microsoft/RoslynClrHeapAllocationAnalyzer

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

Open

#12 ouverte le 3 mai 2015

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C# (63 forks)github user discovery
enhancementhelp wanted

Métriques du dépôt

Stars
 (654 stars)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Inspectez la logique de capture de fermeture dans l'analyseur pour trouver où 'this' est capturé. Au lieu de pointer vers le nœud de méthode, modifiez le pour pointer vers le nœud de déclaration de classe ou de structure. Pour les classes partielles, utilisez la déclaration de classe dans le fichier actuel.
Stack technique
csharp
Domaine
developer experiencetooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Roslyn
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60