Skip to content

Fix an EN grammar error & add an item to place expr context list#2189

Merged
ehuss merged 3 commits intorust-lang:masterfrom
bitcp:add-to-place-expr-context
Mar 1, 2026
Merged

Fix an EN grammar error & add an item to place expr context list#2189
ehuss merged 3 commits intorust-lang:masterfrom
bitcp:add-to-place-expr-context

Conversation

@bitcp
Copy link
Contributor

@bitcp bitcp commented Feb 24, 2026

The reference calls tuple components fields at some places, but the syntax distinguishes between FieldExprssion, IndexExpression and TupleIndexingExpression.

Not sure whether this adding is a redundancy.

@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Feb 24, 2026
Copy link
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

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

Thanks! Yea, it was a little ambiguous here, and seems reasonable to be more explicit (even if internally the compiler combines the different field kinds).

I added a small commit to add links to the expressions.

View changes since this review

@ehuss ehuss enabled auto-merge March 1, 2026 22:36
@ehuss ehuss added this pull request to the merge queue Mar 1, 2026
Merged via the queue into rust-lang:master with commit 5092757 Mar 1, 2026
6 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Mar 1, 2026
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.

3 participants