[INFO] fetching crate memory-cache-rust 0.1.0-alpha... [INFO] testing memory-cache-rust-0.1.0-alpha against try#f23b22458132fb20faeb8a915b1c252478696f25 for pr-131095 [INFO] extracting crate memory-cache-rust 0.1.0-alpha into /workspace/builds/worker-4-tc2/source [INFO] validating manifest of crates.io crate memory-cache-rust 0.1.0-alpha on toolchain f23b22458132fb20faeb8a915b1c252478696f25 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+f23b22458132fb20faeb8a915b1c252478696f25" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking crates.io crate memory-cache-rust 0.1.0-alpha [INFO] finished tweaking crates.io crate memory-cache-rust 0.1.0-alpha [INFO] tweaked toml for crates.io crate memory-cache-rust 0.1.0-alpha written to /workspace/builds/worker-4-tc2/source/Cargo.toml [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+f23b22458132fb20faeb8a915b1c252478696f25" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Updating crates.io index [INFO] [stderr] Locking 129 packages to latest compatible versions [INFO] [stderr] Adding ahash v0.7.8 (available: v0.8.11) [INFO] [stderr] Adding criterion v0.3.6 (available: v0.5.1) [INFO] [stderr] Adding hashbrown v0.13.2 (available: v0.15.0) [INFO] [stderr] Adding seize v0.2.5 (available: v0.4.5) [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+f23b22458132fb20faeb8a915b1c252478696f25" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:6252d7ea7fb8caaf7af6d800c5fb250a9cd862b9a7f9508afb3c54fa7fe1102e" "/opt/rustwide/cargo-home/bin/cargo" "+f23b22458132fb20faeb8a915b1c252478696f25" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] fbf88ad70fde699c96ebafd74c366f105076cea268d1675a6094af754f5e5434 [INFO] running `Command { std: "docker" "start" "-a" "fbf88ad70fde699c96ebafd74c366f105076cea268d1675a6094af754f5e5434", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "fbf88ad70fde699c96ebafd74c366f105076cea268d1675a6094af754f5e5434", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "fbf88ad70fde699c96ebafd74c366f105076cea268d1675a6094af754f5e5434", kill_on_drop: false }` [INFO] [stdout] fbf88ad70fde699c96ebafd74c366f105076cea268d1675a6094af754f5e5434 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:6252d7ea7fb8caaf7af6d800c5fb250a9cd862b9a7f9508afb3c54fa7fe1102e" "/opt/rustwide/cargo-home/bin/cargo" "+f23b22458132fb20faeb8a915b1c252478696f25" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] b51d46420898a05fde2f8bec801d0987f01e45b8ab663101bf377fdb4ba93d2b [INFO] running `Command { std: "docker" "start" "-a" "b51d46420898a05fde2f8bec801d0987f01e45b8ab663101bf377fdb4ba93d2b", kill_on_drop: false }` [INFO] [stderr] Compiling proc-macro2 v1.0.86 [INFO] [stderr] Compiling byteorder v1.5.0 [INFO] [stderr] Compiling typenum v1.17.0 [INFO] [stderr] Compiling generic-array v0.14.7 [INFO] [stderr] Compiling crossbeam-utils v0.8.20 [INFO] [stderr] Compiling num-traits v0.2.19 [INFO] [stderr] Compiling getrandom v0.2.15 [INFO] [stderr] Compiling ahash v0.8.11 [INFO] [stderr] Compiling rand v0.4.6 [INFO] [stderr] Compiling ahash v0.7.8 [INFO] [stderr] Compiling rand_core v0.6.4 [INFO] [stderr] Compiling iana-time-zone v0.1.61 [INFO] [stderr] Compiling cpufeatures v0.2.14 [INFO] [stderr] Compiling serde_json v1.0.128 [INFO] [stderr] Compiling num_cpus v1.16.0 [INFO] [stderr] Compiling crossbeam v0.3.2 [INFO] [stderr] Compiling crossbeam-epoch v0.9.18 [INFO] [stderr] Compiling quote v1.0.37 [INFO] [stderr] Compiling crossbeam-deque v0.8.5 [INFO] [stderr] Compiling rand v0.3.23 [INFO] [stderr] Compiling crossbeam-queue v0.3.11 [INFO] [stderr] Compiling syn v2.0.79 [INFO] [stderr] Compiling crossbeam-channel v0.5.13 [INFO] [stderr] Compiling libmath v0.2.1 [INFO] [stderr] Compiling concache v0.2.1 [INFO] [stderr] Compiling chrono v0.4.38 [INFO] [stderr] Compiling seize v0.2.5 [INFO] [stderr] Compiling syncpool v0.1.6 [INFO] [stderr] Compiling xxhash-rust v0.8.12 [INFO] [stderr] Compiling seahash v4.1.0 [INFO] [stderr] Compiling crossbeam v0.8.4 [INFO] [stderr] Compiling inout v0.1.3 [INFO] [stderr] Compiling crypto-common v0.1.6 [INFO] [stderr] Compiling cipher v0.4.4 [INFO] [stderr] Compiling aes v0.8.4 [INFO] [stderr] Compiling timer v0.2.0 [INFO] [stderr] Compiling zerocopy-derive v0.7.35 [INFO] [stderr] Compiling serde_derive v1.0.210 [INFO] [stderr] Compiling uuid-macro-internal v1.10.0 [INFO] [stderr] Compiling zerocopy v0.7.35 [INFO] [stderr] Compiling ppv-lite86 v0.2.20 [INFO] [stderr] Compiling hashbrown v0.13.2 [INFO] [stderr] Compiling rand_chacha v0.3.1 [INFO] [stderr] Compiling rand v0.8.5 [INFO] [stderr] Compiling aes-prng v0.2.1 [INFO] [stderr] Compiling uuid v1.10.0 [INFO] [stderr] Compiling serde v1.0.210 [INFO] [stderr] Compiling memory-cache-rust v0.1.0-alpha (/opt/rustwide/workdir) [INFO] [stdout] warning: unnecessary parentheses around index expression [INFO] [stdout] --> src/cmsketch.rs:94:51 [INFO] [stdout] | [INFO] [stdout] 94 | s.push_str(format!("{:#02} ", (self.0[(i / 2)] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 94 - s.push_str(format!("{:#02} ", (self.0[(i / 2)] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] 94 + s.push_str(format!("{:#02} ", (self.0[i / 2] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable statement [INFO] [stdout] --> src/ttl.rs:96:13 [INFO] [stdout] | [INFO] [stdout] 94 | break; [INFO] [stdout] | ----- any code following this expression is unreachable [INFO] [stdout] 95 | [INFO] [stdout] 96 | break; [INFO] [stdout] | ^^^^^^ unreachable statement [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unreachable_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `syncpool::prelude` [INFO] [stdout] --> src/ring.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use syncpool::prelude::*; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Add` [INFO] [stdout] --> src/cache.rs:6:16 [INFO] [stdout] | [INFO] [stdout] 6 | use std::ops::{Add, Deref}; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `entries` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:35:17 [INFO] [stdout] | [INFO] [stdout] 35 | let mut entries = 0; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `locs` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:36:17 [INFO] [stdout] | [INFO] [stdout] 36 | let mut locs = 0; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/bloom/bbloom.rs:133:9 [INFO] [stdout] | [INFO] [stdout] 133 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_unsafe)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/bloom/bbloom.rs:153:32 [INFO] [stdout] | [INFO] [stdout] 153 | bj.filter_set[i] = unsafe { ptr.wrapping_offset(i as isize) as u8 } [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function cannot return without recursing [INFO] [stdout] --> src/store.rs:71:5 [INFO] [stdout] | [INFO] [stdout] 71 | fn deref_mut(&mut self) -> &mut Self::Target { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stdout] 72 | self [INFO] [stdout] | ---- recursive call site [INFO] [stdout] | [INFO] [stdout] = help: a `loop` may express intention better if this is on purpose [INFO] [stdout] = note: `#[warn(unconditional_recursion)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `expiration` is never read [INFO] [stdout] --> src/cache.rs:592:17 [INFO] [stdout] | [INFO] [stdout] 592 | let mut expiration: Option = None; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function cannot return without recursing [INFO] [stdout] --> src/cache.rs:742:5 [INFO] [stdout] | [INFO] [stdout] 742 | pub fn clear<'g>(&'g self, guard: &'g Guard) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stdout] ... [INFO] [stdout] 761 | self.clear(guard); [INFO] [stdout] | ----------------- recursive call site [INFO] [stdout] | [INFO] [stdout] = help: a `loop` may express intention better if this is on purpose [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/policy.rs:191:17 [INFO] [stdout] | [INFO] [stdout] 191 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:46:25 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] 45 | let p = self.cons.load(Ordering::SeqCst, guard); [INFO] [stdout] 46 | let p = unsafe {p.as_ptr()}; [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:47:25 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 47 | let p = unsafe {p.as_mut().unwrap()}; [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:49:39 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 49 | if data.is_null() || !unsafe { data.deref() }.is_empty() { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `size_exp` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:19:5 [INFO] [stdout] | [INFO] [stdout] 16 | pub struct Bloom { [INFO] [stdout] | ----- field in this struct [INFO] [stdout] ... [INFO] [stdout] 19 | size_exp: u64, [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `json_encoder` is never used [INFO] [stdout] --> src/bloom/bbloom.rs:145:8 [INFO] [stdout] | [INFO] [stdout] 32 | impl Bloom { [INFO] [stdout] | ---------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 145 | fn json_encoder(&mut self) -> Vec { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `any_as_u8_slice` is never used [INFO] [stdout] --> src/bloom/rutil.rs:7:11 [INFO] [stdout] | [INFO] [stdout] 7 | unsafe fn any_as_u8_slice(p: &T) -> &[u8] { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `is_string` is never used [INFO] [stdout] --> src/bloom/hasher.rs:58:4 [INFO] [stdout] | [INFO] [stdout] 58 | fn is_string(_s: &T) -> bool { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `new` is never used [INFO] [stdout] --> src/store.rs:23:19 [INFO] [stdout] | [INFO] [stdout] 22 | impl Node { [INFO] [stdout] | --------------- associated function in this implementation [INFO] [stdout] 23 | pub(crate) fn new(key: u64, conflict: u64, value: AV, expiration: Option) -> Self [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `into_box` and `compare_exchange` are never used [INFO] [stdout] --> src/reclaim.rs:23:26 [INFO] [stdout] | [INFO] [stdout] 10 | impl Atomic { [INFO] [stdout] | ----------------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 23 | pub(crate) unsafe fn into_box(self) -> Box> { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 36 | pub(crate) fn compare_exchange<'g>( [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `CompareExchangeError` is never constructed [INFO] [stdout] --> src/reclaim.rs:81:19 [INFO] [stdout] | [INFO] [stdout] 81 | pub(crate) struct CompareExchangeError<'g, T> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `RetireShared` is never used [INFO] [stdout] --> src/reclaim.rs:146:18 [INFO] [stdout] | [INFO] [stdout] 146 | pub(crate) trait RetireShared { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `GuardRef` is never used [INFO] [stdout] --> src/reclaim.rs:156:17 [INFO] [stdout] | [INFO] [stdout] 156 | pub(crate) enum GuardRef<'g> { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `get` and `SetsDropped` are never used [INFO] [stdout] --> src/cache.rs:917:19 [INFO] [stdout] | [INFO] [stdout] 910 | impl Metrics { [INFO] [stdout] | ------------ methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 917 | pub(crate) fn get<'g>(&'g self, t: MetricType, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 931 | pub(crate) fn SetsDropped<'g>(&'g self, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `Policy` is never used [INFO] [stdout] --> src/policy.rs:16:11 [INFO] [stdout] | [INFO] [stdout] 16 | pub trait Policy { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `number_counters` and `max_cost` are never read [INFO] [stdout] --> src/policy.rs:46:5 [INFO] [stdout] | [INFO] [stdout] 40 | pub struct DefaultPolicy { [INFO] [stdout] | ------------- fields in this struct [INFO] [stdout] ... [INFO] [stdout] 46 | number_counters: i64, [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] 47 | lock: Mutex<()>, [INFO] [stdout] 48 | max_cost: i64, [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `has`, `close`, and `cap` are never used [INFO] [stdout] --> src/policy.rs:218:12 [INFO] [stdout] | [INFO] [stdout] 53 | impl DefaultPolicy { [INFO] [stdout] | ------------------------ methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 218 | pub fn has(&self, key: u64, _guard: &Guard) -> bool { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 236 | pub fn close(&mut self) { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 246 | pub fn cap(&self) -> i64 { [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `reset` is never used [INFO] [stdout] --> src/cmsketch.rs:58:13 [INFO] [stdout] | [INFO] [stdout] 16 | impl CmSketch { [INFO] [stdout] | ------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 58 | pub fn reset(&mut self) { [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `reset` and `string` are never used [INFO] [stdout] --> src/cmsketch.rs:85:8 [INFO] [stdout] | [INFO] [stdout] 71 | impl CmRows { [INFO] [stdout] | ----------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 85 | fn reset(&mut self) { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 91 | fn string(&self) -> String { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type alias `RingConsumer` is never used [INFO] [stdout] --> src/ring.rs:10:10 [INFO] [stdout] | [INFO] [stdout] 10 | pub type RingConsumer = Box) -> bool>; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `getSize` should have a snake case name [INFO] [stdout] --> src/bloom/bbloom.rs:170:4 [INFO] [stdout] | [INFO] [stdout] 170 | fn getSize(mut u_i64: u64) -> (u64, u64) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `get_size` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/bloom/haskey.rs:17:19 [INFO] [stdout] | [INFO] [stdout] 17 | let key = *key.deref(); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `T` does not implement `Deref`, so calling `deref` on `&T` 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: comparison is useless due to type limits [INFO] [stdout] --> src/cache.rs:593:12 [INFO] [stdout] | [INFO] [stdout] 593 | if ttl.as_millis() < 0 { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_comparisons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `delVal` should have a snake case name [INFO] [stdout] --> src/cache.rs:666:29 [INFO] [stdout] | [INFO] [stdout] 666 | let delVal = dstore.del(&victims[i].key, &0, guard); [INFO] [stdout] | ^^^^^^ help: convert the identifier to snake case: `del_val` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/cache.rs:675:125 [INFO] [stdout] | [INFO] [stdout] 675 | ... (self.on_evict.unwrap())(victims[i].key, victims[i].conflict, unsafe { v.deref().deref().deref() }, victims[i].cost) [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `V` does not implement `Deref`, so calling `deref` on `&V` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `delVal` should have a snake case name [INFO] [stdout] --> src/cache.rs:811:29 [INFO] [stdout] | [INFO] [stdout] 811 | let delVal = store.del(&victims[i].key, &0, guard); [INFO] [stdout] | ^^^^^^ help: convert the identifier to snake case: `del_val` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/cache.rs:820:125 [INFO] [stdout] | [INFO] [stdout] 820 | ... (self.on_evict.unwrap())(victims[i].key, victims[i].conflict, unsafe { v.deref().deref().deref() }, victims[i].cost) [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `V` does not implement `Deref`, so calling `deref` on `&V` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `SetsDropped` should have a snake case name [INFO] [stdout] --> src/cache.rs:931:19 [INFO] [stdout] | [INFO] [stdout] 931 | pub(crate) fn SetsDropped<'g>(&'g self, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^^^^^^^^^ help: convert the identifier to snake case: `sets_dropped` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `cmDepth` should have an upper case name [INFO] [stdout] --> src/cmsketch.rs:6:7 [INFO] [stdout] | [INFO] [stdout] 6 | const cmDepth: usize = 4; [INFO] [stdout] | ^^^^^^^ help: convert the identifier to upper case: `CM_DEPTH` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_upper_case_globals)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 31.84s [INFO] running `Command { std: "docker" "inspect" "b51d46420898a05fde2f8bec801d0987f01e45b8ab663101bf377fdb4ba93d2b", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "b51d46420898a05fde2f8bec801d0987f01e45b8ab663101bf377fdb4ba93d2b", kill_on_drop: false }` [INFO] [stdout] b51d46420898a05fde2f8bec801d0987f01e45b8ab663101bf377fdb4ba93d2b [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:6252d7ea7fb8caaf7af6d800c5fb250a9cd862b9a7f9508afb3c54fa7fe1102e" "/opt/rustwide/cargo-home/bin/cargo" "+f23b22458132fb20faeb8a915b1c252478696f25" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 74c9483be7b6fe97c21be2f3911870c6273fd35607b34d27c90b5616dad7646f [INFO] running `Command { std: "docker" "start" "-a" "74c9483be7b6fe97c21be2f3911870c6273fd35607b34d27c90b5616dad7646f", kill_on_drop: false }` [INFO] [stderr] Compiling crossbeam-utils v0.8.20 [INFO] [stderr] Compiling num-traits v0.2.19 [INFO] [stderr] Compiling rayon-core v1.12.1 [INFO] [stderr] Compiling plotters-backend v0.3.7 [INFO] [stderr] Compiling regex-syntax v0.8.5 [INFO] [stderr] Compiling textwrap v0.11.0 [INFO] [stderr] Compiling csv-core v0.1.11 [INFO] [stderr] Compiling half v1.8.3 [INFO] [stderr] Compiling cast v0.3.0 [INFO] [stderr] Compiling clap v2.34.0 [INFO] [stderr] Compiling plotters-svg v0.3.7 [INFO] [stderr] Compiling criterion-plot v0.4.5 [INFO] [stderr] Compiling serde_cbor v0.11.2 [INFO] [stderr] Compiling csv v1.3.0 [INFO] [stderr] Compiling tinytemplate v1.2.1 [INFO] [stderr] Compiling crossbeam-epoch v0.9.18 [INFO] [stderr] Compiling crossbeam-channel v0.5.13 [INFO] [stderr] Compiling chrono v0.4.38 [INFO] [stderr] Compiling crossbeam-queue v0.3.11 [INFO] [stderr] Compiling plotters v0.3.7 [INFO] [stderr] Compiling crossbeam-deque v0.8.5 [INFO] [stderr] Compiling oorandom v11.1.4 [INFO] [stderr] Compiling crossbeam v0.8.4 [INFO] [stderr] Compiling regex-automata v0.4.8 [INFO] [stderr] Compiling rayon v1.10.0 [INFO] [stderr] Compiling timer v0.2.0 [INFO] [stderr] Compiling memory-cache-rust v0.1.0-alpha (/opt/rustwide/workdir) [INFO] [stdout] warning: unnecessary parentheses around index expression [INFO] [stdout] --> src/cmsketch.rs:94:51 [INFO] [stdout] | [INFO] [stdout] 94 | s.push_str(format!("{:#02} ", (self.0[(i / 2)] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 94 - s.push_str(format!("{:#02} ", (self.0[(i / 2)] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] 94 + s.push_str(format!("{:#02} ", (self.0[i / 2] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable statement [INFO] [stdout] --> src/ttl.rs:96:13 [INFO] [stdout] | [INFO] [stdout] 94 | break; [INFO] [stdout] | ----- any code following this expression is unreachable [INFO] [stdout] 95 | [INFO] [stdout] 96 | break; [INFO] [stdout] | ^^^^^^ unreachable statement [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unreachable_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `syncpool::prelude` [INFO] [stdout] --> src/ring.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use syncpool::prelude::*; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Add` [INFO] [stdout] --> src/cache.rs:6:16 [INFO] [stdout] | [INFO] [stdout] 6 | use std::ops::{Add, Deref}; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `entries` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:35:17 [INFO] [stdout] | [INFO] [stdout] 35 | let mut entries = 0; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `locs` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:36:17 [INFO] [stdout] | [INFO] [stdout] 36 | let mut locs = 0; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/bloom/bbloom.rs:133:9 [INFO] [stdout] | [INFO] [stdout] 133 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_unsafe)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/bloom/bbloom.rs:153:32 [INFO] [stdout] | [INFO] [stdout] 153 | bj.filter_set[i] = unsafe { ptr.wrapping_offset(i as isize) as u8 } [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function cannot return without recursing [INFO] [stdout] --> src/store.rs:71:5 [INFO] [stdout] | [INFO] [stdout] 71 | fn deref_mut(&mut self) -> &mut Self::Target { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stdout] 72 | self [INFO] [stdout] | ---- recursive call site [INFO] [stdout] | [INFO] [stdout] = help: a `loop` may express intention better if this is on purpose [INFO] [stdout] = note: `#[warn(unconditional_recursion)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `expiration` is never read [INFO] [stdout] --> src/cache.rs:592:17 [INFO] [stdout] | [INFO] [stdout] 592 | let mut expiration: Option = None; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function cannot return without recursing [INFO] [stdout] --> src/cache.rs:742:5 [INFO] [stdout] | [INFO] [stdout] 742 | pub fn clear<'g>(&'g self, guard: &'g Guard) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stdout] ... [INFO] [stdout] 761 | self.clear(guard); [INFO] [stdout] | ----------------- recursive call site [INFO] [stdout] | [INFO] [stdout] = help: a `loop` may express intention better if this is on purpose [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/policy.rs:191:17 [INFO] [stdout] | [INFO] [stdout] 191 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:46:25 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] 45 | let p = self.cons.load(Ordering::SeqCst, guard); [INFO] [stdout] 46 | let p = unsafe {p.as_ptr()}; [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:47:25 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 47 | let p = unsafe {p.as_mut().unwrap()}; [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:49:39 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 49 | if data.is_null() || !unsafe { data.deref() }.is_empty() { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `size_exp` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:19:5 [INFO] [stdout] | [INFO] [stdout] 16 | pub struct Bloom { [INFO] [stdout] | ----- field in this struct [INFO] [stdout] ... [INFO] [stdout] 19 | size_exp: u64, [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `json_encoder` is never used [INFO] [stdout] --> src/bloom/bbloom.rs:145:8 [INFO] [stdout] | [INFO] [stdout] 32 | impl Bloom { [INFO] [stdout] | ---------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 145 | fn json_encoder(&mut self) -> Vec { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `any_as_u8_slice` is never used [INFO] [stdout] --> src/bloom/rutil.rs:7:11 [INFO] [stdout] | [INFO] [stdout] 7 | unsafe fn any_as_u8_slice(p: &T) -> &[u8] { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `is_string` is never used [INFO] [stdout] --> src/bloom/hasher.rs:58:4 [INFO] [stdout] | [INFO] [stdout] 58 | fn is_string(_s: &T) -> bool { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `new` is never used [INFO] [stdout] --> src/store.rs:23:19 [INFO] [stdout] | [INFO] [stdout] 22 | impl Node { [INFO] [stdout] | --------------- associated function in this implementation [INFO] [stdout] 23 | pub(crate) fn new(key: u64, conflict: u64, value: AV, expiration: Option) -> Self [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `into_box` and `compare_exchange` are never used [INFO] [stdout] --> src/reclaim.rs:23:26 [INFO] [stdout] | [INFO] [stdout] 10 | impl Atomic { [INFO] [stdout] | ----------------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 23 | pub(crate) unsafe fn into_box(self) -> Box> { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 36 | pub(crate) fn compare_exchange<'g>( [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `CompareExchangeError` is never constructed [INFO] [stdout] --> src/reclaim.rs:81:19 [INFO] [stdout] | [INFO] [stdout] 81 | pub(crate) struct CompareExchangeError<'g, T> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `RetireShared` is never used [INFO] [stdout] --> src/reclaim.rs:146:18 [INFO] [stdout] | [INFO] [stdout] 146 | pub(crate) trait RetireShared { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `GuardRef` is never used [INFO] [stdout] --> src/reclaim.rs:156:17 [INFO] [stdout] | [INFO] [stdout] 156 | pub(crate) enum GuardRef<'g> { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `get` and `SetsDropped` are never used [INFO] [stdout] --> src/cache.rs:917:19 [INFO] [stdout] | [INFO] [stdout] 910 | impl Metrics { [INFO] [stdout] | ------------ methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 917 | pub(crate) fn get<'g>(&'g self, t: MetricType, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 931 | pub(crate) fn SetsDropped<'g>(&'g self, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `Policy` is never used [INFO] [stdout] --> src/policy.rs:16:11 [INFO] [stdout] | [INFO] [stdout] 16 | pub trait Policy { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `number_counters` and `max_cost` are never read [INFO] [stdout] --> src/policy.rs:46:5 [INFO] [stdout] | [INFO] [stdout] 40 | pub struct DefaultPolicy { [INFO] [stdout] | ------------- fields in this struct [INFO] [stdout] ... [INFO] [stdout] 46 | number_counters: i64, [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] 47 | lock: Mutex<()>, [INFO] [stdout] 48 | max_cost: i64, [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `has`, `close`, and `cap` are never used [INFO] [stdout] --> src/policy.rs:218:12 [INFO] [stdout] | [INFO] [stdout] 53 | impl DefaultPolicy { [INFO] [stdout] | ------------------------ methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 218 | pub fn has(&self, key: u64, _guard: &Guard) -> bool { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 236 | pub fn close(&mut self) { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 246 | pub fn cap(&self) -> i64 { [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `reset` is never used [INFO] [stdout] --> src/cmsketch.rs:58:13 [INFO] [stdout] | [INFO] [stdout] 16 | impl CmSketch { [INFO] [stdout] | ------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 58 | pub fn reset(&mut self) { [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `reset` and `string` are never used [INFO] [stdout] --> src/cmsketch.rs:85:8 [INFO] [stdout] | [INFO] [stdout] 71 | impl CmRows { [INFO] [stdout] | ----------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 85 | fn reset(&mut self) { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 91 | fn string(&self) -> String { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type alias `RingConsumer` is never used [INFO] [stdout] --> src/ring.rs:10:10 [INFO] [stdout] | [INFO] [stdout] 10 | pub type RingConsumer = Box) -> bool>; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `getSize` should have a snake case name [INFO] [stdout] --> src/bloom/bbloom.rs:170:4 [INFO] [stdout] | [INFO] [stdout] 170 | fn getSize(mut u_i64: u64) -> (u64, u64) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `get_size` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/bloom/haskey.rs:17:19 [INFO] [stdout] | [INFO] [stdout] 17 | let key = *key.deref(); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `T` does not implement `Deref`, so calling `deref` on `&T` 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: comparison is useless due to type limits [INFO] [stdout] --> src/cache.rs:593:12 [INFO] [stdout] | [INFO] [stdout] 593 | if ttl.as_millis() < 0 { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_comparisons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `delVal` should have a snake case name [INFO] [stdout] --> src/cache.rs:666:29 [INFO] [stdout] | [INFO] [stdout] 666 | let delVal = dstore.del(&victims[i].key, &0, guard); [INFO] [stdout] | ^^^^^^ help: convert the identifier to snake case: `del_val` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/cache.rs:675:125 [INFO] [stdout] | [INFO] [stdout] 675 | ... (self.on_evict.unwrap())(victims[i].key, victims[i].conflict, unsafe { v.deref().deref().deref() }, victims[i].cost) [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `V` does not implement `Deref`, so calling `deref` on `&V` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `delVal` should have a snake case name [INFO] [stdout] --> src/cache.rs:811:29 [INFO] [stdout] | [INFO] [stdout] 811 | let delVal = store.del(&victims[i].key, &0, guard); [INFO] [stdout] | ^^^^^^ help: convert the identifier to snake case: `del_val` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/cache.rs:820:125 [INFO] [stdout] | [INFO] [stdout] 820 | ... (self.on_evict.unwrap())(victims[i].key, victims[i].conflict, unsafe { v.deref().deref().deref() }, victims[i].cost) [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `V` does not implement `Deref`, so calling `deref` on `&V` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `SetsDropped` should have a snake case name [INFO] [stdout] --> src/cache.rs:931:19 [INFO] [stdout] | [INFO] [stdout] 931 | pub(crate) fn SetsDropped<'g>(&'g self, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^^^^^^^^^ help: convert the identifier to snake case: `sets_dropped` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `cmDepth` should have an upper case name [INFO] [stdout] --> src/cmsketch.rs:6:7 [INFO] [stdout] | [INFO] [stdout] 6 | const cmDepth: usize = 4; [INFO] [stdout] | ^^^^^^^ help: convert the identifier to upper case: `CM_DEPTH` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_upper_case_globals)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Compiling regex v1.11.0 [INFO] [stderr] Compiling criterion v0.3.6 [INFO] [stdout] warning: unnecessary trailing semicolon [INFO] [stdout] --> src/bloom/haskey.rs:173:9 [INFO] [stdout] | [INFO] [stdout] 173 | ; [INFO] [stdout] | ^ help: remove this semicolon [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(redundant_semicolons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around index expression [INFO] [stdout] --> src/cmsketch.rs:94:51 [INFO] [stdout] | [INFO] [stdout] 94 | s.push_str(format!("{:#02} ", (self.0[(i / 2)] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 94 - s.push_str(format!("{:#02} ", (self.0[(i / 2)] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] 94 + s.push_str(format!("{:#02} ", (self.0[i / 2] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable statement [INFO] [stdout] --> src/ttl.rs:96:13 [INFO] [stdout] | [INFO] [stdout] 94 | break; [INFO] [stdout] | ----- any code following this expression is unreachable [INFO] [stdout] 95 | [INFO] [stdout] 96 | break; [INFO] [stdout] | ^^^^^^ unreachable statement [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unreachable_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `syncpool::prelude` [INFO] [stdout] --> src/ring.rs:5:5 [INFO] [stdout] | [INFO] [stdout] 5 | use syncpool::prelude::*; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Add` [INFO] [stdout] --> src/cache.rs:6:16 [INFO] [stdout] | [INFO] [stdout] 6 | use std::ops::{Add, Deref}; [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `entries` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:35:17 [INFO] [stdout] | [INFO] [stdout] 35 | let mut entries = 0; [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] = note: `#[warn(unused_assignments)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `locs` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:36:17 [INFO] [stdout] | [INFO] [stdout] 36 | let mut locs = 0; [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/bloom/bbloom.rs:133:9 [INFO] [stdout] | [INFO] [stdout] 133 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_unsafe)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/bloom/bbloom.rs:153:32 [INFO] [stdout] | [INFO] [stdout] 153 | bj.filter_set[i] = unsafe { ptr.wrapping_offset(i as isize) as u8 } [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function cannot return without recursing [INFO] [stdout] --> src/store.rs:71:5 [INFO] [stdout] | [INFO] [stdout] 71 | fn deref_mut(&mut self) -> &mut Self::Target { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stdout] 72 | self [INFO] [stdout] | ---- recursive call site [INFO] [stdout] | [INFO] [stdout] = help: a `loop` may express intention better if this is on purpose [INFO] [stdout] = note: `#[warn(unconditional_recursion)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: value assigned to `expiration` is never read [INFO] [stdout] --> src/cache.rs:592:17 [INFO] [stdout] | [INFO] [stdout] 592 | let mut expiration: Option = None; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: maybe it is overwritten before being read? [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function cannot return without recursing [INFO] [stdout] --> src/cache.rs:742:5 [INFO] [stdout] | [INFO] [stdout] 742 | pub fn clear<'g>(&'g self, guard: &'g Guard) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stdout] ... [INFO] [stdout] 761 | self.clear(guard); [INFO] [stdout] | ----------------- recursive call site [INFO] [stdout] | [INFO] [stdout] = help: a `loop` may express intention better if this is on purpose [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/policy.rs:191:17 [INFO] [stdout] | [INFO] [stdout] 191 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:46:25 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] 45 | let p = self.cons.load(Ordering::SeqCst, guard); [INFO] [stdout] 46 | let p = unsafe {p.as_ptr()}; [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:47:25 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 47 | let p = unsafe {p.as_mut().unwrap()}; [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/ring.rs:49:39 [INFO] [stdout] | [INFO] [stdout] 44 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 49 | if data.is_null() || !unsafe { data.deref() }.is_empty() { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `size_exp` is never read [INFO] [stdout] --> src/bloom/bbloom.rs:19:5 [INFO] [stdout] | [INFO] [stdout] 16 | pub struct Bloom { [INFO] [stdout] | ----- field in this struct [INFO] [stdout] ... [INFO] [stdout] 19 | size_exp: u64, [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `json_encoder` is never used [INFO] [stdout] --> src/bloom/bbloom.rs:145:8 [INFO] [stdout] | [INFO] [stdout] 32 | impl Bloom { [INFO] [stdout] | ---------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 145 | fn json_encoder(&mut self) -> Vec { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `any_as_u8_slice` is never used [INFO] [stdout] --> src/bloom/rutil.rs:7:11 [INFO] [stdout] | [INFO] [stdout] 7 | unsafe fn any_as_u8_slice(p: &T) -> &[u8] { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `is_string` is never used [INFO] [stdout] --> src/bloom/hasher.rs:58:4 [INFO] [stdout] | [INFO] [stdout] 58 | fn is_string(_s: &T) -> bool { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ITER` is never used [INFO] [stdout] --> src/store.rs:283:11 [INFO] [stdout] | [INFO] [stdout] 283 | const ITER: u64 = 32 * 1024; [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `into_box` and `compare_exchange` are never used [INFO] [stdout] --> src/reclaim.rs:23:26 [INFO] [stdout] | [INFO] [stdout] 10 | impl Atomic { [INFO] [stdout] | ----------------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 23 | pub(crate) unsafe fn into_box(self) -> Box> { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 36 | pub(crate) fn compare_exchange<'g>( [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `CompareExchangeError` is never constructed [INFO] [stdout] --> src/reclaim.rs:81:19 [INFO] [stdout] | [INFO] [stdout] 81 | pub(crate) struct CompareExchangeError<'g, T> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `RetireShared` is never used [INFO] [stdout] --> src/reclaim.rs:146:18 [INFO] [stdout] | [INFO] [stdout] 146 | pub(crate) trait RetireShared { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `GuardRef` is never used [INFO] [stdout] --> src/reclaim.rs:156:17 [INFO] [stdout] | [INFO] [stdout] 156 | pub(crate) enum GuardRef<'g> { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `get` and `SetsDropped` are never used [INFO] [stdout] --> src/cache.rs:917:19 [INFO] [stdout] | [INFO] [stdout] 910 | impl Metrics { [INFO] [stdout] | ------------ methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 917 | pub(crate) fn get<'g>(&'g self, t: MetricType, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 931 | pub(crate) fn SetsDropped<'g>(&'g self, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0` and `1` are never read [INFO] [stdout] --> src/cache.rs:1199:17 [INFO] [stdout] | [INFO] [stdout] 1199 | struct Pair(Box, Box); [INFO] [stdout] | ---- ^^^^^^^^ ^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this struct [INFO] [stdout] | [INFO] [stdout] = help: consider removing these fields [INFO] [stdout] = note: `Pair` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `Policy` is never used [INFO] [stdout] --> src/policy.rs:16:11 [INFO] [stdout] | [INFO] [stdout] 16 | pub trait Policy { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `number_counters` and `max_cost` are never read [INFO] [stdout] --> src/policy.rs:46:5 [INFO] [stdout] | [INFO] [stdout] 40 | pub struct DefaultPolicy { [INFO] [stdout] | ------------- fields in this struct [INFO] [stdout] ... [INFO] [stdout] 46 | number_counters: i64, [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] 47 | lock: Mutex<()>, [INFO] [stdout] 48 | max_cost: i64, [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `close` is never used [INFO] [stdout] --> src/policy.rs:236:12 [INFO] [stdout] | [INFO] [stdout] 53 | impl DefaultPolicy { [INFO] [stdout] | ------------------------ method in this implementation [INFO] [stdout] ... [INFO] [stdout] 236 | pub fn close(&mut self) { [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: type alias `RingConsumer` is never used [INFO] [stdout] --> src/ring.rs:10:10 [INFO] [stdout] | [INFO] [stdout] 10 | pub type RingConsumer = Box) -> bool>; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `getSize` should have a snake case name [INFO] [stdout] --> src/bloom/bbloom.rs:170:4 [INFO] [stdout] | [INFO] [stdout] 170 | fn getSize(mut u_i64: u64) -> (u64, u64) { [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case: `get_size` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/bloom/haskey.rs:17:19 [INFO] [stdout] | [INFO] [stdout] 17 | let key = *key.deref(); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `T` does not implement `Deref`, so calling `deref` on `&T` 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: comparison is useless due to type limits [INFO] [stdout] --> src/cache.rs:593:12 [INFO] [stdout] | [INFO] [stdout] 593 | if ttl.as_millis() < 0 { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_comparisons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `delVal` should have a snake case name [INFO] [stdout] --> src/cache.rs:666:29 [INFO] [stdout] | [INFO] [stdout] 666 | let delVal = dstore.del(&victims[i].key, &0, guard); [INFO] [stdout] | ^^^^^^ help: convert the identifier to snake case: `del_val` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/cache.rs:675:125 [INFO] [stdout] | [INFO] [stdout] 675 | ... (self.on_evict.unwrap())(victims[i].key, victims[i].conflict, unsafe { v.deref().deref().deref() }, victims[i].cost) [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `V` does not implement `Deref`, so calling `deref` on `&V` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `delVal` should have a snake case name [INFO] [stdout] --> src/cache.rs:811:29 [INFO] [stdout] | [INFO] [stdout] 811 | let delVal = store.del(&victims[i].key, &0, guard); [INFO] [stdout] | ^^^^^^ help: convert the identifier to snake case: `del_val` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stdout] --> src/cache.rs:820:125 [INFO] [stdout] | [INFO] [stdout] 820 | ... (self.on_evict.unwrap())(victims[i].key, victims[i].conflict, unsafe { v.deref().deref().deref() }, victims[i].cost) [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `V` does not implement `Deref`, so calling `deref` on `&V` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `SetsDropped` should have a snake case name [INFO] [stdout] --> src/cache.rs:931:19 [INFO] [stdout] | [INFO] [stdout] 931 | pub(crate) fn SetsDropped<'g>(&'g self, guard: &'g Guard) -> u64 { [INFO] [stdout] | ^^^^^^^^^^^ help: convert the identifier to snake case: `sets_dropped` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/cache.rs:1080:9 [INFO] [stdout] | [INFO] [stdout] 1080 | t1.join(); [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 1080 | let _ = t1.join(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/cache.rs:1081:9 [INFO] [stdout] | [INFO] [stdout] 1081 | t2.join(); [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 1081 | let _ = t2.join(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/cache.rs:1082:9 [INFO] [stdout] | [INFO] [stdout] 1082 | t3.join(); [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 1082 | let _ = t3.join(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/cache.rs:1083:9 [INFO] [stdout] | [INFO] [stdout] 1083 | t4.join(); [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 1083 | let _ = t4.join(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:467:9 [INFO] [stdout] | [INFO] [stdout] 467 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] = note: `#[warn(dropping_copy_types)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:519:9 [INFO] [stdout] | [INFO] [stdout] 519 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:534:9 [INFO] [stdout] | [INFO] [stdout] 534 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:552:9 [INFO] [stdout] | [INFO] [stdout] 552 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:570:9 [INFO] [stdout] | [INFO] [stdout] 570 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:594:9 [INFO] [stdout] | [INFO] [stdout] 594 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:612:9 [INFO] [stdout] | [INFO] [stdout] 612 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:631:9 [INFO] [stdout] | [INFO] [stdout] 631 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:650:9 [INFO] [stdout] | [INFO] [stdout] 650 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stdout] --> src/policy.rs:670:9 [INFO] [stdout] | [INFO] [stdout] 670 | drop(Box::into_raw(shard_metric)) [INFO] [stdout] | ^^^^^---------------------------^ [INFO] [stdout] | | [INFO] [stdout] | argument has type `*mut Metrics` [INFO] [stdout] | [INFO] [stdout] = note: use `let _ = ...` to ignore the expression or result [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `cmDepth` should have an upper case name [INFO] [stdout] --> src/cmsketch.rs:6:7 [INFO] [stdout] | [INFO] [stdout] 6 | const cmDepth: usize = 4; [INFO] [stdout] | ^^^^^^^ help: convert the identifier to upper case: `CM_DEPTH` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_upper_case_globals)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `TestSketchClear` should have a snake case name [INFO] [stdout] --> src/cmsketch.rs:132:8 [INFO] [stdout] | [INFO] [stdout] 132 | fn TestSketchClear() { [INFO] [stdout] | ^^^^^^^^^^^^^^^ help: convert the identifier to snake case: `test_sketch_clear` [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `test` profile [unoptimized + debuginfo] target(s) in 35.08s [INFO] running `Command { std: "docker" "inspect" "74c9483be7b6fe97c21be2f3911870c6273fd35607b34d27c90b5616dad7646f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "74c9483be7b6fe97c21be2f3911870c6273fd35607b34d27c90b5616dad7646f", kill_on_drop: false }` [INFO] [stdout] 74c9483be7b6fe97c21be2f3911870c6273fd35607b34d27c90b5616dad7646f [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:6252d7ea7fb8caaf7af6d800c5fb250a9cd862b9a7f9508afb3c54fa7fe1102e" "/opt/rustwide/cargo-home/bin/cargo" "+f23b22458132fb20faeb8a915b1c252478696f25" "test" "--frozen", kill_on_drop: false }` [INFO] [stdout] b885bf63622a06f5d81ce7d276685cb0d0ce0ecc65bdef22a27852e68ba0fd58 [INFO] running `Command { std: "docker" "start" "-a" "b885bf63622a06f5d81ce7d276685cb0d0ce0ecc65bdef22a27852e68ba0fd58", kill_on_drop: false }` [INFO] [stderr] warning: unnecessary parentheses around index expression [INFO] [stderr] --> src/cmsketch.rs:94:51 [INFO] [stderr] | [INFO] [stderr] 94 | s.push_str(format!("{:#02} ", (self.0[(i / 2)] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stderr] | ^ ^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_parens)]` on by default [INFO] [stderr] help: remove these parentheses [INFO] [stderr] | [INFO] [stderr] 94 - s.push_str(format!("{:#02} ", (self.0[(i / 2)] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stderr] 94 + s.push_str(format!("{:#02} ", (self.0[i / 2] >> ((i & 1) * 4)) & 0x0f).as_str()); [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: unreachable statement [INFO] [stderr] --> src/ttl.rs:96:13 [INFO] [stderr] | [INFO] [stderr] 94 | break; [INFO] [stderr] | ----- any code following this expression is unreachable [INFO] [stderr] 95 | [INFO] [stderr] 96 | break; [INFO] [stderr] | ^^^^^^ unreachable statement [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unreachable_code)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused import: `syncpool::prelude` [INFO] [stderr] --> src/ring.rs:5:5 [INFO] [stderr] | [INFO] [stderr] 5 | use syncpool::prelude::*; [INFO] [stderr] | ^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_imports)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unused import: `Add` [INFO] [stderr] --> src/cache.rs:6:16 [INFO] [stderr] | [INFO] [stderr] 6 | use std::ops::{Add, Deref}; [INFO] [stderr] | ^^^ [INFO] [stderr] [INFO] [stderr] warning: value assigned to `entries` is never read [INFO] [stderr] --> src/bloom/bbloom.rs:35:17 [INFO] [stderr] | [INFO] [stderr] 35 | let mut entries = 0; [INFO] [stderr] | ^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: maybe it is overwritten before being read? [INFO] [stderr] = note: `#[warn(unused_assignments)]` on by default [INFO] [stderr] [INFO] [stderr] warning: value assigned to `locs` is never read [INFO] [stderr] --> src/bloom/bbloom.rs:36:17 [INFO] [stderr] | [INFO] [stderr] 36 | let mut locs = 0; [INFO] [stderr] | ^^^^ [INFO] [stderr] | [INFO] [stderr] = help: maybe it is overwritten before being read? [INFO] [stderr] [INFO] [stderr] warning: unnecessary `unsafe` block [INFO] [stderr] --> src/bloom/bbloom.rs:133:9 [INFO] [stderr] | [INFO] [stderr] 133 | unsafe { [INFO] [stderr] | ^^^^^^ unnecessary `unsafe` block [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_unsafe)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unnecessary `unsafe` block [INFO] [stderr] --> src/bloom/bbloom.rs:153:32 [INFO] [stderr] | [INFO] [stderr] 153 | bj.filter_set[i] = unsafe { ptr.wrapping_offset(i as isize) as u8 } [INFO] [stderr] | ^^^^^^ unnecessary `unsafe` block [INFO] [stderr] [INFO] [stderr] warning: function cannot return without recursing [INFO] [stderr] --> src/store.rs:71:5 [INFO] [stderr] | [INFO] [stderr] 71 | fn deref_mut(&mut self) -> &mut Self::Target { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stderr] 72 | self [INFO] [stderr] | ---- recursive call site [INFO] [stderr] | [INFO] [stderr] = help: a `loop` may express intention better if this is on purpose [INFO] [stderr] = note: `#[warn(unconditional_recursion)]` on by default [INFO] [stderr] [INFO] [stderr] warning: value assigned to `expiration` is never read [INFO] [stderr] --> src/cache.rs:592:17 [INFO] [stderr] | [INFO] [stderr] 592 | let mut expiration: Option = None; [INFO] [stderr] | ^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: maybe it is overwritten before being read? [INFO] [stderr] [INFO] [stderr] warning: function cannot return without recursing [INFO] [stderr] --> src/cache.rs:742:5 [INFO] [stderr] | [INFO] [stderr] 742 | pub fn clear<'g>(&'g self, guard: &'g Guard) { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stderr] ... [INFO] [stderr] 761 | self.clear(guard); [INFO] [stderr] | ----------------- recursive call site [INFO] [stderr] | [INFO] [stderr] = help: a `loop` may express intention better if this is on purpose [INFO] [stderr] [INFO] [stderr] warning: unnecessary `unsafe` block [INFO] [stderr] --> src/policy.rs:191:17 [INFO] [stderr] | [INFO] [stderr] 191 | unsafe { [INFO] [stderr] | ^^^^^^ unnecessary `unsafe` block [INFO] [stderr] [INFO] [stderr] warning: unnecessary `unsafe` block [INFO] [stderr] --> src/ring.rs:46:25 [INFO] [stderr] | [INFO] [stderr] 44 | unsafe { [INFO] [stderr] | ------ because it's nested under this `unsafe` block [INFO] [stderr] 45 | let p = self.cons.load(Ordering::SeqCst, guard); [INFO] [stderr] 46 | let p = unsafe {p.as_ptr()}; [INFO] [stderr] | ^^^^^^ unnecessary `unsafe` block [INFO] [stderr] [INFO] [stderr] warning: unnecessary `unsafe` block [INFO] [stderr] --> src/ring.rs:47:25 [INFO] [stderr] | [INFO] [stderr] 44 | unsafe { [INFO] [stderr] | ------ because it's nested under this `unsafe` block [INFO] [stderr] ... [INFO] [stderr] 47 | let p = unsafe {p.as_mut().unwrap()}; [INFO] [stderr] | ^^^^^^ unnecessary `unsafe` block [INFO] [stderr] [INFO] [stderr] warning: unnecessary `unsafe` block [INFO] [stderr] --> src/ring.rs:49:39 [INFO] [stderr] | [INFO] [stderr] 44 | unsafe { [INFO] [stderr] | ------ because it's nested under this `unsafe` block [INFO] [stderr] ... [INFO] [stderr] 49 | if data.is_null() || !unsafe { data.deref() }.is_empty() { [INFO] [stderr] | ^^^^^^ unnecessary `unsafe` block [INFO] [stderr] [INFO] [stderr] warning: field `size_exp` is never read [INFO] [stderr] --> src/bloom/bbloom.rs:19:5 [INFO] [stderr] | [INFO] [stderr] 16 | pub struct Bloom { [INFO] [stderr] | ----- field in this struct [INFO] [stderr] ... [INFO] [stderr] 19 | size_exp: u64, [INFO] [stderr] | ^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(dead_code)]` on by default [INFO] [stderr] [INFO] [stderr] warning: method `json_encoder` is never used [INFO] [stderr] --> src/bloom/bbloom.rs:145:8 [INFO] [stderr] | [INFO] [stderr] 32 | impl Bloom { [INFO] [stderr] | ---------- method in this implementation [INFO] [stderr] ... [INFO] [stderr] 145 | fn json_encoder(&mut self) -> Vec { [INFO] [stderr] | ^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function `any_as_u8_slice` is never used [INFO] [stderr] --> src/bloom/rutil.rs:7:11 [INFO] [stderr] | [INFO] [stderr] 7 | unsafe fn any_as_u8_slice(p: &T) -> &[u8] { [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function `is_string` is never used [INFO] [stderr] --> src/bloom/hasher.rs:58:4 [INFO] [stderr] | [INFO] [stderr] 58 | fn is_string(_s: &T) -> bool { [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: associated function `new` is never used [INFO] [stderr] --> src/store.rs:23:19 [INFO] [stderr] | [INFO] [stderr] 22 | impl Node { [INFO] [stderr] | --------------- associated function in this implementation [INFO] [stderr] 23 | pub(crate) fn new(key: u64, conflict: u64, value: AV, expiration: Option) -> Self [INFO] [stderr] | ^^^ [INFO] [stderr] [INFO] [stderr] warning: methods `into_box` and `compare_exchange` are never used [INFO] [stderr] --> src/reclaim.rs:23:26 [INFO] [stderr] | [INFO] [stderr] 10 | impl Atomic { [INFO] [stderr] | ----------------- methods in this implementation [INFO] [stderr] ... [INFO] [stderr] 23 | pub(crate) unsafe fn into_box(self) -> Box> { [INFO] [stderr] | ^^^^^^^^ [INFO] [stderr] ... [INFO] [stderr] 36 | pub(crate) fn compare_exchange<'g>( [INFO] [stderr] | ^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: struct `CompareExchangeError` is never constructed [INFO] [stderr] --> src/reclaim.rs:81:19 [INFO] [stderr] | [INFO] [stderr] 81 | pub(crate) struct CompareExchangeError<'g, T> { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: trait `RetireShared` is never used [INFO] [stderr] --> src/reclaim.rs:146:18 [INFO] [stderr] | [INFO] [stderr] 146 | pub(crate) trait RetireShared { [INFO] [stderr] | ^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: enum `GuardRef` is never used [INFO] [stderr] --> src/reclaim.rs:156:17 [INFO] [stderr] | [INFO] [stderr] 156 | pub(crate) enum GuardRef<'g> { [INFO] [stderr] | ^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: methods `get` and `SetsDropped` are never used [INFO] [stderr] --> src/cache.rs:917:19 [INFO] [stderr] | [INFO] [stderr] 910 | impl Metrics { [INFO] [stderr] | ------------ methods in this implementation [INFO] [stderr] ... [INFO] [stderr] 917 | pub(crate) fn get<'g>(&'g self, t: MetricType, guard: &'g Guard) -> u64 { [INFO] [stderr] | ^^^ [INFO] [stderr] ... [INFO] [stderr] 931 | pub(crate) fn SetsDropped<'g>(&'g self, guard: &'g Guard) -> u64 { [INFO] [stderr] | ^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: trait `Policy` is never used [INFO] [stderr] --> src/policy.rs:16:11 [INFO] [stderr] | [INFO] [stderr] 16 | pub trait Policy { [INFO] [stderr] | ^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: fields `number_counters` and `max_cost` are never read [INFO] [stderr] --> src/policy.rs:46:5 [INFO] [stderr] | [INFO] [stderr] 40 | pub struct DefaultPolicy { [INFO] [stderr] | ------------- fields in this struct [INFO] [stderr] ... [INFO] [stderr] 46 | number_counters: i64, [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] 47 | lock: Mutex<()>, [INFO] [stderr] 48 | max_cost: i64, [INFO] [stderr] | ^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: methods `has`, `close`, and `cap` are never used [INFO] [stderr] --> src/policy.rs:218:12 [INFO] [stderr] | [INFO] [stderr] 53 | impl DefaultPolicy { [INFO] [stderr] | ------------------------ methods in this implementation [INFO] [stderr] ... [INFO] [stderr] 218 | pub fn has(&self, key: u64, _guard: &Guard) -> bool { [INFO] [stderr] | ^^^ [INFO] [stderr] ... [INFO] [stderr] 236 | pub fn close(&mut self) { [INFO] [stderr] | ^^^^^ [INFO] [stderr] ... [INFO] [stderr] 246 | pub fn cap(&self) -> i64 { [INFO] [stderr] | ^^^ [INFO] [stderr] [INFO] [stderr] warning: method `reset` is never used [INFO] [stderr] --> src/cmsketch.rs:58:13 [INFO] [stderr] | [INFO] [stderr] 16 | impl CmSketch { [INFO] [stderr] | ------------- method in this implementation [INFO] [stderr] ... [INFO] [stderr] 58 | pub fn reset(&mut self) { [INFO] [stderr] | ^^^^^ [INFO] [stderr] [INFO] [stderr] warning: methods `reset` and `string` are never used [INFO] [stderr] --> src/cmsketch.rs:85:8 [INFO] [stderr] | [INFO] [stderr] 71 | impl CmRows { [INFO] [stderr] | ----------- methods in this implementation [INFO] [stderr] ... [INFO] [stderr] 85 | fn reset(&mut self) { [INFO] [stderr] | ^^^^^ [INFO] [stderr] ... [INFO] [stderr] 91 | fn string(&self) -> String { [INFO] [stderr] | ^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: type alias `RingConsumer` is never used [INFO] [stderr] --> src/ring.rs:10:10 [INFO] [stderr] | [INFO] [stderr] 10 | pub type RingConsumer = Box) -> bool>; [INFO] [stderr] | ^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function `getSize` should have a snake case name [INFO] [stderr] --> src/bloom/bbloom.rs:170:4 [INFO] [stderr] | [INFO] [stderr] 170 | fn getSize(mut u_i64: u64) -> (u64, u64) { [INFO] [stderr] | ^^^^^^^ help: convert the identifier to snake case: `get_size` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(non_snake_case)]` on by default [INFO] [stderr] [INFO] [stderr] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stderr] --> src/bloom/haskey.rs:17:19 [INFO] [stderr] | [INFO] [stderr] 17 | let key = *key.deref(); [INFO] [stderr] | ^^^^^^^^ help: remove this redundant call [INFO] [stderr] | [INFO] [stderr] = note: the type `T` does not implement `Deref`, so calling `deref` on `&T` copies the reference, which does not do anything and can be removed [INFO] [stderr] = note: `#[warn(noop_method_call)]` on by default [INFO] [stderr] [INFO] [stderr] warning: comparison is useless due to type limits [INFO] [stderr] --> src/cache.rs:593:12 [INFO] [stderr] | [INFO] [stderr] 593 | if ttl.as_millis() < 0 { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_comparisons)]` on by default [INFO] [stderr] [INFO] [stderr] warning: variable `delVal` should have a snake case name [INFO] [stderr] --> src/cache.rs:666:29 [INFO] [stderr] | [INFO] [stderr] 666 | let delVal = dstore.del(&victims[i].key, &0, guard); [INFO] [stderr] | ^^^^^^ help: convert the identifier to snake case: `del_val` [INFO] [stderr] [INFO] [stderr] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stderr] --> src/cache.rs:675:125 [INFO] [stderr] | [INFO] [stderr] 675 | ... (self.on_evict.unwrap())(victims[i].key, victims[i].conflict, unsafe { v.deref().deref().deref() }, victims[i].cost) [INFO] [stderr] | ^^^^^^^^ help: remove this redundant call [INFO] [stderr] | [INFO] [stderr] = note: the type `V` does not implement `Deref`, so calling `deref` on `&V` copies the reference, which does not do anything and can be removed [INFO] [stderr] [INFO] [stderr] warning: variable `delVal` should have a snake case name [INFO] [stderr] --> src/cache.rs:811:29 [INFO] [stderr] | [INFO] [stderr] 811 | let delVal = store.del(&victims[i].key, &0, guard); [INFO] [stderr] | ^^^^^^ help: convert the identifier to snake case: `del_val` [INFO] [stderr] [INFO] [stderr] warning: call to `.deref()` on a reference in this situation does nothing [INFO] [stderr] --> src/cache.rs:820:125 [INFO] [stderr] | [INFO] [stderr] 820 | ... (self.on_evict.unwrap())(victims[i].key, victims[i].conflict, unsafe { v.deref().deref().deref() }, victims[i].cost) [INFO] [stderr] | ^^^^^^^^ help: remove this redundant call [INFO] [stderr] | [INFO] [stderr] = note: the type `V` does not implement `Deref`, so calling `deref` on `&V` copies the reference, which does not do anything and can be removed [INFO] [stderr] [INFO] [stderr] warning: method `SetsDropped` should have a snake case name [INFO] [stderr] --> src/cache.rs:931:19 [INFO] [stderr] | [INFO] [stderr] 931 | pub(crate) fn SetsDropped<'g>(&'g self, guard: &'g Guard) -> u64 { [INFO] [stderr] | ^^^^^^^^^^^ help: convert the identifier to snake case: `sets_dropped` [INFO] [stderr] [INFO] [stderr] warning: constant `cmDepth` should have an upper case name [INFO] [stderr] --> src/cmsketch.rs:6:7 [INFO] [stderr] | [INFO] [stderr] 6 | const cmDepth: usize = 4; [INFO] [stderr] | ^^^^^^^ help: convert the identifier to upper case: `CM_DEPTH` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(non_upper_case_globals)]` on by default [INFO] [stderr] [INFO] [stderr] warning: `memory-cache-rust` (lib) generated 40 warnings (run `cargo fix --lib -p memory-cache-rust` to apply 4 suggestions) [INFO] [stderr] warning: unnecessary trailing semicolon [INFO] [stderr] --> src/bloom/haskey.rs:173:9 [INFO] [stderr] | [INFO] [stderr] 173 | ; [INFO] [stderr] | ^ help: remove this semicolon [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(redundant_semicolons)]` on by default [INFO] [stderr] [INFO] [stderr] warning: constant `ITER` is never used [INFO] [stderr] --> src/store.rs:283:11 [INFO] [stderr] | [INFO] [stderr] 283 | const ITER: u64 = 32 * 1024; [INFO] [stderr] | ^^^^ [INFO] [stderr] [INFO] [stderr] warning: fields `0` and `1` are never read [INFO] [stderr] --> src/cache.rs:1199:17 [INFO] [stderr] | [INFO] [stderr] 1199 | struct Pair(Box, Box); [INFO] [stderr] | ---- ^^^^^^^^ ^^^^^^^^ [INFO] [stderr] | | [INFO] [stderr] | fields in this struct [INFO] [stderr] | [INFO] [stderr] = help: consider removing these fields [INFO] [stderr] = note: `Pair` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis [INFO] [stderr] [INFO] [stderr] warning: method `close` is never used [INFO] [stderr] --> src/policy.rs:236:12 [INFO] [stderr] | [INFO] [stderr] 53 | impl DefaultPolicy { [INFO] [stderr] | ------------------------ method in this implementation [INFO] [stderr] ... [INFO] [stderr] 236 | pub fn close(&mut self) { [INFO] [stderr] | ^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/cache.rs:1080:9 [INFO] [stderr] | [INFO] [stderr] 1080 | t1.join(); [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] = note: `#[warn(unused_must_use)]` on by default [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 1080 | let _ = t1.join(); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/cache.rs:1081:9 [INFO] [stderr] | [INFO] [stderr] 1081 | t2.join(); [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 1081 | let _ = t2.join(); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/cache.rs:1082:9 [INFO] [stderr] | [INFO] [stderr] 1082 | t3.join(); [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 1082 | let _ = t3.join(); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/cache.rs:1083:9 [INFO] [stderr] | [INFO] [stderr] 1083 | t4.join(); [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 1083 | let _ = t4.join(); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:467:9 [INFO] [stderr] | [INFO] [stderr] 467 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] = note: `#[warn(dropping_copy_types)]` on by default [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:519:9 [INFO] [stderr] | [INFO] [stderr] 519 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:534:9 [INFO] [stderr] | [INFO] [stderr] 534 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:552:9 [INFO] [stderr] | [INFO] [stderr] 552 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:570:9 [INFO] [stderr] | [INFO] [stderr] 570 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:594:9 [INFO] [stderr] | [INFO] [stderr] 594 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:612:9 [INFO] [stderr] | [INFO] [stderr] 612 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:631:9 [INFO] [stderr] | [INFO] [stderr] 631 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:650:9 [INFO] [stderr] | [INFO] [stderr] 650 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: calls to `std::mem::drop` with a value that implements `Copy` does nothing [INFO] [stderr] --> src/policy.rs:670:9 [INFO] [stderr] | [INFO] [stderr] 670 | drop(Box::into_raw(shard_metric)) [INFO] [stderr] | ^^^^^---------------------------^ [INFO] [stderr] | | [INFO] [stderr] | argument has type `*mut Metrics` [INFO] [stderr] | [INFO] [stderr] = note: use `let _ = ...` to ignore the expression or result [INFO] [stderr] [INFO] [stderr] warning: function `TestSketchClear` should have a snake case name [INFO] [stderr] --> src/cmsketch.rs:132:8 [INFO] [stderr] | [INFO] [stderr] 132 | fn TestSketchClear() { [INFO] [stderr] | ^^^^^^^^^^^^^^^ help: convert the identifier to snake case: `test_sketch_clear` [INFO] [stderr] [INFO] [stderr] warning: `memory-cache-rust` (lib test) generated 55 warnings (36 duplicates) [INFO] [stderr] Finished `test` profile [unoptimized + debuginfo] target(s) in 0.25s [INFO] [stderr] Running unittests src/lib.rs (/opt/rustwide/target/debug/deps/memory_cache_rust-8bf91d16faba2e0a) [INFO] [stdout] [INFO] [stdout] running 37 tests [INFO] [stdout] test bloom::bbloom::tests::oprator_test ... ok [INFO] [stdout] test bloom::haskey::tests::test_hash ... ok [INFO] [stdout] test bloom::hasher::tests::test_hash ... ok [INFO] [stdout] test cache::tests::check ... ok [INFO] [stdout] test bloom::bbloom::tests::test_has ... ok [INFO] [stdout] test cache::tests::testclone ... ok [INFO] [stdout] test cmsketch::tests::TestSketchClear ... ok [INFO] [stdout] test cmsketch::tests::test_next_2_power ... ok [INFO] [stdout] test cmsketch::tests::test_sketch ... ok [INFO] [stdout] test cmsketch::tests::test_sketch_estimate ... ok [INFO] [stdout] test cmsketch::tests::test_sketch_increment ... ok [INFO] [stdout] test cmsketch::tests::test_sketch_reset ... ok [INFO] [stdout] test policy::tests::test_lfu_add ... ok [INFO] [stdout] test policy::tests::test_lfu_clear ... ok [INFO] [stdout] test policy::tests::test_lfu_del ... ok [INFO] [stdout] test policy::tests::test_lfu_update ... ok [INFO] [stdout] test policy::tests::test_policy_cap ... ok [INFO] [stdout] test policy::tests::test_policy_clear ... ok [INFO] [stdout] test policy::tests::test_policy_cost ... ok [INFO] [stdout] test policy::tests::test_policy_del ... ok [INFO] [stdout] test cache::tests::test_sotre_set_get_thread ... ok [INFO] [stdout] test cache::tests::test_cache_key_to_hash ... ok [INFO] [stdout] test policy::tests::test_policy_policy_add ... ok [INFO] [stdout] test policy::tests::test_policy_policy_push ... ok [INFO] [stdout] test store::tests::test_set_clear ... ok [INFO] [stdout] test store::tests::test_set_collision ... ok [INFO] [stdout] test ring::tests::test_ring_drain ... ok [INFO] [stdout] test policy::tests::test_policy_update ... ok [INFO] [stdout] test store::tests::test_set_get ... ok [INFO] [stdout] test store::tests::test_set_update ... ok [INFO] [stdout] test tests::it_works ... ok [INFO] [stdout] test cache::tests::test_cache_insert_thread ... ok [INFO] [stdout] test store::tests::test_set_del ... ok [INFO] [stderr] error: test failed, to rerun pass `--lib` [INFO] [stderr] [INFO] [stderr] Caused by: [INFO] [stderr] process didn't exit successfully: `/opt/rustwide/target/debug/deps/memory_cache_rust-8bf91d16faba2e0a` (signal: 11, SIGSEGV: invalid memory reference) [INFO] running `Command { std: "docker" "inspect" "b885bf63622a06f5d81ce7d276685cb0d0ce0ecc65bdef22a27852e68ba0fd58", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "b885bf63622a06f5d81ce7d276685cb0d0ce0ecc65bdef22a27852e68ba0fd58", kill_on_drop: false }` [INFO] [stdout] b885bf63622a06f5d81ce7d276685cb0d0ce0ecc65bdef22a27852e68ba0fd58