[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] building pajowu/snaprust against master#44f415c1d617ebc7b931a243b7b321ef8a6ca47c for pr-142134-abi-ast-error [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] validating manifest of git repo https://github.com/pajowu/snaprust on toolchain 44f415c1d617ebc7b931a243b7b321ef8a6ca47c [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+44f415c1d617ebc7b931a243b7b321ef8a6ca47c" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [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] 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" "+44f415c1d617ebc7b931a243b7b321ef8a6ca47c" "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] Downloading crates ... [INFO] [stderr] Downloaded alsa-sys v0.1.1 [INFO] [stderr] Downloaded simplelog v0.4.2 [INFO] [stderr] Downloaded coreaudio-rs v0.6.0 [INFO] [stderr] Downloaded hound v1.1.0 [INFO] [stderr] Downloaded ogg v0.4.1 [INFO] [stderr] Downloaded cpal v0.4.5 [INFO] [stderr] Downloaded hound v3.1.0 [INFO] [stderr] Downloaded alsa v0.1.10 [INFO] [stderr] Downloaded lewton v0.5.2 [INFO] [stderr] Downloaded claxon v0.3.1 [INFO] [stderr] Downloaded coreaudio-sys v0.1.2 [INFO] [stderr] Downloaded rodio v0.5.2 [INFO] [stderr] Downloaded clap v2.26.1 [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:eea15d5475069c3ef791f10c2c6b5af2ee421ef2da1e680ecce1cba46243983b" "/opt/rustwide/cargo-home/bin/cargo" "+44f415c1d617ebc7b931a243b7b321ef8a6ca47c" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 32ffadedad7e59f2410d17818acabffefab33f6a3ea1a994cfc1f22cf291ebdd [INFO] running `Command { std: "docker" "start" "-a" "32ffadedad7e59f2410d17818acabffefab33f6a3ea1a994cfc1f22cf291ebdd", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "32ffadedad7e59f2410d17818acabffefab33f6a3ea1a994cfc1f22cf291ebdd", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "32ffadedad7e59f2410d17818acabffefab33f6a3ea1a994cfc1f22cf291ebdd", kill_on_drop: false }` [INFO] [stdout] 32ffadedad7e59f2410d17818acabffefab33f6a3ea1a994cfc1f22cf291ebdd [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=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:eea15d5475069c3ef791f10c2c6b5af2ee421ef2da1e680ecce1cba46243983b" "/opt/rustwide/cargo-home/bin/cargo" "+44f415c1d617ebc7b931a243b7b321ef8a6ca47c" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 29efd115a8bdefe63e197302557847c17922e0de2a56934a76d8a9cff540f781 [INFO] running `Command { std: "docker" "start" "-a" "29efd115a8bdefe63e197302557847c17922e0de2a56934a76d8a9cff540f781", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024 [INFO] [stderr] Compiling libc v0.2.30 [INFO] [stderr] Compiling pkg-config v0.3.9 [INFO] [stderr] Compiling unicode-xid v0.0.4 [INFO] [stderr] Compiling byteorder v1.1.0 [INFO] [stderr] Compiling num-traits v0.1.40 [INFO] [stderr] Compiling quote v0.3.15 [INFO] [stderr] Compiling unicode-width v0.1.4 [INFO] [stderr] Compiling lazy_static v0.2.9 [INFO] [stderr] Compiling synom v0.11.3 [INFO] [stderr] Compiling approx v0.1.1 [INFO] [stderr] Compiling futures v0.1.16 [INFO] [stderr] Compiling lazy_static v0.1.16 [INFO] [stderr] Compiling ogg v0.4.1 [INFO] [stderr] Compiling itoa v0.3.4 [INFO] [stderr] Compiling strsim v0.6.0 [INFO] [stderr] Compiling rand v0.3.16 [INFO] [stderr] Compiling alsa-sys v0.1.1 [INFO] [stderr] Compiling term_size v0.3.0 [INFO] [stderr] Compiling syn v0.11.11 [INFO] [stderr] Compiling textwrap v0.8.0 [INFO] [stderr] Compiling lewton v0.5.2 [INFO] [stderr] Compiling cgmath v0.14.1 [INFO] [stderr] Compiling time v0.1.38 [INFO] [stderr] Compiling atty v0.2.2 [INFO] [stderr] Compiling ansi_term v0.9.0 [INFO] [stderr] Compiling hound v1.1.0 [INFO] [stderr] Compiling term v0.4.6 [INFO] [stderr] Compiling cpal v0.4.5 [INFO] [stderr] Compiling bitflags v0.8.2 [INFO] [stderr] Compiling serde_derive_internals v0.16.0 [INFO] [stderr] Compiling claxon v0.3.1 [INFO] [stderr] Compiling dtoa v0.4.2 [INFO] [stderr] Compiling serde v1.0.15 [INFO] [stderr] Compiling bitflags v0.9.1 [INFO] [stderr] Compiling vec_map v0.8.0 [INFO] [stderr] Compiling log v0.3.8 [INFO] [stderr] Compiling alsa v0.1.10 [INFO] [stderr] Compiling simplelog v0.4.2 [INFO] [stderr] Compiling clap v2.26.1 [INFO] [stderr] Compiling serde_derive v1.0.15 [INFO] [stderr] Compiling rodio v0.5.2 [INFO] [stderr] Compiling hound v3.1.0 [INFO] [stderr] Compiling serde_json v1.0.3 [INFO] [stderr] Compiling 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)]` 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)]` 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)]` 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)]` 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)]` 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)]` 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)]` 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)]` 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)]` 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 18.20s [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" "29efd115a8bdefe63e197302557847c17922e0de2a56934a76d8a9cff540f781", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "29efd115a8bdefe63e197302557847c17922e0de2a56934a76d8a9cff540f781", kill_on_drop: false }` [INFO] [stdout] 29efd115a8bdefe63e197302557847c17922e0de2a56934a76d8a9cff540f781 [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=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:eea15d5475069c3ef791f10c2c6b5af2ee421ef2da1e680ecce1cba46243983b" "/opt/rustwide/cargo-home/bin/cargo" "+44f415c1d617ebc7b931a243b7b321ef8a6ca47c" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 221635db5633e348bf10b911292ba5e8e98944696e95fd6ea12c4c2d63087e03 [INFO] running `Command { std: "docker" "start" "-a" "221635db5633e348bf10b911292ba5e8e98944696e95fd6ea12c4c2d63087e03", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024 [INFO] [stderr] Compiling 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)]` 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)]` 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)]` 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)]` 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)]` 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)]` 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)]` 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)]` 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)]` 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 `test` profile [unoptimized + debuginfo] target(s) in 0.58s [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" "221635db5633e348bf10b911292ba5e8e98944696e95fd6ea12c4c2d63087e03", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "221635db5633e348bf10b911292ba5e8e98944696e95fd6ea12c4c2d63087e03", kill_on_drop: false }` [INFO] [stdout] 221635db5633e348bf10b911292ba5e8e98944696e95fd6ea12c4c2d63087e03