rust-lang/rust-analyzer

matching_brace function not correct when jumping from open brace to close brace

Aperta

#1942 aperta il 2 ott 2019

Vedi su GitHubGuida contributor
 (8 commenti) (0 reazioni) (0 assegnatari)Rust (1726 fork)batch import
E-has-instructionsE-mediumS-actionablegood first issue

Metriche repository

Star
 (14.924 stelle)
Metriche merge PR
 (Merge medio 11g 5h) (96 PR mergiate in 30 g)

Descrizione

https://github.com/rust-analyzer/rust-analyzer/blob/31f22d85491d9e7eaadf5fd4f9754c83fc0f3ea6/crates/ra_ide_api/src/matching_brace.rs#L41

swap the order of do_check parameters the makes the test case failed

do_check("struct Foo <|>{ a: i32, }", "struct Foo { a: i32, }<|>");

this matching behavior should be symmetrical

https://github.com/rust-analyzer/rust-analyzer/blob/31f22d85491d9e7eaadf5fd4f9754c83fc0f3ea6/crates/ra_ide_api/src/matching_brace.rs#L18-L19

if the matching node is a left brace, we use start(), if the matching node is a right brace, we should use end()

Guida contributor

Direzione di ricerca
Ispeziona il file matching brace.rs, in particolare la funzione do check e la logica per la gestione delle parentesi graffe destre. Assicura la simmetria usando end() per le parentesi graffe destre invece di start().
Tech stack
rust
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70