dotnet/roslyn

Use Site Diagnostics are missed in tuple symbols in VB

Open

#61.112 aperta il 3 mag 2022

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-CompilersBughelp wanted

Metriche repository

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

Descrizione

https://github.com/dotnet/roslyn/blob/main/src/Compilers/VisualBasic/Portable/Symbols/Tuples/TupleEventSymbol.vb#L87 https://github.com/dotnet/roslyn/blob/main/src/Compilers/VisualBasic/Portable/Symbols/Tuples/TupleFieldSymbol.vb#L102 https://github.com/dotnet/roslyn/blob/main/src/Compilers/VisualBasic/Portable/Symbols/Tuples/TupleMethodSymbol.vb#L133 https://github.com/dotnet/roslyn/blob/main/src/Compilers/VisualBasic/Portable/Symbols/Tuples/TuplePropertySymbol.vb#L115

All of these call MergeUseSiteInfo and discard the result. In the VB compiler, that method does not take use site info by ref, it returns a new copy, so all of these locations will miss any use site info that should be reported.

Guida contributor

Direzione di ricerca
La correzione comporta l'aggiornamento di quattro file: TupleEventSymbol.vb, TupleFieldSymbol.vb, TupleMethodSymbol.vb e TuplePropertySymbol.vb. In ciascuno, la chiamata a MergeUseSiteInfo dovrebbe catturare il suo valore di ritorno invece di scartarlo. Il valore di ritorno dovrebbe essere unito in modo appropriato. Dopo la correzione, verificare che le diagnostiche use site vengano ora segnalate correttamente per i simboli di tupla in VB. Controllare anche se ci sono altri pattern simili nel compilatore VB che potrebbero avere lo stesso problema.
Tech stack
csharp
Dominio
developer experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Visual Basic compiler internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40