Code Review Request for 4533691 (add Collections.EMPTY_SORTED_SET)

Jason Mehrens jason_mehrens at hotmail.com
Sat Oct 29 16:03:52 UTC 2011


Darryl,
 
I would get rid of the public static field so this class can be lazy loaded like EmptyIterator and ReverseComparator (in 1.7).  What about accepting a comparator as an argument?  I bet the bug report is predates 1.6, so maybe you should target NavigableSet instead of sorted set.
 
Regards,
 
Jason
 

> Date: Fri, 28 Oct 2011 15:44:25 -0700
> From: darryl.mocek at oracle.com
> To: core-libs-dev at openjdk.java.net
> Subject: Code Review Request for 4533691 (add Collections.EMPTY_SORTED_SET)
> 
> Hello. Please review this patch to add empty sorted set to the 
> Collections class. Test case provided.
> 
> Webrev: http://cr.openjdk.java.net/~mduigou/4533691/1/webrev/
> 
> Additional Notes to Reviewers:
> The sets resulting from tailSet() headSet() and subSet() normally 
> include the range which was used to create them. Using these methods 
> with emptySortedSet() does not currently set a range on the resulting sets.
> 
> Thanks,
> Darryl
 		 	   		  


More information about the core-libs-dev mailing list