Code Review Request Bug #7129185:(coll) Please add Collections.emptyNavigableSet()
Jason Mehrens
jason_mehrens at hotmail.com
Tue Jan 24 17:54:52 UTC 2012
Darryl,
Here is my list of bugs:
1. The comparator method is using raw types. Should be Comparator<? super E>.
2. The readResolve method comment is just wrong and the method implementation is redundant of the default behavior.
3. What if I want to create an empty set navigable set with supplied comparator? Extending is not an option.
Here are my lists of RFEs:
5. Why not extend EmptySet so you don't have to reimplement the optimized methods?
6. Why not create a default access static final reference named EMPTY_NAVIGABLE_SET inside of the EmptyNavigableSet and use that in readResolve and in Collections.emptyNavigableSet? That gives you the nice singleton behavior and on demand class loading.
7. CCE lacks a descriptive message that you normally get if you used Class.cast or just an implicit cast.
Jason
> Date: Mon, 23 Jan 2012 15:19:01 -0800
> From: darryl.mocek at oracle.com
> To: core-libs-dev at openjdk.java.net
> Subject: Code Review Request Bug #7129185:(coll) Please add Collections.emptyNavigableSet()
>
> Re-sending this with the synopsis in the subject line (and the correct
> bug #).
>
> Hello core-libs. Please review this patch to fix Bug #7129185. This
> fix addresses comments made by Jason Mehrens to the commit of the fix
> for bug #4533691, including adding a Collections.emptyNavigableSet
> method. Tests are included.
>
> Webrev, can be found here:
> http://cr.openjdk.java.net/~dmocek/7129185/webrev.00
>
> Thanks,
> Darryl
More information about the core-libs-dev
mailing list