New Lint: blacklisted_method

[ilknarf]

What it does

Blacklist Method Disallowed Method (see #5896)
What does this lint do?
Disallows certain methods, as defined in clippy.toml

Categories (optional)

• Kind: Nursery

Checks for method calls of specified methods
For example:

• SomeStruct::bad_method
• SomeTrait::bad_method

Drawbacks

This might take some significant overhead to implement (regex or some manual string parsing), and the main purpose would probably be to patch in temporary lints, so it might not be super useful.

Example

# clippy.toml

disallowed-methods = ["Foo::bar", "Baz::bar"]

...
let f = Foo::default();
f.bar();
...

results in a lint

[flip1995]

Should only the usage be blacklisted or also the definition?

[ilknarf]

We were mostly looking at usage. I can pick it up too.

[ghost]

Apparently we don't use the term "blacklist" anymore - #5896 . Maybe choose something else if you don't want someone to come along and rename it.

[ilknarf]

Ok I vibe with that. Would banned_method be appropriate nomenclature for the future?

[ghost]

The other PR used the term 'disallowed' so maybe go with disallowed_method. I probably not the right person to answer this.

[ilknarf]

Many thanks! I will go with that, then.