[INFO] fetching crate gte-w65c02s 0.17.0...
[INFO] linting gte-w65c02s-0.17.0 against nightly for clippy-nonminimal_bool-denied
[INFO] extracting crate gte-w65c02s 0.17.0 into /workspace/builds/worker-1-tc1/source
[INFO] started tweaking crates.io crate gte-w65c02s 0.17.0
[INFO] finished tweaking crates.io crate gte-w65c02s 0.17.0
[INFO] tweaked toml for crates.io crate gte-w65c02s 0.17.0 written to /workspace/builds/worker-1-tc1/source/Cargo.toml
[INFO] validating manifest of crates.io crate gte-w65c02s 0.17.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 gte-w65c02s 0.17.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] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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] 28fcad4a71c828af0f8edfaa583af2d03fac1a6a33ab5f82a1ceea55a4c0fafb
[INFO] running `Command { std: "docker" "start" "-a" "28fcad4a71c828af0f8edfaa583af2d03fac1a6a33ab5f82a1ceea55a4c0fafb", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "28fcad4a71c828af0f8edfaa583af2d03fac1a6a33ab5f82a1ceea55a4c0fafb", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "28fcad4a71c828af0f8edfaa583af2d03fac1a6a33ab5f82a1ceea55a4c0fafb", kill_on_drop: false }`
[INFO] [stdout] 28fcad4a71c828af0f8edfaa583af2d03fac1a6a33ab5f82a1ceea55a4c0fafb
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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] f0beb4e0d041ded5e41924f609572811803ee88e33237210665eab47de23882a
[INFO] running `Command { std: "docker" "start" "-a" "f0beb4e0d041ded5e41924f609572811803ee88e33237210665eab47de23882a", kill_on_drop: false }`
[INFO] [stderr]     Checking gte-w65c02s v0.17.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unneeded unit return type
[INFO] [stdout]   --> src/addressing_modes.rs:68:65
[INFO] [stdout]    |
[INFO] [stdout] 68 |     fn get_operand<S: System>(system: &mut S, cpu: &mut W65C02S) -> () {
[INFO] [stdout]    |                                                                 ^^^^^^ help: remove the `-> ()`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit
[INFO] [stdout]    = note: `#[warn(clippy::unused_unit)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded unit return type
[INFO] [stdout]   --> src/addressing_modes.rs:75:58
[INFO] [stdout]    |
[INFO] [stdout] 75 |     fn get_operand<S: System>(_: &mut S, _: &mut W65C02S) -> () {}
[INFO] [stdout]    |                                                          ^^^^^^ help: remove the `-> ()`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/addressing_modes.rs:329:13
[INFO] [stdout]     |
[INFO] [stdout] 329 |             cpu.irq_pending = cpu.irq_pending | old_irq_pending;
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `cpu.irq_pending |= old_irq_pending`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout]     = note: `#[warn(clippy::assign_op_pattern)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/addressing_modes.rs:367:13
[INFO] [stdout]     |
[INFO] [stdout] 367 |             cpu.irq_pending = cpu.irq_pending | old_irq_pending;
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `cpu.irq_pending |= old_irq_pending`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/instructions.rs:118:33
[INFO] [stdout]     |
[INFO] [stdout] 118 |         if data & self.a == 0 { self.p = self.p | P_Z }
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.p |= P_Z`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/instructions.rs:119:16
[INFO] [stdout]     |
[INFO] [stdout] 119 |         else { self.p = self.p & !P_Z }
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.p &= !P_Z`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/instructions.rs:126:33
[INFO] [stdout]     |
[INFO] [stdout] 126 |         if data & self.a == 0 { self.p = self.p | P_Z }
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.p |= P_Z`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/instructions.rs:127:16
[INFO] [stdout]     |
[INFO] [stdout] 127 |         else { self.p = self.p & !P_Z }
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.p &= !P_Z`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual `Range::contains` implementation
[INFO] [stdout]    --> src/instructions.rs:426:16
[INFO] [stdout]     |
[INFO] [stdout] 426 |             if val >= 0x80 && val < 0xFF80 { self.p |= P_V }
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use: `(0x80..0xFF80).contains(&val)`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains
[INFO] [stdout]     = note: `#[warn(clippy::manual_range_contains)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: you should consider adding a `Default` implementation for `W65C02S`
[INFO] [stdout]    --> src/lib.rs:269:5
[INFO] [stdout]     |
[INFO] [stdout] 269 | /     pub fn new() -> W65C02S {
[INFO] [stdout] 270 | |         W65C02S {
[INFO] [stdout] 271 | |             state: State::HasBeenReset,
[INFO] [stdout] 272 | |             pc: 0xFFFF,
[INFO] [stdout] ...   |
[INFO] [stdout] 281 | |     }
[INFO] [stdout]     | |_____^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default
[INFO] [stdout]     = note: `#[warn(clippy::new_without_default)]` on by default
[INFO] [stdout] help: try adding this
[INFO] [stdout]     |
[INFO] [stdout] 265 + impl Default for W65C02S {
[INFO] [stdout] 266 +     fn default() -> Self {
[INFO] [stdout] 267 +         Self::new()
[INFO] [stdout] 268 +     }
[INFO] [stdout] 269 + }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this boolean expression can be simplified
[INFO] [stdout]    --> src/lib.rs:390:25
[INFO] [stdout]     |
[INFO] [stdout] 390 |         self.nmi_edge = self.nmi_edge || (!self.nmi_edge && nmi);
[INFO] [stdout]     |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[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] help: try
[INFO] [stdout]     |
[INFO] [stdout] 390 -         self.nmi_edge = self.nmi_edge || (!self.nmi_edge && nmi);
[INFO] [stdout] 390 +         self.nmi_edge = !(!self.nmi_edge && !nmi);
[INFO] [stdout]     |
[INFO] [stdout] 390 -         self.nmi_edge = self.nmi_edge || (!self.nmi_edge && nmi);
[INFO] [stdout] 390 +         self.nmi_edge = self.nmi_edge || nmi;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded unit return type
[INFO] [stdout]   --> src/addressing_modes.rs:68:65
[INFO] [stdout]    |
[INFO] [stdout] 68 |     fn get_operand<S: System>(system: &mut S, cpu: &mut W65C02S) -> () {
[INFO] [stdout]    |                                                                 ^^^^^^ help: remove the `-> ()`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit
[INFO] [stdout]    = note: `#[warn(clippy::unused_unit)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unneeded unit return type
[INFO] [stdout]   --> src/addressing_modes.rs:75:58
[INFO] [stdout]    |
[INFO] [stdout] 75 |     fn get_operand<S: System>(_: &mut S, _: &mut W65C02S) -> () {}
[INFO] [stdout]    |                                                          ^^^^^^ help: remove the `-> ()`
[INFO] [stdout]    |
[INFO] [stdout]    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_unit
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/addressing_modes.rs:329:13
[INFO] [stdout]     |
[INFO] [stdout] 329 |             cpu.irq_pending = cpu.irq_pending | old_irq_pending;
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `cpu.irq_pending |= old_irq_pending`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout]     = note: `#[warn(clippy::assign_op_pattern)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/addressing_modes.rs:367:13
[INFO] [stdout]     |
[INFO] [stdout] 367 |             cpu.irq_pending = cpu.irq_pending | old_irq_pending;
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `cpu.irq_pending |= old_irq_pending`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/instructions.rs:118:33
[INFO] [stdout]     |
[INFO] [stdout] 118 |         if data & self.a == 0 { self.p = self.p | P_Z }
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.p |= P_Z`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/instructions.rs:119:16
[INFO] [stdout]     |
[INFO] [stdout] 119 |         else { self.p = self.p & !P_Z }
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.p &= !P_Z`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/instructions.rs:126:33
[INFO] [stdout]     |
[INFO] [stdout] 126 |         if data & self.a == 0 { self.p = self.p | P_Z }
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.p |= P_Z`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual implementation of an assign operation
[INFO] [stdout]    --> src/instructions.rs:127:16
[INFO] [stdout]     |
[INFO] [stdout] 127 |         else { self.p = self.p & !P_Z }
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `self.p &= !P_Z`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: manual `Range::contains` implementation
[INFO] [stdout]    --> src/instructions.rs:426:16
[INFO] [stdout]     |
[INFO] [stdout] 426 |             if val >= 0x80 && val < 0xFF80 { self.p |= P_V }
[INFO] [stdout]     |                ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use: `(0x80..0xFF80).contains(&val)`
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains
[INFO] [stdout]     = note: `#[warn(clippy::manual_range_contains)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: you should consider adding a `Default` implementation for `W65C02S`
[INFO] [stdout]    --> src/lib.rs:269:5
[INFO] [stdout]     |
[INFO] [stdout] 269 | /     pub fn new() -> W65C02S {
[INFO] [stdout] 270 | |         W65C02S {
[INFO] [stdout] 271 | |             state: State::HasBeenReset,
[INFO] [stdout] 272 | |             pc: 0xFFFF,
[INFO] [stdout] ...   |
[INFO] [stdout] 281 | |     }
[INFO] [stdout]     | |_____^
[INFO] [stdout]     |
[INFO] [stdout]     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default
[INFO] [stdout]     = note: `#[warn(clippy::new_without_default)]` on by default
[INFO] [stdout] help: try adding this
[INFO] [stdout]     |
[INFO] [stdout] 265 + impl Default for W65C02S {
[INFO] [stdout] 266 +     fn default() -> Self {
[INFO] [stdout] 267 +         Self::new()
[INFO] [stdout] 268 +     }
[INFO] [stdout] 269 + }
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: this boolean expression can be simplified
[INFO] [stdout]    --> src/lib.rs:390:25
[INFO] [stdout]     |
[INFO] [stdout] 390 |         self.nmi_edge = self.nmi_edge || (!self.nmi_edge && nmi);
[INFO] [stdout]     |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[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] help: try
[INFO] [stdout]     |
[INFO] [stdout] 390 -         self.nmi_edge = self.nmi_edge || (!self.nmi_edge && nmi);
[INFO] [stdout] 390 +         self.nmi_edge = !(!self.nmi_edge && !nmi);
[INFO] [stdout]     |
[INFO] [stdout] 390 -         self.nmi_edge = self.nmi_edge || (!self.nmi_edge && nmi);
[INFO] [stdout] 390 +         self.nmi_edge = self.nmi_edge || nmi;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.53s
[INFO] running `Command { std: "docker" "inspect" "f0beb4e0d041ded5e41924f609572811803ee88e33237210665eab47de23882a", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "f0beb4e0d041ded5e41924f609572811803ee88e33237210665eab47de23882a", kill_on_drop: false }`
[INFO] [stdout] f0beb4e0d041ded5e41924f609572811803ee88e33237210665eab47de23882a