Skip to content
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

Fix TODO: Use Comparators.lexicographical instead of Ordering.from in DeclaredMembers #1836

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

panic08
Copy link

@panic08 panic08 commented Oct 24, 2024

Here we fix the TODO and change the use of Ordering.from to the more simplistic Comparators.lexicographical

@panic08 panic08 closed this Oct 24, 2024
@panic08 panic08 force-pushed the fix/comparators-lexicographical branch from 8932496 to d83f677 Compare October 24, 2024 20:21
@panic08 panic08 reopened this Oct 24, 2024
@panic08 panic08 force-pushed the fix/comparators-lexicographical branch from f2a71af to 69b8f5a Compare October 24, 2024 20:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant