incompatible_msrv false positive: Trait methods of the same name

[joshlf]

Summary

The incompatible_msrv lint fires when a method was stabilized more recently than the MSRV, but there is also a trait in scope with a method of the same name. In this case, on toolchains prior to stabilization, the trait's method will be chosen instead, and so compilation will still succeed. This technique is sometimes used to implement polyfills.

To repro, run cargo +nightly-2024-02-12 clippy -p zerocopy-derive at this commit:

$ cargo +nightly-2024-02-12 clippy -p zerocopy-derive
   Compiling proc-macro2 v1.0.76
    Checking unicode-ident v1.0.12
    Checking quote v1.0.35
    Checking syn v2.0.48
    Checking zerocopy-derive v0.8.0-alpha.4 (/Users/josh/workspace/zerocopy/zerocopy-derive)
error: current MSRV (Minimum Supported Rust Version) is `1.56.0` but this item is stable since `1.62.0`
   --> zerocopy-derive/src/lib.rs:943:83
    |
943 |     let padding_check_bound = padding_check.and_then(|check| (!fields.is_empty()).then_some(check)).map(|check| {
    |                                                                                   ^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incompatible_msrv
note: the lint level is defined here
   --> zerocopy-derive/src/lib.rs:19:9
    |
19  | #![deny(clippy::all, clippy::missing_safety_doc, clippy::undocumented_unsafe_blocks)]
    |         ^^^^^^^^^^^
    = note: `#[deny(clippy::incompatible_msrv)]` implied by `#[deny(clippy::all)]`

error: could not compile `zerocopy-derive` (lib) due to 1 previous error

The following code also exists in the same module:

// A polyfill for `Option::then_some`, which was added after our MSRV.
//
// The `#[allow(unused)]` is necessary because, on sufficiently recent toolchain
// versions, `b.then_some(...)` resolves to the inherent method rather than to
// this trait, and so this trait is considered unused.
//
// TODO(#67): Remove this once our MSRV is >= 1.62.
#[allow(unused)]
trait BoolExt {
    fn then_some<T>(self, t: T) -> Option<T>;
}

impl BoolExt for bool {
    fn then_some<T>(self, t: T) -> Option<T> {
        if self {
            Some(t)
        } else {
            None
        }
    }
}

Lint Name

incompatible_msrv

Version

nightly-2024-02-12

[bluss]

This also happens for the new NonNull methods vs the PointerExt methods of the same name.