Skip to content
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

Include 'new' in core scenarios for target-typed static name lookup, and editorial updates #9199

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jnm2
Copy link
Contributor

@jnm2 jnm2 commented Mar 6, 2025

No description provided.

@jnm2 jnm2 requested a review from a team as a code owner March 6, 2025 00:51
@jnm2 jnm2 changed the title Target typed static lookup/updates Include 'new' in core scenarios for target-typed static name lookup, and editorial updates Mar 6, 2025

This would continue to be target-typed static member access (since nested types are members of their containing type), which is distinct from target-typed `new` since a definite type is provided to the `new` operator.

TODO: spec the conversion
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are other todos in this file. I'm happy merging as-is, since it doesn't seem like there will be an issue in principle coming up with the conversion, and the proposal is being discussed at a higher level currently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant