microsoft/RoslynClrHeapAllocationAnalyzer

Closure capture of "this" points to the method node, instead pick class decl node

Open

#12 aberto em 3 de mai. de 2015

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C# (63 forks)github user discovery
enhancementhelp wanted

Métricas do repositório

Stars
 (654 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

When a closure captures the this parameter, we lazily just point to the method declaration syntax node, but we really should point to the class/struct declaration node.

If the class is a partial class, then we should point to the class declaration node in the current file.

Guia do colaborador

Direção de pesquisa
Inspecione a lógica de captura de closure no analisador para encontrar onde 'this' é capturado. Em vez de apontar para o nó do método, modifique para apontar para o nó de declaração da classe ou struct. Para classes parciais, use a declaração de classe no arquivo atual.
Pilha de tecnologia
csharp
Domain
developer experiencetooling
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Roslyn
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60