RFR (2nd round) 8009736: Comparator API cleanup

Henry Jen henry.jen at oracle.com
Sat Jun 15 23:28:54 UTC 2013


Reflecting feedbacks received so far, please see

http://cr.openjdk.java.net/~henryjen/ccc/8009736.3/webrev/
http://cr.openjdk.java.net/~henryjen/ccc/8009736.3/specdiff

Cheers,
Henry


On 06/11/2013 02:04 PM, Henry Jen wrote:
> 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