dotnet/roslyn

VB boolean literals not highlighted as keywords in SymbolDisplay.ToDisplayParts

Aberta

#30.960 aberto em 5 de nov. de 2018

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

Create a C# project and a VB project referencing it. C# file:

public class CS
{
    public const bool Const = true;
}

VB with completion invoked at |.

Module VB
    Sub M()
        CS.|
    End Sub
End Module

image Expected behavior: True is colored as a keyword Actual behavior: it's not

This is because in the result of calling SymbolDisplay.ToDisplayParts on the field, the part with the true keyword has a kind of SymbolDisplayPartKind.NumericLiteral as opposed to SymbolDisplayPartKind.Keyword (so this bug is in the compiler).

Guia do colaborador

Direção de pesquisa
Examine o método SymbolDisplay.ToDisplayParts no código fonte do compilador Roslyn. O bug é que, para literais booleanos do VB como 'True', o tipo da parte é definido incorretamente como NumericLiteral em vez de Keyword. Concentre se no código que determina o tipo para valores constantes, provavelmente no modelo semântico ou nos utilitários de exibição. O problema tem uma reprodução clara e um comportamento esperado, portanto, uma correção envolveria atualizar a condição que atribui o tipo para constantes booleanas.
Pilha de tecnologia
csharp
Domain
api
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.
3
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.
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.
35