-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update IsImplicitlyDeclared doc #48199
Conversation
@@ -127,7 +127,7 @@ public interface ISymbol : IEquatable<ISymbol?> | |||
|
|||
/// <summary> | |||
/// Returns true if this symbol was automatically created by the compiler, and does not have | |||
/// an explicit corresponding source code declaration. | |||
/// an explicit corresponding source code declaration. This doesn't include the generated method and class for top-level statements. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the key point is that these are not generated methods. A generated method and class is something like PrivateImplementationDetails
. The top-level <Program>$
and <Main>$
are created because of a specific user action: writing a top-level main. That means create these things. It's not implicitly created.
If we need to add a note, I'd prefer it go down in the remarks below.
Co-authored-by: Fred Silberberg <fred@silberberg.xyz>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (commit 3). @dotnet/roslyn-compiler for a second review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (iteration 3)
@Youssef1313 Thanks for the contribution. |
Closes #48193