dotnet/roslyn

Simplify logic for deconstruction variables in ExpressionVariableFinder

Open

#15.064 aperta il 7 nov 2016

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-CompilersConcept-Design DebtFeature - Tupleshelp wanted

Metriche repository

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

Descrizione

Per discussion in PR https://github.com/dotnet/roslyn/pull/14871

  • Maybe we don't need MakeOutVariable and MakeDeconstructionVariable, instead one may suffice, by looking up the syntax tree.
  • Maybe we don't need CollectVariablesFromDeconstruction in ExpressionVariableFinder, instead we just need a VisitVariableDesignation which looks up the syntax tree to determine if it is an out or a deconstruction variable.
  • Then maybe we can re-use ExpressionVariableFinder instead of ForEachLoopBinder.CollectLocalsFromDeconstruction.

Guida contributor

Direzione di ricerca
Leggi la discussione della PR collegata, quindi rifattorizza ExpressionVariableFinder unendo MakeOutVariable e MakeDeconstructionVariable in un unico metodo che utilizza le ricerche nell'albero sintattico e considera di riutilizzare ExpressionVariableFinder per ForEachLoopBinder.CollectLocalsFromDeconstruction.
Tech stack
csharp
Dominio
compilers
Tipo issue
Refactoring
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#RoslynGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55