[INFO] fetching crate pixel-sig 0.4.0...
[INFO] checking pixel-sig-0.4.0 against try#d1d7143153307f561233bd4d93e2a3b24157b081 for pr-151539-1
[INFO] extracting crate pixel-sig 0.4.0 into /workspace/builds/worker-0-tc2/source
[INFO] started tweaking crates.io crate pixel-sig 0.4.0
[INFO] finished tweaking crates.io crate pixel-sig 0.4.0
[INFO] tweaked toml for crates.io crate pixel-sig 0.4.0 written to /workspace/builds/worker-0-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate pixel-sig 0.4.0 on toolchain d1d7143153307f561233bd4d93e2a3b24157b081
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+d1d7143153307f561233bd4d93e2a3b24157b081" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+d1d7143153307f561233bd4d93e2a3b24157b081" "generate-lockfile" "--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]      Locking 58 packages to latest compatible versions
[INFO] [stderr]       Adding amcl_wrapper v0.3.5 (available: v0.4.0)
[INFO] [stderr]       Adding rand v0.7.3 (available: v0.10.1)
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+d1d7143153307f561233bd4d93e2a3b24157b081" "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/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/target:/opt/rustwide/target:rw,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" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:3a6becf2bc8dde7f3fa57ede90e4f284e72d296796fc446bbb1e2c7cc0530151" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 99bb3e641820e1fa0aea8658424151d9aa033a9563a1f5fd64782b74c6082855
[INFO] running `Command { std: "docker" "start" "99bb3e641820e1fa0aea8658424151d9aa033a9563a1f5fd64782b74c6082855", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "99bb3e641820e1fa0aea8658424151d9aa033a9563a1f5fd64782b74c6082855" "/opt/rustwide/cargo-home/bin/cargo" "+d1d7143153307f561233bd4d93e2a3b24157b081" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "99bb3e641820e1fa0aea8658424151d9aa033a9563a1f5fd64782b74c6082855", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-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" "-w" "/opt/rustwide/workdir" "--user" "0:0" "99bb3e641820e1fa0aea8658424151d9aa033a9563a1f5fd64782b74c6082855" "/opt/rustwide/cargo-home/bin/cargo" "+d1d7143153307f561233bd4d93e2a3b24157b081" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling syn v1.0.109
[INFO] [stderr]     Checking memchr v2.8.2
[INFO] [stderr]     Checking byte-tools v0.3.1
[INFO] [stderr]    Compiling libc v0.2.186
[INFO] [stderr]     Checking generic-array v0.12.4
[INFO] [stderr]    Compiling failure_derive v0.1.8
[INFO] [stderr]     Checking miniz_oxide v0.8.9
[INFO] [stderr]    Compiling syn v2.0.118
[INFO] [stderr]     Checking either v1.16.0
[INFO] [stderr]     Checking opaque-debug v0.2.3
[INFO] [stderr]     Checking subtle-encoding v0.5.1
[INFO] [stderr]     Checking miracl_amcl v3.2.5
[INFO] [stderr]     Checking block-padding v0.1.5
[INFO] [stderr]     Checking rayon v1.12.0
[INFO] [stderr]     Checking block-buffer v0.7.3
[INFO] [stderr]     Checking digest v0.8.1
[INFO] [stderr]     Checking sha3 v0.8.2
[INFO] [stderr]     Checking object v0.37.3
[INFO] [stderr]     Checking serde_json v1.0.150
[INFO] [stderr]     Checking getrandom v0.1.16
[INFO] [stderr]     Checking rand_core v0.5.1
[INFO] [stderr]     Checking rand_chacha v0.2.2
[INFO] [stderr]     Checking rand v0.7.3
[INFO] [stderr]    Compiling synstructure v0.12.6
[INFO] [stderr]    Compiling serde_derive v1.0.228
[INFO] [stderr]     Checking backtrace v0.3.76
[INFO] [stderr]     Checking failure v0.1.8
[INFO] [stderr]     Checking amcl_wrapper v0.3.5
[INFO] [stderr]     Checking secret_sharing v0.2.0
[INFO] [stderr]     Checking pixel-sig v0.4.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused `#[macro_use]` import
[INFO] [stdout]  --> src/lib.rs:4:1
[INFO] [stdout]   |
[INFO] [stdout] 4 | #[macro_use]
[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: `failure::Error`
[INFO] [stdout]   --> src/errors.rs:10:5
[INFO] [stdout]    |
[INFO] [stdout] 10 | use failure::Error;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around assigned value
[INFO] [stdout]    --> src/keys.rs:243:40
[INFO] [stdout]     |
[INFO] [stdout] 243 |             sk_right_prime_prime[0] += (&path_factor * &r);
[INFO] [stdout]     |                                        ^                 ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 243 -             sk_right_prime_prime[0] += (&path_factor * &r);
[INFO] [stdout] 243 +             sk_right_prime_prime[0] += &path_factor * &r ;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around assigned value
[INFO] [stdout]    --> src/keys.rs:393:14
[INFO] [stdout]     |
[INFO] [stdout] 393 |         d += (&path_factor * &r);
[INFO] [stdout]     |              ^                 ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 393 -         d += (&path_factor * &r);
[INFO] [stdout] 393 +         d += &path_factor * &r ;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `from_node_num_to_path`
[INFO] [stdout]  --> src/signature.rs:9:52
[INFO] [stdout]   |
[INFO] [stdout] 9 | use crate::util::{calculate_path_factor_using_t_l, from_node_num_to_path, GeneratorSet};
[INFO] [stdout]   |                                                    ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `amcl_wrapper::field_elem::FieldElement`
[INFO] [stdout]  --> src/util.rs:3:5
[INFO] [stdout]   |
[INFO] [stdout] 3 | use amcl_wrapper::field_elem::FieldElement;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `#[macro_use]` import
[INFO] [stdout]  --> src/lib.rs:4:1
[INFO] [stdout]   |
[INFO] [stdout] 4 | #[macro_use]
[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: `failure::Error`
[INFO] [stdout]   --> src/errors.rs:10:5
[INFO] [stdout]    |
[INFO] [stdout] 10 | use failure::Error;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around assigned value
[INFO] [stdout]    --> src/keys.rs:243:40
[INFO] [stdout]     |
[INFO] [stdout] 243 |             sk_right_prime_prime[0] += (&path_factor * &r);
[INFO] [stdout]     |                                        ^                 ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 243 -             sk_right_prime_prime[0] += (&path_factor * &r);
[INFO] [stdout] 243 +             sk_right_prime_prime[0] += &path_factor * &r ;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around assigned value
[INFO] [stdout]    --> src/keys.rs:393:14
[INFO] [stdout]     |
[INFO] [stdout] 393 |         d += (&path_factor * &r);
[INFO] [stdout]     |              ^                 ^
[INFO] [stdout]     |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 393 -         d += (&path_factor * &r);
[INFO] [stdout] 393 +         d += &path_factor * &r ;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Duration`
[INFO] [stdout]    --> src/keys.rs:486:21
[INFO] [stdout]     |
[INFO] [stdout] 486 |     use std::time::{Duration, Instant};
[INFO] [stdout]     |                     ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `from_node_num_to_path`
[INFO] [stdout]  --> src/signature.rs:9:52
[INFO] [stdout]   |
[INFO] [stdout] 9 | use crate::util::{calculate_path_factor_using_t_l, from_node_num_to_path, GeneratorSet};
[INFO] [stdout]   |                                                    ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Duration`
[INFO] [stdout]    --> src/signature.rs:217:21
[INFO] [stdout]     |
[INFO] [stdout] 217 |     use std::time::{Duration, Instant};
[INFO] [stdout]     |                     ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `amcl_wrapper::field_elem::FieldElement`
[INFO] [stdout]  --> src/util.rs:3:5
[INFO] [stdout]   |
[INFO] [stdout] 3 | use amcl_wrapper::field_elem::FieldElement;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `rand::rngs::ThreadRng`
[INFO] [stdout]    --> src/threshold_sig.rs:135:9
[INFO] [stdout]     |
[INFO] [stdout] 135 |     use rand::rngs::ThreadRng;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `GroupElementVector`
[INFO] [stdout]  --> src/keys.rs:5:46
[INFO] [stdout]   |
[INFO] [stdout] 5 | use amcl_wrapper::group_elem::{GroupElement, GroupElementVector};
[INFO] [stdout]   |                                              ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `GroupElement`
[INFO] [stdout]  --> src/threshold_sig.rs:5:32
[INFO] [stdout]   |
[INFO] [stdout] 5 | use amcl_wrapper::group_elem::{GroupElement, GroupElementVector};
[INFO] [stdout]   |                                ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `T` is never read
[INFO] [stdout]    --> src/keys.rs:177:5
[INFO] [stdout]     |
[INFO] [stdout] 175 | pub struct SigkeyManager {
[INFO] [stdout]     |            ------------- field in this struct
[INFO] [stdout] 176 |     l: u8,
[INFO] [stdout] 177 |     T: u128,
[INFO] [stdout]     |     ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) 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/errors.rs:12:17
[INFO] [stdout]    |
[INFO] [stdout] 12 | #[derive(Debug, Fail)]
[INFO] [stdout]    |                 ^---
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `Fail` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_PixelError`
[INFO] [stdout] 13 | pub enum PixelError {
[INFO] [stdout]    |          ---------- `PixelError` is not local
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (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/errors.rs:12:17
[INFO] [stdout]    |
[INFO] [stdout] 12 | #[derive(Debug, Fail)]
[INFO] [stdout]    |                 ^---
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `Display` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_PixelError`
[INFO] [stdout] 13 | pub enum PixelError {
[INFO] [stdout]    |          ---------- `PixelError` is not local
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `PrefixPoP` should have an upper case name
[INFO] [stdout]   --> src/keys.rs:93:7
[INFO] [stdout]    |
[INFO] [stdout] 93 | const PrefixPoP: &[u8] = b"PoP";
[INFO] [stdout]    |       ^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_upper_case_globals)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 93 - const PrefixPoP: &[u8] = b"PoP";
[INFO] [stdout] 93 + const PREFIX_PO_P: &[u8] = b"PoP";
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `GroupElementVector`
[INFO] [stdout]  --> src/keys.rs:5:46
[INFO] [stdout]   |
[INFO] [stdout] 5 | use amcl_wrapper::group_elem::{GroupElement, GroupElementVector};
[INFO] [stdout]   |                                              ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `GroupElement`
[INFO] [stdout]  --> src/threshold_sig.rs:5:32
[INFO] [stdout]   |
[INFO] [stdout] 5 | use amcl_wrapper::group_elem::{GroupElement, GroupElementVector};
[INFO] [stdout]   |                                ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `set1`
[INFO] [stdout]    --> src/keys.rs:531:20
[INFO] [stdout]     |
[INFO] [stdout] 531 |         let (_, _, set1, _) = setup::<ThreadRng>(T1, "test_pixel", &mut rng, &mut db1).unwrap();
[INFO] [stdout]     |                    ^^^^ help: if this is intentional, prefix it with an underscore: `_set1`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `set2`
[INFO] [stdout]    --> src/keys.rs:538:20
[INFO] [stdout]     |
[INFO] [stdout] 538 |         let (_, _, set2, _) = setup::<ThreadRng>(T2, "test_pixel", &mut rng, &mut db2).unwrap();
[INFO] [stdout]     |                    ^^^^ help: if this is intentional, prefix it with an underscore: `_set2`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `l`
[INFO] [stdout]    --> src/keys.rs:740:13
[INFO] [stdout]     |
[INFO] [stdout] 740 |         let l = calculate_l(T).unwrap();
[INFO] [stdout]     |             ^ help: if this is intentional, prefix it with an underscore: `_l`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: value assigned to `t` is never read
[INFO] [stdout]    --> src/keys.rs:742:21
[INFO] [stdout]     |
[INFO] [stdout] 742 |         let mut t = 1u128;
[INFO] [stdout]     |                     ^^^^^ this value is reassigned later and never used
[INFO] [stdout] ...
[INFO] [stdout] 748 |             t = 3;
[INFO] [stdout]     |             ----- `t` is overwritten here before the previous value is read
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_assignments)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: value assigned to `t` is never read
[INFO] [stdout]    --> src/keys.rs:793:21
[INFO] [stdout]     |
[INFO] [stdout] 793 |         let mut t = 1u128;
[INFO] [stdout]     |                     ^^^^^ this value is reassigned later and never used
[INFO] [stdout] ...
[INFO] [stdout] 800 |             t = 2;
[INFO] [stdout]     |             ----- `t` is overwritten here before the previous value is read
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `set`
[INFO] [stdout]    --> src/signature.rs:220:9
[INFO] [stdout]     |
[INFO] [stdout] 220 |         set: &SigkeyManager,
[INFO] [stdout]     |         ^^^ help: if this is intentional, prefix it with an underscore: `_set`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: value assigned to `t` is never read
[INFO] [stdout]    --> src/signature.rs:417:21
[INFO] [stdout]     |
[INFO] [stdout] 417 |         let mut t = 1u128;
[INFO] [stdout]     |                     ^^^^^ this value is reassigned later and never used
[INFO] [stdout] ...
[INFO] [stdout] 424 |             t = 3;
[INFO] [stdout]     |             ----- `t` is overwritten here before the previous value is read
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: value assigned to `t` is never read
[INFO] [stdout]    --> src/signature.rs:470:21
[INFO] [stdout]     |
[INFO] [stdout] 470 |         let mut t = 1u128;
[INFO] [stdout]     |                     ^^^^^ this value is reassigned later and never used
[INFO] [stdout] ...
[INFO] [stdout] 475 |         t = 2;
[INFO] [stdout]     |         ----- `t` is overwritten here before the previous value is read
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: value assigned to `t` is never read
[INFO] [stdout]    --> src/signature.rs:490:21
[INFO] [stdout]     |
[INFO] [stdout] 490 |         let mut t = 1u128;
[INFO] [stdout]     |                     ^^^^^ this value is reassigned later and never used
[INFO] [stdout] ...
[INFO] [stdout] 497 |             t = 3;
[INFO] [stdout]     |             ----- `t` is overwritten here before the previous value is read
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: value assigned to `t` is never read
[INFO] [stdout]    --> src/signature.rs:528:21
[INFO] [stdout]     |
[INFO] [stdout] 528 |         let mut t = 1u128;
[INFO] [stdout]     |                     ^^^^^ this value is reassigned later and never used
[INFO] [stdout] ...
[INFO] [stdout] 535 |             t = 4;
[INFO] [stdout]     |             ----- `t` is overwritten here before the previous value is read
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `secret_x`
[INFO] [stdout]    --> src/threshold_sig.rs:240:14
[INFO] [stdout]     |
[INFO] [stdout] 240 |         let (secret_x, signers) = trusted_party_SSS_keygen(threshold, total, &mut rng, &generators.0, &generators.1).unwrap();
[INFO] [stdout]     |              ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_secret_x`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `T` is never read
[INFO] [stdout]    --> src/keys.rs:177:5
[INFO] [stdout]     |
[INFO] [stdout] 175 | pub struct SigkeyManager {
[INFO] [stdout]     |            ------------- field in this struct
[INFO] [stdout] 176 |     l: u8,
[INFO] [stdout] 177 |     T: u128,
[INFO] [stdout]     |     ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) 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/errors.rs:12:17
[INFO] [stdout]    |
[INFO] [stdout] 12 | #[derive(Debug, Fail)]
[INFO] [stdout]    |                 ^---
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `Fail` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_PixelError`
[INFO] [stdout] 13 | pub enum PixelError {
[INFO] [stdout]    |          ---------- `PixelError` is not local
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (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/errors.rs:12:17
[INFO] [stdout]    |
[INFO] [stdout] 12 | #[derive(Debug, Fail)]
[INFO] [stdout]    |                 ^---
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `Display` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_PixelError`
[INFO] [stdout] 13 | pub enum PixelError {
[INFO] [stdout]    |          ---------- `PixelError` is not local
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `PrefixPoP` should have an upper case name
[INFO] [stdout]   --> src/keys.rs:93:7
[INFO] [stdout]    |
[INFO] [stdout] 93 | const PrefixPoP: &[u8] = b"PoP";
[INFO] [stdout]    |       ^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_upper_case_globals)]` (part of `#[warn(nonstandard_style)]`) on by default
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 93 - const PrefixPoP: &[u8] = b"PoP";
[INFO] [stdout] 93 + const PREFIX_PO_P: &[u8] = b"PoP";
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 14.38s
[INFO] running `Command { std: "docker" "inspect" "99bb3e641820e1fa0aea8658424151d9aa033a9563a1f5fd64782b74c6082855", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "99bb3e641820e1fa0aea8658424151d9aa033a9563a1f5fd64782b74c6082855", kill_on_drop: false }`
[INFO] [stdout] 99bb3e641820e1fa0aea8658424151d9aa033a9563a1f5fd64782b74c6082855
