eigenlsp: definition/symbol range for a function covers 'define', not the name
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 72/100
Direzione di ricerca
Start in src/eigenlsp.c around lines 1209, 1261, 1339, and 1725, tracing how a symbol's col is used for definition, documentSymbol, workspaceSymbol, reference, and rename ranges. Add or update coverage in tests/test_lsp.py for definition, documentSymbol, and references on a define, then run that test and confirm ranges use the helper name rather than define.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Found by the builder on #1368.
textDocument/definition on a call to helper returns {start:{0,0}, end:{0,6}} for define helper(a) as:. That range covers the define keyword. s->col is the define STATEMENT's column, and the end is computed as col + strlen(name). This happens with space indentation too. The same s->col feeds documentSymbol, workspaceSymbol and the symbol-sourced reference and rename ranges (src/eigenlsp.c around lines 1209, 1261, 1339, 1725). Any edit built from those ranges would rewrite define.
Done when
- definition of
helperreturns{0,7}..{0,13}(the name's bytes). A test in tests/test_lsp.py covers definition, documentSymbol and references on adefine. - Every range built from a symbol's
colis audited, and each one either uses the name's column or states why the statement column is right.
- Lingua principale
- C
- Stelle
- 3
- Fork
- 7
- Merge medio
- 8h 44m
- PR unite (30g)
- 252
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 InauguralSystems/EigenScript
-
UB: (int) cast of a user double outside int range in dispatch / vm_index_is_int / flat_index_atApertaarea:runtime-vm
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
InauguralSystems/EigenScript#1748 ·
I maintainer di solito rispondono entro 1 giorno
-
area:stdlib found-by:critic kind:gate-defect
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
InauguralSystems/EigenScript#1693 ·
I maintainer di solito rispondono entro 1 giorno
-
area:gfx bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
InauguralSystems/EigenScript#1684 ·
I maintainer di solito rispondono entro 1 giorno
-
area:stdlib bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
InauguralSystems/EigenScript#1679 ·
I maintainer di solito rispondono entro 1 giorno
-
area:consumer
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
InauguralSystems/EigenScript#1658 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di InauguralSystems/EigenScript
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 2/5 Mezza giornata Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
foomatic-rip: dstrtrim_right() underflows mid-loop on an all-whitespace stringForse già presa @ManasBagul23 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
OpenPrinting/cups-filters#759 ·
I maintainer di solito rispondono entro 11 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
EchoTools/nevr-runtime#547 ·
I maintainer di solito rispondono entro 1 giorno
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100