Quick info on `field` exposes generated member name, unlike on accessors

[jnm2]

Version Used: 17.12 Preview 3.0

The metadata name of the backing field is being displayed:

But accessors don't show the metadata name of the generated accessor methods:

Perhaps instead, this could show int C.P.field?

[RikkiGibson]

Probably the fix belongs in SymbolDisplay?

[kyoyama-kazusa]

Just be curious: what will be displayed in quick info for such backing field? int C.field or just int field?

[jnm2]

Also:

[RikkiGibson]

Just be curious: what will be displayed in quick info for such backing field? int C.field or just int field?

I think int C.P.field would be best for symbol display. The property is a logical container for the field. @cston in case you have any additional thoughts here.

[RikkiGibson]

Doing a quick check for what additional behaviors need to be specified, using property accessors (e.g. int C.P.get) as a starting point for the way the field name is displayed.

• https://github.com/dotnet/roslyn/blob/main/src/Compilers/Core/Portable/SymbolDisplay/SymbolDisplayMemberOptions.cs
• https://github.com/dotnet/roslyn/blob/main/src/Compilers/Core/Portable/SymbolDisplay/SymbolDisplayMiscellaneousOptions.cs
• https://github.com/dotnet/roslyn/blob/main/src/Compilers/CSharp/Test/Symbol/SymbolDisplay/SymbolDisplayTests.cs

I am not seeing any options or tests which apply customization of how a property accessor name is displayed. I don't think we should offer any option to, for example, omit the field name component from symbol display. But, let's bring the question up with API review.

I am also thinking that the symbol display behavior should be independent of LangVersion. I do not think that SymbolDisplayVisitor.Members has any existing cases of language version affecting the way symbols are displayed.