Skip to content
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

Re-enable clippy after upstream crash fix #932

Merged
merged 4 commits into from
Sep 12, 2021
Merged

Re-enable clippy after upstream crash fix #932

merged 4 commits into from
Sep 12, 2021

Conversation

dtolnay
Copy link
Owner

@dtolnay dtolnay commented Sep 12, 2021

The Clippy crash that caused us to disable it (rust-lang/rust-clippy#7423) was fixed in rust-lang/rust-clippy#7428 + rust-lang/rust#87152 in nightly-2021-07-16.

    error: this `if` has identical blocks
       --> gen/build/src/gen/write.rs:704:33
        |
    704 |           if arg.ty == RustString {
        |  _________________________________^
    705 | |             write!(out, "const ");
    706 | |         } else if let Type::RustVec(_) = arg.ty {
        | |_________^
        |
        = note: `-D clippy::if-same-then-else` implied by `-D clippy::all`
    note: same as this
       --> gen/build/src/gen/write.rs:706:49
        |
    706 |           } else if let Type::RustVec(_) = arg.ty {
        |  _________________________________________________^
    707 | |             write!(out, "const ");
    708 | |         }
        | |_________^
        = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#if_same_then_else

    error: this `if` has identical blocks
       --> macro/src/expand.rs:430:33
        |
    430 |           if arg.ty == RustString {
        |  _________________________________^
    431 | |             quote!(#var #colon *const #ty)
    432 | |         } else if let Type::RustVec(_) = arg.ty {
        | |_________^
        |
        = note: `-D clippy::if-same-then-else` implied by `-D clippy::all`
    note: same as this
       --> macro/src/expand.rs:432:49
        |
    432 |           } else if let Type::RustVec(_) = arg.ty {
        |  _________________________________________________^
    433 | |             quote!(#var #colon *const #ty)
    434 | |         } else if let Type::Fn(_) = arg.ty {
        | |_________^
        = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#if_same_then_else
    warning: lint `clippy::filter_map` has been removed: this lint has been replaced by `manual_filter_map`, a more specific lint
     --> macro/src/lib.rs:6:5
      |
    6 |     clippy::filter_map,
      |     ^^^^^^^^^^^^^^^^^^
      |
      = note: `#[warn(renamed_and_removed_lints)]` on by default
@dtolnay dtolnay merged commit 5d71b1f into master Sep 12, 2021
@dtolnay dtolnay deleted the clippy branch September 12, 2021 12:31
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.

1 participant