[INFO] crate rtow 0.1.0 is already in cache [INFO] extracting crate rtow 0.1.0 into work/ex/clippy-test-run/sources/stable/reg/rtow/0.1.0 [INFO] extracting crate rtow 0.1.0 into work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/reg/rtow/0.1.0 [INFO] validating manifest of rtow-0.1.0 on toolchain stable [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "read-manifest" "--manifest-path" "Cargo.toml"` [INFO] validating manifest of rtow-0.1.0 on toolchain stable+rustflags=-Dclippy::into_iter_on_array [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "read-manifest" "--manifest-path" "Cargo.toml"` [INFO] started frobbing rtow-0.1.0 [INFO] finished frobbing rtow-0.1.0 [INFO] frobbed toml for rtow-0.1.0 written to work/ex/clippy-test-run/sources/stable/reg/rtow/0.1.0/Cargo.toml [INFO] started frobbing rtow-0.1.0 [INFO] finished frobbing rtow-0.1.0 [INFO] frobbed toml for rtow-0.1.0 written to work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/reg/rtow/0.1.0/Cargo.toml [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "generate-lockfile" "--manifest-path" "Cargo.toml" "-Zno-index-update"` [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "generate-lockfile" "--manifest-path" "Cargo.toml" "-Zno-index-update"` [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "fetch" "--locked" "--manifest-path" "Cargo.toml"` [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "fetch" "--locked" "--manifest-path" "Cargo.toml"` [INFO] linting rtow-0.1.0 against stable for clippy-test-run [INFO] running `"docker" "create" "-v" "/mnt/big/crater/work/local/target-dirs/clippy-test-run/worker-6/stable:/opt/crater/target:rw,Z" "-v" "/mnt/big/crater/work/ex/clippy-test-run/sources/stable/reg/rtow/0.1.0:/opt/crater/workdir:ro,Z" "-v" "/mnt/big/crater/work/local/cargo-home:/opt/crater/cargo-home:ro,Z" "-v" "/mnt/big/crater/work/local/rustup-home:/opt/crater/rustup-home:ro,Z" "-e" "USER_ID=1000" "-e" "SOURCE_DIR=/opt/crater/workdir" "-e" "MAP_USER_ID=1000" "-e" "CARGO_TARGET_DIR=/opt/crater/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/crater/cargo-home" "-e" "RUSTUP_HOME=/opt/crater/rustup-home" "-w" "/opt/crater/workdir" "-m" "1536M" "--network" "none" "rustops/crates-build-env" "/opt/crater/cargo-home/bin/cargo" "+stable" "clippy" "--frozen" "--all" "--all-targets"` [INFO] [stdout] fa0fb874c41b2c996570056cc9af71716e0138dd3e2d75079a7c51507b6be886 [INFO] running `"docker" "start" "-a" "fa0fb874c41b2c996570056cc9af71716e0138dd3e2d75079a7c51507b6be886"` [INFO] [stderr] Checking pbr v1.0.1 [INFO] [stderr] Checking chan v0.1.23 [INFO] [stderr] Checking nalgebra v0.14.4 [INFO] [stderr] Checking rtow v0.1.0 (/opt/crater/workdir) [INFO] [stderr] warning: needlessly taken reference of left operand [INFO] [stderr] --> src/ray.rs:33:9 [INFO] [stderr] | [INFO] [stderr] 33 | &self.origin + t * &self.direction [INFO] [stderr] | ------------^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: use the left value directly: `self.origin` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::op_ref)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#op_ref [INFO] [stderr] [INFO] [stderr] warning: taken reference of right operand [INFO] [stderr] --> src/ray.rs:33:24 [INFO] [stderr] | [INFO] [stderr] 33 | &self.origin + t * &self.direction [INFO] [stderr] | ^^^^--------------- [INFO] [stderr] | | [INFO] [stderr] | help: use the right value directly: `self.direction` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#op_ref [INFO] [stderr] [INFO] [stderr] warning: identical conversion [INFO] [stderr] --> src/sphere.rs:39:9 [INFO] [stderr] | [INFO] [stderr] 39 | / (0..) [INFO] [stderr] 40 | | .into_iter() [INFO] [stderr] | |________________________^ help: consider removing `.into_iter()`: `(0..)` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::identity_conversion)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion [INFO] [stderr] [INFO] [stderr] warning: useless use of `vec!` [INFO] [stderr] --> src/sphere.rs:69:26 [INFO] [stderr] | [INFO] [stderr] 69 | for t in vec![(-b - delta.sqrt()) / a, (-b + delta.sqrt()) / a] { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: you can use a slice directly: `&[(-b - delta.sqrt()) / a, (-b + delta.sqrt()) / a]` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::useless_vec)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec [INFO] [stderr] [INFO] [stderr] warning: using `clone` on a `Copy` type [INFO] [stderr] --> src/camera.rs:42:18 [INFO] [stderr] | [INFO] [stderr] 42 | Ray::new(self.origin.clone(), direction) [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `self.origin` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::clone_on_copy)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy [INFO] [stderr] [INFO] [stderr] warning: you don't need to add `&` to all patterns [INFO] [stderr] --> src/material.rs:47:9 [INFO] [stderr] | [INFO] [stderr] 47 | / match self { [INFO] [stderr] 48 | | &Material::Lambertian(attenuation) => { [INFO] [stderr] 49 | | let direction = hitpoint.normal + Sphere::random_point_in_unit_sphere(); [INFO] [stderr] 50 | | let scattered_ray = Ray::new(hitpoint.p, direction); [INFO] [stderr] ... | [INFO] [stderr] 97 | | } [INFO] [stderr] 98 | | } [INFO] [stderr] | |_________^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::match_ref_pats)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats [INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression [INFO] [stderr] | [INFO] [stderr] 47 | match *self { [INFO] [stderr] 48 | Material::Lambertian(attenuation) => { [INFO] [stderr] 49 | let direction = hitpoint.normal + Sphere::random_point_in_unit_sphere(); [INFO] [stderr] 50 | let scattered_ray = Ray::new(hitpoint.p, direction); [INFO] [stderr] 51 | Some((scattered_ray, attenuation.clone())) [INFO] [stderr] 52 | } [INFO] [stderr] ... [INFO] [stderr] [INFO] [stderr] warning: using `clone` on a `Copy` type [INFO] [stderr] --> src/material.rs:51:38 [INFO] [stderr] | [INFO] [stderr] 51 | Some((scattered_ray, attenuation.clone())) [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `attenuation` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy [INFO] [stderr] [INFO] [stderr] warning: using `clone` on a `Copy` type [INFO] [stderr] --> src/material.rs:64:69 [INFO] [stderr] | [INFO] [stderr] 64 | Some(Ordering::Greater) => Some((scattered_ray, attenuation.clone())), [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `attenuation` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy [INFO] [stderr] [INFO] [stderr] warning: `if _ { .. } else { .. }` is an expression [INFO] [stderr] --> src/material.rs:75:17 [INFO] [stderr] | [INFO] [stderr] 75 | / let mut cosine = [INFO] [stderr] 76 | | -ray.direction().dot(&hitpoint.normal) / vec_util::length(ray.direction()); [INFO] [stderr] 77 | | [INFO] [stderr] 78 | | if ray.direction().dot(&hitpoint.normal) > 0.0 { [INFO] [stderr] ... | [INFO] [stderr] 82 | | / vec_util::length(ray.direction()); [INFO] [stderr] 83 | | } [INFO] [stderr] | |_________________^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::useless_let_if_seq)] on by default [INFO] [stderr] = note: you might not need `mut` at all [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_let_if_seq [INFO] [stderr] help: it is more idiomatic to write [INFO] [stderr] | [INFO] [stderr] 75 | let cosine = if ray.direction().dot(&hitpoint.normal) > 0.0 { ..; refractive_index * ray.direction().dot(&hitpoint.normal) [INFO] [stderr] 76 | / vec_util::length(ray.direction()) } else { -ray.direction().dot(&hitpoint.normal) / vec_util::length(ray.direction()) }; [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: using `clone` on a `Copy` type [INFO] [stderr] --> src/vec_util.rs:6:5 [INFO] [stderr] | [INFO] [stderr] 6 | v.clone() / length(&v) [INFO] [stderr] | ^^^^^^^^^ help: try dereferencing it: `*v` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy [INFO] [stderr] [INFO] [stderr] warning: `ref` on an entire `let` pattern is discouraged, take a reference with `&` instead [INFO] [stderr] --> src/lib.rs:32:9 [INFO] [stderr] | [INFO] [stderr] 32 | let ref mut fout = File::create(&cfg.output_filename).unwrap(); [INFO] [stderr] | ----^^^^^^^^^^^^----------------------------------------------- help: try: `let fout = &mut File::create(&cfg.output_filename).unwrap();` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::toplevel_ref_arg)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#toplevel_ref_arg [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:85:24 [INFO] [stderr] | [INFO] [stderr] 85 | let aspect_ratio = cfg.resolution.width as f64 / cfg.resolution.height as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.resolution.width)` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::cast_lossless)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:85:54 [INFO] [stderr] | [INFO] [stderr] 85 | let aspect_ratio = cfg.resolution.width as f64 / cfg.resolution.height as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.resolution.height)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: identical conversion [INFO] [stderr] --> src/lib.rs:93:13 [INFO] [stderr] | [INFO] [stderr] 93 | / image::DynamicImage::new_rgb8(cfg.resolution.width, cfg.resolution.height) [INFO] [stderr] 94 | | .pixels() [INFO] [stderr] 95 | | .into_iter() [INFO] [stderr] | |____________________________^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion [INFO] [stderr] help: consider removing `.into_iter()` [INFO] [stderr] | [INFO] [stderr] 93 | image::DynamicImage::new_rgb8(cfg.resolution.width, cfg.resolution.height) [INFO] [stderr] 94 | .pixels() [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: identical conversion [INFO] [stderr] --> src/lib.rs:119:47 [INFO] [stderr] | [INFO] [stderr] 119 | let mut color: Vector3 = (0..cfg.n_samples) [INFO] [stderr] | _______________________________________________^ [INFO] [stderr] 120 | | .into_iter() [INFO] [stderr] | |________________________________^ help: consider removing `.into_iter()`: `(0..cfg.n_samples)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:123:34 [INFO] [stderr] | [INFO] [stderr] 123 | let u = (pixel.0 as f64 + rng.gen::()) / cfg.resolution.width as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^ help: try: `f64::from(pixel.0)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:123:71 [INFO] [stderr] | [INFO] [stderr] 123 | let u = (pixel.0 as f64 + rng.gen::()) / cfg.resolution.width as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.resolution.width)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:124:34 [INFO] [stderr] | [INFO] [stderr] 124 | let v = (pixel.1 as f64 + rng.gen::()) / cfg.resolution.height as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^ help: try: `f64::from(pixel.1)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:124:71 [INFO] [stderr] | [INFO] [stderr] 124 | let v = (pixel.1 as f64 + rng.gen::()) / cfg.resolution.height as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.resolution.height)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:130:26 [INFO] [stderr] | [INFO] [stderr] 130 | color /= cfg.n_samples as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.n_samples)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: needlessly taken reference of left operand [INFO] [stderr] --> src/ray.rs:33:9 [INFO] [stderr] | [INFO] [stderr] 33 | &self.origin + t * &self.direction [INFO] [stderr] | ------------^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: use the left value directly: `self.origin` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::op_ref)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#op_ref [INFO] [stderr] [INFO] [stderr] warning: taken reference of right operand [INFO] [stderr] --> src/ray.rs:33:24 [INFO] [stderr] | [INFO] [stderr] 33 | &self.origin + t * &self.direction [INFO] [stderr] | ^^^^--------------- [INFO] [stderr] | | [INFO] [stderr] | help: use the right value directly: `self.direction` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#op_ref [INFO] [stderr] [INFO] [stderr] warning: identical conversion [INFO] [stderr] --> src/sphere.rs:39:9 [INFO] [stderr] | [INFO] [stderr] 39 | / (0..) [INFO] [stderr] 40 | | .into_iter() [INFO] [stderr] | |________________________^ help: consider removing `.into_iter()`: `(0..)` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::identity_conversion)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion [INFO] [stderr] [INFO] [stderr] warning: useless use of `vec!` [INFO] [stderr] --> src/sphere.rs:69:26 [INFO] [stderr] | [INFO] [stderr] 69 | for t in vec![(-b - delta.sqrt()) / a, (-b + delta.sqrt()) / a] { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: you can use a slice directly: `&[(-b - delta.sqrt()) / a, (-b + delta.sqrt()) / a]` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::useless_vec)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec [INFO] [stderr] [INFO] [stderr] warning: using `clone` on a `Copy` type [INFO] [stderr] --> src/camera.rs:42:18 [INFO] [stderr] | [INFO] [stderr] 42 | Ray::new(self.origin.clone(), direction) [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `self.origin` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::clone_on_copy)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy [INFO] [stderr] [INFO] [stderr] warning: you don't need to add `&` to all patterns [INFO] [stderr] --> src/material.rs:47:9 [INFO] [stderr] | [INFO] [stderr] 47 | / match self { [INFO] [stderr] 48 | | &Material::Lambertian(attenuation) => { [INFO] [stderr] 49 | | let direction = hitpoint.normal + Sphere::random_point_in_unit_sphere(); [INFO] [stderr] 50 | | let scattered_ray = Ray::new(hitpoint.p, direction); [INFO] [stderr] ... | [INFO] [stderr] 97 | | } [INFO] [stderr] 98 | | } [INFO] [stderr] | |_________^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::match_ref_pats)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_ref_pats [INFO] [stderr] help: instead of prefixing all patterns with `&`, you can dereference the expression [INFO] [stderr] | [INFO] [stderr] 47 | match *self { [INFO] [stderr] 48 | Material::Lambertian(attenuation) => { [INFO] [stderr] 49 | let direction = hitpoint.normal + Sphere::random_point_in_unit_sphere(); [INFO] [stderr] 50 | let scattered_ray = Ray::new(hitpoint.p, direction); [INFO] [stderr] 51 | Some((scattered_ray, attenuation.clone())) [INFO] [stderr] 52 | } [INFO] [stderr] ... [INFO] [stderr] [INFO] [stderr] warning: using `clone` on a `Copy` type [INFO] [stderr] --> src/material.rs:51:38 [INFO] [stderr] | [INFO] [stderr] 51 | Some((scattered_ray, attenuation.clone())) [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `attenuation` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy [INFO] [stderr] [INFO] [stderr] warning: using `clone` on a `Copy` type [INFO] [stderr] --> src/material.rs:64:69 [INFO] [stderr] | [INFO] [stderr] 64 | Some(Ordering::Greater) => Some((scattered_ray, attenuation.clone())), [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `attenuation` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy [INFO] [stderr] [INFO] [stderr] warning: `if _ { .. } else { .. }` is an expression [INFO] [stderr] --> src/material.rs:75:17 [INFO] [stderr] | [INFO] [stderr] 75 | / let mut cosine = [INFO] [stderr] 76 | | -ray.direction().dot(&hitpoint.normal) / vec_util::length(ray.direction()); [INFO] [stderr] 77 | | [INFO] [stderr] 78 | | if ray.direction().dot(&hitpoint.normal) > 0.0 { [INFO] [stderr] ... | [INFO] [stderr] 82 | | / vec_util::length(ray.direction()); [INFO] [stderr] 83 | | } [INFO] [stderr] | |_________________^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::useless_let_if_seq)] on by default [INFO] [stderr] = note: you might not need `mut` at all [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_let_if_seq [INFO] [stderr] help: it is more idiomatic to write [INFO] [stderr] | [INFO] [stderr] 75 | let cosine = if ray.direction().dot(&hitpoint.normal) > 0.0 { ..; refractive_index * ray.direction().dot(&hitpoint.normal) [INFO] [stderr] 76 | / vec_util::length(ray.direction()) } else { -ray.direction().dot(&hitpoint.normal) / vec_util::length(ray.direction()) }; [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: using `clone` on a `Copy` type [INFO] [stderr] --> src/vec_util.rs:6:5 [INFO] [stderr] | [INFO] [stderr] 6 | v.clone() / length(&v) [INFO] [stderr] | ^^^^^^^^^ help: try dereferencing it: `*v` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy [INFO] [stderr] [INFO] [stderr] warning: `ref` on an entire `let` pattern is discouraged, take a reference with `&` instead [INFO] [stderr] --> src/lib.rs:32:9 [INFO] [stderr] | [INFO] [stderr] 32 | let ref mut fout = File::create(&cfg.output_filename).unwrap(); [INFO] [stderr] | ----^^^^^^^^^^^^----------------------------------------------- help: try: `let fout = &mut File::create(&cfg.output_filename).unwrap();` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::toplevel_ref_arg)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#toplevel_ref_arg [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:85:24 [INFO] [stderr] | [INFO] [stderr] 85 | let aspect_ratio = cfg.resolution.width as f64 / cfg.resolution.height as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.resolution.width)` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::cast_lossless)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:85:54 [INFO] [stderr] | [INFO] [stderr] 85 | let aspect_ratio = cfg.resolution.width as f64 / cfg.resolution.height as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.resolution.height)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: identical conversion [INFO] [stderr] --> src/lib.rs:93:13 [INFO] [stderr] | [INFO] [stderr] 93 | / image::DynamicImage::new_rgb8(cfg.resolution.width, cfg.resolution.height) [INFO] [stderr] 94 | | .pixels() [INFO] [stderr] 95 | | .into_iter() [INFO] [stderr] | |____________________________^ [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion [INFO] [stderr] help: consider removing `.into_iter()` [INFO] [stderr] | [INFO] [stderr] 93 | image::DynamicImage::new_rgb8(cfg.resolution.width, cfg.resolution.height) [INFO] [stderr] 94 | .pixels() [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: identical conversion [INFO] [stderr] --> src/lib.rs:119:47 [INFO] [stderr] | [INFO] [stderr] 119 | let mut color: Vector3 = (0..cfg.n_samples) [INFO] [stderr] | _______________________________________________^ [INFO] [stderr] 120 | | .into_iter() [INFO] [stderr] | |________________________________^ help: consider removing `.into_iter()`: `(0..cfg.n_samples)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:123:34 [INFO] [stderr] | [INFO] [stderr] 123 | let u = (pixel.0 as f64 + rng.gen::()) / cfg.resolution.width as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^ help: try: `f64::from(pixel.0)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:123:71 [INFO] [stderr] | [INFO] [stderr] 123 | let u = (pixel.0 as f64 + rng.gen::()) / cfg.resolution.width as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.resolution.width)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:124:34 [INFO] [stderr] | [INFO] [stderr] 124 | let v = (pixel.1 as f64 + rng.gen::()) / cfg.resolution.height as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^ help: try: `f64::from(pixel.1)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:124:71 [INFO] [stderr] | [INFO] [stderr] 124 | let v = (pixel.1 as f64 + rng.gen::()) / cfg.resolution.height as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.resolution.height)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] warning: casting u32 to f64 may become silently lossy if types change [INFO] [stderr] --> src/lib.rs:130:26 [INFO] [stderr] | [INFO] [stderr] 130 | color /= cfg.n_samples as f64; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^ help: try: `f64::from(cfg.n_samples)` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless [INFO] [stderr] [INFO] [stderr] Finished dev [unoptimized + debuginfo] target(s) in 51.10s [INFO] running `"docker" "inspect" "fa0fb874c41b2c996570056cc9af71716e0138dd3e2d75079a7c51507b6be886"` [INFO] running `"docker" "rm" "-f" "fa0fb874c41b2c996570056cc9af71716e0138dd3e2d75079a7c51507b6be886"` [INFO] [stdout] fa0fb874c41b2c996570056cc9af71716e0138dd3e2d75079a7c51507b6be886