address a bunch of lints

[djc]

I'd prefer to leave out "avoid using match on bools" and "remove neeedless pass by value", please remove those commits.

Is this clippy at pedantic level or something?

[danieleades]

I'd prefer to leave out "avoid using match on bools" and "remove neeedless pass by value", please remove those commits.

Is this clippy at pedantic level or something?

it's a combination of pedantic and nursery lints, as well as some standard lints triggered by edits made in fixing other lints.

Some of them are just newer lints- the CI job is using the stable clippy version, i'm using nightly

i've dropped those two commits

[djc]

Thanks!