rust-lang/rust-analyzer

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

Open

#1 942 ouverte le 2 oct. 2019

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)Rust (1 726 forks)batch import
E-has-instructionsE-mediumS-actionablegood first issue

Métriques du dépôt

Stars
 (14 924 stars)
Métriques de merge PR
 (Merge moyen 3j 11h) (113 PRs mergées en 30 j)

Description

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()

Guide contributeur

Direction de recherche
Le bug se trouve dans crates/ra ide api/src/matching brace.rs. La fonction do check utilise actuellement start() pour l'accolade gauche, mais devrait utiliser end() pour l'accolade droite pour assurer la symétrie. La correction implique d'échanger l'ordre des paramètres dans le cas de test et d'ajuster la vérification pour l'accolade droite pour utiliser end(). Aucune PR liée ou question de mainteneur ; le problème est autonome.
Stack technique
rust
Domaine
developer experiencetooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rustrust analyzer codebase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80