[INFO] fetching crate thrussh 0.49.0...
[INFO] checking thrussh-0.49.0 against c60b92c03ca8a3b101eb9d57f51e2232690638c7 for pr-164148-crater-rollup
[INFO] extracting crate thrussh 0.49.0 into /workspace/builds/worker-5-tc2/source
[INFO] started tweaking crates.io crate thrussh 0.49.0
[INFO] finished tweaking crates.io crate thrussh 0.49.0
[INFO] tweaked toml for crates.io crate thrussh 0.49.0 written to /workspace/builds/worker-5-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate thrussh 0.49.0 on toolchain c60b92c03ca8a3b101eb9d57f51e2232690638c7
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate crates.io crate thrussh 0.49.0 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" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-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] 120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48
[INFO] running `Command { std: "docker" "start" "120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48", 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" "120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48" "/opt/rustwide/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48", 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" "120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48" "/opt/rustwide/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]     Checking subtle v2.4.1
[INFO] [stderr]    Compiling find-msvc-tools v0.1.1
[INFO] [stderr]    Compiling num-traits v0.2.19
[INFO] [stderr]    Compiling winapi v0.3.9
[INFO] [stderr]     Checking hybrid-array v0.4.14
[INFO] [stderr]     Checking futures-channel v0.3.31
[INFO] [stderr]    Compiling serde_core v1.0.225
[INFO] [stderr]     Checking crypto-common v0.1.6
[INFO] [stderr]     Checking cipher v0.3.0
[INFO] [stderr]     Checking inout v0.1.4
[INFO] [stderr]     Checking getrandom v0.3.3
[INFO] [stderr]     Checking digest v0.9.0
[INFO] [stderr]    Compiling syn v2.0.106
[INFO] [stderr]     Checking signal-hook-registry v1.4.6
[INFO] [stderr]     Checking block-buffer v0.9.0
[INFO] [stderr]     Checking regex-syntax v0.8.6
[INFO] [stderr]     Checking base64ct v1.8.0
[INFO] [stderr]     Checking rand_core v0.6.4
[INFO] [stderr]     Checking rand_core v0.9.3
[INFO] [stderr]    Compiling serde v1.0.225
[INFO] [stderr]     Checking cipher v0.4.4
[INFO] [stderr]     Checking sha2 v0.9.9
[INFO] [stderr]     Checking ctr v0.8.0
[INFO] [stderr]     Checking digest v0.10.7
[INFO] [stderr]     Checking crypto-mac v0.11.1
[INFO] [stderr]     Checking hmac v0.11.0
[INFO] [stderr]     Checking dirs-sys v0.3.7
[INFO] [stderr]     Checking keccak v0.2.1
[INFO] [stderr]     Checking bit-vec v0.6.3
[INFO] [stderr]     Checking dirs v3.0.2
[INFO] [stderr]    Compiling cc v1.2.37
[INFO] [stderr]     Checking blowfish v0.9.1
[INFO] [stderr]     Checking rand_chacha v0.9.0
[INFO] [stderr]     Checking password-hash v0.2.3
[INFO] [stderr]     Checking block-modes v0.8.1
[INFO] [stderr]     Checking aes v0.7.5
[INFO] [stderr]     Checking anstyle-query v1.1.4
[INFO] [stderr]     Checking md5 v0.7.0
[INFO] [stderr]     Checking anstream v0.6.20
[INFO] [stderr]     Checking cryptovec v0.7.1
[INFO] [stderr]     Checking rand v0.9.2
[INFO] [stderr]     Checking flate2 v1.1.2
[INFO] [stderr]     Checking jiff v0.2.15
[INFO] [stderr]     Checking sha2 v0.10.9
[INFO] [stderr]     Checking pbkdf2 v0.12.2
[INFO] [stderr]     Checking pbkdf2 v0.8.0
[INFO] [stderr]     Checking bcrypt-pbkdf v0.10.0
[INFO] [stderr]     Checking crypto-common v0.2.2
[INFO] [stderr]     Checking block-buffer v0.12.1
[INFO] [stderr]    Compiling libsodium-sys v0.2.7
[INFO] [stderr]     Checking num-integer v0.1.46
[INFO] [stderr]     Checking module-lattice v0.2.3
[INFO] [stderr]     Checking regex-automata v0.4.10
[INFO] [stderr]     Checking ssh-libsodium v0.4.0
[INFO] [stderr]     Checking num-bigint v0.4.6
[INFO] [stderr]     Checking digest v0.11.3
[INFO] [stderr]     Checking kem v0.3.0
[INFO] [stderr]     Checking sha3 v0.11.0
[INFO] [stderr]     Checking ml-kem v0.3.2
[INFO] [stderr]     Checking yasna v0.4.0
[INFO] [stderr]    Compiling futures-macro v0.3.31
[INFO] [stderr]    Compiling tokio-macros v2.5.0
[INFO] [stderr]    Compiling thiserror-impl v1.0.69
[INFO] [stderr]    Compiling serde_derive v1.0.225
[INFO] [stderr]     Checking tokio v1.47.1
[INFO] [stderr]     Checking thiserror v1.0.69
[INFO] [stderr]     Checking futures-util v0.3.31
[INFO] [stderr]     Checking regex v1.11.2
[INFO] [stderr]     Checking env_filter v0.1.3
[INFO] [stderr]     Checking env_logger v0.11.8
[INFO] [stderr]     Checking futures-executor v0.3.31
[INFO] [stderr]     Checking futures v0.3.31
[INFO] [stderr]     Checking tokio-stream v0.1.17
[INFO] [stderr]     Checking thrussh-keys v0.44.0
[INFO] [stderr]     Checking thrussh v0.49.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `super::super::*`
[INFO] [stdout]   --> src/server/encrypted.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use super::super::*;
[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: `super::super::*`
[INFO] [stdout]   --> src/server/encrypted.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use super::super::*;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u32::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:215:44
[INFO] [stdout]     |
[INFO] [stdout] 215 |         assert!(packet_length <= std::u32::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u8::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:218:44
[INFO] [stdout]     |
[INFO] [stdout] 218 |         assert!(padding_length <= std::u8::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u32::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:215:44
[INFO] [stdout]     |
[INFO] [stdout] 215 |         assert!(packet_length <= std::u32::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u8::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:218:44
[INFO] [stdout]     |
[INFO] [stdout] 218 |         assert!(padding_length <= std::u8::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:194:24
[INFO] [stdout]     |
[INFO] [stdout] 194 |             return Err(Error::NoCommonKexAlgo.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] 194 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 194 +             return Err(Error::NoCommonKexAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 194 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 194 +             return Err(<Error as Into<_>>::into(Error::NoCommonKexAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:231:24
[INFO] [stdout]     |
[INFO] [stdout] 231 |             return Err(Error::NoCommonKeyAlgo.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] 231 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 231 +             return Err(Error::NoCommonKeyAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 231 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 231 +             return Err(<Error as Into<_>>::into(Error::NoCommonKeyAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:242:24
[INFO] [stdout]     |
[INFO] [stdout] 242 |             return Err(Error::NoCommonCipher.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] 242 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 242 +             return Err(Error::NoCommonCipher);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 242 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 242 +             return Err(<Error as Into<_>>::into(Error::NoCommonCipher));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:254:24
[INFO] [stdout]     |
[INFO] [stdout] 254 |             return Err(Error::NoCommonMac.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] 254 -             return Err(Error::NoCommonMac.into());
[INFO] [stdout] 254 +             return Err(Error::NoCommonMac);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 254 -             return Err(Error::NoCommonMac.into());
[INFO] [stdout] 254 +             return Err(<Error as Into<_>>::into(Error::NoCommonMac));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:265:28
[INFO] [stdout]     |
[INFO] [stdout] 265 |                 return Err(Error::NoCommonCompression.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] 265 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 265 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 265 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 265 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:273:28
[INFO] [stdout]     |
[INFO] [stdout] 273 |                 return Err(Error::NoCommonCompression.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] 273 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 273 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 273 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 273 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:296:22
[INFO] [stdout]     |
[INFO] [stdout] 296 |             _ => Err(Error::KexInit.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] 296 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 296 +             _ => Err(Error::KexInit),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 296 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 296 +             _ => Err(<Error as Into<_>>::into(Error::KexInit)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/ssh_read.rs:125:28
[INFO] [stdout]     |
[INFO] [stdout] 125 |                 return Err(Error::Disconnect.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] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/server/kex.rs:43:28
[INFO] [stdout]    |
[INFO] [stdout] 43 |                 return Err(Error::UnknownKey.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] 43 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 43 +                 return Err(Error::UnknownKey);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 43 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 43 +                 return Err(<Error as Into<_>>::into(Error::UnknownKey));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `<S as Signer>::Error` thrussh
[INFO] [stdout]    --> src/client/mod.rs:292:55
[INFO] [stdout]     |
[INFO] [stdout] 292 |                         Err(e) => return (future, Err(e.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] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(e)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(<<S as Signer>::Error as Into<_>>::into(e))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/client/mod.rs:326:32
[INFO] [stdout]     |
[INFO] [stdout] 326 |                     return Err(Error::Disconnect.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] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/client/session.rs:29:33
[INFO] [stdout]    |
[INFO] [stdout] 29 |                 _ => return Err(Error::NotAuthenticated.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] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/client/session.rs:32:24
[INFO] [stdout]    |
[INFO] [stdout] 32 |             return Err(Error::Inconsistent.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] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/client/session.rs:69:33
[INFO] [stdout]    |
[INFO] [stdout] 69 |                 _ => return Err(Error::NotAuthenticated.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] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/client/session.rs:72:24
[INFO] [stdout]    |
[INFO] [stdout] 72 |             return Err(Error::Inconsistent.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] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/client/session.rs:113:33
[INFO] [stdout]     |
[INFO] [stdout] 113 |                 _ => return Err(Error::NotAuthenticated.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] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/client/session.rs:116:24
[INFO] [stdout]     |
[INFO] [stdout] 116 |             return Err(Error::Inconsistent.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] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(Error::Inconsistent);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:194:24
[INFO] [stdout]     |
[INFO] [stdout] 194 |             return Err(Error::NoCommonKexAlgo.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] 194 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 194 +             return Err(Error::NoCommonKexAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 194 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 194 +             return Err(<Error as Into<_>>::into(Error::NoCommonKexAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:231:24
[INFO] [stdout]     |
[INFO] [stdout] 231 |             return Err(Error::NoCommonKeyAlgo.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] 231 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 231 +             return Err(Error::NoCommonKeyAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 231 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 231 +             return Err(<Error as Into<_>>::into(Error::NoCommonKeyAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:242:24
[INFO] [stdout]     |
[INFO] [stdout] 242 |             return Err(Error::NoCommonCipher.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] 242 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 242 +             return Err(Error::NoCommonCipher);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 242 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 242 +             return Err(<Error as Into<_>>::into(Error::NoCommonCipher));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:254:24
[INFO] [stdout]     |
[INFO] [stdout] 254 |             return Err(Error::NoCommonMac.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] 254 -             return Err(Error::NoCommonMac.into());
[INFO] [stdout] 254 +             return Err(Error::NoCommonMac);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 254 -             return Err(Error::NoCommonMac.into());
[INFO] [stdout] 254 +             return Err(<Error as Into<_>>::into(Error::NoCommonMac));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:265:28
[INFO] [stdout]     |
[INFO] [stdout] 265 |                 return Err(Error::NoCommonCompression.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] 265 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 265 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 265 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 265 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:273:28
[INFO] [stdout]     |
[INFO] [stdout] 273 |                 return Err(Error::NoCommonCompression.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] 273 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 273 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 273 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 273 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/negotiation.rs:296:22
[INFO] [stdout]     |
[INFO] [stdout] 296 |             _ => Err(Error::KexInit.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] 296 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 296 +             _ => Err(Error::KexInit),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 296 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 296 +             _ => Err(<Error as Into<_>>::into(Error::KexInit)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/ssh_read.rs:125:28
[INFO] [stdout]     |
[INFO] [stdout] 125 |                 return Err(Error::Disconnect.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] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/server/kex.rs:43:28
[INFO] [stdout]    |
[INFO] [stdout] 43 |                 return Err(Error::UnknownKey.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] 43 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 43 +                 return Err(Error::UnknownKey);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 43 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 43 +                 return Err(<Error as Into<_>>::into(Error::UnknownKey));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `<S as Signer>::Error` thrussh
[INFO] [stdout]    --> src/client/mod.rs:292:55
[INFO] [stdout]     |
[INFO] [stdout] 292 |                         Err(e) => return (future, Err(e.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] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(e)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(<<S as Signer>::Error as Into<_>>::into(e))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/client/mod.rs:326:32
[INFO] [stdout]     |
[INFO] [stdout] 326 |                     return Err(Error::Disconnect.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] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/client/session.rs:29:33
[INFO] [stdout]    |
[INFO] [stdout] 29 |                 _ => return Err(Error::NotAuthenticated.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] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/client/session.rs:32:24
[INFO] [stdout]    |
[INFO] [stdout] 32 |             return Err(Error::Inconsistent.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] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/client/session.rs:69:33
[INFO] [stdout]    |
[INFO] [stdout] 69 |                 _ => return Err(Error::NotAuthenticated.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] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]   --> src/client/session.rs:72:24
[INFO] [stdout]    |
[INFO] [stdout] 72 |             return Err(Error::Inconsistent.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] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/client/session.rs:113:33
[INFO] [stdout]     |
[INFO] [stdout] 113 |                 _ => return Err(Error::NotAuthenticated.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] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error` thrussh
[INFO] [stdout]    --> src/client/session.rs:116:24
[INFO] [stdout]     |
[INFO] [stdout] 116 |             return Err(Error::Inconsistent.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] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(Error::Inconsistent);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `thrussh` (lib test) due to 17 previous errors; 3 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stderr] error: could not compile `thrussh` (lib) due to 17 previous errors; 3 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48", kill_on_drop: false }`
[INFO] [stdout] 120f10cabf04dffbb7933eb6d0d51c2a6d45d81306601457a5952485a8a87b48
