[INFO] cloning repository https://github.com/d2ci8xc5/chip8vm [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/d2ci8xc5/chip8vm" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fd2ci8xc5%2Fchip8vm", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fd2ci8xc5%2Fchip8vm'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] ab98e3f549bdef2e6383a2b9d6e77c71642765c1 [INFO] checking d2ci8xc5/chip8vm against master#2e6fc425411280c658d852813de01c5157255088 for pr-128013 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fd2ci8xc5%2Fchip8vm" "/workspace/builds/worker-4-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-4-tc1/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/d2ci8xc5/chip8vm on toolchain 2e6fc425411280c658d852813de01c5157255088 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+2e6fc425411280c658d852813de01c5157255088" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/d2ci8xc5/chip8vm [INFO] finished tweaking git repo https://github.com/d2ci8xc5/chip8vm [INFO] tweaked toml for git repo https://github.com/d2ci8xc5/chip8vm written to /workspace/builds/worker-4-tc1/source/Cargo.toml [INFO] crate git repo https://github.com/d2ci8xc5/chip8vm 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" "+2e6fc425411280c658d852813de01c5157255088" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Updating git repository `https://github.com/AngryLawyer/rust-sdl2` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:dff56e7819e73ed36160586b3445e93eb0be776c16704aeeded9c3fb668b2384" "/opt/rustwide/cargo-home/bin/cargo" "+2e6fc425411280c658d852813de01c5157255088" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 427fe918363680bc872f4f88a2cee48f5318b568601fdf3d6890bcb2e5487289 [INFO] running `Command { std: "docker" "start" "-a" "427fe918363680bc872f4f88a2cee48f5318b568601fdf3d6890bcb2e5487289", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "427fe918363680bc872f4f88a2cee48f5318b568601fdf3d6890bcb2e5487289", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "427fe918363680bc872f4f88a2cee48f5318b568601fdf3d6890bcb2e5487289", kill_on_drop: false }` [INFO] [stdout] 427fe918363680bc872f4f88a2cee48f5318b568601fdf3d6890bcb2e5487289 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:dff56e7819e73ed36160586b3445e93eb0be776c16704aeeded9c3fb668b2384" "/opt/rustwide/cargo-home/bin/cargo" "+2e6fc425411280c658d852813de01c5157255088" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 075c65da72a1b957faad818c29e76b9f335870d506ec7c0b0bf344fadd4a57b8 [INFO] running `Command { std: "docker" "start" "-a" "075c65da72a1b957faad818c29e76b9f335870d506ec7c0b0bf344fadd4a57b8", kill_on_drop: false }` [INFO] [stderr] Compiling autocfg v0.1.7 [INFO] [stderr] Checking rand_core v0.4.2 [INFO] [stderr] Compiling libc v0.2.65 [INFO] [stderr] Compiling getrandom v0.1.13 [INFO] [stderr] Compiling cfg-if v0.1.10 [INFO] [stderr] Checking ppv-lite86 v0.2.6 [INFO] [stderr] Compiling sdl2 v0.32.2 (https://github.com/AngryLawyer/rust-sdl2#72d199e5) [INFO] [stderr] Compiling sdl2-sys v0.32.6 (https://github.com/AngryLawyer/rust-sdl2#72d199e5) [INFO] [stderr] Checking rand_core v0.3.1 [INFO] [stderr] Checking rand_jitter v0.1.4 [INFO] [stderr] Checking rand_xorshift v0.1.1 [INFO] [stderr] Checking rand_hc v0.1.0 [INFO] [stderr] Checking rand_isaac v0.1.1 [INFO] [stderr] Compiling num-traits v0.2.8 [INFO] [stderr] Compiling num-integer v0.1.41 [INFO] [stderr] Compiling rand_chacha v0.1.1 [INFO] [stderr] Compiling num-iter v0.1.39 [INFO] [stderr] Compiling rand_pcg v0.1.2 [INFO] [stderr] Compiling rand v0.6.5 [INFO] [stderr] Checking c2-chacha v0.2.3 [INFO] [stderr] Checking rand_os v0.1.3 [INFO] [stderr] Checking rand_core v0.5.1 [INFO] [stderr] Checking rand_chacha v0.2.1 [INFO] [stderr] Checking rand v0.7.2 [INFO] [stderr] Checking num v0.1.42 [INFO] [stderr] Checking chip8vm v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: type `color` should have an upper camel case name [INFO] [stdout] --> src/color.rs:3:10 [INFO] [stdout] | [INFO] [stdout] 3 | pub enum color { [INFO] [stdout] | ^^^^^ help: convert the identifier to upper camel case (notice the capitalization): `Color` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `red` should have an upper camel case name [INFO] [stdout] --> src/color.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | red, [INFO] [stdout] | ^^^ help: convert the identifier to upper camel case: `Red` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `green` should have an upper camel case name [INFO] [stdout] --> src/color.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | green, [INFO] [stdout] | ^^^^^ help: convert the identifier to upper camel case: `Green` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `blue` should have an upper camel case name [INFO] [stdout] --> src/color.rs:6:5 [INFO] [stdout] | [INFO] [stdout] 6 | blue, [INFO] [stdout] | ^^^^ help: convert the identifier to upper camel case: `Blue` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `black` should have an upper camel case name [INFO] [stdout] --> src/color.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | black, [INFO] [stdout] | ^^^^^ help: convert the identifier to upper camel case: `Black` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `color` should have an upper camel case name [INFO] [stdout] --> src/color.rs:3:10 [INFO] [stdout] | [INFO] [stdout] 3 | pub enum color { [INFO] [stdout] | ^^^^^ help: convert the identifier to upper camel case (notice the capitalization): `Color` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `cpu` should have an upper camel case name [INFO] [stdout] --> src/cpu.rs:8:12 [INFO] [stdout] | [INFO] [stdout] 8 | pub struct cpu { [INFO] [stdout] | ^^^ help: convert the identifier to upper camel case (notice the capitalization): `Cpu` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `red` should have an upper camel case name [INFO] [stdout] --> src/color.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | red, [INFO] [stdout] | ^^^ help: convert the identifier to upper camel case: `Red` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `green` should have an upper camel case name [INFO] [stdout] --> src/color.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | green, [INFO] [stdout] | ^^^^^ help: convert the identifier to upper camel case: `Green` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around assigned value [INFO] [stdout] --> src/cpu.rs:282:24 [INFO] [stdout] | [INFO] [stdout] 282 | self.v[0x0f] = (self.v[x] & (1 << 7) >> 7); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 282 - self.v[0x0f] = (self.v[x] & (1 << 7) >> 7); [INFO] [stdout] 282 + self.v[0x0f] = self.v[x] & (1 << 7) >> 7; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `blue` should have an upper camel case name [INFO] [stdout] --> src/color.rs:6:5 [INFO] [stdout] | [INFO] [stdout] 6 | blue, [INFO] [stdout] | ^^^^ help: convert the identifier to upper camel case: `Blue` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `black` should have an upper camel case name [INFO] [stdout] --> src/color.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | black, [INFO] [stdout] | ^^^^^ help: convert the identifier to upper camel case: `Black` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `cpu` should have an upper camel case name [INFO] [stdout] --> src/cpu.rs:8:12 [INFO] [stdout] | [INFO] [stdout] 8 | pub struct cpu { [INFO] [stdout] | ^^^ help: convert the identifier to upper camel case (notice the capitalization): `Cpu` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around assigned value [INFO] [stdout] --> src/cpu.rs:311:21 [INFO] [stdout] | [INFO] [stdout] 311 | self.v[x] = (kk & random_byte); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 311 - self.v[x] = (kk & random_byte); [INFO] [stdout] 311 + self.v[x] = kk & random_byte; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around assigned value [INFO] [stdout] --> src/cpu.rs:282:24 [INFO] [stdout] | [INFO] [stdout] 282 | self.v[0x0f] = (self.v[x] & (1 << 7) >> 7); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 282 - self.v[0x0f] = (self.v[x] & (1 << 7) >> 7); [INFO] [stdout] 282 + self.v[0x0f] = self.v[x] & (1 << 7) >> 7; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around index expression [INFO] [stdout] --> src/cpu.rs:319:37 [INFO] [stdout] | [INFO] [stdout] 319 | let pixel = self.memory[(self.i + (y_offset as usize))]; [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 319 - let pixel = self.memory[(self.i + (y_offset as usize))]; [INFO] [stdout] 319 + let pixel = self.memory[self.i + (y_offset as usize)]; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around assigned value [INFO] [stdout] --> src/cpu.rs:372:32 [INFO] [stdout] | [INFO] [stdout] 372 | self.keypad_register = (x as u8); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 372 - self.keypad_register = (x as u8); [INFO] [stdout] 372 + self.keypad_register = x as u8; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around assigned value [INFO] [stdout] --> src/cpu.rs:311:21 [INFO] [stdout] | [INFO] [stdout] 311 | self.v[x] = (kk & random_byte); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 311 - self.v[x] = (kk & random_byte); [INFO] [stdout] 311 + self.v[x] = kk & random_byte; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::cpu::cpu` [INFO] [stdout] --> src/display.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use crate::cpu::cpu; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `display` should have an upper camel case name [INFO] [stdout] --> src/display.rs:9:12 [INFO] [stdout] | [INFO] [stdout] 9 | pub struct display { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to upper camel case: `Display` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around index expression [INFO] [stdout] --> src/cpu.rs:319:37 [INFO] [stdout] | [INFO] [stdout] 319 | let pixel = self.memory[(self.i + (y_offset as usize))]; [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 319 - let pixel = self.memory[(self.i + (y_offset as usize))]; [INFO] [stdout] 319 + let pixel = self.memory[self.i + (y_offset as usize)]; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `keypad` should have an upper camel case name [INFO] [stdout] --> src/keypad.rs:5:12 [INFO] [stdout] | [INFO] [stdout] 5 | pub struct keypad { [INFO] [stdout] | ^^^^^^ help: convert the identifier to upper camel case (notice the capitalization): `Keypad` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `cpu::cpu as _cpu` [INFO] [stdout] --> src/main.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use cpu::cpu as _cpu; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around assigned value [INFO] [stdout] --> src/cpu.rs:372:32 [INFO] [stdout] | [INFO] [stdout] 372 | self.keypad_register = (x as u8); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 372 - self.keypad_register = (x as u8); [INFO] [stdout] 372 + self.keypad_register = x as u8; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::cpu::cpu` [INFO] [stdout] --> src/display.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use crate::cpu::cpu; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `display` should have an upper camel case name [INFO] [stdout] --> src/display.rs:9:12 [INFO] [stdout] | [INFO] [stdout] 9 | pub struct display { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to upper camel case: `Display` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `keypad` should have an upper camel case name [INFO] [stdout] --> src/keypad.rs:5:12 [INFO] [stdout] | [INFO] [stdout] 5 | pub struct keypad { [INFO] [stdout] | ^^^^^^ help: convert the identifier to upper camel case (notice the capitalization): `Keypad` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `cpu::cpu as _cpu` [INFO] [stdout] --> src/main.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use cpu::cpu as _cpu; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable pattern [INFO] [stdout] --> src/cpu.rs:114:13 [INFO] [stdout] | [INFO] [stdout] 114 | (0x0f, _, 0x03, 0x03) => self.op_Fx33(x), [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unreachable_patterns)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `poll_keypad` is assigned to, but never used [INFO] [stdout] --> src/cpu.rs:129:17 [INFO] [stdout] | [INFO] [stdout] 129 | let poll_keypad; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: consider using `_poll_keypad` instead [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `poll_keypad` is never read [INFO] [stdout] --> src/cpu.rs:131:26 [INFO] [stdout] | [INFO] [stdout] 131 | Ok(x) => poll_keypad = x, [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `y` [INFO] [stdout] --> src/cpu.rs:262:37 [INFO] [stdout] | [INFO] [stdout] 262 | fn op_8xy6(&mut self, x: usize, y: usize) { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_y` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable pattern [INFO] [stdout] --> src/cpu.rs:114:13 [INFO] [stdout] | [INFO] [stdout] 114 | (0x0f, _, 0x03, 0x03) => self.op_Fx33(x), [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unreachable_patterns)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `y` [INFO] [stdout] --> src/cpu.rs:281:37 [INFO] [stdout] | [INFO] [stdout] 281 | fn op_8xyE(&mut self, x: usize, y: usize) { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_y` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `poll_keypad` is assigned to, but never used [INFO] [stdout] --> src/cpu.rs:129:17 [INFO] [stdout] | [INFO] [stdout] 129 | let poll_keypad; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: consider using `_poll_keypad` instead [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `poll_keypad` is never read [INFO] [stdout] --> src/cpu.rs:131:26 [INFO] [stdout] | [INFO] [stdout] 131 | Ok(x) => poll_keypad = x, [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `y` [INFO] [stdout] --> src/cpu.rs:262:37 [INFO] [stdout] | [INFO] [stdout] 262 | fn op_8xy6(&mut self, x: usize, y: usize) { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_y` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `y` [INFO] [stdout] --> src/cpu.rs:281:37 [INFO] [stdout] | [INFO] [stdout] 281 | fn op_8xyE(&mut self, x: usize, y: usize) { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_y` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `x_true` [INFO] [stdout] --> src/display.rs:34:21 [INFO] [stdout] | [INFO] [stdout] 34 | let x_true = (x as u32) * SCALE; [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_x_true` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `y_true` [INFO] [stdout] --> src/display.rs:35:21 [INFO] [stdout] | [INFO] [stdout] 35 | let y_true = (y as u32) * SCALE; [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_y_true` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `draw_color` is never read [INFO] [stdout] --> src/display.rs:38:25 [INFO] [stdout] | [INFO] [stdout] 38 | let mut draw_color = color::black.value(); [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `x` [INFO] [stdout] --> src/display.rs:50:24 [INFO] [stdout] | [INFO] [stdout] 50 | Ok(x) => continue, [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_x` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `gfx` [INFO] [stdout] --> src/display.rs:57:29 [INFO] [stdout] | [INFO] [stdout] 57 | pub fn clear(&mut self, gfx: [u8; 32*64]) { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_gfx` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `x_true` [INFO] [stdout] --> src/display.rs:34:21 [INFO] [stdout] | [INFO] [stdout] 34 | let x_true = (x as u32) * SCALE; [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_x_true` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `y_true` [INFO] [stdout] --> src/display.rs:35:21 [INFO] [stdout] | [INFO] [stdout] 35 | let y_true = (y as u32) * SCALE; [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_y_true` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `draw_color` is never read [INFO] [stdout] --> src/display.rs:38:25 [INFO] [stdout] | [INFO] [stdout] 38 | let mut draw_color = color::black.value(); [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `x` [INFO] [stdout] --> src/display.rs:50:24 [INFO] [stdout] | [INFO] [stdout] 50 | Ok(x) => continue, [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_x` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `gfx` [INFO] [stdout] --> src/display.rs:57:29 [INFO] [stdout] | [INFO] [stdout] 57 | pub fn clear(&mut self, gfx: [u8; 32*64]) { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_gfx` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `sdl_context` [INFO] [stdout] --> src/main.rs:20:9 [INFO] [stdout] | [INFO] [stdout] 20 | let sdl_context = sdl2::init().unwrap(); [INFO] [stdout] | ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_sdl_context` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variants `red` and `blue` are never constructed [INFO] [stdout] --> src/color.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 3 | pub enum color { [INFO] [stdout] | ----- variants in this enum [INFO] [stdout] 4 | red, [INFO] [stdout] | ^^^ [INFO] [stdout] 5 | green, [INFO] [stdout] 6 | blue, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `display` is never read [INFO] [stdout] --> src/cpu.rs:25:5 [INFO] [stdout] | [INFO] [stdout] 8 | pub struct cpu { [INFO] [stdout] | --- field in this struct [INFO] [stdout] ... [INFO] [stdout] 25 | display: display [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: comparison is useless due to type limits [INFO] [stdout] --> src/cpu.rs:242:12 [INFO] [stdout] | [INFO] [stdout] 242 | if addition > 0xff { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_comparisons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_8xyE` should have a snake case name [INFO] [stdout] --> src/cpu.rs:281:8 [INFO] [stdout] | [INFO] [stdout] 281 | fn op_8xyE(&mut self, x: usize, y: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_8xy_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Annn` should have a snake case name [INFO] [stdout] --> src/cpu.rs:298:8 [INFO] [stdout] | [INFO] [stdout] 298 | fn op_Annn(&mut self, nnn: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_annn` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Bnnn` should have a snake case name [INFO] [stdout] --> src/cpu.rs:304:8 [INFO] [stdout] | [INFO] [stdout] 304 | fn op_Bnnn(&mut self, nnn: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_bnnn` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Cxkk` should have a snake case name [INFO] [stdout] --> src/cpu.rs:309:8 [INFO] [stdout] | [INFO] [stdout] 309 | fn op_Cxkk(&mut self, x: usize, kk: u8) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_cxkk` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Dxyn` should have a snake case name [INFO] [stdout] --> src/cpu.rs:316:8 [INFO] [stdout] | [INFO] [stdout] 316 | fn op_Dxyn(&mut self, x: usize, y: usize, n: u16) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_dxyn` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Ex9E` should have a snake case name [INFO] [stdout] --> src/cpu.rs:342:8 [INFO] [stdout] | [INFO] [stdout] 342 | fn op_Ex9E(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_ex9_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_ExA1` should have a snake case name [INFO] [stdout] --> src/cpu.rs:352:8 [INFO] [stdout] | [INFO] [stdout] 352 | fn op_ExA1(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_ex_a1` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx07` should have a snake case name [INFO] [stdout] --> src/cpu.rs:362:8 [INFO] [stdout] | [INFO] [stdout] 362 | fn op_Fx07(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx07` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx0A` should have a snake case name [INFO] [stdout] --> src/cpu.rs:369:8 [INFO] [stdout] | [INFO] [stdout] 369 | fn op_Fx0A(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_fx0_a` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx15` should have a snake case name [INFO] [stdout] --> src/cpu.rs:377:8 [INFO] [stdout] | [INFO] [stdout] 377 | fn op_Fx15(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx15` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx18` should have a snake case name [INFO] [stdout] --> src/cpu.rs:383:8 [INFO] [stdout] | [INFO] [stdout] 383 | fn op_Fx18(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx18` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx1E` should have a snake case name [INFO] [stdout] --> src/cpu.rs:389:8 [INFO] [stdout] | [INFO] [stdout] 389 | fn op_Fx1E(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_fx1_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx29` should have a snake case name [INFO] [stdout] --> src/cpu.rs:395:8 [INFO] [stdout] | [INFO] [stdout] 395 | fn op_Fx29(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx29` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx33` should have a snake case name [INFO] [stdout] --> src/cpu.rs:401:8 [INFO] [stdout] | [INFO] [stdout] 401 | fn op_Fx33(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx33` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx55` should have a snake case name [INFO] [stdout] --> src/cpu.rs:408:8 [INFO] [stdout] | [INFO] [stdout] 408 | fn op_Fx55(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx55` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx65` should have a snake case name [INFO] [stdout] --> src/cpu.rs:415:8 [INFO] [stdout] | [INFO] [stdout] 415 | fn op_Fx65(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx65` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `font` should have an upper case name [INFO] [stdout] --> src/font.rs:1:11 [INFO] [stdout] | [INFO] [stdout] 1 | pub const font: [u8; 80] = [ [INFO] [stdout] | ^^^^ help: convert the identifier to upper case: `FONT` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_upper_case_globals)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `sdl_context` [INFO] [stdout] --> src/main.rs:20:9 [INFO] [stdout] | [INFO] [stdout] 20 | let sdl_context = sdl2::init().unwrap(); [INFO] [stdout] | ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_sdl_context` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 45 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variants `red` and `blue` are never constructed [INFO] [stdout] --> src/color.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 3 | pub enum color { [INFO] [stdout] | ----- variants in this enum [INFO] [stdout] 4 | red, [INFO] [stdout] | ^^^ [INFO] [stdout] 5 | green, [INFO] [stdout] 6 | blue, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `opcode` and `display` are never read [INFO] [stdout] --> src/cpu.rs:9:5 [INFO] [stdout] | [INFO] [stdout] 8 | pub struct cpu { [INFO] [stdout] | --- fields in this struct [INFO] [stdout] 9 | opcode: u16, [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 25 | display: display [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: comparison is useless due to type limits [INFO] [stdout] --> src/cpu.rs:242:12 [INFO] [stdout] | [INFO] [stdout] 242 | if addition > 0xff { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_comparisons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_8xyE` should have a snake case name [INFO] [stdout] --> src/cpu.rs:281:8 [INFO] [stdout] | [INFO] [stdout] 281 | fn op_8xyE(&mut self, x: usize, y: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_8xy_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Annn` should have a snake case name [INFO] [stdout] --> src/cpu.rs:298:8 [INFO] [stdout] | [INFO] [stdout] 298 | fn op_Annn(&mut self, nnn: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_annn` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Bnnn` should have a snake case name [INFO] [stdout] --> src/cpu.rs:304:8 [INFO] [stdout] | [INFO] [stdout] 304 | fn op_Bnnn(&mut self, nnn: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_bnnn` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Cxkk` should have a snake case name [INFO] [stdout] --> src/cpu.rs:309:8 [INFO] [stdout] | [INFO] [stdout] 309 | fn op_Cxkk(&mut self, x: usize, kk: u8) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_cxkk` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Dxyn` should have a snake case name [INFO] [stdout] --> src/cpu.rs:316:8 [INFO] [stdout] | [INFO] [stdout] 316 | fn op_Dxyn(&mut self, x: usize, y: usize, n: u16) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_dxyn` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Ex9E` should have a snake case name [INFO] [stdout] --> src/cpu.rs:342:8 [INFO] [stdout] | [INFO] [stdout] 342 | fn op_Ex9E(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_ex9_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_ExA1` should have a snake case name [INFO] [stdout] --> src/cpu.rs:352:8 [INFO] [stdout] | [INFO] [stdout] 352 | fn op_ExA1(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_ex_a1` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx07` should have a snake case name [INFO] [stdout] --> src/cpu.rs:362:8 [INFO] [stdout] | [INFO] [stdout] 362 | fn op_Fx07(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx07` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx0A` should have a snake case name [INFO] [stdout] --> src/cpu.rs:369:8 [INFO] [stdout] | [INFO] [stdout] 369 | fn op_Fx0A(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_fx0_a` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx15` should have a snake case name [INFO] [stdout] --> src/cpu.rs:377:8 [INFO] [stdout] | [INFO] [stdout] 377 | fn op_Fx15(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx15` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx18` should have a snake case name [INFO] [stdout] --> src/cpu.rs:383:8 [INFO] [stdout] | [INFO] [stdout] 383 | fn op_Fx18(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx18` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx1E` should have a snake case name [INFO] [stdout] --> src/cpu.rs:389:8 [INFO] [stdout] | [INFO] [stdout] 389 | fn op_Fx1E(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `op_fx1_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx29` should have a snake case name [INFO] [stdout] --> src/cpu.rs:395:8 [INFO] [stdout] | [INFO] [stdout] 395 | fn op_Fx29(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx29` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx33` should have a snake case name [INFO] [stdout] --> src/cpu.rs:401:8 [INFO] [stdout] | [INFO] [stdout] 401 | fn op_Fx33(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx33` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx55` should have a snake case name [INFO] [stdout] --> src/cpu.rs:408:8 [INFO] [stdout] | [INFO] [stdout] 408 | fn op_Fx55(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx55` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `op_Fx65` should have a snake case name [INFO] [stdout] --> src/cpu.rs:415:8 [INFO] [stdout] | [INFO] [stdout] 415 | fn op_Fx65(&mut self, x: usize) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `op_fx65` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `font` should have an upper case name [INFO] [stdout] --> src/font.rs:1:11 [INFO] [stdout] | [INFO] [stdout] 1 | pub const font: [u8; 80] = [ [INFO] [stdout] | ^^^^ help: convert the identifier to upper case: `FONT` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_upper_case_globals)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 45 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 2.10s [INFO] running `Command { std: "docker" "inspect" "075c65da72a1b957faad818c29e76b9f335870d506ec7c0b0bf344fadd4a57b8", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "075c65da72a1b957faad818c29e76b9f335870d506ec7c0b0bf344fadd4a57b8", kill_on_drop: false }` [INFO] [stdout] 075c65da72a1b957faad818c29e76b9f335870d506ec7c0b0bf344fadd4a57b8