dotnet/roslyn

`IsPreprocessorKeyword` documentation comment doesn't match implementation

Aberta

#54.604 aberto em 4 de jul. de 2021

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)C# (4.257 forks)batch import
Area-CompilersBughelp 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

https://github.com/dotnet/roslyn/blob/43afd6dfd4b79380ff48078dfb264c60c518948c/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Services/SyntaxFacts/ISyntaxFacts.cs#L81-L85

The documentation says: "The set of identifiers that have special meaning directly after the # token". However, the implementation delegates to the following:

https://github.com/dotnet/roslyn/blob/43afd6dfd4b79380ff48078dfb264c60c518948c/src/Compilers/CSharp/Portable/Syntax/SyntaxKindFacts.cs#L68-L101

which returns true for things that, as far as I know, doesn't come "directly after" the # (for example TrueKeyword).

Guia do colaborador

Direção de pesquisa
Compare o comentário da documentação em ISyntaxFacts.cs com a implementação em SyntaxKindFacts.cs para identificar discrepâncias. Atualize a documentação ou a implementação para alinhá las.
Pilha de tecnologia
csharp
Domain
compilers
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.
1
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
None
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90