governor icon indicating copy to clipboard operation
governor copied to clipboard

All clippy warnings that currently block compliation

Open koshell opened this issue 1 year ago • 1 comments
trafficstars

Versions: rustc 1.79.0 clippy 0.1.79

Warnings: clippy::default_constructed_unit_structs clippy::single_char_pattern clippy::len_zero clippy::clone_on_copy clippy::duplicated_attributes

After going back through to check for these I realised all of them, except clippy::duplicated_attributes, occur in tests and therefore can likely be ignored.

Changing

#![deny(warnings)]

to

#![cfg_attr(not(test), deny(warnings))]

would allow the crate to compile (assuming clippy::duplicated_attributes is resolved).

However I would recommend reading over this and considering if you even want to keep deny(warnings) in general.

koshell avatar Jun 18 '24 00:06 koshell

Hrm, it seems that I was building this with the wrong clippy version all along, can now repro them. I think your suggestion of turning the deny(warnings) off in tests is sound - it's not like we care much about perf or other little nits there.

duplicated_attributes is worth fixing though, of course (:

antifuchs avatar Jun 18 '24 02:06 antifuchs