[INFO] cloning repository https://github.com/Jax-Hax/First-Compiler [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/Jax-Hax/First-Compiler" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FJax-Hax%2FFirst-Compiler", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FJax-Hax%2FFirst-Compiler'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] a71be2cd0e9fb7ed51e7d4a4138888c34062b988 [INFO] checking Jax-Hax/First-Compiler against try#e117153a45c546e883c1f91d82611775fcaeffe0 for pr-146470 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FJax-Hax%2FFirst-Compiler" "/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/Jax-Hax/First-Compiler [INFO] finished tweaking git repo https://github.com/Jax-Hax/First-Compiler [INFO] tweaked toml for git repo https://github.com/Jax-Hax/First-Compiler written to /workspace/builds/worker-2-tc2/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/Jax-Hax/First-Compiler on toolchain e117153a45c546e883c1f91d82611775fcaeffe0 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+e117153a45c546e883c1f91d82611775fcaeffe0" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/Jax-Hax/First-Compiler 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" "+e117153a45c546e883c1f91d82611775fcaeffe0" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:4848fb76d95f26979359cc7e45710b1dbc8f3acb7aeedee7c460d7702230f228" "/opt/rustwide/cargo-home/bin/cargo" "+e117153a45c546e883c1f91d82611775fcaeffe0" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 4271f931651e050b7667235af489566e1d3dbc19f2f2663d762a1ddbc8c368b1 [INFO] running `Command { std: "docker" "start" "-a" "4271f931651e050b7667235af489566e1d3dbc19f2f2663d762a1ddbc8c368b1", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "4271f931651e050b7667235af489566e1d3dbc19f2f2663d762a1ddbc8c368b1", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "4271f931651e050b7667235af489566e1d3dbc19f2f2663d762a1ddbc8c368b1", kill_on_drop: false }` [INFO] [stdout] 4271f931651e050b7667235af489566e1d3dbc19f2f2663d762a1ddbc8c368b1 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-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" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:4848fb76d95f26979359cc7e45710b1dbc8f3acb7aeedee7c460d7702230f228" "/opt/rustwide/cargo-home/bin/cargo" "+e117153a45c546e883c1f91d82611775fcaeffe0" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 24224e7df14689fd76f0b46977cd37c4f11997c8c46fdae44f9104ba37517065 [INFO] running `Command { std: "docker" "start" "-a" "24224e7df14689fd76f0b46977cd37c4f11997c8c46fdae44f9104ba37517065", kill_on_drop: false }` [INFO] [stderr] Checking First-Compiler v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused variable: `expr_precendence` [INFO] [stdout] --> src/ast.rs:91:39 [INFO] [stdout] | [INFO] [stdout] 91 | fn parse_binary_op_rhs(&mut self, expr_precendence: i32, lhs: Node) -> Node { [INFO] [stdout] | ^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_expr_precendence` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `lhs` [INFO] [stdout] --> src/ast.rs:91:62 [INFO] [stdout] | [INFO] [stdout] 91 | fn parse_binary_op_rhs(&mut self, expr_precendence: i32, lhs: Node) -> Node { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_lhs` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `tok_precedence` [INFO] [stdout] --> src/ast.rs:93:13 [INFO] [stdout] | [INFO] [stdout] 93 | let tok_precedence = self.get_tok_precedence(); [INFO] [stdout] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_tok_precedence` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `expr_precendence` [INFO] [stdout] --> src/ast.rs:91:39 [INFO] [stdout] | [INFO] [stdout] 91 | fn parse_binary_op_rhs(&mut self, expr_precendence: i32, lhs: Node) -> Node { [INFO] [stdout] | ^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_expr_precendence` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `lhs` [INFO] [stdout] --> src/ast.rs:91:62 [INFO] [stdout] | [INFO] [stdout] 91 | fn parse_binary_op_rhs(&mut self, expr_precendence: i32, lhs: Node) -> Node { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_lhs` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `tok_precedence` [INFO] [stdout] --> src/ast.rs:93:13 [INFO] [stdout] | [INFO] [stdout] 93 | let tok_precedence = self.get_tok_precedence(); [INFO] [stdout] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_tok_precedence` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/ast.rs:154:9 [INFO] [stdout] | [INFO] [stdout] 154 | Val(f32), [INFO] [stdout] | --- ^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Node` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 154 - Val(f32), [INFO] [stdout] 154 + Val(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `name` and `args` are never read [INFO] [stdout] --> src/ast.rs:166:9 [INFO] [stdout] | [INFO] [stdout] 165 | FunctionCall { [INFO] [stdout] | ------------ fields in this variant [INFO] [stdout] 166 | name: String, [INFO] [stdout] | ^^^^ [INFO] [stdout] 167 | args: Vec>, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Node` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/ast.rs:173:14 [INFO] [stdout] | [INFO] [stdout] 173 | Variable(String), [INFO] [stdout] | -------- ^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Node` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 173 - Variable(String), [INFO] [stdout] 173 + Variable(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variants `Eof`, `UnaryExpr`, `BinaryExpr`, and `Function` are never constructed [INFO] [stdout] --> src/ast.rs:155:5 [INFO] [stdout] | [INFO] [stdout] 153 | pub enum Node { [INFO] [stdout] | ---- variants in this enum [INFO] [stdout] 154 | Val(f32), [INFO] [stdout] 155 | Eof, [INFO] [stdout] | ^^^ [INFO] [stdout] 156 | UnaryExpr { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 160 | BinaryExpr { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 169 | Function { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Node` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `name` and `args` are never read [INFO] [stdout] --> src/ast.rs:178:9 [INFO] [stdout] | [INFO] [stdout] 177 | pub struct FunctionParams { [INFO] [stdout] | -------------- fields in this struct [INFO] [stdout] 178 | pub name: String, [INFO] [stdout] | ^^^^ [INFO] [stdout] 179 | pub args: Vec, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `FunctionParams` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over an `Option`. This is more readably written as an `if let` statement [INFO] [stdout] --> src/main.rs:20:17 [INFO] [stdout] | [INFO] [stdout] 20 | for node in nodes { [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 20 - for node in nodes { [INFO] [stdout] 20 + while let Some(node) = nodes { [INFO] [stdout] | [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 20 - for node in nodes { [INFO] [stdout] 20 + if let Some(node) = nodes { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/ast.rs:154:9 [INFO] [stdout] | [INFO] [stdout] 154 | Val(f32), [INFO] [stdout] | --- ^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Node` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 154 - Val(f32), [INFO] [stdout] 154 + Val(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `name` and `args` are never read [INFO] [stdout] --> src/ast.rs:166:9 [INFO] [stdout] | [INFO] [stdout] 165 | FunctionCall { [INFO] [stdout] | ------------ fields in this variant [INFO] [stdout] 166 | name: String, [INFO] [stdout] | ^^^^ [INFO] [stdout] 167 | args: Vec>, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Node` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/ast.rs:173:14 [INFO] [stdout] | [INFO] [stdout] 173 | Variable(String), [INFO] [stdout] | -------- ^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Node` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 173 - Variable(String), [INFO] [stdout] 173 + Variable(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variants `Eof`, `UnaryExpr`, `BinaryExpr`, and `Function` are never constructed [INFO] [stdout] --> src/ast.rs:155:5 [INFO] [stdout] | [INFO] [stdout] 153 | pub enum Node { [INFO] [stdout] | ---- variants in this enum [INFO] [stdout] 154 | Val(f32), [INFO] [stdout] 155 | Eof, [INFO] [stdout] | ^^^ [INFO] [stdout] 156 | UnaryExpr { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 160 | BinaryExpr { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 169 | Function { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Node` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `name` and `args` are never read [INFO] [stdout] --> src/ast.rs:178:9 [INFO] [stdout] | [INFO] [stdout] 177 | pub struct FunctionParams { [INFO] [stdout] | -------------- fields in this struct [INFO] [stdout] 178 | pub name: String, [INFO] [stdout] | ^^^^ [INFO] [stdout] 179 | pub args: Vec, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `FunctionParams` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over an `Option`. This is more readably written as an `if let` statement [INFO] [stdout] --> src/main.rs:20:17 [INFO] [stdout] | [INFO] [stdout] 20 | for node in nodes { [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 20 - for node in nodes { [INFO] [stdout] 20 + while let Some(node) = nodes { [INFO] [stdout] | [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 20 - for node in nodes { [INFO] [stdout] 20 + if let Some(node) = nodes { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.08s [INFO] running `Command { std: "docker" "inspect" "24224e7df14689fd76f0b46977cd37c4f11997c8c46fdae44f9104ba37517065", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "24224e7df14689fd76f0b46977cd37c4f11997c8c46fdae44f9104ba37517065", kill_on_drop: false }` [INFO] [stdout] 24224e7df14689fd76f0b46977cd37c4f11997c8c46fdae44f9104ba37517065