Code Review Request Bug #7129185:(coll) Please add Collections.emptyNavigableSet()
Darryl Mocek
darryl.mocek at oracle.com
Tue Jan 24 18:51:27 UTC 2012
Jason,
see inline and updated webrev:
http://cr.openjdk.java.net/~dmocek/7129185/webrev.01
Darryl
On Tue 24 Jan 2012 09:54:52 AM PST, Jason Mehrens wrote:
>
> Darryl,
>
> Here is my list of bugs:
> 1. The comparator method is using raw types. Should be Comparator<?
> super E>.
The webrev didn't pick up on this change. The method is this:
public Comparator<? super E> comparator() {
return null;
}
> 2. The readResolve method comment is just wrong and the method
> implementation is redundant of the default behavior.
readResolve has been removed.
> 3. What if I want to create an empty set navigable set with supplied
> comparator? Extending is not an option.
This is the one issue I wanted to discuss...is this necessary? I was
thinking about how this would be implemented. You would need to supply
a comparator to the emptyNavigableSet. Other empty* methods don't take
parameters and adding a method to supply a comparator would require an
additional method.
Darryl
>
> 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