`unwrap_or_default` suggests replacing `T::new`

Summary

unwrap_or_default suggests replacing unwrap_or_else(T::new) with unwrap_or_default() when Default is implemented via new.

Lint Name

unwrap_or_default

Reproducer

I tried this code (playground):

pub struct Foo(u32);

impl Foo {
    fn new() -> Self {
        Self(0)
    }
}

impl Default for Foo {
    fn default() -> Self {
        Self::new()
    }
}

pub fn foo(x: Option<Foo>) -> Foo {
    x.unwrap_or_else(Foo::new)
}

I saw this happen:

warning: use of `unwrap_or_else` to construct default value
  --> src/lib.rs:16:7
   |
16 |     x.unwrap_or_else(Foo::new)
   |       ^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `unwrap_or_default()`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_or_default
   = note: `#[warn(clippy::unwrap_or_default)]` on by default

I expected to see no warning. The implementation of Default could be changed to something other than new, potentially making all code that was changed to unwrap_or_default invalid.

Version

rustc 1.73.0 (cc66ad468 2023-10-03)
binary: rustc
commit-hash: cc66ad468955717ab92600c770da8c1601a4ff33
commit-date: 2023-10-03
host: aarch64-apple-darwin
release: 1.73.0
LLVM version: 17.0.2

Additional Labels

No response

Start with the unwrap_or_default lint implementation and the reproducer in the issue. Check how the lint decides that Foo::new constructs a default value, then verify the behavior against the shown Rust example. Done means the lint no longer suggests unwrap_or_default() for this case while preserving its valid suggestions.