Can we get rid of the std::mem::replace warning?

[drwilco]

My compiler keeps complaining about:

cached/src/stores.rs

Line 630 in 21bd5c6

std::mem::replace(self, HashMap::new());

warning: unused return value of `std::mem::replace` that must be used
   --> src/stores.rs:630:9
    |
630 |         std::mem::replace(self, HashMap::new());
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: `#[warn(unused_must_use)]` on by default
    = note: if you don't need the old value, you can just assign the new value directly

warning: 1 warning emitted

Is there a reason for using std::mem::replace instead of just *self = HashMap::new();?

For convenience, incoming PR

[Stargateur]

None in this case, in my defense it's a new warning or I'm blind.

[drwilco]

It could be new, I'm on 1.45.0. Anyways, PR #53 will get rid of it.