[INFO] cloning repository https://github.com/pajowu/snaprust [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/pajowu/snaprust" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fpajowu%2Fsnaprust", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fpajowu%2Fsnaprust'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 832a60b34551a97bde28a493b368f53fd9430779 [INFO] checking pajowu/snaprust against master#bca37a20bd376ce3fd138e7cdee7fe704e0f8814 for pr-139493-3 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fpajowu%2Fsnaprust" "/workspace/builds/worker-3-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-3-tc1/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/pajowu/snaprust [INFO] finished tweaking git repo https://github.com/pajowu/snaprust [INFO] tweaked toml for git repo https://github.com/pajowu/snaprust written to /workspace/builds/worker-3-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/pajowu/snaprust on toolchain bca37a20bd376ce3fd138e7cdee7fe704e0f8814 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+bca37a20bd376ce3fd138e7cdee7fe704e0f8814" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/pajowu/snaprust already has a lockfile, it will not be regenerated [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+bca37a20bd376ce3fd138e7cdee7fe704e0f8814" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024 [INFO] [stderr] Updating crates.io index [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded simplelog v0.4.2 [INFO] [stderr] Downloaded dtoa v0.4.2 [INFO] [stderr] Downloaded redox_syscall v0.1.31 [INFO] [stderr] Downloaded textwrap v0.8.0 [INFO] [stderr] Downloaded pkg-config v0.3.9 [INFO] [stderr] Downloaded alsa-sys v0.1.1 [INFO] [stderr] Downloaded bitflags v0.8.2 [INFO] [stderr] Downloaded lazy_static v0.2.9 [INFO] [stderr] Downloaded hound v3.1.0 [INFO] [stderr] Downloaded cgmath v0.14.1 [INFO] [stderr] Downloaded alsa v0.1.10 [INFO] [stderr] Downloaded claxon v0.3.1 [INFO] [stderr] Downloaded coreaudio-sys v0.1.2 [INFO] [stderr] Downloaded lewton v0.5.2 [INFO] [stderr] Downloaded syn v0.11.11 [INFO] [stderr] Downloaded serde v1.0.15 [INFO] [stderr] Downloaded futures v0.1.16 [INFO] [stderr] Downloaded clap v2.26.1 [INFO] [stderr] Downloaded libc v0.2.30 [INFO] [stderr] Downloaded serde_json v1.0.3 [INFO] [stderr] Downloaded cpal v0.4.5 [INFO] [stderr] Downloaded hound v1.1.0 [INFO] [stderr] Downloaded ogg v0.4.1 [INFO] [stderr] Downloaded serde_derive_internals v0.16.0 [INFO] [stderr] Downloaded synom v0.11.3 [INFO] [stderr] Downloaded term v0.4.6 [INFO] [stderr] Downloaded coreaudio-rs v0.6.0 [INFO] [stderr] Downloaded quote v0.3.15 [INFO] [stderr] Downloaded unicode-xid v0.0.4 [INFO] [stderr] Downloaded ole32-sys v0.2.0 [INFO] [stderr] Downloaded approx v0.1.1 [INFO] [stderr] Downloaded time v0.1.38 [INFO] [stderr] Downloaded serde_derive v1.0.15 [INFO] [stderr] Downloaded lazy_static v0.1.16 [INFO] [stderr] Downloaded rodio v0.5.2 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:aa71247004a7fa38d13ec170f48f06cdedf5bc50b2a8645e56ed7e992e6fa513" "/opt/rustwide/cargo-home/bin/cargo" "+bca37a20bd376ce3fd138e7cdee7fe704e0f8814" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 320a5fb92d20cd636a0f74f24674c55ee91461cbe67d09f8bf1bcb3bf45bec6d [INFO] running `Command { std: "docker" "start" "-a" "320a5fb92d20cd636a0f74f24674c55ee91461cbe67d09f8bf1bcb3bf45bec6d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "320a5fb92d20cd636a0f74f24674c55ee91461cbe67d09f8bf1bcb3bf45bec6d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "320a5fb92d20cd636a0f74f24674c55ee91461cbe67d09f8bf1bcb3bf45bec6d", kill_on_drop: false }` [INFO] [stdout] 320a5fb92d20cd636a0f74f24674c55ee91461cbe67d09f8bf1bcb3bf45bec6d [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:aa71247004a7fa38d13ec170f48f06cdedf5bc50b2a8645e56ed7e992e6fa513" "/opt/rustwide/cargo-home/bin/cargo" "+bca37a20bd376ce3fd138e7cdee7fe704e0f8814" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 676800df93603fc10b433da679dfcc3bc999b4dc4ccfecc049d6155c60d66935 [INFO] running `Command { std: "docker" "start" "-a" "676800df93603fc10b433da679dfcc3bc999b4dc4ccfecc049d6155c60d66935", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024 [INFO] [stderr] Checking libc v0.2.30 [INFO] [stderr] Compiling pkg-config v0.3.9 [INFO] [stderr] Compiling unicode-xid v0.0.4 [INFO] [stderr] Compiling quote v0.3.15 [INFO] [stderr] Checking num-traits v0.1.40 [INFO] [stderr] Checking byteorder v1.1.0 [INFO] [stderr] Checking lazy_static v0.2.9 [INFO] [stderr] Checking unicode-width v0.1.4 [INFO] [stderr] Checking approx v0.1.1 [INFO] [stderr] Checking futures v0.1.16 [INFO] [stderr] Checking dtoa v0.4.2 [INFO] [stderr] Checking term v0.4.6 [INFO] [stderr] Checking itoa v0.3.4 [INFO] [stderr] Checking strsim v0.6.0 [INFO] [stderr] Checking lazy_static v0.1.16 [INFO] [stderr] Checking serde v1.0.15 [INFO] [stderr] Compiling synom v0.11.3 [INFO] [stderr] Checking log v0.3.8 [INFO] [stderr] Checking claxon v0.3.1 [INFO] [stderr] Checking hound v1.1.0 [INFO] [stderr] Checking bitflags v0.8.2 [INFO] [stderr] Checking bitflags v0.9.1 [INFO] [stderr] Checking vec_map v0.8.0 [INFO] [stderr] Checking rand v0.3.16 [INFO] [stderr] Checking term_size v0.3.0 [INFO] [stderr] Checking time v0.1.38 [INFO] [stderr] Checking atty v0.2.2 [INFO] [stderr] Checking ansi_term v0.9.0 [INFO] [stderr] Checking hound v3.1.0 [INFO] [stderr] Checking textwrap v0.8.0 [INFO] [stderr] Compiling syn v0.11.11 [INFO] [stderr] Checking clap v2.26.1 [INFO] [stderr] Checking ogg v0.4.1 [INFO] [stderr] Compiling alsa-sys v0.1.1 [INFO] [stderr] Checking simplelog v0.4.2 [INFO] [stderr] Checking cgmath v0.14.1 [INFO] [stderr] Checking lewton v0.5.2 [INFO] [stderr] Checking alsa v0.1.10 [INFO] [stderr] Compiling serde_derive_internals v0.16.0 [INFO] [stderr] Checking cpal v0.4.5 [INFO] [stderr] Checking rodio v0.5.2 [INFO] [stderr] Compiling serde_derive v1.0.15 [INFO] [stderr] Checking serde_json v1.0.3 [INFO] [stderr] Checking snaprust v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused imports: `Access`, `Format`, and `State` [INFO] [stdout] --> src/main.rs:8:32 [INFO] [stdout] | [INFO] [stdout] 8 | use alsa::pcm::{PCM, HwParams, Format, Access, State}; [INFO] [stdout] | ^^^^^^ ^^^^^^ ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `WriteLogger` [INFO] [stdout] --> src/main.rs:13:37 [INFO] [stdout] | [INFO] [stdout] 13 | use simplelog::{Config, TermLogger, WriteLogger, CombinedLogger, LogLevelFilter}; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SubCommand` [INFO] [stdout] --> src/main.rs:17:22 [INFO] [stdout] | [INFO] [stdout] 17 | use clap::{Arg, App, SubCommand}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `ValueOr` [INFO] [stdout] --> src/main.rs:18:23 [INFO] [stdout] | [INFO] [stdout] 18 | use alsa::{Direction, ValueOr}; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::net::TcpStream` [INFO] [stdout] --> src/main.rs:19:5 [INFO] [stdout] | [INFO] [stdout] 19 | use std::net::TcpStream; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Read` and `Write` [INFO] [stdout] --> src/main.rs:20:15 [INFO] [stdout] | [INFO] [stdout] 20 | use std::io::{Write, Read}; [INFO] [stdout] | ^^^^^ ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::sync::mpsc` [INFO] [stdout] --> src/main.rs:23:5 [INFO] [stdout] | [INFO] [stdout] 23 | use std::sync::mpsc; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::collections::VecDeque` [INFO] [stdout] --> src/main.rs:24:5 [INFO] [stdout] | [INFO] [stdout] 24 | use std::collections::VecDeque; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::sync::Mutex` [INFO] [stdout] --> src/main.rs:25:5 [INFO] [stdout] | [INFO] [stdout] 25 | use std::sync::Mutex; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `serde` [INFO] [stdout] --> src/message.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use serde; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `serde_derive` [INFO] [stdout] --> src/message.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use serde_derive; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Write` [INFO] [stdout] --> src/message.rs:8:15 [INFO] [stdout] | [INFO] [stdout] 8 | use std::io::{Write, Read, Error}; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead [INFO] [stdout] --> src/message.rs:190:25 [INFO] [stdout] | [INFO] [stdout] 190 | let refers_to = try!(socket.read_u16::()); [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(deprecated)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `SnapMessageData` and `TimeVal` [INFO] [stdout] --> src/main.rs:29:15 [INFO] [stdout] | [INFO] [stdout] 29 | use message::{SnapMessageData, TimeVal}; [INFO] [stdout] | ^^^^^^^^^^^^^^^ ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::collections::VecDeque` [INFO] [stdout] --> src/network_handler.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use std::collections::VecDeque; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::sync::Mutex` [INFO] [stdout] --> src/network_handler.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::sync::Mutex; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `SystemTime` and `UNIX_EPOCH` [INFO] [stdout] --> src/network_handler.rs:8:17 [INFO] [stdout] | [INFO] [stdout] 8 | use std::time::{SystemTime, UNIX_EPOCH, Instant}; [INFO] [stdout] | ^^^^^^^^^^ ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `log` [INFO] [stdout] --> src/network_handler.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | use log; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `simplelog` [INFO] [stdout] --> src/network_handler.rs:13:5 [INFO] [stdout] | [INFO] [stdout] 13 | use simplelog; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::ops::Range` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use std::ops::Range; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `hound` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use hound; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::io::Cursor` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:6:5 [INFO] [stdout] | [INFO] [stdout] 6 | use std::io::Cursor; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `PCM` and `State` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:9:17 [INFO] [stdout] | [INFO] [stdout] 9 | use alsa::pcm::{PCM, HwParams, Format, Access, State}; [INFO] [stdout] | ^^^ ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Direction` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:10:12 [INFO] [stdout] | [INFO] [stdout] 10 | use alsa::{Direction, ValueOr}; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Access`, `Format`, `PCM`, and `State` [INFO] [stdout] --> src/decoder/mod.rs:6:17 [INFO] [stdout] | [INFO] [stdout] 6 | use alsa::pcm::{PCM, HwParams, Format, Access, State}; [INFO] [stdout] | ^^^ ^^^^^^ ^^^^^^ ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::vec::IntoIter` [INFO] [stdout] --> src/time_provider.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::vec::IntoIter; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/main.rs:246:84 [INFO] [stdout] | [INFO] [stdout] 246 | fn handleCodecHeader(data: message::CodecHeaderData, hwp: &HwParams) -> Option> { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: `#[warn(bare_trait_objects)]` (part of `#[warn(rust_2021_compatibility)]`) on by default [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 246 | fn handleCodecHeader(data: message::CodecHeaderData, hwp: &HwParams) -> Option> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/main.rs:255:41 [INFO] [stdout] | [INFO] [stdout] 255 | fn handleWireChunk(decoder: &Option>, data: message::WireChunkData) -> Option<(usize,Vec)> { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 255 | fn handleWireChunk(decoder: &Option>, data: message::WireChunkData) -> Option<(usize,Vec)> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Access`, `Format`, and `State` [INFO] [stdout] --> src/main.rs:8:32 [INFO] [stdout] | [INFO] [stdout] 8 | use alsa::pcm::{PCM, HwParams, Format, Access, State}; [INFO] [stdout] | ^^^^^^ ^^^^^^ ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `WriteLogger` [INFO] [stdout] --> src/main.rs:13:37 [INFO] [stdout] | [INFO] [stdout] 13 | use simplelog::{Config, TermLogger, WriteLogger, CombinedLogger, LogLevelFilter}; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SubCommand` [INFO] [stdout] --> src/main.rs:17:22 [INFO] [stdout] | [INFO] [stdout] 17 | use clap::{Arg, App, SubCommand}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `ValueOr` [INFO] [stdout] --> src/main.rs:18:23 [INFO] [stdout] | [INFO] [stdout] 18 | use alsa::{Direction, ValueOr}; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::net::TcpStream` [INFO] [stdout] --> src/main.rs:19:5 [INFO] [stdout] | [INFO] [stdout] 19 | use std::net::TcpStream; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Read` and `Write` [INFO] [stdout] --> src/main.rs:20:15 [INFO] [stdout] | [INFO] [stdout] 20 | use std::io::{Write, Read}; [INFO] [stdout] | ^^^^^ ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::sync::mpsc` [INFO] [stdout] --> src/main.rs:23:5 [INFO] [stdout] | [INFO] [stdout] 23 | use std::sync::mpsc; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::collections::VecDeque` [INFO] [stdout] --> src/main.rs:24:5 [INFO] [stdout] | [INFO] [stdout] 24 | use std::collections::VecDeque; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::sync::Mutex` [INFO] [stdout] --> src/main.rs:25:5 [INFO] [stdout] | [INFO] [stdout] 25 | use std::sync::Mutex; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `serde` [INFO] [stdout] --> src/message.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use serde; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `serde_derive` [INFO] [stdout] --> src/message.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use serde_derive; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Write` [INFO] [stdout] --> src/message.rs:8:15 [INFO] [stdout] | [INFO] [stdout] 8 | use std::io::{Write, Read, Error}; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead [INFO] [stdout] --> src/message.rs:190:25 [INFO] [stdout] | [INFO] [stdout] 190 | let refers_to = try!(socket.read_u16::()); [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(deprecated)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `SnapMessageData` and `TimeVal` [INFO] [stdout] --> src/main.rs:29:15 [INFO] [stdout] | [INFO] [stdout] 29 | use message::{SnapMessageData, TimeVal}; [INFO] [stdout] | ^^^^^^^^^^^^^^^ ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::collections::VecDeque` [INFO] [stdout] --> src/network_handler.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use std::collections::VecDeque; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::sync::Mutex` [INFO] [stdout] --> src/network_handler.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::sync::Mutex; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `SystemTime` and `UNIX_EPOCH` [INFO] [stdout] --> src/network_handler.rs:8:17 [INFO] [stdout] | [INFO] [stdout] 8 | use std::time::{SystemTime, UNIX_EPOCH, Instant}; [INFO] [stdout] | ^^^^^^^^^^ ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `log` [INFO] [stdout] --> src/network_handler.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | use log; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `simplelog` [INFO] [stdout] --> src/network_handler.rs:13:5 [INFO] [stdout] | [INFO] [stdout] 13 | use simplelog; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::ops::Range` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use std::ops::Range; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `hound` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use hound; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::io::Cursor` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:6:5 [INFO] [stdout] | [INFO] [stdout] 6 | use std::io::Cursor; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/message.rs:88:17 [INFO] [stdout] | [INFO] [stdout] 88 | let t: &SnapMessageData = match self { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 88 | let t: &dyn SnapMessageData = match self { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `PCM` and `State` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:9:17 [INFO] [stdout] | [INFO] [stdout] 9 | use alsa::pcm::{PCM, HwParams, Format, Access, State}; [INFO] [stdout] | ^^^ ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Direction` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:10:12 [INFO] [stdout] | [INFO] [stdout] 10 | use alsa::{Direction, ValueOr}; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Access`, `Format`, `PCM`, and `State` [INFO] [stdout] --> src/decoder/mod.rs:6:17 [INFO] [stdout] | [INFO] [stdout] 6 | use alsa::pcm::{PCM, HwParams, Format, Access, State}; [INFO] [stdout] | ^^^ ^^^^^^ ^^^^^^ ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::vec::IntoIter` [INFO] [stdout] --> src/time_provider.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::vec::IntoIter; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/main.rs:246:84 [INFO] [stdout] | [INFO] [stdout] 246 | fn handleCodecHeader(data: message::CodecHeaderData, hwp: &HwParams) -> Option> { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: `#[warn(bare_trait_objects)]` (part of `#[warn(rust_2021_compatibility)]`) on by default [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 246 | fn handleCodecHeader(data: message::CodecHeaderData, hwp: &HwParams) -> Option> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/main.rs:255:41 [INFO] [stdout] | [INFO] [stdout] 255 | fn handleWireChunk(decoder: &Option>, data: message::WireChunkData) -> Option<(usize,Vec)> { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 255 | fn handleWireChunk(decoder: &Option>, data: message::WireChunkData) -> Option<(usize,Vec)> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/message.rs:88:17 [INFO] [stdout] | [INFO] [stdout] 88 | let t: &SnapMessageData = match self { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 88 | let t: &dyn SnapMessageData = match self { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/main.rs:183:33 [INFO] [stdout] | [INFO] [stdout] 183 | let mut decoder: Option> = None; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 183 | let mut decoder: Option> = None; [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable statement [INFO] [stdout] --> src/main.rs:242:5 [INFO] [stdout] | [INFO] [stdout] 190 | / loop { [INFO] [stdout] 191 | | while let Ok(msg) = msg_rx.try_recv() { [INFO] [stdout] 192 | | debug!("Got message: {:?}", msg); [INFO] [stdout] 193 | | match msg.type_ { [INFO] [stdout] ... | [INFO] [stdout] 239 | | thread::sleep(time::Duration::from_millis(1000)); [INFO] [stdout] 240 | | } [INFO] [stdout] | |_____- any code following this expression is unreachable [INFO] [stdout] 241 | [INFO] [stdout] 242 | t.join(); [INFO] [stdout] | ^^^^^^^^^ unreachable statement [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unreachable_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/main.rs:247:22 [INFO] [stdout] | [INFO] [stdout] 247 | let decoder: Box = match data.codec.as_str() { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 247 | let decoder: Box = match data.codec.as_str() { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `ReadBytesExt` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:3:42 [INFO] [stdout] | [INFO] [stdout] 3 | use byteorder::{ByteOrder, LittleEndian, ReadBytesExt}; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/message.rs:160:20 [INFO] [stdout] | [INFO] [stdout] 160 | fn deserialize(data: &[u8]) -> Self { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/message.rs:181:24 [INFO] [stdout] | [INFO] [stdout] 181 | pub fn deserialize(data: Vec) { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/message.rs:201:13 [INFO] [stdout] | [INFO] [stdout] 201 | let data = socket.read(&mut buf); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/main.rs:183:33 [INFO] [stdout] | [INFO] [stdout] 183 | let mut decoder: Option> = None; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 183 | let mut decoder: Option> = None; [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable statement [INFO] [stdout] --> src/main.rs:242:5 [INFO] [stdout] | [INFO] [stdout] 190 | / loop { [INFO] [stdout] 191 | | while let Ok(msg) = msg_rx.try_recv() { [INFO] [stdout] 192 | | debug!("Got message: {:?}", msg); [INFO] [stdout] 193 | | match msg.type_ { [INFO] [stdout] ... | [INFO] [stdout] 239 | | thread::sleep(time::Duration::from_millis(1000)); [INFO] [stdout] 240 | | } [INFO] [stdout] | |_____- any code following this expression is unreachable [INFO] [stdout] 241 | [INFO] [stdout] 242 | t.join(); [INFO] [stdout] | ^^^^^^^^^ unreachable statement [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unreachable_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated [INFO] [stdout] --> src/main.rs:247:22 [INFO] [stdout] | [INFO] [stdout] 247 | let decoder: Box = match data.codec.as_str() { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021! [INFO] [stdout] = note: for more information, see [INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn` [INFO] [stdout] | [INFO] [stdout] 247 | let decoder: Box = match data.codec.as_str() { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `ReadBytesExt` [INFO] [stdout] --> src/decoder/pcm_decoder.rs:3:42 [INFO] [stdout] | [INFO] [stdout] 3 | use byteorder::{ByteOrder, LittleEndian, ReadBytesExt}; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/message.rs:160:20 [INFO] [stdout] | [INFO] [stdout] 160 | fn deserialize(data: &[u8]) -> Self { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/message.rs:181:24 [INFO] [stdout] | [INFO] [stdout] 181 | pub fn deserialize(data: Vec) { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/message.rs:201:13 [INFO] [stdout] | [INFO] [stdout] 201 | let data = socket.read(&mut buf); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/message.rs:311:13 [INFO] [stdout] | [INFO] [stdout] 311 | let mut data = Vec::from(data); [INFO] [stdout] | ----^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/message.rs:339:13 [INFO] [stdout] | [INFO] [stdout] 339 | let mut data = Vec::from(data); [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/network_handler.rs:25:13 [INFO] [stdout] | [INFO] [stdout] 25 | let mut stream = TcpStream::connect(host).unwrap(); [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/decoder/pcm_decoder.rs:35:9 [INFO] [stdout] | [INFO] [stdout] 35 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_unsafe)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/decoder/pcm_decoder.rs:41:25 [INFO] [stdout] | [INFO] [stdout] 41 | fn setHeader(&self, mut header: message::CodecHeaderData) { [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `chunk` [INFO] [stdout] --> src/decoder/mod.rs:18:22 [INFO] [stdout] | [INFO] [stdout] 18 | fn decode(&self, chunk: Vec) -> Vec { [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_chunk` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `header` [INFO] [stdout] --> src/decoder/mod.rs:21:25 [INFO] [stdout] | [INFO] [stdout] 21 | fn setHeader(&self, header: message::CodecHeaderData) { [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_header` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `pcm` [INFO] [stdout] --> src/decoder/mod.rs:27:28 [INFO] [stdout] | [INFO] [stdout] 27 | fn get_hwparams(&self, pcm: &HwParams) { } [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_pcm` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `host` [INFO] [stdout] --> src/main.rs:72:9 [INFO] [stdout] | [INFO] [stdout] 72 | let host = matches.value_of("HOST").unwrap(); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_host` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `t` [INFO] [stdout] --> src/main.rs:138:9 [INFO] [stdout] | [INFO] [stdout] 138 | let t = thread::spawn(move || { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_t` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/main.rs:273:24 [INFO] [stdout] | [INFO] [stdout] 273 | fn handleServerSetting(data: message::ServerSettingsData) { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `deserialize_u16` is never used [INFO] [stdout] --> src/message.rs:32:8 [INFO] [stdout] | [INFO] [stdout] 32 | pub fn deserialize_u16(buf: &[u8]) -> u16 { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `deserialize_u32` is never used [INFO] [stdout] --> src/message.rs:36:8 [INFO] [stdout] | [INFO] [stdout] 36 | pub fn deserialize_u32(buf: &[u8]) -> u32 { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `deserialize_i32` is never used [INFO] [stdout] --> src/message.rs:40:8 [INFO] [stdout] | [INFO] [stdout] 40 | pub fn deserialize_i32(buf: &[u8]) -> i32 { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `BASE_MESSAGE_SIZE` is never used [INFO] [stdout] --> src/message.rs:150:7 [INFO] [stdout] | [INFO] [stdout] 150 | const BASE_MESSAGE_SIZE: usize = 26; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `deserialize` is never used [INFO] [stdout] --> src/message.rs:181:12 [INFO] [stdout] | [INFO] [stdout] 165 | impl Message { [INFO] [stdout] | ------------ associated function in this implementation [INFO] [stdout] ... [INFO] [stdout] 181 | pub fn deserialize(data: Vec) { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `fill_queue` is never used [INFO] [stdout] --> src/network_handler.rs:81:8 [INFO] [stdout] | [INFO] [stdout] 81 | pub fn fill_queue(sender: mpsc::Sender, stream: TcpStream) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `new` is never used [INFO] [stdout] --> src/decoder/mod.rs:11:8 [INFO] [stdout] | [INFO] [stdout] 8 | pub trait Decoder { [INFO] [stdout] | ------- associated function in this trait [INFO] [stdout] ... [INFO] [stdout] 11 | fn new() -> Self where Self: Sized; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/message.rs:311:13 [INFO] [stdout] | [INFO] [stdout] 311 | let mut data = Vec::from(data); [INFO] [stdout] | ----^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/message.rs:339:13 [INFO] [stdout] | [INFO] [stdout] 339 | let mut data = Vec::from(data); [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/network_handler.rs:25:13 [INFO] [stdout] | [INFO] [stdout] 25 | let mut stream = TcpStream::connect(host).unwrap(); [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/decoder/pcm_decoder.rs:35:9 [INFO] [stdout] | [INFO] [stdout] 35 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_unsafe)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/decoder/pcm_decoder.rs:41:25 [INFO] [stdout] | [INFO] [stdout] 41 | fn setHeader(&self, mut header: message::CodecHeaderData) { [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `chunk` [INFO] [stdout] --> src/decoder/mod.rs:18:22 [INFO] [stdout] | [INFO] [stdout] 18 | fn decode(&self, chunk: Vec) -> Vec { [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_chunk` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `header` [INFO] [stdout] --> src/decoder/mod.rs:21:25 [INFO] [stdout] | [INFO] [stdout] 21 | fn setHeader(&self, header: message::CodecHeaderData) { [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_header` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `pcm` [INFO] [stdout] --> src/decoder/mod.rs:27:28 [INFO] [stdout] | [INFO] [stdout] 27 | fn get_hwparams(&self, pcm: &HwParams) { } [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_pcm` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/main.rs:160:5 [INFO] [stdout] | [INFO] [stdout] 160 | msg_tx.send(hello_msg); [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)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 160 | let _ = msg_tx.send(hello_msg); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/main.rs:177:9 [INFO] [stdout] | [INFO] [stdout] 177 | msg_tx.send(time_msg); [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 _ = msg_tx.send(time_msg); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/main.rs:234:13 [INFO] [stdout] | [INFO] [stdout] 234 | pcm.recover(e.code(), true); [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] 234 | let _ = pcm.recover(e.code(), true); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/main.rs:242:5 [INFO] [stdout] | [INFO] [stdout] 242 | t.join(); [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] 242 | let _ = t.join(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handleCodecHeader` should have a snake case name [INFO] [stdout] --> src/main.rs:246:4 [INFO] [stdout] | [INFO] [stdout] 246 | fn handleCodecHeader(data: message::CodecHeaderData, hwp: &HwParams) -> Option> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ help: convert the identifier to snake case: `handle_codec_header` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handleWireChunk` should have a snake case name [INFO] [stdout] --> src/main.rs:255:4 [INFO] [stdout] | [INFO] [stdout] 255 | fn handleWireChunk(decoder: &Option>, data: message::WireChunkData) -> Option<(usize,Vec)> { [INFO] [stdout] | ^^^^^^^^^^^^^^^ help: convert the identifier to snake case: `handle_wire_chunk` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handleServerSetting` should have a snake case name [INFO] [stdout] --> src/main.rs:273:4 [INFO] [stdout] | [INFO] [stdout] 273 | fn handleServerSetting(data: message::ServerSettingsData) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ help: convert the identifier to snake case: `handle_server_setting` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handleTime` should have a snake case name [INFO] [stdout] --> src/main.rs:276:4 [INFO] [stdout] | [INFO] [stdout] 276 | fn handleTime(data: message::TimeData, time_provider: &mut TimeProvider) { [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `handle_time` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/message.rs:221:24 [INFO] [stdout] | [INFO] [stdout] 221 | #[derive(Debug, Clone, Serialize, Deserialize)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `HelloData` is not local [INFO] [stdout] | `Serialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_SERIALIZE_FOR_HelloData` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Serialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `Serialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/message.rs:221:35 [INFO] [stdout] | [INFO] [stdout] 221 | #[derive(Debug, Clone, Serialize, Deserialize)] [INFO] [stdout] | -^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `HelloData` is not local [INFO] [stdout] | `Deserialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_HelloData` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/message.rs:255:24 [INFO] [stdout] | [INFO] [stdout] 255 | #[derive(Debug, Clone, Serialize, Deserialize)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ServerSettingsData` is not local [INFO] [stdout] | `Serialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_SERIALIZE_FOR_ServerSettingsData` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Serialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Serialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/message.rs:255:35 [INFO] [stdout] | [INFO] [stdout] 255 | #[derive(Debug, Clone, Serialize, Deserialize)] [INFO] [stdout] | -^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ServerSettingsData` is not local [INFO] [stdout] | `Deserialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_ServerSettingsData` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/network_handler.rs:26:9 [INFO] [stdout] | [INFO] [stdout] 26 | stream.set_nonblocking(true); [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.set_nonblocking(true); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/network_handler.rs:60:17 [INFO] [stdout] | [INFO] [stdout] 60 | self.stream.write_all(&msg[..]); [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] 60 | let _ = self.stream.write_all(&msg[..]); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/network_handler.rs:68:17 [INFO] [stdout] | [INFO] [stdout] 68 | self.stream.write_all(&msg[..]); [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] 68 | let _ = self.stream.write_all(&msg[..]); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/network_handler.rs:73:17 [INFO] [stdout] | [INFO] [stdout] 73 | self.recv_message_channel.send(msg); [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] 73 | let _ = self.recv_message_channel.send(msg); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait method `setHeader` should have a snake case name [INFO] [stdout] --> src/decoder/mod.rs:10:8 [INFO] [stdout] | [INFO] [stdout] 10 | fn setHeader(&self, header: message::CodecHeaderData); [INFO] [stdout] | ^^^^^^^^^ help: convert the identifier to snake case: `set_header` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `host` [INFO] [stdout] --> src/main.rs:72:9 [INFO] [stdout] | [INFO] [stdout] 72 | let host = matches.value_of("HOST").unwrap(); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_host` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `t` [INFO] [stdout] --> src/main.rs:138:9 [INFO] [stdout] | [INFO] [stdout] 138 | let t = thread::spawn(move || { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_t` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `data` [INFO] [stdout] --> src/main.rs:273:24 [INFO] [stdout] | [INFO] [stdout] 273 | fn handleServerSetting(data: message::ServerSettingsData) { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_data` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `deserialize_u16` is never used [INFO] [stdout] --> src/message.rs:32:8 [INFO] [stdout] | [INFO] [stdout] 32 | pub fn deserialize_u16(buf: &[u8]) -> u16 { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `deserialize_u32` is never used [INFO] [stdout] --> src/message.rs:36:8 [INFO] [stdout] | [INFO] [stdout] 36 | pub fn deserialize_u32(buf: &[u8]) -> u32 { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `deserialize_i32` is never used [INFO] [stdout] --> src/message.rs:40:8 [INFO] [stdout] | [INFO] [stdout] 40 | pub fn deserialize_i32(buf: &[u8]) -> i32 { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `BASE_MESSAGE_SIZE` is never used [INFO] [stdout] --> src/message.rs:150:7 [INFO] [stdout] | [INFO] [stdout] 150 | const BASE_MESSAGE_SIZE: usize = 26; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `deserialize` is never used [INFO] [stdout] --> src/message.rs:181:12 [INFO] [stdout] | [INFO] [stdout] 165 | impl Message { [INFO] [stdout] | ------------ associated function in this implementation [INFO] [stdout] ... [INFO] [stdout] 181 | pub fn deserialize(data: Vec) { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `fill_queue` is never used [INFO] [stdout] --> src/network_handler.rs:81:8 [INFO] [stdout] | [INFO] [stdout] 81 | pub fn fill_queue(sender: mpsc::Sender, stream: TcpStream) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `new` is never used [INFO] [stdout] --> src/decoder/mod.rs:11:8 [INFO] [stdout] | [INFO] [stdout] 8 | pub trait Decoder { [INFO] [stdout] | ------- associated function in this trait [INFO] [stdout] ... [INFO] [stdout] 11 | fn new() -> Self where Self: Sized; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/main.rs:160:5 [INFO] [stdout] | [INFO] [stdout] 160 | msg_tx.send(hello_msg); [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)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 160 | let _ = msg_tx.send(hello_msg); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/main.rs:177:9 [INFO] [stdout] | [INFO] [stdout] 177 | msg_tx.send(time_msg); [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 _ = msg_tx.send(time_msg); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/main.rs:234:13 [INFO] [stdout] | [INFO] [stdout] 234 | pcm.recover(e.code(), true); [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] 234 | let _ = pcm.recover(e.code(), true); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/main.rs:242:5 [INFO] [stdout] | [INFO] [stdout] 242 | t.join(); [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] 242 | let _ = t.join(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handleCodecHeader` should have a snake case name [INFO] [stdout] --> src/main.rs:246:4 [INFO] [stdout] | [INFO] [stdout] 246 | fn handleCodecHeader(data: message::CodecHeaderData, hwp: &HwParams) -> Option> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ help: convert the identifier to snake case: `handle_codec_header` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handleWireChunk` should have a snake case name [INFO] [stdout] --> src/main.rs:255:4 [INFO] [stdout] | [INFO] [stdout] 255 | fn handleWireChunk(decoder: &Option>, data: message::WireChunkData) -> Option<(usize,Vec)> { [INFO] [stdout] | ^^^^^^^^^^^^^^^ help: convert the identifier to snake case: `handle_wire_chunk` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handleServerSetting` should have a snake case name [INFO] [stdout] --> src/main.rs:273:4 [INFO] [stdout] | [INFO] [stdout] 273 | fn handleServerSetting(data: message::ServerSettingsData) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ help: convert the identifier to snake case: `handle_server_setting` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `handleTime` should have a snake case name [INFO] [stdout] --> src/main.rs:276:4 [INFO] [stdout] | [INFO] [stdout] 276 | fn handleTime(data: message::TimeData, time_provider: &mut TimeProvider) { [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `handle_time` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/message.rs:221:24 [INFO] [stdout] | [INFO] [stdout] 221 | #[derive(Debug, Clone, Serialize, Deserialize)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `HelloData` is not local [INFO] [stdout] | `Serialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_SERIALIZE_FOR_HelloData` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Serialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `Serialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/message.rs:221:35 [INFO] [stdout] | [INFO] [stdout] 221 | #[derive(Debug, Clone, Serialize, Deserialize)] [INFO] [stdout] | -^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `HelloData` is not local [INFO] [stdout] | `Deserialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_HelloData` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/message.rs:255:24 [INFO] [stdout] | [INFO] [stdout] 255 | #[derive(Debug, Clone, Serialize, Deserialize)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ServerSettingsData` is not local [INFO] [stdout] | `Serialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_SERIALIZE_FOR_ServerSettingsData` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Serialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Serialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/message.rs:255:35 [INFO] [stdout] | [INFO] [stdout] 255 | #[derive(Debug, Clone, Serialize, Deserialize)] [INFO] [stdout] | -^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ServerSettingsData` is not local [INFO] [stdout] | `Deserialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_ServerSettingsData` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/network_handler.rs:26:9 [INFO] [stdout] | [INFO] [stdout] 26 | stream.set_nonblocking(true); [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.set_nonblocking(true); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/network_handler.rs:60:17 [INFO] [stdout] | [INFO] [stdout] 60 | self.stream.write_all(&msg[..]); [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] 60 | let _ = self.stream.write_all(&msg[..]); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/network_handler.rs:68:17 [INFO] [stdout] | [INFO] [stdout] 68 | self.stream.write_all(&msg[..]); [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] 68 | let _ = self.stream.write_all(&msg[..]); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/network_handler.rs:73:17 [INFO] [stdout] | [INFO] [stdout] 73 | self.recv_message_channel.send(msg); [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] 73 | let _ = self.recv_message_channel.send(msg); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait method `setHeader` should have a snake case name [INFO] [stdout] --> src/decoder/mod.rs:10:8 [INFO] [stdout] | [INFO] [stdout] 10 | fn setHeader(&self, header: message::CodecHeaderData); [INFO] [stdout] | ^^^^^^^^^ help: convert the identifier to snake case: `set_header` [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 7.33s [INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: bitflags v0.8.2 [INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 1` [INFO] running `Command { std: "docker" "inspect" "676800df93603fc10b433da679dfcc3bc999b4dc4ccfecc049d6155c60d66935", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "676800df93603fc10b433da679dfcc3bc999b4dc4ccfecc049d6155c60d66935", kill_on_drop: false }` [INFO] [stdout] 676800df93603fc10b433da679dfcc3bc999b4dc4ccfecc049d6155c60d66935