Squish a Bunch of Clippy Errors and Warnings

[Pi-Cla]

There are some remaining such as these:

error: &-masking with zero
   --> wezterm-input-types/src/lib.rs:479:1
    |
479 | / bitflags! {
480 | |     #[cfg_attr(feature="serde", derive(Serialize, Deserialize))]
481 | |     #[derive(Default, FromDynamic, ToDynamic)]
482 | |     #[dynamic(into="String", try_from="String")]
...   |
498 | |     }
499 | | }
    | |_^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bad_bit_mask
    = note: `#[deny(clippy::bad_bit_mask)]` on by default
    = note: this error originates in the macro `__impl_bitflags` which comes from the expansion of the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)

error: &-masking with zero
    --> wezterm-input-types/src/lib.rs:1174:1
     |
1174 | / bitflags! {
1175 | |     #[derive(Default)]
1176 | |     pub struct MouseButtons: u8 {
1177 | |         const NONE = 0;
...    |
1184 | |     }
1185 | | }
     | |_^
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bad_bit_mask
     = note: this error originates in the macro `__impl_bitflags` which comes from the expansion of the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
...

But I got rid of a lot of them

[wez]

I appreciate the time you spent on this, but I generally do not want to take clippy diffs like this. I find that much of what clippy suggests is highly opinionated, at-odds with my own style, often makes things more verbose, and is not generally useful here.
These giant clippy diffs are time consuming to review to try to find net positives, and I think I've only ever found one actually useful snippet in the history of this kind of PR.

So I'm now just going to close them without looking in depth.

Sorry if this is disappointing to you, but it's not worth the effort to me.