Skip to content
This repository has been archived by the owner on Sep 11, 2023. It is now read-only.
This repository has been archived by the owner on Sep 11, 2023. It is now read-only.

Missing Ord and PartialOrd impls #4

Closed
bill-myers opened this issue Mar 21, 2018 · 2 comments
Closed

Missing Ord and PartialOrd impls #4

bill-myers opened this issue Mar 21, 2018 · 2 comments
Labels
enhancement wontfix

Comments

@bill-myers
Copy link

Ord and PartialOrd impls are missing.
@KamilaBorowska
Copy link
Owner

I believe I did it that way to match HashMap which don't implement an Ord interface. But I guess this is different as in EnumMap has a proper order. Will see if it can be feasibly implemented.

@KamilaBorowska
Copy link
Owner

I don't think there is a reasonable comparison function for enum maps -- note that PartialOrd implementation doesn't have to keep elements in order. If you need this for sorting or whatever, feel free to provide comparison function that compares slices.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement wontfix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@KamilaBorowska @bill-myers