[INFO] cloning repository https://github.com/kam1tsur3/rust-nes-emulator [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/kam1tsur3/rust-nes-emulator" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkam1tsur3%2Frust-nes-emulator", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkam1tsur3%2Frust-nes-emulator'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 00ceb2d721ab02d10e1031fdd073508b77724ab2 [INFO] building kam1tsur3/rust-nes-emulator against master#11663cd3bfefef7d34e8f0892c250bf698049392+rustflags=-Dtail-expr-drop-order for pr-134523 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkam1tsur3%2Frust-nes-emulator" "/workspace/builds/worker-7-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc1/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/kam1tsur3/rust-nes-emulator on toolchain 11663cd3bfefef7d34e8f0892c250bf698049392 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+11663cd3bfefef7d34e8f0892c250bf698049392" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/kam1tsur3/rust-nes-emulator [INFO] finished tweaking git repo https://github.com/kam1tsur3/rust-nes-emulator [INFO] tweaked toml for git repo https://github.com/kam1tsur3/rust-nes-emulator written to /workspace/builds/worker-7-tc1/source/Cargo.toml [INFO] crate git repo https://github.com/kam1tsur3/rust-nes-emulator 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" "+11663cd3bfefef7d34e8f0892c250bf698049392" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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:f3a9d4ad9d972b27faf3965f35b62e55ba32bbce8f20bc8fe909558a86702fde" "/opt/rustwide/cargo-home/bin/cargo" "+11663cd3bfefef7d34e8f0892c250bf698049392" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] b468a85e2208e329d7df6d9ef3f920b1d92fcf8e6e64525eabd6661d19027463 [INFO] running `Command { std: "docker" "start" "-a" "b468a85e2208e329d7df6d9ef3f920b1d92fcf8e6e64525eabd6661d19027463", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "b468a85e2208e329d7df6d9ef3f920b1d92fcf8e6e64525eabd6661d19027463", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "b468a85e2208e329d7df6d9ef3f920b1d92fcf8e6e64525eabd6661d19027463", kill_on_drop: false }` [INFO] [stdout] b468a85e2208e329d7df6d9ef3f920b1d92fcf8e6e64525eabd6661d19027463 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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 -Dtail-expr-drop-order" "-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:f3a9d4ad9d972b27faf3965f35b62e55ba32bbce8f20bc8fe909558a86702fde" "/opt/rustwide/cargo-home/bin/cargo" "+11663cd3bfefef7d34e8f0892c250bf698049392" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 4a4ded069893abeaeb6fad8fb1c64354750e1a8416171571cdb579d8fa4786d4 [INFO] running `Command { std: "docker" "start" "-a" "4a4ded069893abeaeb6fad8fb1c64354750e1a8416171571cdb579d8fa4786d4", kill_on_drop: false }` [INFO] [stderr] Compiling libc v0.2.139 [INFO] [stderr] Compiling version-compare v0.0.10 [INFO] [stderr] Compiling getrandom v0.1.16 [INFO] [stderr] Compiling cfg-if v0.1.10 [INFO] [stderr] Compiling cfg-if v1.0.0 [INFO] [stderr] Compiling ppv-lite86 v0.2.17 [INFO] [stderr] Compiling sdl2 v0.34.5 [INFO] [stderr] Compiling bitflags v1.3.2 [INFO] [stderr] Compiling lazy_static v1.4.0 [INFO] [stderr] Compiling sdl2-sys v0.34.5 [INFO] [stderr] Compiling rand_core v0.5.1 [INFO] [stderr] Compiling rand_chacha v0.2.2 [INFO] [stderr] Compiling rand v0.7.3 [INFO] [stderr] Compiling nes-emulator-rs v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `std::ops::Add` [INFO] [stdout] --> src/cpu.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::ops::Add; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `FOUR_SCREEN` should have an upper camel case name [INFO] [stdout] --> src/cartridge.rs:9:3 [INFO] [stdout] | [INFO] [stdout] 9 | FOUR_SCREEN, [INFO] [stdout] | ^^^^^^^^^^^ help: convert the identifier to upper camel case: `FourScreen` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/cpu.rs:530:15 [INFO] [stdout] | [INFO] [stdout] 530 | let data = self.mem_read(addr); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `address_lo` [INFO] [stdout] --> src/trace.rs:65:11 [INFO] [stdout] | [INFO] [stdout] 65 | let address_lo = cpu.mem_read(cpu.program_counter + 1); [INFO] [stdout] | ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_address_lo` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `address_hi` [INFO] [stdout] --> src/trace.rs:66:11 [INFO] [stdout] | [INFO] [stdout] 66 | let address_hi = cpu.mem_read(cpu.program_counter + 2); [INFO] [stdout] | ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_address_hi` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `event_pump` [INFO] [stdout] --> src/main.rs:94:11 [INFO] [stdout] | [INFO] [stdout] 94 | let mut event_pump = sdl_context.event_pump().unwrap(); [INFO] [stdout] | ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_event_pump` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `texture` [INFO] [stdout] --> src/main.rs:98:11 [INFO] [stdout] | [INFO] [stdout] 98 | let mut texture = creator [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_texture` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:94:7 [INFO] [stdout] | [INFO] [stdout] 94 | let mut event_pump = sdl_context.event_pump().unwrap(); [INFO] [stdout] | ----^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:98:7 [INFO] [stdout] | [INFO] [stdout] 98 | let mut texture = creator [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `color` is never used [INFO] [stdout] --> src/main.rs:28:4 [INFO] [stdout] | [INFO] [stdout] 28 | fn color(byte: u8) -> Color { [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `read_screen_state` is never used [INFO] [stdout] --> src/main.rs:42:4 [INFO] [stdout] | [INFO] [stdout] 42 | fn read_screen_state(cpu: &mut CPU, frame: &mut [u8; 32 * 3 * 32]) -> bool { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handle_user_input` is never used [INFO] [stdout] --> src/main.rs:59:4 [INFO] [stdout] | [INFO] [stdout] 59 | fn handle_user_input(cpu: &mut CPU, event_pump: &mut EventPump) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `PPU_REGISTERS` is never used [INFO] [stdout] --> src/bus.rs:14:7 [INFO] [stdout] | [INFO] [stdout] 14 | const PPU_REGISTERS: u16 = 0x2000; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/trace.rs:17:5 [INFO] [stdout] | [INFO] [stdout] 17 | write!(byte_codes,"{:02x} ", cpu.mem_read(cpu.program_counter+(i as u16))); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] = note: this warning originates in the macro `write` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/trace.rs:22:3 [INFO] [stdout] | [INFO] [stdout] 22 | write!(asm_str, "{} ", opcode.mnemonic); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: this warning originates in the macro `write` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 5.38s [INFO] running `Command { std: "docker" "inspect" "4a4ded069893abeaeb6fad8fb1c64354750e1a8416171571cdb579d8fa4786d4", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "4a4ded069893abeaeb6fad8fb1c64354750e1a8416171571cdb579d8fa4786d4", kill_on_drop: false }` [INFO] [stdout] 4a4ded069893abeaeb6fad8fb1c64354750e1a8416171571cdb579d8fa4786d4 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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 -Dtail-expr-drop-order" "-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:f3a9d4ad9d972b27faf3965f35b62e55ba32bbce8f20bc8fe909558a86702fde" "/opt/rustwide/cargo-home/bin/cargo" "+11663cd3bfefef7d34e8f0892c250bf698049392" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 80cb462460bcc373062dce32555c8b0f3fb6e8927ae9f08769a201f800ee9e22 [INFO] running `Command { std: "docker" "start" "-a" "80cb462460bcc373062dce32555c8b0f3fb6e8927ae9f08769a201f800ee9e22", kill_on_drop: false }` [INFO] [stderr] Compiling nes-emulator-rs v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `std::ops::Add` [INFO] [stdout] --> src/cpu.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::ops::Add; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `FOUR_SCREEN` should have an upper camel case name [INFO] [stdout] --> src/cartridge.rs:9:3 [INFO] [stdout] | [INFO] [stdout] 9 | FOUR_SCREEN, [INFO] [stdout] | ^^^^^^^^^^^ help: convert the identifier to upper camel case: `FourScreen` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/cpu.rs:530:15 [INFO] [stdout] | [INFO] [stdout] 530 | let data = self.mem_read(addr); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `address_lo` [INFO] [stdout] --> src/trace.rs:65:11 [INFO] [stdout] | [INFO] [stdout] 65 | let address_lo = cpu.mem_read(cpu.program_counter + 1); [INFO] [stdout] | ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_address_lo` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `address_hi` [INFO] [stdout] --> src/trace.rs:66:11 [INFO] [stdout] | [INFO] [stdout] 66 | let address_hi = cpu.mem_read(cpu.program_counter + 2); [INFO] [stdout] | ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_address_hi` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `event_pump` [INFO] [stdout] --> src/main.rs:94:11 [INFO] [stdout] | [INFO] [stdout] 94 | let mut event_pump = sdl_context.event_pump().unwrap(); [INFO] [stdout] | ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_event_pump` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `texture` [INFO] [stdout] --> src/main.rs:98:11 [INFO] [stdout] | [INFO] [stdout] 98 | let mut texture = creator [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_texture` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:94:7 [INFO] [stdout] | [INFO] [stdout] 94 | let mut event_pump = sdl_context.event_pump().unwrap(); [INFO] [stdout] | ----^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:98:7 [INFO] [stdout] | [INFO] [stdout] 98 | let mut texture = creator [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `color` is never used [INFO] [stdout] --> src/main.rs:28:4 [INFO] [stdout] | [INFO] [stdout] 28 | fn color(byte: u8) -> Color { [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `read_screen_state` is never used [INFO] [stdout] --> src/main.rs:42:4 [INFO] [stdout] | [INFO] [stdout] 42 | fn read_screen_state(cpu: &mut CPU, frame: &mut [u8; 32 * 3 * 32]) -> bool { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handle_user_input` is never used [INFO] [stdout] --> src/main.rs:59:4 [INFO] [stdout] | [INFO] [stdout] 59 | fn handle_user_input(cpu: &mut CPU, event_pump: &mut EventPump) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `PPU_REGISTERS` is never used [INFO] [stdout] --> src/bus.rs:14:7 [INFO] [stdout] | [INFO] [stdout] 14 | const PPU_REGISTERS: u16 = 0x2000; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/trace.rs:17:5 [INFO] [stdout] | [INFO] [stdout] 17 | write!(byte_codes,"{:02x} ", cpu.mem_read(cpu.program_counter+(i as u16))); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] = note: this warning originates in the macro `write` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/trace.rs:22:3 [INFO] [stdout] | [INFO] [stdout] 22 | write!(asm_str, "{} ", opcode.mnemonic); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: this warning originates in the macro `write` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `test` profile [unoptimized + debuginfo] target(s) in 0.87s [INFO] running `Command { std: "docker" "inspect" "80cb462460bcc373062dce32555c8b0f3fb6e8927ae9f08769a201f800ee9e22", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "80cb462460bcc373062dce32555c8b0f3fb6e8927ae9f08769a201f800ee9e22", kill_on_drop: false }` [INFO] [stdout] 80cb462460bcc373062dce32555c8b0f3fb6e8927ae9f08769a201f800ee9e22