Warn if function name default() is used without implementing the Default trait

[kamulos]

Description

Just stumbled on this in a code review. Would it make sense to check the naming of a member function? If a default() function is used (especially as a constructor) I would expect, that the Default trait is implemented.

Not sure if this would be it's own lint or integrated in an existing one

Version

No response

Additional Labels

No response

[smoelius]

Personally, I would generalize this to: any inherent method decalartion whose name matches a std trait method name.

There may be reasons why this is too general, but I can't think of any offhand.

[smoelius]

Today, I noticed this:

rust-clippy/clippy_lints/src/methods/mod.rs

Line 2642 in f07ee8a

// FIXME: default doesn't work

Apparently, should_implement_trait has been around for a while. This seems to be the PR that introduced it: #228

[kraktus]

@rustbot claim