[INFO] cloning repository https://github.com/nielsdos/snowman [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/nielsdos/snowman" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnielsdos%2Fsnowman", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnielsdos%2Fsnowman'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] f88479c9bd34b67750ef72de760f330ab42a3f4f [INFO] checking nielsdos/snowman against try#9d63f825c0d409b0e1743a2c84b33f86965e5929 for pr-136968 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnielsdos%2Fsnowman" "/workspace/builds/worker-0-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-0-tc2/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/nielsdos/snowman on toolchain 9d63f825c0d409b0e1743a2c84b33f86965e5929 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+9d63f825c0d409b0e1743a2c84b33f86965e5929" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/nielsdos/snowman [INFO] finished tweaking git repo https://github.com/nielsdos/snowman [INFO] tweaked toml for git repo https://github.com/nielsdos/snowman written to /workspace/builds/worker-0-tc2/source/Cargo.toml [INFO] crate git repo https://github.com/nielsdos/snowman 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" "+9d63f825c0d409b0e1743a2c84b33f86965e5929" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:3959728d45483eb673ba100a951a6b9f5012970368db2a9e08e0ac9b67c8a5ad" "/opt/rustwide/cargo-home/bin/cargo" "+9d63f825c0d409b0e1743a2c84b33f86965e5929" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] c7e1ff7b08972cf004e95158228a6261cd9026d4a59e118e473cbcac7a2b8d4f [INFO] running `Command { std: "docker" "start" "-a" "c7e1ff7b08972cf004e95158228a6261cd9026d4a59e118e473cbcac7a2b8d4f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "c7e1ff7b08972cf004e95158228a6261cd9026d4a59e118e473cbcac7a2b8d4f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "c7e1ff7b08972cf004e95158228a6261cd9026d4a59e118e473cbcac7a2b8d4f", kill_on_drop: false }` [INFO] [stdout] c7e1ff7b08972cf004e95158228a6261cd9026d4a59e118e473cbcac7a2b8d4f [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:3959728d45483eb673ba100a951a6b9f5012970368db2a9e08e0ac9b67c8a5ad" "/opt/rustwide/cargo-home/bin/cargo" "+9d63f825c0d409b0e1743a2c84b33f86965e5929" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 1cee5aaa4cef565d933a10abcf84b7247a51fedb2a973ea555cce1219fd58c68 [INFO] running `Command { std: "docker" "start" "-a" "1cee5aaa4cef565d933a10abcf84b7247a51fedb2a973ea555cce1219fd58c68", kill_on_drop: false }` [INFO] [stderr] Compiling proc-macro2 v1.0.40 [INFO] [stderr] Compiling unicode-ident v1.0.2 [INFO] [stderr] Compiling version_check v0.9.4 [INFO] [stderr] Compiling quote v1.0.20 [INFO] [stderr] Compiling autocfg v1.1.0 [INFO] [stderr] Compiling libc v0.2.126 [INFO] [stderr] Compiling syn v1.0.98 [INFO] [stderr] Compiling version-compare v0.1.0 [INFO] [stderr] Compiling cfg-if v1.0.0 [INFO] [stderr] Compiling sdl2 v0.35.2 [INFO] [stderr] Checking bitflags v1.3.2 [INFO] [stderr] Checking lazy_static v1.4.0 [INFO] [stderr] Compiling proc-macro-error-attr v1.0.4 [INFO] [stderr] Compiling proc-macro-error v1.0.4 [INFO] [stderr] Compiling num-traits v0.2.15 [INFO] [stderr] Compiling num-integer v0.1.45 [INFO] [stderr] Compiling sdl2-sys v0.35.2 [INFO] [stderr] Checking time v0.1.44 [INFO] [stderr] Checking chrono v0.4.19 [INFO] [stderr] Compiling num-derive v0.3.3 [INFO] [stderr] Compiling syscall v0.1.0 (/opt/rustwide/workdir/lib/syscall) [INFO] [stderr] Checking snowman_emulator v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused variable: `accessor` [INFO] [stdout] --> src/emulated_kernel.rs:226:9 [INFO] [stdout] | [INFO] [stdout] 226 | accessor: EmulatorAccessor, [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_accessor` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `accessor` [INFO] [stdout] --> src/emulated_kernel.rs:226:9 [INFO] [stdout] | [INFO] [stdout] 226 | accessor: EmulatorAccessor, [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_accessor` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `deallocate` is never used [INFO] [stdout] --> src/bitvector_allocator.rs:29:12 [INFO] [stdout] | [INFO] [stdout] 7 | impl BitVectorAllocator { [INFO] [stdout] | ----------------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 29 | pub fn deallocate(&mut self, bit: usize) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `cmp_r8_imm8` is never used [INFO] [stdout] --> src/emulator.rs:743:8 [INFO] [stdout] | [INFO] [stdout] 31 | impl<'a> Emulator<'a> { [INFO] [stdout] | --------------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 743 | fn cmp_r8_imm8(&mut self, reg: u8) -> Result<(), EmulatorError> { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `overwrite_u8` is never used [INFO] [stdout] --> src/executable.rs:75:12 [INFO] [stdout] | [INFO] [stdout] 24 | impl<'a> Executable<'a> { [INFO] [stdout] | ----------------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 75 | pub fn overwrite_u8(&mut self, offset: usize, data: u8) -> Result<(), ExecutableFormatError> { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `width` is never read [INFO] [stdout] --> src/object_environment.rs:34:9 [INFO] [stdout] | [INFO] [stdout] 32 | pub struct Pen { [INFO] [stdout] | --- field in this struct [INFO] [stdout] 33 | // TODO: style [INFO] [stdout] 34 | pub width: u16, [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/byte_string.rs:48:18 [INFO] [stdout] | [INFO] [stdout] 48 | self.data.deref() [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `[u8]` does not implement `Deref`, so calling `deref` on `&[u8]` copies the reference, which does not do anything and can be removed [INFO] [stdout] = note: `#[warn(noop_method_call)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/constants.rs:55:32 [INFO] [stdout] | [INFO] [stdout] 55 | #[derive(Debug, Eq, PartialEq, FromPrimitive)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `FromPrimitive` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_NUM_FromPrimitive_FOR_SystemColors` [INFO] [stdout] 56 | pub enum SystemColors { [INFO] [stdout] | ------------ `SystemColors` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `FromPrimitive` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `FromPrimitive` may come from an old version of the `num_derive` crate, try updating your dependency with `cargo update -p num_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `FromPrimitive` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/constants.rs:97:45 [INFO] [stdout] | [INFO] [stdout] 97 | #[derive(Debug, Copy, Clone, Eq, PartialEq, FromPrimitive)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `FromPrimitive` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_NUM_FromPrimitive_FOR_RasterOp` [INFO] [stdout] 98 | pub enum RasterOp { [INFO] [stdout] | -------- `RasterOp` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `FromPrimitive` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `FromPrimitive` may come from an old version of the `num_derive` crate, try updating your dependency with `cargo update -p num_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `FromPrimitive` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `deallocate` is never used [INFO] [stdout] --> src/bitvector_allocator.rs:29:12 [INFO] [stdout] | [INFO] [stdout] 7 | impl BitVectorAllocator { [INFO] [stdout] | ----------------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 29 | pub fn deallocate(&mut self, bit: usize) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `cmp_r8_imm8` is never used [INFO] [stdout] --> src/emulator.rs:743:8 [INFO] [stdout] | [INFO] [stdout] 31 | impl<'a> Emulator<'a> { [INFO] [stdout] | --------------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 743 | fn cmp_r8_imm8(&mut self, reg: u8) -> Result<(), EmulatorError> { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `overwrite_u8` is never used [INFO] [stdout] --> src/executable.rs:75:12 [INFO] [stdout] | [INFO] [stdout] 24 | impl<'a> Executable<'a> { [INFO] [stdout] | ----------------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 75 | pub fn overwrite_u8(&mut self, offset: usize, data: u8) -> Result<(), ExecutableFormatError> { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `width` is never read [INFO] [stdout] --> src/object_environment.rs:34:9 [INFO] [stdout] | [INFO] [stdout] 32 | pub struct Pen { [INFO] [stdout] | --- field in this struct [INFO] [stdout] 33 | // TODO: style [INFO] [stdout] 34 | pub width: u16, [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/byte_string.rs:48:18 [INFO] [stdout] | [INFO] [stdout] 48 | self.data.deref() [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `[u8]` does not implement `Deref`, so calling `deref` on `&[u8]` copies the reference, which does not do anything and can be removed [INFO] [stdout] = note: `#[warn(noop_method_call)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/constants.rs:55:32 [INFO] [stdout] | [INFO] [stdout] 55 | #[derive(Debug, Eq, PartialEq, FromPrimitive)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `FromPrimitive` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_NUM_FromPrimitive_FOR_SystemColors` [INFO] [stdout] 56 | pub enum SystemColors { [INFO] [stdout] | ------------ `SystemColors` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `FromPrimitive` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `FromPrimitive` may come from an old version of the `num_derive` crate, try updating your dependency with `cargo update -p num_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `FromPrimitive` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/constants.rs:97:45 [INFO] [stdout] | [INFO] [stdout] 97 | #[derive(Debug, Copy, Clone, Eq, PartialEq, FromPrimitive)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `FromPrimitive` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_NUM_FromPrimitive_FOR_RasterOp` [INFO] [stdout] 98 | pub enum RasterOp { [INFO] [stdout] | -------- `RasterOp` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `FromPrimitive` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `FromPrimitive` may come from an old version of the `num_derive` crate, try updating your dependency with `cargo update -p num_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `FromPrimitive` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 7.42s [INFO] running `Command { std: "docker" "inspect" "1cee5aaa4cef565d933a10abcf84b7247a51fedb2a973ea555cce1219fd58c68", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "1cee5aaa4cef565d933a10abcf84b7247a51fedb2a973ea555cce1219fd58c68", kill_on_drop: false }` [INFO] [stdout] 1cee5aaa4cef565d933a10abcf84b7247a51fedb2a973ea555cce1219fd58c68