-
Notifications
You must be signed in to change notification settings - Fork 168
Full Consistency With HashMap #19
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Comments
#28 implements Ser/De |
I can't find any discrepancies between HashSet and OrderSet. A second close look is of course welcome, but I think @cuviper already did this carefully when implementing OrderSet, so it should already be fine. |
|
Just an oversight, please check all iterators you can and PRs welcome. |
I think we are as close as we can get at this point. New additions to |
Missing methods (stable):
with_hasher
Add with_hasher, hasher methods #35hasher
Add with_hasher, hasher methods #35shrink_to_fit
is_empty
Implement method .is_empty() #22drain
Adds a OrderMap::drain() method #27values
,values_mut
Consistency with HashMap: missingvalues
iterator. #17New methods:
As a rule, we don't implement unstable methods; they are implemented when they are stable in Rust beta
get_key_value
Tracking issue for {HashMap, BTreeMap}::get_key_value stabilization rust-lang/rust#49347Entry::and_modify
(unstable)Entry::or_default
(unstable)try_reserve
Tracking issue fortry_reserve
: RFC 2116 fallible collection allocation rust-lang/rust#48043remove_entry
Tracking issue for HashMap::remove_entry rust-lang/rust#46344Trait impls
PartialEq + Eq
Implement PartialEq and Eq #23Extend<(&K, &V)>
Implement Extend<(&K, &V)> for Copy types #34Type Conflicts:
Entry<K, V>
vsEntry<K, V, S>
(not important) Factor out "OrderMapCore" and other generics bloat saving measures #59retain
's&K
vs&mut K
Add trait MutableKeys #39drain
's signature (not important)OccupiedEntry::insert
's signature Signature of OccupiedEntry::insert #68The text was updated successfully, but these errors were encountered: