icsharpcode/ILSpy

When navigating to an overridden virtual method from the overriding derived class, navigate to the overridden method

Open

#819 ouverte le 5 juil. 2017

Voir sur GitHub
 (1 commentaire) (2 réactions) (0 assignés)C# (3 645 forks)batch import
Help WantedUI

Métriques du dépôt

Stars
 (25 162 stars)
Métriques de merge PR
 (Merge moyen 15j 22h) (61 PRs mergées en 30 j)

Description

Consider the following class structure:

class Base 
{
    public virtual int Foo() { return 42; }
}

sealed class Derived : Base
{
    public override int Foo() { return 123; }
    public int Bar() { return this.Foo(); }
}

Suppose I'm looking at Derived::Bar, and I click Foo(). I am currently taken to Base::Foo which is misleading because in actuality Derived::Foo will be called.

I made Derived sealed just to drive my point further, obviously even if it's non-sealed Derived::Foo would be a better guess.

Guide contributeur

Direction de recherche
Étudiez le code de navigation dans ILSpy qui résout les références de méthode lors d'un clic sur un appel. Actuellement, il semble naviguer vers la méthode de base. La correction implique d'ajouter une résolution contextuelle qui préfère la méthode de substitution dans la classe dérivée lorsque l'appel est effectué depuis une classe dérivée. Recherchez les fichiers pertinents dans le code source d'ILSpy, comme ceux gérant la navigation des symboles ou la décompilation. Vérifiez les commentaires existants ou les problèmes liés pour obtenir un contexte supplémentaire.
Stack technique
csharp
Domaine
toolingdeveloper experience
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70