Hi Stuart,

Looks fine.  seems more of a bug fix so I am not sure a ccc is needed but I am 
sure Joe will let us know :-)

On Jun 24, 2015, at 4:37 PM, Stuart Marks <stuart.ma...@oracle.com> 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



Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
lance.ander...@oracle.com



Reply via email to