RFR doclint issues in j.u.Deque & Queue

Martin Buchholz martinrb at google.com
Fri Aug 2 22:57:04 UTC 2013


On Thu, Aug 1, 2013 at 3:34 AM, Chris Hegarty <chris.hegarty at oracle.com>wrote:

> These changes are already committed in the jsr166 CVS, this is a request
> to pull them into jdk8.
>
> Specdiff has been run, and there is nothing surprising.
> The doclint issues are specifically around the addition of a caption to
> the tables, the remaining changes are cleanup, <code> -> {@code ..}, etc.
>
> Note: there are some minor doc updates around throwing an
> IllegalStateException if no space is currently available in Deque, which is
> already part of the spec.
>
>
Of course, I'm also happy to see the changes from jsr166 making it into
jdk8.


> Martin, Doug,
>  The 166 CVS is trivially missing 6546713,
>
http://cr.openjdk.java.net/~**mduigou/6546713/0/webrev/src/**
> share/classes/java/util/Deque.**java.sdiff.html<http://cr.openjdk.java.net/~mduigou/6546713/0/webrev/src/share/classes/java/util/Deque.java.sdiff.html>.
> Once the changes below are in jdk8, you can easily sync with the 8 version,
> or apply the diffs from the webrev link above.
>
>
jsr166 CVS now also has these changes to Deque.java.  Although....... I am
not entirely happy about them because they create broken links if jsr166 is
built independently, i.e. not as part of openjdk, and I don't see an easy
way to fix that.  But this is one set of broken links we can just live with.


> I have already reviewed these changes, but more reviewers are welcome.
>



More information about the core-libs-dev mailing list