[INFO] cloning repository https://github.com/cmr/one-hour-roguelike [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/cmr/one-hour-roguelike" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fcmr%2Fone-hour-roguelike", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fcmr%2Fone-hour-roguelike'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 06d314a5750343c7da3e7688de2d99620e2d2f3e [INFO] checking cmr/one-hour-roguelike/06d314a5750343c7da3e7688de2d99620e2d2f3e against master#2dece5bb62f234f5622a08289c5a3d1555cd7843 for pr-132325 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fcmr%2Fone-hour-roguelike" "/workspace/builds/worker-6-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-6-tc1/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/cmr/one-hour-roguelike on toolchain 2dece5bb62f234f5622a08289c5a3d1555cd7843 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+2dece5bb62f234f5622a08289c5a3d1555cd7843" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/cmr/one-hour-roguelike [INFO] finished tweaking git repo https://github.com/cmr/one-hour-roguelike [INFO] tweaked toml for git repo https://github.com/cmr/one-hour-roguelike written to /workspace/builds/worker-6-tc1/source/Cargo.toml [INFO] crate git repo https://github.com/cmr/one-hour-roguelike 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" "+2dece5bb62f234f5622a08289c5a3d1555cd7843" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2021 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-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:4a844ea9eb2546a2d2c7022eacef16ef2e8229c7fbb2c7d4d55a9ceca922f72d" "/opt/rustwide/cargo-home/bin/cargo" "+2dece5bb62f234f5622a08289c5a3d1555cd7843" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 8e5a85423fc67faa2dabc2251cb90a5324e49a72d409740458a9ae59b439ac7c [INFO] running `Command { std: "docker" "start" "-a" "8e5a85423fc67faa2dabc2251cb90a5324e49a72d409740458a9ae59b439ac7c", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "8e5a85423fc67faa2dabc2251cb90a5324e49a72d409740458a9ae59b439ac7c", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "8e5a85423fc67faa2dabc2251cb90a5324e49a72d409740458a9ae59b439ac7c", kill_on_drop: false }` [INFO] [stdout] 8e5a85423fc67faa2dabc2251cb90a5324e49a72d409740458a9ae59b439ac7c [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-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:4a844ea9eb2546a2d2c7022eacef16ef2e8229c7fbb2c7d4d55a9ceca922f72d" "/opt/rustwide/cargo-home/bin/cargo" "+2dece5bb62f234f5622a08289c5a3d1555cd7843" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] bd59d1ddf329405bf62ece22f75d6943fa6815ede29354dafe811eb1a47c8ec4 [INFO] running `Command { std: "docker" "start" "-a" "bd59d1ddf329405bf62ece22f75d6943fa6815ede29354dafe811eb1a47c8ec4", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2021 [INFO] [stderr] Checking rogue v0.0.1 (/opt/rustwide/workdir) [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:172:9 [INFO] [stdout] | [INFO] [stdout] 172 | let mut stdin = std::io::stdin(); [INFO] [stdout] | ----^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:172:9 [INFO] [stdout] | [INFO] [stdout] 172 | let mut stdin = std::io::stdin(); [INFO] [stdout] | ----^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/main.rs:14:10 [INFO] [stdout] | [INFO] [stdout] 14 | Gold(u8), [INFO] [stdout] | ---- ^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Cell` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 14 | Gold(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/main.rs:14:10 [INFO] [stdout] | [INFO] [stdout] 14 | Gold(u8), [INFO] [stdout] | ---- ^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Cell` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 14 | Gold(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/main.rs:15:12 [INFO] [stdout] | [INFO] [stdout] 15 | Player(P), [INFO] [stdout] | ------ ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Cell` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 15 | Player(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/main.rs:15:12 [INFO] [stdout] | [INFO] [stdout] 15 | Player(P), [INFO] [stdout] | ------ ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Cell` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 15 | Player(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/main.rs:16:9 [INFO] [stdout] | [INFO] [stdout] 16 | Emu(P), [INFO] [stdout] | --- ^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Cell` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 16 | Emu(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/main.rs:16:9 [INFO] [stdout] | [INFO] [stdout] 16 | Emu(P), [INFO] [stdout] | --- ^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Cell` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 16 | Emu(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `emus` is never read [INFO] [stdout] --> src/main.rs:34:5 [INFO] [stdout] | [INFO] [stdout] 31 | struct Game { [INFO] [stdout] | ---- field in this struct [INFO] [stdout] ... [INFO] [stdout] 34 | emus: Vec>>, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Game` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `emus` is never read [INFO] [stdout] --> src/main.rs:34:5 [INFO] [stdout] | [INFO] [stdout] 31 | struct Game { [INFO] [stdout] | ---- field in this struct [INFO] [stdout] ... [INFO] [stdout] 34 | emus: Vec>>, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Game` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `hp`, `dam`, and `gold` are never read [INFO] [stdout] --> src/main.rs:78:5 [INFO] [stdout] | [INFO] [stdout] 76 | struct Player { [INFO] [stdout] | ------ fields in this struct [INFO] [stdout] 77 | pos: (usize, usize), [INFO] [stdout] 78 | hp: usize, [INFO] [stdout] | ^^ [INFO] [stdout] 79 | dam: usize, [INFO] [stdout] | ^^^ [INFO] [stdout] 80 | gold: usize, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Player` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `hp`, `dam`, and `gold` are never read [INFO] [stdout] --> src/main.rs:78:5 [INFO] [stdout] | [INFO] [stdout] 76 | struct Player { [INFO] [stdout] | ------ fields in this struct [INFO] [stdout] 77 | pos: (usize, usize), [INFO] [stdout] 78 | hp: usize, [INFO] [stdout] | ^^ [INFO] [stdout] 79 | dam: usize, [INFO] [stdout] | ^^^ [INFO] [stdout] 80 | gold: usize, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Player` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `hp` and `dam` are never read [INFO] [stdout] --> src/main.rs:85:5 [INFO] [stdout] | [INFO] [stdout] 84 | struct Emu { [INFO] [stdout] | --- fields in this struct [INFO] [stdout] 85 | hp: usize, [INFO] [stdout] | ^^ [INFO] [stdout] 86 | dam: usize, [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Emu` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `hp` and `dam` are never read [INFO] [stdout] --> src/main.rs:85:5 [INFO] [stdout] | [INFO] [stdout] 84 | struct Emu { [INFO] [stdout] | --- fields in this struct [INFO] [stdout] 85 | hp: usize, [INFO] [stdout] | ^^ [INFO] [stdout] 86 | dam: usize, [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Emu` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `P` should have a snake case name [INFO] [stdout] --> src/main.rs:6:4 [INFO] [stdout] | [INFO] [stdout] 6 | fn P(val: T) -> P { [INFO] [stdout] | ^ help: convert the identifier to snake case: `p` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `P` should have a snake case name [INFO] [stdout] --> src/main.rs:6:4 [INFO] [stdout] | [INFO] [stdout] 6 | fn P(val: T) -> P { [INFO] [stdout] | ^ help: convert the identifier to snake case: `p` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/main.rs:69:9 [INFO] [stdout] | [INFO] [stdout] 69 | / std::mem::replace(&mut self.map.data[oy * self.map.width + ox], [INFO] [stdout] 70 | | new); [INFO] [stdout] | |______________________________^ [INFO] [stdout] | [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 69 | let _ = std::mem::replace(&mut self.map.data[oy * self.map.width + ox], [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:169:5 [INFO] [stdout] | [INFO] [stdout] 169 | f.read_to_end(&mut loaded); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 169 | let _ = f.read_to_end(&mut loaded); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:178:9 [INFO] [stdout] | [INFO] [stdout] 178 | stdin.read_line(&mut line); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 178 | let _ = stdin.read_line(&mut line); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/main.rs:69:9 [INFO] [stdout] | [INFO] [stdout] 69 | / std::mem::replace(&mut self.map.data[oy * self.map.width + ox], [INFO] [stdout] 70 | | new); [INFO] [stdout] | |______________________________^ [INFO] [stdout] | [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 69 | let _ = std::mem::replace(&mut self.map.data[oy * self.map.width + ox], [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:169:5 [INFO] [stdout] | [INFO] [stdout] 169 | f.read_to_end(&mut loaded); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 169 | let _ = f.read_to_end(&mut loaded); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:178:9 [INFO] [stdout] | [INFO] [stdout] 178 | stdin.read_line(&mut line); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 178 | let _ = stdin.read_line(&mut line); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.07s [INFO] running `Command { std: "docker" "inspect" "bd59d1ddf329405bf62ece22f75d6943fa6815ede29354dafe811eb1a47c8ec4", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "bd59d1ddf329405bf62ece22f75d6943fa6815ede29354dafe811eb1a47c8ec4", kill_on_drop: false }` [INFO] [stdout] bd59d1ddf329405bf62ece22f75d6943fa6815ede29354dafe811eb1a47c8ec4