RFR(s): 8080450: doc for Double/Int/LongSummaryStatistics.toString has errors

Joseph D. Darcy joe.darcy at oracle.com
Wed Jun 24 20:56:33 UTC 2015


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




More information about the core-libs-dev mailing list