Skip to content

Extend format arg help for simple tuple index access expression#122556

Merged
bors merged 1 commit into
rust-lang:masterfrom
jieyouxu:non-identifier-format-arg
Mar 19, 2024
Merged

Extend format arg help for simple tuple index access expression#122556
bors merged 1 commit into
rust-lang:masterfrom
jieyouxu:non-identifier-format-arg

Conversation

@jieyouxu

@jieyouxu jieyouxu commented Mar 15, 2024

Copy link
Copy Markdown
Member

The help is only applicable for simple field access a.b and (with this PR) simple tuple index access expressions a.0.

Closes #122535.

@rustbot

rustbot commented Mar 15, 2024

Copy link
Copy Markdown
Collaborator

r? @petrochenkov

rustbot has assigned @petrochenkov.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 15, 2024
@jieyouxu jieyouxu force-pushed the non-identifier-format-arg branch from 2648798 to 04b2f79 Compare March 16, 2024 00:43
Comment thread compiler/rustc_parse_format/src/lib.rs Outdated
@jieyouxu jieyouxu force-pushed the non-identifier-format-arg branch from 04b2f79 to 52a1125 Compare March 16, 2024 22:33
@petrochenkov

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Mar 19, 2024

Copy link
Copy Markdown
Collaborator

📌 Commit 52a1125 has been approved by petrochenkov

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 19, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 19, 2024
…iaskrgr

Rollup of 10 pull requests

Successful merges:

 - rust-lang#122435 (Don't trigger `unused_qualifications` on global paths)
 - rust-lang#122556 (Extend format arg help for simple tuple index access expression)
 - rust-lang#122634 (compiletest: Add support for `//@ aux-bin: foo.rs`)
 - rust-lang#122677 (Fix incorrect mutable suggestion information for binding in ref pattern.)
 - rust-lang#122691 (Fix ICE: `global_asm!()` Don't Panic When Unable to Evaluate Constant)
 - rust-lang#122695 (Change only_local to a enum type.)
 - rust-lang#122717 (Ensure stack before parsing dot-or-call)
 - rust-lang#122719 (Ensure nested statics have a HIR node to prevent various queries from ICEing)
 - rust-lang#122720 ([doc]:fix error code example)
 - rust-lang#122724 (add test for casting pointer to union with unsized tail)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 45e005d into rust-lang:master Mar 19, 2024
@rustbot rustbot added this to the 1.79.0 milestone Mar 19, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Mar 19, 2024
Rollup merge of rust-lang#122556 - jieyouxu:non-identifier-format-arg, r=petrochenkov

Extend format arg help for simple tuple index access expression

The help is only applicable for simple field access `a.b` and (with this PR) simple tuple index access expressions `a.0`.

Closes rust-lang#122535.
@jieyouxu jieyouxu deleted the non-identifier-format-arg branch March 19, 2024 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unclear error message when using . in format string

5 participants