JDK 8 code review request for java.math doclint / accessibility cleanup

Lance Andersen Lance.Andersen at oracle.com
Wed Jun 26 20:06:54 UTC 2013


Hi Joe,

Looks fine

Best
Lance
On Jun 26, 2013, at 3:21 PM, Joe Darcy <joe.darcy at oracle.com> wrote:

> Hello,
> 
> Please review the changes in the patch below; these change address the doclint and HTML accessibility issues in java.math.
> 
> I'll find an existing bug or if one is not present, file a new bug to cover this work.
> 
> Thanks,
> 
> -Joe
> 
> diff -r 336e5a862013 src/share/classes/java/math/BigDecimal.java
> --- a/src/share/classes/java/math/BigDecimal.java    Wed Jun 26 11:21:01 2013 -0700
> +++ b/src/share/classes/java/math/BigDecimal.java    Wed Jun 26 12:20:28 2013 -0700
> @@ -2572,6 +2572,9 @@
>      * ({@code this} * 10<sup>n</sup>).  The scale of
>      * the result is {@code (this.scale() - n)}.
>      *
> +     * @param n the exponent power to ten to scale by
> +     * @return a BigDecimal whose numerical value is equal to
> +     * ({@code this} * 10<sup>n</sup>)
>      * @throws ArithmeticException if the scale would be
>      *         outside the range of a 32-bit integer.
>      *
> diff -r 336e5a862013 src/share/classes/java/math/RoundingMode.java
> --- a/src/share/classes/java/math/RoundingMode.java    Wed Jun 26 11:21:01 2013 -0700
> +++ b/src/share/classes/java/math/RoundingMode.java    Wed Jun 26 12:20:28 2013 -0700
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2003, 2011, Oracle and/or its affiliates. All rights reserved.
> + * Copyright (c) 2003, 2013, Oracle and/or its affiliates. All rights reserved.
>  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>  *
>  * This code is free software; you can redistribute it and/or modify it
> @@ -101,6 +101,7 @@
>          *
>          *<p>Example:
>          *<table border>
> +         * <caption><b>Rounding mode UP Examples</b></caption>
>          *<tr valign=top><th>Input Number</th>
>          *    <th>Input rounded to one digit<br> with {@code UP} rounding
>          *<tr align=right><td>5.5</td> <td>6</td>
> @@ -124,6 +125,7 @@
>          *
>          *<p>Example:
>          *<table border>
> +         * <caption><b>Rounding mode DOWN Examples</b></caption>
>          *<tr valign=top><th>Input Number</th>
>          *    <th>Input rounded to one digit<br> with {@code DOWN} rounding
>          *<tr align=right><td>5.5</td> <td>5</td>
> @@ -148,6 +150,7 @@
>          *
>          *<p>Example:
>          *<table border>
> +         * <caption><b>Rounding mode CEILING Examples</b></caption>
>          *<tr valign=top><th>Input Number</th>
>          *    <th>Input rounded to one digit<br> with {@code CEILING} rounding
>          *<tr align=right><td>5.5</td> <td>6</td>
> @@ -172,6 +175,7 @@
>          *
>          *<p>Example:
>          *<table border>
> +         * <caption><b>Rounding mode FLOOR Examples</b></caption>
>          *<tr valign=top><th>Input Number</th>
>          *    <th>Input rounded to one digit<br> with {@code FLOOR} rounding
>          *<tr align=right><td>5.5</td> <td>5</td>
> @@ -198,6 +202,7 @@
>          *
>          *<p>Example:
>          *<table border>
> +         * <caption><b>Rounding mode HALF_UP Examples</b></caption>
>          *<tr valign=top><th>Input Number</th>
>          *    <th>Input rounded to one digit<br> with {@code HALF_UP} rounding
>          *<tr align=right><td>5.5</td> <td>6</td>
> @@ -223,6 +228,7 @@
>          *
>          *<p>Example:
>          *<table border>
> +         * <caption><b>Rounding mode HALF_DOWN Examples</b></caption>
>          *<tr valign=top><th>Input Number</th>
>          *    <th>Input rounded to one digit<br> with {@code HALF_DOWN} rounding
>          *<tr align=right><td>5.5</td> <td>5</td>
> @@ -255,6 +261,7 @@
>          *
>          *<p>Example:
>          *<table border>
> +         * <caption><b>Rounding mode HALF_EVEN Examples</b></caption>
>          *<tr valign=top><th>Input Number</th>
>          *    <th>Input rounded to one digit<br> with {@code HALF_EVEN} rounding
>          *<tr align=right><td>5.5</td> <td>6</td>
> @@ -278,6 +285,7 @@
>          * {@code ArithmeticException} is thrown.
>          *<p>Example:
>          *<table border>
> +         * <caption><b>Rounding mode UNNECESSARY Examples</b></caption>
>          *<tr valign=top><th>Input Number</th>
>          *    <th>Input rounded to one digit<br> with {@code UNNECESSARY} rounding
>          *<tr align=right><td>5.5</td> <td>throw {@code ArithmeticException}</td>
> 




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







More information about the core-libs-dev mailing list