[INFO] cloning repository https://github.com/botiapa/neo
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/botiapa/neo" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fbotiapa%2Fneo", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fbotiapa%2Fneo'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] f00a48e937a9f4cf4351139434d248e18c069799
[INFO] checking botiapa/neo against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fbotiapa%2Fneo" "/workspace/builds/worker-2-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-2-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/botiapa/neo
[INFO] finished tweaking git repo https://github.com/botiapa/neo
[INFO] tweaked toml for git repo https://github.com/botiapa/neo written to /workspace/builds/worker-2-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/botiapa/neo 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/botiapa/neo 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-2-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-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] 47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c
[INFO] running `Command { std: "docker" "start" "47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c", 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" "47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c", 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" "47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling proc-macro2 v1.0.94
[INFO] [stderr]     Checking once_cell v1.21.1
[INFO] [stderr]     Checking regex-syntax v0.8.5
[INFO] [stderr]     Checking log v0.4.27
[INFO] [stderr]     Checking smallvec v1.14.0
[INFO] [stderr]     Checking regex-automata v0.1.10
[INFO] [stderr]     Checking tracing-core v0.1.33
[INFO] [stderr]     Checking thread_local v1.1.8
[INFO] [stderr]     Checking tracing-log v0.2.0
[INFO] [stderr]     Checking matchers v0.1.0
[INFO] [stderr]    Compiling quote v1.0.40
[INFO] [stderr]    Compiling syn v2.0.100
[INFO] [stderr]     Checking regex-automata v0.4.9
[INFO] [stderr]     Checking regex v1.11.1
[INFO] [stderr]    Compiling tracing-attributes v0.1.28
[INFO] [stderr]     Checking tracing v0.1.41
[INFO] [stderr]     Checking tracing-subscriber v0.3.19
[INFO] [stderr]     Checking neo v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `crate::tokenizer::Token`
[INFO] [stdout]  --> src/expression/helpers.rs:1:5
[INFO] [stdout]   |
[INFO] [stdout] 1 | use crate::tokenizer::Token;
[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: `Parser`
[INFO] [stdout]  --> src/expression/helpers.rs:4:78
[INFO] [stdout]   |
[INFO] [stdout] 4 |     Args, BinaryOp, EnumDeclaration, EnumVariant, Expr, FunctionDeclaration, Parser, UnaryOp,
[INFO] [stdout]   |                                                                              ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Args`
[INFO] [stdout]  --> src/interpreter/functions.rs:3:25
[INFO] [stdout]   |
[INFO] [stdout] 3 | use crate::expression::{Args, Expr, FunctionDeclaration};
[INFO] [stdout]   |                         ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `crate::tokenizer::Token`
[INFO] [stdout]  --> src/expression/helpers.rs:1:5
[INFO] [stdout]   |
[INFO] [stdout] 1 | use crate::tokenizer::Token;
[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: `Parser`
[INFO] [stdout]  --> src/expression/helpers.rs:4:78
[INFO] [stdout]   |
[INFO] [stdout] 4 |     Args, BinaryOp, EnumDeclaration, EnumVariant, Expr, FunctionDeclaration, Parser, UnaryOp,
[INFO] [stdout]   |                                                                              ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Args`
[INFO] [stdout]  --> src/interpreter/functions.rs:3:25
[INFO] [stdout]   |
[INFO] [stdout] 3 | use crate::expression::{Args, Expr, FunctionDeclaration};
[INFO] [stdout]   |                         ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `function_call`
[INFO] [stdout]    --> src/interpreter.rs:354:64
[INFO] [stdout]     |
[INFO] [stdout] 354 |         self, binary, block, bool_lit, enum_variant, function, function_call, iden, num_lit,
[INFO] [stdout]     |                                                                ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `crate::tokenizer::Token`
[INFO] [stdout]    --> src/interpreter.rs:358:9
[INFO] [stdout]     |
[INFO] [stdout] 358 |     use crate::tokenizer::Token;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `empty` is never used
[INFO] [stdout]    --> src/expression.rs:122:8
[INFO] [stdout]     |
[INFO] [stdout]  82 | impl Parser {
[INFO] [stdout]     | ----------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 122 |     fn empty(&self) -> bool {
[INFO] [stdout]     |        ^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `num_lit` is never used
[INFO] [stdout]  --> src/expression/helpers.rs:8:15
[INFO] [stdout]   |
[INFO] [stdout] 8 | pub(crate) fn num_lit(n: i32) -> Expr {
[INFO] [stdout]   |               ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `string_lit` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:12:15
[INFO] [stdout]    |
[INFO] [stdout] 12 | pub(crate) fn string_lit(s: String) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `iden` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:20:15
[INFO] [stdout]    |
[INFO] [stdout] 20 | pub(crate) fn iden(id: &str) -> Expr {
[INFO] [stdout]    |               ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `iden_with_path` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:24:15
[INFO] [stdout]    |
[INFO] [stdout] 24 | pub(crate) fn iden_with_path(id: &str, path: Vec<String>) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `unary` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:28:15
[INFO] [stdout]    |
[INFO] [stdout] 28 | pub(crate) fn unary(op: UnaryOp, expr: Expr) -> Expr {
[INFO] [stdout]    |               ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `binary` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:32:15
[INFO] [stdout]    |
[INFO] [stdout] 32 | pub(crate) fn binary(op: BinaryOp, left: Expr, right: Expr) -> Expr {
[INFO] [stdout]    |               ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `function` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:36:15
[INFO] [stdout]    |
[INFO] [stdout] 36 | pub(crate) fn function(name: String, args: Vec<Expr>) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `function_call` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:40:15
[INFO] [stdout]    |
[INFO] [stdout] 40 | pub(crate) fn function_call(name: String, args: Vec<Expr>) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `block` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:44:15
[INFO] [stdout]    |
[INFO] [stdout] 44 | pub(crate) fn block(exprs: &[Expr]) -> Expr {
[INFO] [stdout]    |               ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `assignment` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:48:15
[INFO] [stdout]    |
[INFO] [stdout] 48 | pub(crate) fn assignment(id: String, expr: Expr) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `assignment_typed` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:52:15
[INFO] [stdout]    |
[INFO] [stdout] 52 | pub(crate) fn assignment_typed(id: String, expr: Expr, var_type: VarType) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `if_expr` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:56:15
[INFO] [stdout]    |
[INFO] [stdout] 56 | pub(crate) fn if_expr(cond: Expr, then: Expr, else_block: Option<Expr>) -> Expr {
[INFO] [stdout]    |               ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `while_expr` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:60:15
[INFO] [stdout]    |
[INFO] [stdout] 60 | pub(crate) fn while_expr(cond: Expr, block: Expr) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `enum_dec` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:64:15
[INFO] [stdout]    |
[INFO] [stdout] 64 | pub(crate) fn enum_dec(name: String, variants: Vec<(String, Option<Vec<String>>)>) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `no_op` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:68:15
[INFO] [stdout]    |
[INFO] [stdout] 68 | pub(crate) fn no_op() -> Expr {
[INFO] [stdout]    |               ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `function_declaration` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:72:15
[INFO] [stdout]    |
[INFO] [stdout] 72 | pub(crate) fn function_declaration(name: String, args: Args, body: Expr) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `generic_function_declaration` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:81:15
[INFO] [stdout]    |
[INFO] [stdout] 81 | pub(crate) fn generic_function_declaration(
[INFO] [stdout]    |               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `enum_variant` is never used
[INFO] [stdout]   --> src/expression/helpers.rs:95:15
[INFO] [stdout]    |
[INFO] [stdout] 95 | pub(crate) fn enum_variant(enum_name: &str, variant_name: &str, values: Vec<Expr>) -> Expr {
[INFO] [stdout]    |               ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `is_expr` is never used
[INFO] [stdout]    --> src/expression/helpers.rs:103:15
[INFO] [stdout]     |
[INFO] [stdout] 103 | pub(crate) fn is_expr(iden: &str, enum_variant: Expr) -> Expr {
[INFO] [stdout]     |               ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated functions `new_immutable` and `new_immutable_typed` are never used
[INFO] [stdout]   --> src/interpreter/variable.rs:21:19
[INFO] [stdout]    |
[INFO] [stdout] 12 | impl Variable {
[INFO] [stdout]    | ------------- associated functions in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 21 |     pub(super) fn new_immutable(value: Option<Expr>) -> Option<Self> {
[INFO] [stdout]    |                   ^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 44 |     pub(super) fn new_immutable_typed(value: Option<Expr>, var_type: Type) -> Result<Self, String> {
[INFO] [stdout]    |                   ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:20:54
[INFO] [stdout]    |
[INFO] [stdout] 20 |             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(Expr::NumLit(op_fn(*a, *b)).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] 20 -             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(Expr::NumLit(op_fn(*a, *b)).into()),
[INFO] [stdout] 20 +             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(Expr::NumLit(op_fn(*a, *b))),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 20 -             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(Expr::NumLit(op_fn(*a, *b)).into()),
[INFO] [stdout] 20 +             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(<Expr as Into<_>>::into(Expr::NumLit(op_fn(*a, *b)))),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:22:20
[INFO] [stdout]    |
[INFO] [stdout] 22 |                 Ok(Expr::StringLit(b.to_string() + &a.to_string()).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] 22 -                 Ok(Expr::StringLit(b.to_string() + &a.to_string()).into())
[INFO] [stdout] 22 +                 Ok(Expr::StringLit(b.to_string() + &a.to_string()))
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 22 -                 Ok(Expr::StringLit(b.to_string() + &a.to_string()).into())
[INFO] [stdout] 22 +                 Ok(<Expr as Into<_>>::into(Expr::StringLit(b.to_string() + &a.to_string())))
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:25:20
[INFO] [stdout]    |
[INFO] [stdout] 25 |                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).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] 25 -                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).into())
[INFO] [stdout] 25 +                 Ok(Expr::StringLit(a.to_string() + &b.to_string()))
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 25 -                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).into())
[INFO] [stdout] 25 +                 Ok(<Expr as Into<_>>::into(Expr::StringLit(a.to_string() + &b.to_string())))
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:28:20
[INFO] [stdout]    |
[INFO] [stdout] 28 |                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).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] 28 -                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).into())
[INFO] [stdout] 28 +                 Ok(Expr::StringLit(a.to_string() + &b.to_string()))
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 28 -                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).into())
[INFO] [stdout] 28 +                 Ok(<Expr as Into<_>>::into(Expr::StringLit(a.to_string() + &b.to_string())))
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:46:23
[INFO] [stdout]    |
[INFO] [stdout] 46 |             return Ok(Expr::BoolLit(op_fn(a, b)).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] 46 -             return Ok(Expr::BoolLit(op_fn(a, b)).into());
[INFO] [stdout] 46 +             return Ok(Expr::BoolLit(op_fn(a, b)));
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 46 -             return Ok(Expr::BoolLit(op_fn(a, b)).into());
[INFO] [stdout] 46 +             return Ok(<Expr as Into<_>>::into(Expr::BoolLit(op_fn(a, b))));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Expr`
[INFO] [stdout]    --> src/interpreter.rs:186:48
[INFO] [stdout]     |
[INFO] [stdout] 186 |                     last = self.interpret_expr(expr.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] 186 -                     last = self.interpret_expr(expr.into())?;
[INFO] [stdout] 186 +                     last = self.interpret_expr(expr)?;
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 186 -                     last = self.interpret_expr(expr.into())?;
[INFO] [stdout] 186 +                     last = self.interpret_expr(<Expr as Into<_>>::into(expr))?;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `neo` (bin "neo") due to 6 previous errors; 24 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: associated functions `new_immutable` and `new_immutable_typed` are never used
[INFO] [stdout]   --> src/interpreter/variable.rs:21:19
[INFO] [stdout]    |
[INFO] [stdout] 12 | impl Variable {
[INFO] [stdout]    | ------------- associated functions in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 21 |     pub(super) fn new_immutable(value: Option<Expr>) -> Option<Self> {
[INFO] [stdout]    |                   ^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 44 |     pub(super) fn new_immutable_typed(value: Option<Expr>, var_type: Type) -> Result<Self, String> {
[INFO] [stdout]    |                   ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `expression::Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:20:54
[INFO] [stdout]    |
[INFO] [stdout] 20 |             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(Expr::NumLit(op_fn(*a, *b)).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] 20 -             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(Expr::NumLit(op_fn(*a, *b)).into()),
[INFO] [stdout] 20 +             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(Expr::NumLit(op_fn(*a, *b))),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 20 -             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(Expr::NumLit(op_fn(*a, *b)).into()),
[INFO] [stdout] 20 +             (Expr::NumLit(a), Expr::NumLit(b)) => Ok(<expression::Expr as Into<_>>::into(Expr::NumLit(op_fn(*a, *b)))),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `expression::Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:22:20
[INFO] [stdout]    |
[INFO] [stdout] 22 |                 Ok(Expr::StringLit(b.to_string() + &a.to_string()).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] 22 -                 Ok(Expr::StringLit(b.to_string() + &a.to_string()).into())
[INFO] [stdout] 22 +                 Ok(Expr::StringLit(b.to_string() + &a.to_string()))
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 22 -                 Ok(Expr::StringLit(b.to_string() + &a.to_string()).into())
[INFO] [stdout] 22 +                 Ok(<expression::Expr as Into<_>>::into(Expr::StringLit(b.to_string() + &a.to_string())))
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `expression::Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:25:20
[INFO] [stdout]    |
[INFO] [stdout] 25 |                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).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] 25 -                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).into())
[INFO] [stdout] 25 +                 Ok(Expr::StringLit(a.to_string() + &b.to_string()))
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 25 -                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).into())
[INFO] [stdout] 25 +                 Ok(<expression::Expr as Into<_>>::into(Expr::StringLit(a.to_string() + &b.to_string())))
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `expression::Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:28:20
[INFO] [stdout]    |
[INFO] [stdout] 28 |                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).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] 28 -                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).into())
[INFO] [stdout] 28 +                 Ok(Expr::StringLit(a.to_string() + &b.to_string()))
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 28 -                 Ok(Expr::StringLit(a.to_string() + &b.to_string()).into())
[INFO] [stdout] 28 +                 Ok(<expression::Expr as Into<_>>::into(Expr::StringLit(a.to_string() + &b.to_string())))
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `expression::Expr`
[INFO] [stdout]   --> src/interpreter/operators.rs:46:23
[INFO] [stdout]    |
[INFO] [stdout] 46 |             return Ok(Expr::BoolLit(op_fn(a, b)).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] 46 -             return Ok(Expr::BoolLit(op_fn(a, b)).into());
[INFO] [stdout] 46 +             return Ok(Expr::BoolLit(op_fn(a, b)));
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 46 -             return Ok(Expr::BoolLit(op_fn(a, b)).into());
[INFO] [stdout] 46 +             return Ok(<expression::Expr as Into<_>>::into(Expr::BoolLit(op_fn(a, b))));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `expression::Expr`
[INFO] [stdout]    --> src/interpreter.rs:186:48
[INFO] [stdout]     |
[INFO] [stdout] 186 |                     last = self.interpret_expr(expr.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] 186 -                     last = self.interpret_expr(expr.into())?;
[INFO] [stdout] 186 +                     last = self.interpret_expr(expr)?;
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 186 -                     last = self.interpret_expr(expr.into())?;
[INFO] [stdout] 186 +                     last = self.interpret_expr(<expression::Expr as Into<_>>::into(expr))?;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `neo` (bin "neo" test) due to 6 previous errors; 6 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c", kill_on_drop: false }`
[INFO] [stdout] 47bb08f6e54641d567b323356793ea0e58cadf30f0a1d32d95b6b4f61fcca29c
