[INFO] cloning repository https://github.com/GachiLord/pipa [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/GachiLord/pipa" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FGachiLord%2Fpipa", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FGachiLord%2Fpipa'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 348aefddd8d06834dc76036f7617fa6efd8835df [INFO] checking GachiLord/pipa against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FGachiLord%2Fpipa" "/workspace/builds/worker-1-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-1-tc2/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/GachiLord/pipa [INFO] finished tweaking git repo https://github.com/GachiLord/pipa [INFO] tweaked toml for git repo https://github.com/GachiLord/pipa written to /workspace/builds/worker-1-tc2/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/GachiLord/pipa 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/GachiLord/pipa 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-1-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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] e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed [INFO] running `Command { std: "docker" "start" "e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed", 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" "e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed", 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" "e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stderr] Checking unicode-segmentation v1.12.0 [INFO] [stderr] Checking pipa v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `std::collections::HashSet` [INFO] [stdout] --> src/analysis.rs:118:9 [INFO] [stdout] | [INFO] [stdout] 118 | use std::collections::HashSet; [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 imports: `InnerNode`, `Node`, `evaluate_expr`, and `unique_constants_expr` [INFO] [stdout] --> src/analysis.rs:119:27 [INFO] [stdout] | [INFO] [stdout] 119 | use crate::analysis::{Node, unique_constants_expr, InnerNode, evaluate_expr}; [INFO] [stdout] | ^^^^ ^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^ ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::syntax::ast` [INFO] [stdout] --> src/analysis.rs:120:9 [INFO] [stdout] | [INFO] [stdout] 120 | use crate::syntax::ast; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:135:25 [INFO] [stdout] | [INFO] [stdout] 135 | let value = node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd]).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] 135 - let value = node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd]).into(); [INFO] [stdout] 135 + let value = node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd]); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 135 - let value = node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd]).into(); [INFO] [stdout] 135 + let value = >::into(node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd])); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:146:42 [INFO] [stdout] | [INFO] [stdout] 146 | ops.push(Op::PutStr { value: value.to_string().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] 146 - ops.push(Op::PutStr { value: value.to_string().into() }); [INFO] [stdout] 146 + ops.push(Op::PutStr { value: value.to_string() }); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 146 - ops.push(Op::PutStr { value: value.to_string().into() }); [INFO] [stdout] 146 + ops.push(Op::PutStr { value: >::into(value.to_string()) }); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:265:79 [INFO] [stdout] | [INFO] [stdout] 265 | ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: name.clone().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] 265 - ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: name.clone().into() }); [INFO] [stdout] 265 + ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: name.clone() }); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 265 - ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: name.clone().into() }); [INFO] [stdout] 265 + ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: >::into(name.clone()) }); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:287:60 [INFO] [stdout] | [INFO] [stdout] 287 | ops.push(Op::LoadArrayItem { name: name.clone().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] 287 - ops.push(Op::LoadArrayItem { name: name.clone().into() }); [INFO] [stdout] 287 + ops.push(Op::LoadArrayItem { name: name.clone() }); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 287 - ops.push(Op::LoadArrayItem { name: name.clone().into() }); [INFO] [stdout] 287 + ops.push(Op::LoadArrayItem { name: >::into(name.clone()) }); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:300:56 [INFO] [stdout] | [INFO] [stdout] 300 | ops.push(Op::LoadArrayItem { name: name.clone().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] 300 - ops.push(Op::LoadArrayItem { name: name.clone().into() }); [INFO] [stdout] 300 + ops.push(Op::LoadArrayItem { name: name.clone() }); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 300 - ops.push(Op::LoadArrayItem { name: name.clone().into() }); [INFO] [stdout] 300 + ops.push(Op::LoadArrayItem { name: >::into(name.clone()) }); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/vm.rs:105:33 [INFO] [stdout] | [INFO] [stdout] 105 | self.stack.push(output.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] 105 - self.stack.push(output.into()); [INFO] [stdout] 105 + self.stack.push(output); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 105 - self.stack.push(output.into()); [INFO] [stdout] 105 + self.stack.push(>::into(output)); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/vm.rs:121:33 [INFO] [stdout] | [INFO] [stdout] 121 | self.stack.push(output.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] 121 - self.stack.push(output.into()); [INFO] [stdout] 121 + self.stack.push(output); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 121 - self.stack.push(output.into()); [INFO] [stdout] 121 + self.stack.push(>::into(output)); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/vm.rs:130:33 [INFO] [stdout] | [INFO] [stdout] 130 | self.stack.push(self.counter.to_string().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 - self.stack.push(self.counter.to_string().into()); [INFO] [stdout] 130 + self.stack.push(self.counter.to_string()); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 130 - self.stack.push(self.counter.to_string().into()); [INFO] [stdout] 130 + self.stack.push(>::into(self.counter.to_string())); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/vm.rs:159:49 [INFO] [stdout] | [INFO] [stdout] 159 | self.scope.insert(name.clone(), var.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] 159 - self.scope.insert(name.clone(), var.into()); [INFO] [stdout] 159 + self.scope.insert(name.clone(), var); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 159 - self.scope.insert(name.clone(), var.into()); [INFO] [stdout] 159 + self.scope.insert(name.clone(), >::into(var)); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `pipa` (lib test) due to 9 previous errors [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:135:25 [INFO] [stdout] | [INFO] [stdout] 135 | let value = node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd]).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] 135 - let value = node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd]).into(); [INFO] [stdout] 135 + let value = node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd]); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 135 - let value = node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd]).into(); [INFO] [stdout] 135 + let value = >::into(node.as_escaped_string(code, &[TokenType::CodeBegin, TokenType::CodeEnd])); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:146:42 [INFO] [stdout] | [INFO] [stdout] 146 | ops.push(Op::PutStr { value: value.to_string().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] 146 - ops.push(Op::PutStr { value: value.to_string().into() }); [INFO] [stdout] 146 + ops.push(Op::PutStr { value: value.to_string() }); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 146 - ops.push(Op::PutStr { value: value.to_string().into() }); [INFO] [stdout] 146 + ops.push(Op::PutStr { value: >::into(value.to_string()) }); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:265:79 [INFO] [stdout] | [INFO] [stdout] 265 | ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: name.clone().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] 265 - ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: name.clone().into() }); [INFO] [stdout] 265 + ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: name.clone() }); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 265 - ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: name.clone().into() }); [INFO] [stdout] 265 + ops.push(Op::CmpArrayEmptyJmp{ op_index: 0, start, end, name: >::into(name.clone()) }); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:287:60 [INFO] [stdout] | [INFO] [stdout] 287 | ops.push(Op::LoadArrayItem { name: name.clone().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] 287 - ops.push(Op::LoadArrayItem { name: name.clone().into() }); [INFO] [stdout] 287 + ops.push(Op::LoadArrayItem { name: name.clone() }); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 287 - ops.push(Op::LoadArrayItem { name: name.clone().into() }); [INFO] [stdout] 287 + ops.push(Op::LoadArrayItem { name: >::into(name.clone()) }); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/ir.rs:300:56 [INFO] [stdout] | [INFO] [stdout] 300 | ops.push(Op::LoadArrayItem { name: name.clone().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] 300 - ops.push(Op::LoadArrayItem { name: name.clone().into() }); [INFO] [stdout] 300 + ops.push(Op::LoadArrayItem { name: name.clone() }); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 300 - ops.push(Op::LoadArrayItem { name: name.clone().into() }); [INFO] [stdout] 300 + ops.push(Op::LoadArrayItem { name: >::into(name.clone()) }); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/vm.rs:105:33 [INFO] [stdout] | [INFO] [stdout] 105 | self.stack.push(output.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] 105 - self.stack.push(output.into()); [INFO] [stdout] 105 + self.stack.push(output); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 105 - self.stack.push(output.into()); [INFO] [stdout] 105 + self.stack.push(>::into(output)); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/vm.rs:121:33 [INFO] [stdout] | [INFO] [stdout] 121 | self.stack.push(output.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] 121 - self.stack.push(output.into()); [INFO] [stdout] 121 + self.stack.push(output); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 121 - self.stack.push(output.into()); [INFO] [stdout] 121 + self.stack.push(>::into(output)); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/vm.rs:130:33 [INFO] [stdout] | [INFO] [stdout] 130 | self.stack.push(self.counter.to_string().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 - self.stack.push(self.counter.to_string().into()); [INFO] [stdout] 130 + self.stack.push(self.counter.to_string()); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 130 - self.stack.push(self.counter.to_string().into()); [INFO] [stdout] 130 + self.stack.push(>::into(self.counter.to_string())); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: useless conversion to the same type: `String` [INFO] [stdout] --> src/vm.rs:159:49 [INFO] [stdout] | [INFO] [stdout] 159 | self.scope.insert(name.clone(), var.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] 159 - self.scope.insert(name.clone(), var.into()); [INFO] [stdout] 159 + self.scope.insert(name.clone(), var); [INFO] [stdout] | [INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint [INFO] [stdout] | [INFO] [stdout] 159 - self.scope.insert(name.clone(), var.into()); [INFO] [stdout] 159 + self.scope.insert(name.clone(), >::into(var)); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `pipa` (lib) due to 9 previous errors; 3 warnings emitted [INFO] running `Command { std: "docker" "inspect" "e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed", kill_on_drop: false }` [INFO] [stdout] e10c4e7b5700ff74c88366f5986c632a7b1943d9e019f8b490d423b14cc0e0ed