[INFO] fetching crate nixpkgs-fmt-rnix 1.2.0...
[INFO] linting nixpkgs-fmt-rnix-1.2.0 against nightly for clippy-nonminimal_bool-denied
[INFO] extracting crate nixpkgs-fmt-rnix 1.2.0 into /workspace/builds/worker-3-tc1/source
[INFO] started tweaking crates.io crate nixpkgs-fmt-rnix 1.2.0
[INFO] finished tweaking crates.io crate nixpkgs-fmt-rnix 1.2.0
[INFO] tweaked toml for crates.io crate nixpkgs-fmt-rnix 1.2.0 written to /workspace/builds/worker-3-tc1/source/Cargo.toml
[INFO] validating manifest of crates.io crate nixpkgs-fmt-rnix 1.2.0 on toolchain nightly
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+nightly" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate crates.io crate nixpkgs-fmt-rnix 1.2.0 already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+nightly" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded text-size v1.1.0
[INFO] [stderr]   Downloaded winapi-util v0.1.3
[INFO] [stderr]   Downloaded memoffset v0.6.1
[INFO] [stderr]   Downloaded smol_str v0.1.17
[INFO] [stderr]   Downloaded unindent v0.1.5
[INFO] [stderr]   Downloaded hermit-abi v0.1.8
[INFO] [stderr]   Downloaded globset v0.4.5
[INFO] [stderr]   Downloaded serde v1.0.105
[INFO] [stderr]   Downloaded ignore v0.4.13
[INFO] [stderr]   Downloaded serde_json v1.0.48
[INFO] [stderr]   Downloaded crossbeam-channel v0.4.2
[INFO] [stderr]   Downloaded rnix v0.10.1
[INFO] [stderr]   Downloaded regex v1.3.5
[INFO] [stderr]   Downloaded bstr v0.2.12
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] cf748e4b6e1770b044d2efe8fac5a3ed06cb91a98c884fcad82b53f08d4b24cd
[INFO] running `Command { std: "docker" "start" "-a" "cf748e4b6e1770b044d2efe8fac5a3ed06cb91a98c884fcad82b53f08d4b24cd", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "cf748e4b6e1770b044d2efe8fac5a3ed06cb91a98c884fcad82b53f08d4b24cd", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "cf748e4b6e1770b044d2efe8fac5a3ed06cb91a98c884fcad82b53f08d4b24cd", kill_on_drop: false }`
[INFO] [stdout] cf748e4b6e1770b044d2efe8fac5a3ed06cb91a98c884fcad82b53f08d4b24cd
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "clippy" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 825c5bd0081075c913e1a29b5fdcf005502a11d456d7f92e5d3a5dc143a0f7f3
[INFO] running `Command { std: "docker" "start" "-a" "825c5bd0081075c913e1a29b5fdcf005502a11d456d7f92e5d3a5dc143a0f7f3", kill_on_drop: false }`
[INFO] [stderr]    Compiling autocfg v1.0.0
[INFO] [stderr]    Compiling serde v1.0.105
[INFO] [stderr]    Compiling libc v0.2.68
[INFO] [stderr]    Compiling log v0.4.8
[INFO] [stderr]    Compiling ryu v1.0.3
[INFO] [stderr]     Checking regex-syntax v0.6.17
[INFO] [stderr]     Checking unicode-width v0.1.7
[INFO] [stderr]     Checking hashbrown v0.9.1
[INFO] [stderr]     Checking countme v2.0.4
[INFO] [stderr]     Checking fnv v1.0.6
[INFO] [stderr]     Checking crossbeam-utils v0.6.6
[INFO] [stderr]     Checking walkdir v2.3.1
[INFO] [stderr]     Checking aho-corasick v0.7.10
[INFO] [stderr]     Checking bstr v0.2.12
[INFO] [stderr]     Checking itoa v0.4.5
[INFO] [stderr]     Checking smol_str v0.1.17
[INFO] [stderr]     Checking vec_map v0.8.1
[INFO] [stderr]     Checking crossbeam-channel v0.3.9
[INFO] [stderr]     Checking textwrap v0.11.0
[INFO] [stderr]     Checking unindent v0.1.5
[INFO] [stderr]    Compiling crossbeam-utils v0.7.2
[INFO] [stderr]    Compiling num-traits v0.2.11
[INFO] [stderr]    Compiling memoffset v0.6.1
[INFO] [stderr]     Checking atty v0.2.14
[INFO] [stderr]     Checking clap v2.33.0
[INFO] [stderr]     Checking crossbeam-channel v0.4.2
[INFO] [stderr]     Checking cbitset v0.2.0
[INFO] [stderr]     Checking regex v1.3.5
[INFO] [stderr]     Checking globset v0.4.5
[INFO] [stderr]     Checking ignore v0.4.13
[INFO] [stderr]     Checking text-size v1.1.0
[INFO] [stderr]     Checking serde_json v1.0.48
[INFO] [stderr]     Checking rowan v0.12.6
[INFO] [stderr]     Checking rnix v0.10.1
[INFO] [stderr]     Checking nixpkgs-fmt-rnix v1.2.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unnecessary parentheses around type
[INFO] [stdout]   --> src/pattern.rs:28:19
[INFO] [stdout]    |
[INFO] [stdout] 28 |     pred: Arc<dyn (Fn(&SyntaxElement) -> bool)>,
[INFO] [stdout]    |                   ^                          ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 28 -     pred: Arc<dyn (Fn(&SyntaxElement) -> bool)>,
[INFO] [stdout] 28 +     pred: Arc<dyn Fn(&SyntaxElement) -> bool>,
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around type
[INFO] [stdout]   --> src/pattern.rs:28:19
[INFO] [stdout]    |
[INFO] [stdout] 28 |     pred: Arc<dyn (Fn(&SyntaxElement) -> bool)>,
[INFO] [stdout]    |                   ^                          ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 28 -     pred: Arc<dyn (Fn(&SyntaxElement) -> bool)>,
[INFO] [stdout] 28 +     pred: Arc<dyn Fn(&SyntaxElement) -> bool>,
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `indent_value` is never read
[INFO] [stdout]    --> src/dsl.rs:300:16
[INFO] [stdout]     |
[INFO] [stdout] 275 | pub(crate) struct IndentRule {
[INFO] [stdout]     |                   ---------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 300 |     pub(crate) indent_value: IndentValue,
[INFO] [stdout]     |                ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `IndentRule` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/engine/indentation.rs:206:5
[INFO] [stdout]     |
[INFO] [stdout] 206 |     return;
[INFO] [stdout]     |     ^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout]     = note: `#[warn(clippy::needless_return)]` on by default
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 205 -     }
[INFO] [stdout] 206 -     return;
[INFO] [stdout] 205 +     }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler
[INFO] [stdout]    --> src/engine/indentation.rs:194:45
[INFO] [stdout]     |
[INFO] [stdout] 194 |     let anchor_indent = match indent_anchor(&syntax_element, model, anchor_set) {
[INFO] [stdout]     |                                             ^^^^^^^^^^^^^^^ help: change this to: `syntax_element`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
[INFO] [stdout]     = note: `#[warn(clippy::needless_borrow)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler
[INFO] [stdout]    --> src/engine/indentation.rs:198:33
[INFO] [stdout]     |
[INFO] [stdout] 198 |     let block = model.block_for(&syntax_element, BlockPosition::Before);
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^ help: change this to: `syntax_element`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this `map_or` can be simplified
[INFO] [stdout]   --> src/engine/spacing.rs:66:39
[INFO] [stdout]    |
[INFO] [stdout] 66 |             let parent_is_multiline = element.parent().map_or(false, |it| has_newline(&it));
[INFO] [stdout]    |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_map_or
[INFO] [stdout]    = note: `#[warn(clippy::unnecessary_map_or)]` on by default
[INFO] [stdout] help: use `is_some_and` instead
[INFO] [stdout]    |
[INFO] [stdout] 66 -             let parent_is_multiline = element.parent().map_or(false, |it| has_newline(&it));
[INFO] [stdout] 66 +             let parent_is_multiline = element.parent().is_some_and(|it| has_newline(&it));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this `map_or` can be simplified
[INFO] [stdout]   --> src/engine/spacing.rs:74:39
[INFO] [stdout]    |
[INFO] [stdout] 74 |             let parent_is_multiline = element.parent().map_or(false, |it| has_newline(&it));
[INFO] [stdout]    |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_map_or
[INFO] [stdout] help: use `is_some_and` instead
[INFO] [stdout]    |
[INFO] [stdout] 74 -             let parent_is_multiline = element.parent().map_or(false, |it| has_newline(&it));
[INFO] [stdout] 74 +             let parent_is_multiline = element.parent().is_some_and(|it| has_newline(&it));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary closure used to substitute value for `Option::None`
[INFO] [stdout]    --> src/engine/fixes.rs:178:26
[INFO] [stdout]     |
[INFO] [stdout] 178 |         let indent_len = s.find(|c| c != ' ').unwrap_or_else(|| s.len());
[INFO] [stdout]     |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_lazy_evaluations
[INFO] [stdout]     = note: `#[warn(clippy::unnecessary_lazy_evaluations)]` on by default
[INFO] [stdout] help: use `unwrap_or` instead
[INFO] [stdout]     |
[INFO] [stdout] 178 -         let indent_len = s.find(|c| c != ' ').unwrap_or_else(|| s.len());
[INFO] [stdout] 178 +         let indent_len = s.find(|c| c != ' ').unwrap_or(s.len());
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `indent_value` is never read
[INFO] [stdout]    --> src/dsl.rs:300:16
[INFO] [stdout]     |
[INFO] [stdout] 275 | pub(crate) struct IndentRule {
[INFO] [stdout]     |                   ---------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 300 |     pub(crate) indent_value: IndentValue,
[INFO] [stdout]     |                ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `IndentRule` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:169:5
[INFO] [stdout]     |
[INFO] [stdout] 169 | /     return if let Some(body) = prev.clone().and_then(With::cast).and_then(|w| w.body()) {
[INFO] [stdout] 170 | |         is_literal(body.kind())
[INFO] [stdout] 171 | |     } else {
[INFO] [stdout] 172 | |         prev.map(|it| is_literal(it.kind())) == Some(true)
[INFO] [stdout] 173 | |     };
[INFO] [stdout]     | |_____^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 169 ~     if let Some(body) = prev.clone().and_then(With::cast).and_then(|w| w.body()) {
[INFO] [stdout] 170 +         is_literal(body.kind())
[INFO] [stdout] 171 +     } else {
[INFO] [stdout] 172 +         prev.map(|it| is_literal(it.kind())) == Some(true)
[INFO] [stdout] 173 ~     }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this boolean expression can be simplified
[INFO] [stdout]    --> src/rules.rs:192:33
[INFO] [stdout]     |
[INFO] [stdout] 192 |                     _ => return true && before_token_has_newline(&it.into()),
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `before_token_has_newline(&it.into())`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool
[INFO] [stdout]     = note: `#[warn(clippy::nonminimal_bool)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:195:13
[INFO] [stdout]     |
[INFO] [stdout] 195 |             return false;
[INFO] [stdout]     |             ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 195 -             return false;
[INFO] [stdout] 195 +             false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: returning the result of a `let` binding from a block
[INFO] [stdout]    --> src/rules.rs:208:9
[INFO] [stdout]     |
[INFO] [stdout] 203 | /         let inline_attr = element
[INFO] [stdout] 204 | |             .parent()?
[INFO] [stdout] 205 | |             .descendants_with_tokens()
[INFO] [stdout] 206 | |             .find(|e| e.kind() == NODE_ATTR_SET)
[INFO] [stdout] 207 | |             .map(|t| before_token_has_newline(&t));
[INFO] [stdout]     | |___________________________________________________- unnecessary `let` binding
[INFO] [stdout] 208 |           inline_attr
[INFO] [stdout]     |           ^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
[INFO] [stdout]     = note: `#[warn(clippy::let_and_return)]` on by default
[INFO] [stdout] help: return the expression directly
[INFO] [stdout]     |
[INFO] [stdout] 203 ~         
[INFO] [stdout] 204 ~         element
[INFO] [stdout] 205 +             .parent()?
[INFO] [stdout] 206 +             .descendants_with_tokens()
[INFO] [stdout] 207 +             .find(|e| e.kind() == NODE_ATTR_SET)
[INFO] [stdout] 208 +             .map(|t| before_token_has_newline(&t))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:249:17
[INFO] [stdout]     |
[INFO] [stdout] 249 |                 return false;
[INFO] [stdout]     |                 ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 249 -                 return false;
[INFO] [stdout] 249 +                 false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:251:22
[INFO] [stdout]     |
[INFO] [stdout] 251 |             false => return has_newline(&parent),
[INFO] [stdout]     |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 251 -             false => return has_newline(&parent),
[INFO] [stdout] 251 +             false => has_newline(&parent),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: redundant closure
[INFO] [stdout]    --> src/rules.rs:241:59
[INFO] [stdout]     |
[INFO] [stdout] 241 | ...                   return node.as_node().map(|t| has_newline(t)).unwrap_or(false);
[INFO] [stdout]     |                                                 ^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `has_newline`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
[INFO] [stdout]     = note: `#[warn(clippy::redundant_closure)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: returning the result of a `let` binding from a block
[INFO] [stdout]    --> src/rules.rs:273:5
[INFO] [stdout]     |
[INFO] [stdout] 265 | /     let is_last_argument = match element.parent() {
[INFO] [stdout] 266 | |         None => false,
[INFO] [stdout] 267 | |         Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout] 268 | |             None => true,
[INFO] [stdout] 269 | |             _ => false,
[INFO] [stdout] 270 | |         },
[INFO] [stdout] 271 | |     };
[INFO] [stdout]     | |______- unnecessary `let` binding
[INFO] [stdout] 272 |
[INFO] [stdout] 273 |       is_last_argument
[INFO] [stdout]     |       ^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
[INFO] [stdout] help: return the expression directly
[INFO] [stdout]     |
[INFO] [stdout] 265 ~     
[INFO] [stdout] 266 |
[INFO] [stdout] 267 ~     match element.parent() {
[INFO] [stdout] 268 +         None => false,
[INFO] [stdout] 269 +         Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout] 270 +             None => true,
[INFO] [stdout] 271 +             _ => false,
[INFO] [stdout] 272 +         },
[INFO] [stdout] 273 +     }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: match expression looks like `matches!` macro
[INFO] [stdout]    --> src/rules.rs:267:21
[INFO] [stdout]     |
[INFO] [stdout] 267 |           Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout]     |  _____________________^
[INFO] [stdout] 268 | |             None => true,
[INFO] [stdout] 269 | |             _ => false,
[INFO] [stdout] 270 | |         },
[INFO] [stdout]     | |_________^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_like_matches_macro
[INFO] [stdout]     = note: `#[warn(clippy::match_like_matches_macro)]` on by default
[INFO] [stdout] help: use `matches!` directly
[INFO] [stdout]     |
[INFO] [stdout] 267 -         Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout] 268 -             None => true,
[INFO] [stdout] 269 -             _ => false,
[INFO] [stdout] 270 -         },
[INFO] [stdout] 267 +         Some(it) => matches!(next_sibling(&it.into()), None),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: redundant pattern matching, consider using `is_none()`
[INFO] [stdout]    --> src/rules.rs:267:21
[INFO] [stdout]     |
[INFO] [stdout] 267 |           Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout]     |  _____________________^
[INFO] [stdout] 268 | |             None => true,
[INFO] [stdout] 269 | |             _ => false,
[INFO] [stdout] 270 | |         },
[INFO] [stdout]     | |_________^ help: try: `next_sibling(&it.into()).is_none()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching
[INFO] [stdout]     = note: `#[warn(clippy::redundant_pattern_matching)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:289:5
[INFO] [stdout]     |
[INFO] [stdout] 289 |     return false;
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 289 -     return false;
[INFO] [stdout] 289 +     false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: match expression looks like `matches!` macro
[INFO] [stdout]    --> src/rules.rs:298:31
[INFO] [stdout]     |
[INFO] [stdout] 298 |               .filter(|element| match element {
[INFO] [stdout]     |  _______________________________^
[INFO] [stdout] 299 | |                 NodeOrToken::Token(_) => true,
[INFO] [stdout] 300 | |                 _ => false,
[INFO] [stdout] 301 | |             })
[INFO] [stdout]     | |_____________^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_like_matches_macro
[INFO] [stdout] help: use `matches!` directly
[INFO] [stdout]     |
[INFO] [stdout] 298 -             .filter(|element| match element {
[INFO] [stdout] 299 -                 NodeOrToken::Token(_) => true,
[INFO] [stdout] 300 -                 _ => false,
[INFO] [stdout] 301 -             })
[INFO] [stdout] 298 +             .filter(|element| matches!(element, NodeOrToken::Token(_)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:332:17
[INFO] [stdout]     |
[INFO] [stdout] 332 |                 return v;
[INFO] [stdout]     |                 ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 332 -                 return v;
[INFO] [stdout] 332 +                 v
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:335:17
[INFO] [stdout]     |
[INFO] [stdout] 335 |                 return v;
[INFO] [stdout]     |                 ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 335 -                 return v;
[INFO] [stdout] 335 +                 v
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: redundant closure
[INFO] [stdout]    --> src/rules.rs:360:61
[INFO] [stdout]     |
[INFO] [stdout] 360 |         next_non_whitespace_sibling(element)?.as_node().map(|e| has_newline(e))
[INFO] [stdout]     |                                                             ^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `has_newline`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: match expression looks like `matches!` macro
[INFO] [stdout]    --> src/rules.rs:684:31
[INFO] [stdout]     |
[INFO] [stdout] 684 |               .filter(|element| match element {
[INFO] [stdout]     |  _______________________________^
[INFO] [stdout] 685 | |                 NodeOrToken::Token(_) => true,
[INFO] [stdout] 686 | |                 _ => false,
[INFO] [stdout] 687 | |             })
[INFO] [stdout]     | |_____________^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_like_matches_macro
[INFO] [stdout] help: use `matches!` directly
[INFO] [stdout]     |
[INFO] [stdout] 684 -             .filter(|element| match element {
[INFO] [stdout] 685 -                 NodeOrToken::Token(_) => true,
[INFO] [stdout] 686 -                 _ => false,
[INFO] [stdout] 687 -             })
[INFO] [stdout] 684 +             .filter(|element| matches!(element, NodeOrToken::Token(_)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: called `.nth(0)` on a `std::iter::Iterator`, when `.next()` is equivalent
[INFO] [stdout]    --> src/rules.rs:727:24
[INFO] [stdout]     |
[INFO] [stdout] 727 |         let first_el = element.as_node()?.descendants().filter(|e| e.kind() != NODE_BIN_OP).nth(0);
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try calling `.next()` instead of `.nth(0)`: `element.as_node()?.descendants().filter(|e| e.kind() != NODE_BIN_OP).next()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#iter_nth_zero
[INFO] [stdout]     = note: `#[warn(clippy::iter_nth_zero)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/engine/indentation.rs:206:5
[INFO] [stdout]     |
[INFO] [stdout] 206 |     return;
[INFO] [stdout]     |     ^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout]     = note: `#[warn(clippy::needless_return)]` on by default
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 205 -     }
[INFO] [stdout] 206 -     return;
[INFO] [stdout] 205 +     }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler
[INFO] [stdout]    --> src/engine/indentation.rs:194:45
[INFO] [stdout]     |
[INFO] [stdout] 194 |     let anchor_indent = match indent_anchor(&syntax_element, model, anchor_set) {
[INFO] [stdout]     |                                             ^^^^^^^^^^^^^^^ help: change this to: `syntax_element`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
[INFO] [stdout]     = note: `#[warn(clippy::needless_borrow)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler
[INFO] [stdout]    --> src/engine/indentation.rs:198:33
[INFO] [stdout]     |
[INFO] [stdout] 198 |     let block = model.block_for(&syntax_element, BlockPosition::Before);
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^ help: change this to: `syntax_element`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this `map_or` can be simplified
[INFO] [stdout]   --> src/engine/spacing.rs:66:39
[INFO] [stdout]    |
[INFO] [stdout] 66 |             let parent_is_multiline = element.parent().map_or(false, |it| has_newline(&it));
[INFO] [stdout]    |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_map_or
[INFO] [stdout]    = note: `#[warn(clippy::unnecessary_map_or)]` on by default
[INFO] [stdout] help: use `is_some_and` instead
[INFO] [stdout]    |
[INFO] [stdout] 66 -             let parent_is_multiline = element.parent().map_or(false, |it| has_newline(&it));
[INFO] [stdout] 66 +             let parent_is_multiline = element.parent().is_some_and(|it| has_newline(&it));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this `map_or` can be simplified
[INFO] [stdout]   --> src/engine/spacing.rs:74:39
[INFO] [stdout]    |
[INFO] [stdout] 74 |             let parent_is_multiline = element.parent().map_or(false, |it| has_newline(&it));
[INFO] [stdout]    |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_map_or
[INFO] [stdout] help: use `is_some_and` instead
[INFO] [stdout]    |
[INFO] [stdout] 74 -             let parent_is_multiline = element.parent().map_or(false, |it| has_newline(&it));
[INFO] [stdout] 74 +             let parent_is_multiline = element.parent().is_some_and(|it| has_newline(&it));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary closure used to substitute value for `Option::None`
[INFO] [stdout]    --> src/engine/fixes.rs:178:26
[INFO] [stdout]     |
[INFO] [stdout] 178 |         let indent_len = s.find(|c| c != ' ').unwrap_or_else(|| s.len());
[INFO] [stdout]     |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_lazy_evaluations
[INFO] [stdout]     = note: `#[warn(clippy::unnecessary_lazy_evaluations)]` on by default
[INFO] [stdout] help: use `unwrap_or` instead
[INFO] [stdout]     |
[INFO] [stdout] 178 -         let indent_len = s.find(|c| c != ' ').unwrap_or_else(|| s.len());
[INFO] [stdout] 178 +         let indent_len = s.find(|c| c != ' ').unwrap_or(s.len());
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: deref which would be done by auto-deref
[INFO] [stdout]   --> src/lib.rs:97:27
[INFO] [stdout]    |
[INFO] [stdout] 97 |     let ast = rnix::parse(&*text);
[INFO] [stdout]    |                           ^^^^^^ help: try: `&text`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref
[INFO] [stdout]    = note: `#[warn(clippy::explicit_auto_deref)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: deref which would be done by auto-deref
[INFO] [stdout]    --> src/lib.rs:108:27
[INFO] [stdout]     |
[INFO] [stdout] 108 |     let ast = rnix::parse(&*text);
[INFO] [stdout]     |                           ^^^^^^ help: try: `&text`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/lib.rs:153:39
[INFO] [stdout]     |
[INFO] [stdout] 153 | fn convert_to_unix_line_endings(text: &str) -> (Cow<str>, LineEndings) {
[INFO] [stdout]     |                                       ^^^^      ^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                       |
[INFO] [stdout]     |                                       the lifetime is elided here
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 153 | fn convert_to_unix_line_endings(text: &str) -> (Cow<'_, str>, LineEndings) {
[INFO] [stdout]     |                                                     +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:169:5
[INFO] [stdout]     |
[INFO] [stdout] 169 | /     return if let Some(body) = prev.clone().and_then(With::cast).and_then(|w| w.body()) {
[INFO] [stdout] 170 | |         is_literal(body.kind())
[INFO] [stdout] 171 | |     } else {
[INFO] [stdout] 172 | |         prev.map(|it| is_literal(it.kind())) == Some(true)
[INFO] [stdout] 173 | |     };
[INFO] [stdout]     | |_____^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 169 ~     if let Some(body) = prev.clone().and_then(With::cast).and_then(|w| w.body()) {
[INFO] [stdout] 170 +         is_literal(body.kind())
[INFO] [stdout] 171 +     } else {
[INFO] [stdout] 172 +         prev.map(|it| is_literal(it.kind())) == Some(true)
[INFO] [stdout] 173 ~     }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this boolean expression can be simplified
[INFO] [stdout]    --> src/rules.rs:192:33
[INFO] [stdout]     |
[INFO] [stdout] 192 |                     _ => return true && before_token_has_newline(&it.into()),
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `before_token_has_newline(&it.into())`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool
[INFO] [stdout]     = note: `#[warn(clippy::nonminimal_bool)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:195:13
[INFO] [stdout]     |
[INFO] [stdout] 195 |             return false;
[INFO] [stdout]     |             ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 195 -             return false;
[INFO] [stdout] 195 +             false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: returning the result of a `let` binding from a block
[INFO] [stdout]    --> src/rules.rs:208:9
[INFO] [stdout]     |
[INFO] [stdout] 203 | /         let inline_attr = element
[INFO] [stdout] 204 | |             .parent()?
[INFO] [stdout] 205 | |             .descendants_with_tokens()
[INFO] [stdout] 206 | |             .find(|e| e.kind() == NODE_ATTR_SET)
[INFO] [stdout] 207 | |             .map(|t| before_token_has_newline(&t));
[INFO] [stdout]     | |___________________________________________________- unnecessary `let` binding
[INFO] [stdout] 208 |           inline_attr
[INFO] [stdout]     |           ^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
[INFO] [stdout]     = note: `#[warn(clippy::let_and_return)]` on by default
[INFO] [stdout] help: return the expression directly
[INFO] [stdout]     |
[INFO] [stdout] 203 ~         
[INFO] [stdout] 204 ~         element
[INFO] [stdout] 205 +             .parent()?
[INFO] [stdout] 206 +             .descendants_with_tokens()
[INFO] [stdout] 207 +             .find(|e| e.kind() == NODE_ATTR_SET)
[INFO] [stdout] 208 +             .map(|t| before_token_has_newline(&t))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:249:17
[INFO] [stdout]     |
[INFO] [stdout] 249 |                 return false;
[INFO] [stdout]     |                 ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 249 -                 return false;
[INFO] [stdout] 249 +                 false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:251:22
[INFO] [stdout]     |
[INFO] [stdout] 251 |             false => return has_newline(&parent),
[INFO] [stdout]     |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 251 -             false => return has_newline(&parent),
[INFO] [stdout] 251 +             false => has_newline(&parent),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: redundant closure
[INFO] [stdout]    --> src/rules.rs:241:59
[INFO] [stdout]     |
[INFO] [stdout] 241 | ...                   return node.as_node().map(|t| has_newline(t)).unwrap_or(false);
[INFO] [stdout]     |                                                 ^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `has_newline`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
[INFO] [stdout]     = note: `#[warn(clippy::redundant_closure)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: returning the result of a `let` binding from a block
[INFO] [stdout]    --> src/rules.rs:273:5
[INFO] [stdout]     |
[INFO] [stdout] 265 | /     let is_last_argument = match element.parent() {
[INFO] [stdout] 266 | |         None => false,
[INFO] [stdout] 267 | |         Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout] 268 | |             None => true,
[INFO] [stdout] 269 | |             _ => false,
[INFO] [stdout] 270 | |         },
[INFO] [stdout] 271 | |     };
[INFO] [stdout]     | |______- unnecessary `let` binding
[INFO] [stdout] 272 |
[INFO] [stdout] 273 |       is_last_argument
[INFO] [stdout]     |       ^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
[INFO] [stdout] help: return the expression directly
[INFO] [stdout]     |
[INFO] [stdout] 265 ~     
[INFO] [stdout] 266 |
[INFO] [stdout] 267 ~     match element.parent() {
[INFO] [stdout] 268 +         None => false,
[INFO] [stdout] 269 +         Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout] 270 +             None => true,
[INFO] [stdout] 271 +             _ => false,
[INFO] [stdout] 272 +         },
[INFO] [stdout] 273 +     }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: match expression looks like `matches!` macro
[INFO] [stdout]    --> src/rules.rs:267:21
[INFO] [stdout]     |
[INFO] [stdout] 267 |           Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout]     |  _____________________^
[INFO] [stdout] 268 | |             None => true,
[INFO] [stdout] 269 | |             _ => false,
[INFO] [stdout] 270 | |         },
[INFO] [stdout]     | |_________^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_like_matches_macro
[INFO] [stdout]     = note: `#[warn(clippy::match_like_matches_macro)]` on by default
[INFO] [stdout] help: use `matches!` directly
[INFO] [stdout]     |
[INFO] [stdout] 267 -         Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout] 268 -             None => true,
[INFO] [stdout] 269 -             _ => false,
[INFO] [stdout] 270 -         },
[INFO] [stdout] 267 +         Some(it) => matches!(next_sibling(&it.into()), None),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: redundant pattern matching, consider using `is_none()`
[INFO] [stdout]    --> src/rules.rs:267:21
[INFO] [stdout]     |
[INFO] [stdout] 267 |           Some(it) => match next_sibling(&it.into()) {
[INFO] [stdout]     |  _____________________^
[INFO] [stdout] 268 | |             None => true,
[INFO] [stdout] 269 | |             _ => false,
[INFO] [stdout] 270 | |         },
[INFO] [stdout]     | |_________^ help: try: `next_sibling(&it.into()).is_none()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching
[INFO] [stdout]     = note: `#[warn(clippy::redundant_pattern_matching)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:289:5
[INFO] [stdout]     |
[INFO] [stdout] 289 |     return false;
[INFO] [stdout]     |     ^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 289 -     return false;
[INFO] [stdout] 289 +     false
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: match expression looks like `matches!` macro
[INFO] [stdout]    --> src/rules.rs:298:31
[INFO] [stdout]     |
[INFO] [stdout] 298 |               .filter(|element| match element {
[INFO] [stdout]     |  _______________________________^
[INFO] [stdout] 299 | |                 NodeOrToken::Token(_) => true,
[INFO] [stdout] 300 | |                 _ => false,
[INFO] [stdout] 301 | |             })
[INFO] [stdout]     | |_____________^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_like_matches_macro
[INFO] [stdout] help: use `matches!` directly
[INFO] [stdout]     |
[INFO] [stdout] 298 -             .filter(|element| match element {
[INFO] [stdout] 299 -                 NodeOrToken::Token(_) => true,
[INFO] [stdout] 300 -                 _ => false,
[INFO] [stdout] 301 -             })
[INFO] [stdout] 298 +             .filter(|element| matches!(element, NodeOrToken::Token(_)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:332:17
[INFO] [stdout]     |
[INFO] [stdout] 332 |                 return v;
[INFO] [stdout]     |                 ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 332 -                 return v;
[INFO] [stdout] 332 +                 v
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded `return` statement
[INFO] [stdout]    --> src/rules.rs:335:17
[INFO] [stdout]     |
[INFO] [stdout] 335 |                 return v;
[INFO] [stdout]     |                 ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
[INFO] [stdout] help: remove `return`
[INFO] [stdout]     |
[INFO] [stdout] 335 -                 return v;
[INFO] [stdout] 335 +                 v
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: redundant closure
[INFO] [stdout]    --> src/rules.rs:360:61
[INFO] [stdout]     |
[INFO] [stdout] 360 |         next_non_whitespace_sibling(element)?.as_node().map(|e| has_newline(e))
[INFO] [stdout]     |                                                             ^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `has_newline`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: match expression looks like `matches!` macro
[INFO] [stdout]    --> src/rules.rs:684:31
[INFO] [stdout]     |
[INFO] [stdout] 684 |               .filter(|element| match element {
[INFO] [stdout]     |  _______________________________^
[INFO] [stdout] 685 | |                 NodeOrToken::Token(_) => true,
[INFO] [stdout] 686 | |                 _ => false,
[INFO] [stdout] 687 | |             })
[INFO] [stdout]     | |_____________^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_like_matches_macro
[INFO] [stdout] help: use `matches!` directly
[INFO] [stdout]     |
[INFO] [stdout] 684 -             .filter(|element| match element {
[INFO] [stdout] 685 -                 NodeOrToken::Token(_) => true,
[INFO] [stdout] 686 -                 _ => false,
[INFO] [stdout] 687 -             })
[INFO] [stdout] 684 +             .filter(|element| matches!(element, NodeOrToken::Token(_)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: called `.nth(0)` on a `std::iter::Iterator`, when `.next()` is equivalent
[INFO] [stdout]    --> src/rules.rs:727:24
[INFO] [stdout]     |
[INFO] [stdout] 727 |         let first_el = element.as_node()?.descendants().filter(|e| e.kind() != NODE_BIN_OP).nth(0);
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try calling `.next()` instead of `.nth(0)`: `element.as_node()?.descendants().filter(|e| e.kind() != NODE_BIN_OP).next()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#iter_nth_zero
[INFO] [stdout]     = note: `#[warn(clippy::iter_nth_zero)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: length comparison to zero
[INFO] [stdout]    --> src/rules.rs:926:21
[INFO] [stdout]     |
[INFO] [stdout] 926 |             assert!(res.len() > 0);
[INFO] [stdout]     |                     ^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!res.is_empty()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
[INFO] [stdout]     = note: `#[warn(clippy::len_zero)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: called `.as_ref().map(|it| it.as_str())` on an `Option` value
[INFO] [stdout]    --> src/rules.rs:931:24
[INFO] [stdout]     |
[INFO] [stdout] 931 |             let name = self.name.as_ref().map(|it| it.as_str()).unwrap_or("");
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using as_deref: `self.name.as_deref()`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#option_as_ref_deref
[INFO] [stdout]     = note: `#[warn(clippy::option_as_ref_deref)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: deref which would be done by auto-deref
[INFO] [stdout]   --> src/lib.rs:97:27
[INFO] [stdout]    |
[INFO] [stdout] 97 |     let ast = rnix::parse(&*text);
[INFO] [stdout]    |                           ^^^^^^ help: try: `&text`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref
[INFO] [stdout]    = note: `#[warn(clippy::explicit_auto_deref)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: deref which would be done by auto-deref
[INFO] [stdout]    --> src/lib.rs:108:27
[INFO] [stdout]     |
[INFO] [stdout] 108 |     let ast = rnix::parse(&*text);
[INFO] [stdout]     |                           ^^^^^^ help: try: `&text`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/lib.rs:153:39
[INFO] [stdout]     |
[INFO] [stdout] 153 | fn convert_to_unix_line_endings(text: &str) -> (Cow<str>, LineEndings) {
[INFO] [stdout]     |                                       ^^^^      ^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                       |
[INFO] [stdout]     |                                       the lifetime is elided here
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout] 153 | fn convert_to_unix_line_endings(text: &str) -> (Cow<'_, str>, LineEndings) {
[INFO] [stdout]     |                                                     +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: panic message is not a string literal
[INFO] [stdout]    --> src/rules.rs:813:29
[INFO] [stdout]     |
[INFO] [stdout] 813 |         .map_err(|e| panic!(e))
[INFO] [stdout]     |                             ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021
[INFO] [stdout]     = note: for more information, see <https://doc.rust-lang.org/edition-guide/rust-2021/panic-macro-consistency.html>
[INFO] [stdout]     = note: for more information, see <https://doc.rust-lang.org/edition-guide/rust-2021/panic-macro-consistency.html>
[INFO] [stdout]     = note: `#[warn(non_fmt_panics)]` (part of `#[warn(rust_2021_compatibility)]`) on by default
[INFO] [stdout] help: add a "{}" format string to `Display` the message
[INFO] [stdout]     |
[INFO] [stdout] 813 |         .map_err(|e| panic!("{}", e))
[INFO] [stdout]     |                             +++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
[INFO] [stdout]    --> src/main.rs:239:17
[INFO] [stdout]     |
[INFO] [stdout] 239 | /                 match reformat_dir_entry(entry, write_changes, &s) {
[INFO] [stdout] 240 | |                     Err(err) => eprintln!("error: {}", err),
[INFO] [stdout] 241 | |                     Ok(()) => {}
[INFO] [stdout] 242 | |                 }
[INFO] [stdout]     | |_________________^ help: try: `if let Err(err) = reformat_dir_entry(entry, write_changes, &s) { eprintln!("error: {}", err) }`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
[INFO] [stdout]     = note: `#[warn(clippy::single_match)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
[INFO] [stdout]    --> src/main.rs:239:17
[INFO] [stdout]     |
[INFO] [stdout] 239 | /                 match reformat_dir_entry(entry, write_changes, &s) {
[INFO] [stdout] 240 | |                     Err(err) => eprintln!("error: {}", err),
[INFO] [stdout] 241 | |                     Ok(()) => {}
[INFO] [stdout] 242 | |                 }
[INFO] [stdout]     | |_________________^ help: try: `if let Err(err) = reformat_dir_entry(entry, write_changes, &s) { eprintln!("error: {}", err) }`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
[INFO] [stdout]     = note: `#[warn(clippy::single_match)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 7.04s
[INFO] running `Command { std: "docker" "inspect" "825c5bd0081075c913e1a29b5fdcf005502a11d456d7f92e5d3a5dc143a0f7f3", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "825c5bd0081075c913e1a29b5fdcf005502a11d456d7f92e5d3a5dc143a0f7f3", kill_on_drop: false }`
[INFO] [stdout] 825c5bd0081075c913e1a29b5fdcf005502a11d456d7f92e5d3a5dc143a0f7f3