Repository: actix/actix (language: rust)

Issue:
chore: Update hashbrown to 0.5
Closes: #264


Discussion / hints:
could you add entry to changes,md
@fafhrd91 sure, one moment.
@fafhrd91 done!
Looks good to me but the branch is out-of-date.
Note, in rustc 1.36, the `std::collections::HashMap` is hashbrown but with the default `SipHash` instead of `FxHash`. We could potentially just use the one in `std` and use `FxHash` as the hasher (or one of our own choosing) where appropriate. Bringing the hasher implementation internal to this crate could then reduce out the additional dependency.
Reducing dependencies is good. What about the above comment? @fafhrd91 
I think this is good idea
`hashbrown` has been deleted in #277. If updating `derive_more` is still reasonable, we should drop `hashbrown` from here.
I would like to push this branch to an end. @ignatenkobrain could you update the PR and take @JohnTitor's advice? 

Fix patch:
diff --git a/CHANGES.md b/CHANGES.md
index 7879d59b8..8ba99ca53 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -9,6 +9,7 @@
 ### Changed
 
 * Remove `hashbrown` crate from dependencies
+* Upgrade `derive_more` to 0.15.0
 
 ### Fixed
 * Added `#[must_use]` attribute to `ActorFuture` and `ActorStream`
diff --git a/Cargo.toml b/Cargo.toml
index 1536d22fb..c9ae714a4 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -42,7 +42,7 @@ actix-rt = "0.2.2"
 actix_derive = "0.4"
 bytes = "0.4"
 crossbeam-channel = "0.3"
-derive_more = "0.14.0"
+derive_more = "0.15.0"
 futures = "0.1.25"
 log = "0.4"
 lazy_static = "1.2"


Test patch: