On 28/10/2011 23:44, 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/

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.
Just to add to Jason's comment, would it be better to just leave out EMPTY_SORTED_SET from this patch? It would be nice not to have to load yet another class when Collections is first used and I assume we would encourage folks to use Collections.emptySortedSet() anyway so that they get type safety.

-Alan.

Reply via email to