dotnet/roslyn

Extract method does not respect whitespace when extracting a return statement

Aberta

#40.023 aberto em 26 de nov. de 2019

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)C# (4.257 forks)batch import
Area-IDEBugFeature - Extract Methodhelp wanted

Métricas do repositório

Stars
 (20.414 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (256 fundiu PRs em 30d)

Description

Version Used: 16.4.0 P6

Steps to Reproduce:

  1. Highlight a single return statement
  2. Ctrl+. and select extract method code action

Expected Behavior: Newline above return statement preserved

Actual Behavior: Newline not preserved. Seems to be an edge case since extracting regular statements seems to correctly preserve the newline if present.

newlinenotrespected

Guia do colaborador

Direção de pesquisa
Encontre a ação de código 'Extract Method' na base de código do Roslyn e rastreie a lógica para lidar com espaços em branco ao redor de instruções return. Procure testes existentes para extração de método e adicione um teste com falha que reproduza o problema. Em seguida, corrija o código para preservar os espaços em branco.
Pilha de tecnologia
csharp
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60