RFR 8009736: Comparator API cleanup (original) (raw)

Jason Mehrens jason_mehrens at hotmail.com
Fri Jun 14 17:10:14 UTC 2013


Any chance the static method Comparator.reverseOrder() will be renamed to something like reversedNaturalOrder? Now all of the calls to Comparator.reverseOrder() take on new meaning.

Jason

Date: Tue, 11 Jun 2013 14:04:38 -0700 From: henry.jen at oracle.com To: lambda-dev at openjdk.java.net; core-libs-dev at openjdk.java.net Subject: RFR 8009736: Comparator API cleanup

Hi, Please review http://cr.openjdk.java.net/~henryjen/ccc/8009736.2/webrev/ Highlight of changes, - Comparators class is now only package-private implementations. The public static methods have been move to other arguably more appropriate places, mostly in Comparator. - Comparator.reverseOrder() is renamed to Comparator.reversed() - nullsFirst(Comparator) and nullsLast(Comparator) are introduced to wrap up a comparator to be null-friendly. To see the API changes, found the specdiff at http://cr.openjdk.java.net/~henryjen/ccc/8009736.2/specdiff/overview-summary.html Cheers, Henry



More information about the core-libs-dev mailing list