Skip to content

Fix uninlined_format_args in stable_mir - #131733

Merged
bors merged 1 commit into
rust-lang:masterfrom
practicalrs:fix_uninlined_format_args
Oct 15, 2024
Merged

Fix uninlined_format_args in stable_mir#131733
bors merged 1 commit into
rust-lang:masterfrom
practicalrs:fix_uninlined_format_args

Conversation

@practicalrs

Copy link
Copy Markdown
Contributor

Hi,

This PR fixes some clippy warnings

warning: variables can be used directly in the `format!` string
--> compiler/stable_mir/src/mir/pretty.rs:362:13
|
362 | write!(writer, "{kind}{:?}", place)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
= note: requested on the command line with `-W clippy::uninlined-format-args`
help: change this to
|
362 - write!(writer, "{kind}{:?}", place)
362 + write!(writer, "{kind}{place:?}")
|

Best regards,
Michal

@rustbot

Copy link
Copy Markdown
Collaborator

Failed to set assignee to ouz-a: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@rustbot

Copy link
Copy Markdown
Collaborator

r? @ouz-a

rustbot has assigned @ouz-a.
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

@rustbotrustbot 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 Oct 15, 2024
@rustbot

Copy link
Copy Markdown
Collaborator

This PR changes Stable MIR

cc @oli-obk, @celinval, @ouz-a

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

@jieyouxu

Copy link
Copy Markdown
Member

r? jieyouxu
@bors r+ rollup

@bors

bors commented Oct 15, 2024

Copy link
Copy Markdown
Collaborator

📌 Commit d3d5905 has been approved by jieyouxu

It is now in the queue for this repository.

@borsbors 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 Oct 15, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 15, 2024
…mpiler-errors
Rollup of 7 pull requests
Successful merges:
- rust-lang#129794 (uefi: Implement getcwd and chdir)
- rust-lang#130568 (Make some float methods unstable `const fn`)
- rust-lang#131521 (rename RcBox to RcInner for consistency)
- rust-lang#131701 (Don't report `on_unimplemented` message for negative traits)
- rust-lang#131705 (Fix most ui tests on emscripten target)
- rust-lang#131733 (Fix uninlined_format_args in stable_mir)
- rust-lang#131734 (Update `arm64e-apple-tvos` maintainer)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 53d1a66 into rust-lang:masterOct 15, 2024
@rustbotrustbot added this to the 1.84.0 milestone Oct 15, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Oct 15, 2024
Rollup merge of rust-lang#131733 - practicalrs:fix_uninlined_format_args, r=jieyouxu
Fix uninlined_format_args in stable_mir
Hi,
This PR fixes some clippy warnings
```
warning: variables can be used directly in the `format!` string
--> compiler/stable_mir/src/mir/pretty.rs:362:13
|
362 | write!(writer, "{kind}{:?}", place)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
= note: requested on the command line with `-W clippy::uninlined-format-args`
help: change this to
|
362 - write!(writer, "{kind}{:?}", place)
362 + write!(writer, "{kind}{place:?}")
|
```
Best regards,
Michal
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant 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.

4 participants

@practicalrs@rustbot@jieyouxu@bors