[INFO] cloning repository https://github.com/Ngalstyan4/emulator101.rs
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/Ngalstyan4/emulator101.rs" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FNgalstyan4%2Femulator101.rs", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FNgalstyan4%2Femulator101.rs'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 38d52100b6f443d94d6077d7ff4c5a7a9ec82595
[INFO] checking Ngalstyan4/emulator101.rs against master#bca37a20bd376ce3fd138e7cdee7fe704e0f8814 for pr-139493-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FNgalstyan4%2Femulator101.rs" "/workspace/builds/worker-3-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-3-tc1/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/Ngalstyan4/emulator101.rs
[INFO] finished tweaking git repo https://github.com/Ngalstyan4/emulator101.rs
[INFO] tweaked toml for git repo https://github.com/Ngalstyan4/emulator101.rs written to /workspace/builds/worker-3-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/Ngalstyan4/emulator101.rs on toolchain bca37a20bd376ce3fd138e7cdee7fe704e0f8814
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+bca37a20bd376ce3fd138e7cdee7fe704e0f8814" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/Ngalstyan4/emulator101.rs 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" "+bca37a20bd376ce3fd138e7cdee7fe704e0f8814" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded num-integer v0.1.41
[INFO] [stderr]   Downloaded num v0.1.42
[INFO] [stderr]   Downloaded num-iter v0.1.39
[INFO] [stderr]   Downloaded drawille v0.2.3
[INFO] [stderr]   Downloaded num-traits v0.2.10
[INFO] [stderr]   Downloaded sdl2 v0.32.2
[INFO] [stderr]   Downloaded sdl2-sys v0.32.6
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-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:aa71247004a7fa38d13ec170f48f06cdedf5bc50b2a8645e56ed7e992e6fa513" "/opt/rustwide/cargo-home/bin/cargo" "+bca37a20bd376ce3fd138e7cdee7fe704e0f8814" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] 7f486a80f4be57837cc06f6d3991d71e6be8821547821b2d3d87698bbbdee50c
[INFO] running `Command { std: "docker" "start" "-a" "7f486a80f4be57837cc06f6d3991d71e6be8821547821b2d3d87698bbbdee50c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "7f486a80f4be57837cc06f6d3991d71e6be8821547821b2d3d87698bbbdee50c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "7f486a80f4be57837cc06f6d3991d71e6be8821547821b2d3d87698bbbdee50c", kill_on_drop: false }`
[INFO] [stdout] 7f486a80f4be57837cc06f6d3991d71e6be8821547821b2d3d87698bbbdee50c
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-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:aa71247004a7fa38d13ec170f48f06cdedf5bc50b2a8645e56ed7e992e6fa513" "/opt/rustwide/cargo-home/bin/cargo" "+bca37a20bd376ce3fd138e7cdee7fe704e0f8814" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 609cdc31dea641ad20564fd2098b82ca1c01732545af02a62c60a6e529f0ac2c
[INFO] running `Command { std: "docker" "start" "-a" "609cdc31dea641ad20564fd2098b82ca1c01732545af02a62c60a6e529f0ac2c", 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 cfg-if v0.1.10
[INFO] [stderr]    Compiling sdl2 v0.32.2
[INFO] [stderr]     Checking drawille v0.2.3
[INFO] [stderr]    Compiling sdl2-sys v0.32.6
[INFO] [stderr]    Compiling num-traits v0.2.10
[INFO] [stderr]    Compiling num-integer v0.1.41
[INFO] [stderr]    Compiling num-iter v0.1.39
[INFO] [stderr]    Compiling rand_pcg v0.1.2
[INFO] [stderr]    Compiling rand_chacha v0.1.1
[INFO] [stderr]     Checking rand_core v0.3.1
[INFO] [stderr]    Compiling rand v0.6.5
[INFO] [stderr]     Checking rand_hc v0.1.0
[INFO] [stderr]     Checking rand_isaac v0.1.1
[INFO] [stderr]     Checking rand_xorshift v0.1.1
[INFO] [stderr]     Checking rand_jitter v0.1.4
[INFO] [stderr]     Checking rand_os v0.1.3
[INFO] [stderr]     Checking num v0.1.42
[INFO] [stderr]     Checking emulator v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `std::time::Duration`
[INFO] [stdout]  --> src/sdl2_screen.rs:7:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | use std::time::Duration;
[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 imports: `thread` and `time`
[INFO] [stdout]  --> src/sdl2_screen.rs:9:11
[INFO] [stdout]   |
[INFO] [stdout] 9 | use std::{thread, time};
[INFO] [stdout]   |           ^^^^^^  ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::time::Duration`
[INFO] [stdout]   --> src/main.rs:45:9
[INFO] [stdout]    |
[INFO] [stdout] 45 |     use std::time::Duration;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: the feature `try_from` has been stable since 1.34.0 and no longer requires an attribute to enable
[INFO] [stdout]  --> src/main.rs:1:12
[INFO] [stdout]   |
[INFO] [stdout] 1 | #![feature(try_from)]
[INFO] [stdout]   |            ^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(stable_features)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::time::Duration`
[INFO] [stdout]  --> src/sdl2_screen.rs:7:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | use std::time::Duration;
[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 imports: `thread` and `time`
[INFO] [stdout]  --> src/sdl2_screen.rs:9:11
[INFO] [stdout]   |
[INFO] [stdout] 9 | use std::{thread, time};
[INFO] [stdout]   |           ^^^^^^  ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::time::Duration`
[INFO] [stdout]   --> src/main.rs:45:9
[INFO] [stdout]    |
[INFO] [stdout] 45 |     use std::time::Duration;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: the feature `try_from` has been stable since 1.34.0 and no longer requires an attribute to enable
[INFO] [stdout]  --> src/main.rs:1:12
[INFO] [stdout]   |
[INFO] [stdout] 1 | #![feature(try_from)]
[INFO] [stdout]   |            ^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(stable_features)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `c`
[INFO] [stdout]   --> src/sdl2_screen.rs:40:63
[INFO] [stdout]    |
[INFO] [stdout] 40 |     fn draw_elems<'a>(&mut self, elems: Vec<(usize, &'a u8)>, c:Color) {
[INFO] [stdout]    |                                                               ^ help: if this is intentional, prefix it with an underscore: `_c`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `v`
[INFO] [stdout]   --> src/sdl2_screen.rs:93:66
[INFO] [stdout]    |
[INFO] [stdout] 93 |         let elems = framebuffer.iter().enumerate().filter(|&(i, &v)| (i *8 % 256 < 112 )== top).collect::<Vec<_>>();
[INFO] [stdout]    |                                                                  ^ help: if this is intentional, prefix it with an underscore: `_v`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `num_instr`
[INFO] [stdout]   --> src/main.rs:21:9
[INFO] [stdout]    |
[INFO] [stdout] 21 |     let num_instr = args[1].parse().unwrap_or(50000004);
[INFO] [stdout]    |         ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_num_instr`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `get_cycles` is never used
[INFO] [stdout]    --> src/cpu8080.rs:228:12
[INFO] [stdout]     |
[INFO] [stdout] 172 | impl State8080 {
[INFO] [stdout]     | -------------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 228 |     pub fn get_cycles(&self) -> u64 {
[INFO] [stdout]     |            ^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `main` is never used
[INFO] [stdout]     --> src/cpu8080.rs:1799:4
[INFO] [stdout]      |
[INFO] [stdout] 1799 | fn main() {
[INFO] [stdout]      |    ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated function `new` is never used
[INFO] [stdout]  --> src/screen.rs:2:8
[INFO] [stdout]   |
[INFO] [stdout] 1 | pub trait Screen {
[INFO] [stdout]   |           ------ associated function in this trait
[INFO] [stdout] 2 |     fn new() -> Self; //  Q:: is that the right thing to do? // should this be out of the trait?
[INFO] [stdout]   |        ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `SoC` is never constructed
[INFO] [stdout]  --> src/soc.rs:5:8
[INFO] [stdout]   |
[INFO] [stdout] 5 | struct SoC<S: Screen> {
[INFO] [stdout]   |        ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `emulate` is never used
[INFO] [stdout]   --> src/soc.rs:15:8
[INFO] [stdout]    |
[INFO] [stdout] 14 | impl SoC<Sdl2Screen> {
[INFO] [stdout]    | -------------------- method in this implementation
[INFO] [stdout] 15 |     fn emulate(self) {
[INFO] [stdout]    |        ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `_LogicFlagsA` should have a snake case name
[INFO] [stdout]    --> src/cpu8080.rs:248:8
[INFO] [stdout]     |
[INFO] [stdout] 248 |     fn _LogicFlagsA(&mut self) {
[INFO] [stdout]     |        ^^^^^^^^^^^^ help: convert the identifier to snake case: `_logic_flags_a`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `_arithFlagsA` should have a snake case name
[INFO] [stdout]    --> src/cpu8080.rs:256:8
[INFO] [stdout]     |
[INFO] [stdout] 256 |     fn _arithFlagsA(&mut self, res: u16) {
[INFO] [stdout]     |        ^^^^^^^^^^^^ help: convert the identifier to snake case: `_arith_flags_a`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `_arithNoCarry` should have a snake case name
[INFO] [stdout]    --> src/cpu8080.rs:263:8
[INFO] [stdout]     |
[INFO] [stdout] 263 |     fn _arithNoCarry(&mut self, res: u16) {
[INFO] [stdout]     |        ^^^^^^^^^^^^^ help: convert the identifier to snake case: `_arith_no_carry`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `c`
[INFO] [stdout]   --> src/sdl2_screen.rs:40:63
[INFO] [stdout]    |
[INFO] [stdout] 40 |     fn draw_elems<'a>(&mut self, elems: Vec<(usize, &'a u8)>, c:Color) {
[INFO] [stdout]    |                                                               ^ help: if this is intentional, prefix it with an underscore: `_c`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `v`
[INFO] [stdout]   --> src/sdl2_screen.rs:93:66
[INFO] [stdout]    |
[INFO] [stdout] 93 |         let elems = framebuffer.iter().enumerate().filter(|&(i, &v)| (i *8 % 256 < 112 )== top).collect::<Vec<_>>();
[INFO] [stdout]    |                                                                  ^ help: if this is intentional, prefix it with an underscore: `_v`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `num_instr`
[INFO] [stdout]   --> src/main.rs:21:9
[INFO] [stdout]    |
[INFO] [stdout] 21 |     let num_instr = args[1].parse().unwrap_or(50000004);
[INFO] [stdout]    |         ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_num_instr`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `get_cycles` is never used
[INFO] [stdout]    --> src/cpu8080.rs:228:12
[INFO] [stdout]     |
[INFO] [stdout] 172 | impl State8080 {
[INFO] [stdout]     | -------------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 228 |     pub fn get_cycles(&self) -> u64 {
[INFO] [stdout]     |            ^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `main` is never used
[INFO] [stdout]     --> src/cpu8080.rs:1799:4
[INFO] [stdout]      |
[INFO] [stdout] 1799 | fn main() {
[INFO] [stdout]      |    ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated function `new` is never used
[INFO] [stdout]  --> src/screen.rs:2:8
[INFO] [stdout]   |
[INFO] [stdout] 1 | pub trait Screen {
[INFO] [stdout]   |           ------ associated function in this trait
[INFO] [stdout] 2 |     fn new() -> Self; //  Q:: is that the right thing to do? // should this be out of the trait?
[INFO] [stdout]   |        ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `SoC` is never constructed
[INFO] [stdout]  --> src/soc.rs:5:8
[INFO] [stdout]   |
[INFO] [stdout] 5 | struct SoC<S: Screen> {
[INFO] [stdout]   |        ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `emulate` is never used
[INFO] [stdout]   --> src/soc.rs:15:8
[INFO] [stdout]    |
[INFO] [stdout] 14 | impl SoC<Sdl2Screen> {
[INFO] [stdout]    | -------------------- method in this implementation
[INFO] [stdout] 15 |     fn emulate(self) {
[INFO] [stdout]    |        ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `_LogicFlagsA` should have a snake case name
[INFO] [stdout]    --> src/cpu8080.rs:248:8
[INFO] [stdout]     |
[INFO] [stdout] 248 |     fn _LogicFlagsA(&mut self) {
[INFO] [stdout]     |        ^^^^^^^^^^^^ help: convert the identifier to snake case: `_logic_flags_a`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `_arithFlagsA` should have a snake case name
[INFO] [stdout]    --> src/cpu8080.rs:256:8
[INFO] [stdout]     |
[INFO] [stdout] 256 |     fn _arithFlagsA(&mut self, res: u16) {
[INFO] [stdout]     |        ^^^^^^^^^^^^ help: convert the identifier to snake case: `_arith_flags_a`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `_arithNoCarry` should have a snake case name
[INFO] [stdout]    --> src/cpu8080.rs:263:8
[INFO] [stdout]     |
[INFO] [stdout] 263 |     fn _arithNoCarry(&mut self, res: u16) {
[INFO] [stdout]     |        ^^^^^^^^^^^^^ help: convert the identifier to snake case: `_arith_no_carry`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 7.11s
[INFO] running `Command { std: "docker" "inspect" "609cdc31dea641ad20564fd2098b82ca1c01732545af02a62c60a6e529f0ac2c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "609cdc31dea641ad20564fd2098b82ca1c01732545af02a62c60a6e529f0ac2c", kill_on_drop: false }`
[INFO] [stdout] 609cdc31dea641ad20564fd2098b82ca1c01732545af02a62c60a6e529f0ac2c
