[INFO] cloning repository https://github.com/TartanLlama/lines-of-action [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/TartanLlama/lines-of-action" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FTartanLlama%2Flines-of-action", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FTartanLlama%2Flines-of-action'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 4d757dd5a5f6b2ab72074cfd64eaaa64318ce220 [INFO] testing TartanLlama/lines-of-action against beta-2023-10-04 for beta-1.74-1 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FTartanLlama%2Flines-of-action" "/workspace/builds/worker-7-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc2/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/TartanLlama/lines-of-action on toolchain beta-2023-10-04 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+beta-2023-10-04" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/TartanLlama/lines-of-action [INFO] finished tweaking git repo https://github.com/TartanLlama/lines-of-action [INFO] tweaked toml for git repo https://github.com/TartanLlama/lines-of-action written to /workspace/builds/worker-7-tc2/source/Cargo.toml [INFO] crate git repo https://github.com/TartanLlama/lines-of-action 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" "+beta-2023-10-04" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/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:4f8da3c744d2da974e54ce020a79f4ad54313e914ebeaab8fc086f07e8d8b066" "/opt/rustwide/cargo-home/bin/cargo" "+beta-2023-10-04" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 5160001f9d3244786b9e25ae9cb684e2b579fd114d92e1aa6626fdc5584d747d [INFO] running `Command { std: "docker" "start" "-a" "5160001f9d3244786b9e25ae9cb684e2b579fd114d92e1aa6626fdc5584d747d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "5160001f9d3244786b9e25ae9cb684e2b579fd114d92e1aa6626fdc5584d747d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "5160001f9d3244786b9e25ae9cb684e2b579fd114d92e1aa6626fdc5584d747d", kill_on_drop: false }` [INFO] [stdout] 5160001f9d3244786b9e25ae9cb684e2b579fd114d92e1aa6626fdc5584d747d [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/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=warn" "-e" "RUSTDOCFLAGS=--cap-lints=warn" "-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:4f8da3c744d2da974e54ce020a79f4ad54313e914ebeaab8fc086f07e8d8b066" "/opt/rustwide/cargo-home/bin/cargo" "+beta-2023-10-04" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 28e80010bca9205e5c6d0772b3524464f4f03c39b2331f1140d6f8fce541a3f0 [INFO] running `Command { std: "docker" "start" "-a" "28e80010bca9205e5c6d0772b3524464f4f03c39b2331f1140d6f8fce541a3f0", kill_on_drop: false }` [INFO] [stderr] Compiling libc v0.1.10 [INFO] [stderr] Compiling advapi32-sys v0.1.2 [INFO] [stderr] Compiling winapi v0.2.2 [INFO] [stderr] Compiling regex-syntax v0.2.1 [INFO] [stderr] Compiling memchr v0.1.6 [INFO] [stderr] Compiling log v0.3.2 [INFO] [stderr] Compiling aho-corasick v0.3.2 [INFO] [stderr] Compiling rand v0.3.11 [INFO] [stderr] Compiling regex v0.1.41 [INFO] [stderr] Compiling env_logger v0.3.1 [INFO] [stderr] Compiling sfml v0.9.3 [INFO] [stderr] Compiling lines-of-action v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused imports: `Display`, `Error`, `Formatter` [INFO] [stdout] --> src/board.rs:2:16 [INFO] [stdout] | [INFO] [stdout] 2 | use std::fmt::{Formatter,Error,Display}; [INFO] [stdout] | ^^^^^^^^^ ^^^^^ ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SocketAddr` [INFO] [stdout] --> src/server.rs:44:40 [INFO] [stdout] | [INFO] [stdout] 44 | use std::net::{TcpListener, TcpStream, SocketAddr}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Captures` [INFO] [stdout] --> src/server.rs:47:25 [INFO] [stdout] | [INFO] [stdout] 47 | use self::regex::{Regex,Captures}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Captures` [INFO] [stdout] --> src/api.rs:3:25 [INFO] [stdout] | [INFO] [stdout] 3 | use self::regex::{Regex,Captures}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SocketAddr` [INFO] [stdout] --> src/api.rs:4:27 [INFO] [stdout] | [INFO] [stdout] 4 | use std::net::{TcpStream, SocketAddr, ToSocketAddrs}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `handle` is never read [INFO] [stdout] --> src/server.rs:180:5 [INFO] [stdout] | [INFO] [stdout] 178 | pub struct GameServer { [INFO] [stdout] | ---------- field in this struct [INFO] [stdout] 179 | command_receiver: Receiver, [INFO] [stdout] 180 | handle: thread::JoinHandle<()>, [INFO] [stdout] | ^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:82:15 [INFO] [stdout] | [INFO] [stdout] 82 | { stream.write(b"Move successful;\n"); }, [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)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 82 | { let _ = stream.write(b"Move successful;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:84:15 [INFO] [stdout] | [INFO] [stdout] 84 | { stream.write(b"You can't move air, ya numpty;\n"); }, [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] 84 | { let _ = stream.write(b"You can't move air, ya numpty;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:86:15 [INFO] [stdout] | [INFO] [stdout] 86 | { stream.write(b"Stick to your own pieces!;\n"); }, [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] 86 | { let _ = stream.write(b"Stick to your own pieces!;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:101:15 [INFO] [stdout] | [INFO] [stdout] 101 | { stream.write(format!("Oh, hai {}!;\n",name).as_bytes()); }, [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] 101 | { let _ = stream.write(format!("Oh, hai {}!;\n",name).as_bytes()); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:103:15 [INFO] [stdout] | [INFO] [stdout] 103 | { stream.write(b"You've already registered, you asshat;\n"); }, [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] 103 | { let _ = stream.write(b"You've already registered, you asshat;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:105:15 [INFO] [stdout] | [INFO] [stdout] 105 | { stream.write(b"The game is full, get tae;\n"); }, [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] 105 | { let _ = stream.write(b"The game is full, get tae;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:106:49 [INFO] [stdout] | [INFO] [stdout] 106 | Err(CommandErr::Other(s)) => panic!(s), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 106 | Err(CommandErr::Other(s)) => panic!("{}", s), [INFO] [stdout] | +++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:146:15 [INFO] [stdout] | [INFO] [stdout] 146 | { stream.write(format!("You are {:?} and {} wants to batter you;\n",col,opponent).as_bytes()); }, [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] 146 | { let _ = stream.write(format!("You are {:?} and {} wants to batter you;\n",col,opponent).as_bytes()); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:148:15 [INFO] [stdout] | [INFO] [stdout] 148 | { stream.write(b"Who even are you?;\n"); }, [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] 148 | { let _ = stream.write(b"Who even are you?;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:156:9 [INFO] [stdout] | [INFO] [stdout] 156 | stream.write(b"You wot m8?;\n"); [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] 156 | let _ = stream.write(b"You wot m8?;\n"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:173:30 [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => panic!(e), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => panic!("{}", e), [INFO] [stdout] | +++++ [INFO] [stdout] help: or use std::panic::panic_any instead [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => std::panic::panic_any(e), [INFO] [stdout] | ~~~~~~~~~~~~~~~~~~~~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:201:40 [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { panic!(e); } [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { panic!("{}", e); } [INFO] [stdout] | +++++ [INFO] [stdout] help: or use std::panic::panic_any instead [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { std::panic::panic_any(e); } [INFO] [stdout] | ~~~~~~~~~~~~~~~~~~~~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:23:9 [INFO] [stdout] | [INFO] [stdout] 23 | stream.read(&mut c); [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] 23 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:26:13 [INFO] [stdout] | [INFO] [stdout] 26 | stream.read(&mut c); [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] 26 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:29:9 [INFO] [stdout] | [INFO] [stdout] 29 | stream.read(&mut c); [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] 29 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 21 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around method argument [INFO] [stdout] --> src/main.rs:179:40 [INFO] [stdout] | [INFO] [stdout] 179 | self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 179 - self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stdout] 179 + self.players.insert(id,Player::new(name.to_string())); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:42:13 [INFO] [stdout] | [INFO] [stdout] 42 | let mut window = match RenderWindow::new(VideoMode::new_init(SCREEN_SIZE, SCREEN_SIZE, 32), [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:50:13 [INFO] [stdout] | [INFO] [stdout] 50 | let mut board = Board::new(); [INFO] [stdout] | ----^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:51:13 [INFO] [stdout] | [INFO] [stdout] 51 | let mut players = BTreeMap::new(); [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `id` [INFO] [stdout] --> src/main.rs:87:46 [INFO] [stdout] | [INFO] [stdout] 87 | CommandData::Message(id,msg) => [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_id` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `id` [INFO] [stdout] --> src/main.rs:204:14 [INFO] [stdout] | [INFO] [stdout] 204 | for (id, p) in &self.players { [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_id` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `turn` and `game_started` are never read [INFO] [stdout] --> src/main.rs:26:5 [INFO] [stdout] | [INFO] [stdout] 21 | struct Engine { [INFO] [stdout] | ------ fields in this struct [INFO] [stdout] ... [INFO] [stdout] 26 | turn: Colour, [INFO] [stdout] | ^^^^ [INFO] [stdout] 27 | game_started: bool, [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/main.rs:95:59 [INFO] [stdout] | [INFO] [stdout] 95 | Err(GetCommandErr::Failed(why)) => panic!(why), [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 95 | Err(GetCommandErr::Failed(why)) => panic!("{}", why), [INFO] [stdout] | +++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/main.rs:104:34 [INFO] [stdout] | [INFO] [stdout] 104 | Err(s) => panic!(s), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 104 | Err(s) => panic!("{}", s), [INFO] [stdout] | +++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:141:9 [INFO] [stdout] | [INFO] [stdout] 141 | / reply.send(match self.board.move_piece(&mov,&player_colour) { [INFO] [stdout] 142 | | Ok(()) => { [INFO] [stdout] 143 | | Ok(CommandOk::None) [INFO] [stdout] 144 | | }, [INFO] [stdout] 145 | | Err(MoveErr::NoPiece) => Err(CommandErr::NoPiece), [INFO] [stdout] 146 | | Err(MoveErr::WrongPiece) => Err(CommandErr::WrongPiece), [INFO] [stdout] 147 | | }); [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)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 141 | let _ = reply.send(match self.board.move_piece(&mov,&player_colour) { [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:156:21 [INFO] [stdout] | [INFO] [stdout] 156 | other.reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [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] 156 | let _ = other.reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:173:13 [INFO] [stdout] | [INFO] [stdout] 173 | reply.send(Err(CommandErr::AlreadyRegistered)); [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] 173 | let _ = reply.send(Err(CommandErr::AlreadyRegistered)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:177:9 [INFO] [stdout] | [INFO] [stdout] 177 | / reply.send( [INFO] [stdout] 178 | | if self.players.len() < 2 { [INFO] [stdout] 179 | | self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stdout] 180 | | debug!("Registered new player: {}", name); [INFO] [stdout] ... | [INFO] [stdout] 183 | | Err(CommandErr::GameFull) [INFO] [stdout] 184 | | }); [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] 177 | let _ = reply.send( [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:189:9 [INFO] [stdout] | [INFO] [stdout] 189 | reply.send(Ok(CommandOk::None)); [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] 189 | let _ = reply.send(Ok(CommandOk::None)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:195:23 [INFO] [stdout] | [INFO] [stdout] 195 | None => { reply.send(Err(CommandErr::NotRegistered)); return; }, [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] 195 | None => { let _ = reply.send(Err(CommandErr::NotRegistered)); return; }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:218:13 [INFO] [stdout] | [INFO] [stdout] 218 | p1.reply.clone().unwrap().send(Ok(CommandOk::Ready(p2_name, p1.colour.clone()))); [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] 218 | let _ = p1.reply.clone().unwrap().send(Ok(CommandOk::Ready(p2_name, p1.colour.clone()))); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:225:13 [INFO] [stdout] | [INFO] [stdout] 225 | p2.reply.clone().unwrap().send(Ok(CommandOk::Ready(p1_name, p2.colour))); [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] 225 | let _ = p2.reply.clone().unwrap().send(Ok(CommandOk::Ready(p1_name, p2.colour))); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:239:25 [INFO] [stdout] | [INFO] [stdout] 239 | self.players.get(&i).unwrap().reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [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] 239 | let _ = self.players.get(&i).unwrap().reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 18 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished dev [unoptimized + debuginfo] target(s) in 2.22s [INFO] running `Command { std: "docker" "inspect" "28e80010bca9205e5c6d0772b3524464f4f03c39b2331f1140d6f8fce541a3f0", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "28e80010bca9205e5c6d0772b3524464f4f03c39b2331f1140d6f8fce541a3f0", kill_on_drop: false }` [INFO] [stdout] 28e80010bca9205e5c6d0772b3524464f4f03c39b2331f1140d6f8fce541a3f0 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/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=warn" "-e" "RUSTDOCFLAGS=--cap-lints=warn" "-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:4f8da3c744d2da974e54ce020a79f4ad54313e914ebeaab8fc086f07e8d8b066" "/opt/rustwide/cargo-home/bin/cargo" "+beta-2023-10-04" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] c61b50c9da069a6f23ab4be2b4dacfebb7baf8a615a472d37097330cf7492c95 [INFO] running `Command { std: "docker" "start" "-a" "c61b50c9da069a6f23ab4be2b4dacfebb7baf8a615a472d37097330cf7492c95", kill_on_drop: false }` [INFO] [stdout] warning: unused imports: `Display`, `Error`, `Formatter` [INFO] [stdout] --> src/board.rs:2:16 [INFO] [stdout] | [INFO] [stdout] 2 | use std::fmt::{Formatter,Error,Display}; [INFO] [stdout] | ^^^^^^^^^ ^^^^^ ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SocketAddr` [INFO] [stdout] --> src/server.rs:44:40 [INFO] [stdout] | [INFO] [stdout] 44 | use std::net::{TcpListener, TcpStream, SocketAddr}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Captures` [INFO] [stdout] --> src/server.rs:47:25 [INFO] [stdout] | [INFO] [stdout] 47 | use self::regex::{Regex,Captures}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Captures` [INFO] [stdout] --> src/api.rs:3:25 [INFO] [stdout] | [INFO] [stdout] 3 | use self::regex::{Regex,Captures}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SocketAddr` [INFO] [stdout] --> src/api.rs:4:27 [INFO] [stdout] | [INFO] [stdout] 4 | use std::net::{TcpStream, SocketAddr, ToSocketAddrs}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `handle` is never read [INFO] [stdout] --> src/server.rs:180:5 [INFO] [stdout] | [INFO] [stdout] 178 | pub struct GameServer { [INFO] [stdout] | ---------- field in this struct [INFO] [stdout] 179 | command_receiver: Receiver, [INFO] [stdout] 180 | handle: thread::JoinHandle<()>, [INFO] [stdout] | ^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:82:15 [INFO] [stdout] | [INFO] [stdout] 82 | { stream.write(b"Move successful;\n"); }, [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)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 82 | { let _ = stream.write(b"Move successful;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:84:15 [INFO] [stdout] | [INFO] [stdout] 84 | { stream.write(b"You can't move air, ya numpty;\n"); }, [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] 84 | { let _ = stream.write(b"You can't move air, ya numpty;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:86:15 [INFO] [stdout] | [INFO] [stdout] 86 | { stream.write(b"Stick to your own pieces!;\n"); }, [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] 86 | { let _ = stream.write(b"Stick to your own pieces!;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:101:15 [INFO] [stdout] | [INFO] [stdout] 101 | { stream.write(format!("Oh, hai {}!;\n",name).as_bytes()); }, [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] 101 | { let _ = stream.write(format!("Oh, hai {}!;\n",name).as_bytes()); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:103:15 [INFO] [stdout] | [INFO] [stdout] 103 | { stream.write(b"You've already registered, you asshat;\n"); }, [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] 103 | { let _ = stream.write(b"You've already registered, you asshat;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:105:15 [INFO] [stdout] | [INFO] [stdout] 105 | { stream.write(b"The game is full, get tae;\n"); }, [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] 105 | { let _ = stream.write(b"The game is full, get tae;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:106:49 [INFO] [stdout] | [INFO] [stdout] 106 | Err(CommandErr::Other(s)) => panic!(s), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 106 | Err(CommandErr::Other(s)) => panic!("{}", s), [INFO] [stdout] | +++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:146:15 [INFO] [stdout] | [INFO] [stdout] 146 | { stream.write(format!("You are {:?} and {} wants to batter you;\n",col,opponent).as_bytes()); }, [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] 146 | { let _ = stream.write(format!("You are {:?} and {} wants to batter you;\n",col,opponent).as_bytes()); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:148:15 [INFO] [stdout] | [INFO] [stdout] 148 | { stream.write(b"Who even are you?;\n"); }, [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] 148 | { let _ = stream.write(b"Who even are you?;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:156:9 [INFO] [stdout] | [INFO] [stdout] 156 | stream.write(b"You wot m8?;\n"); [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] 156 | let _ = stream.write(b"You wot m8?;\n"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:173:30 [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => panic!(e), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => panic!("{}", e), [INFO] [stdout] | +++++ [INFO] [stdout] help: or use std::panic::panic_any instead [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => std::panic::panic_any(e), [INFO] [stdout] | ~~~~~~~~~~~~~~~~~~~~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:201:40 [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { panic!(e); } [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { panic!("{}", e); } [INFO] [stdout] | +++++ [INFO] [stdout] help: or use std::panic::panic_any instead [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { std::panic::panic_any(e); } [INFO] [stdout] | ~~~~~~~~~~~~~~~~~~~~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:23:9 [INFO] [stdout] | [INFO] [stdout] 23 | stream.read(&mut c); [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] 23 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:26:13 [INFO] [stdout] | [INFO] [stdout] 26 | stream.read(&mut c); [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] 26 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:29:9 [INFO] [stdout] | [INFO] [stdout] 29 | stream.read(&mut c); [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] 29 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Compiling lines-of-action v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: 21 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Display`, `Error`, `Formatter` [INFO] [stdout] --> src/board.rs:2:16 [INFO] [stdout] | [INFO] [stdout] 2 | use std::fmt::{Formatter,Error,Display}; [INFO] [stdout] | ^^^^^^^^^ ^^^^^ ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SocketAddr` [INFO] [stdout] --> src/server.rs:44:40 [INFO] [stdout] | [INFO] [stdout] 44 | use std::net::{TcpListener, TcpStream, SocketAddr}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Captures` [INFO] [stdout] --> src/server.rs:47:25 [INFO] [stdout] | [INFO] [stdout] 47 | use self::regex::{Regex,Captures}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Captures` [INFO] [stdout] --> src/api.rs:3:25 [INFO] [stdout] | [INFO] [stdout] 3 | use self::regex::{Regex,Captures}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SocketAddr` [INFO] [stdout] --> src/api.rs:4:27 [INFO] [stdout] | [INFO] [stdout] 4 | use std::net::{TcpStream, SocketAddr, ToSocketAddrs}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around method argument [INFO] [stdout] --> src/main.rs:179:40 [INFO] [stdout] | [INFO] [stdout] 179 | self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 179 - self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stdout] 179 + self.players.insert(id,Player::new(name.to_string())); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:42:13 [INFO] [stdout] | [INFO] [stdout] 42 | let mut window = match RenderWindow::new(VideoMode::new_init(SCREEN_SIZE, SCREEN_SIZE, 32), [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:50:13 [INFO] [stdout] | [INFO] [stdout] 50 | let mut board = Board::new(); [INFO] [stdout] | ----^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:51:13 [INFO] [stdout] | [INFO] [stdout] 51 | let mut players = BTreeMap::new(); [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `id` [INFO] [stdout] --> src/main.rs:87:46 [INFO] [stdout] | [INFO] [stdout] 87 | CommandData::Message(id,msg) => [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_id` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `id` [INFO] [stdout] --> src/main.rs:204:14 [INFO] [stdout] | [INFO] [stdout] 204 | for (id, p) in &self.players { [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_id` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `turn` and `game_started` are never read [INFO] [stdout] --> src/main.rs:26:5 [INFO] [stdout] | [INFO] [stdout] 21 | struct Engine { [INFO] [stdout] | ------ fields in this struct [INFO] [stdout] ... [INFO] [stdout] 26 | turn: Colour, [INFO] [stdout] | ^^^^ [INFO] [stdout] 27 | game_started: bool, [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/main.rs:95:59 [INFO] [stdout] | [INFO] [stdout] 95 | Err(GetCommandErr::Failed(why)) => panic!(why), [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 95 | Err(GetCommandErr::Failed(why)) => panic!("{}", why), [INFO] [stdout] | +++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/main.rs:104:34 [INFO] [stdout] | [INFO] [stdout] 104 | Err(s) => panic!(s), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 104 | Err(s) => panic!("{}", s), [INFO] [stdout] | +++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:141:9 [INFO] [stdout] | [INFO] [stdout] 141 | / reply.send(match self.board.move_piece(&mov,&player_colour) { [INFO] [stdout] 142 | | Ok(()) => { [INFO] [stdout] 143 | | Ok(CommandOk::None) [INFO] [stdout] 144 | | }, [INFO] [stdout] 145 | | Err(MoveErr::NoPiece) => Err(CommandErr::NoPiece), [INFO] [stdout] 146 | | Err(MoveErr::WrongPiece) => Err(CommandErr::WrongPiece), [INFO] [stdout] 147 | | }); [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)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 141 | let _ = reply.send(match self.board.move_piece(&mov,&player_colour) { [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:156:21 [INFO] [stdout] | [INFO] [stdout] 156 | other.reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [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] 156 | let _ = other.reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:173:13 [INFO] [stdout] | [INFO] [stdout] 173 | reply.send(Err(CommandErr::AlreadyRegistered)); [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] 173 | let _ = reply.send(Err(CommandErr::AlreadyRegistered)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:177:9 [INFO] [stdout] | [INFO] [stdout] 177 | / reply.send( [INFO] [stdout] 178 | | if self.players.len() < 2 { [INFO] [stdout] 179 | | self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stdout] 180 | | debug!("Registered new player: {}", name); [INFO] [stdout] ... | [INFO] [stdout] 183 | | Err(CommandErr::GameFull) [INFO] [stdout] 184 | | }); [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] 177 | let _ = reply.send( [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:189:9 [INFO] [stdout] | [INFO] [stdout] 189 | reply.send(Ok(CommandOk::None)); [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] 189 | let _ = reply.send(Ok(CommandOk::None)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:195:23 [INFO] [stdout] | [INFO] [stdout] 195 | None => { reply.send(Err(CommandErr::NotRegistered)); return; }, [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] 195 | None => { let _ = reply.send(Err(CommandErr::NotRegistered)); return; }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:218:13 [INFO] [stdout] | [INFO] [stdout] 218 | p1.reply.clone().unwrap().send(Ok(CommandOk::Ready(p2_name, p1.colour.clone()))); [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] 218 | let _ = p1.reply.clone().unwrap().send(Ok(CommandOk::Ready(p2_name, p1.colour.clone()))); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:225:13 [INFO] [stdout] | [INFO] [stdout] 225 | p2.reply.clone().unwrap().send(Ok(CommandOk::Ready(p1_name, p2.colour))); [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] 225 | let _ = p2.reply.clone().unwrap().send(Ok(CommandOk::Ready(p1_name, p2.colour))); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:239:25 [INFO] [stdout] | [INFO] [stdout] 239 | self.players.get(&i).unwrap().reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [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] 239 | let _ = self.players.get(&i).unwrap().reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `handle` is never read [INFO] [stdout] --> src/server.rs:180:5 [INFO] [stdout] | [INFO] [stdout] 178 | pub struct GameServer { [INFO] [stdout] | ---------- field in this struct [INFO] [stdout] 179 | command_receiver: Receiver, [INFO] [stdout] 180 | handle: thread::JoinHandle<()>, [INFO] [stdout] | ^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:82:15 [INFO] [stdout] | [INFO] [stdout] 82 | { stream.write(b"Move successful;\n"); }, [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)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 82 | { let _ = stream.write(b"Move successful;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:84:15 [INFO] [stdout] | [INFO] [stdout] 84 | { stream.write(b"You can't move air, ya numpty;\n"); }, [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] 84 | { let _ = stream.write(b"You can't move air, ya numpty;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:86:15 [INFO] [stdout] | [INFO] [stdout] 86 | { stream.write(b"Stick to your own pieces!;\n"); }, [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] 86 | { let _ = stream.write(b"Stick to your own pieces!;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:101:15 [INFO] [stdout] | [INFO] [stdout] 101 | { stream.write(format!("Oh, hai {}!;\n",name).as_bytes()); }, [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] 101 | { let _ = stream.write(format!("Oh, hai {}!;\n",name).as_bytes()); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:103:15 [INFO] [stdout] | [INFO] [stdout] 103 | { stream.write(b"You've already registered, you asshat;\n"); }, [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] 103 | { let _ = stream.write(b"You've already registered, you asshat;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:105:15 [INFO] [stdout] | [INFO] [stdout] 105 | { stream.write(b"The game is full, get tae;\n"); }, [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] 105 | { let _ = stream.write(b"The game is full, get tae;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:106:49 [INFO] [stdout] | [INFO] [stdout] 106 | Err(CommandErr::Other(s)) => panic!(s), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 106 | Err(CommandErr::Other(s)) => panic!("{}", s), [INFO] [stdout] | +++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:146:15 [INFO] [stdout] | [INFO] [stdout] 146 | { stream.write(format!("You are {:?} and {} wants to batter you;\n",col,opponent).as_bytes()); }, [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] 146 | { let _ = stream.write(format!("You are {:?} and {} wants to batter you;\n",col,opponent).as_bytes()); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:148:15 [INFO] [stdout] | [INFO] [stdout] 148 | { stream.write(b"Who even are you?;\n"); }, [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] 148 | { let _ = stream.write(b"Who even are you?;\n"); }, [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:156:9 [INFO] [stdout] | [INFO] [stdout] 156 | stream.write(b"You wot m8?;\n"); [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] 156 | let _ = stream.write(b"You wot m8?;\n"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:173:30 [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => panic!(e), [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => panic!("{}", e), [INFO] [stdout] | +++++ [INFO] [stdout] help: or use std::panic::panic_any instead [INFO] [stdout] | [INFO] [stdout] 173 | Err(e) => std::panic::panic_any(e), [INFO] [stdout] | ~~~~~~~~~~~~~~~~~~~~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/server.rs:201:40 [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { panic!(e); } [INFO] [stdout] | ^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: add a "{}" format string to `Display` the message [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { panic!("{}", e); } [INFO] [stdout] | +++++ [INFO] [stdout] help: or use std::panic::panic_any instead [INFO] [stdout] | [INFO] [stdout] 201 | Err(e) => { std::panic::panic_any(e); } [INFO] [stdout] | ~~~~~~~~~~~~~~~~~~~~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:23:9 [INFO] [stdout] | [INFO] [stdout] 23 | stream.read(&mut c); [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] 23 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:26:13 [INFO] [stdout] | [INFO] [stdout] 26 | stream.read(&mut c); [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] 26 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/api.rs:29:9 [INFO] [stdout] | [INFO] [stdout] 29 | stream.read(&mut c); [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] 29 | let _ = stream.read(&mut c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 18 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 21 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished test [unoptimized + debuginfo] target(s) in 0.35s [INFO] running `Command { std: "docker" "inspect" "c61b50c9da069a6f23ab4be2b4dacfebb7baf8a615a472d37097330cf7492c95", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "c61b50c9da069a6f23ab4be2b4dacfebb7baf8a615a472d37097330cf7492c95", kill_on_drop: false }` [INFO] [stdout] c61b50c9da069a6f23ab4be2b4dacfebb7baf8a615a472d37097330cf7492c95 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/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=warn" "-e" "RUSTDOCFLAGS=--cap-lints=warn" "-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:4f8da3c744d2da974e54ce020a79f4ad54313e914ebeaab8fc086f07e8d8b066" "/opt/rustwide/cargo-home/bin/cargo" "+beta-2023-10-04" "test" "--frozen", kill_on_drop: false }` [INFO] [stdout] 6b900caa1da47b70af65f8d8a4f84380375fd46137090b9c289038029ec9574d [INFO] running `Command { std: "docker" "start" "-a" "6b900caa1da47b70af65f8d8a4f84380375fd46137090b9c289038029ec9574d", kill_on_drop: false }` [INFO] [stderr] warning: unused imports: `Display`, `Error`, `Formatter` [INFO] [stderr] --> src/board.rs:2:16 [INFO] [stderr] | [INFO] [stderr] 2 | use std::fmt::{Formatter,Error,Display}; [INFO] [stderr] | ^^^^^^^^^ ^^^^^ ^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_imports)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused import: `SocketAddr` [INFO] [stderr] --> src/server.rs:44:40 [INFO] [stderr] | [INFO] [stderr] 44 | use std::net::{TcpListener, TcpStream, SocketAddr}; [INFO] [stderr] | ^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused import: `Captures` [INFO] [stderr] --> src/server.rs:47:25 [INFO] [stderr] | [INFO] [stderr] 47 | use self::regex::{Regex,Captures}; [INFO] [stderr] | ^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused import: `Captures` [INFO] [stderr] --> src/api.rs:3:25 [INFO] [stderr] | [INFO] [stderr] 3 | use self::regex::{Regex,Captures}; [INFO] [stderr] | ^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused import: `SocketAddr` [INFO] [stderr] --> src/api.rs:4:27 [INFO] [stderr] | [INFO] [stderr] 4 | use std::net::{TcpStream, SocketAddr, ToSocketAddrs}; [INFO] [stderr] | ^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: field `handle` is never read [INFO] [stderr] --> src/server.rs:180:5 [INFO] [stderr] | [INFO] [stderr] 178 | pub struct GameServer { [INFO] [stderr] | ---------- field in this struct [INFO] [stderr] 179 | command_receiver: Receiver, [INFO] [stderr] 180 | handle: thread::JoinHandle<()>, [INFO] [stderr] | ^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(dead_code)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:82:15 [INFO] [stderr] | [INFO] [stderr] 82 | { stream.write(b"Move successful;\n"); }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] = note: `#[warn(unused_must_use)]` on by default [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 82 | { let _ = stream.write(b"Move successful;\n"); }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:84:15 [INFO] [stderr] | [INFO] [stderr] 84 | { stream.write(b"You can't move air, ya numpty;\n"); }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 84 | { let _ = stream.write(b"You can't move air, ya numpty;\n"); }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:86:15 [INFO] [stderr] | [INFO] [stderr] 86 | { stream.write(b"Stick to your own pieces!;\n"); }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 86 | { let _ = stream.write(b"Stick to your own pieces!;\n"); }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:101:15 [INFO] [stderr] | [INFO] [stderr] 101 | { stream.write(format!("Oh, hai {}!;\n",name).as_bytes()); }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 101 | { let _ = stream.write(format!("Oh, hai {}!;\n",name).as_bytes()); }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:103:15 [INFO] [stderr] | [INFO] [stderr] 103 | { stream.write(b"You've already registered, you asshat;\n"); }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 103 | { let _ = stream.write(b"You've already registered, you asshat;\n"); }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:105:15 [INFO] [stderr] | [INFO] [stderr] 105 | { stream.write(b"The game is full, get tae;\n"); }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 105 | { let _ = stream.write(b"The game is full, get tae;\n"); }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: panic message is not a string literal [INFO] [stderr] --> src/server.rs:106:49 [INFO] [stderr] | [INFO] [stderr] 106 | Err(CommandErr::Other(s)) => panic!(s), [INFO] [stderr] | ^ [INFO] [stderr] | [INFO] [stderr] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stderr] = note: for more information, see [INFO] [stderr] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stderr] help: add a "{}" format string to `Display` the message [INFO] [stderr] | [INFO] [stderr] 106 | Err(CommandErr::Other(s)) => panic!("{}", s), [INFO] [stderr] | +++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:146:15 [INFO] [stderr] | [INFO] [stderr] 146 | { stream.write(format!("You are {:?} and {} wants to batter you;\n",col,opponent).as_bytes()); }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 146 | { let _ = stream.write(format!("You are {:?} and {} wants to batter you;\n",col,opponent).as_bytes()); }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:148:15 [INFO] [stderr] | [INFO] [stderr] 148 | { stream.write(b"Who even are you?;\n"); }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 148 | { let _ = stream.write(b"Who even are you?;\n"); }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/server.rs:156:9 [INFO] [stderr] | [INFO] [stderr] 156 | stream.write(b"You wot m8?;\n"); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 156 | let _ = stream.write(b"You wot m8?;\n"); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: panic message is not a string literal [INFO] [stderr] --> src/server.rs:173:30 [INFO] [stderr] | [INFO] [stderr] 173 | Err(e) => panic!(e), [INFO] [stderr] | ^ [INFO] [stderr] | [INFO] [stderr] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stderr] = note: for more information, see [INFO] [stderr] help: add a "{}" format string to `Display` the message [INFO] [stderr] | [INFO] [stderr] 173 | Err(e) => panic!("{}", e), [INFO] [stderr] | +++++ [INFO] [stderr] help: or use std::panic::panic_any instead [INFO] [stderr] | [INFO] [stderr] 173 | Err(e) => std::panic::panic_any(e), [INFO] [stderr] | ~~~~~~~~~~~~~~~~~~~~~ [INFO] [stderr] [INFO] [stderr] warning: panic message is not a string literal [INFO] [stderr] --> src/server.rs:201:40 [INFO] [stderr] | [INFO] [stderr] 201 | Err(e) => { panic!(e); } [INFO] [stderr] | ^ [INFO] [stderr] | [INFO] [stderr] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stderr] = note: for more information, see [INFO] [stderr] help: add a "{}" format string to `Display` the message [INFO] [stderr] | [INFO] [stderr] 201 | Err(e) => { panic!("{}", e); } [INFO] [stderr] | +++++ [INFO] [stderr] help: or use std::panic::panic_any instead [INFO] [stderr] | [INFO] [stderr] 201 | Err(e) => { std::panic::panic_any(e); } [INFO] [stderr] | ~~~~~~~~~~~~~~~~~~~~~ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/api.rs:23:9 [INFO] [stderr] | [INFO] [stderr] 23 | stream.read(&mut c); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 23 | let _ = stream.read(&mut c); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/api.rs:26:13 [INFO] [stderr] | [INFO] [stderr] 26 | stream.read(&mut c); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 26 | let _ = stream.read(&mut c); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/api.rs:29:9 [INFO] [stderr] | [INFO] [stderr] 29 | stream.read(&mut c); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 29 | let _ = stream.read(&mut c); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: `lines-of-action` (lib) generated 21 warnings (run `cargo fix --lib -p lines-of-action` to apply 8 suggestions) [INFO] [stderr] warning: `lines-of-action` (lib test) generated 21 warnings (21 duplicates) [INFO] [stderr] warning: unnecessary parentheses around method argument [INFO] [stderr] --> src/main.rs:179:40 [INFO] [stderr] | [INFO] [stderr] 179 | self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stderr] | ^ ^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_parens)]` on by default [INFO] [stderr] help: remove these parentheses [INFO] [stderr] | [INFO] [stderr] 179 - self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stderr] 179 + self.players.insert(id,Player::new(name.to_string())); [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/main.rs:42:13 [INFO] [stderr] | [INFO] [stderr] 42 | let mut window = match RenderWindow::new(VideoMode::new_init(SCREEN_SIZE, SCREEN_SIZE, 32), [INFO] [stderr] | ----^^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_mut)]` on by default [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/main.rs:50:13 [INFO] [stderr] | [INFO] [stderr] 50 | let mut board = Board::new(); [INFO] [stderr] | ----^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/main.rs:51:13 [INFO] [stderr] | [INFO] [stderr] 51 | let mut players = BTreeMap::new(); [INFO] [stderr] | ----^^^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] [INFO] [stderr] warning: unused variable: `id` [INFO] [stderr] --> src/main.rs:87:46 [INFO] [stderr] | [INFO] [stderr] 87 | CommandData::Message(id,msg) => [INFO] [stderr] | ^^ help: if this is intentional, prefix it with an underscore: `_id` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_variables)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused variable: `id` [INFO] [stderr] --> src/main.rs:204:14 [INFO] [stderr] | [INFO] [stderr] 204 | for (id, p) in &self.players { [INFO] [stderr] | ^^ help: if this is intentional, prefix it with an underscore: `_id` [INFO] [stderr] [INFO] [stderr] warning: fields `turn` and `game_started` are never read [INFO] [stderr] --> src/main.rs:26:5 [INFO] [stderr] | [INFO] [stderr] 21 | struct Engine { [INFO] [stderr] | ------ fields in this struct [INFO] [stderr] ... [INFO] [stderr] 26 | turn: Colour, [INFO] [stderr] | ^^^^ [INFO] [stderr] 27 | game_started: bool, [INFO] [stderr] | ^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(dead_code)]` on by default [INFO] [stderr] [INFO] [stderr] warning: panic message is not a string literal [INFO] [stderr] --> src/main.rs:95:59 [INFO] [stderr] | [INFO] [stderr] 95 | Err(GetCommandErr::Failed(why)) => panic!(why), [INFO] [stderr] | ^^^ [INFO] [stderr] | [INFO] [stderr] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stderr] = note: for more information, see [INFO] [stderr] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stderr] help: add a "{}" format string to `Display` the message [INFO] [stderr] | [INFO] [stderr] 95 | Err(GetCommandErr::Failed(why)) => panic!("{}", why), [INFO] [stderr] | +++++ [INFO] [stderr] [INFO] [stderr] warning: panic message is not a string literal [INFO] [stderr] --> src/main.rs:104:34 [INFO] [stderr] | [INFO] [stderr] 104 | Err(s) => panic!(s), [INFO] [stderr] | ^ [INFO] [stderr] | [INFO] [stderr] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stderr] = note: for more information, see [INFO] [stderr] help: add a "{}" format string to `Display` the message [INFO] [stderr] | [INFO] [stderr] 104 | Err(s) => panic!("{}", s), [INFO] [stderr] | +++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:141:9 [INFO] [stderr] | [INFO] [stderr] 141 | / reply.send(match self.board.move_piece(&mov,&player_colour) { [INFO] [stderr] 142 | | Ok(()) => { [INFO] [stderr] 143 | | Ok(CommandOk::None) [INFO] [stderr] 144 | | }, [INFO] [stderr] 145 | | Err(MoveErr::NoPiece) => Err(CommandErr::NoPiece), [INFO] [stderr] 146 | | Err(MoveErr::WrongPiece) => Err(CommandErr::WrongPiece), [INFO] [stderr] 147 | | }); [INFO] [stderr] | |__________^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] = note: `#[warn(unused_must_use)]` on by default [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 141 | let _ = reply.send(match self.board.move_piece(&mov,&player_colour) { [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:156:21 [INFO] [stderr] | [INFO] [stderr] 156 | other.reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 156 | let _ = other.reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:173:13 [INFO] [stderr] | [INFO] [stderr] 173 | reply.send(Err(CommandErr::AlreadyRegistered)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 173 | let _ = reply.send(Err(CommandErr::AlreadyRegistered)); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:177:9 [INFO] [stderr] | [INFO] [stderr] 177 | / reply.send( [INFO] [stderr] 178 | | if self.players.len() < 2 { [INFO] [stderr] 179 | | self.players.insert(id,(Player::new(name.to_string()))); [INFO] [stderr] 180 | | debug!("Registered new player: {}", name); [INFO] [stderr] ... | [INFO] [stderr] 183 | | Err(CommandErr::GameFull) [INFO] [stderr] 184 | | }); [INFO] [stderr] | |______________^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 177 | let _ = reply.send( [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:189:9 [INFO] [stderr] | [INFO] [stderr] 189 | reply.send(Ok(CommandOk::None)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 189 | let _ = reply.send(Ok(CommandOk::None)); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:195:23 [INFO] [stderr] | [INFO] [stderr] 195 | None => { reply.send(Err(CommandErr::NotRegistered)); return; }, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 195 | None => { let _ = reply.send(Err(CommandErr::NotRegistered)); return; }, [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:218:13 [INFO] [stderr] | [INFO] [stderr] 218 | p1.reply.clone().unwrap().send(Ok(CommandOk::Ready(p2_name, p1.colour.clone()))); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 218 | let _ = p1.reply.clone().unwrap().send(Ok(CommandOk::Ready(p2_name, p1.colour.clone()))); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:225:13 [INFO] [stderr] | [INFO] [stderr] 225 | p2.reply.clone().unwrap().send(Ok(CommandOk::Ready(p1_name, p2.colour))); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 225 | let _ = p2.reply.clone().unwrap().send(Ok(CommandOk::Ready(p1_name, p2.colour))); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:239:25 [INFO] [stderr] | [INFO] [stderr] 239 | self.players.get(&i).unwrap().reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 239 | let _ = self.players.get(&i).unwrap().reply.clone().unwrap().send(Ok(CommandOk::Move(mov.clone()))); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: `lines-of-action` (bin "loa-server" test) generated 18 warnings (run `cargo fix --bin "loa-server" --tests` to apply 8 suggestions) [INFO] [stderr] Finished test [unoptimized + debuginfo] target(s) in 0.01s [INFO] [stderr] Running unittests src/lib.rs (/opt/rustwide/target/debug/deps/lines_of_action-30a3a6d11c5d0f79) [INFO] [stdout] [INFO] [stdout] running 0 tests [INFO] [stdout] [INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s [INFO] [stdout] [INFO] [stderr] Running unittests src/main.rs (/opt/rustwide/target/debug/deps/loa_server-fae2f4c975ce6eb2) [INFO] [stdout] [INFO] [stdout] running 0 tests [INFO] [stdout] [INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s [INFO] [stdout] [INFO] [stderr] Doc-tests lines_of_action [INFO] [stdout] [INFO] [stdout] running 0 tests [INFO] [stdout] [INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s [INFO] [stdout] [INFO] running `Command { std: "docker" "inspect" "6b900caa1da47b70af65f8d8a4f84380375fd46137090b9c289038029ec9574d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "6b900caa1da47b70af65f8d8a4f84380375fd46137090b9c289038029ec9574d", kill_on_drop: false }` [INFO] [stdout] 6b900caa1da47b70af65f8d8a4f84380375fd46137090b9c289038029ec9574d