rust-windowing / winit

Window handling library in pure Rust
https://docs.rs/winit/
Apache License 2.0
4.87k stars 910 forks source link

style: simplify string formatting for readability #4001

Closed hamirmahal closed 6 hours ago

hamirmahal commented 2 days ago

Closes #4000

madsmtm commented 2 days ago

It sounds like you're using an automated tool to find these? From a quick look I found at least the similar https://github.com/tembo-io/pgmq/pull/341, https://github.com/stalwartlabs/mail-server/pull/909, https://github.com/mullvad/pfctl-rs/pull/119 and https://github.com/amodm/webbrowser-rs/pull/91.

It would be really nice if more context was given (that these were found in an automated fashion, if so), otherwise it feels disingenuous?

hamirmahal commented 1 day ago

Oh, I found these with clippy.

madsmtm commented 1 day ago

Ah, makes sense. Would you be up for adding the specific lints to lib.rs, then?

hamirmahal commented 1 day ago

Sure! I'll take a look at that right now.

hamirmahal commented 6 hours ago

You're welcome.