dotnet/roslyn

VB boolean literals not highlighted as keywords in SymbolDisplay.ToDisplayParts

Aperta

#30.960 aperta il 5 nov 2018

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
4 - In ReviewArea-CompilersBughelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il metodo SymbolDisplay.ToDisplayParts nel codice sorgente del compilatore Roslyn. Il bug è che per i letterali booleani VB come 'True', il tipo di parte è impostato erroneamente su NumericLiteral invece che su Keyword. Concentrati sul codice che determina il tipo per i valori costanti, probabilmente nel modello semantico o negli strumenti di visualizzazione. Il problema ha una riproduzione chiara e un comportamento atteso, quindi una correzione comporterebbe l'aggiornamento della condizione che assegna il tipo per le costanti booleane.
Tech stack
csharp
Dominio
api
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Nessuno
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35