Code Review Request Bug #7129185:(coll) Please add Collections.emptyNavigableSet()
Joseph Darcy
joe.darcy at oracle.com
Mon Jan 23 23:44:31 UTC 2012
Hello Darryl,
On 1/23/2012 3:19 PM, Darryl Mocek wrote:
> 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
>
>
I recommend using some different javadoc idioms. For example, I'd replace
3209 * <pre>
3210 * NavigableSet<String> ns =
Collections.emptyNavigableSet();
3211 * </pre>
with
<blockquote>
{@code NavigableSet<String> ns = Collections.emptyNavigableSet();}
</blockquote>
and replace <tt>Foo</tt> with {@code Foo}.
-Joe
More information about the core-libs-dev
mailing list