[INFO] cloning repository https://github.com/nsmryan/RustTcodTutorial [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/nsmryan/RustTcodTutorial" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnsmryan%2FRustTcodTutorial", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnsmryan%2FRustTcodTutorial'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 73c1f608292044de140475a5a3ee57179a101f23 [INFO] checking nsmryan/RustTcodTutorial against master#642c19bfc3a5c1de985bf5d0cc8207ac9d22708a for pr-148577 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnsmryan%2FRustTcodTutorial" "/workspace/builds/worker-5-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc1/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/nsmryan/RustTcodTutorial [INFO] finished tweaking git repo https://github.com/nsmryan/RustTcodTutorial [INFO] tweaked toml for git repo https://github.com/nsmryan/RustTcodTutorial written to /workspace/builds/worker-5-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/nsmryan/RustTcodTutorial on toolchain 642c19bfc3a5c1de985bf5d0cc8207ac9d22708a [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+642c19bfc3a5c1de985bf5d0cc8207ac9d22708a" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/nsmryan/RustTcodTutorial 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" "+642c19bfc3a5c1de985bf5d0cc8207ac9d22708a" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Updating crates.io index [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded parse-display v0.5.5 [INFO] [stderr] Downloaded parse-display-derive v0.5.5 [INFO] [stderr] Downloaded structmeta-derive v0.1.5 [INFO] [stderr] Downloaded structmeta v0.1.5 [INFO] [stderr] Downloaded fixedbitset v0.3.2 [INFO] [stderr] Downloaded logging_timer_proc_macros v1.1.0 [INFO] [stderr] Downloaded serde_yaml v0.8.23 [INFO] [stderr] Downloaded gumdrop v0.8.0 [INFO] [stderr] Downloaded bmp v0.5.0 [INFO] [stderr] Downloaded rmp v0.8.10 [INFO] [stderr] Downloaded logging_timer v1.1.0 [INFO] [stderr] Downloaded symmetric-shadowcasting v0.2.0 [INFO] [stderr] Downloaded gumdrop_derive v0.8.0 [INFO] [stderr] Downloaded wfc-rs v0.6.1 [INFO] [stderr] Downloaded pathfinding v2.0.5 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:4848fb76d95f26979359cc7e45710b1dbc8f3acb7aeedee7c460d7702230f228" "/opt/rustwide/cargo-home/bin/cargo" "+642c19bfc3a5c1de985bf5d0cc8207ac9d22708a" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] e861b56aed628c82166a58513f796f13d49de9308325ad56550488cf66c119cb [INFO] running `Command { std: "docker" "start" "-a" "e861b56aed628c82166a58513f796f13d49de9308325ad56550488cf66c119cb", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "e861b56aed628c82166a58513f796f13d49de9308325ad56550488cf66c119cb", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "e861b56aed628c82166a58513f796f13d49de9308325ad56550488cf66c119cb", kill_on_drop: false }` [INFO] [stdout] e861b56aed628c82166a58513f796f13d49de9308325ad56550488cf66c119cb [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:4848fb76d95f26979359cc7e45710b1dbc8f3acb7aeedee7c460d7702230f228" "/opt/rustwide/cargo-home/bin/cargo" "+642c19bfc3a5c1de985bf5d0cc8207ac9d22708a" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 7d86411e4bb13081d965232a30147929aa83d7f99af0c60451769e87115022ed [INFO] running `Command { std: "docker" "start" "-a" "7d86411e4bb13081d965232a30147929aa83d7f99af0c60451769e87115022ed", kill_on_drop: false }` [INFO] [stderr] Compiling syn v1.0.85 [INFO] [stderr] Compiling libc v0.2.112 [INFO] [stderr] Compiling serde_derive v1.0.133 [INFO] [stderr] Compiling indexmap v1.8.0 [INFO] [stderr] Compiling serde v1.0.133 [INFO] [stderr] Compiling regex-syntax v0.6.26 [INFO] [stderr] Compiling num-integer v0.1.44 [INFO] [stderr] Compiling once_cell v1.12.0 [INFO] [stderr] Checking yaml-rust v0.4.5 [INFO] [stderr] Checking fixedbitset v0.3.2 [INFO] [stderr] Compiling aho-corasick v0.7.18 [INFO] [stderr] Checking itertools v0.9.0 [INFO] [stderr] Compiling log v0.4.14 [INFO] [stderr] Checking smallvec v1.8.0 [INFO] [stderr] Compiling sdl2-sys v0.34.5 [INFO] [stderr] Compiling cc v1.0.72 [INFO] [stderr] Compiling sdl2 v0.34.5 [INFO] [stderr] Checking rmp v0.8.10 [INFO] [stderr] Checking bmp v0.5.0 [INFO] [stderr] Checking num-rational v0.3.2 [INFO] [stderr] Checking thread-id v3.3.0 [INFO] [stderr] Checking getrandom v0.2.4 [INFO] [stderr] Checking simple-logging v2.0.2 [INFO] [stderr] Checking rand_core v0.6.3 [INFO] [stderr] Checking pathfinding v2.0.5 [INFO] [stderr] Checking rand_chacha v0.3.1 [INFO] [stderr] Checking symmetric-shadowcasting v0.2.0 [INFO] [stderr] Compiling wfc-rs v0.6.1 [INFO] [stderr] Checking rand v0.8.4 [INFO] [stderr] Checking regex v1.5.6 [INFO] [stderr] Compiling structmeta-derive v0.1.5 [INFO] [stderr] Compiling logging_timer_proc_macros v1.1.0 [INFO] [stderr] Compiling gumdrop_derive v0.8.0 [INFO] [stderr] Checking logging_timer v1.1.0 [INFO] [stderr] Compiling structmeta v0.1.5 [INFO] [stderr] Checking gumdrop v0.8.0 [INFO] [stderr] Compiling parse-display-derive v0.5.5 [INFO] [stderr] Checking parse-display v0.5.5 [INFO] [stderr] Checking serde_yaml v0.8.23 [INFO] [stderr] Checking rmp-serde v0.15.5 [INFO] [stderr] Checking roguelike_utils v0.1.0 (/opt/rustwide/workdir/roguelike_utils) [INFO] [stderr] Checking roguelike_map v0.1.0 (/opt/rustwide/workdir/roguelike_map) [INFO] [stderr] Checking roguelike_core v0.1.0 (/opt/rustwide/workdir/roguelike_core) [INFO] [stderr] Checking roguelike_draw v0.1.0 (/opt/rustwide/workdir/roguelike_draw) [INFO] [stdout] warning: unused import: `std::str::FromStr` [INFO] [stdout] --> roguelike_draw/src/drawcmd.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use std::str::FromStr; [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::io::BufRead` [INFO] [stdout] --> roguelike_draw/src/drawcmd.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::io::BufRead; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SpriteKey` [INFO] [stdout] --> roguelike_draw/src/drawcmd.rs:16:32 [INFO] [stdout] | [INFO] [stdout] 16 | use crate::animation::{Sprite, SpriteKey}; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `lookup_spritekey` and `parse_atlas_file` [INFO] [stdout] --> roguelike_draw/src/drawcmd.rs:18:39 [INFO] [stdout] | [INFO] [stdout] 18 | use crate::spritesheet::{SpriteSheet, lookup_spritekey, parse_atlas_file}; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `PlayerDirection` [INFO] [stdout] --> roguelike_draw/src/spritesheet.rs:7:21 [INFO] [stdout] | [INFO] [stdout] 7 | use roguelike_map::{PlayerDirection}; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Sprite` [INFO] [stdout] --> roguelike_draw/src/spritesheet.rs:9:24 [INFO] [stdout] | [INFO] [stdout] 9 | use crate::animation::{Sprite, SpriteKey}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Panel` [INFO] [stdout] --> roguelike_draw/src/spritesheet.rs:10:20 [INFO] [stdout] | [INFO] [stdout] 10 | use crate::panel::{Panel}; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::str::FromStr` [INFO] [stdout] --> roguelike_draw/src/drawcmd.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use std::str::FromStr; [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::io::BufRead` [INFO] [stdout] --> roguelike_draw/src/drawcmd.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::io::BufRead; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SpriteKey` [INFO] [stdout] --> roguelike_draw/src/drawcmd.rs:16:32 [INFO] [stdout] | [INFO] [stdout] 16 | use crate::animation::{Sprite, SpriteKey}; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `lookup_spritekey` and `parse_atlas_file` [INFO] [stdout] --> roguelike_draw/src/drawcmd.rs:18:39 [INFO] [stdout] | [INFO] [stdout] 18 | use crate::spritesheet::{SpriteSheet, lookup_spritekey, parse_atlas_file}; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `PlayerDirection` [INFO] [stdout] --> roguelike_draw/src/spritesheet.rs:7:21 [INFO] [stdout] | [INFO] [stdout] 7 | use roguelike_map::{PlayerDirection}; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Sprite` [INFO] [stdout] --> roguelike_draw/src/spritesheet.rs:9:24 [INFO] [stdout] | [INFO] [stdout] 9 | use crate::animation::{Sprite, SpriteKey}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Panel` [INFO] [stdout] --> roguelike_draw/src/spritesheet.rs:10:20 [INFO] [stdout] | [INFO] [stdout] 10 | use crate::panel::{Panel}; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over an `Option`. This is more readably written as an `if let` statement [INFO] [stdout] --> roguelike_core/src/level.rs:100:41 [INFO] [stdout] | [INFO] [stdout] 100 | for offset in reach.move_with_reach(&direction) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 100 - for offset in reach.move_with_reach(&direction) { [INFO] [stdout] 100 + while let Some(offset) = reach.move_with_reach(&direction) { [INFO] [stdout] | [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 100 - for offset in reach.move_with_reach(&direction) { [INFO] [stdout] 100 + if let Some(offset) = reach.move_with_reach(&direction) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over an `Option`. This is more readably written as an `if let` statement [INFO] [stdout] --> roguelike_core/src/level.rs:100:41 [INFO] [stdout] | [INFO] [stdout] 100 | for offset in reach.move_with_reach(&direction) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 100 - for offset in reach.move_with_reach(&direction) { [INFO] [stdout] 100 + while let Some(offset) = reach.move_with_reach(&direction) { [INFO] [stdout] | [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 100 - for offset in reach.move_with_reach(&direction) { [INFO] [stdout] 100 + if let Some(offset) = reach.move_with_reach(&direction) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Checking roguelike_engine v0.1.0 (/opt/rustwide/workdir/roguelike_engine) [INFO] [stderr] Checking roguelike_display v0.1.0 (/opt/rustwide/workdir/roguelike_display) [INFO] [stdout] warning: unused import: `roguelike_core::utils::*` [INFO] [stdout] --> roguelike_engine/src/step.rs:18:5 [INFO] [stdout] | [INFO] [stdout] 18 | use roguelike_core::utils::*; [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: `roguelike_core::movement::*` [INFO] [stdout] --> roguelike_engine/src/step.rs:22:5 [INFO] [stdout] | [INFO] [stdout] 22 | use roguelike_core::movement::*; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `tile` is assigned to, but never used [INFO] [stdout] --> roguelike_display/src/display.rs:1524:17 [INFO] [stdout] | [INFO] [stdout] 1524 | let mut tile = summary_tiles[chr_index as usize]; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: consider using `_tile` instead [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `tile` is never read [INFO] [stdout] --> roguelike_display/src/display.rs:1525:13 [INFO] [stdout] | [INFO] [stdout] 1525 | tile.explored = explored; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `tile` is assigned to, but never used [INFO] [stdout] --> roguelike_display/src/display.rs:1524:17 [INFO] [stdout] | [INFO] [stdout] 1524 | let mut tile = summary_tiles[chr_index as usize]; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: consider using `_tile` instead [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `tile` is never read [INFO] [stdout] --> roguelike_display/src/display.rs:1525:13 [INFO] [stdout] | [INFO] [stdout] 1525 | tile.explored = explored; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> roguelike_display/src/render.rs:1216:21 [INFO] [stdout] | [INFO] [stdout] 1216 | let mut color = color; [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: variable `attack_highlight_color` is assigned to, but never used [INFO] [stdout] --> roguelike_display/src/render.rs:1487:13 [INFO] [stdout] | [INFO] [stdout] 1487 | let mut attack_highlight_color = config.color_red; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: consider using `_attack_highlight_color` instead [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `attack_highlight_color` is never read [INFO] [stdout] --> roguelike_display/src/render.rs:1488:9 [INFO] [stdout] | [INFO] [stdout] 1488 | attack_highlight_color.a = config.grid_alpha_overlay; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `highlight_color` is assigned to, but never used [INFO] [stdout] --> roguelike_display/src/render.rs:1724:9 [INFO] [stdout] | [INFO] [stdout] 1724 | let mut highlight_color: Color = config.color_warm_grey; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: consider using `_highlight_color` instead [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `highlight_color` is never read [INFO] [stdout] --> roguelike_display/src/render.rs:1725:5 [INFO] [stdout] | [INFO] [stdout] 1725 | highlight_color.a = config.highlight_player_move; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> roguelike_display/src/render.rs:1216:21 [INFO] [stdout] | [INFO] [stdout] 1216 | let mut color = color; [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: variable `attack_highlight_color` is assigned to, but never used [INFO] [stdout] --> roguelike_display/src/render.rs:1487:13 [INFO] [stdout] | [INFO] [stdout] 1487 | let mut attack_highlight_color = config.color_red; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: consider using `_attack_highlight_color` instead [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `attack_highlight_color` is never read [INFO] [stdout] --> roguelike_display/src/render.rs:1488:9 [INFO] [stdout] | [INFO] [stdout] 1488 | attack_highlight_color.a = config.grid_alpha_overlay; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `highlight_color` is assigned to, but never used [INFO] [stdout] --> roguelike_display/src/render.rs:1724:9 [INFO] [stdout] | [INFO] [stdout] 1724 | let mut highlight_color: Color = config.color_warm_grey; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: consider using `_highlight_color` instead [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `highlight_color` is never read [INFO] [stdout] --> roguelike_display/src/render.rs:1725:5 [INFO] [stdout] | [INFO] [stdout] 1725 | highlight_color.a = config.highlight_player_move; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `index` [INFO] [stdout] --> roguelike_engine/src/map_construct.rs:143:17 [INFO] [stdout] | [INFO] [stdout] 143 | for index in 0..1000 { [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_index` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `index` [INFO] [stdout] --> roguelike_engine/src/map_construct.rs:143:17 [INFO] [stdout] | [INFO] [stdout] 143 | for index in 0..1000 { [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_index` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> roguelike_engine/src/resolve.rs:204:25 [INFO] [stdout] | [INFO] [stdout] 204 | if let Some(mut status) = game.level.entities.status.get_mut(&entity_id) { [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: variable does not need to be mutable [INFO] [stdout] --> roguelike_engine/src/resolve.rs:204:25 [INFO] [stdout] | [INFO] [stdout] 204 | if let Some(mut status) = game.level.entities.status.get_mut(&entity_id) { [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: variable does not need to be mutable [INFO] [stdout] --> roguelike_engine/src/step.rs:91:21 [INFO] [stdout] | [INFO] [stdout] 91 | if let Some(mut status) = game.level.entities.status.get_mut(entity_id) { [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `col` [INFO] [stdout] --> roguelike_engine/src/step.rs:525:9 [INFO] [stdout] | [INFO] [stdout] 525 | let col = make_column(&mut game.level.entities, &game.config, col_pos, &mut game.msg_log); [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_col` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> roguelike_engine/src/step.rs:91:21 [INFO] [stdout] | [INFO] [stdout] 91 | if let Some(mut status) = game.level.entities.status.get_mut(entity_id) { [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over an `Option`. This is more readably written as an `if let` statement [INFO] [stdout] --> roguelike_engine/src/resolve.rs:1052:25 [INFO] [stdout] | [INFO] [stdout] 1052 | for other_id in level.has_entity(interact_pos) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 1052 - for other_id in level.has_entity(interact_pos) { [INFO] [stdout] 1052 + while let Some(other_id) = level.has_entity(interact_pos) { [INFO] [stdout] | [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 1052 - for other_id in level.has_entity(interact_pos) { [INFO] [stdout] 1052 + if let Some(other_id) = level.has_entity(interact_pos) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over an `Option`. This is more readably written as an `if let` statement [INFO] [stdout] --> roguelike_engine/src/resolve.rs:1052:25 [INFO] [stdout] | [INFO] [stdout] 1052 | for other_id in level.has_entity(interact_pos) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 1052 - for other_id in level.has_entity(interact_pos) { [INFO] [stdout] 1052 + while let Some(other_id) = level.has_entity(interact_pos) { [INFO] [stdout] | [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 1052 - for other_id in level.has_entity(interact_pos) { [INFO] [stdout] 1052 + if let Some(other_id) = level.has_entity(interact_pos) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Checking roguelike_lib v0.1.0 (/opt/rustwide/workdir/roguelike_lib) [INFO] [stderr] Checking roguelike_main v0.1.0 (/opt/rustwide/workdir/roguelike_main) [INFO] [stdout] warning: unused import: `roguelike_core::movement::*` [INFO] [stdout] --> roguelike_main/src/replay.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | use roguelike_core::movement::*; [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 variable: `display_state` [INFO] [stdout] --> roguelike_main/src/replay.rs:124:16 [INFO] [stdout] | [INFO] [stdout] 124 | let (game, display_state) = recording.backward(); [INFO] [stdout] | ^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_display_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: `display_state` [INFO] [stdout] --> roguelike_main/src/replay.rs:144:16 [INFO] [stdout] | [INFO] [stdout] 144 | let (game, display_state) = recording.forward().unwrap(); [INFO] [stdout] | ^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_display_state` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `display_state` [INFO] [stdout] --> roguelike_main/src/replay.rs:150:16 [INFO] [stdout] | [INFO] [stdout] 150 | let (game, display_state) = recording.forward().unwrap(); [INFO] [stdout] | ^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_display_state` [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 22.12s [INFO] running `Command { std: "docker" "inspect" "7d86411e4bb13081d965232a30147929aa83d7f99af0c60451769e87115022ed", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "7d86411e4bb13081d965232a30147929aa83d7f99af0c60451769e87115022ed", kill_on_drop: false }` [INFO] [stdout] 7d86411e4bb13081d965232a30147929aa83d7f99af0c60451769e87115022ed