Proposal: remove requirement for exact instruction count for disassemble response
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 32/100
Hướng nghiên cứu
Bắt đầu với định nghĩa yêu cầu disassemble và đặc tả instructionCount được trích dẫn trong issue, sau đó xem xét cách chế độ xem disassembly của VS Code diễn giải các địa chỉ được trả về. Xác định phản hồi được mong đợi khi có ít lệnh hơn khả dụng và làm rõ các quy tắc về địa chỉ; công việc được coi là hoàn tất khi đặc tả giao thức không còn mơ hồ đối với cả client và debug adapter.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The spec requires debug adapters to return exactly the number of instructions requested.
/**
* Number of instructions to disassemble starting at the specified location
* and offset.
* An adapter must return exactly this number of instructions - any
* unavailable instructions should be replaced with an implementation-defined
* 'invalid instruction' value.
*/
instructionCount: number;
This presents an issue when there are not enough instructions available. The DA is then required to return an "implementation-defined" invalid instruction. It is unclear how those instructions will be used, and why they are even necessary to return. Scrolling through invalid instructions does not provide any additional information to users.
Also what the addresses should be for the invalid instructions is not defined in the spec so it is unclear how the editor can interpret them. For example, the VS Code implementation of a disassembly view will attempt to do a binary search using the addresses, which requires that the invalid instructions obey certain rules to implement this. This may also result in having to return negative addresses which also does not really make sense for an instruction.
Removing the requirement to return exactly the number of requested instructions would better define the expected disassembled instructions for both the client and server.
- Ngôn ngữ chính
- HTML
- Star
- 1.8k
- Fork
- 173
- Merge trung bình
- 7 ngày 7 giờ
- Pull request đã merge (30 ngày)
- 2
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của microsoft/debug-adapter-protocol
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
microsoft/debug-adapter-protocol#633 · 1 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
microsoft/debug-adapter-protocol#599 · 8 reaction ·
-
under-discussion
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 48/100
microsoft/debug-adapter-protocol#596 · 9 bình luận ·
Tất cả issue của microsoft/debug-adapter-protocol
Issue tương tự
-
schema_dict_to_pydantic_model TypeError on JSON Schema type arrays → extract falls back to free-text Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
browser-use/browser-use#5904 ·
-
type: enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
ynput/ayon-python-api#363 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
bug severity:low
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100