[INFO] cloning repository https://github.com/jkf16m/tree-data-structures [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/jkf16m/tree-data-structures" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjkf16m%2Ftree-data-structures", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjkf16m%2Ftree-data-structures'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 11f6eb0eb4699b4cef254ad2fa0f4483fd49e5b3 [INFO] checking jkf16m/tree-data-structures against master#ed141926048597e3649bb238ca3dc417904cd56c for pr-134300 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjkf16m%2Ftree-data-structures" "/workspace/builds/worker-1-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-1-tc1/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/jkf16m/tree-data-structures on toolchain ed141926048597e3649bb238ca3dc417904cd56c [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+ed141926048597e3649bb238ca3dc417904cd56c" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/jkf16m/tree-data-structures [INFO] finished tweaking git repo https://github.com/jkf16m/tree-data-structures [INFO] tweaked toml for git repo https://github.com/jkf16m/tree-data-structures written to /workspace/builds/worker-1-tc1/source/Cargo.toml [INFO] crate git repo https://github.com/jkf16m/tree-data-structures 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" "+ed141926048597e3649bb238ca3dc417904cd56c" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/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:f3a9d4ad9d972b27faf3965f35b62e55ba32bbce8f20bc8fe909558a86702fde" "/opt/rustwide/cargo-home/bin/cargo" "+ed141926048597e3649bb238ca3dc417904cd56c" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 1910225b8f0fa0410ea27cdeb77119a986e7c8e9c712fd6195ecd7b463d7719f [INFO] running `Command { std: "docker" "start" "-a" "1910225b8f0fa0410ea27cdeb77119a986e7c8e9c712fd6195ecd7b463d7719f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "1910225b8f0fa0410ea27cdeb77119a986e7c8e9c712fd6195ecd7b463d7719f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "1910225b8f0fa0410ea27cdeb77119a986e7c8e9c712fd6195ecd7b463d7719f", kill_on_drop: false }` [INFO] [stdout] 1910225b8f0fa0410ea27cdeb77119a986e7c8e9c712fd6195ecd7b463d7719f [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc1/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:f3a9d4ad9d972b27faf3965f35b62e55ba32bbce8f20bc8fe909558a86702fde" "/opt/rustwide/cargo-home/bin/cargo" "+ed141926048597e3649bb238ca3dc417904cd56c" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 9b4716ac825fc59382832a7589ab0bfb0f1e1e986c4a44086fe64e186e9c41f9 [INFO] running `Command { std: "docker" "start" "-a" "9b4716ac825fc59382832a7589ab0bfb0f1e1e986c4a44086fe64e186e9c41f9", kill_on_drop: false }` [INFO] [stderr] Checking tree-data-structures v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/tree.rs:181:16 [INFO] [stdout] | [INFO] [stdout] 181 | if (current_nodes_id.len() > 0){ [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 181 - if (current_nodes_id.len() > 0){ [INFO] [stdout] 181 + if current_nodes_id.len() > 0 { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/tree.rs:181:16 [INFO] [stdout] | [INFO] [stdout] 181 | if (current_nodes_id.len() > 0){ [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 181 - if (current_nodes_id.len() > 0){ [INFO] [stdout] 181 + if current_nodes_id.len() > 0 { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/tree.rs:104:13 [INFO] [stdout] | [INFO] [stdout] 104 | let mut parent = &mut self.nodes[parent_id]; [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `current_node_children_id` [INFO] [stdout] --> src/tree.rs:178:17 [INFO] [stdout] | [INFO] [stdout] 178 | let mut current_node_children_id: Vec = Vec::new(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_current_node_children_id` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/tree.rs:178:13 [INFO] [stdout] | [INFO] [stdout] 178 | let mut current_node_children_id: Vec = Vec::new(); [INFO] [stdout] | ----^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `parent_id` is never read [INFO] [stdout] --> src/tree.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 2 | pub struct Node{ [INFO] [stdout] | ---- field in this struct [INFO] [stdout] 3 | depth_level: usize, [INFO] [stdout] 4 | parent_id: Option, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Node` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: crate `tree_DS` should have a snake case name [INFO] [stdout] | [INFO] [stdout] = help: convert the identifier to snake case: `tree_ds` [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/tree.rs:104:13 [INFO] [stdout] | [INFO] [stdout] 104 | let mut parent = &mut self.nodes[parent_id]; [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `current_node_children_id` [INFO] [stdout] --> src/tree.rs:178:17 [INFO] [stdout] | [INFO] [stdout] 178 | let mut current_node_children_id: Vec = Vec::new(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_current_node_children_id` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/tree.rs:178:13 [INFO] [stdout] | [INFO] [stdout] 178 | let mut current_node_children_id: Vec = Vec::new(); [INFO] [stdout] | ----^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/tree.rs:181:16 [INFO] [stdout] | [INFO] [stdout] 181 | if (current_nodes_id.len() > 0){ [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 181 - if (current_nodes_id.len() > 0){ [INFO] [stdout] 181 + if current_nodes_id.len() > 0 { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `tree::Tree` [INFO] [stdout] --> src/main.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use tree::Tree; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/tree.rs:181:16 [INFO] [stdout] | [INFO] [stdout] 181 | if (current_nodes_id.len() > 0){ [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 181 - if (current_nodes_id.len() > 0){ [INFO] [stdout] 181 + if current_nodes_id.len() > 0 { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `tree::Tree` [INFO] [stdout] --> src/main.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use tree::Tree; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/tree.rs:104:13 [INFO] [stdout] | [INFO] [stdout] 104 | let mut parent = &mut self.nodes[parent_id]; [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `current_node_children_id` [INFO] [stdout] --> src/tree.rs:178:17 [INFO] [stdout] | [INFO] [stdout] 178 | let mut current_node_children_id: Vec = Vec::new(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_current_node_children_id` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/tree.rs:178:13 [INFO] [stdout] | [INFO] [stdout] 178 | let mut current_node_children_id: Vec = Vec::new(); [INFO] [stdout] | ----^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/tree.rs:104:13 [INFO] [stdout] | [INFO] [stdout] 104 | let mut parent = &mut self.nodes[parent_id]; [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `current_node_children_id` [INFO] [stdout] --> src/tree.rs:178:17 [INFO] [stdout] | [INFO] [stdout] 178 | let mut current_node_children_id: Vec = Vec::new(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_current_node_children_id` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `new_empty`, `get_current_max_depth`, and `matches_children` are never used [INFO] [stdout] --> src/tree.rs:14:12 [INFO] [stdout] | [INFO] [stdout] 13 | impl Tree{ [INFO] [stdout] | --------------- associated items in this implementation [INFO] [stdout] 14 | pub fn new_empty()->Tree{ [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 32 | pub fn get_current_max_depth(&self)->usize{ [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 61 | pub fn matches_children(&self, parent_id: usize, value:T)->Option where T: PartialEq{ [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/tree.rs:178:13 [INFO] [stdout] | [INFO] [stdout] 178 | let mut current_node_children_id: Vec = Vec::new(); [INFO] [stdout] | ----^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Node` is never constructed [INFO] [stdout] --> src/tree.rs:2:12 [INFO] [stdout] | [INFO] [stdout] 2 | pub struct Node{ [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Node` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Tree` is never constructed [INFO] [stdout] --> src/tree.rs:9:12 [INFO] [stdout] | [INFO] [stdout] 9 | pub struct Tree{ [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: multiple associated items are never used [INFO] [stdout] --> src/tree.rs:14:12 [INFO] [stdout] | [INFO] [stdout] 13 | impl Tree{ [INFO] [stdout] | --------------- associated items in this implementation [INFO] [stdout] 14 | pub fn new_empty()->Tree{ [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 19 | pub fn new(value: T)->Tree{ [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 32 | pub fn get_current_max_depth(&self)->usize{ [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 43 | pub fn get_children(&self, parent_id: usize)->Vec<&Node>{ [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 61 | pub fn matches_children(&self, parent_id: usize, value:T)->Option where T: PartialEq{ [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 94 | pub fn add(mut self, value: T, parent_id: usize)->Tree{ [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 109 | pub fn find(&self, predicate: &dyn Fn(&T)->bool)->Option{ [INFO] [stdout] | ^^^^ [INFO] [stdout] ... [INFO] [stdout] 141 | pub fn matches_branch(&self, branch: Vec)->Option> where T: PartialEq + Clone{ [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 176 | pub fn matches_branch_predicated(&self, branch:Vec, predicate: &dyn Fn(&U, &T)->bool)->Option> where T: Clone{ [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.33s [INFO] running `Command { std: "docker" "inspect" "9b4716ac825fc59382832a7589ab0bfb0f1e1e986c4a44086fe64e186e9c41f9", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "9b4716ac825fc59382832a7589ab0bfb0f1e1e986c4a44086fe64e186e9c41f9", kill_on_drop: false }` [INFO] [stdout] 9b4716ac825fc59382832a7589ab0bfb0f1e1e986c4a44086fe64e186e9c41f9