[14] RFR: DecimalFormat.setGroupingSize(int) allows setting negative grouping size

naoto.sato at oracle.com naoto.sato at oracle.com
Tue Oct 8 15:59:58 UTC 2019


Hi Roger,

Thank you for the review. In fact, Joe commented about the validity of 
zero on the CSR, so I will need to modify the method description such as:

diff -r 9576895d0f9a 
src/java.base/share/classes/java/text/DecimalFormat.java
--- a/src/java.base/share/classes/java/text/DecimalFormat.java
+++ b/src/java.base/share/classes/java/text/DecimalFormat.java
@@ -2770,10 +2770,13 @@
      /**
       * Set the grouping size. Grouping size is the number of digits 
between
       * grouping separators in the integer portion of a number.  For 
example,
-     * in the number "123,456.78", the grouping size is 3.
-     * <br>
+     * in the number "123,456.78", the grouping size is 3. Grouping size of
+     * zero designates that grouping is not used, which provides the same
+     * formatting as if calling {@link #setGroupingUsed(boolean)
+     * setGroupingUsed(false)}.
+     * <p>
       * The value passed in is converted to a byte, which may lose 
information.
-     * Invalid value, i.e., negative or greater than
+     * Values that are negative or greater than
       * {@link java.lang.Byte#MAX_VALUE Byte.MAX_VALUE}, will throw an
       * {@code IllegalArgumentException}.
       *

I will file a follow-on CSR and merge changesets.

Naoto

On 10/8/19 6:59 AM, Roger Riggs wrote:
> Hi Naoto,
> 
> DecimalFormat.java: 2776:  "Invalid value, i.e.," -> "Values that are".
> 
> Otherwise looks fine. No need for another webrev.
> 
> Thanks, Roger
> 
> 
> 
> 
> On 10/4/19 6:54 PM, naoto.sato at oracle.com wrote:
>> Hello,
>>
>> Please review the fix to the following issue:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8212749
>>
>> The proposed CSR and changeset are located at:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8231851
>> https://cr.openjdk.java.net/~naoto/8212749/webrev.00/
>>
>> Naoto
> 


More information about the core-libs-dev mailing list