rust-lang/rust-analyzer

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

Open

#1942 aperta il 2 ott 2019

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

Metriche repository

Star
 (14.924 star)
Metriche merge PR
 (Merge medio 3g 11h) (113 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
Il bug si trova in crates/ra ide api/src/matching brace.rs. La funzione do check attualmente utilizza start() per la parentesi sinistra, ma dovrebbe utilizzare end() per la parentesi destra per garantire la simmetria. La correzione prevede di scambiare l'ordine dei parametri nel caso di test e di adattare il controllo per la parentesi destra per utilizzare end(). Nessun PR collegato o domande del maintainer; il problema è autonomo.
Tech stack
rust
Dominio
developer experiencetooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rustrust analyzer codebase
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80