-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Revive suggestion on explicit_write #4542
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
Labels
C-enhancement
Category: Enhancement of lints, like adding more cases or adding help messages
L-suggestion
Lint: Improving, adding or fixing lint suggestions
Comments
JoshMcguigan
added a commit
to JoshMcguigan/rust-clippy
that referenced
this issue
Sep 15, 2019
JoshMcguigan
added a commit
to JoshMcguigan/rust-clippy
that referenced
this issue
Sep 17, 2019
JoshMcguigan
added a commit
to JoshMcguigan/rust-clippy
that referenced
this issue
Sep 17, 2019
JoshMcguigan
added a commit
to JoshMcguigan/rust-clippy
that referenced
this issue
Sep 17, 2019
JoshMcguigan
added a commit
to JoshMcguigan/rust-clippy
that referenced
this issue
Sep 17, 2019
JoshMcguigan
added a commit
to JoshMcguigan/rust-clippy
that referenced
this issue
Sep 18, 2019
bors
added a commit
that referenced
this issue
Sep 19, 2019
#4542 remove machine applicable suggestion This helps #4542 (but does not completely resolve) by removing the machine applicable suggestion (which was incorrect) for that case. I would have preferred to fix the machine applicable suggestion to handle format strings, but that's a bit beyond my current understanding of the clippy codebase. I'd be happy to give it a try given some guidance.
flip1995
added a commit
to flip1995/rust-clippy
that referenced
this issue
Sep 19, 2019
rust-lang#4542 remove machine applicable suggestion This helps rust-lang#4542 (but does not completely resolve) by removing the machine applicable suggestion (which was incorrect) for that case. I would have preferred to fix the machine applicable suggestion to handle format strings, but that's a bit beyond my current understanding of the clippy codebase. I'd be happy to give it a try given some guidance.
flip1995
added a commit
to flip1995/rust-clippy
that referenced
this issue
Sep 19, 2019
rust-lang#4542 remove machine applicable suggestion This helps rust-lang#4542 (but does not completely resolve) by removing the machine applicable suggestion (which was incorrect) for that case. I would have preferred to fix the machine applicable suggestion to handle format strings, but that's a bit beyond my current understanding of the clippy codebase. I'd be happy to give it a try given some guidance. changelog: only produce machine applicable suggestions on `explicit_write` lint
bors
added a commit
that referenced
this issue
Sep 19, 2019
Rollup of 4 pull requests Successful merges: - #4511 (New lint: mem_replace_with_uninit) - #4535 (New lint: Require `# Safety` section in pub unsafe fn docs) - #4539 (Changes cast-lossless to a pedantic lint) - #4544 (#4542 remove machine applicable suggestion) Failed merges: r? @ghost changelog: none
bors
added a commit
that referenced
this issue
Sep 19, 2019
#4542 remove machine applicable suggestion This helps #4542 (but does not completely resolve) by removing the machine applicable suggestion (which was incorrect) for that case. I would have preferred to fix the machine applicable suggestion to handle format strings, but that's a bit beyond my current understanding of the clippy codebase. I'd be happy to give it a try given some guidance. changelog: only produce machine applicable suggestions on `explicit_write` lint
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
C-enhancement
Category: Enhancement of lints, like adding more cases or adding help messages
L-suggestion
Lint: Improving, adding or fixing lint suggestions
Uh oh!
There was an error while loading. Please reload this page.
should be
but clippy suggests
It should be
(Playground)
UPDATE
#4544 removed the wrong suggestion. We should find the right way and revive suggestions.
The text was updated successfully, but these errors were encountered: