dotnet/roslyn

CSharpDecompiledSourceService doesn't support nested types

Aberta

#46.076 aberto em 17 de jul. de 2020

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)C# (4.257 forks)batch import
Area-IDEBughelp wanted

Métricas do repositório

Stars
 (20.414 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (256 fundiu PRs em 30d)

Description

Version Used: 16.6.3

Steps to Reproduce: Reference a nested type from a .dll and try to decompile it

InvalidOperationException: Could not find type definition A.NestedClass in type system.

  | CSharpDecompiler.DecompileType Line 817
-- | --
  | CSharpDecompiler.DecompileTypeAsString Line 840
  | CSharpDecompiledSourceService.PerformDecompilation Line 115
  | CSharpDecompiledSourceService.<AddSourceToAsync>d__3.MoveNext Line 74

This method should append the names of parent types (and probably use the + separator?) https://github.com/dotnet/roslyn/blob/8bedf286254d901c48f2cfd562cf3425de2b2e10/src/EditorFeatures/CSharp/DecompiledSource/CSharpDecompiledSourceService.cs#L160-L175

Guia do colaborador

Direção de pesquisa
Inspecione o método na linha vinculada (L160 L175) em CSharpDecompiledSourceService.cs e modifique o para anexar nomes de tipos pai usando um separador '+' para tipos aninhados.
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#
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70