[INFO] cloning repository https://github.com/ZacJoffe/gameboy-emulator [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/ZacJoffe/gameboy-emulator" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FZacJoffe%2Fgameboy-emulator", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FZacJoffe%2Fgameboy-emulator'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 367a93884a4ba543759d7b0a20824e66e8e61af5 [INFO] checking ZacJoffe/gameboy-emulator/367a93884a4ba543759d7b0a20824e66e8e61af5 against master#f609b7e0586f81fefb3523e3e17adf779ac416be for pr-129466-2 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FZacJoffe%2Fgameboy-emulator" "/workspace/builds/worker-5-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc1/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/ZacJoffe/gameboy-emulator on toolchain f609b7e0586f81fefb3523e3e17adf779ac416be [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+f609b7e0586f81fefb3523e3e17adf779ac416be" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/ZacJoffe/gameboy-emulator [INFO] finished tweaking git repo https://github.com/ZacJoffe/gameboy-emulator [INFO] tweaked toml for git repo https://github.com/ZacJoffe/gameboy-emulator written to /workspace/builds/worker-5-tc1/source/Cargo.toml [INFO] crate git repo https://github.com/ZacJoffe/gameboy-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" "+f609b7e0586f81fefb3523e3e17adf779ac416be" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-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:6252d7ea7fb8caaf7af6d800c5fb250a9cd862b9a7f9508afb3c54fa7fe1102e" "/opt/rustwide/cargo-home/bin/cargo" "+f609b7e0586f81fefb3523e3e17adf779ac416be" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 87ca42c229f113bbcc6848764734a3c9e8c048c0683e1868d38d16d03062a291 [INFO] running `Command { std: "docker" "start" "-a" "87ca42c229f113bbcc6848764734a3c9e8c048c0683e1868d38d16d03062a291", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "87ca42c229f113bbcc6848764734a3c9e8c048c0683e1868d38d16d03062a291", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "87ca42c229f113bbcc6848764734a3c9e8c048c0683e1868d38d16d03062a291", kill_on_drop: false }` [INFO] [stdout] 87ca42c229f113bbcc6848764734a3c9e8c048c0683e1868d38d16d03062a291 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-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:6252d7ea7fb8caaf7af6d800c5fb250a9cd862b9a7f9508afb3c54fa7fe1102e" "/opt/rustwide/cargo-home/bin/cargo" "+f609b7e0586f81fefb3523e3e17adf779ac416be" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] a5ff02b7a0888ad981ac9eeeea975bba249851778157e91679c80bf4015c373b [INFO] running `Command { std: "docker" "start" "-a" "a5ff02b7a0888ad981ac9eeeea975bba249851778157e91679c80bf4015c373b", kill_on_drop: false }` [INFO] [stderr] Checking gameboy-emulator v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] error[E0616]: field `vram` of struct `GPU` is private [INFO] [stdout] --> src/memory_bus.rs:60:47 [INFO] [stdout] | [INFO] [stdout] 60 | VRAM_START..=VRAM_END => self.gpu.vram[(address - VRAM_START) as usize], [INFO] [stdout] | ^^^^ private field [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0616]: field `vram` of struct `GPU` is private [INFO] [stdout] --> src/memory_bus.rs:64:45 [INFO] [stdout] | [INFO] [stdout] 64 | OAM_START..=OAM_END => self.gpu.vram[(address - OAM_START) as usize], [INFO] [stdout] | ^^^^ private field [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0616]: field `vram` of struct `GPU` is private [INFO] [stdout] --> src/memory_bus.rs:60:47 [INFO] [stdout] | [INFO] [stdout] 60 | VRAM_START..=VRAM_END => self.gpu.vram[(address - VRAM_START) as usize], [INFO] [stdout] | ^^^^ private field [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0616]: field `vram` of struct `GPU` is private [INFO] [stdout] --> src/memory_bus.rs:64:45 [INFO] [stdout] | [INFO] [stdout] 64 | OAM_START..=OAM_END => self.gpu.vram[(address - OAM_START) as usize], [INFO] [stdout] | ^^^^ private field [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `result` [INFO] [stdout] --> src/cpu.rs:83:21 [INFO] [stdout] | [INFO] [stdout] 83 | let result = self.sp.wrapping_add(value); [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_result` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `result` [INFO] [stdout] --> src/cpu.rs:83:21 [INFO] [stdout] | [INFO] [stdout] 83 | let result = self.sp.wrapping_add(value); [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_result` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0382]: use of moved value: `target` [INFO] [stdout] --> src/cpu.rs:140:23 [INFO] [stdout] | [INFO] [stdout] 138 | Instruction::INC(target) => { [INFO] [stdout] | ------ move occurs because `target` has type `instructions::IncDecTarget`, which does not implement the `Copy` trait [INFO] [stdout] 139 | self.inc(target); [INFO] [stdout] | ------ value moved here [INFO] [stdout] 140 | match target { [INFO] [stdout] | ^^^^^^ value used here after move [INFO] [stdout] | [INFO] [stdout] note: consider changing this parameter type in method `inc` to borrow instead if owning the value isn't necessary [INFO] [stdout] --> src/cpu.rs:970:31 [INFO] [stdout] | [INFO] [stdout] 970 | fn inc(&mut self, target: IncDecTarget) { [INFO] [stdout] | --- in this method ^^^^^^^^^^^^ this parameter takes ownership of the value [INFO] [stdout] note: if `instructions::IncDecTarget` implemented `Clone`, you could clone the value [INFO] [stdout] --> src/instructions.rs:81:1 [INFO] [stdout] | [INFO] [stdout] 81 | pub enum IncDecTarget { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ consider implementing `Clone` for this type [INFO] [stdout] | [INFO] [stdout] ::: src/cpu.rs:139:26 [INFO] [stdout] | [INFO] [stdout] 139 | self.inc(target); [INFO] [stdout] | ------ you could clone this value [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0382]: use of moved value: `target` [INFO] [stdout] --> src/cpu.rs:140:23 [INFO] [stdout] | [INFO] [stdout] 138 | Instruction::INC(target) => { [INFO] [stdout] | ------ move occurs because `target` has type `instructions::IncDecTarget`, which does not implement the `Copy` trait [INFO] [stdout] 139 | self.inc(target); [INFO] [stdout] | ------ value moved here [INFO] [stdout] 140 | match target { [INFO] [stdout] | ^^^^^^ value used here after move [INFO] [stdout] | [INFO] [stdout] note: consider changing this parameter type in method `inc` to borrow instead if owning the value isn't necessary [INFO] [stdout] --> src/cpu.rs:970:31 [INFO] [stdout] | [INFO] [stdout] 970 | fn inc(&mut self, target: IncDecTarget) { [INFO] [stdout] | --- in this method ^^^^^^^^^^^^ this parameter takes ownership of the value [INFO] [stdout] note: if `instructions::IncDecTarget` implemented `Clone`, you could clone the value [INFO] [stdout] --> src/instructions.rs:81:1 [INFO] [stdout] | [INFO] [stdout] 81 | pub enum IncDecTarget { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ consider implementing `Clone` for this type [INFO] [stdout] | [INFO] [stdout] ::: src/cpu.rs:139:26 [INFO] [stdout] | [INFO] [stdout] 139 | self.inc(target); [INFO] [stdout] | ------ you could clone this value [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0382]: use of moved value: `target` [INFO] [stdout] --> src/cpu.rs:148:23 [INFO] [stdout] | [INFO] [stdout] 146 | Instruction::DEC(target) => { [INFO] [stdout] | ------ move occurs because `target` has type `instructions::IncDecTarget`, which does not implement the `Copy` trait [INFO] [stdout] 147 | self.dec(target); [INFO] [stdout] | ------ value moved here [INFO] [stdout] 148 | match target { [INFO] [stdout] | ^^^^^^ value used here after move [INFO] [stdout] | [INFO] [stdout] note: consider changing this parameter type in method `dec` to borrow instead if owning the value isn't necessary [INFO] [stdout] --> src/cpu.rs:1135:31 [INFO] [stdout] | [INFO] [stdout] 1135 | fn dec(&mut self, target: IncDecTarget) { [INFO] [stdout] | --- in this method ^^^^^^^^^^^^ this parameter takes ownership of the value [INFO] [stdout] note: if `instructions::IncDecTarget` implemented `Clone`, you could clone the value [INFO] [stdout] --> src/instructions.rs:81:1 [INFO] [stdout] | [INFO] [stdout] 81 | pub enum IncDecTarget { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ consider implementing `Clone` for this type [INFO] [stdout] | [INFO] [stdout] ::: src/cpu.rs:147:26 [INFO] [stdout] | [INFO] [stdout] 147 | self.dec(target); [INFO] [stdout] | ------ you could clone this value [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0382]: use of moved value: `target` [INFO] [stdout] --> src/cpu.rs:148:23 [INFO] [stdout] | [INFO] [stdout] 146 | Instruction::DEC(target) => { [INFO] [stdout] | ------ move occurs because `target` has type `instructions::IncDecTarget`, which does not implement the `Copy` trait [INFO] [stdout] 147 | self.dec(target); [INFO] [stdout] | ------ value moved here [INFO] [stdout] 148 | match target { [INFO] [stdout] | ^^^^^^ value used here after move [INFO] [stdout] | [INFO] [stdout] note: consider changing this parameter type in method `dec` to borrow instead if owning the value isn't necessary [INFO] [stdout] --> src/cpu.rs:1135:31 [INFO] [stdout] | [INFO] [stdout] 1135 | fn dec(&mut self, target: IncDecTarget) { [INFO] [stdout] | --- in this method ^^^^^^^^^^^^ this parameter takes ownership of the value [INFO] [stdout] note: if `instructions::IncDecTarget` implemented `Clone`, you could clone the value [INFO] [stdout] --> src/instructions.rs:81:1 [INFO] [stdout] | [INFO] [stdout] 81 | pub enum IncDecTarget { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ consider implementing `Clone` for this type [INFO] [stdout] | [INFO] [stdout] ::: src/cpu.rs:147:26 [INFO] [stdout] | [INFO] [stdout] 147 | self.dec(target); [INFO] [stdout] | ------ you could clone this value [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `address` [INFO] [stdout] --> src/memory_bus.rs:91:32 [INFO] [stdout] | [INFO] [stdout] 91 | fn read_io_register(&self, address: u16) -> u8 { [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_address` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `address` [INFO] [stdout] --> src/memory_bus.rs:96:37 [INFO] [stdout] | [INFO] [stdout] 96 | fn write_io_register(&mut self, address: u16, new_byte: u8) { [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_address` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `new_byte` [INFO] [stdout] --> src/memory_bus.rs:96:51 [INFO] [stdout] | [INFO] [stdout] 96 | fn write_io_register(&mut self, address: u16, new_byte: u8) { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_new_byte` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0004]: non-exhaustive patterns: `4_u8..=u8::MAX` not covered [INFO] [stdout] --> src/gpu.rs:64:15 [INFO] [stdout] | [INFO] [stdout] 64 | match self.STAT & 0x3 { [INFO] [stdout] | ^^^^^^^^^^^^^^^ pattern `4_u8..=u8::MAX` not covered [INFO] [stdout] | [INFO] [stdout] = note: the matched value is of type `u8` [INFO] [stdout] help: ensure that all possible cases are being handled by adding a match arm with a wildcard pattern or an explicit pattern as shown [INFO] [stdout] | [INFO] [stdout] 68 ~ 3 => Mode::Transfer, [INFO] [stdout] 69 + 4_u8..=u8::MAX => todo!() [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `cpu` [INFO] [stdout] --> src/main.rs:41:13 [INFO] [stdout] | [INFO] [stdout] 41 | let mut cpu = CPU::new(bios_buffer, game_buffer); [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_cpu` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `address` [INFO] [stdout] --> src/memory_bus.rs:91:32 [INFO] [stdout] | [INFO] [stdout] 91 | fn read_io_register(&self, address: u16) -> u8 { [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_address` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `address` [INFO] [stdout] --> src/memory_bus.rs:96:37 [INFO] [stdout] | [INFO] [stdout] 96 | fn write_io_register(&mut self, address: u16, new_byte: u8) { [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_address` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `new_byte` [INFO] [stdout] --> src/memory_bus.rs:96:51 [INFO] [stdout] | [INFO] [stdout] 96 | fn write_io_register(&mut self, address: u16, new_byte: u8) { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_new_byte` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:41:9 [INFO] [stdout] | [INFO] [stdout] 41 | let mut cpu = CPU::new(bios_buffer, game_buffer); [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] error[E0004]: non-exhaustive patterns: `4_u8..=u8::MAX` not covered [INFO] [stdout] --> src/gpu.rs:64:15 [INFO] [stdout] | [INFO] [stdout] 64 | match self.STAT & 0x3 { [INFO] [stdout] | ^^^^^^^^^^^^^^^ pattern `4_u8..=u8::MAX` not covered [INFO] [stdout] | [INFO] [stdout] = note: the matched value is of type `u8` [INFO] [stdout] help: ensure that all possible cases are being handled by adding a match arm with a wildcard pattern or an explicit pattern as shown [INFO] [stdout] | [INFO] [stdout] 68 ~ 3 => Mode::Transfer, [INFO] [stdout] 69 + 4_u8..=u8::MAX => todo!() [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `cpu` [INFO] [stdout] --> src/main.rs:41:13 [INFO] [stdout] | [INFO] [stdout] 41 | let mut cpu = CPU::new(bios_buffer, game_buffer); [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_cpu` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: aborting due to 5 previous errors; 6 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] Some errors have detailed explanations: E0004, E0382, E0616. [INFO] [stdout] [INFO] [stdout] For more information about an error, try `rustc --explain E0004`. [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:41:9 [INFO] [stdout] | [INFO] [stdout] 41 | let mut cpu = CPU::new(bios_buffer, game_buffer); [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] error: aborting due to 5 previous errors; 6 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] Some errors have detailed explanations: E0004, E0382, E0616. [INFO] [stdout] [INFO] [stdout] For more information about an error, try `rustc --explain E0004`. [INFO] [stdout] [INFO] [stderr] error: could not compile `gameboy-emulator` (bin "gameboy-emulator") due to 6 previous errors; 6 warnings emitted [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stderr] error: could not compile `gameboy-emulator` (bin "gameboy-emulator" test) due to 6 previous errors; 6 warnings emitted [INFO] running `Command { std: "docker" "inspect" "a5ff02b7a0888ad981ac9eeeea975bba249851778157e91679c80bf4015c373b", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "a5ff02b7a0888ad981ac9eeeea975bba249851778157e91679c80bf4015c373b", kill_on_drop: false }` [INFO] [stdout] a5ff02b7a0888ad981ac9eeeea975bba249851778157e91679c80bf4015c373b