Skip to content

Better suggestion for panic!(format!(..)) #82110

New issue
New issue
Closed
#82113
Closed
Better suggestion for panic!(format!(..))#82110
#82113
Assignees
m-ou-se
Labels
A-suggestion-diagnosticsArea: Suggestions generated by the compiler applied by `cargo fix`D-confusingDiagnostics: Confusing error or lint that should be reworked.
@m-ou-se

Description

@m-ou-se
m-ou-se
opened on Feb 14, 2021
Member

The non_fmt_panic lint should have a special case for panic!(format!(..)) to suggest panic!(..) by removing format( and ).

Activity

m-ou-se
added
A-suggestion-diagnosticsArea: Suggestions generated by the compiler applied by `cargo fix`
D-confusingDiagnostics: Confusing error or lint that should be reworked.
on Feb 14, 2021
m-ou-se
self-assigned this
on Feb 14, 2021
197g
added a commit that references this issue on Feb 14, 2021

Fix forward compatibility of panicking

fa9f629
197g
mentioned this on Feb 14, 2021
m-ou-se
mentioned this on Feb 14, 2021
Dylan-DPC-zz
added a commit that references this issue on Feb 23, 2021

Rollup merge of rust-lang#82113 - m-ou-se:panic-format-lint, r=estebank

547b3ad
bors
closed this as completedin 547b3adon Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

A-suggestion-diagnosticsArea: Suggestions generated by the compiler applied by `cargo fix`D-confusingDiagnostics: Confusing error or lint that should be reworked.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

    Development

    Participants

    @m-ou-se

    Issue actions
      Better suggestion for `panic!(format!(..))` · Issue #82110 · rust-lang/rust