[INFO] cloning repository https://github.com/lilyquu/gltf_decimation [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/lilyquu/gltf_decimation" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Flilyquu%2Fgltf_decimation", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Flilyquu%2Fgltf_decimation'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] b74c352786e8aaba5657dfbd0923d2b536e6177f [INFO] linting lilyquu/gltf_decimation against nightly for clippy-nonminimal_bool-denied [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Flilyquu%2Fgltf_decimation" "/workspace/builds/worker-1-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-1-tc1/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/lilyquu/gltf_decimation [INFO] finished tweaking git repo https://github.com/lilyquu/gltf_decimation [INFO] tweaked toml for git repo https://github.com/lilyquu/gltf_decimation written to /workspace/builds/worker-1-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/lilyquu/gltf_decimation on toolchain nightly [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+nightly" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/lilyquu/gltf_decimation 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" "+nightly" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] [stderr] Updating crates.io index [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded simba v0.8.1 [INFO] [stderr] Downloaded itoa v1.0.10 [INFO] [stderr] Downloaded clap_lex v0.7.0 [INFO] [stderr] Downloaded anstyle-query v1.0.2 [INFO] [stderr] Downloaded utf8parse v0.2.1 [INFO] [stderr] Downloaded windows-targets v0.52.4 [INFO] [stderr] Downloaded anstyle-wincon v3.0.2 [INFO] [stderr] Downloaded nalgebra-macros v0.2.1 [INFO] [stderr] Downloaded colorchoice v1.0.0 [INFO] [stderr] Downloaded anstyle v1.0.6 [INFO] [stderr] Downloaded strsim v0.11.0 [INFO] [stderr] Downloaded autocfg v1.2.0 [INFO] [stderr] Downloaded clap_derive v4.5.3 [INFO] [stderr] Downloaded anstream v0.6.13 [INFO] [stderr] Downloaded anstyle-parse v0.2.3 [INFO] [stderr] Downloaded num-rational v0.4.1 [INFO] [stderr] Downloaded wide v0.7.15 [INFO] [stderr] Downloaded num-complex v0.4.5 [INFO] [stderr] Downloaded bytemuck v1.15.0 [INFO] [stderr] Downloaded matrixmultiply v0.3.8 [INFO] [stderr] Downloaded clap v4.5.3 [INFO] [stderr] Downloaded num-traits v0.2.18 [INFO] [stderr] Downloaded safe_arch v0.7.1 [INFO] [stderr] Downloaded indexmap v2.2.6 [INFO] [stderr] Downloaded json v0.12.4 [INFO] [stderr] Downloaded hashbrown v0.14.3 [INFO] [stderr] Downloaded serde_json v1.0.114 [INFO] [stderr] Downloaded clap_builder v4.5.2 [INFO] [stderr] Downloaded syn v2.0.53 [INFO] [stderr] Downloaded nalgebra v0.32.5 [INFO] [stderr] Downloaded windows_aarch64_gnullvm v0.52.4 [INFO] [stderr] Downloaded windows_x86_64_gnullvm v0.52.4 [INFO] [stderr] Downloaded windows_x86_64_msvc v0.52.4 [INFO] [stderr] Downloaded windows_aarch64_msvc v0.52.4 [INFO] [stderr] Downloaded windows_x86_64_gnu v0.52.4 [INFO] [stderr] Downloaded windows_i686_gnu v0.52.4 [INFO] [stderr] Downloaded windows_i686_msvc v0.52.4 [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:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 00d9777f18793a6f8c8e778fcf0f701ba5ea3b04527a14ba57256f944ea7b2a9 [INFO] running `Command { std: "docker" "start" "-a" "00d9777f18793a6f8c8e778fcf0f701ba5ea3b04527a14ba57256f944ea7b2a9", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "00d9777f18793a6f8c8e778fcf0f701ba5ea3b04527a14ba57256f944ea7b2a9", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "00d9777f18793a6f8c8e778fcf0f701ba5ea3b04527a14ba57256f944ea7b2a9", kill_on_drop: false }` [INFO] [stdout] 00d9777f18793a6f8c8e778fcf0f701ba5ea3b04527a14ba57256f944ea7b2a9 [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:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "clippy" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] b05db8d48d00e4d485c81d11dfc5befc958d5ab33799a0655a622cf2e789dabc [INFO] running `Command { std: "docker" "start" "-a" "b05db8d48d00e4d485c81d11dfc5befc958d5ab33799a0655a622cf2e789dabc", kill_on_drop: false }` [INFO] [stderr] Compiling autocfg v1.2.0 [INFO] [stderr] Compiling proc-macro2 v1.0.79 [INFO] [stderr] Compiling unicode-ident v1.0.12 [INFO] [stderr] Checking utf8parse v0.2.1 [INFO] [stderr] Compiling serde v1.0.197 [INFO] [stderr] Checking bytemuck v1.15.0 [INFO] [stderr] Compiling paste v1.0.14 [INFO] [stderr] Checking anstyle-query v1.0.2 [INFO] [stderr] Checking anstyle-parse v0.2.3 [INFO] [stderr] Checking colorchoice v1.0.0 [INFO] [stderr] Compiling typenum v1.17.0 [INFO] [stderr] Checking anstyle v1.0.6 [INFO] [stderr] Checking strsim v0.11.0 [INFO] [stderr] Checking safe_arch v0.7.1 [INFO] [stderr] Checking equivalent v1.0.1 [INFO] [stderr] Checking anstream v0.6.13 [INFO] [stderr] Compiling num-traits v0.2.18 [INFO] [stderr] Compiling matrixmultiply v0.3.8 [INFO] [stderr] Compiling num-rational v0.4.1 [INFO] [stderr] Compiling serde_json v1.0.114 [INFO] [stderr] Compiling quote v1.0.35 [INFO] [stderr] Checking wide v0.7.15 [INFO] [stderr] Compiling heck v0.5.0 [INFO] [stderr] Checking rawpointer v0.2.1 [INFO] [stderr] Checking clap_lex v0.7.0 [INFO] [stderr] Checking hashbrown v0.14.3 [INFO] [stderr] Checking clap_builder v4.5.2 [INFO] [stderr] Compiling syn v1.0.109 [INFO] [stderr] Compiling syn v2.0.53 [INFO] [stderr] Checking ryu v1.0.17 [INFO] [stderr] Checking itoa v1.0.10 [INFO] [stderr] Checking num-complex v0.4.5 [INFO] [stderr] Checking approx v0.5.1 [INFO] [stderr] Checking num-integer v0.1.46 [INFO] [stderr] Checking json v0.12.4 [INFO] [stderr] Checking simba v0.8.1 [INFO] [stderr] Checking indexmap v2.2.6 [INFO] [stderr] Compiling clap_derive v4.5.3 [INFO] [stderr] Compiling nalgebra-macros v0.2.1 [INFO] [stderr] Checking clap v4.5.3 [INFO] [stderr] Checking nalgebra v0.32.5 [INFO] [stderr] Checking decimation_gltf v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `prelude::*` [INFO] [stdout] --> src/main.rs:4:34 [INFO] [stdout] | [INFO] [stdout] 4 | use std::io::{self, Write, Read, prelude::*}; [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: `Number` [INFO] [stdout] --> src/main.rs:5:31 [INFO] [stdout] | [INFO] [stdout] 5 | use serde_json::{json, Value, Number, to_vec, Map}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Hash` and `Hasher` [INFO] [stdout] --> src/main.rs:6:17 [INFO] [stdout] | [INFO] [stdout] 6 | use std::hash::{Hash, Hasher}; [INFO] [stdout] | ^^^^ ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Matrix1x4` and `Unit` [INFO] [stdout] --> src/main.rs:7:16 [INFO] [stdout] | [INFO] [stdout] 7 | use nalgebra::{Matrix1x4, Matrix4, Matrix4x1, Unit, Vector2, Vector3, Vector4}; [INFO] [stdout] | ^^^^^^^^^ ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `json::JsonValue` [INFO] [stdout] --> src/main.rs:11:5 [INFO] [stdout] | [INFO] [stdout] 11 | use json::JsonValue; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:120:63 [INFO] [stdout] | [INFO] [stdout] 120 | let (mut index_list, mut normal_list, mut position_list, mut tangent_list, mut texcoord_0_list, mut json, mut primitives) = un... [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:120:102 [INFO] [stdout] | [INFO] [stdout] 120 | ...tion_list, mut tangent_list, mut texcoord_0_list, mut json, mut primitives) = unpack_gltf(path); [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:120:112 [INFO] [stdout] | [INFO] [stdout] 120 | ... mut tangent_list, mut texcoord_0_list, mut json, mut primitives) = unpack_gltf(path); [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:122:9 [INFO] [stdout] | [INFO] [stdout] 122 | let mut valid_edge = get_valid_edge(&index_list, &position_list, &vertex_list); [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:282:13 [INFO] [stdout] | [INFO] [stdout] 282 | let mut new_p = Vector3::new((p1[0]+p2[0])/2.0, (p1[1]+p2[1])/2.0, (p1[2]+p2[2])/2.0); [INFO] [stdout] | ----^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `i` [INFO] [stdout] --> src/main.rs:139:9 [INFO] [stdout] | [INFO] [stdout] 139 | for i in 0..position_list.len() { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_i` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `file` [INFO] [stdout] --> src/main.rs:399:9 [INFO] [stdout] | [INFO] [stdout] 399 | let file = repack_gltf(json, index_ref, index_list, &normal_list, &position_list, &texcoord_0_list, &primitives); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_file` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:410:29 [INFO] [stdout] | [INFO] [stdout] 410 | let index = if let Some(mut index) = json["meshes"][0]["primitives"][0]["indices"].as_i64(){ [INFO] [stdout] | ----^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `prelude::*` [INFO] [stdout] --> src/main.rs:4:34 [INFO] [stdout] | [INFO] [stdout] 4 | use std::io::{self, Write, Read, prelude::*}; [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: `Number` [INFO] [stdout] --> src/main.rs:5:31 [INFO] [stdout] | [INFO] [stdout] 5 | use serde_json::{json, Value, Number, to_vec, Map}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Hash` and `Hasher` [INFO] [stdout] --> src/main.rs:6:17 [INFO] [stdout] | [INFO] [stdout] 6 | use std::hash::{Hash, Hasher}; [INFO] [stdout] | ^^^^ ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Matrix1x4` and `Unit` [INFO] [stdout] --> src/main.rs:7:16 [INFO] [stdout] | [INFO] [stdout] 7 | use nalgebra::{Matrix1x4, Matrix4, Matrix4x1, Unit, Vector2, Vector3, Vector4}; [INFO] [stdout] | ^^^^^^^^^ ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `json::JsonValue` [INFO] [stdout] --> src/main.rs:11:5 [INFO] [stdout] | [INFO] [stdout] 11 | use json::JsonValue; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `tangent` [INFO] [stdout] --> src/main.rs:431:9 [INFO] [stdout] | [INFO] [stdout] 431 | let tangent = if let Some(tangent) = json["meshes"][0]["primitives"][0]["attributes"]["TANGENT"].as_i64(){ [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_tangent` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `cost` is never read [INFO] [stdout] --> src/main.rs:702:20 [INFO] [stdout] | [INFO] [stdout] 702 | let mut cost = f32::MAX; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `cost` is never read [INFO] [stdout] --> src/main.rs:712:13 [INFO] [stdout] | [INFO] [stdout] 712 | / cost = q[0]*v[0]*v[0] + [INFO] [stdout] 713 | | 2.0*q[1]*v[0]*v[1] + [INFO] [stdout] 714 | | 2.0*q[2]*v[0]*v[2] + [INFO] [stdout] 715 | | 2.0*q[3]*v[0] + [INFO] [stdout] ... | [INFO] [stdout] 719 | | q[7]*v[2]*v[2] + [INFO] [stdout] 720 | | 2.0*q[8]*v[2] + q[9]; [INFO] [stdout] | |________________________________________^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `cost` is never read [INFO] [stdout] --> src/main.rs:723:13 [INFO] [stdout] | [INFO] [stdout] 723 | / cost = q[0]*new_pos[0]*new_pos[0] + [INFO] [stdout] 724 | | 2.0*q[1]*new_pos[0]*new_pos[1] + [INFO] [stdout] 725 | | 2.0*q[2]*new_pos[0]*new_pos[2] + [INFO] [stdout] 726 | | 2.0*q[3]*new_pos[0] + [INFO] [stdout] ... | [INFO] [stdout] 730 | | q[7]*new_pos[2]*new_pos[2] + [INFO] [stdout] 731 | | 2.0*q[8]*new_pos[2] + q[9]; [INFO] [stdout] | |______________________________________________^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `normal_list` [INFO] [stdout] --> src/main.rs:746:38 [INFO] [stdout] | [INFO] [stdout] 746 | fn initialize(index_list:&Vec, normal_list:&HashMap>, [INFO] [stdout] | ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_normal_list` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `tangent_list` [INFO] [stdout] --> src/main.rs:748:14 [INFO] [stdout] | [INFO] [stdout] 748 | tangent_list:&HashMap>, [INFO] [stdout] | ^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_tangent_list` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `texcoord_0_list` [INFO] [stdout] --> src/main.rs:749:14 [INFO] [stdout] | [INFO] [stdout] 749 | texcoord_0_list:&HashMap>) -> (HashMap, u32) { [INFO] [stdout] | ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_texcoord_0_list` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:871:9 [INFO] [stdout] | [INFO] [stdout] 871 | let mut json:Value = serde_json::from_slice(json_chunk).expect("Failed to parse JSON"); [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:894:31 [INFO] [stdout] | [INFO] [stdout] 894 | let indices = if let Some(mut indices) = json["meshes"][0]["primitives"][0]["indices"].as_i64(){ [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:964:13 [INFO] [stdout] | [INFO] [stdout] 964 | let mut t = primitives.get("indices").unwrap().componentType; [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:965:13 [INFO] [stdout] | [INFO] [stdout] 965 | let mut num = primitives.get("indices").unwrap().count; [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `byteOffset`, `normalized`, and `prim_type` are never read [INFO] [stdout] --> src/main.rs:39:5 [INFO] [stdout] | [INFO] [stdout] 37 | struct Prim { [INFO] [stdout] | ---- fields in this struct [INFO] [stdout] 38 | bufferView: u32, [INFO] [stdout] 39 | byteOffset: u32, [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] 40 | componentType: u32, [INFO] [stdout] 41 | normalized: bool, [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] 42 | count: u32, [INFO] [stdout] 43 | prim_type: String, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Prim` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `buffer` and `byteLength` are never read [INFO] [stdout] --> src/main.rs:62:5 [INFO] [stdout] | [INFO] [stdout] 61 | struct View { [INFO] [stdout] | ---- fields in this struct [INFO] [stdout] 62 | buffer: u32, [INFO] [stdout] | ^^^^^^ [INFO] [stdout] 63 | byteLength: u32, [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `View` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `tri_normal` is never used [INFO] [stdout] --> src/main.rs:646:4 [INFO] [stdout] | [INFO] [stdout] 646 | fn tri_normal(v0: Vector3, v1: Vector3, v2: Vector3) -> Vector3 { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-canonical implementation of `partial_cmp` on an `Ord` type [INFO] [stdout] --> src/main.rs:24:1 [INFO] [stdout] | [INFO] [stdout] 24 | / impl PartialOrd for Remove { [INFO] [stdout] 25 | | fn partial_cmp(&self, other: &Self) -> Option { [INFO] [stdout] 26 | | self.2.partial_cmp(&other.2) [INFO] [stdout] 27 | | } [INFO] [stdout] 28 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_canonical_partial_ord_impl [INFO] [stdout] = note: `#[warn(clippy::non_canonical_partial_ord_impl)]` on by default [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 25 - fn partial_cmp(&self, other: &Self) -> Option { [INFO] [stdout] 26 - self.2.partial_cmp(&other.2) [INFO] [stdout] 27 - } [INFO] [stdout] 25 + fn partial_cmp(&self, other: &Self) -> Option { Some(self.cmp(other)) } [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler [INFO] [stdout] --> src/main.rs:112:42 [INFO] [stdout] | [INFO] [stdout] 112 | "gltf"| "glb" => decimation_gltf(&path, &method, limit), [INFO] [stdout] | ^^^^^ help: change this to: `path` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow [INFO] [stdout] = note: `#[warn(clippy::needless_borrow)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler [INFO] [stdout] --> src/main.rs:112:49 [INFO] [stdout] | [INFO] [stdout] 112 | "gltf"| "glb" => decimation_gltf(&path, &method, limit), [INFO] [stdout] | ^^^^^^^ help: change this to: `method` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this boolean expression can be simplified [INFO] [stdout] --> src/main.rs:314:16 [INFO] [stdout] | [INFO] [stdout] 314 | if (index_list[i*3] == remove.0 && index_list[i*3+1] == remove.0) || [INFO] [stdout] | ________________^ [INFO] [stdout] 315 | | (index_list[i*3+1] == remove.0 && index_list[i*3+2] == remove.0) || [INFO] [stdout] | |________________________________________________________________________________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stdout] = note: `#[warn(clippy::nonminimal_bool)]` on by default [INFO] [stdout] help: try [INFO] [stdout] | [INFO] [stdout] 314 - if (index_list[i*3] == remove.0 && index_list[i*3+1] == remove.0) || [INFO] [stdout] 315 - (index_list[i*3+1] == remove.0 && index_list[i*3+2] == remove.0) || [INFO] [stdout] 314 + if !(index_list[i*3+1] != remove.0 || index_list[i*3] != remove.0 && index_list[i*3+2] != remove.0) || [INFO] [stdout] | [INFO] [stdout] 314 - if (index_list[i*3] == remove.0 && index_list[i*3+1] == remove.0) || [INFO] [stdout] 315 - (index_list[i*3+1] == remove.0 && index_list[i*3+2] == remove.0) || [INFO] [stdout] 314 + if (index_list[i*3+2] == remove.0 || index_list[i*3] == remove.0) && index_list[i*3+1] == remove.0 || [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this call to `as_ref` does nothing [INFO] [stdout] --> src/main.rs:131:11 [INFO] [stdout] | [INFO] [stdout] 131 | match method.as_ref() { [INFO] [stdout] | ^^^^^^^^^^^^^^^ help: try: `method` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_asref [INFO] [stdout] = note: `#[warn(clippy::useless_asref)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `j` is used to index `index_ref` [INFO] [stdout] --> src/main.rs:145:18 [INFO] [stdout] | [INFO] [stdout] 145 | for j in i+1..position_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] = note: `#[warn(clippy::needless_range_loop)]` on by default [INFO] [stdout] help: consider using an iterator and enumerate() [INFO] [stdout] | [INFO] [stdout] 145 - for j in i+1..position_list.len() { [INFO] [stdout] 145 + for (j, ) in index_ref.iter_mut().enumerate().take(position_list.len()).skip(i+1) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `k` is only used to index `index_list` [INFO] [stdout] --> src/main.rs:152:26 [INFO] [stdout] | [INFO] [stdout] 152 | for k in 0..index_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 152 - for k in 0..index_list.len() { [INFO] [stdout] 152 + for in &mut index_list { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is only used to index `index_list` [INFO] [stdout] --> src/main.rs:308:18 [INFO] [stdout] | [INFO] [stdout] 308 | for i in 0..index_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 308 - for i in 0..index_list.len() { [INFO] [stdout] 308 + for in &mut index_list { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: implicitly performing saturating subtraction [INFO] [stdout] --> src/main.rs:317:17 [INFO] [stdout] | [INFO] [stdout] 317 | / if goal > 0 { [INFO] [stdout] 318 | | goal = goal-1; [INFO] [stdout] 319 | | } [INFO] [stdout] | |_________________^ help: try: `goal = goal.saturating_sub(1);` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#implicit_saturating_sub [INFO] [stdout] = note: `#[warn(clippy::implicit_saturating_sub)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:318:21 [INFO] [stdout] | [INFO] [stdout] 318 | goal = goal-1; [INFO] [stdout] | ^^^^^^^^^^^^^ help: replace it with: `goal -= 1` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] = note: `#[warn(clippy::assign_op_pattern)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manually filling a slice [INFO] [stdout] --> src/main.rs:459:5 [INFO] [stdout] | [INFO] [stdout] 459 | / for i in 0..index_ref.len() { [INFO] [stdout] 460 | | index_ref[i] = false; [INFO] [stdout] 461 | | } [INFO] [stdout] | |_____^ help: try: `index_ref.fill(false);` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_slice_fill [INFO] [stdout] = note: `#[warn(clippy::manual_slice_fill)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is only used to index `index_ref` [INFO] [stdout] --> src/main.rs:459:14 [INFO] [stdout] | [INFO] [stdout] 459 | for i in 0..index_ref.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 459 - for i in 0..index_ref.len() { [INFO] [stdout] 459 + for in &mut index_ref { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is used to index `index_ref` [INFO] [stdout] --> src/main.rs:466:14 [INFO] [stdout] | [INFO] [stdout] 466 | for i in 0..index_ref.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator and enumerate() [INFO] [stdout] | [INFO] [stdout] 466 - for i in 0..index_ref.len() { [INFO] [stdout] 466 + for (i, ) in index_ref.iter().enumerate() { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: equality checks against true are unnecessary [INFO] [stdout] --> src/main.rs:467:12 [INFO] [stdout] | [INFO] [stdout] 467 | if index_ref[i] == true { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: try: `index_ref[i]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bool_comparison [INFO] [stdout] = note: `#[warn(clippy::bool_comparison)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:470:13 [INFO] [stdout] | [INFO] [stdout] 470 | k = k+1; [INFO] [stdout] | ^^^^^^^ help: replace it with: `k += 1` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is only used to index `index_list` [INFO] [stdout] --> src/main.rs:475:14 [INFO] [stdout] | [INFO] [stdout] 475 | for i in 0..index_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 475 - for i in 0..index_list.len() { [INFO] [stdout] 475 + for in &index_list { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `j` is only used to index `exist` [INFO] [stdout] --> src/main.rs:478:18 [INFO] [stdout] | [INFO] [stdout] 478 | for j in 0..exist.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 478 - for j in 0..exist.len() { [INFO] [stdout] 478 + for in &exist { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: equality checks against false can be replaced by a negation [INFO] [stdout] --> src/main.rs:483:12 [INFO] [stdout] | [INFO] [stdout] 483 | if same == false { [INFO] [stdout] | ^^^^^^^^^^^^^ help: try: `!same` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bool_comparison [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:484:13 [INFO] [stdout] | [INFO] [stdout] 484 | index_number = index_number + 1; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `index_number += 1` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is only used to index `new_index_list` [INFO] [stdout] --> src/main.rs:496:22 [INFO] [stdout] | [INFO] [stdout] 496 | for i in 0..new_index_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 496 - for i in 0..new_index_list.len() { [INFO] [stdout] 496 + for in &new_index_list { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:571:13 [INFO] [stdout] | [INFO] [stdout] 571 | offset = offset + index_list.len()*2; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `offset += index_list.len()*2` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:574:13 [INFO] [stdout] | [INFO] [stdout] 574 | offset = offset + index_number*12; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `offset += index_number*12` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:577:13 [INFO] [stdout] | [INFO] [stdout] 577 | offset = offset + index_number*12; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `offset += index_number*12` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:580:13 [INFO] [stdout] | [INFO] [stdout] 580 | offset = offset + index_number*8; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `offset += index_number*8` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do [INFO] [stdout] --> src/main.rs:654:30 [INFO] [stdout] | [INFO] [stdout] 654 | fn get_valid_edge(index_list:&Vec, position_list:&HashMap>, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stdout] = note: `#[warn(clippy::ptr_arg)]` on by default [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 654 - fn get_valid_edge(index_list:&Vec, position_list:&HashMap>, [INFO] [stdout] 654 + fn get_valid_edge(index_list:&[u32], position_list:&HashMap>, [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do [INFO] [stdout] --> src/main.rs:691:19 [INFO] [stdout] | [INFO] [stdout] 691 | fn update_cost(q1:&Vec, q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 691 - fn update_cost(q1:&Vec, q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] 691 + fn update_cost(q1:&[f32], q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do [INFO] [stdout] --> src/main.rs:691:33 [INFO] [stdout] | [INFO] [stdout] 691 | fn update_cost(q1:&Vec, q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 691 - fn update_cost(q1:&Vec, q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] 691 + fn update_cost(q1:&Vec, q2:&[f32], new_pos:&Vector3) -> f32{ [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do [INFO] [stdout] --> src/main.rs:746:26 [INFO] [stdout] | [INFO] [stdout] 746 | fn initialize(index_list:&Vec, normal_list:&HashMap>, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 746 - fn initialize(index_list:&Vec, normal_list:&HashMap>, [INFO] [stdout] 746 + fn initialize(index_list:&[u32], normal_list:&HashMap>, [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:785:21 [INFO] [stdout] | [INFO] [stdout] 785 | q_matrix[x] = q_matrix[x] + k_list[j][x]; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `q_matrix[x] += k_list[j][x]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:800:21 [INFO] [stdout] | [INFO] [stdout] 800 | q_matrix[x] = q_matrix[x] + k_list[j][x]; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `q_matrix[x] += k_list[j][x]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:815:21 [INFO] [stdout] | [INFO] [stdout] 815 | q_matrix[x] = q_matrix[x] + k_list[j][x]; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `q_matrix[x] += k_list[j][x]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `push` immediately after creation [INFO] [stdout] --> src/main.rs:838:5 [INFO] [stdout] | [INFO] [stdout] 838 | / let mut k = Vec::with_capacity(10); [INFO] [stdout] ... | [INFO] [stdout] 852 | | k.push(c*d); [INFO] [stdout] 853 | | k.push(d*d); [INFO] [stdout] | |________________^ help: consider using the `vec![]` macro: `let k = vec![..];` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#vec_init_then_push [INFO] [stdout] = note: `#[warn(clippy::vec_init_then_push)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: very complex type used. Consider factoring parts into `type` definitions [INFO] [stdout] --> src/main.rs:857:31 [INFO] [stdout] | [INFO] [stdout] 857 | fn unpack_gltf(path:&Path) -> (Vec, HashMap>, HashMap>, [INFO] [stdout] | _______________________________^ [INFO] [stdout] 858 | | HashMap>, HashMap>, [INFO] [stdout] 859 | | Value, HashMap) { [INFO] [stdout] | |____________________________________________________________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity [INFO] [stdout] = note: `#[warn(clippy::type_complexity)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the variable `i` is used as a loop counter [INFO] [stdout] --> src/main.rs:881:9 [INFO] [stdout] | [INFO] [stdout] 881 | for buffer_view in buffer_views { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `for (i, buffer_view) in buffer_views.iter().enumerate()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_counter_loop [INFO] [stdout] = note: `#[warn(clippy::explicit_counter_loop)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:952:13 [INFO] [stdout] | [INFO] [stdout] 952 | i = i+1; [INFO] [stdout] | ^^^^^^^ help: replace it with: `i += 1` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `vec!` [INFO] [stdout] --> src/main.rs:698:17 [INFO] [stdout] | [INFO] [stdout] 698 | let mut q = vec![0.0; 10]; [INFO] [stdout] | ^^^^^^^^^^^^^ help: you can use an array directly: `[0.0; 10]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec [INFO] [stdout] = note: `#[warn(clippy::useless_vec)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `bufferView` should have a snake case name [INFO] [stdout] --> src/main.rs:38:5 [INFO] [stdout] | [INFO] [stdout] 38 | bufferView: u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `buffer_view` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:39:5 [INFO] [stdout] | [INFO] [stdout] 39 | byteOffset: u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `componentType` should have a snake case name [INFO] [stdout] --> src/main.rs:40:5 [INFO] [stdout] | [INFO] [stdout] 40 | componentType: u32, [INFO] [stdout] | ^^^^^^^^^^^^^ help: convert the identifier to snake case: `component_type` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `bufferView` should have a snake case name [INFO] [stdout] --> src/main.rs:47:12 [INFO] [stdout] | [INFO] [stdout] 47 | fn new(bufferView:u32, byteOffset:u32, componentType:u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `buffer_view` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:47:28 [INFO] [stdout] | [INFO] [stdout] 47 | fn new(bufferView:u32, byteOffset:u32, componentType:u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `componentType` should have a snake case name [INFO] [stdout] --> src/main.rs:47:44 [INFO] [stdout] | [INFO] [stdout] 47 | fn new(bufferView:u32, byteOffset:u32, componentType:u32, [INFO] [stdout] | ^^^^^^^^^^^^^ help: convert the identifier to snake case: `component_type` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `byteLength` should have a snake case name [INFO] [stdout] --> src/main.rs:63:5 [INFO] [stdout] | [INFO] [stdout] 63 | byteLength: u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_length` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:64:5 [INFO] [stdout] | [INFO] [stdout] 64 | byteOffset: u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `byteLength` should have a snake case name [INFO] [stdout] --> src/main.rs:68:24 [INFO] [stdout] | [INFO] [stdout] 68 | fn new(buffer:u32, byteLength:u32, byteOffset:u32) -> Self { [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_length` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:68:40 [INFO] [stdout] | [INFO] [stdout] 68 | fn new(buffer:u32, byteLength:u32, byteOffset:u32) -> Self { [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `NORMAL` should have a snake case name [INFO] [stdout] --> src/main.rs:447:9 [INFO] [stdout] | [INFO] [stdout] 447 | let NORMAL = primitives.get("NORMAL").unwrap(); [INFO] [stdout] | ^^^^^^ help: convert the identifier to snake case: `normal` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `POSITION` should have a snake case name [INFO] [stdout] --> src/main.rs:448:9 [INFO] [stdout] | [INFO] [stdout] 448 | let POSITION = primitives.get("POSITION").unwrap(); [INFO] [stdout] | ^^^^^^^^ help: convert the identifier to snake case: `position` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `TEXCOORD_0` should have a snake case name [INFO] [stdout] --> src/main.rs:449:9 [INFO] [stdout] | [INFO] [stdout] 449 | let TEXCOORD_0 = primitives.get("TEXCOORD_0").unwrap(); [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `texcoord_0` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `bufferView` should have a snake case name [INFO] [stdout] --> src/main.rs:934:17 [INFO] [stdout] | [INFO] [stdout] 934 | let bufferView = accessor["bufferView"].as_u64().unwrap_or(0) as u32; [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `buffer_view` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:935:17 [INFO] [stdout] | [INFO] [stdout] 935 | let byteOffset = accessor["byteOffset"].as_u64().unwrap_or(0) as u32; [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `componentType` should have a snake case name [INFO] [stdout] --> src/main.rs:936:17 [INFO] [stdout] | [INFO] [stdout] 936 | let componentType = accessor["componentType"].as_u64().unwrap() as u32; [INFO] [stdout] | ^^^^^^^^^^^^^ help: convert the identifier to snake case: `component_type` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:120:63 [INFO] [stdout] | [INFO] [stdout] 120 | let (mut index_list, mut normal_list, mut position_list, mut tangent_list, mut texcoord_0_list, mut json, mut primitives) = un... [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:120:102 [INFO] [stdout] | [INFO] [stdout] 120 | ...tion_list, mut tangent_list, mut texcoord_0_list, mut json, mut primitives) = unpack_gltf(path); [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:120:112 [INFO] [stdout] | [INFO] [stdout] 120 | ... mut tangent_list, mut texcoord_0_list, mut json, mut primitives) = unpack_gltf(path); [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:122:9 [INFO] [stdout] | [INFO] [stdout] 122 | let mut valid_edge = get_valid_edge(&index_list, &position_list, &vertex_list); [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:282:13 [INFO] [stdout] | [INFO] [stdout] 282 | let mut new_p = Vector3::new((p1[0]+p2[0])/2.0, (p1[1]+p2[1])/2.0, (p1[2]+p2[2])/2.0); [INFO] [stdout] | ----^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `i` [INFO] [stdout] --> src/main.rs:139:9 [INFO] [stdout] | [INFO] [stdout] 139 | for i in 0..position_list.len() { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_i` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `file` [INFO] [stdout] --> src/main.rs:399:9 [INFO] [stdout] | [INFO] [stdout] 399 | let file = repack_gltf(json, index_ref, index_list, &normal_list, &position_list, &texcoord_0_list, &primitives); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_file` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:410:29 [INFO] [stdout] | [INFO] [stdout] 410 | let index = if let Some(mut index) = json["meshes"][0]["primitives"][0]["indices"].as_i64(){ [INFO] [stdout] | ----^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `tangent` [INFO] [stdout] --> src/main.rs:431:9 [INFO] [stdout] | [INFO] [stdout] 431 | let tangent = if let Some(tangent) = json["meshes"][0]["primitives"][0]["attributes"]["TANGENT"].as_i64(){ [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_tangent` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `cost` is never read [INFO] [stdout] --> src/main.rs:702:20 [INFO] [stdout] | [INFO] [stdout] 702 | let mut cost = f32::MAX; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `cost` is never read [INFO] [stdout] --> src/main.rs:712:13 [INFO] [stdout] | [INFO] [stdout] 712 | / cost = q[0]*v[0]*v[0] + [INFO] [stdout] 713 | | 2.0*q[1]*v[0]*v[1] + [INFO] [stdout] 714 | | 2.0*q[2]*v[0]*v[2] + [INFO] [stdout] 715 | | 2.0*q[3]*v[0] + [INFO] [stdout] ... | [INFO] [stdout] 719 | | q[7]*v[2]*v[2] + [INFO] [stdout] 720 | | 2.0*q[8]*v[2] + q[9]; [INFO] [stdout] | |________________________________________^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `cost` is never read [INFO] [stdout] --> src/main.rs:723:13 [INFO] [stdout] | [INFO] [stdout] 723 | / cost = q[0]*new_pos[0]*new_pos[0] + [INFO] [stdout] 724 | | 2.0*q[1]*new_pos[0]*new_pos[1] + [INFO] [stdout] 725 | | 2.0*q[2]*new_pos[0]*new_pos[2] + [INFO] [stdout] 726 | | 2.0*q[3]*new_pos[0] + [INFO] [stdout] ... | [INFO] [stdout] 730 | | q[7]*new_pos[2]*new_pos[2] + [INFO] [stdout] 731 | | 2.0*q[8]*new_pos[2] + q[9]; [INFO] [stdout] | |______________________________________________^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `normal_list` [INFO] [stdout] --> src/main.rs:746:38 [INFO] [stdout] | [INFO] [stdout] 746 | fn initialize(index_list:&Vec, normal_list:&HashMap>, [INFO] [stdout] | ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_normal_list` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `tangent_list` [INFO] [stdout] --> src/main.rs:748:14 [INFO] [stdout] | [INFO] [stdout] 748 | tangent_list:&HashMap>, [INFO] [stdout] | ^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_tangent_list` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `texcoord_0_list` [INFO] [stdout] --> src/main.rs:749:14 [INFO] [stdout] | [INFO] [stdout] 749 | texcoord_0_list:&HashMap>) -> (HashMap, u32) { [INFO] [stdout] | ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_texcoord_0_list` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/main.rs:871:9 [INFO] [stdout] | [INFO] [stdout] 871 | let mut json:Value = serde_json::from_slice(json_chunk).expect("Failed to parse JSON"); [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:894:31 [INFO] [stdout] | [INFO] [stdout] 894 | let indices = if let Some(mut indices) = json["meshes"][0]["primitives"][0]["indices"].as_i64(){ [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:964:13 [INFO] [stdout] | [INFO] [stdout] 964 | let mut t = primitives.get("indices").unwrap().componentType; [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:965:13 [INFO] [stdout] | [INFO] [stdout] 965 | let mut num = primitives.get("indices").unwrap().count; [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `byteOffset`, `normalized`, and `prim_type` are never read [INFO] [stdout] --> src/main.rs:39:5 [INFO] [stdout] | [INFO] [stdout] 37 | struct Prim { [INFO] [stdout] | ---- fields in this struct [INFO] [stdout] 38 | bufferView: u32, [INFO] [stdout] 39 | byteOffset: u32, [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] 40 | componentType: u32, [INFO] [stdout] 41 | normalized: bool, [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] 42 | count: u32, [INFO] [stdout] 43 | prim_type: String, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Prim` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `buffer` and `byteLength` are never read [INFO] [stdout] --> src/main.rs:62:5 [INFO] [stdout] | [INFO] [stdout] 61 | struct View { [INFO] [stdout] | ---- fields in this struct [INFO] [stdout] 62 | buffer: u32, [INFO] [stdout] | ^^^^^^ [INFO] [stdout] 63 | byteLength: u32, [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `View` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `tri_normal` is never used [INFO] [stdout] --> src/main.rs:646:4 [INFO] [stdout] | [INFO] [stdout] 646 | fn tri_normal(v0: Vector3, v1: Vector3, v2: Vector3) -> Vector3 { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-canonical implementation of `partial_cmp` on an `Ord` type [INFO] [stdout] --> src/main.rs:24:1 [INFO] [stdout] | [INFO] [stdout] 24 | / impl PartialOrd for Remove { [INFO] [stdout] 25 | | fn partial_cmp(&self, other: &Self) -> Option { [INFO] [stdout] 26 | | self.2.partial_cmp(&other.2) [INFO] [stdout] 27 | | } [INFO] [stdout] 28 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_canonical_partial_ord_impl [INFO] [stdout] = note: `#[warn(clippy::non_canonical_partial_ord_impl)]` on by default [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 25 - fn partial_cmp(&self, other: &Self) -> Option { [INFO] [stdout] 26 - self.2.partial_cmp(&other.2) [INFO] [stdout] 27 - } [INFO] [stdout] 25 + fn partial_cmp(&self, other: &Self) -> Option { Some(self.cmp(other)) } [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler [INFO] [stdout] --> src/main.rs:112:42 [INFO] [stdout] | [INFO] [stdout] 112 | "gltf"| "glb" => decimation_gltf(&path, &method, limit), [INFO] [stdout] | ^^^^^ help: change this to: `path` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow [INFO] [stdout] = note: `#[warn(clippy::needless_borrow)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler [INFO] [stdout] --> src/main.rs:112:49 [INFO] [stdout] | [INFO] [stdout] 112 | "gltf"| "glb" => decimation_gltf(&path, &method, limit), [INFO] [stdout] | ^^^^^^^ help: change this to: `method` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this boolean expression can be simplified [INFO] [stdout] --> src/main.rs:314:16 [INFO] [stdout] | [INFO] [stdout] 314 | if (index_list[i*3] == remove.0 && index_list[i*3+1] == remove.0) || [INFO] [stdout] | ________________^ [INFO] [stdout] 315 | | (index_list[i*3+1] == remove.0 && index_list[i*3+2] == remove.0) || [INFO] [stdout] | |________________________________________________________________________________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stdout] = note: `#[warn(clippy::nonminimal_bool)]` on by default [INFO] [stdout] help: try [INFO] [stdout] | [INFO] [stdout] 314 - if (index_list[i*3] == remove.0 && index_list[i*3+1] == remove.0) || [INFO] [stdout] 315 - (index_list[i*3+1] == remove.0 && index_list[i*3+2] == remove.0) || [INFO] [stdout] 314 + if !(index_list[i*3+1] != remove.0 || index_list[i*3] != remove.0 && index_list[i*3+2] != remove.0) || [INFO] [stdout] | [INFO] [stdout] 314 - if (index_list[i*3] == remove.0 && index_list[i*3+1] == remove.0) || [INFO] [stdout] 315 - (index_list[i*3+1] == remove.0 && index_list[i*3+2] == remove.0) || [INFO] [stdout] 314 + if (index_list[i*3+2] == remove.0 || index_list[i*3] == remove.0) && index_list[i*3+1] == remove.0 || [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this call to `as_ref` does nothing [INFO] [stdout] --> src/main.rs:131:11 [INFO] [stdout] | [INFO] [stdout] 131 | match method.as_ref() { [INFO] [stdout] | ^^^^^^^^^^^^^^^ help: try: `method` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_asref [INFO] [stdout] = note: `#[warn(clippy::useless_asref)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `j` is used to index `index_ref` [INFO] [stdout] --> src/main.rs:145:18 [INFO] [stdout] | [INFO] [stdout] 145 | for j in i+1..position_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] = note: `#[warn(clippy::needless_range_loop)]` on by default [INFO] [stdout] help: consider using an iterator and enumerate() [INFO] [stdout] | [INFO] [stdout] 145 - for j in i+1..position_list.len() { [INFO] [stdout] 145 + for (j, ) in index_ref.iter_mut().enumerate().take(position_list.len()).skip(i+1) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `k` is only used to index `index_list` [INFO] [stdout] --> src/main.rs:152:26 [INFO] [stdout] | [INFO] [stdout] 152 | for k in 0..index_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 152 - for k in 0..index_list.len() { [INFO] [stdout] 152 + for in &mut index_list { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is only used to index `index_list` [INFO] [stdout] --> src/main.rs:308:18 [INFO] [stdout] | [INFO] [stdout] 308 | for i in 0..index_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 308 - for i in 0..index_list.len() { [INFO] [stdout] 308 + for in &mut index_list { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: implicitly performing saturating subtraction [INFO] [stdout] --> src/main.rs:317:17 [INFO] [stdout] | [INFO] [stdout] 317 | / if goal > 0 { [INFO] [stdout] 318 | | goal = goal-1; [INFO] [stdout] 319 | | } [INFO] [stdout] | |_________________^ help: try: `goal = goal.saturating_sub(1);` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#implicit_saturating_sub [INFO] [stdout] = note: `#[warn(clippy::implicit_saturating_sub)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:318:21 [INFO] [stdout] | [INFO] [stdout] 318 | goal = goal-1; [INFO] [stdout] | ^^^^^^^^^^^^^ help: replace it with: `goal -= 1` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] = note: `#[warn(clippy::assign_op_pattern)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manually filling a slice [INFO] [stdout] --> src/main.rs:459:5 [INFO] [stdout] | [INFO] [stdout] 459 | / for i in 0..index_ref.len() { [INFO] [stdout] 460 | | index_ref[i] = false; [INFO] [stdout] 461 | | } [INFO] [stdout] | |_____^ help: try: `index_ref.fill(false);` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_slice_fill [INFO] [stdout] = note: `#[warn(clippy::manual_slice_fill)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is only used to index `index_ref` [INFO] [stdout] --> src/main.rs:459:14 [INFO] [stdout] | [INFO] [stdout] 459 | for i in 0..index_ref.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 459 - for i in 0..index_ref.len() { [INFO] [stdout] 459 + for in &mut index_ref { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is used to index `index_ref` [INFO] [stdout] --> src/main.rs:466:14 [INFO] [stdout] | [INFO] [stdout] 466 | for i in 0..index_ref.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator and enumerate() [INFO] [stdout] | [INFO] [stdout] 466 - for i in 0..index_ref.len() { [INFO] [stdout] 466 + for (i, ) in index_ref.iter().enumerate() { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: equality checks against true are unnecessary [INFO] [stdout] --> src/main.rs:467:12 [INFO] [stdout] | [INFO] [stdout] 467 | if index_ref[i] == true { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: try: `index_ref[i]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bool_comparison [INFO] [stdout] = note: `#[warn(clippy::bool_comparison)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:470:13 [INFO] [stdout] | [INFO] [stdout] 470 | k = k+1; [INFO] [stdout] | ^^^^^^^ help: replace it with: `k += 1` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is only used to index `index_list` [INFO] [stdout] --> src/main.rs:475:14 [INFO] [stdout] | [INFO] [stdout] 475 | for i in 0..index_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 475 - for i in 0..index_list.len() { [INFO] [stdout] 475 + for in &index_list { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `j` is only used to index `exist` [INFO] [stdout] --> src/main.rs:478:18 [INFO] [stdout] | [INFO] [stdout] 478 | for j in 0..exist.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 478 - for j in 0..exist.len() { [INFO] [stdout] 478 + for in &exist { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: equality checks against false can be replaced by a negation [INFO] [stdout] --> src/main.rs:483:12 [INFO] [stdout] | [INFO] [stdout] 483 | if same == false { [INFO] [stdout] | ^^^^^^^^^^^^^ help: try: `!same` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bool_comparison [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:484:13 [INFO] [stdout] | [INFO] [stdout] 484 | index_number = index_number + 1; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `index_number += 1` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the loop variable `i` is only used to index `new_index_list` [INFO] [stdout] --> src/main.rs:496:22 [INFO] [stdout] | [INFO] [stdout] 496 | for i in 0..new_index_list.len() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop [INFO] [stdout] help: consider using an iterator [INFO] [stdout] | [INFO] [stdout] 496 - for i in 0..new_index_list.len() { [INFO] [stdout] 496 + for in &new_index_list { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:571:13 [INFO] [stdout] | [INFO] [stdout] 571 | offset = offset + index_list.len()*2; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `offset += index_list.len()*2` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:574:13 [INFO] [stdout] | [INFO] [stdout] 574 | offset = offset + index_number*12; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `offset += index_number*12` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:577:13 [INFO] [stdout] | [INFO] [stdout] 577 | offset = offset + index_number*12; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `offset += index_number*12` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:580:13 [INFO] [stdout] | [INFO] [stdout] 580 | offset = offset + index_number*8; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `offset += index_number*8` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do [INFO] [stdout] --> src/main.rs:654:30 [INFO] [stdout] | [INFO] [stdout] 654 | fn get_valid_edge(index_list:&Vec, position_list:&HashMap>, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stdout] = note: `#[warn(clippy::ptr_arg)]` on by default [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 654 - fn get_valid_edge(index_list:&Vec, position_list:&HashMap>, [INFO] [stdout] 654 + fn get_valid_edge(index_list:&[u32], position_list:&HashMap>, [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do [INFO] [stdout] --> src/main.rs:691:19 [INFO] [stdout] | [INFO] [stdout] 691 | fn update_cost(q1:&Vec, q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 691 - fn update_cost(q1:&Vec, q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] 691 + fn update_cost(q1:&[f32], q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do [INFO] [stdout] --> src/main.rs:691:33 [INFO] [stdout] | [INFO] [stdout] 691 | fn update_cost(q1:&Vec, q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 691 - fn update_cost(q1:&Vec, q2:&Vec, new_pos:&Vector3) -> f32{ [INFO] [stdout] 691 + fn update_cost(q1:&Vec, q2:&[f32], new_pos:&Vector3) -> f32{ [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do [INFO] [stdout] --> src/main.rs:746:26 [INFO] [stdout] | [INFO] [stdout] 746 | fn initialize(index_list:&Vec, normal_list:&HashMap>, [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg [INFO] [stdout] help: change this to [INFO] [stdout] | [INFO] [stdout] 746 - fn initialize(index_list:&Vec, normal_list:&HashMap>, [INFO] [stdout] 746 + fn initialize(index_list:&[u32], normal_list:&HashMap>, [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:785:21 [INFO] [stdout] | [INFO] [stdout] 785 | q_matrix[x] = q_matrix[x] + k_list[j][x]; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `q_matrix[x] += k_list[j][x]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:800:21 [INFO] [stdout] | [INFO] [stdout] 800 | q_matrix[x] = q_matrix[x] + k_list[j][x]; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `q_matrix[x] += k_list[j][x]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:815:21 [INFO] [stdout] | [INFO] [stdout] 815 | q_matrix[x] = q_matrix[x] + k_list[j][x]; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `q_matrix[x] += k_list[j][x]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `push` immediately after creation [INFO] [stdout] --> src/main.rs:838:5 [INFO] [stdout] | [INFO] [stdout] 838 | / let mut k = Vec::with_capacity(10); [INFO] [stdout] ... | [INFO] [stdout] 852 | | k.push(c*d); [INFO] [stdout] 853 | | k.push(d*d); [INFO] [stdout] | |________________^ help: consider using the `vec![]` macro: `let k = vec![..];` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#vec_init_then_push [INFO] [stdout] = note: `#[warn(clippy::vec_init_then_push)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: very complex type used. Consider factoring parts into `type` definitions [INFO] [stdout] --> src/main.rs:857:31 [INFO] [stdout] | [INFO] [stdout] 857 | fn unpack_gltf(path:&Path) -> (Vec, HashMap>, HashMap>, [INFO] [stdout] | _______________________________^ [INFO] [stdout] 858 | | HashMap>, HashMap>, [INFO] [stdout] 859 | | Value, HashMap) { [INFO] [stdout] | |____________________________________________________________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity [INFO] [stdout] = note: `#[warn(clippy::type_complexity)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the variable `i` is used as a loop counter [INFO] [stdout] --> src/main.rs:881:9 [INFO] [stdout] | [INFO] [stdout] 881 | for buffer_view in buffer_views { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `for (i, buffer_view) in buffer_views.iter().enumerate()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_counter_loop [INFO] [stdout] = note: `#[warn(clippy::explicit_counter_loop)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual implementation of an assign operation [INFO] [stdout] --> src/main.rs:952:13 [INFO] [stdout] | [INFO] [stdout] 952 | i = i+1; [INFO] [stdout] | ^^^^^^^ help: replace it with: `i += 1` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assign_op_pattern [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `vec!` [INFO] [stdout] --> src/main.rs:698:17 [INFO] [stdout] | [INFO] [stdout] 698 | let mut q = vec![0.0; 10]; [INFO] [stdout] | ^^^^^^^^^^^^^ help: you can use an array directly: `[0.0; 10]` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_vec [INFO] [stdout] = note: `#[warn(clippy::useless_vec)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `bufferView` should have a snake case name [INFO] [stdout] --> src/main.rs:38:5 [INFO] [stdout] | [INFO] [stdout] 38 | bufferView: u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `buffer_view` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:39:5 [INFO] [stdout] | [INFO] [stdout] 39 | byteOffset: u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `componentType` should have a snake case name [INFO] [stdout] --> src/main.rs:40:5 [INFO] [stdout] | [INFO] [stdout] 40 | componentType: u32, [INFO] [stdout] | ^^^^^^^^^^^^^ help: convert the identifier to snake case: `component_type` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `bufferView` should have a snake case name [INFO] [stdout] --> src/main.rs:47:12 [INFO] [stdout] | [INFO] [stdout] 47 | fn new(bufferView:u32, byteOffset:u32, componentType:u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `buffer_view` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:47:28 [INFO] [stdout] | [INFO] [stdout] 47 | fn new(bufferView:u32, byteOffset:u32, componentType:u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `componentType` should have a snake case name [INFO] [stdout] --> src/main.rs:47:44 [INFO] [stdout] | [INFO] [stdout] 47 | fn new(bufferView:u32, byteOffset:u32, componentType:u32, [INFO] [stdout] | ^^^^^^^^^^^^^ help: convert the identifier to snake case: `component_type` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `byteLength` should have a snake case name [INFO] [stdout] --> src/main.rs:63:5 [INFO] [stdout] | [INFO] [stdout] 63 | byteLength: u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_length` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:64:5 [INFO] [stdout] | [INFO] [stdout] 64 | byteOffset: u32, [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `byteLength` should have a snake case name [INFO] [stdout] --> src/main.rs:68:24 [INFO] [stdout] | [INFO] [stdout] 68 | fn new(buffer:u32, byteLength:u32, byteOffset:u32) -> Self { [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_length` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:68:40 [INFO] [stdout] | [INFO] [stdout] 68 | fn new(buffer:u32, byteLength:u32, byteOffset:u32) -> Self { [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `NORMAL` should have a snake case name [INFO] [stdout] --> src/main.rs:447:9 [INFO] [stdout] | [INFO] [stdout] 447 | let NORMAL = primitives.get("NORMAL").unwrap(); [INFO] [stdout] | ^^^^^^ help: convert the identifier to snake case: `normal` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `POSITION` should have a snake case name [INFO] [stdout] --> src/main.rs:448:9 [INFO] [stdout] | [INFO] [stdout] 448 | let POSITION = primitives.get("POSITION").unwrap(); [INFO] [stdout] | ^^^^^^^^ help: convert the identifier to snake case: `position` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `TEXCOORD_0` should have a snake case name [INFO] [stdout] --> src/main.rs:449:9 [INFO] [stdout] | [INFO] [stdout] 449 | let TEXCOORD_0 = primitives.get("TEXCOORD_0").unwrap(); [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `texcoord_0` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `bufferView` should have a snake case name [INFO] [stdout] --> src/main.rs:934:17 [INFO] [stdout] | [INFO] [stdout] 934 | let bufferView = accessor["bufferView"].as_u64().unwrap_or(0) as u32; [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `buffer_view` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `byteOffset` should have a snake case name [INFO] [stdout] --> src/main.rs:935:17 [INFO] [stdout] | [INFO] [stdout] 935 | let byteOffset = accessor["byteOffset"].as_u64().unwrap_or(0) as u32; [INFO] [stdout] | ^^^^^^^^^^ help: convert the identifier to snake case: `byte_offset` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `componentType` should have a snake case name [INFO] [stdout] --> src/main.rs:936:17 [INFO] [stdout] | [INFO] [stdout] 936 | let componentType = accessor["componentType"].as_u64().unwrap() as u32; [INFO] [stdout] | ^^^^^^^^^^^^^ help: convert the identifier to snake case: `component_type` [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 31.68s [INFO] running `Command { std: "docker" "inspect" "b05db8d48d00e4d485c81d11dfc5befc958d5ab33799a0655a622cf2e789dabc", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "b05db8d48d00e4d485c81d11dfc5befc958d5ab33799a0655a622cf2e789dabc", kill_on_drop: false }` [INFO] [stdout] b05db8d48d00e4d485c81d11dfc5befc958d5ab33799a0655a622cf2e789dabc