[INFO] cloning repository https://github.com/dspencer001/rschip16
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/dspencer001/rschip16" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fdspencer001%2Frschip16", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fdspencer001%2Frschip16'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 7fef790cbbc1a05b7cb587f70827bcaf1bf4919b
[INFO] checking dspencer001/rschip16 against 1.95.0 for pr-155915
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fdspencer001%2Frschip16" "/workspace/builds/worker-7-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc1/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/dspencer001/rschip16
[INFO] finished tweaking git repo https://github.com/dspencer001/rschip16
[INFO] tweaked toml for git repo https://github.com/dspencer001/rschip16 written to /workspace/builds/worker-7-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/dspencer001/rschip16 on toolchain 1.95.0
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+1.95.0" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/dspencer001/rschip16 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" "+1.95.0" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded binrw v0.11.1
[INFO] [stderr]   Downloaded binrw_derive v0.11.1
[INFO] [stderr]   Downloaded sdl2 v0.35.2
[INFO] [stderr]   Downloaded libc v0.2.142
[INFO] [stderr]   Downloaded sdl2-sys v0.35.2
[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:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+1.95.0" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] b9e1e1adb275f95375574776a6180e6a385371970d8e8060759abe35f7a0532d
[INFO] running `Command { std: "docker" "start" "-a" "b9e1e1adb275f95375574776a6180e6a385371970d8e8060759abe35f7a0532d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "b9e1e1adb275f95375574776a6180e6a385371970d8e8060759abe35f7a0532d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "b9e1e1adb275f95375574776a6180e6a385371970d8e8060759abe35f7a0532d", kill_on_drop: false }`
[INFO] [stdout] b9e1e1adb275f95375574776a6180e6a385371970d8e8060759abe35f7a0532d
[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" "-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" "+1.95.0" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 593b57f4d1166ebdbcccdcd3fad2f78cf737251111d2d7d894dff6a07a103c97
[INFO] running `Command { std: "docker" "start" "-a" "593b57f4d1166ebdbcccdcd3fad2f78cf737251111d2d7d894dff6a07a103c97", kill_on_drop: false }`
[INFO] [stderr]    Compiling libc v0.2.142
[INFO] [stderr]    Compiling proc-macro2 v1.0.56
[INFO] [stderr]    Compiling unicode-ident v1.0.8
[INFO] [stderr]    Compiling quote v1.0.26
[INFO] [stderr]    Compiling io-lifetimes v1.0.10
[INFO] [stderr]    Compiling rustix v0.37.19
[INFO] [stderr]    Compiling version-compare v0.1.1
[INFO] [stderr]     Checking linux-raw-sys v0.3.7
[INFO] [stderr]     Checking anstyle-parse v0.2.0
[INFO] [stderr]    Compiling sdl2-sys v0.35.2
[INFO] [stderr]    Compiling binrw_derive v0.11.1
[INFO] [stderr]     Checking anstyle-query v1.0.0
[INFO] [stderr]     Checking anstyle v1.0.0
[INFO] [stderr]    Compiling sdl2 v0.35.2
[INFO] [stderr]     Checking clap_lex v0.4.1
[INFO] [stderr]    Compiling owo-colors v3.5.0
[INFO] [stderr]    Compiling either v1.8.1
[INFO] [stderr]    Compiling binrw v0.11.1
[INFO] [stderr]     Checking once_cell v1.17.1
[INFO] [stderr]     Checking bytemuck v1.13.1
[INFO] [stderr]     Checking array-init v2.1.0
[INFO] [stderr]    Compiling syn v1.0.109
[INFO] [stderr]    Compiling syn v2.0.15
[INFO] [stderr]     Checking getrandom v0.2.9
[INFO] [stderr]     Checking rand_core v0.6.4
[INFO] [stderr]     Checking rand_chacha v0.3.1
[INFO] [stderr]     Checking rand v0.8.5
[INFO] [stderr]     Checking is-terminal v0.4.7
[INFO] [stderr]     Checking anstream v0.3.2
[INFO] [stderr]     Checking clap_builder v4.2.7
[INFO] [stderr]    Compiling clap_derive v4.2.0
[INFO] [stderr]     Checking clap v4.2.7
[INFO] [stderr]     Checking chip16 v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `sdl2::audio::AudioSpec`
[INFO] [stdout]  --> src/audio.rs:6:5
[INFO] [stdout]   |
[INFO] [stdout] 6 | use sdl2::audio::AudioSpec;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::f32::MAX`
[INFO] [stdout]  --> src/audio.rs:7:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | use std::f32::MAX;
[INFO] [stdout]   |     ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::audio::AudioQueue`
[INFO] [stdout]   --> src/audio.rs:12:5
[INFO] [stdout]    |
[INFO] [stdout] 12 | use sdl2::audio::AudioQueue;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `crate::FPS`
[INFO] [stdout]   --> src/audio.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use crate::FPS;
[INFO] [stdout]    |     ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::fs::File`
[INFO] [stdout]  --> src/cpu.rs:1:5
[INFO] [stdout]   |
[INFO] [stdout] 1 | use std::fs::File;
[INFO] [stdout]   |     ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/cpu.rs:934:8
[INFO] [stdout]     |
[INFO] [stdout] 934 |     if (load_mem(state, a) != val) {
[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] 934 -     if (load_mem(state, a) != val) {
[INFO] [stdout] 934 +     if load_mem(state, a) != val  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::event::Event`
[INFO] [stdout]  --> src/renderer.rs:1:5
[INFO] [stdout]   |
[INFO] [stdout] 1 | use sdl2::event::Event;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::keyboard::Keycode`
[INFO] [stdout]  --> src/renderer.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use sdl2::keyboard::Keycode;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::EventPump`
[INFO] [stdout]  --> src/renderer.rs:7:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | use sdl2::EventPump;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `rand::Rng`
[INFO] [stdout]   --> src/main.rs:21:5
[INFO] [stdout]    |
[INFO] [stdout] 21 | use rand::Rng;
[INFO] [stdout]    |     ^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::audio::AudioSpec`
[INFO] [stdout]  --> src/audio.rs:6:5
[INFO] [stdout]   |
[INFO] [stdout] 6 | use sdl2::audio::AudioSpec;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::f32::MAX`
[INFO] [stdout]  --> src/audio.rs:7:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | use std::f32::MAX;
[INFO] [stdout]   |     ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::audio::AudioQueue`
[INFO] [stdout]   --> src/audio.rs:12:5
[INFO] [stdout]    |
[INFO] [stdout] 12 | use sdl2::audio::AudioQueue;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `crate::FPS`
[INFO] [stdout]   --> src/audio.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use crate::FPS;
[INFO] [stdout]    |     ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::fs::File`
[INFO] [stdout]  --> src/cpu.rs:1:5
[INFO] [stdout]   |
[INFO] [stdout] 1 | use std::fs::File;
[INFO] [stdout]   |     ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/cpu.rs:934:8
[INFO] [stdout]     |
[INFO] [stdout] 934 |     if (load_mem(state, a) != val) {
[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] 934 -     if (load_mem(state, a) != val) {
[INFO] [stdout] 934 +     if load_mem(state, a) != val  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::event::Event`
[INFO] [stdout]  --> src/renderer.rs:1:5
[INFO] [stdout]   |
[INFO] [stdout] 1 | use sdl2::event::Event;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::keyboard::Keycode`
[INFO] [stdout]  --> src/renderer.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use sdl2::keyboard::Keycode;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `sdl2::EventPump`
[INFO] [stdout]  --> src/renderer.rs:7:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | use sdl2::EventPump;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `rand::Rng`
[INFO] [stdout]   --> src/main.rs:21:5
[INFO] [stdout]    |
[INFO] [stdout] 21 | use rand::Rng;
[INFO] [stdout]    |     ^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `state`
[INFO] [stdout]   --> src/cpu.rs:51:8
[INFO] [stdout]    |
[INFO] [stdout] 51 | fn nop(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |        ^^^^^ help: if this is intentional, prefix it with an underscore: `_state`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]   --> src/cpu.rs:51:25
[INFO] [stdout]    |
[INFO] [stdout] 51 | fn nop(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |                         ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `state`
[INFO] [stdout]   --> src/cpu.rs:56:10
[INFO] [stdout]    |
[INFO] [stdout] 56 | fn error(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |          ^^^^^ help: if this is intentional, prefix it with an underscore: `_state`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]   --> src/cpu.rs:56:27
[INFO] [stdout]    |
[INFO] [stdout] 56 | fn error(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |                           ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]   --> src/cpu.rs:61:25
[INFO] [stdout]    |
[INFO] [stdout] 61 | fn cls(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |                         ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]   --> src/cpu.rs:68:27
[INFO] [stdout]    |
[INFO] [stdout] 68 | fn vblnk(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |                           ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]    --> src/cpu.rs:137:26
[INFO] [stdout]     |
[INFO] [stdout] 137 | fn snd0(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]     |                          ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]    --> src/cpu.rs:718:26
[INFO] [stdout]     |
[INFO] [stdout] 718 | fn popf(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]     |                          ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/main.rs:47:9
[INFO] [stdout]    |
[INFO] [stdout] 47 |     let mut has_header: bool;
[INFO] [stdout]    |         ----^^^^^^^^^^
[INFO] [stdout]    |         |
[INFO] [stdout]    |         help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `spec`
[INFO] [stdout]   --> src/main.rs:83:80
[INFO] [stdout]    |
[INFO] [stdout] 83 |     let mut audio_device = audio_subsystem.open_playback(None, &desired_spec, |spec| {
[INFO] [stdout]    |                                                                                ^^^^ help: if this is intentional, prefix it with an underscore: `_spec`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `magic_number`, `reserved`, `specification_version`, `rom_size`, and `rom_crc` are never read
[INFO] [stdout]   --> src/main.rs:37:5
[INFO] [stdout]    |
[INFO] [stdout] 36 | pub struct ROMHeader {
[INFO] [stdout]    |            --------- fields in this struct
[INFO] [stdout] 37 |     magic_number: [u8; 4],
[INFO] [stdout]    |     ^^^^^^^^^^^^
[INFO] [stdout] 38 |     reserved: u8,
[INFO] [stdout]    |     ^^^^^^^^
[INFO] [stdout] 39 |     specification_version: u8,
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 40 |     rom_size: u32,
[INFO] [stdout]    |     ^^^^^^^^
[INFO] [stdout] 41 |     start_address: [u8; 2],
[INFO] [stdout] 42 |     rom_crc: [u8; 4],
[INFO] [stdout]    |     ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/audio.rs:62:34
[INFO] [stdout]    |
[INFO] [stdout] 62 | pub fn default_wave(output_file: &mut File) -> Wave {
[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] 62 | pub fn default_wave(output_file: &mut File) -> Wave<'_> {
[INFO] [stdout]    |                                                    ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: structure field `C` should have a snake case name
[INFO] [stdout]     --> src/cpu.rs:1046:5
[INFO] [stdout]      |
[INFO] [stdout] 1046 |     C: bool,
[INFO] [stdout]      |     ^ help: convert the identifier to snake case (notice the capitalization): `c`
[INFO] [stdout]      |
[INFO] [stdout]      = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: structure field `Z` should have a snake case name
[INFO] [stdout]     --> src/cpu.rs:1047:5
[INFO] [stdout]      |
[INFO] [stdout] 1047 |     Z: bool,
[INFO] [stdout]      |     ^ help: convert the identifier to snake case (notice the capitalization): `z`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: structure field `O` should have a snake case name
[INFO] [stdout]     --> src/cpu.rs:1048:5
[INFO] [stdout]      |
[INFO] [stdout] 1048 |     O: bool,
[INFO] [stdout]      |     ^ help: convert the identifier to snake case (notice the capitalization): `o`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: structure field `N` should have a snake case name
[INFO] [stdout]     --> src/cpu.rs:1049:5
[INFO] [stdout]      |
[INFO] [stdout] 1049 |     N: bool,
[INFO] [stdout]      |     ^ help: convert the identifier to snake case: `n`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `Result` that must be used
[INFO] [stdout]   --> src/renderer.rs:50:21
[INFO] [stdout]    |
[INFO] [stdout] 50 |                     self.draw_dot(x as u32, y as u32);
[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)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]    |
[INFO] [stdout] 50 |                     let _ = self.draw_dot(x as u32, y as u32);
[INFO] [stdout]    |                     +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `state`
[INFO] [stdout]   --> src/cpu.rs:51:8
[INFO] [stdout]    |
[INFO] [stdout] 51 | fn nop(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |        ^^^^^ help: if this is intentional, prefix it with an underscore: `_state`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]   --> src/cpu.rs:51:25
[INFO] [stdout]    |
[INFO] [stdout] 51 | fn nop(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |                         ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `state`
[INFO] [stdout]   --> src/cpu.rs:56:10
[INFO] [stdout]    |
[INFO] [stdout] 56 | fn error(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |          ^^^^^ help: if this is intentional, prefix it with an underscore: `_state`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]   --> src/cpu.rs:56:27
[INFO] [stdout]    |
[INFO] [stdout] 56 | fn error(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |                           ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]   --> src/cpu.rs:61:25
[INFO] [stdout]    |
[INFO] [stdout] 61 | fn cls(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |                         ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]   --> src/cpu.rs:68:27
[INFO] [stdout]    |
[INFO] [stdout] 68 | fn vblnk(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]    |                           ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]    --> src/cpu.rs:137:26
[INFO] [stdout]     |
[INFO] [stdout] 137 | fn snd0(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]     |                          ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `instruction`
[INFO] [stdout]    --> src/cpu.rs:718:26
[INFO] [stdout]     |
[INFO] [stdout] 718 | fn popf(state: &mut CPU, instruction: &Instruction) -> Result<(), String> {
[INFO] [stdout]     |                          ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_instruction`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/main.rs:47:9
[INFO] [stdout]    |
[INFO] [stdout] 47 |     let mut has_header: bool;
[INFO] [stdout]    |         ----^^^^^^^^^^
[INFO] [stdout]    |         |
[INFO] [stdout]    |         help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `spec`
[INFO] [stdout]   --> src/main.rs:83:80
[INFO] [stdout]    |
[INFO] [stdout] 83 |     let mut audio_device = audio_subsystem.open_playback(None, &desired_spec, |spec| {
[INFO] [stdout]    |                                                                                ^^^^ help: if this is intentional, prefix it with an underscore: `_spec`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `magic_number`, `reserved`, `specification_version`, `rom_size`, and `rom_crc` are never read
[INFO] [stdout]   --> src/main.rs:37:5
[INFO] [stdout]    |
[INFO] [stdout] 36 | pub struct ROMHeader {
[INFO] [stdout]    |            --------- fields in this struct
[INFO] [stdout] 37 |     magic_number: [u8; 4],
[INFO] [stdout]    |     ^^^^^^^^^^^^
[INFO] [stdout] 38 |     reserved: u8,
[INFO] [stdout]    |     ^^^^^^^^
[INFO] [stdout] 39 |     specification_version: u8,
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 40 |     rom_size: u32,
[INFO] [stdout]    |     ^^^^^^^^
[INFO] [stdout] 41 |     start_address: [u8; 2],
[INFO] [stdout] 42 |     rom_crc: [u8; 4],
[INFO] [stdout]    |     ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/audio.rs:62:34
[INFO] [stdout]    |
[INFO] [stdout] 62 | pub fn default_wave(output_file: &mut File) -> Wave {
[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] 62 | pub fn default_wave(output_file: &mut File) -> Wave<'_> {
[INFO] [stdout]    |                                                    ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: structure field `C` should have a snake case name
[INFO] [stdout]     --> src/cpu.rs:1046:5
[INFO] [stdout]      |
[INFO] [stdout] 1046 |     C: bool,
[INFO] [stdout]      |     ^ help: convert the identifier to snake case (notice the capitalization): `c`
[INFO] [stdout]      |
[INFO] [stdout]      = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: structure field `Z` should have a snake case name
[INFO] [stdout]     --> src/cpu.rs:1047:5
[INFO] [stdout]      |
[INFO] [stdout] 1047 |     Z: bool,
[INFO] [stdout]      |     ^ help: convert the identifier to snake case (notice the capitalization): `z`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: structure field `O` should have a snake case name
[INFO] [stdout]     --> src/cpu.rs:1048:5
[INFO] [stdout]      |
[INFO] [stdout] 1048 |     O: bool,
[INFO] [stdout]      |     ^ help: convert the identifier to snake case (notice the capitalization): `o`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: structure field `N` should have a snake case name
[INFO] [stdout]     --> src/cpu.rs:1049:5
[INFO] [stdout]      |
[INFO] [stdout] 1049 |     N: bool,
[INFO] [stdout]      |     ^ help: convert the identifier to snake case: `n`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `Result` that must be used
[INFO] [stdout]   --> src/renderer.rs:50:21
[INFO] [stdout]    |
[INFO] [stdout] 50 |                     self.draw_dot(x as u32, y as u32);
[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)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]    |
[INFO] [stdout] 50 |                     let _ = self.draw_dot(x as u32, y as u32);
[INFO] [stdout]    |                     +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 30.05s
[INFO] running `Command { std: "docker" "inspect" "593b57f4d1166ebdbcccdcd3fad2f78cf737251111d2d7d894dff6a07a103c97", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "593b57f4d1166ebdbcccdcd3fad2f78cf737251111d2d7d894dff6a07a103c97", kill_on_drop: false }`
[INFO] [stdout] 593b57f4d1166ebdbcccdcd3fad2f78cf737251111d2d7d894dff6a07a103c97
