[INFO] fetching crate art-stamps 0.1.0... [INFO] testing art-stamps-0.1.0 against 1.60.0 for beta-1.61-1 [INFO] extracting crate art-stamps 0.1.0 into /workspace/builds/worker-37/source [INFO] validating manifest of crates.io crate art-stamps 0.1.0 on toolchain 1.60.0 [INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+1.60.0" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking crates.io crate art-stamps 0.1.0 [INFO] finished tweaking crates.io crate art-stamps 0.1.0 [INFO] tweaked toml for crates.io crate art-stamps 0.1.0 written to /workspace/builds/worker-37/source/Cargo.toml [INFO] crate crates.io crate art-stamps 0.1.0 already has a lockfile, it will not be regenerated [INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+1.60.0" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded serde-xml-rs v0.3.1 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-37/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-37/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:b0c94ce3c1162fcb8e57cac5b65ec2f72eabb1eebea4fcc35e269e823f681646" "/opt/rustwide/cargo-home/bin/cargo" "+1.60.0" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] b12d71debc5fec24fe6b9343e77f2350797088a1ea516c626a4cf5fc07bfccf6 [INFO] running `Command { std: "docker" "start" "-a" "b12d71debc5fec24fe6b9343e77f2350797088a1ea516c626a4cf5fc07bfccf6", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "b12d71debc5fec24fe6b9343e77f2350797088a1ea516c626a4cf5fc07bfccf6", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "b12d71debc5fec24fe6b9343e77f2350797088a1ea516c626a4cf5fc07bfccf6", kill_on_drop: false }` [INFO] [stdout] b12d71debc5fec24fe6b9343e77f2350797088a1ea516c626a4cf5fc07bfccf6 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-37/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-37/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=warn" "-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:b0c94ce3c1162fcb8e57cac5b65ec2f72eabb1eebea4fcc35e269e823f681646" "/opt/rustwide/cargo-home/bin/cargo" "+1.60.0" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 3040cb81574d9d1b83ccfe1a213384a001e60196b0620d08e3e58bceb7e224ba [INFO] running `Command { std: "docker" "start" "-a" "3040cb81574d9d1b83ccfe1a213384a001e60196b0620d08e3e58bceb7e224ba", kill_on_drop: false }` [INFO] [stderr] Compiling cfg-if v0.1.10 [INFO] [stderr] Compiling libc v0.2.77 [INFO] [stderr] Compiling version-compare v0.0.10 [INFO] [stderr] Compiling proc-macro2 v1.0.21 [INFO] [stderr] Compiling memchr v2.3.3 [INFO] [stderr] Compiling gimli v0.22.0 [INFO] [stderr] Compiling unicode-xid v0.2.1 [INFO] [stderr] Compiling adler v0.2.3 [INFO] [stderr] Compiling log v0.4.11 [INFO] [stderr] Compiling serde v1.0.116 [INFO] [stderr] Compiling rustc-demangle v0.1.16 [INFO] [stderr] Compiling syn v1.0.41 [INFO] [stderr] Compiling lazy_static v1.4.0 [INFO] [stderr] Compiling object v0.20.0 [INFO] [stderr] Compiling sdl2 v0.34.3 [INFO] [stderr] Compiling serde_derive v1.0.116 [INFO] [stderr] Compiling regex-syntax v0.6.18 [INFO] [stderr] Compiling xml-rs v0.8.3 [INFO] [stderr] Compiling miniz_oxide v0.4.2 [INFO] [stderr] Compiling thread_local v1.0.1 [INFO] [stderr] Compiling sdl2-sys v0.34.3 [INFO] [stderr] Compiling aho-corasick v0.7.13 [INFO] [stderr] Compiling quote v1.0.7 [INFO] [stderr] Compiling regex v1.3.9 [INFO] [stderr] Compiling addr2line v0.13.0 [INFO] [stderr] Compiling backtrace v0.3.50 [INFO] [stderr] Compiling error-chain v0.10.0 [INFO] [stderr] Compiling serde-xml-rs v0.3.1 [INFO] [stderr] Compiling art-stamps v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `std::path::Path` [INFO] [stdout] --> src/polygonsvg.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::path::Path; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::collections::HashMap` [INFO] [stdout] --> src/polygonsvg.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::collections::HashMap; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `super::serde_xml_rs::from_str` [INFO] [stdout] --> src/polygonsvg.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::io::Read` [INFO] [stdout] --> src/polygonsvg.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::io::Read; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::convert::TryFrom` [INFO] [stdout] --> src/polygonsvg.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | use std::convert::TryFrom; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `defs` should have an upper camel case name [INFO] [stdout] --> src/polygonsvg.rs:102:12 [INFO] [stdout] | [INFO] [stdout] 102 | pub struct defs { [INFO] [stdout] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the item `from_str` is imported redundantly [INFO] [stdout] --> src/polygonsvg.rs:238:13 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ----------------------------- the item `from_str` is already imported here [INFO] [stdout] ... [INFO] [stdout] 238 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `super::serde_xml_rs::from_str` [INFO] [stdout] --> src/svgxml.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `defs` should have an upper camel case name [INFO] [stdout] --> src/svgxml.rs:326:12 [INFO] [stdout] | [INFO] [stdout] 326 | pub struct defs { [INFO] [stdout] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the item `from_str` is imported redundantly [INFO] [stdout] --> src/svgxml.rs:387:13 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ----------------------------- the item `from_str` is already imported here [INFO] [stdout] ... [INFO] [stdout] 387 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Color`, `Mask`, `SourceStamp` [INFO] [stdout] --> src/svgxml.rs:470:26 [INFO] [stdout] | [INFO] [stdout] 470 | use super::{F64Point,Color, SourceStamp, Mask}; [INFO] [stdout] | ^^^^^ ^^^^^^^^^^^ ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused macro definition: `assert_approx_eq` [INFO] [stdout] --> src/svgxml.rs:705:16 [INFO] [stdout] | [INFO] [stdout] 705 | macro_rules! assert_approx_eq { [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_macros)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::fmt::Write` [INFO] [stdout] --> src/polygonsvg.rs:8:5 [INFO] [stdout] | [INFO] [stdout] 8 | use std::fmt::Write; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `segment_to_end` [INFO] [stdout] --> src/polygonsvg.rs:534:9 [INFO] [stdout] | [INFO] [stdout] 534 | let segment_to_end = sub2d(perp_dir, segment.0); [INFO] [stdout] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_segment_to_end` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `rest` [INFO] [stdout] --> src/svgxml.rs:78:16 [INFO] [stdout] | [INFO] [stdout] 78 | let (bstr, rest) = rest.split_at(2); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_rest` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `scratch` [INFO] [stdout] --> src/svgxml.rs:291:17 [INFO] [stdout] | [INFO] [stdout] 291 | let mut scratch = String::new(); [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_scratch` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/polygonsvg.rs:242:13 [INFO] [stdout] | [INFO] [stdout] 242 | let mut ret = self.section.to_polygon(); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/polygonsvg.rs:368:21 [INFO] [stdout] | [INFO] [stdout] 368 | let mut ta = (a.1 - origin.1) / dir.1; [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/polygonsvg.rs:369:21 [INFO] [stdout] | [INFO] [stdout] 369 | let mut tb = (b.1 - origin.1) / dir.1; [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/polygonsvg.rs:488:9 [INFO] [stdout] | [INFO] [stdout] 488 | let mut a_p_intersection = ray_vs_polygon(a, sub2d(b, a), poly_transform, poly); [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/polygonsvg.rs:489:9 [INFO] [stdout] | [INFO] [stdout] 489 | let mut b_p_intersection = ray_vs_polygon(b, sub2d(a, b), poly_transform, poly); [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/svgxml.rs:291:13 [INFO] [stdout] | [INFO] [stdout] 291 | let mut scratch = String::new(); [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function is never used: `read_to_string` [INFO] [stdout] --> src/polygonsvg.rs:721:6 [INFO] [stdout] | [INFO] [stdout] 721 | fn read_to_string(filename: &Path) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `clipPath` should have a snake case name [INFO] [stdout] --> src/svgxml.rs:328:9 [INFO] [stdout] | [INFO] [stdout] 328 | pub clipPath: Vec, [INFO] [stdout] | ^^^^^^^^ help: convert the identifier to snake case: `clip_path` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 24 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `key` [INFO] [stdout] --> src/bin/art-stamps.rs:777:22 [INFO] [stdout] | [INFO] [stdout] 777 | for (key,_)in keys_down.iter() { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_key` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `mask` [INFO] [stdout] --> src/bin/art-stamps.rs:463:21 [INFO] [stdout] | [INFO] [stdout] 463 | for mask in self.mask_transforms.iter_mut() { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_mask` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field is never read: `name` [INFO] [stdout] --> src/bin/art-stamps.rs:97:5 [INFO] [stdout] | [INFO] [stdout] 97 | name: String, [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] note: `InventoryKey` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis [INFO] [stdout] --> src/bin/art-stamps.rs:95:10 [INFO] [stdout] | [INFO] [stdout] 95 | #[derive(Clone, Debug)] [INFO] [stdout] | ^^^^^ ^^^^^ [INFO] [stdout] = note: this warning originates in the derive macro `Clone` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field is never read: `clip` [INFO] [stdout] --> src/bin/art-stamps.rs:98:5 [INFO] [stdout] | [INFO] [stdout] 98 | clip: String, [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] note: `InventoryKey` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis [INFO] [stdout] --> src/bin/art-stamps.rs:95:10 [INFO] [stdout] | [INFO] [stdout] 95 | #[derive(Clone, Debug)] [INFO] [stdout] | ^^^^^ ^^^^^ [INFO] [stdout] = note: this warning originates in the derive macro `Clone` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 4 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished dev [unoptimized + debuginfo] target(s) in 14.41s [INFO] running `Command { std: "docker" "inspect" "3040cb81574d9d1b83ccfe1a213384a001e60196b0620d08e3e58bceb7e224ba", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "3040cb81574d9d1b83ccfe1a213384a001e60196b0620d08e3e58bceb7e224ba", kill_on_drop: false }` [INFO] [stdout] 3040cb81574d9d1b83ccfe1a213384a001e60196b0620d08e3e58bceb7e224ba [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-37/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-37/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=warn" "-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:b0c94ce3c1162fcb8e57cac5b65ec2f72eabb1eebea4fcc35e269e823f681646" "/opt/rustwide/cargo-home/bin/cargo" "+1.60.0" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 593bc7228cb26308e6427ead14943d9f9794231f5a2dbda25ecc20081e981682 [INFO] running `Command { std: "docker" "start" "-a" "593bc7228cb26308e6427ead14943d9f9794231f5a2dbda25ecc20081e981682", kill_on_drop: false }` [INFO] [stdout] warning: unused import: `std::path::Path` [INFO] [stdout] --> src/polygonsvg.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::path::Path; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::collections::HashMap` [INFO] [stdout] --> src/polygonsvg.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::collections::HashMap; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `super::serde_xml_rs::from_str` [INFO] [stdout] --> src/polygonsvg.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::io::Read` [INFO] [stdout] --> src/polygonsvg.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::io::Read; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::convert::TryFrom` [INFO] [stdout] --> src/polygonsvg.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | use std::convert::TryFrom; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `defs` should have an upper camel case name [INFO] [stdout] --> src/polygonsvg.rs:102:12 [INFO] [stdout] | [INFO] [stdout] 102 | pub struct defs { [INFO] [stdout] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the item `from_str` is imported redundantly [INFO] [stdout] --> src/polygonsvg.rs:238:13 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ----------------------------- the item `from_str` is already imported here [INFO] [stdout] ... [INFO] [stdout] 238 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `super::serde_xml_rs::from_str` [INFO] [stdout] --> src/svgxml.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `defs` should have an upper camel case name [INFO] [stdout] --> src/svgxml.rs:326:12 [INFO] [stdout] | [INFO] [stdout] 326 | pub struct defs { [INFO] [stdout] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the item `from_str` is imported redundantly [INFO] [stdout] --> src/svgxml.rs:387:13 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ----------------------------- the item `from_str` is already imported here [INFO] [stdout] ... [INFO] [stdout] 387 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `Color`, `Mask`, `SourceStamp` [INFO] [stdout] --> src/svgxml.rs:470:26 [INFO] [stdout] | [INFO] [stdout] 470 | use super::{F64Point,Color, SourceStamp, Mask}; [INFO] [stdout] | ^^^^^ ^^^^^^^^^^^ ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused macro definition: `assert_approx_eq` [INFO] [stdout] --> src/svgxml.rs:705:16 [INFO] [stdout] | [INFO] [stdout] 705 | macro_rules! assert_approx_eq { [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_macros)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::fmt::Write` [INFO] [stdout] --> src/polygonsvg.rs:8:5 [INFO] [stdout] | [INFO] [stdout] 8 | use std::fmt::Write; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `segment_to_end` [INFO] [stdout] --> src/polygonsvg.rs:534:9 [INFO] [stdout] | [INFO] [stdout] 534 | let segment_to_end = sub2d(perp_dir, segment.0); [INFO] [stdout] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_segment_to_end` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `rest` [INFO] [stdout] --> src/svgxml.rs:78:16 [INFO] [stdout] | [INFO] [stdout] 78 | let (bstr, rest) = rest.split_at(2); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_rest` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `scratch` [INFO] [stdout] --> src/svgxml.rs:291:17 [INFO] [stdout] | [INFO] [stdout] 291 | let mut scratch = String::new(); [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_scratch` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/polygonsvg.rs:242:13 [INFO] [stdout] | [INFO] [stdout] 242 | let mut ret = self.section.to_polygon(); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/polygonsvg.rs:368:21 [INFO] [stdout] | [INFO] [stdout] 368 | let mut ta = (a.1 - origin.1) / dir.1; [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/polygonsvg.rs:369:21 [INFO] [stdout] | [INFO] [stdout] 369 | let mut tb = (b.1 - origin.1) / dir.1; [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/polygonsvg.rs:488:9 [INFO] [stdout] | [INFO] [stdout] 488 | let mut a_p_intersection = ray_vs_polygon(a, sub2d(b, a), poly_transform, poly); [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/polygonsvg.rs:489:9 [INFO] [stdout] | [INFO] [stdout] 489 | let mut b_p_intersection = ray_vs_polygon(b, sub2d(a, b), poly_transform, poly); [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/svgxml.rs:291:13 [INFO] [stdout] | [INFO] [stdout] 291 | let mut scratch = String::new(); [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Compiling art-stamps v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: function is never used: `read_to_string` [INFO] [stdout] --> src/polygonsvg.rs:721:6 [INFO] [stdout] | [INFO] [stdout] 721 | fn read_to_string(filename: &Path) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `clipPath` should have a snake case name [INFO] [stdout] --> src/svgxml.rs:328:9 [INFO] [stdout] | [INFO] [stdout] 328 | pub clipPath: Vec, [INFO] [stdout] | ^^^^^^^^ help: convert the identifier to snake case: `clip_path` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 24 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::path::Path` [INFO] [stdout] --> src/polygonsvg.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::path::Path; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::collections::HashMap` [INFO] [stdout] --> src/polygonsvg.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::collections::HashMap; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `super::serde_xml_rs::from_str` [INFO] [stdout] --> src/polygonsvg.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::io::Read` [INFO] [stdout] --> src/polygonsvg.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::io::Read; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::convert::TryFrom` [INFO] [stdout] --> src/polygonsvg.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | use std::convert::TryFrom; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `defs` should have an upper camel case name [INFO] [stdout] --> src/polygonsvg.rs:102:12 [INFO] [stdout] | [INFO] [stdout] 102 | pub struct defs { [INFO] [stdout] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the item `from_str` is imported redundantly [INFO] [stdout] --> src/polygonsvg.rs:238:13 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ----------------------------- the item `from_str` is already imported here [INFO] [stdout] ... [INFO] [stdout] 238 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `super::serde_xml_rs::from_str` [INFO] [stdout] --> src/svgxml.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type `defs` should have an upper camel case name [INFO] [stdout] --> src/svgxml.rs:326:12 [INFO] [stdout] | [INFO] [stdout] 326 | pub struct defs { [INFO] [stdout] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the item `from_str` is imported redundantly [INFO] [stdout] --> src/svgxml.rs:387:13 [INFO] [stdout] | [INFO] [stdout] 5 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ----------------------------- the item `from_str` is already imported here [INFO] [stdout] ... [INFO] [stdout] 387 | use super::serde_xml_rs::from_str; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SourceStamp` [INFO] [stdout] --> src/svgxml.rs:470:33 [INFO] [stdout] | [INFO] [stdout] 470 | use super::{F64Point,Color, SourceStamp, Mask}; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `super::Transform` [INFO] [stdout] --> src/svgxml.rs:754:13 [INFO] [stdout] | [INFO] [stdout] 754 | use super::Transform; [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `key` [INFO] [stdout] --> src/bin/art-stamps.rs:777:22 [INFO] [stdout] | [INFO] [stdout] 777 | for (key,_)in keys_down.iter() { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_key` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::fmt::Write` [INFO] [stdout] --> src/polygonsvg.rs:8:5 [INFO] [stdout] | [INFO] [stdout] 8 | use std::fmt::Write; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `segment_to_end` [INFO] [stdout] --> src/polygonsvg.rs:534:9 [INFO] [stdout] | [INFO] [stdout] 534 | let segment_to_end = sub2d(perp_dir, segment.0); [INFO] [stdout] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_segment_to_end` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `rest` [INFO] [stdout] --> src/svgxml.rs:78:16 [INFO] [stdout] | [INFO] [stdout] 78 | let (bstr, rest) = rest.split_at(2); [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_rest` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `mask` [INFO] [stdout] --> src/bin/art-stamps.rs:463:21 [INFO] [stdout] | [INFO] [stdout] 463 | for mask in self.mask_transforms.iter_mut() { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_mask` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `scratch` [INFO] [stdout] --> src/svgxml.rs:291:17 [INFO] [stdout] | [INFO] [stdout] 291 | let mut scratch = String::new(); [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_scratch` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/polygonsvg.rs:242:13 [INFO] [stdout] | [INFO] [stdout] 242 | let mut ret = self.section.to_polygon(); [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field is never read: `name` [INFO] [stdout] --> src/bin/art-stamps.rs:97:5 [INFO] [stdout] | [INFO] [stdout] 97 | name: String, [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] note: `InventoryKey` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis [INFO] [stdout] --> src/bin/art-stamps.rs:95:10 [INFO] [stdout] | [INFO] [stdout] 95 | #[derive(Clone, Debug)] [INFO] [stdout] | ^^^^^ ^^^^^ [INFO] [stdout] = note: this warning originates in the derive macro `Clone` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/polygonsvg.rs:368:21 [INFO] [stdout] | [INFO] [stdout] 368 | let mut ta = (a.1 - origin.1) / dir.1; [INFO] [stdout] | ----^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field is never read: `clip` [INFO] [stdout] --> src/bin/art-stamps.rs:98:5 [INFO] [stdout] | [INFO] [stdout] 98 | clip: String, [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] note: `InventoryKey` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis [INFO] [stdout] --> src/bin/art-stamps.rs:95:10 [INFO] [stdout] | [INFO] [stdout] 95 | #[derive(Clone, Debug)] [INFO] [stdout] | ^^^^^ ^^^^^ [INFO] [stdout] = note: this warning originates in the derive macro `Clone` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/polygonsvg.rs:369:21 [INFO] [stdout] | [INFO] [stdout] 369 | let mut tb = (b.1 - origin.1) / dir.1; [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/polygonsvg.rs:488:9 [INFO] [stdout] | [INFO] [stdout] 488 | let mut a_p_intersection = ray_vs_polygon(a, sub2d(b, a), poly_transform, poly); [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/polygonsvg.rs:489:9 [INFO] [stdout] | [INFO] [stdout] 489 | let mut b_p_intersection = ray_vs_polygon(b, sub2d(a, b), poly_transform, poly); [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/svgxml.rs:291:13 [INFO] [stdout] | [INFO] [stdout] 291 | let mut scratch = String::new(); [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: structure field `clipPath` should have a snake case name [INFO] [stdout] --> src/svgxml.rs:328:9 [INFO] [stdout] | [INFO] [stdout] 328 | pub clipPath: Vec, [INFO] [stdout] | ^^^^^^^^ help: convert the identifier to snake case: `clip_path` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 4 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 23 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished test [unoptimized + debuginfo] target(s) in 44.60s [INFO] running `Command { std: "docker" "inspect" "593bc7228cb26308e6427ead14943d9f9794231f5a2dbda25ecc20081e981682", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "593bc7228cb26308e6427ead14943d9f9794231f5a2dbda25ecc20081e981682", kill_on_drop: false }` [INFO] [stdout] 593bc7228cb26308e6427ead14943d9f9794231f5a2dbda25ecc20081e981682 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-37/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-37/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=warn" "-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:b0c94ce3c1162fcb8e57cac5b65ec2f72eabb1eebea4fcc35e269e823f681646" "/opt/rustwide/cargo-home/bin/cargo" "+1.60.0" "test" "--frozen", kill_on_drop: false }` [INFO] [stdout] 5e72b7f8a3e8a59509d9db750451298b740955340b09fc466cb0001cd397857f [INFO] running `Command { std: "docker" "start" "-a" "5e72b7f8a3e8a59509d9db750451298b740955340b09fc466cb0001cd397857f", kill_on_drop: false }` [INFO] [stderr] warning: unused import: `std::path::Path` [INFO] [stderr] --> src/polygonsvg.rs:2:5 [INFO] [stderr] | [INFO] [stderr] 2 | use std::path::Path; [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_imports)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused import: `std::collections::HashMap` [INFO] [stderr] --> src/polygonsvg.rs:3:5 [INFO] [stderr] | [INFO] [stderr] 3 | use std::collections::HashMap; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused import: `super::serde_xml_rs::from_str` [INFO] [stderr] --> src/polygonsvg.rs:5:5 [INFO] [stderr] | [INFO] [stderr] 5 | use super::serde_xml_rs::from_str; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused import: `std::io::Read` [INFO] [stderr] --> src/polygonsvg.rs:7:5 [INFO] [stderr] | [INFO] [stderr] 7 | use std::io::Read; [INFO] [stderr] | ^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused import: `std::convert::TryFrom` [INFO] [stderr] --> src/polygonsvg.rs:12:5 [INFO] [stderr] | [INFO] [stderr] 12 | use std::convert::TryFrom; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: type `defs` should have an upper camel case name [INFO] [stderr] --> src/polygonsvg.rs:102:12 [INFO] [stderr] | [INFO] [stderr] 102 | pub struct defs { [INFO] [stderr] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stderr] [INFO] [stderr] warning: the item `from_str` is imported redundantly [INFO] [stderr] --> src/polygonsvg.rs:238:13 [INFO] [stderr] | [INFO] [stderr] 5 | use super::serde_xml_rs::from_str; [INFO] [stderr] | ----------------------------- the item `from_str` is already imported here [INFO] [stderr] ... [INFO] [stderr] 238 | use super::serde_xml_rs::from_str; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused import: `super::serde_xml_rs::from_str` [INFO] [stderr] --> src/svgxml.rs:5:5 [INFO] [stderr] | [INFO] [stderr] 5 | use super::serde_xml_rs::from_str; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: type `defs` should have an upper camel case name [INFO] [stderr] --> src/svgxml.rs:326:12 [INFO] [stderr] | [INFO] [stderr] 326 | pub struct defs { [INFO] [stderr] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stderr] [INFO] [stderr] warning: the item `from_str` is imported redundantly [INFO] [stderr] --> src/svgxml.rs:387:13 [INFO] [stderr] | [INFO] [stderr] 5 | use super::serde_xml_rs::from_str; [INFO] [stderr] | ----------------------------- the item `from_str` is already imported here [INFO] [stderr] ... [INFO] [stderr] 387 | use super::serde_xml_rs::from_str; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused imports: `Color`, `Mask`, `SourceStamp` [INFO] [stderr] --> src/svgxml.rs:470:26 [INFO] [stderr] | [INFO] [stderr] 470 | use super::{F64Point,Color, SourceStamp, Mask}; [INFO] [stderr] | ^^^^^ ^^^^^^^^^^^ ^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused macro definition: `assert_approx_eq` [INFO] [stderr] --> src/svgxml.rs:705:16 [INFO] [stderr] | [INFO] [stderr] 705 | macro_rules! assert_approx_eq { [INFO] [stderr] | ^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_macros)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused import: `std::fmt::Write` [INFO] [stderr] --> src/polygonsvg.rs:8:5 [INFO] [stderr] | [INFO] [stderr] 8 | use std::fmt::Write; [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused variable: `segment_to_end` [INFO] [stderr] --> src/polygonsvg.rs:534:9 [INFO] [stderr] | [INFO] [stderr] 534 | let segment_to_end = sub2d(perp_dir, segment.0); [INFO] [stderr] | ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_segment_to_end` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_variables)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused variable: `rest` [INFO] [stderr] --> src/svgxml.rs:78:16 [INFO] [stderr] | [INFO] [stderr] 78 | let (bstr, rest) = rest.split_at(2); [INFO] [stderr] | ^^^^ help: if this is intentional, prefix it with an underscore: `_rest` [INFO] [stderr] [INFO] [stderr] warning: unused variable: `scratch` [INFO] [stderr] --> src/svgxml.rs:291:17 [INFO] [stderr] | [INFO] [stderr] 291 | let mut scratch = String::new(); [INFO] [stderr] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_scratch` [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/polygonsvg.rs:242:13 [INFO] [stderr] | [INFO] [stderr] 242 | let mut ret = self.section.to_polygon(); [INFO] [stderr] | ----^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_mut)]` on by default [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/polygonsvg.rs:368:21 [INFO] [stderr] | [INFO] [stderr] 368 | let mut ta = (a.1 - origin.1) / dir.1; [INFO] [stderr] | ----^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/polygonsvg.rs:369:21 [INFO] [stderr] | [INFO] [stderr] 369 | let mut tb = (b.1 - origin.1) / dir.1; [INFO] [stderr] | ----^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/polygonsvg.rs:488:9 [INFO] [stderr] | [INFO] [stderr] 488 | let mut a_p_intersection = ray_vs_polygon(a, sub2d(b, a), poly_transform, poly); [INFO] [stderr] | ----^^^^^^^^^^^^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/polygonsvg.rs:489:9 [INFO] [stderr] | [INFO] [stderr] 489 | let mut b_p_intersection = ray_vs_polygon(b, sub2d(a, b), poly_transform, poly); [INFO] [stderr] | ----^^^^^^^^^^^^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/svgxml.rs:291:13 [INFO] [stderr] | [INFO] [stderr] 291 | let mut scratch = String::new(); [INFO] [stderr] | ----^^^^^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] [INFO] [stderr] warning: function is never used: `read_to_string` [INFO] [stderr] --> src/polygonsvg.rs:721:6 [INFO] [stderr] | [INFO] [stderr] 721 | fn read_to_string(filename: &Path) -> Result { [INFO] [stderr] | ^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(dead_code)]` on by default [INFO] [stderr] [INFO] [stderr] warning: structure field `clipPath` should have a snake case name [INFO] [stderr] --> src/svgxml.rs:328:9 [INFO] [stderr] | [INFO] [stderr] 328 | pub clipPath: Vec, [INFO] [stderr] | ^^^^^^^^ help: convert the identifier to snake case: `clip_path` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(non_snake_case)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused import: `SourceStamp` [INFO] [stderr] --> src/svgxml.rs:470:33 [INFO] [stderr] | [INFO] [stderr] 470 | use super::{F64Point,Color, SourceStamp, Mask}; [INFO] [stderr] | ^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused import: `super::Transform` [INFO] [stderr] --> src/svgxml.rs:754:13 [INFO] [stderr] | [INFO] [stderr] 754 | use super::Transform; [INFO] [stderr] | ^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: `art-stamps` (lib) generated 24 warnings [INFO] [stderr] warning: `art-stamps` (lib test) generated 23 warnings (21 duplicates) [INFO] [stderr] warning: unused variable: `key` [INFO] [stderr] --> src/bin/art-stamps.rs:777:22 [INFO] [stderr] | [INFO] [stderr] 777 | for (key,_)in keys_down.iter() { [INFO] [stderr] | ^^^ help: if this is intentional, prefix it with an underscore: `_key` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_variables)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused variable: `mask` [INFO] [stderr] --> src/bin/art-stamps.rs:463:21 [INFO] [stderr] | [INFO] [stderr] 463 | for mask in self.mask_transforms.iter_mut() { [INFO] [stderr] | ^^^^ help: if this is intentional, prefix it with an underscore: `_mask` [INFO] [stderr] [INFO] [stderr] warning: field is never read: `name` [INFO] [stderr] --> src/bin/art-stamps.rs:97:5 [INFO] [stderr] | [INFO] [stderr] 97 | name: String, [INFO] [stderr] | ^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(dead_code)]` on by default [INFO] [stderr] note: `InventoryKey` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis [INFO] [stderr] --> src/bin/art-stamps.rs:95:10 [INFO] [stderr] | [INFO] [stderr] 95 | #[derive(Clone, Debug)] [INFO] [stderr] | ^^^^^ ^^^^^ [INFO] [stderr] = note: this warning originates in the derive macro `Clone` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: field is never read: `clip` [INFO] [stderr] --> src/bin/art-stamps.rs:98:5 [INFO] [stderr] | [INFO] [stderr] 98 | clip: String, [INFO] [stderr] | ^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] note: `InventoryKey` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis [INFO] [stderr] --> src/bin/art-stamps.rs:95:10 [INFO] [stderr] | [INFO] [stderr] 95 | #[derive(Clone, Debug)] [INFO] [stderr] | ^^^^^ ^^^^^ [INFO] [stderr] = note: this warning originates in the derive macro `Clone` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: `art-stamps` (bin "art-stamps" test) generated 4 warnings [INFO] [stderr] Finished test [unoptimized + debuginfo] target(s) in 0.08s [INFO] [stderr] Running unittests (/opt/rustwide/target/debug/deps/art_stamps-ec4fd650c89b99df) [INFO] [stdout] [INFO] [stdout] running 14 tests [INFO] [stdout] test polygonsvg::test::test_segment_intersect ... ok [INFO] [stdout] test polygonsvg::test::test_parse_bad_polygon_points ... ok [INFO] [stdout] test polygonsvg::test::test_segment_inside_polygon ... ok [INFO] [stdout] test polygonsvg::test::test_capture_segment_inside_polygon ... ok [INFO] [stdout] test polygonsvg::test::test_ray_polygon_intersect ... ok [INFO] [stdout] test polygonsvg::test::test_parse_polygon_points ... ok [INFO] [stdout] test svgxml::test::test_attr_escape ... ok [INFO] [stdout] test polygonsvg::test::test_regex ... ok [INFO] [stdout] test svgxml::test::test_pack_polygon_points ... ok [INFO] [stdout] test svgxml::test::test_basic_serde ... ok [INFO] [stdout] test svgxml::test::test_transform ... ok [INFO] [stdout] test svgxml::test::test_clip_mask_serde ... ok [INFO] [stdout] test svgxml::test::test_intersect ... ok [INFO] [stdout] test polygonsvg::test::test_parse_assets ... ok [INFO] [stdout] [INFO] [stdout] test result: ok. 14 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.25s [INFO] [stdout] [INFO] [stderr] Running unittests (/opt/rustwide/target/debug/deps/art_stamps-ad85068c80a6b8ff) [INFO] [stdout] [INFO] [stderr] Doc-tests art-stamps [INFO] [stdout] running 0 tests [INFO] [stdout] [INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s [INFO] [stdout] [INFO] [stderr] warning: type `defs` should have an upper camel case name [INFO] [stderr] --> /opt/rustwide/workdir/src/polygonsvg.rs:102:12 [INFO] [stderr] | [INFO] [stderr] 102 | pub struct defs { [INFO] [stderr] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(non_camel_case_types)]` on by default [INFO] [stderr] [INFO] [stderr] warning: type `defs` should have an upper camel case name [INFO] [stderr] --> /opt/rustwide/workdir/src/svgxml.rs:326:12 [INFO] [stderr] | [INFO] [stderr] 326 | pub struct defs { [INFO] [stderr] | ^^^^ help: convert the identifier to upper camel case: `Defs` [INFO] [stderr] [INFO] [stderr] warning: 2 warnings emitted [INFO] [stderr] [INFO] [stdout] [INFO] [stdout] running 0 tests [INFO] [stdout] [INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s [INFO] [stdout] [INFO] running `Command { std: "docker" "inspect" "5e72b7f8a3e8a59509d9db750451298b740955340b09fc466cb0001cd397857f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "5e72b7f8a3e8a59509d9db750451298b740955340b09fc466cb0001cd397857f", kill_on_drop: false }` [INFO] [stdout] 5e72b7f8a3e8a59509d9db750451298b740955340b09fc466cb0001cd397857f