Joe, great, I'll go ahead and push. Thanks.

Lance, thanks for looking as well.

s'marks


On 6/24/15 1:56 PM, Joseph D. Darcy wrote:
Hi Stuart,

The changes look good; a ccc is not necessary IMO.

Thanks,

-Joe

On 6/24/2015 1:37 PM, Stuart Marks wrote:
Hi all,

Please review this small javadoc change to the
Double/Int/LongSummaryStatistics classes' toString() method specifications.
Basically they all inherit the Object.toString() doc, which is mostly
irrelevant, and in two cases the doc comment is placed erroneously and so is
entirely ignored.

See the bug report for details:

    https://bugs.openjdk.java.net/browse/JDK-8080450

Webrev:

    http://cr.openjdk.java.net/~smarks/reviews/8080450/webrev.0/

Javadoc output with changes in place:


http://cr.openjdk.java.net/~smarks/reviews/8080450/javadoc.0/java/util/DoubleSummaryStatistics.html#toString--


http://cr.openjdk.java.net/~smarks/reviews/8080450/javadoc.0/java/util/IntSummaryStatistics.html#toString--


http://cr.openjdk.java.net/~smarks/reviews/8080450/javadoc.0/java/util/LongSummaryStatistics.html#toString--


For comparison, here are the JDK 8 versions of the docs:


http://docs.oracle.com/javase/8/docs/api/java/util/DoubleSummaryStatistics.html#toString--


http://docs.oracle.com/javase/8/docs/api/java/util/IntSummaryStatistics.html#toString--


http://docs.oracle.com/javase/8/docs/api/java/util/LongSummaryStatistics.html#toString--


(I expect this will also require a CCC request, since this is formally a
change in the specification.)

s'marks

Reply via email to