-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Full Consistency With HashMap #19
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: