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

Records spec missing scope changes #45087

Closed
gafter opened this issue Jun 11, 2020 · 3 comments
Closed

Records spec missing scope changes #45087

gafter opened this issue Jun 11, 2020 · 3 comments

Comments

@gafter
Copy link
Member

gafter commented Jun 11, 2020

Although dotnet/csharplang#3548 expresses informally the intent to make some scope changes to the language in support of records, the records specification at https://github.com/dotnet/csharplang/blob/master/proposals/records.md still contains no modification to the name lookup rule in https://github.com/dotnet/csharplang/blob/master/spec/expressions.md#simple-names that would be needed to describe the intended language impact.

See also #44687. Once the spec is written the compiler would need to be implemented (if there is any mismatch) and tested against the specification.

/cc @AlekseyTs @agocke

@AlekseyTs
Copy link
Contributor

This issue is too general to be actionable.

@AlekseyTs AlekseyTs assigned jcouv and unassigned AlekseyTs Oct 30, 2020
@jcouv
Copy link
Member

jcouv commented Oct 30, 2020

@AlekseyTs The ask is to update the records speclet with the changes from this PR: #44906

@jcouv
Copy link
Member

jcouv commented Oct 30, 2020

From offline discussion, the speclet was already updated to include the information.

@jcouv jcouv closed this as completed Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants