[INFO] fetching crate rsnbs 0.1.2...
[INFO] building rsnbs-0.1.2 against master#c7f6aa2869acdbf014d094c6e427e554e160b6db for pr-146237-2
[INFO] extracting crate rsnbs 0.1.2 into /workspace/builds/worker-4-tc1/source
[INFO] started tweaking crates.io crate rsnbs 0.1.2
[INFO] finished tweaking crates.io crate rsnbs 0.1.2
[INFO] tweaked toml for crates.io crate rsnbs 0.1.2 written to /workspace/builds/worker-4-tc1/source/Cargo.toml
[INFO] validating manifest of crates.io crate rsnbs 0.1.2 on toolchain c7f6aa2869acdbf014d094c6e427e554e160b6db
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+c7f6aa2869acdbf014d094c6e427e554e160b6db" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+c7f6aa2869acdbf014d094c6e427e554e160b6db" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]      Locking 3 packages to latest compatible versions
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+c7f6aa2869acdbf014d094c6e427e554e160b6db" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+c7f6aa2869acdbf014d094c6e427e554e160b6db" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] eec92cb25ba104a4b925c0761a72bf578f0ded71bc0e15c210d8f94ec1ddca95
[INFO] running `Command { std: "docker" "start" "-a" "eec92cb25ba104a4b925c0761a72bf578f0ded71bc0e15c210d8f94ec1ddca95", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "eec92cb25ba104a4b925c0761a72bf578f0ded71bc0e15c210d8f94ec1ddca95", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "eec92cb25ba104a4b925c0761a72bf578f0ded71bc0e15c210d8f94ec1ddca95", kill_on_drop: false }`
[INFO] [stdout] eec92cb25ba104a4b925c0761a72bf578f0ded71bc0e15c210d8f94ec1ddca95
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+c7f6aa2869acdbf014d094c6e427e554e160b6db" "build" "--frozen" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 589a6069f655678c35a225675bcd16b77c6c084a7f524c848b66d139de54180a
[INFO] running `Command { std: "docker" "start" "-a" "589a6069f655678c35a225675bcd16b77c6c084a7f524c848b66d139de54180a", kill_on_drop: false }`
[INFO] [stderr]    Compiling rsnbs v0.1.2 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `Header`
[INFO] [stdout]  --> src/parser.rs:1:37
[INFO] [stdout]   |
[INFO] [stdout] 1 | use super::{Song, BinaryMut, Layer, Header, Note, Instrument};
[INFO] [stdout]   |                                     ^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::mem::transmute`
[INFO] [stdout]   --> src/parser.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use std::mem::transmute;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait `from_bytes` should have an upper camel case name
[INFO] [stdout]   --> src/parser.rs:17:7
[INFO] [stdout]    |
[INFO] [stdout] 17 | trait from_bytes<T> {fn from_bytes(bytes: Vec<u8>) -> T;}
[INFO] [stdout]    |       ^^^^^^^^^^ help: convert the identifier to upper camel case: `FromBytes`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_camel_case_types)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: denote infinite loops with `loop { ... }`
[INFO] [stdout]   --> src/parser.rs:73:5
[INFO] [stdout]    |
[INFO] [stdout] 73 |     while true {
[INFO] [stdout]    |     ^^^^^^^^^^ help: use `loop`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(while_true)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `super::super::parser::read_nbs`
[INFO] [stdout]  --> src/tests.rs:2:9
[INFO] [stdout]   |
[INFO] [stdout] 2 |     use super::super::parser::read_nbs;
[INFO] [stdout]   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `super::super::Song`
[INFO] [stdout]  --> src/tests.rs:3:9
[INFO] [stdout]   |
[INFO] [stdout] 3 |     use super::super::Song;
[INFO] [stdout]   |         ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `diff::chars`
[INFO] [stdout]  --> src/tests.rs:4:9
[INFO] [stdout]   |
[INFO] [stdout] 4 |     use diff::chars;
[INFO] [stdout]   |         ^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait `writable_bin` should have an upper camel case name
[INFO] [stdout]   --> src/writer.rs:27:7
[INFO] [stdout]    |
[INFO] [stdout] 27 | trait writable_bin {
[INFO] [stdout]    |       ^^^^^^^^^^^^ help: convert the identifier to upper camel case: `WritableBin`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/parser.rs:21:16
[INFO] [stdout]    |
[INFO] [stdout] 21 | fn read_string(mut file: File) -> Option<String> {
[INFO] [stdout]    |                ----^^^^
[INFO] [stdout]    |                |
[INFO] [stdout]    |                help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `version`
[INFO] [stdout]   --> src/parser.rs:43:18
[INFO] [stdout]    |
[INFO] [stdout] 43 |     for (binary, version) in part {
[INFO] [stdout]    |                  ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_version`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:15:34
[INFO] [stdout]    |
[INFO] [stdout] 15 |                 fn write_default(mut file: File) -> Option<Self> {
[INFO] [stdout]    |                                  ----^^^^
[INFO] [stdout]    |                                  |
[INFO] [stdout]    |                                  help: remove this `mut`
[INFO] [stdout] ...
[INFO] [stdout] 32 | impl_writable_bin!([u8, i8, i16, i32,]);
[INFO] [stdout]    | --------------------------------------- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = note: this warning originates in the macro `impl_writable_bin` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:43:22
[INFO] [stdout]    |
[INFO] [stdout] 43 |     fn write_default(mut file: File) -> Option<Self> {
[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/writer.rs:57:22
[INFO] [stdout]    |
[INFO] [stdout] 57 |     fn write_default(mut file: File) -> Option<Self> {
[INFO] [stdout]    |                      ----^^^^
[INFO] [stdout]    |                      |
[INFO] [stdout]    |                      help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `version`
[INFO] [stdout]   --> src/writer.rs:74:18
[INFO] [stdout]    |
[INFO] [stdout] 74 |     for (binary, version) in part {
[INFO] [stdout]    |                  ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_version`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:73:15
[INFO] [stdout]    |
[INFO] [stdout] 73 | fn write_part(mut file: File, part: Vec<(Binary, u8)>) -> Option<()> {
[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/writer.rs:96:13
[INFO] [stdout]    |
[INFO] [stdout] 96 |         let mut file = match OpenOptions::new().write(true).open(filename) {
[INFO] [stdout]    |             ----^^^^
[INFO] [stdout]    |             |
[INFO] [stdout]    |             help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `TEST_FILES` is never used
[INFO] [stdout]  --> src/tests.rs:5:11
[INFO] [stdout]   |
[INFO] [stdout] 5 |     const TEST_FILES: [&str;2] = ["test_files/Megalovania.nbs", "test_files/DB.nbs"];
[INFO] [stdout]   |           ^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/fields.rs:34:35
[INFO] [stdout]    |
[INFO] [stdout] 34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]    |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                                     |
[INFO] [stdout]    |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 64 | / create_iterable_struct!(
[INFO] [stdout] 65 | |     Header,
[INFO] [stdout] 66 | |     [
[INFO] [stdout] 67 | |         (classic_length: i16: Short): 0,
[INFO] [stdout] ...  |
[INFO] [stdout] 90 | | );
[INFO] [stdout]    | |_- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout]    = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]    |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/fields.rs:41:35
[INFO] [stdout]    |
[INFO] [stdout] 41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]    |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                                     |
[INFO] [stdout]    |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 64 | / create_iterable_struct!(
[INFO] [stdout] 65 | |     Header,
[INFO] [stdout] 66 | |     [
[INFO] [stdout] 67 | |         (classic_length: i16: Short): 0,
[INFO] [stdout] ...  |
[INFO] [stdout] 90 | | );
[INFO] [stdout]    | |_- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]    |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout]  92 | / create_iterable_struct!(
[INFO] [stdout]  93 | |     Note,
[INFO] [stdout]  94 | |     [   
[INFO] [stdout]  95 | |         (tick: i32: Integer): 127,
[INFO] [stdout] ...   |
[INFO] [stdout] 103 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout]  92 | / create_iterable_struct!(
[INFO] [stdout]  93 | |     Note,
[INFO] [stdout]  94 | |     [   
[INFO] [stdout]  95 | |         (tick: i32: Integer): 127,
[INFO] [stdout] ...   |
[INFO] [stdout] 103 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 105 | / create_iterable_struct!(
[INFO] [stdout] 106 | |     Layer,
[INFO] [stdout] 107 | |     [
[INFO] [stdout] 108 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 113 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 105 | / create_iterable_struct!(
[INFO] [stdout] 106 | |     Layer,
[INFO] [stdout] 107 | |     [
[INFO] [stdout] 108 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 113 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 115 | / create_iterable_struct!(
[INFO] [stdout] 116 | |     Instrument,
[INFO] [stdout] 117 | |     [
[INFO] [stdout] 118 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 123 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 115 | / create_iterable_struct!(
[INFO] [stdout] 116 | |     Instrument,
[INFO] [stdout] 117 | |     [
[INFO] [stdout] 118 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 123 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.66s
[INFO] running `Command { std: "docker" "inspect" "589a6069f655678c35a225675bcd16b77c6c084a7f524c848b66d139de54180a", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "589a6069f655678c35a225675bcd16b77c6c084a7f524c848b66d139de54180a", kill_on_drop: false }`
[INFO] [stdout] 589a6069f655678c35a225675bcd16b77c6c084a7f524c848b66d139de54180a
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+c7f6aa2869acdbf014d094c6e427e554e160b6db" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 97357163c80ce0404986e07037060e50f2e7e121f0f74aed512b1bc7514a2d9d
[INFO] running `Command { std: "docker" "start" "-a" "97357163c80ce0404986e07037060e50f2e7e121f0f74aed512b1bc7514a2d9d", kill_on_drop: false }`
[INFO] [stdout] warning: unused import: `Header`
[INFO] [stdout]  --> src/parser.rs:1:37
[INFO] [stdout]   |
[INFO] [stdout] 1 | use super::{Song, BinaryMut, Layer, Header, Note, Instrument};
[INFO] [stdout]   |                                     ^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::mem::transmute`
[INFO] [stdout]   --> src/parser.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use std::mem::transmute;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait `from_bytes` should have an upper camel case name
[INFO] [stdout]   --> src/parser.rs:17:7
[INFO] [stdout]    |
[INFO] [stdout] 17 | trait from_bytes<T> {fn from_bytes(bytes: Vec<u8>) -> T;}
[INFO] [stdout]    |       ^^^^^^^^^^ help: convert the identifier to upper camel case: `FromBytes`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_camel_case_types)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: denote infinite loops with `loop { ... }`
[INFO] [stdout]   --> src/parser.rs:73:5
[INFO] [stdout]    |
[INFO] [stdout] 73 |     while true {
[INFO] [stdout]    |     ^^^^^^^^^^ help: use `loop`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(while_true)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `super::super::parser::read_nbs`
[INFO] [stdout]  --> src/tests.rs:2:9
[INFO] [stdout]   |
[INFO] [stdout] 2 |     use super::super::parser::read_nbs;
[INFO] [stdout]   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `super::super::Song`
[INFO] [stdout]  --> src/tests.rs:3:9
[INFO] [stdout]   |
[INFO] [stdout] 3 |     use super::super::Song;
[INFO] [stdout]   |         ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `diff::chars`
[INFO] [stdout]  --> src/tests.rs:4:9
[INFO] [stdout]   |
[INFO] [stdout] 4 |     use diff::chars;
[INFO] [stdout]   |         ^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait `writable_bin` should have an upper camel case name
[INFO] [stdout]   --> src/writer.rs:27:7
[INFO] [stdout]    |
[INFO] [stdout] 27 | trait writable_bin {
[INFO] [stdout]    |       ^^^^^^^^^^^^ help: convert the identifier to upper camel case: `WritableBin`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/parser.rs:21:16
[INFO] [stdout]    |
[INFO] [stdout] 21 | fn read_string(mut file: File) -> Option<String> {
[INFO] [stdout]    |                ----^^^^
[INFO] [stdout]    |                |
[INFO] [stdout]    |                help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `version`
[INFO] [stdout]   --> src/parser.rs:43:18
[INFO] [stdout]    |
[INFO] [stdout] 43 |     for (binary, version) in part {
[INFO] [stdout]    |                  ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_version`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:15:34
[INFO] [stdout]    |
[INFO] [stdout] 15 |                 fn write_default(mut file: File) -> Option<Self> {
[INFO] [stdout]    |                                  ----^^^^
[INFO] [stdout]    |                                  |
[INFO] [stdout]    |                                  help: remove this `mut`
[INFO] [stdout] ...
[INFO] [stdout] 32 | impl_writable_bin!([u8, i8, i16, i32,]);
[INFO] [stdout]    | --------------------------------------- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = note: this warning originates in the macro `impl_writable_bin` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:43:22
[INFO] [stdout]    |
[INFO] [stdout] 43 |     fn write_default(mut file: File) -> Option<Self> {
[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/writer.rs:57:22
[INFO] [stdout]    |
[INFO] [stdout] 57 |     fn write_default(mut file: File) -> Option<Self> {
[INFO] [stdout]    |                      ----^^^^
[INFO] [stdout]    |                      |
[INFO] [stdout]    |                      help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `version`
[INFO] [stdout]   --> src/writer.rs:74:18
[INFO] [stdout]    |
[INFO] [stdout] 74 |     for (binary, version) in part {
[INFO] [stdout]    |                  ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_version`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:73:15
[INFO] [stdout]    |
[INFO] [stdout] 73 | fn write_part(mut file: File, part: Vec<(Binary, u8)>) -> Option<()> {
[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/writer.rs:96:13
[INFO] [stdout]    |
[INFO] [stdout] 96 |         let mut file = match OpenOptions::new().write(true).open(filename) {
[INFO] [stdout]    |             ----^^^^
[INFO] [stdout]    |             |
[INFO] [stdout]    |             help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `TEST_FILES` is never used
[INFO] [stdout]  --> src/tests.rs:5:11
[INFO] [stdout]   |
[INFO] [stdout] 5 |     const TEST_FILES: [&str;2] = ["test_files/Megalovania.nbs", "test_files/DB.nbs"];
[INFO] [stdout]   |           ^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/fields.rs:34:35
[INFO] [stdout]    |
[INFO] [stdout] 34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]    |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                                     |
[INFO] [stdout]    |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 64 | / create_iterable_struct!(
[INFO] [stdout] 65 | |     Header,
[INFO] [stdout] 66 | |     [
[INFO] [stdout] 67 | |         (classic_length: i16: Short): 0,
[INFO] [stdout] ...  |
[INFO] [stdout] 90 | | );
[INFO] [stdout]    | |_- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout]    = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]    |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/fields.rs:41:35
[INFO] [stdout]    |
[INFO] [stdout] 41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]    |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                                     |
[INFO] [stdout]    |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 64 | / create_iterable_struct!(
[INFO] [stdout] 65 | |     Header,
[INFO] [stdout] 66 | |     [
[INFO] [stdout] 67 | |         (classic_length: i16: Short): 0,
[INFO] [stdout] ...  |
[INFO] [stdout] 90 | | );
[INFO] [stdout]    | |_- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]    |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout]  92 | / create_iterable_struct!(
[INFO] [stdout]  93 | |     Note,
[INFO] [stdout]  94 | |     [   
[INFO] [stdout]  95 | |         (tick: i32: Integer): 127,
[INFO] [stdout] ...   |
[INFO] [stdout] 103 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout]  92 | / create_iterable_struct!(
[INFO] [stdout]  93 | |     Note,
[INFO] [stdout]  94 | |     [   
[INFO] [stdout]  95 | |         (tick: i32: Integer): 127,
[INFO] [stdout] ...   |
[INFO] [stdout] 103 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 105 | / create_iterable_struct!(
[INFO] [stdout] 106 | |     Layer,
[INFO] [stdout] 107 | |     [
[INFO] [stdout] 108 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 113 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 105 | / create_iterable_struct!(
[INFO] [stdout] 106 | |     Layer,
[INFO] [stdout] 107 | |     [
[INFO] [stdout] 108 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 113 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 115 | / create_iterable_struct!(
[INFO] [stdout] 116 | |     Instrument,
[INFO] [stdout] 117 | |     [
[INFO] [stdout] 118 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 123 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]    Compiling rsnbs v0.1.2 (/opt/rustwide/workdir)
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 115 | / create_iterable_struct!(
[INFO] [stdout] 116 | |     Instrument,
[INFO] [stdout] 117 | |     [
[INFO] [stdout] 118 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 123 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Header`
[INFO] [stdout]  --> src/parser.rs:1:37
[INFO] [stdout]   |
[INFO] [stdout] 1 | use super::{Song, BinaryMut, Layer, Header, Note, Instrument};
[INFO] [stdout]   |                                     ^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::mem::transmute`
[INFO] [stdout]   --> src/parser.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use std::mem::transmute;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait `from_bytes` should have an upper camel case name
[INFO] [stdout]   --> src/parser.rs:17:7
[INFO] [stdout]    |
[INFO] [stdout] 17 | trait from_bytes<T> {fn from_bytes(bytes: Vec<u8>) -> T;}
[INFO] [stdout]    |       ^^^^^^^^^^ help: convert the identifier to upper camel case: `FromBytes`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_camel_case_types)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: denote infinite loops with `loop { ... }`
[INFO] [stdout]   --> src/parser.rs:73:5
[INFO] [stdout]    |
[INFO] [stdout] 73 |     while true {
[INFO] [stdout]    |     ^^^^^^^^^^ help: use `loop`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(while_true)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `super::super::Song`
[INFO] [stdout]  --> src/tests.rs:3:9
[INFO] [stdout]   |
[INFO] [stdout] 3 |     use super::super::Song;
[INFO] [stdout]   |         ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around assigned value
[INFO] [stdout]   --> src/tests.rs:11:30
[INFO] [stdout]    |
[INFO] [stdout] 11 |         let test_file_dest = (test_file.to_string() + "Test");
[INFO] [stdout]    |                              ^                              ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 11 -         let test_file_dest = (test_file.to_string() + "Test");
[INFO] [stdout] 11 +         let test_file_dest = test_file.to_string() + "Test" ;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait `writable_bin` should have an upper camel case name
[INFO] [stdout]   --> src/writer.rs:27:7
[INFO] [stdout]    |
[INFO] [stdout] 27 | trait writable_bin {
[INFO] [stdout]    |       ^^^^^^^^^^^^ help: convert the identifier to upper camel case: `WritableBin`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/parser.rs:21:16
[INFO] [stdout]    |
[INFO] [stdout] 21 | fn read_string(mut file: File) -> Option<String> {
[INFO] [stdout]    |                ----^^^^
[INFO] [stdout]    |                |
[INFO] [stdout]    |                help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `version`
[INFO] [stdout]   --> src/parser.rs:43:18
[INFO] [stdout]    |
[INFO] [stdout] 43 |     for (binary, version) in part {
[INFO] [stdout]    |                  ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_version`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:15:34
[INFO] [stdout]    |
[INFO] [stdout] 15 |                 fn write_default(mut file: File) -> Option<Self> {
[INFO] [stdout]    |                                  ----^^^^
[INFO] [stdout]    |                                  |
[INFO] [stdout]    |                                  help: remove this `mut`
[INFO] [stdout] ...
[INFO] [stdout] 32 | impl_writable_bin!([u8, i8, i16, i32,]);
[INFO] [stdout]    | --------------------------------------- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = note: this warning originates in the macro `impl_writable_bin` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:43:22
[INFO] [stdout]    |
[INFO] [stdout] 43 |     fn write_default(mut file: File) -> Option<Self> {
[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/writer.rs:57:22
[INFO] [stdout]    |
[INFO] [stdout] 57 |     fn write_default(mut file: File) -> Option<Self> {
[INFO] [stdout]    |                      ----^^^^
[INFO] [stdout]    |                      |
[INFO] [stdout]    |                      help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `version`
[INFO] [stdout]   --> src/writer.rs:74:18
[INFO] [stdout]    |
[INFO] [stdout] 74 |     for (binary, version) in part {
[INFO] [stdout]    |                  ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_version`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/writer.rs:73:15
[INFO] [stdout]    |
[INFO] [stdout] 73 | fn write_part(mut file: File, part: Vec<(Binary, u8)>) -> Option<()> {
[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/writer.rs:96:13
[INFO] [stdout]    |
[INFO] [stdout] 96 |         let mut file = match OpenOptions::new().write(true).open(filename) {
[INFO] [stdout]    |             ----^^^^
[INFO] [stdout]    |             |
[INFO] [stdout]    |             help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/fields.rs:34:35
[INFO] [stdout]    |
[INFO] [stdout] 34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]    |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                                     |
[INFO] [stdout]    |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 64 | / create_iterable_struct!(
[INFO] [stdout] 65 | |     Header,
[INFO] [stdout] 66 | |     [
[INFO] [stdout] 67 | |         (classic_length: i16: Short): 0,
[INFO] [stdout] ...  |
[INFO] [stdout] 90 | | );
[INFO] [stdout]    | |_- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout]    = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]    |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/fields.rs:41:35
[INFO] [stdout]    |
[INFO] [stdout] 41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]    |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                                     |
[INFO] [stdout]    |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 64 | / create_iterable_struct!(
[INFO] [stdout] 65 | |     Header,
[INFO] [stdout] 66 | |     [
[INFO] [stdout] 67 | |         (classic_length: i16: Short): 0,
[INFO] [stdout] ...  |
[INFO] [stdout] 90 | | );
[INFO] [stdout]    | |_- in this macro invocation
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]    |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout]  92 | / create_iterable_struct!(
[INFO] [stdout]  93 | |     Note,
[INFO] [stdout]  94 | |     [   
[INFO] [stdout]  95 | |         (tick: i32: Integer): 127,
[INFO] [stdout] ...   |
[INFO] [stdout] 103 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout]  92 | / create_iterable_struct!(
[INFO] [stdout]  93 | |     Note,
[INFO] [stdout]  94 | |     [   
[INFO] [stdout]  95 | |         (tick: i32: Integer): 127,
[INFO] [stdout] ...   |
[INFO] [stdout] 103 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 105 | / create_iterable_struct!(
[INFO] [stdout] 106 | |     Layer,
[INFO] [stdout] 107 | |     [
[INFO] [stdout] 108 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 113 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 105 | / create_iterable_struct!(
[INFO] [stdout] 106 | |     Layer,
[INFO] [stdout] 107 | |     [
[INFO] [stdout] 108 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 113 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:34:35
[INFO] [stdout]     |
[INFO] [stdout]  34 |                   pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut, u8)> {
[INFO] [stdout]     |                                     ^^^^^^^^^                       ^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 115 | / create_iterable_struct!(
[INFO] [stdout] 116 | |     Instrument,
[INFO] [stdout] 117 | |     [
[INFO] [stdout] 118 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 123 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  34 |                 pub fn as_mut_vec(&mut self, version: u8) -> Vec<(BinaryMut<'_>, u8)> {
[INFO] [stdout]     |                                                                            ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]    --> src/fields.rs:41:35
[INFO] [stdout]     |
[INFO] [stdout]  41 |                   pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary, u8)> {
[INFO] [stdout]     |                                     ^^^^^                       ^^^^^^ the same lifetime is hidden here
[INFO] [stdout]     |                                     |
[INFO] [stdout]     |                                     the lifetime is elided here
[INFO] [stdout] ...
[INFO] [stdout] 115 | / create_iterable_struct!(
[INFO] [stdout] 116 | |     Instrument,
[INFO] [stdout] 117 | |     [
[INFO] [stdout] 118 | |         (name: String: String): 0,
[INFO] [stdout] ...   |
[INFO] [stdout] 123 | | );
[INFO] [stdout]     | |_- in this macro invocation
[INFO] [stdout]     |
[INFO] [stdout]     = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]     = note: this warning originates in the macro `create_iterable_struct` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]     |
[INFO] [stdout]  41 |                 pub fn as_ref_vec(&self, version: u8) -> Vec<(Binary<'_>, u8)> {
[INFO] [stdout]     |                                                                     ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `test` profile [unoptimized + debuginfo] target(s) in 0.78s
[INFO] running `Command { std: "docker" "inspect" "97357163c80ce0404986e07037060e50f2e7e121f0f74aed512b1bc7514a2d9d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "97357163c80ce0404986e07037060e50f2e7e121f0f74aed512b1bc7514a2d9d", kill_on_drop: false }`
[INFO] [stdout] 97357163c80ce0404986e07037060e50f2e7e121f0f74aed512b1bc7514a2d9d