dotnet/roslyn

Change signature adds unnecessary argument names when removing a parameter

Aperta

#46.455 aperta il 30 lug 2020

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-IDEBughelp wanted

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Riproduci il problema rimuovendo il parametro e osserva le modifiche generate. Analizza la logica di refactoring di ChangeSignature per capire perché aggiunge nomi di argomenti non necessari nei siti di chiamata per il parametro rimosso.
Tech stack
csharp
Dominio
compilers
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60