Update field references in property accessors#108413
Conversation
ghost
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 |
| 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 <stoub@microsoft.com>
* Update field references in property accessors * Update field references * Use @ * Rename field --------- Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update field references in property accessors * Update field references * Use @ * Rename field --------- Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update field references in property accessors * Update field references in property accessors (#108225) * Update field references in property accessors (#108413) * Update field references in property accessors * Update field references * Use @ * Rename field --------- Co-authored-by: Stephen Toub <stoub@microsoft.com> --------- Co-authored-by: Charles Stoner <10732005+cston@users.noreply.github.com> Co-authored-by: Stephen Toub <stoub@microsoft.com>
4 participants
Rename or qualify
fieldreferences in property accessors to avoid conflict withfieldkeyword in C# compiler preview.Continuation of #108219, #108225.