icsharpcode/ILSpy

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

Open

#819 geöffnet am 5. Juli 2017

Auf GitHub ansehen
 (1 Kommentar) (2 Reaktionen) (0 zugewiesene Personen)C# (3.645 Forks)batch import
Help WantedUI

Repository-Metriken

Stars
 (25.162 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 15T 22h) (61 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche den Navigationscode in ILSpy, der Methodenreferenzen beim Klicken auf einen Aufruf auflöst. Derzeit scheint er zur Basismethode zu navigieren. Die Lösung besteht darin, eine kontextbewusste Auflösung hinzuzufügen, die die überschreibende Methode in der abgeleiteten Klasse bevorzugt, wenn der Aufruf von einer abgeleiteten Klasse aus erfolgt. Suche nach relevanten Dateien im ILSpy Quellcode, wie z. B. Dateien, die die Symbolnavigation oder Dekompilierung behandeln. Überprüfe vorhandene Kommentare oder verknüpfte Issues für zusätzlichen Kontext.
Tech Stack
csharp
Domain
toolingdeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70