dotnet/roslyn

Fix VB Parsing of Format Specifiers

Aberta

#58.043 aberto em 30 de nov. de 2021

Ver no GitHubGuia do colaborador
 (16 comentários) (0 reação) (0 responsável)C# (4.257 forks)batch import
Area-CompilersBugLanguage-VBhelp wanted

Métricas do repositório

Stars
 (20.414 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (256 fundiu PRs em 30d)

Description

VB has the same parsing "bug" as detailed in https://github.com/dotnet/roslyn/issues/57750, but unlike C# it does not support interpolated string handlers, so there are no runtime impacts to the parsing issue. We should update the VB parser to be in sync with the C# parser and the runtime here, but it's not high priority so the team is not going to focus on it. We'd accept a community PR to implement this fix.

Guia do colaborador

Direção de pesquisa
Examine a correção em C# na issue #57750 para entender a alteração de análise. Localize a lógica correspondente do analisador VB na base de código do Roslyn, provavelmente em src/Compilers/VisualBasic/Portable/Parsing. Implemente a mesma atualização para VB, garantindo que não haja impacto em tempo de execução, pois VB não possui manipuladores de string interpolada. Teste a alteração com os testes de análise VB existentes.
Pilha de tecnologia
csharp
Domain
backend
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75