dotnet/roslyn

CSharpDecompiledSourceService doesn't support nested types

Offen

#46.076 geöffnet am 17.07.2020

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-IDEBughelp wanted

Repository-Metriken

Stars
 (20.414 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (256 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Prüfe die Methode in der verlinkten Zeile (L160 L175) in CSharpDecompiledSourceService.cs und modifiziere sie, um die Namen der übergeordneten Typen mit einem '+' Trennzeichen für verschachtelte Typen anzuhängen.
Tech Stack
csharp
Domain
developer experiencetooling
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.
Frisch
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