Code Review Request for 4533691 (add Collections.EMPTY_SORTED_SET)
David Holmes
david.holmes at oracle.com
Mon Oct 31 00:52:08 UTC 2011
Darryl,
On 29/10/2011 8:44 AM, Darryl Mocek wrote:
> 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/
This was an RFE from 2001 - pre-generics. I don't think it makes sense
to add to the pre-generics pollution by defining EMPTY_SORTED_SET.
> 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.
What is the range on an empty set?
David
-----
> Thanks,
> Darryl
More information about the core-libs-dev
mailing list