dotnet/roslyn

Simplify logic for deconstruction variables in ExpressionVariableFinder

Ouverte

#15 064 ouverte le 7 nov. 2016

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-CompilersConcept-Design DebtFeature - Tupleshelp wanted

Métriques du dépôt

Stars
 (20 414 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Lisez la discussion de la PR liée, puis refactorisez ExpressionVariableFinder en fusionnant MakeOutVariable et MakeDeconstructionVariable en une seule méthode utilisant des recherches dans l'arbre syntaxique, et envisagez de réutiliser ExpressionVariableFinder pour ForEachLoopBinder.CollectLocalsFromDeconstruction.
Stack technique
csharp
Domaine
compilers
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#RoslynGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55