icsharpcode/ILSpy

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

Open

#819 aberto em 5 de jul. de 2017

Ver no GitHub
 (1 comment) (2 reactions) (0 assignees)C# (3.645 forks)batch import
Help WantedUI

Métricas do repositório

Stars
 (25.162 stars)
Métricas de merge de PR
 (Mesclagem média 15d 22h) (61 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue o código de navegação no ILSpy que resolve referências de método ao clicar em uma chamada. Atualmente, parece navegar para o método base. A correção envolve adicionar uma resolução sensível ao contexto que prefere o método de substituição na classe derivada quando a chamada é feita a partir de uma classe derivada. Procure por arquivos relevantes no código fonte do ILSpy, como aqueles que lidam com navegação de símbolos ou descompilação. Verifique quaisquer comentários existentes ou issues vinculadas para contexto adicional.
Pilha de tecnologia
csharp
Domain
toolingdeveloper experience
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.
70