[INFO] cloning repository https://github.com/kennyvv/Rust-Server-Ping [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/kennyvv/Rust-Server-Ping" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkennyvv%2FRust-Server-Ping", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkennyvv%2FRust-Server-Ping'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 06341c64880609f37dac70946829a9702f2be722 [INFO] checking kennyvv/Rust-Server-Ping/06341c64880609f37dac70946829a9702f2be722 against master#568f6a8641e391ffcdcdb03e79d0901731d8d399 for pr-117164 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkennyvv%2FRust-Server-Ping" "/workspace/builds/worker-4-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-4-tc1/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/kennyvv/Rust-Server-Ping on toolchain 568f6a8641e391ffcdcdb03e79d0901731d8d399 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+568f6a8641e391ffcdcdb03e79d0901731d8d399" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/kennyvv/Rust-Server-Ping [INFO] finished tweaking git repo https://github.com/kennyvv/Rust-Server-Ping [INFO] tweaked toml for git repo https://github.com/kennyvv/Rust-Server-Ping written to /workspace/builds/worker-4-tc1/source/Cargo.toml [INFO] crate git repo https://github.com/kennyvv/Rust-Server-Ping 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" "+568f6a8641e391ffcdcdb03e79d0901731d8d399" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:2788e3201cd34a07e3172128adcd8b3090168a8e3bcc40d7c032b9dda1df7d1c" "/opt/rustwide/cargo-home/bin/cargo" "+568f6a8641e391ffcdcdb03e79d0901731d8d399" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] e7bdd874bbd6f59c78ceb39d02260495a2d1ef885df8cec9a23707497506c08d [INFO] running `Command { std: "docker" "start" "-a" "e7bdd874bbd6f59c78ceb39d02260495a2d1ef885df8cec9a23707497506c08d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "e7bdd874bbd6f59c78ceb39d02260495a2d1ef885df8cec9a23707497506c08d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "e7bdd874bbd6f59c78ceb39d02260495a2d1ef885df8cec9a23707497506c08d", kill_on_drop: false }` [INFO] [stdout] e7bdd874bbd6f59c78ceb39d02260495a2d1ef885df8cec9a23707497506c08d [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:2788e3201cd34a07e3172128adcd8b3090168a8e3bcc40d7c032b9dda1df7d1c" "/opt/rustwide/cargo-home/bin/cargo" "+568f6a8641e391ffcdcdb03e79d0901731d8d399" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 339ee6d20ea3d8d8d7cb66a0843a3eff7809b11f83b99cc9ddc3173d5c8929c6 [INFO] running `Command { std: "docker" "start" "-a" "339ee6d20ea3d8d8d7cb66a0843a3eff7809b11f83b99cc9ddc3173d5c8929c6", kill_on_drop: false }` [INFO] [stderr] Checking Rust-Test v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `BufReader` [INFO] [stdout] --> src/net/client_connection.rs:2:15 [INFO] [stdout] | [INFO] [stdout] 2 | use std::io::{BufReader, Write}; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::io::mem_stream::MemStream` [INFO] [stdout] --> src/net/client_connection.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use crate::io::mem_stream::MemStream; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Packet` [INFO] [stdout] --> src/net/client_connection.rs:6:27 [INFO] [stdout] | [INFO] [stdout] 6 | use crate::net::packets::{Packet, Handshake, read_handshake, read_ping_request}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `ReadError` [INFO] [stdout] --> src/net/packets.rs:1:53 [INFO] [stdout] | [INFO] [stdout] 1 | use crate::net::minecraft_stream::{MinecraftStream, ReadError}; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `BufReader` [INFO] [stdout] --> src/net/client_connection.rs:2:15 [INFO] [stdout] | [INFO] [stdout] 2 | use std::io::{BufReader, Write}; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::io::mem_stream::MemStream` [INFO] [stdout] --> src/net/client_connection.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use crate::io::mem_stream::MemStream; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Packet` [INFO] [stdout] --> src/net/client_connection.rs:6:27 [INFO] [stdout] | [INFO] [stdout] 6 | use crate::net::packets::{Packet, Handshake, read_handshake, read_ping_request}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `ReadError` [INFO] [stdout] --> src/net/packets.rs:1:53 [INFO] [stdout] | [INFO] [stdout] 1 | use crate::net::minecraft_stream::{MinecraftStream, ReadError}; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `uncompressed_length` is never read [INFO] [stdout] --> src/net/client_connection.rs:45:21 [INFO] [stdout] | [INFO] [stdout] 45 | let mut uncompressed_length = 0; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `uncompressed_length` is never read [INFO] [stdout] --> src/net/client_connection.rs:45:21 [INFO] [stdout] | [INFO] [stdout] 45 | let mut uncompressed_length = 0; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/net/client_connection.rs:39:13 [INFO] [stdout] | [INFO] [stdout] 39 | let mut packet_length = self.stream.read_varint().expect("Failed to read packet length"); [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/net/client_connection.rs:39:13 [INFO] [stdout] | [INFO] [stdout] 39 | let mut packet_length = self.stream.read_varint().expect("Failed to read packet length"); [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: value assigned to `packet_id` is never read [INFO] [stdout] --> src/net/client_connection.rs:78:17 [INFO] [stdout] | [INFO] [stdout] 78 | let mut packet_id: i32 = 0; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `packet_id` is never read [INFO] [stdout] --> src/net/client_connection.rs:78:17 [INFO] [stdout] | [INFO] [stdout] 78 | let mut packet_id: i32 = 0; [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] --> src/net/client_connection.rs:112:17 [INFO] [stdout] | [INFO] [stdout] 112 | let mut buffer: Vec = Vec::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/net/client_connection.rs:121:17 [INFO] [stdout] | [INFO] [stdout] 121 | let mut buffer: Vec = Vec::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/net/client_connection.rs:112:17 [INFO] [stdout] | [INFO] [stdout] 112 | let mut buffer: Vec = Vec::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/net/client_connection.rs:121:17 [INFO] [stdout] | [INFO] [stdout] 121 | let mut buffer: Vec = Vec::new(); [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/net/minecraft_server.rs:37:21 [INFO] [stdout] | [INFO] [stdout] 37 | Err(e) => { /* connection failed */ } [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/net/minecraft_server.rs:37:21 [INFO] [stdout] | [INFO] [stdout] 37 | Err(e) => { /* connection failed */ } [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `read` is never read [INFO] [stdout] --> src/net/minecraft_stream.rs:110:17 [INFO] [stdout] | [INFO] [stdout] 110 | let mut read: u8 = 0; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `read` is never read [INFO] [stdout] --> src/net/minecraft_stream.rs:110:17 [INFO] [stdout] | [INFO] [stdout] 110 | let mut read: u8 = 0; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `read` is never read [INFO] [stdout] --> src/net/minecraft_stream.rs:156:17 [INFO] [stdout] | [INFO] [stdout] 156 | let mut read: u8 = 0; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `read` is never read [INFO] [stdout] --> src/net/minecraft_stream.rs:156:17 [INFO] [stdout] | [INFO] [stdout] 156 | let mut read: u8 = 0; [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] --> src/net/packets.rs:30:23 [INFO] [stdout] | [INFO] [stdout] 30 | pub fn read_handshake(mut stream: impl MinecraftStream) -> Handshake { [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/net/packets.rs:30:23 [INFO] [stdout] | [INFO] [stdout] 30 | pub fn read_handshake(mut stream: impl MinecraftStream) -> Handshake { [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/net/packets.rs:53:26 [INFO] [stdout] | [INFO] [stdout] 53 | pub fn read_ping_request(mut stream: impl MinecraftStream) -> PingRequest { [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/net/packets.rs:53:26 [INFO] [stdout] | [INFO] [stdout] 53 | pub fn read_ping_request(mut stream: impl MinecraftStream) -> PingRequest { [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `compression_threshold` is never read [INFO] [stdout] --> src/net/client_connection.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 8 | pub struct ClientConnection{ [INFO] [stdout] | ---------------- field in this struct [INFO] [stdout] ... [INFO] [stdout] 12 | compression_threshold: i32, [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `VarLongTooBig` is never constructed [INFO] [stdout] --> src/net/minecraft_stream.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 7 | pub enum ReadError { [INFO] [stdout] | --------- variant in this enum [INFO] [stdout] ... [INFO] [stdout] 10 | VarLongTooBig [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `ReadError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `compression_threshold` is never read [INFO] [stdout] --> src/net/client_connection.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 8 | pub struct ClientConnection{ [INFO] [stdout] | ---------------- field in this struct [INFO] [stdout] ... [INFO] [stdout] 12 | compression_threshold: i32, [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: crate `Rust_Test` should have a snake case name [INFO] [stdout] | [INFO] [stdout] = help: convert the identifier to snake case: `rust_test` [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `VarLongTooBig` is never constructed [INFO] [stdout] --> src/net/minecraft_stream.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 7 | pub enum ReadError { [INFO] [stdout] | --------- variant in this enum [INFO] [stdout] ... [INFO] [stdout] 10 | VarLongTooBig [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `ReadError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: crate `Rust_Test` should have a snake case name [INFO] [stdout] | [INFO] [stdout] = help: convert the identifier to snake case: `rust_test` [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 17 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 17 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished dev [unoptimized + debuginfo] target(s) in 0.07s [INFO] running `Command { std: "docker" "inspect" "339ee6d20ea3d8d8d7cb66a0843a3eff7809b11f83b99cc9ddc3173d5c8929c6", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "339ee6d20ea3d8d8d7cb66a0843a3eff7809b11f83b99cc9ddc3173d5c8929c6", kill_on_drop: false }` [INFO] [stdout] 339ee6d20ea3d8d8d7cb66a0843a3eff7809b11f83b99cc9ddc3173d5c8929c6