-
Notifications
You must be signed in to change notification settings - Fork 614
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
Support sorting of primitive lists by indirect comparison. #908
Comments
This looks fine as a starting point. Please raise a PR. Thanks! |
vmzakharov
added a commit
to vmzakharov/eclipse-collections
that referenced
this issue
Jun 11, 2020
…mparison Signed-off-by: vmzakharov <zakharov.vladimir.m@gmail.com>
vmzakharov
added a commit
to vmzakharov/eclipse-collections
that referenced
this issue
Jun 11, 2020
…mparison Signed-off-by: vmzakharov <zakharov.vladimir.m@gmail.com>
vmzakharov
added a commit
to vmzakharov/eclipse-collections
that referenced
this issue
Jun 12, 2020
…mparison Signed-off-by: vmzakharov <zakharov.vladimir.m@gmail.com>
vmzakharov
added a commit
to vmzakharov/eclipse-collections
that referenced
this issue
Jun 13, 2020
…mparison Signed-off-by: vmzakharov <zakharov.vladimir.m@gmail.com>
vmzakharov
added a commit
to vmzakharov/eclipse-collections
that referenced
this issue
Jun 13, 2020
…mparison Signed-off-by: vmzakharov <zakharov.vladimir.m@gmail.com>
donraab
added a commit
that referenced
this issue
Jun 14, 2020
Closes #908: Support sorting of primitive lists by indirect comparison
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The proposal is to add the following APIs on PrimitiveLists (in addition to the existing
sortThis()
):PrimitiveList sortThis(PrimitiveComparator comparator)
<T> PrimitiveList sortThisBy(PrimitiveFunction<T> keyExtractor) // sorts by the natural order of the keys
<T> PrimitiveList sortThisBy(PrimitiveFunction<T> keyExtractor, Comparator<? super T> keyComparator)
Where
PrimitiveComparator
is a Single Abstract Method interface that looks likeTo support this a new sort implementation will be required.
The text was updated successfully, but these errors were encountered: