[INFO] cloning repository https://github.com/lzqlzzq/midiparser [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/lzqlzzq/midiparser" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Flzqlzzq%2Fmidiparser", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Flzqlzzq%2Fmidiparser'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 01c9cbde3f56bf24ccc50551ad7a7f4a4915edb0 [INFO] testing lzqlzzq/midiparser against master#d98a5da813da67eb189387b8ccfb73cf481275d8+rustflags=-Copt-level=3 for pr-138759-retry [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Flzqlzzq%2Fmidiparser" "/workspace/builds/worker-5-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc1/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/lzqlzzq/midiparser [INFO] finished tweaking git repo https://github.com/lzqlzzq/midiparser [INFO] tweaked toml for git repo https://github.com/lzqlzzq/midiparser written to /workspace/builds/worker-5-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/lzqlzzq/midiparser on toolchain d98a5da813da67eb189387b8ccfb73cf481275d8 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/lzqlzzq/midiparser 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" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Updating crates.io index [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded ryu v1.0.15 [INFO] [stderr] Downloaded itoa v1.0.9 [INFO] [stderr] Downloaded unindent v0.1.11 [INFO] [stderr] Downloaded pyo3-macros v0.18.3 [INFO] [stderr] Downloaded memoffset v0.8.0 [INFO] [stderr] Downloaded indoc v1.0.9 [INFO] [stderr] Downloaded unsafe-libyaml v0.2.9 [INFO] [stderr] Downloaded lock_api v0.4.10 [INFO] [stderr] Downloaded parking_lot_core v0.9.8 [INFO] [stderr] Downloaded pyo3-ffi v0.18.3 [INFO] [stderr] Downloaded serde v1.0.183 [INFO] [stderr] Downloaded pyo3-macros-backend v0.18.3 [INFO] [stderr] Downloaded serde_yaml v0.9.25 [INFO] [stderr] Downloaded smallvec v1.11.0 [INFO] [stderr] Downloaded once_cell v1.18.0 [INFO] [stderr] Downloaded serde_json v1.0.105 [INFO] [stderr] Downloaded target-lexicon v0.12.11 [INFO] [stderr] Downloaded syn v2.0.29 [INFO] [stderr] Downloaded pyo3-build-config v0.18.3 [INFO] [stderr] Downloaded pyo3 v0.18.3 [INFO] [stderr] Downloaded serde_derive v1.0.183 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] c81ebe446c87264b350f460f32e2f41012d84a8c120711ea6e913c5d9fac0eb2 [INFO] running `Command { std: "docker" "start" "-a" "c81ebe446c87264b350f460f32e2f41012d84a8c120711ea6e913c5d9fac0eb2", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "c81ebe446c87264b350f460f32e2f41012d84a8c120711ea6e913c5d9fac0eb2", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "c81ebe446c87264b350f460f32e2f41012d84a8c120711ea6e913c5d9fac0eb2", kill_on_drop: false }` [INFO] [stdout] c81ebe446c87264b350f460f32e2f41012d84a8c120711ea6e913c5d9fac0eb2 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid -Copt-level=3" "-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:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 873a125f3ca579ead2cbc7469f845d7ba292a5b9a7d27355550321054fc4ff52 [INFO] running `Command { std: "docker" "start" "-a" "873a125f3ca579ead2cbc7469f845d7ba292a5b9a7d27355550321054fc4ff52", kill_on_drop: false }` [INFO] [stderr] Compiling target-lexicon v0.12.11 [INFO] [stderr] Compiling autocfg v1.1.0 [INFO] [stderr] Compiling once_cell v1.18.0 [INFO] [stderr] Compiling proc-macro2 v1.0.66 [INFO] [stderr] Compiling unicode-ident v1.0.11 [INFO] [stderr] Compiling libc v0.2.147 [INFO] [stderr] Compiling syn v1.0.109 [INFO] [stderr] Compiling parking_lot_core v0.9.8 [INFO] [stderr] Compiling serde v1.0.183 [INFO] [stderr] Compiling smallvec v1.11.0 [INFO] [stderr] Compiling serde_derive v1.0.183 [INFO] [stderr] Compiling hashbrown v0.14.0 [INFO] [stderr] Compiling ryu v1.0.15 [INFO] [stderr] Compiling serde_json v1.0.105 [INFO] [stderr] Compiling itoa v1.0.9 [INFO] [stderr] Compiling indoc v1.0.9 [INFO] [stderr] Compiling quote v1.0.33 [INFO] [stderr] Compiling indexmap v2.0.0 [INFO] [stderr] Compiling lock_api v0.4.10 [INFO] [stderr] Compiling memoffset v0.8.0 [INFO] [stderr] Compiling pyo3-build-config v0.18.3 [INFO] [stderr] Compiling unsafe-libyaml v0.2.9 [INFO] [stderr] Compiling parking_lot v0.12.1 [INFO] [stderr] Compiling unindent v0.1.11 [INFO] [stderr] Compiling serde_yaml v0.9.25 [INFO] [stderr] Compiling pyo3-macros-backend v0.18.3 [INFO] [stderr] Compiling pyo3-ffi v0.18.3 [INFO] [stderr] Compiling pyo3 v0.18.3 [INFO] [stderr] Compiling pyo3-macros v0.18.3 [INFO] [stderr] Compiling midiparse v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `MetaStatus` [INFO] [stdout] --> src/io.rs:3:60 [INFO] [stdout] | [INFO] [stdout] 3 | use crate::message::{MIDIFormat, EventStatus, MIDIMessage, MetaStatus}; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/sequence.rs:276:43 [INFO] [stdout] | [INFO] [stdout] 276 | let mut next_timesig_time = if(self.time_signatures.len() > 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 276 - let mut next_timesig_time = if(self.time_signatures.len() > 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] 276 + let mut next_timesig_time = if self.time_signatures.len() > 1 {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/sequence.rs:280:19 [INFO] [stdout] | [INFO] [stdout] 280 | if(note.start >= next_timesig_time) { [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 280 - if(note.start >= next_timesig_time) { [INFO] [stdout] 280 + if note.start >= next_timesig_time { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/sequence.rs:283:43 [INFO] [stdout] | [INFO] [stdout] 283 | ... next_timesig_time = if(this_timesig_idx < self.time_signatures.len() - 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 283 - next_timesig_time = if(this_timesig_idx < self.time_signatures.len() - 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] 283 + next_timesig_time = if this_timesig_idx < self.time_signatures.len() - 1 {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/sequence.rs:265:13 [INFO] [stdout] | [INFO] [stdout] 265 | for mut track in self.tracks.iter_mut() { [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: unused variable: `control_number` [INFO] [stdout] --> src/sequence.rs:317:14 [INFO] [stdout] | [INFO] [stdout] 317 | for (control_number, control_change) in self.controls.iter_mut() { [INFO] [stdout] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_control_number` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `track_idx` is never read [INFO] [stdout] --> src/io.rs:15:5 [INFO] [stdout] | [INFO] [stdout] 14 | pub struct MidiTrack { [INFO] [stdout] | --------- field in this struct [INFO] [stdout] 15 | track_idx: u16, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `MidiTrack` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: lifetime flowing from input to output with different syntax can be confusing [INFO] [stdout] --> src/io.rs:30:17 [INFO] [stdout] | [INFO] [stdout] 30 | pub fn iter(&self) -> MidiTrackIter { [INFO] [stdout] | ^^^^^ ------------- the lifetime gets resolved as `'_` [INFO] [stdout] | | [INFO] [stdout] | this lifetime flows to the output [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stdout] | [INFO] [stdout] 30 | pub fn iter(&self) -> MidiTrackIter<'_> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 1m 14s [INFO] running `Command { std: "docker" "inspect" "873a125f3ca579ead2cbc7469f845d7ba292a5b9a7d27355550321054fc4ff52", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "873a125f3ca579ead2cbc7469f845d7ba292a5b9a7d27355550321054fc4ff52", kill_on_drop: false }` [INFO] [stdout] 873a125f3ca579ead2cbc7469f845d7ba292a5b9a7d27355550321054fc4ff52 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid -Copt-level=3" "-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:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] d38774cba867a6181d3343f2690ae303088f05b3a8685e6e29dc77764ec52e82 [INFO] running `Command { std: "docker" "start" "-a" "d38774cba867a6181d3343f2690ae303088f05b3a8685e6e29dc77764ec52e82", kill_on_drop: false }` [INFO] [stderr] Compiling midiparse v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/sequence.rs:276:43 [INFO] [stdout] | [INFO] [stdout] 276 | let mut next_timesig_time = if(self.time_signatures.len() > 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 276 - let mut next_timesig_time = if(self.time_signatures.len() > 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] 276 + let mut next_timesig_time = if self.time_signatures.len() > 1 {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/sequence.rs:280:19 [INFO] [stdout] | [INFO] [stdout] 280 | if(note.start >= next_timesig_time) { [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 280 - if(note.start >= next_timesig_time) { [INFO] [stdout] 280 + if note.start >= next_timesig_time { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/sequence.rs:283:43 [INFO] [stdout] | [INFO] [stdout] 283 | ... next_timesig_time = if(this_timesig_idx < self.time_signatures.len() - 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 283 - next_timesig_time = if(this_timesig_idx < self.time_signatures.len() - 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] 283 + next_timesig_time = if this_timesig_idx < self.time_signatures.len() - 1 {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `serde_json` [INFO] [stdout] --> src/sequence.rs:398:9 [INFO] [stdout] | [INFO] [stdout] 398 | use serde_json; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/sequence.rs:265:13 [INFO] [stdout] | [INFO] [stdout] 265 | for mut track in self.tracks.iter_mut() { [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: unused variable: `control_number` [INFO] [stdout] --> src/sequence.rs:317:14 [INFO] [stdout] | [INFO] [stdout] 317 | for (control_number, control_change) in self.controls.iter_mut() { [INFO] [stdout] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_control_number` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `track_idx` is never read [INFO] [stdout] --> src/io.rs:15:5 [INFO] [stdout] | [INFO] [stdout] 14 | pub struct MidiTrack { [INFO] [stdout] | --------- field in this struct [INFO] [stdout] 15 | track_idx: u16, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `MidiTrack` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: lifetime flowing from input to output with different syntax can be confusing [INFO] [stdout] --> src/io.rs:30:17 [INFO] [stdout] | [INFO] [stdout] 30 | pub fn iter(&self) -> MidiTrackIter { [INFO] [stdout] | ^^^^^ ------------- the lifetime gets resolved as `'_` [INFO] [stdout] | | [INFO] [stdout] | this lifetime flows to the output [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stdout] | [INFO] [stdout] 30 | pub fn iter(&self) -> MidiTrackIter<'_> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `test` profile [unoptimized + debuginfo] target(s) in 6.21s [INFO] running `Command { std: "docker" "inspect" "d38774cba867a6181d3343f2690ae303088f05b3a8685e6e29dc77764ec52e82", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "d38774cba867a6181d3343f2690ae303088f05b3a8685e6e29dc77764ec52e82", kill_on_drop: false }` [INFO] [stdout] d38774cba867a6181d3343f2690ae303088f05b3a8685e6e29dc77764ec52e82 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid -Copt-level=3" "-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:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "test" "--frozen", kill_on_drop: false }` [INFO] [stdout] d60308476c126ae87314f3efca5cf333d6cc76c74ff2099a092b46caa8254476 [INFO] running `Command { std: "docker" "start" "-a" "d60308476c126ae87314f3efca5cf333d6cc76c74ff2099a092b46caa8254476", kill_on_drop: false }` [INFO] [stderr] warning: unnecessary parentheses around `if` condition [INFO] [stderr] --> src/sequence.rs:276:43 [INFO] [stderr] | [INFO] [stderr] 276 | let mut next_timesig_time = if(self.time_signatures.len() > 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stderr] | ^ ^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_parens)]` on by default [INFO] [stderr] help: remove these parentheses [INFO] [stderr] | [INFO] [stderr] 276 - let mut next_timesig_time = if(self.time_signatures.len() > 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stderr] 276 + let mut next_timesig_time = if self.time_signatures.len() > 1 {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: unnecessary parentheses around `if` condition [INFO] [stderr] --> src/sequence.rs:280:19 [INFO] [stderr] | [INFO] [stderr] 280 | if(note.start >= next_timesig_time) { [INFO] [stderr] | ^ ^ [INFO] [stderr] | [INFO] [stderr] help: remove these parentheses [INFO] [stderr] | [INFO] [stderr] 280 - if(note.start >= next_timesig_time) { [INFO] [stderr] 280 + if note.start >= next_timesig_time { [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: unnecessary parentheses around `if` condition [INFO] [stderr] --> src/sequence.rs:283:43 [INFO] [stderr] | [INFO] [stderr] 283 | ... next_timesig_time = if(this_timesig_idx < self.time_signatures.len() - 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stderr] | ^ ^ [INFO] [stderr] | [INFO] [stderr] help: remove these parentheses [INFO] [stderr] | [INFO] [stderr] 283 - next_timesig_time = if(this_timesig_idx < self.time_signatures.len() - 1) {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stderr] 283 + next_timesig_time = if this_timesig_idx < self.time_signatures.len() - 1 {self.time_signatures[1].time} else {f32::MAX}; [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: unused import: `serde_json` [INFO] [stderr] --> src/sequence.rs:398:9 [INFO] [stderr] | [INFO] [stderr] 398 | use serde_json; [INFO] [stderr] | ^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_imports)]` on by default [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/sequence.rs:265:13 [INFO] [stderr] | [INFO] [stderr] 265 | for mut track in self.tracks.iter_mut() { [INFO] [stderr] | ----^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_mut)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused variable: `control_number` [INFO] [stderr] --> src/sequence.rs:317:14 [INFO] [stderr] | [INFO] [stderr] 317 | for (control_number, control_change) in self.controls.iter_mut() { [INFO] [stderr] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_control_number` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_variables)]` on by default [INFO] [stderr] [INFO] [stderr] warning: field `track_idx` is never read [INFO] [stderr] --> src/io.rs:15:5 [INFO] [stderr] | [INFO] [stderr] 14 | pub struct MidiTrack { [INFO] [stderr] | --------- field in this struct [INFO] [stderr] 15 | track_idx: u16, [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `MidiTrack` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis [INFO] [stderr] = note: `#[warn(dead_code)]` on by default [INFO] [stderr] [INFO] [stderr] warning: lifetime flowing from input to output with different syntax can be confusing [INFO] [stderr] --> src/io.rs:30:17 [INFO] [stderr] | [INFO] [stderr] 30 | pub fn iter(&self) -> MidiTrackIter { [INFO] [stderr] | ^^^^^ ------------- the lifetime gets resolved as `'_` [INFO] [stderr] | | [INFO] [stderr] | this lifetime flows to the output [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stderr] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stderr] | [INFO] [stderr] 30 | pub fn iter(&self) -> MidiTrackIter<'_> { [INFO] [stderr] | ++++ [INFO] [stderr] [INFO] [stderr] warning: `midiparse` (lib test) generated 8 warnings (run `cargo fix --lib -p midiparse --tests` to apply 5 suggestions) [INFO] [stderr] Finished `test` profile [unoptimized + debuginfo] target(s) in 0.06s [INFO] [stderr] Running unittests src/lib.rs (/opt/rustwide/target/debug/deps/midiparse_core-1174d82559fb0391) [INFO] [stdout] [INFO] [stdout] running 3 tests [INFO] [stdout] test util::tests::test_read_vlq ... ok [INFO] [stdout] test io::tests::test_read_midi_head ... ok [INFO] [stdout] test sequence::tests::test_midi2seq ... ok [INFO] [stdout] [INFO] [stdout] test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.01s [INFO] [stdout] [INFO] running `Command { std: "docker" "inspect" "d60308476c126ae87314f3efca5cf333d6cc76c74ff2099a092b46caa8254476", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "d60308476c126ae87314f3efca5cf333d6cc76c74ff2099a092b46caa8254476", kill_on_drop: false }` [INFO] [stdout] d60308476c126ae87314f3efca5cf333d6cc76c74ff2099a092b46caa8254476