Fix up some lints on Windows

[RossSmyth]

Lints are not run for Windows in CI so they are missed. There are a couple that have been showing for a few months now so here are some fixes.

The reasoning behind the unused field is because of here:

fd/src/output.rs

Lines 159 to 167 in 9f0fea6

	#[cfg(unix)]
	fn print_entry_uncolorized<W: Write>(
	stdout: &mut W,
	entry: &DirEntry,
	config: &Config,
	) -> io::Result<()> {
	use std::os::unix::ffi::OsStrExt;
	if config.interactive_terminal || config.path_separator.is_some() {

Currently in CI you are only running Clippy for Linux. cargo clippy --all-targets does not lint for all target triples, but rather all compilation targets (binaries, libraries, tests, examples). So for each target you have cfg'd code, to lint that code you need to run Clippy on that target. Usually in the CI matrix in another step. Cross Clippy is theoretically possible but I've never gotten it to work reliably.

[RossSmyth]

oops had the wrong email on my commits