Update field references in property accessors #108413
Conversation
dotnet-issue-labeler
bot
added
the
area-Tools-ILLink
dotnet-policy-service
bot
added
the
linkable-framework
|
Tagging subscribers to this area: @agocke, @sbomer, @vitek-karas |
|
A 3rd PR that should cover the remaining cases of (See test PR #108468 which builds these changes with the latest dotnet/roslyn toolset. That PR has test failures with [wasm] DebuggerTests but those seem to be a known issue #108072.) The 3 PRs for
It looks like only the first one has been ported to |
| @@ -144,9 +144,9 @@ private class SimpleType | |||
| [Kept] | |||
| public int memberKept { | |||
| [Kept] | |||
| get { return field; } | |||
| get { return @field; } | |||
There was a problem hiding this comment.
I'd rename this to "fieldKept" to match the rest of the test - and it will resolve the problem as well.
Yes, once we believe we're done with all the changes in main, we should cherry-pick them into that branch and bring it through tactics. |
|
Merging to unblock ci |
* Update field references in property accessors * Update field references * Use @ * Rename field --------- Co-authored-by: Stephen Toub <[email protected]>
* Update field references in property accessors * Update field references * Use @ * Rename field --------- Co-authored-by: Stephen Toub <[email protected]>
Rename or qualify
fieldreferences in property accessors to avoid conflict withfieldkeyword in C# compiler preview.Continuation of #108219, #108225.