Add a `memoryType: assembly` to `Variable`
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- typescript
Direzione di ricerca
Inizia individuando le definizioni di MemoryType e Variable nel protocollo, quindi esamina le definizioni correlate di EvaluateResponse, Output e SetExpressionResponse indicate nella proposta. Traccia come memoryReference viene documentato e rappresentato attraverso questi punti di ingresso; il lavoro è completato quando il tipo di memoria dell’assembly è specificato in modo coerente per tutti i tipi di risposta proposti e il comportamento esistente di memoryReference rimane chiaro.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Use case
For pointers, the memoryReference attached to a pointer points to the corresponding memory.
Currently, this memory is shown in VS-Code in the hex-editor.
However, for function pointers, it would be more ergonomic to show this memory area in the disassembly view.
Proposal
type MemoryType = 'raw' | 'assembly';
interface Variable {
// ...
/**
* A memory reference associated with this variable.
* For pointer type variables, this is generally a reference to the memory
* address contained in the pointer.
* For executable data, this reference may later be used in a `disassemble`
* request.
* This attribute may be returned by a debug adapter if corresponding
* capability `supportsMemoryReferences` is true.
*/
memoryReference?: string;
/**
* Determines the type of memory which this memory reference is
* referring to.
*/
memoryType?: MemoryType;
}
// Similar for `EvaluateResponse`, `Output`, `SetExpressionResponse`
The UI could then open either the disassembly or the hex-editor for the given memoryReference, as appropriate. (Or even offer a choice to the user whether to use the disassembly or the hex-editor view).
This might also be useful in combination with microsoft/vscode#197287, where extensions might introduce own, customMemoryType. E.g., I could imagine other extensions to contribute a PNG-viewer which allows viewing PNGs directly from within the program's memory. (But I don't have any particular use case in that direction. My main motivation here is to get better default-behavior for function pointers)
- Lingua principale
- HTML
- Stelle
- 1.8k
- Fork
- 173
- Merge medio
- 7g 7h
- PR unite (30g)
- 2
Preparare l'ambiente
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di microsoft/debug-adapter-protocol
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
microsoft/debug-adapter-protocol#633 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
microsoft/debug-adapter-protocol#599 · 8 reazioni ·
-
under-discussion
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
microsoft/debug-adapter-protocol#596 · 9 commenti ·
Tutte le issue di microsoft/debug-adapter-protocol
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
SciML/Evolutionary.jl#182 ·
I maintainer di solito rispondono entro 1 giorno
-
Missing GLM modelsAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
simonw/llm-mistral#43 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
rossoctl/context-guru#346 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
prime-radiant-inc/evener#2883 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
GoogleChromeLabs/project-sesame#216 ·
I maintainer di solito rispondono entro 12 giorni