[INFO] cloning repository https://github.com/LioraLabs/ppu.toys [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/LioraLabs/ppu.toys" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FLioraLabs%2Fppu.toys", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FLioraLabs%2Fppu.toys'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 0b077674b7ca656b49420edb14df70bc89d19541 [INFO] checking LioraLabs/ppu.toys against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FLioraLabs%2Fppu.toys" "/workspace/builds/worker-6-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-6-tc2/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/LioraLabs/ppu.toys [INFO] finished tweaking git repo https://github.com/LioraLabs/ppu.toys [INFO] tweaked toml for git repo https://github.com/LioraLabs/ppu.toys written to /workspace/builds/worker-6-tc2/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/LioraLabs/ppu.toys 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/LioraLabs/ppu.toys 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-6-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-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] c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647 [INFO] running `Command { std: "docker" "start" "c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647", 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" "c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647", 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" "c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stderr] Compiling quote v1.0.46 [INFO] [stderr] Checking once_cell v1.21.4 [INFO] [stderr] Checking zerocopy v0.7.35 [INFO] [stderr] Compiling ahash v0.8.11 [INFO] [stderr] Checking allocator-api2 v0.2.21 [INFO] [stderr] Checking rand_chacha v0.3.1 [INFO] [stderr] Checking memchr v2.8.2 [INFO] [stderr] Checking sptr v0.3.2 [INFO] [stderr] Checking png v0.17.16 [INFO] [stderr] Compiling rustix v1.1.4 [INFO] [stderr] Checking rand v0.8.6 [INFO] [stderr] Compiling cfg_aliases v0.1.1 [INFO] [stderr] Compiling nix v0.28.0 [INFO] [stderr] Compiling syn v2.0.118 [INFO] [stderr] Checking smallvec v1.16.1 [INFO] [stderr] Checking hashbrown v0.14.5 [INFO] [stderr] Checking clap_lex v1.1.1 [INFO] [stderr] Checking serde_json v1.0.150 [INFO] [stderr] Checking nibble_vec v0.1.0 [INFO] [stderr] Checking endian-type v0.1.2 [INFO] [stderr] Checking clap_builder v4.6.7 [INFO] [stderr] Checking radix_trie v0.2.1 [INFO] [stderr] Checking unicode-segmentation v1.13.3 [INFO] [stderr] Checking log v0.4.34 [INFO] [stderr] Compiling getrandom v0.4.3 [INFO] [stderr] Checking fd-lock v4.0.4 [INFO] [stderr] Checking tempfile v3.27.0 [INFO] [stderr] Checking clap v4.6.7 [INFO] [stderr] Checking rustyline v14.0.0 [INFO] [stderr] Compiling synstructure v0.13.2 [INFO] [stderr] Compiling thiserror-impl v1.0.69 [INFO] [stderr] Compiling serde_derive v1.0.228 [INFO] [stderr] Compiling gc-arena-derive v0.5.3 [INFO] [stderr] Checking gc-arena v0.5.3 [INFO] [stderr] Checking thiserror v1.0.69 [INFO] [stderr] Checking piccolo-ppu v0.3.4 (/opt/rustwide/workdir/vendor/piccolo) [INFO] [stdout] warning: use of deprecated module `std::i32`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/compiler/lexer.rs:2:16 [INFO] [stdout] | [INFO] [stdout] 2 | char, fmt, i32, i64, [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(deprecated)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i64`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/compiler/lexer.rs:2:21 [INFO] [stdout] | [INFO] [stdout] 2 | char, fmt, i32, i64, [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::f64::INFINITY`: replaced by the `INFINITY` associated constant on `f64` [INFO] [stdout] --> vendor/piccolo/src/stdlib/math.rs:120:46 [INFO] [stdout] | [INFO] [stdout] 120 | math.set(ctx, "huge", Value::Number(f64::INFINITY)).unwrap(); [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::f64::INFINITY`: replaced by the `INFINITY` associated constant on `f64` [INFO] [stdout] --> vendor/piccolo/src/stdlib/math.rs:140:51 [INFO] [stdout] | [INFO] [stdout] 140 | .try_fold(Value::Number(-f64::INFINITY), |max, entry| { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::f64::INFINITY`: replaced by the `INFINITY` associated constant on `f64` [INFO] [stdout] --> vendor/piccolo/src/stdlib/math.rs:163:50 [INFO] [stdout] | [INFO] [stdout] 163 | .try_fold(Value::Number(f64::INFINITY), |max, entry| { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i64`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/table/raw.rs:1:28 [INFO] [stdout] | [INFO] [stdout] 1 | use std::{fmt, hash::Hash, i64, mem}; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::i64::MAX`: replaced by the `MAX` associated constant on this type [INFO] [stdout] --> vendor/piccolo/src/table/raw.rs:272:32 [INFO] [stdout] | [INFO] [stdout] 272 | if max == i64::MAX { [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::i64::MAX`: replaced by the `MAX` associated constant on this type [INFO] [stdout] --> vendor/piccolo/src/table/raw.rs:277:33 [INFO] [stdout] | [INFO] [stdout] 277 | return i64::MAX; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::i64::MAX`: replaced by the `MAX` associated constant on this type [INFO] [stdout] --> vendor/piccolo/src/table/raw.rs:281:32 [INFO] [stdout] | [INFO] [stdout] 281 | max = i64::MAX; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i64`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/table/table.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | i64, mem, [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i64`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/value.rs:1:21 [INFO] [stdout] | [INFO] [stdout] 1 | use std::{f64, fmt, i64, io, string::String as StdString}; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i32`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/compiler/lexer.rs:2:16 [INFO] [stdout] | [INFO] [stdout] 2 | char, fmt, i32, i64, [INFO] [stdout] | ^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(deprecated)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i64`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/compiler/lexer.rs:2:21 [INFO] [stdout] | [INFO] [stdout] 2 | char, fmt, i32, i64, [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::f64::INFINITY`: replaced by the `INFINITY` associated constant on `f64` [INFO] [stdout] --> vendor/piccolo/src/stdlib/math.rs:120:46 [INFO] [stdout] | [INFO] [stdout] 120 | math.set(ctx, "huge", Value::Number(f64::INFINITY)).unwrap(); [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::f64::INFINITY`: replaced by the `INFINITY` associated constant on `f64` [INFO] [stdout] --> vendor/piccolo/src/stdlib/math.rs:140:51 [INFO] [stdout] | [INFO] [stdout] 140 | .try_fold(Value::Number(-f64::INFINITY), |max, entry| { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::f64::INFINITY`: replaced by the `INFINITY` associated constant on `f64` [INFO] [stdout] --> vendor/piccolo/src/stdlib/math.rs:163:50 [INFO] [stdout] | [INFO] [stdout] 163 | .try_fold(Value::Number(f64::INFINITY), |max, entry| { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i64`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/table/raw.rs:1:28 [INFO] [stdout] | [INFO] [stdout] 1 | use std::{fmt, hash::Hash, i64, mem}; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::i64::MAX`: replaced by the `MAX` associated constant on this type [INFO] [stdout] --> vendor/piccolo/src/table/raw.rs:272:32 [INFO] [stdout] | [INFO] [stdout] 272 | if max == i64::MAX { [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::i64::MAX`: replaced by the `MAX` associated constant on this type [INFO] [stdout] --> vendor/piccolo/src/table/raw.rs:277:33 [INFO] [stdout] | [INFO] [stdout] 277 | return i64::MAX; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::i64::MAX`: replaced by the `MAX` associated constant on this type [INFO] [stdout] --> vendor/piccolo/src/table/raw.rs:281:32 [INFO] [stdout] | [INFO] [stdout] 281 | max = i64::MAX; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i64`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/table/table.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | i64, mem, [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated module `std::i64`: all constants in this module replaced by associated constants on the type [INFO] [stdout] --> vendor/piccolo/src/value.rs:1:21 [INFO] [stdout] | [INFO] [stdout] 1 | use std::{f64, fmt, i64, io, string::String as StdString}; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Checking serde v1.0.228 [INFO] [stdout] error: useless conversion to the same type: `CallbackReturn<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:123:28 [INFO] [stdout] | [INFO] [stdout] 123 | Ok(CallbackReturn::Return.into()) [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 123 - Ok(CallbackReturn::Return.into()) [INFO] [stdout] 123 + Ok(CallbackReturn::Return) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 123 - Ok(CallbackReturn::Return.into()) [INFO] [stdout] 123 + Ok( as Into<_>>::into(CallbackReturn::Return)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `CallbackReturn<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:127:28 [INFO] [stdout] | [INFO] [stdout] 127 | Ok(CallbackReturn::Call { [INFO] [stdout] | ____________________________^ [INFO] [stdout] 128 | | function: call.function, [INFO] [stdout] 129 | | then: None, [INFO] [stdout] 130 | | } [INFO] [stdout] 131 | | .into()) [INFO] [stdout] | |_______________________________^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 130 - } [INFO] [stdout] 131 - .into()) [INFO] [stdout] 130 + }) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 127 ~ Ok( as Into<_>>::into(CallbackReturn::Call { [INFO] [stdout] 128 | function: call.function, [INFO] [stdout] 129 | then: None, [INFO] [stdout] 130 ~ })) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `value::Value<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:136:20 [INFO] [stdout] | [INFO] [stdout] 136 | args: [table.into(), key], [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 136 - args: [table.into(), key], [INFO] [stdout] 136 + args: [table, key], [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 136 - args: [table.into(), key], [INFO] [stdout] 136 + args: [ as Into<_>>::into(table), key], [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `CallbackReturn<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:207:24 [INFO] [stdout] | [INFO] [stdout] 207 | Ok(CallbackReturn::Call { [INFO] [stdout] | ________________________^ [INFO] [stdout] 208 | | function: call.function, [INFO] [stdout] 209 | | then: None, [INFO] [stdout] 210 | | } [INFO] [stdout] 211 | | .into()) [INFO] [stdout] | |___________________________^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 210 - } [INFO] [stdout] 211 - .into()) [INFO] [stdout] 210 + }) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 207 ~ Ok( as Into<_>>::into(CallbackReturn::Call { [INFO] [stdout] 208 | function: call.function, [INFO] [stdout] 209 | then: None, [INFO] [stdout] 210 ~ })) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `value::Value<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:217:20 [INFO] [stdout] | [INFO] [stdout] 217 | args: [table.into(), key, value], [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 217 - args: [table.into(), key, value], [INFO] [stdout] 217 + args: [table, key, value], [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 217 - args: [table.into(), key, value], [INFO] [stdout] 217 + args: [ as Into<_>>::into(table), key, value], [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `CallbackReturn<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:242:20 [INFO] [stdout] | [INFO] [stdout] 242 | Ok(CallbackReturn::Call { [INFO] [stdout] | ____________________^ [INFO] [stdout] 243 | | function: call(ctx, f)?, [INFO] [stdout] 244 | | then: None, [INFO] [stdout] 245 | | } [INFO] [stdout] 246 | | .into()) [INFO] [stdout] | |_______________________^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 245 - } [INFO] [stdout] 246 - .into()) [INFO] [stdout] 245 + }) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 242 ~ Ok( as Into<_>>::into(CallbackReturn::Call { [INFO] [stdout] 243 | function: call(ctx, f)?, [INFO] [stdout] 244 | then: None, [INFO] [stdout] 245 ~ })) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `bool` [INFO] [stdout] --> vendor/piccolo/src/raw_ops.rs:62:10 [INFO] [stdout] | [INFO] [stdout] 62 | Some(lhs.to_constant()?.less_than(&rhs.to_constant()?)?.into()) [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 62 - Some(lhs.to_constant()?.less_than(&rhs.to_constant()?)?.into()) [INFO] [stdout] 62 + Some(lhs.to_constant()?.less_than(&rhs.to_constant()?)?) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 62 - Some(lhs.to_constant()?.less_than(&rhs.to_constant()?)?.into()) [INFO] [stdout] 62 + Some(>::into(lhs.to_constant()?.less_than(&rhs.to_constant()?)?)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `bool` [INFO] [stdout] --> vendor/piccolo/src/raw_ops.rs:66:10 [INFO] [stdout] | [INFO] [stdout] 66 | Some(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?.into()) [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 66 - Some(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?.into()) [INFO] [stdout] 66 + Some(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 66 - Some(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?.into()) [INFO] [stdout] 66 + Some(>::into(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `error::Error<'_>` [INFO] [stdout] --> vendor/piccolo/src/thread/executor.rs:187:72 [INFO] [stdout] | [INFO] [stdout] 187 | ... top_state.frames.push(Frame::Error(err.into())); [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 187 - top_state.frames.push(Frame::Error(err.into())); [INFO] [stdout] 187 + top_state.frames.push(Frame::Error(err)); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 187 - top_state.frames.push(Frame::Error(err.into())); [INFO] [stdout] 187 + top_state.frames.push(Frame::Error( as Into<_>>::into(err))); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `piccolo-ppu` (lib test) due to 9 previous errors; 11 warnings emitted [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stdout] error: useless conversion to the same type: `CallbackReturn<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:123:28 [INFO] [stdout] | [INFO] [stdout] 123 | Ok(CallbackReturn::Return.into()) [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 123 - Ok(CallbackReturn::Return.into()) [INFO] [stdout] 123 + Ok(CallbackReturn::Return) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 123 - Ok(CallbackReturn::Return.into()) [INFO] [stdout] 123 + Ok( as Into<_>>::into(CallbackReturn::Return)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `CallbackReturn<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:127:28 [INFO] [stdout] | [INFO] [stdout] 127 | Ok(CallbackReturn::Call { [INFO] [stdout] | ____________________________^ [INFO] [stdout] 128 | | function: call.function, [INFO] [stdout] 129 | | then: None, [INFO] [stdout] 130 | | } [INFO] [stdout] 131 | | .into()) [INFO] [stdout] | |_______________________________^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 130 - } [INFO] [stdout] 131 - .into()) [INFO] [stdout] 130 + }) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 127 ~ Ok( as Into<_>>::into(CallbackReturn::Call { [INFO] [stdout] 128 | function: call.function, [INFO] [stdout] 129 | then: None, [INFO] [stdout] 130 ~ })) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `value::Value<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:136:20 [INFO] [stdout] | [INFO] [stdout] 136 | args: [table.into(), key], [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 136 - args: [table.into(), key], [INFO] [stdout] 136 + args: [table, key], [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 136 - args: [table.into(), key], [INFO] [stdout] 136 + args: [ as Into<_>>::into(table), key], [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `CallbackReturn<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:207:24 [INFO] [stdout] | [INFO] [stdout] 207 | Ok(CallbackReturn::Call { [INFO] [stdout] | ________________________^ [INFO] [stdout] 208 | | function: call.function, [INFO] [stdout] 209 | | then: None, [INFO] [stdout] 210 | | } [INFO] [stdout] 211 | | .into()) [INFO] [stdout] | |___________________________^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 210 - } [INFO] [stdout] 211 - .into()) [INFO] [stdout] 210 + }) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 207 ~ Ok( as Into<_>>::into(CallbackReturn::Call { [INFO] [stdout] 208 | function: call.function, [INFO] [stdout] 209 | then: None, [INFO] [stdout] 210 ~ })) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `value::Value<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:217:20 [INFO] [stdout] | [INFO] [stdout] 217 | args: [table.into(), key, value], [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 217 - args: [table.into(), key, value], [INFO] [stdout] 217 + args: [table, key, value], [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 217 - args: [table.into(), key, value], [INFO] [stdout] 217 + args: [ as Into<_>>::into(table), key, value], [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `CallbackReturn<'_>` [INFO] [stdout] --> vendor/piccolo/src/meta_ops.rs:242:20 [INFO] [stdout] | [INFO] [stdout] 242 | Ok(CallbackReturn::Call { [INFO] [stdout] | ____________________^ [INFO] [stdout] 243 | | function: call(ctx, f)?, [INFO] [stdout] 244 | | then: None, [INFO] [stdout] 245 | | } [INFO] [stdout] 246 | | .into()) [INFO] [stdout] | |_______________________^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 245 - } [INFO] [stdout] 246 - .into()) [INFO] [stdout] 245 + }) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 242 ~ Ok( as Into<_>>::into(CallbackReturn::Call { [INFO] [stdout] 243 | function: call(ctx, f)?, [INFO] [stdout] 244 | then: None, [INFO] [stdout] 245 ~ })) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `bool` [INFO] [stdout] --> vendor/piccolo/src/raw_ops.rs:62:10 [INFO] [stdout] | [INFO] [stdout] 62 | Some(lhs.to_constant()?.less_than(&rhs.to_constant()?)?.into()) [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 62 - Some(lhs.to_constant()?.less_than(&rhs.to_constant()?)?.into()) [INFO] [stdout] 62 + Some(lhs.to_constant()?.less_than(&rhs.to_constant()?)?) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 62 - Some(lhs.to_constant()?.less_than(&rhs.to_constant()?)?.into()) [INFO] [stdout] 62 + Some(>::into(lhs.to_constant()?.less_than(&rhs.to_constant()?)?)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `bool` [INFO] [stdout] --> vendor/piccolo/src/raw_ops.rs:66:10 [INFO] [stdout] | [INFO] [stdout] 66 | Some(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?.into()) [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 66 - Some(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?.into()) [INFO] [stdout] 66 + Some(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?) [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 66 - Some(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?.into()) [INFO] [stdout] 66 + Some(>::into(lhs.to_constant()?.less_equal(&rhs.to_constant()?)?)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `error::Error<'_>` [INFO] [stdout] --> vendor/piccolo/src/thread/executor.rs:187:72 [INFO] [stdout] | [INFO] [stdout] 187 | ... top_state.frames.push(Frame::Error(err.into())); [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this method call relies on the `impl Into 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] 187 - top_state.frames.push(Frame::Error(err.into())); [INFO] [stdout] 187 + top_state.frames.push(Frame::Error(err)); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 187 - top_state.frames.push(Frame::Error(err.into())); [INFO] [stdout] 187 + top_state.frames.push(Frame::Error( as Into<_>>::into(err))); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `piccolo-ppu` (lib) due to 9 previous errors; 11 warnings emitted [INFO] running `Command { std: "docker" "inspect" "c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647", kill_on_drop: false }` [INFO] [stdout] c97448acf2b5c31571fcbbe38de88a956f9506193a07b2f8142945a362d96647