[INFO] cloning repository https://github.com/faraplay/dearlystars_tool
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/faraplay/dearlystars_tool" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ffaraplay%2Fdearlystars_tool", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ffaraplay%2Fdearlystars_tool'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] a0165392e2e7063d7aaa9213a7e7004dc210270c
[INFO] checking faraplay/dearlystars_tool against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ffaraplay%2Fdearlystars_tool" "/workspace/builds/worker-7-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/faraplay/dearlystars_tool
[INFO] finished tweaking git repo https://github.com/faraplay/dearlystars_tool
[INFO] tweaked toml for git repo https://github.com/faraplay/dearlystars_tool written to /workspace/builds/worker-7-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/faraplay/dearlystars_tool on toolchain a5689b034ad287112f709693384a37f1377eb9cc
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/faraplay/dearlystars_tool 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" "+a5689b034ad287112f709693384a37f1377eb9cc" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/target:/opt/rustwide/target:rw,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" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:77db811e55d90add9212f6832c23229e3de5f1c1f9905e8cb07319ea15df73ac" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d
[INFO] running `Command { std: "docker" "start" "e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-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" "-w" "/opt/rustwide/workdir" "--user" "0:0" "e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]     Checking typenum v1.20.0
[INFO] [stderr]    Compiling proc-macro2 v1.0.106
[INFO] [stderr]    Compiling owo-colors v4.3.0
[INFO] [stderr]     Checking cmov v0.5.3
[INFO] [stderr]    Compiling binrw_derive v0.15.1
[INFO] [stderr]     Checking const-oid v0.10.2
[INFO] [stderr]    Compiling binrw v0.15.1
[INFO] [stderr]     Checking simd-adler32 v0.3.9
[INFO] [stderr]    Compiling either v1.15.0
[INFO] [stderr]     Checking cpufeatures v0.3.0
[INFO] [stderr]    Compiling crc32fast v1.5.0
[INFO] [stderr]     Checking array-init v2.1.0
[INFO] [stderr]     Checking cpubits v0.1.1
[INFO] [stderr]     Checking bytemuck v1.25.0
[INFO] [stderr]     Checking clap_builder v4.6.0
[INFO] [stderr]     Checking bitflags v2.11.1
[INFO] [stderr]     Checking nom v8.0.0
[INFO] [stderr]     Checking encoding_rs v0.8.35
[INFO] [stderr]     Checking miniz_oxide v0.8.9
[INFO] [stderr]     Checking fdeflate v0.3.7
[INFO] [stderr]     Checking ctutils v0.4.2
[INFO] [stderr]     Checking flate2 v1.1.9
[INFO] [stderr]    Compiling quote v1.0.45
[INFO] [stderr]    Compiling syn v2.0.117
[INFO] [stderr]     Checking png v0.18.1
[INFO] [stderr]     Checking hybrid-array v0.4.11
[INFO] [stderr]     Checking crypto-common v0.2.1
[INFO] [stderr]     Checking block-buffer v0.12.0
[INFO] [stderr]     Checking inout v0.2.2
[INFO] [stderr]     Checking digest v0.11.3
[INFO] [stderr]     Checking cipher v0.5.1
[INFO] [stderr]     Checking hmac v0.13.0
[INFO] [stderr]     Checking sha1 v0.11.0
[INFO] [stderr]     Checking aes v0.9.0
[INFO] [stderr]    Compiling clap_derive v4.6.1
[INFO] [stderr]     Checking clap v4.6.1
[INFO] [stderr]     Checking ndstool v0.3.1 (/opt/rustwide/workdir/ndstool)
[INFO] [stderr]     Checking dearlystars v0.5.2 (/opt/rustwide/workdir/dearlystars)
[INFO] [stdout] warning: a method with this name may be added to the standard library in the future
[INFO] [stdout]    --> dearlystars/src/gld.rs:331:40
[INFO] [stdout]     |
[INFO] [stdout] 331 |         let header: GldHeader = reader.read_le()?;
[INFO] [stdout]     |                                        ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: call with fully qualified syntax `binrw::BinReaderExt::read_le(...)` to keep using the current method
[INFO] [stdout]     = warning: once this associated item is added to the standard library, the ambiguity may cause an error or change in behavior!
[INFO] [stdout]     = note: for more information, see issue #48919 <https://github.com/rust-lang/rust/issues/48919>
[INFO] [stdout]     = note: `#[warn(unstable_name_collisions)]` (part of `#[warn(future_incompatible)]`) on by default
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: a method with this name may be added to the standard library in the future
[INFO] [stdout]    --> dearlystars/src/gld.rs:353:40
[INFO] [stdout]     |
[INFO] [stdout] 353 |                     footer.push(reader.read_le::<GldFooterEntry1>()?.into());
[INFO] [stdout]     |                                        ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: call with fully qualified syntax `binrw::BinReaderExt::read_le(...)` to keep using the current method
[INFO] [stdout]     = warning: once this associated item is added to the standard library, the ambiguity may cause an error or change in behavior!
[INFO] [stdout]     = note: for more information, see issue #48919 <https://github.com/rust-lang/rust/issues/48919>
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: a method with this name may be added to the standard library in the future
[INFO] [stdout]    --> dearlystars/src/gld.rs:358:40
[INFO] [stdout]     |
[INFO] [stdout] 358 |                     footer.push(reader.read_le::<GldFooterEntry2>()?.into());
[INFO] [stdout]     |                                        ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: call with fully qualified syntax `binrw::BinReaderExt::read_le(...)` to keep using the current method
[INFO] [stdout]     = warning: once this associated item is added to the standard library, the ambiguity may cause an error or change in behavior!
[INFO] [stdout]     = note: for more information, see issue #48919 <https://github.com/rust-lang/rust/issues/48919>
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: a method with this name may be added to the standard library in the future
[INFO] [stdout]    --> dearlystars/src/gld.rs:331:40
[INFO] [stdout]     |
[INFO] [stdout] 331 |         let header: GldHeader = reader.read_le()?;
[INFO] [stdout]     |                                        ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: call with fully qualified syntax `binrw::BinReaderExt::read_le(...)` to keep using the current method
[INFO] [stdout]     = warning: once this associated item is added to the standard library, the ambiguity may cause an error or change in behavior!
[INFO] [stdout]     = note: for more information, see issue #48919 <https://github.com/rust-lang/rust/issues/48919>
[INFO] [stdout]     = note: `#[warn(unstable_name_collisions)]` (part of `#[warn(future_incompatible)]`) on by default
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: a method with this name may be added to the standard library in the future
[INFO] [stdout]    --> dearlystars/src/gld.rs:353:40
[INFO] [stdout]     |
[INFO] [stdout] 353 |                     footer.push(reader.read_le::<GldFooterEntry1>()?.into());
[INFO] [stdout]     |                                        ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: call with fully qualified syntax `binrw::BinReaderExt::read_le(...)` to keep using the current method
[INFO] [stdout]     = warning: once this associated item is added to the standard library, the ambiguity may cause an error or change in behavior!
[INFO] [stdout]     = note: for more information, see issue #48919 <https://github.com/rust-lang/rust/issues/48919>
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: a method with this name may be added to the standard library in the future
[INFO] [stdout]    --> dearlystars/src/gld.rs:358:40
[INFO] [stdout]     |
[INFO] [stdout] 358 |                     footer.push(reader.read_le::<GldFooterEntry2>()?.into());
[INFO] [stdout]     |                                        ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = help: call with fully qualified syntax `binrw::BinReaderExt::read_le(...)` to keep using the current method
[INFO] [stdout]     = warning: once this associated item is added to the standard library, the ambiguity may cause an error or change in behavior!
[INFO] [stdout]     = note: for more information, see issue #48919 <https://github.com/rust-lang/rust/issues/48919>
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] help: add `#![feature(read_le)]` to the crate attributes to enable `std::io::Read::read_le`
[INFO] [stdout]    --> dearlystars/src/main.rs:1:1
[INFO] [stdout]     |
[INFO] [stdout]   1 + #![feature(read_le)]
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]   --> dearlystars/src/bbq/byte_convert.rs:28:24
[INFO] [stdout]    |
[INFO] [stdout] 28 |             return Err(bbq_error("Size of Type 2 data is not 28!").into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]    = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 28 -             return Err(bbq_error("Size of Type 2 data is not 28!").into());
[INFO] [stdout] 28 +             return Err(bbq_error("Size of Type 2 data is not 28!"));
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 28 -             return Err(bbq_error("Size of Type 2 data is not 28!").into());
[INFO] [stdout] 28 +             return Err(<util::Error as Into<_>>::into(bbq_error("Size of Type 2 data is not 28!")));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]   --> dearlystars/src/bbq/byte_convert.rs:70:24
[INFO] [stdout]    |
[INFO] [stdout] 70 |             return Err(bbq_error("Size of Type 3 data is wrong!").into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 70 -             return Err(bbq_error("Size of Type 3 data is wrong!").into());
[INFO] [stdout] 70 +             return Err(bbq_error("Size of Type 3 data is wrong!"));
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 70 -             return Err(bbq_error("Size of Type 3 data is wrong!").into());
[INFO] [stdout] 70 +             return Err(<util::Error as Into<_>>::into(bbq_error("Size of Type 3 data is wrong!")));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]   --> dearlystars/src/bbq/byte_convert.rs:91:24
[INFO] [stdout]    |
[INFO] [stdout] 91 |             return Err(bbq_error("Type 5 data size is zero!").into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 91 -             return Err(bbq_error("Type 5 data size is zero!").into());
[INFO] [stdout] 91 +             return Err(bbq_error("Type 5 data size is zero!"));
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 91 -             return Err(bbq_error("Type 5 data size is zero!").into());
[INFO] [stdout] 91 +             return Err(<util::Error as Into<_>>::into(bbq_error("Type 5 data size is zero!")));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]    --> dearlystars/src/bbq/byte_convert.rs:147:24
[INFO] [stdout]     |
[INFO] [stdout] 147 |             return Err(bbq_error("Type 6 data parsing read too many bytes!").into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 147 -             return Err(bbq_error("Type 6 data parsing read too many bytes!").into());
[INFO] [stdout] 147 +             return Err(bbq_error("Type 6 data parsing read too many bytes!"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 147 -             return Err(bbq_error("Type 6 data parsing read too many bytes!").into());
[INFO] [stdout] 147 +             return Err(<util::Error as Into<_>>::into(bbq_error("Type 6 data parsing read too many bytes!")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]    --> dearlystars/src/bbq/byte_convert.rs:163:24
[INFO] [stdout]     |
[INFO] [stdout] 163 |             return Err(bbq_error("Type 7 data string does not end in a null byte!").into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 163 -             return Err(bbq_error("Type 7 data string does not end in a null byte!").into());
[INFO] [stdout] 163 +             return Err(bbq_error("Type 7 data string does not end in a null byte!"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 163 -             return Err(bbq_error("Type 7 data string does not end in a null byte!").into());
[INFO] [stdout] 163 +             return Err(<util::Error as Into<_>>::into(bbq_error("Type 7 data string does not end in a null byte!")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]    --> dearlystars/src/bbq/byte_convert.rs:281:24
[INFO] [stdout]     |
[INFO] [stdout] 281 |             return Err(bbq_error("BBQ header is wrong!").into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 281 -             return Err(bbq_error("BBQ header is wrong!").into());
[INFO] [stdout] 281 +             return Err(bbq_error("BBQ header is wrong!"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 281 -             return Err(bbq_error("BBQ header is wrong!").into());
[INFO] [stdout] 281 +             return Err(<util::Error as Into<_>>::into(bbq_error("BBQ header is wrong!")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `dearlystars` (bin "dearlystars") due to 6 previous errors; 3 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]   --> dearlystars/src/bbq/byte_convert.rs:28:24
[INFO] [stdout]    |
[INFO] [stdout] 28 |             return Err(bbq_error("Size of Type 2 data is not 28!").into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]    = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 28 -             return Err(bbq_error("Size of Type 2 data is not 28!").into());
[INFO] [stdout] 28 +             return Err(bbq_error("Size of Type 2 data is not 28!"));
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 28 -             return Err(bbq_error("Size of Type 2 data is not 28!").into());
[INFO] [stdout] 28 +             return Err(<util::Error as Into<_>>::into(bbq_error("Size of Type 2 data is not 28!")));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]   --> dearlystars/src/bbq/byte_convert.rs:70:24
[INFO] [stdout]    |
[INFO] [stdout] 70 |             return Err(bbq_error("Size of Type 3 data is wrong!").into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 70 -             return Err(bbq_error("Size of Type 3 data is wrong!").into());
[INFO] [stdout] 70 +             return Err(bbq_error("Size of Type 3 data is wrong!"));
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 70 -             return Err(bbq_error("Size of Type 3 data is wrong!").into());
[INFO] [stdout] 70 +             return Err(<util::Error as Into<_>>::into(bbq_error("Size of Type 3 data is wrong!")));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]   --> dearlystars/src/bbq/byte_convert.rs:91:24
[INFO] [stdout]    |
[INFO] [stdout] 91 |             return Err(bbq_error("Type 5 data size is zero!").into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 91 -             return Err(bbq_error("Type 5 data size is zero!").into());
[INFO] [stdout] 91 +             return Err(bbq_error("Type 5 data size is zero!"));
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 91 -             return Err(bbq_error("Type 5 data size is zero!").into());
[INFO] [stdout] 91 +             return Err(<util::Error as Into<_>>::into(bbq_error("Type 5 data size is zero!")));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]    --> dearlystars/src/bbq/byte_convert.rs:147:24
[INFO] [stdout]     |
[INFO] [stdout] 147 |             return Err(bbq_error("Type 6 data parsing read too many bytes!").into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 147 -             return Err(bbq_error("Type 6 data parsing read too many bytes!").into());
[INFO] [stdout] 147 +             return Err(bbq_error("Type 6 data parsing read too many bytes!"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 147 -             return Err(bbq_error("Type 6 data parsing read too many bytes!").into());
[INFO] [stdout] 147 +             return Err(<util::Error as Into<_>>::into(bbq_error("Type 6 data parsing read too many bytes!")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]    --> dearlystars/src/bbq/byte_convert.rs:163:24
[INFO] [stdout]     |
[INFO] [stdout] 163 |             return Err(bbq_error("Type 7 data string does not end in a null byte!").into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 163 -             return Err(bbq_error("Type 7 data string does not end in a null byte!").into());
[INFO] [stdout] 163 +             return Err(bbq_error("Type 7 data string does not end in a null byte!"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 163 -             return Err(bbq_error("Type 7 data string does not end in a null byte!").into());
[INFO] [stdout] 163 +             return Err(<util::Error as Into<_>>::into(bbq_error("Type 7 data string does not end in a null byte!")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `util::Error`
[INFO] [stdout]    --> dearlystars/src/bbq/byte_convert.rs:281:24
[INFO] [stdout]     |
[INFO] [stdout] 281 |             return Err(bbq_error("BBQ header is wrong!").into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 281 -             return Err(bbq_error("BBQ header is wrong!").into());
[INFO] [stdout] 281 +             return Err(bbq_error("BBQ header is wrong!"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 281 -             return Err(bbq_error("BBQ header is wrong!").into());
[INFO] [stdout] 281 +             return Err(<util::Error as Into<_>>::into(bbq_error("BBQ header is wrong!")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `dearlystars` (bin "dearlystars" test) due to 6 previous errors; 3 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d", kill_on_drop: false }`
[INFO] [stdout] e619333b5e93f7df17b344681f20372f7b9ac58eac0fac6b38bbb5f155392f6d
