[INFO] cloning repository https://github.com/OleksiiLystopadov/simple-renderer-rs [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/OleksiiLystopadov/simple-renderer-rs" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FOleksiiLystopadov%2Fsimple-renderer-rs", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FOleksiiLystopadov%2Fsimple-renderer-rs'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] bae076e3e0c880ef5ded6b3ba48bf0032ec66078 [INFO] checking OleksiiLystopadov/simple-renderer-rs against try#39605810efb1a0d52f955bfba9e4eec7a01d5deb for pr-153692 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FOleksiiLystopadov%2Fsimple-renderer-rs" "/workspace/builds/worker-7-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc2/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/OleksiiLystopadov/simple-renderer-rs [INFO] finished tweaking git repo https://github.com/OleksiiLystopadov/simple-renderer-rs [INFO] tweaked toml for git repo https://github.com/OleksiiLystopadov/simple-renderer-rs written to /workspace/builds/worker-7-tc2/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/OleksiiLystopadov/simple-renderer-rs on toolchain 39605810efb1a0d52f955bfba9e4eec7a01d5deb [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+39605810efb1a0d52f955bfba9e4eec7a01d5deb" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/OleksiiLystopadov/simple-renderer-rs 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" "+39605810efb1a0d52f955bfba9e4eec7a01d5deb" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+39605810efb1a0d52f955bfba9e4eec7a01d5deb" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 3f1b3d47e83c6e02e14446b570b267345531f5a3dc75bf51173dfc61ef03f02f [INFO] running `Command { std: "docker" "start" "-a" "3f1b3d47e83c6e02e14446b570b267345531f5a3dc75bf51173dfc61ef03f02f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "3f1b3d47e83c6e02e14446b570b267345531f5a3dc75bf51173dfc61ef03f02f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "3f1b3d47e83c6e02e14446b570b267345531f5a3dc75bf51173dfc61ef03f02f", kill_on_drop: false }` [INFO] [stdout] 3f1b3d47e83c6e02e14446b570b267345531f5a3dc75bf51173dfc61ef03f02f [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+39605810efb1a0d52f955bfba9e4eec7a01d5deb" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] d0d8a7ef04290383aa121909f9adba6efa5e8f57a8d3667be2bba389888019f7 [INFO] running `Command { std: "docker" "start" "-a" "d0d8a7ef04290383aa121909f9adba6efa5e8f57a8d3667be2bba389888019f7", kill_on_drop: false }` [INFO] [stderr] Checking simple-renderer v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `std::fs::File` [INFO] [stdout] --> src/main.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::fs::File; [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: `std::io::prelude::*` [INFO] [stdout] --> src/main.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::io::prelude::*; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::io::BufReader` [INFO] [stdout] --> src/main.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use std::io::BufReader; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::error::Error` [INFO] [stdout] --> src/main.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use std::error::Error; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::str::FromStr` [INFO] [stdout] --> src/main.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::str::FromStr; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::path::Path` [INFO] [stdout] --> src/main.rs:8:5 [INFO] [stdout] | [INFO] [stdout] 8 | use std::path::Path; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: `self` parameter type does not contain `Self` [INFO] [stdout] --> src/tga_writer.rs:52:30 [INFO] [stdout] | [INFO] [stdout] 52 | pub fn apply_gamma(self: &mut Image, gamma: f32) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: use `&self`, `&mut self`, or `self: &Self` for correct lifetime elision [INFO] [stdout] = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! [INFO] [stdout] = note: for more information, see issue #140611 [INFO] [stdout] = note: `#[deny(self_lifetime_elision_not_applicable)]` (part of `#[deny(future_incompatible)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: `self` parameter type does not contain `Self` [INFO] [stdout] --> src/tga_writer.rs:64:28 [INFO] [stdout] | [INFO] [stdout] 64 | pub fn set_pixel(self: &mut Image, x: i32, y: i32, c: Color) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: use `&self`, `&mut self`, or `self: &Self` for correct lifetime elision [INFO] [stdout] = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! [INFO] [stdout] = note: for more information, see issue #140611 [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: `self` parameter type does not contain `Self` [INFO] [stdout] --> src/tga_writer.rs:73:31 [INFO] [stdout] | [INFO] [stdout] 73 | pub fn write_to_tga(self: &Image, filename: &str) -> io::Result<()> { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: use `&self`, `&mut self`, or `self: &Self` for correct lifetime elision [INFO] [stdout] = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! [INFO] [stdout] = note: for more information, see issue #140611 [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated macro `r#try`: use the `?` operator instead [INFO] [stdout] --> src/tga_writer.rs:98:21 [INFO] [stdout] | [INFO] [stdout] 98 | let mut f = r#try!(File::create(filename)); [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(deprecated)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated macro `r#try`: use the `?` operator instead [INFO] [stdout] --> src/tga_writer.rs:100:13 [INFO] [stdout] | [INFO] [stdout] 100 | r#try!(f.write_all(struct_to_u8_slice(&h))); [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated macro `r#try`: use the `?` operator instead [INFO] [stdout] --> src/tga_writer.rs:101:13 [INFO] [stdout] | [INFO] [stdout] 101 | r#try!(f.write_all(slice_to_u8_slice(&self.data[..]))); [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/wavefront_parser.rs:34:12 [INFO] [stdout] | [INFO] [stdout] 34 | if (data_str.starts_with("#")) [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 (data_str.starts_with("#")) [INFO] [stdout] 34 + if data_str.starts_with("#") [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/wavefront_parser.rs:37:15 [INFO] [stdout] | [INFO] [stdout] 37 | if(data_str.starts_with("v ")) [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 37 - if(data_str.starts_with("v ")) [INFO] [stdout] 37 + if data_str.starts_with("v ") [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/wavefront_parser.rs:58:20 [INFO] [stdout] | [INFO] [stdout] 58 | else if(data_str.starts_with("f ")) [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 58 - else if(data_str.starts_with("f ")) [INFO] [stdout] 58 + else if data_str.starts_with("f ") [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `wavefront_parser::read` [INFO] [stdout] --> src/main.rs:15:5 [INFO] [stdout] | [INFO] [stdout] 15 | use wavefront_parser::read; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around assigned value [INFO] [stdout] --> src/main.rs:59:30 [INFO] [stdout] | [INFO] [stdout] 59 | let segment_height = (if is_second_half { t2.y - t1.y } else { t1.y - t0.y }) ; [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 59 - let segment_height = (if is_second_half { t2.y - t1.y } else { t1.y - t0.y }) ; [INFO] [stdout] 59 + let segment_height = if is_second_half { t2.y - t1.y } else { t1.y - t0.y } ; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::fs::File` [INFO] [stdout] --> src/main.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::fs::File; [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: `std::io::prelude::*` [INFO] [stdout] --> src/main.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::io::prelude::*; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::io::BufReader` [INFO] [stdout] --> src/main.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use std::io::BufReader; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::error::Error` [INFO] [stdout] --> src/main.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use std::error::Error; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::str::FromStr` [INFO] [stdout] --> src/main.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::str::FromStr; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::path::Path` [INFO] [stdout] --> src/main.rs:8:5 [INFO] [stdout] | [INFO] [stdout] 8 | use std::path::Path; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: `self` parameter type does not contain `Self` [INFO] [stdout] --> src/tga_writer.rs:52:30 [INFO] [stdout] | [INFO] [stdout] 52 | pub fn apply_gamma(self: &mut Image, gamma: f32) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: use `&self`, `&mut self`, or `self: &Self` for correct lifetime elision [INFO] [stdout] = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! [INFO] [stdout] = note: for more information, see issue #140611 [INFO] [stdout] = note: `#[deny(self_lifetime_elision_not_applicable)]` (part of `#[deny(future_incompatible)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: `self` parameter type does not contain `Self` [INFO] [stdout] --> src/tga_writer.rs:64:28 [INFO] [stdout] | [INFO] [stdout] 64 | pub fn set_pixel(self: &mut Image, x: i32, y: i32, c: Color) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: use `&self`, `&mut self`, or `self: &Self` for correct lifetime elision [INFO] [stdout] = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! [INFO] [stdout] = note: for more information, see issue #140611 [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: `self` parameter type does not contain `Self` [INFO] [stdout] --> src/tga_writer.rs:73:31 [INFO] [stdout] | [INFO] [stdout] 73 | pub fn write_to_tga(self: &Image, filename: &str) -> io::Result<()> { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: use `&self`, `&mut self`, or `self: &Self` for correct lifetime elision [INFO] [stdout] = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! [INFO] [stdout] = note: for more information, see issue #140611 [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated macro `r#try`: use the `?` operator instead [INFO] [stdout] --> src/tga_writer.rs:98:21 [INFO] [stdout] | [INFO] [stdout] 98 | let mut f = r#try!(File::create(filename)); [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(deprecated)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated macro `r#try`: use the `?` operator instead [INFO] [stdout] --> src/tga_writer.rs:100:13 [INFO] [stdout] | [INFO] [stdout] 100 | r#try!(f.write_all(struct_to_u8_slice(&h))); [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated macro `r#try`: use the `?` operator instead [INFO] [stdout] --> src/tga_writer.rs:101:13 [INFO] [stdout] | [INFO] [stdout] 101 | r#try!(f.write_all(slice_to_u8_slice(&self.data[..]))); [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/wavefront_parser.rs:34:12 [INFO] [stdout] | [INFO] [stdout] 34 | if (data_str.starts_with("#")) [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 (data_str.starts_with("#")) [INFO] [stdout] 34 + if data_str.starts_with("#") [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/wavefront_parser.rs:37:15 [INFO] [stdout] | [INFO] [stdout] 37 | if(data_str.starts_with("v ")) [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 37 - if(data_str.starts_with("v ")) [INFO] [stdout] 37 + if data_str.starts_with("v ") [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around `if` condition [INFO] [stdout] --> src/wavefront_parser.rs:58:20 [INFO] [stdout] | [INFO] [stdout] 58 | else if(data_str.starts_with("f ")) [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 58 - else if(data_str.starts_with("f ")) [INFO] [stdout] 58 + else if data_str.starts_with("f ") [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `wavefront_parser::read` [INFO] [stdout] --> src/main.rs:15:5 [INFO] [stdout] | [INFO] [stdout] 15 | use wavefront_parser::read; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around assigned value [INFO] [stdout] --> src/main.rs:59:30 [INFO] [stdout] | [INFO] [stdout] 59 | let segment_height = (if is_second_half { t2.y - t1.y } else { t1.y - t0.y }) ; [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 59 - let segment_height = (if is_second_half { t2.y - t1.y } else { t1.y - t0.y }) ; [INFO] [stdout] 59 + let segment_height = if is_second_half { t2.y - t1.y } else { t1.y - t0.y } ; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/wavefront_parser.rs:19:9 [INFO] [stdout] | [INFO] [stdout] 19 | let mut lines = reader.lines(); [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: unused variable: `Error` [INFO] [stdout] --> src/wavefront_parser.rs:29:13 [INFO] [stdout] | [INFO] [stdout] 29 | Error => continue [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_Error` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/wavefront_parser.rs:19:9 [INFO] [stdout] | [INFO] [stdout] 19 | let mut lines = reader.lines(); [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: unused variable: `Error` [INFO] [stdout] --> src/wavefront_parser.rs:29:13 [INFO] [stdout] | [INFO] [stdout] 29 | Error => continue [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_Error` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `draw_line` is never used [INFO] [stdout] --> src/main.rs:23:4 [INFO] [stdout] | [INFO] [stdout] 23 | fn draw_line(mut x0: f32, mut y0: f32, mut x1: f32, mut y1: f32, color: Color, image: &mut Image) { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0`, `1`, and `2` are never read [INFO] [stdout] --> src/tga_writer.rs:17:18 [INFO] [stdout] | [INFO] [stdout] 17 | pub struct Color(u8, u8, u8); [INFO] [stdout] | ----- ^^ ^^ ^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this struct [INFO] [stdout] | [INFO] [stdout] = help: consider removing these fields [INFO] [stdout] = note: `Color` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `apply_gamma` is never used [INFO] [stdout] --> src/tga_writer.rs:52:12 [INFO] [stdout] | [INFO] [stdout] 40 | impl Image { [INFO] [stdout] | ---------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 52 | pub fn apply_gamma(self: &mut Image, gamma: f32) { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `draw_line` is never used [INFO] [stdout] --> src/main.rs:23:4 [INFO] [stdout] | [INFO] [stdout] 23 | fn draw_line(mut x0: f32, mut y0: f32, mut x1: f32, mut y1: f32, color: Color, image: &mut Image) { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0`, `1`, and `2` are never read [INFO] [stdout] --> src/tga_writer.rs:17:18 [INFO] [stdout] | [INFO] [stdout] 17 | pub struct Color(u8, u8, u8); [INFO] [stdout] | ----- ^^ ^^ ^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this struct [INFO] [stdout] | [INFO] [stdout] = help: consider removing these fields [INFO] [stdout] = note: `Color` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `apply_gamma` is never used [INFO] [stdout] --> src/tga_writer.rs:52:12 [INFO] [stdout] | [INFO] [stdout] 40 | impl Image { [INFO] [stdout] | ---------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 52 | pub fn apply_gamma(self: &mut Image, gamma: f32) { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/tga_writer.rs:68:17 [INFO] [stdout] | [INFO] [stdout] 68 | std::mem::replace(pixel, c); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 68 | let _ = std::mem::replace(pixel, c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `lineResult` should have a snake case name [INFO] [stdout] --> src/wavefront_parser.rs:21:9 [INFO] [stdout] | [INFO] [stdout] 21 | for lineResult in lines { [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `line_result` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `Error` should have a snake case name [INFO] [stdout] --> src/wavefront_parser.rs:29:13 [INFO] [stdout] | [INFO] [stdout] 29 | Error => continue [INFO] [stdout] | ^^^^^ help: convert the identifier to snake case: `error` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `dataStr` should have a snake case name [INFO] [stdout] --> src/wavefront_parser.rs:60:25 [INFO] [stdout] | [INFO] [stdout] 60 | let mut dataStr = data_str.split_whitespace(); [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `data_str` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used [INFO] [stdout] --> src/tga_writer.rs:68:17 [INFO] [stdout] | [INFO] [stdout] 68 | std::mem::replace(pixel, c); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: if you don't need the old value, you can just assign the new value directly [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 68 | let _ = std::mem::replace(pixel, c); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `lineResult` should have a snake case name [INFO] [stdout] --> src/wavefront_parser.rs:21:9 [INFO] [stdout] | [INFO] [stdout] 21 | for lineResult in lines { [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `line_result` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `Error` should have a snake case name [INFO] [stdout] --> src/wavefront_parser.rs:29:13 [INFO] [stdout] | [INFO] [stdout] 29 | Error => continue [INFO] [stdout] | ^^^^^ help: convert the identifier to snake case: `error` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `dataStr` should have a snake case name [INFO] [stdout] --> src/wavefront_parser.rs:60:25 [INFO] [stdout] | [INFO] [stdout] 60 | let mut dataStr = data_str.split_whitespace(); [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `data_str` [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `simple-renderer` (bin "simple-renderer" test) due to 3 previous errors; 23 warnings emitted [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stderr] error: could not compile `simple-renderer` (bin "simple-renderer") due to 3 previous errors; 23 warnings emitted [INFO] running `Command { std: "docker" "inspect" "d0d8a7ef04290383aa121909f9adba6efa5e8f57a8d3667be2bba389888019f7", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "d0d8a7ef04290383aa121909f9adba6efa5e8f57a8d3667be2bba389888019f7", kill_on_drop: false }` [INFO] [stdout] d0d8a7ef04290383aa121909f9adba6efa5e8f57a8d3667be2bba389888019f7