dotnet/roslyn

Simplify logic for deconstruction variables in ExpressionVariableFinder

Open

#15,064 创建于 2016年11月7日

在 GitHub 查看
 (0 评论) (0 反应) (0 负责人)C# (4,257 fork)batch import
Area-CompilersConcept-Design DebtFeature - Tupleshelp wanted

仓库指标

Star
 (20,414 star)
PR 合并指标
 (平均合并 6天 17小时) (30 天内合并 256 个 PR)

描述

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.

贡献者指南

研究方向
阅读链接的PR讨论，然后通过使用语法树查找将MakeOutVariable和MakeDeconstructionVariable合并为单个方法来重构ExpressionVariableFinder，并考虑重用ExpressionVariableFinder来处理ForEachLoopBinder.CollectLocalsFromDeconstruction。
技术栈
csharp
领域
compilers
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C#RoslynGit
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
55