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