Useless format!/to_string in argument of function accepting fmt::Display

What it does

Detects the use of format!() and .to_string() in arguments to a function expecting impl fmt::Display.

(Possible duplicate of #3021, #3156)

Lint Name

useless_format

Category

perf, pedantic

Advantage

• Remove useless allocation

Drawbacks

• May be hard to implement?

Example

fn foo(_: impl std::fmt::Display) {}

foo("xyz".to_string());
foo(format!("{} {}", 1, 2));

Could be written as:

fn foo(_: impl std::fmt::Display) {}

foo("xyz");
foo(format_args!("{} {}", 1, 2));

Start by reviewing related issues #3021 and #3156, then examine existing Clippy lint implementations for similar formatting or performance checks. Done means the useless_format lint detects format! and to_string() passed to functions accepting fmt::Display and supports the examples described here.