rust-lang/rust-analyzer

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

Open

#1.942 aberto em 2 de out. de 2019

Ver no GitHub
 (8 comments) (0 reactions) (0 assignees)Rust (1.726 forks)batch import
E-has-instructionsE-mediumS-actionablegood first issue

Métricas do repositório

Stars
 (14.924 stars)
Métricas de merge de PR
 (Mesclagem média 3d 11h) (113 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
O bug está em crates/ra ide api/src/matching brace.rs. A função do check atualmente usa start() para a chave esquerda, mas deve usar end() para a chave direita para garantir simetria. A correção envolve trocar a ordem dos parâmetros no caso de teste e ajustar a verificação para a chave direita para usar end(). Nenhum PR vinculado ou perguntas do mantenedor; o problema é autossuficiente.
Pilha de tecnologia
rust
Domain
developer experiencetooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rustrust analyzer codebase
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80