dotnet/roslyn

Change signature adds unnecessary argument names when removing a parameter

开放

#46,455 创建于 2020年7月30日

在 GitHub 查看贡献者指南
 (0 条评论) (0 个反应) (0 位负责人)C# (4,257 个派生)batch import
Area-IDEBughelp wanted

仓库指标

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

描述

Version Used: Version 16.8.0 Preview 2.0 [30327.8.master]

Steps to Reproduce:

  1. Checkout Roslyn at a7976180954969c9904ee6d449adad28283ddc34

  2. Open src\Features\Core\Portable\SolutionCrawler\WorkCoordinator.WorkItem.cs and find WorkItem constructor on line 95

  3. Remove bool isLowPriority parameter.

  4. Review changes.

src\Features\Core\Portable\SolutionCrawler\WorkCoordinator.SemanticChangeProcessor.cs:

image

src\Features\Core\Portable\SolutionCrawler\WorkCoordinator.cs:

image

贡献者指南

研究方向
通过删除参数重现问题，观察生成的更改。调查ChangeSignature重构逻辑，了解为什么它在调用站点为已删除的参数添加不必要的参数名称。
技术栈
csharp
领域
compilers
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
较久未更新
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
Git
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60