[INFO] cloning repository https://github.com/DaanMiddelkoop/RealisticCircles
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/DaanMiddelkoop/RealisticCircles" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FDaanMiddelkoop%2FRealisticCircles", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FDaanMiddelkoop%2FRealisticCircles'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 418efa2f5d5a1eea0cbbe574096e90787e8c5dc1
[INFO] checking DaanMiddelkoop/RealisticCircles against master#029c9e18dd1f4668e1d42bb187c1c263dfe20093 for pr-157562
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FDaanMiddelkoop%2FRealisticCircles" "/workspace/builds/worker-7-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc1/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/DaanMiddelkoop/RealisticCircles
[INFO] finished tweaking git repo https://github.com/DaanMiddelkoop/RealisticCircles
[INFO] tweaked toml for git repo https://github.com/DaanMiddelkoop/RealisticCircles written to /workspace/builds/worker-7-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/DaanMiddelkoop/RealisticCircles on toolchain 029c9e18dd1f4668e1d42bb187c1c263dfe20093
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+029c9e18dd1f4668e1d42bb187c1c263dfe20093" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/DaanMiddelkoop/RealisticCircles 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" "+029c9e18dd1f4668e1d42bb187c1c263dfe20093" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr] warning: Cargo.toml: `package.edition` is unspecified, defaulting to `2015` while the latest is `2024`
[INFO] [stderr] warning: `RealisticCircles` (manifest) generated 1 warning
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded threads_pool v0.1.7
[INFO] [stderr]   Downloaded smallvec v0.6.5
[INFO] [stderr]   Downloaded rand_core v0.2.1
[INFO] [stderr]   Downloaded crossbeam-channel v0.2.4
[INFO] [stderr]   Downloaded version_check v0.1.4
[INFO] [stderr]   Downloaded lock_api v0.1.3
[INFO] [stderr]   Downloaded crossbeam-utils v0.5.0
[INFO] [stderr]   Downloaded owning_ref v0.3.3
[INFO] [stderr]   Downloaded nodrop v0.1.12
[INFO] [stderr]   Downloaded lazy_static v1.1.0
[INFO] [stderr]   Downloaded parking_lot v0.6.4
[INFO] [stderr]   Downloaded cfg-if v0.1.5
[INFO] [stderr]   Downloaded byteorder v1.2.6
[INFO] [stderr]   Downloaded parking_lot_core v0.3.1
[INFO] [stderr]   Downloaded arrayvec v0.4.7
[INFO] [stderr]   Downloaded crossbeam-epoch v0.5.2
[INFO] [stderr]   Downloaded bmp v0.4.0
[INFO] [stderr]   Downloaded winapi v0.3.5
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/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:3e793bdf10248eb2ddbd6b0faebed8155c11b4589982e2893733958d93e8d540" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] b021bdd723ceb2509844e6c5827f496887c07d42091de4603af92da1fc16c088
[INFO] running `Command { std: "docker" "start" "b021bdd723ceb2509844e6c5827f496887c07d42091de4603af92da1fc16c088", 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" "b021bdd723ceb2509844e6c5827f496887c07d42091de4603af92da1fc16c088" "/opt/rustwide/cargo-home/bin/cargo" "+029c9e18dd1f4668e1d42bb187c1c263dfe20093" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "b021bdd723ceb2509844e6c5827f496887c07d42091de4603af92da1fc16c088", 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" "b021bdd723ceb2509844e6c5827f496887c07d42091de4603af92da1fc16c088" "/opt/rustwide/cargo-home/bin/cargo" "+029c9e18dd1f4668e1d42bb187c1c263dfe20093" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr] warning: Cargo.toml: `package.edition` is unspecified, defaulting to `2015` while the latest is `2024`
[INFO] [stderr] warning: `RealisticCircles` (manifest) generated 1 warning
[INFO] [stderr]    Compiling semver-parser v0.7.0
[INFO] [stderr]    Compiling version_check v0.1.4
[INFO] [stderr]     Checking void v1.0.2
[INFO] [stderr]     Checking rand_core v0.2.1
[INFO] [stderr]     Checking stable_deref_trait v1.1.1
[INFO] [stderr]     Checking libc v0.2.43
[INFO] [stderr]     Checking scopeguard v0.3.3
[INFO] [stderr]     Checking nodrop v0.1.12
[INFO] [stderr]     Checking arrayvec v0.4.7
[INFO] [stderr]     Checking unreachable v1.0.0
[INFO] [stderr]     Checking cfg-if v0.1.5
[INFO] [stderr]     Checking owning_ref v0.3.3
[INFO] [stderr]     Checking crossbeam-utils v0.5.0
[INFO] [stderr]     Checking smallvec v0.6.5
[INFO] [stderr]     Checking memoffset v0.2.1
[INFO] [stderr]    Compiling lazy_static v1.1.0
[INFO] [stderr]     Checking byteorder v1.2.6
[INFO] [stderr]     Checking lock_api v0.1.3
[INFO] [stderr]    Compiling semver v0.9.0
[INFO] [stderr]     Checking crossbeam-epoch v0.5.2
[INFO] [stderr]     Checking bmp v0.4.0
[INFO] [stderr]     Checking rand v0.5.5
[INFO] [stderr]    Compiling rustc_version v0.2.3
[INFO] [stderr]    Compiling parking_lot_core v0.3.1
[INFO] [stderr]     Checking parking_lot v0.6.4
[INFO] [stderr]     Checking crossbeam-channel v0.2.4
[INFO] [stderr]     Checking threads_pool v0.1.7
[INFO] [stderr]     Checking RealisticCircles v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]   --> src/vector.rs:34:12
[INFO] [stdout]    |
[INFO] [stdout] 34 |         if (self.length < 0.0) {
[INFO] [stdout]    |            ^                 ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 34 -         if (self.length < 0.0) {
[INFO] [stdout] 34 +         if self.length < 0.0  {
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:107:8
[INFO] [stdout]     |
[INFO] [stdout] 107 |     if (result.hit) {
[INFO] [stdout]     |        ^          ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 107 -     if (result.hit) {
[INFO] [stdout] 107 +     if result.hit  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:130:12
[INFO] [stdout]     |
[INFO] [stdout] 130 |         if (result.hit && result.distance < closest_dist) {
[INFO] [stdout]     |            ^                                            ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 130 -         if (result.hit && result.distance < closest_dist) {
[INFO] [stdout] 130 +         if result.hit && result.distance < closest_dist  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:148:8
[INFO] [stdout]     |
[INFO] [stdout] 148 |     if (b >= sphere.radius || angle > f32::consts::PI / 2.0) {
[INFO] [stdout]     |        ^                                                   ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 148 -     if (b >= sphere.radius || angle > f32::consts::PI / 2.0) {
[INFO] [stdout] 148 +     if b >= sphere.radius || angle > f32::consts::PI / 2.0  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:170:8
[INFO] [stdout]     |
[INFO] [stdout] 170 |     if (bounces > 0) {
[INFO] [stdout]     |        ^           ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 170 -     if (bounces > 0) {
[INFO] [stdout] 170 +     if bounces > 0  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:188:12
[INFO] [stdout]     |
[INFO] [stdout] 188 |         if (new_result.hit && new_result.distance < new_direction.length()&& new_result.distance > 0.01) {
[INFO] [stdout]     |            ^                                                                                           ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 188 -         if (new_result.hit && new_result.distance < new_direction.length()&& new_result.distance > 0.01) {
[INFO] [stdout] 188 +         if new_result.hit && new_result.distance < new_direction.length()&& new_result.distance > 0.01  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:215:12
[INFO] [stdout]     |
[INFO] [stdout] 215 |         if (new_result.hit) {
[INFO] [stdout]     |            ^              ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 215 -         if (new_result.hit) {
[INFO] [stdout] 215 +         if new_result.hit  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `while` condition
[INFO] [stdout]    --> src/main.rs:243:11
[INFO] [stdout]     |
[INFO] [stdout] 243 |     while ((random.x * random.x) + (random.y * random.y) + (random.z * random.z) > 1.0) {
[INFO] [stdout]     |           ^                                                                           ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 243 -     while ((random.x * random.x) + (random.y * random.y) + (random.z * random.z) > 1.0) {
[INFO] [stdout] 243 +     while (random.x * random.x) + (random.y * random.y) + (random.z * random.z) > 1.0  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/main.rs:24:9
[INFO] [stdout]    |
[INFO] [stdout] 24 |     let mut bitmap = bmp::open("2000x2000.bmp").unwrap();
[INFO] [stdout]    |         ----^^^^^^
[INFO] [stdout]    |         |
[INFO] [stdout]    |         help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/main.rs:39:9
[INFO] [stdout]    |
[INFO] [stdout] 39 |     let mut worker_id = pool.get_first_worker_id().unwrap() as u32;
[INFO] [stdout]    |         ----^^^^^^^^^
[INFO] [stdout]    |         |
[INFO] [stdout]    |         help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `width`
[INFO] [stdout]   --> src/main.rs:34:9
[INFO] [stdout]    |
[INFO] [stdout] 34 |     let width = bitmap.get_width();
[INFO] [stdout]    |         ^^^^^ help: if this is intentional, prefix it with an underscore: `_width`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `worker_id`
[INFO] [stdout]   --> src/main.rs:39:9
[INFO] [stdout]    |
[INFO] [stdout] 39 |     let mut worker_id = pool.get_first_worker_id().unwrap() as u32;
[INFO] [stdout]    |         ^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_worker_id`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `poollock`
[INFO] [stdout]   --> src/main.rs:48:17
[INFO] [stdout]    |
[INFO] [stdout] 48 |             let poollock = c.clone();
[INFO] [stdout]    |                 ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_poollock`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/main.rs:106:9
[INFO] [stdout]     |
[INFO] [stdout] 106 |     let mut result = trace_ray(position, trace_direction, world, 2);
[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/main.rs:178:9
[INFO] [stdout]     |
[INFO] [stdout] 178 |     let mut total_color = Vector::new(1.0, 1.0, 1.0);
[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/main.rs:180:9
[INFO] [stdout]     |
[INFO] [stdout] 180 |     let mut hit_something = false;
[INFO] [stdout]     |         ----^^^^^^^^^^^^^
[INFO] [stdout]     |         |
[INFO] [stdout]     |         help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `total_color`
[INFO] [stdout]    --> src/main.rs:178:9
[INFO] [stdout]     |
[INFO] [stdout] 178 |     let mut total_color = Vector::new(1.0, 1.0, 1.0);
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_total_color`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `hit_something`
[INFO] [stdout]    --> src/main.rs:180:9
[INFO] [stdout]     |
[INFO] [stdout] 180 |     let mut hit_something = false;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_hit_something`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/main.rs:268:9
[INFO] [stdout]     |
[INFO] [stdout] 268 |     let mut result = direction.add(&term);
[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/vector.rs:34:12
[INFO] [stdout]    |
[INFO] [stdout] 34 |         if (self.length < 0.0) {
[INFO] [stdout]    |            ^                 ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 34 -         if (self.length < 0.0) {
[INFO] [stdout] 34 +         if self.length < 0.0  {
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `crossProduct` should have a snake case name
[INFO] [stdout]   --> src/vector.rs:23:12
[INFO] [stdout]    |
[INFO] [stdout] 23 |     pub fn crossProduct(&self, other: &Vector) -> Vector {
[INFO] [stdout]    |            ^^^^^^^^^^^^ help: convert the identifier to snake case: `cross_product`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:107:8
[INFO] [stdout]     |
[INFO] [stdout] 107 |     if (result.hit) {
[INFO] [stdout]     |        ^          ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 107 -     if (result.hit) {
[INFO] [stdout] 107 +     if result.hit  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:130:12
[INFO] [stdout]     |
[INFO] [stdout] 130 |         if (result.hit && result.distance < closest_dist) {
[INFO] [stdout]     |            ^                                            ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 130 -         if (result.hit && result.distance < closest_dist) {
[INFO] [stdout] 130 +         if result.hit && result.distance < closest_dist  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:148:8
[INFO] [stdout]     |
[INFO] [stdout] 148 |     if (b >= sphere.radius || angle > f32::consts::PI / 2.0) {
[INFO] [stdout]     |        ^                                                   ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 148 -     if (b >= sphere.radius || angle > f32::consts::PI / 2.0) {
[INFO] [stdout] 148 +     if b >= sphere.radius || angle > f32::consts::PI / 2.0  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:170:8
[INFO] [stdout]     |
[INFO] [stdout] 170 |     if (bounces > 0) {
[INFO] [stdout]     |        ^           ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 170 -     if (bounces > 0) {
[INFO] [stdout] 170 +     if bounces > 0  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:188:12
[INFO] [stdout]     |
[INFO] [stdout] 188 |         if (new_result.hit && new_result.distance < new_direction.length()&& new_result.distance > 0.01) {
[INFO] [stdout]     |            ^                                                                                           ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 188 -         if (new_result.hit && new_result.distance < new_direction.length()&& new_result.distance > 0.01) {
[INFO] [stdout] 188 +         if new_result.hit && new_result.distance < new_direction.length()&& new_result.distance > 0.01  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/main.rs:215:12
[INFO] [stdout]     |
[INFO] [stdout] 215 |         if (new_result.hit) {
[INFO] [stdout]     |            ^              ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 215 -         if (new_result.hit) {
[INFO] [stdout] 215 +         if new_result.hit  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `while` condition
[INFO] [stdout]    --> src/main.rs:243:11
[INFO] [stdout]     |
[INFO] [stdout] 243 |     while ((random.x * random.x) + (random.y * random.y) + (random.z * random.z) > 1.0) {
[INFO] [stdout]     |           ^                                                                           ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 243 -     while ((random.x * random.x) + (random.y * random.y) + (random.z * random.z) > 1.0) {
[INFO] [stdout] 243 +     while (random.x * random.x) + (random.y * random.y) + (random.z * random.z) > 1.0  {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/main.rs:24:9
[INFO] [stdout]    |
[INFO] [stdout] 24 |     let mut bitmap = bmp::open("2000x2000.bmp").unwrap();
[INFO] [stdout]    |         ----^^^^^^
[INFO] [stdout]    |         |
[INFO] [stdout]    |         help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/main.rs:39:9
[INFO] [stdout]    |
[INFO] [stdout] 39 |     let mut worker_id = pool.get_first_worker_id().unwrap() as u32;
[INFO] [stdout]    |         ----^^^^^^^^^
[INFO] [stdout]    |         |
[INFO] [stdout]    |         help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `width`
[INFO] [stdout]   --> src/main.rs:34:9
[INFO] [stdout]    |
[INFO] [stdout] 34 |     let width = bitmap.get_width();
[INFO] [stdout]    |         ^^^^^ help: if this is intentional, prefix it with an underscore: `_width`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `worker_id`
[INFO] [stdout]   --> src/main.rs:39:9
[INFO] [stdout]    |
[INFO] [stdout] 39 |     let mut worker_id = pool.get_first_worker_id().unwrap() as u32;
[INFO] [stdout]    |         ^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_worker_id`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `poollock`
[INFO] [stdout]   --> src/main.rs:48:17
[INFO] [stdout]    |
[INFO] [stdout] 48 |             let poollock = c.clone();
[INFO] [stdout]    |                 ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_poollock`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/main.rs:106:9
[INFO] [stdout]     |
[INFO] [stdout] 106 |     let mut result = trace_ray(position, trace_direction, world, 2);
[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/main.rs:178:9
[INFO] [stdout]     |
[INFO] [stdout] 178 |     let mut total_color = Vector::new(1.0, 1.0, 1.0);
[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/main.rs:180:9
[INFO] [stdout]     |
[INFO] [stdout] 180 |     let mut hit_something = false;
[INFO] [stdout]     |         ----^^^^^^^^^^^^^
[INFO] [stdout]     |         |
[INFO] [stdout]     |         help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `total_color`
[INFO] [stdout]    --> src/main.rs:178:9
[INFO] [stdout]     |
[INFO] [stdout] 178 |     let mut total_color = Vector::new(1.0, 1.0, 1.0);
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_total_color`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `hit_something`
[INFO] [stdout]    --> src/main.rs:180:9
[INFO] [stdout]     |
[INFO] [stdout] 180 |     let mut hit_something = false;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_hit_something`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/main.rs:268:9
[INFO] [stdout]     |
[INFO] [stdout] 268 |     let mut result = direction.add(&term);
[INFO] [stdout]     |         ----^^^^^^
[INFO] [stdout]     |         |
[INFO] [stdout]     |         help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `crossProduct` should have a snake case name
[INFO] [stdout]   --> src/vector.rs:23:12
[INFO] [stdout]    |
[INFO] [stdout] 23 |     pub fn crossProduct(&self, other: &Vector) -> Vector {
[INFO] [stdout]    |            ^^^^^^^^^^^^ help: convert the identifier to snake case: `cross_product`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 4.88s
[INFO] running `Command { std: "docker" "inspect" "b021bdd723ceb2509844e6c5827f496887c07d42091de4603af92da1fc16c088", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "b021bdd723ceb2509844e6c5827f496887c07d42091de4603af92da1fc16c088", kill_on_drop: false }`
[INFO] [stdout] b021bdd723ceb2509844e6c5827f496887c07d42091de4603af92da1fc16c088
