<i18n dev> [10] RFR JDK-8186713: Document default rounding mode in NumberFormat

Nishit Jain nishit.jain at oracle.com
Tue Aug 29 07:23:07 UTC 2017


Thanks Brian, Naoto for the review. The below updated patch is pushed to 
the repository.

http://cr.openjdk.java.net/~nishjain/8186713/webrev.02/

Regards,
Nishit Jain
On 29-08-2017 04:56, Brian Burkhalter wrote:
> Hi Nishit,
>
> I suggest these changes in NumberFormat.java:
>
> 184:delete the line
> 186:s/{@code NumberFormat}/The default implementation/
> 187-188:s/By default, it uses/It uses/
>
> With these changes I am fine with the patch and no need to send an 
> updated webrev unless you prefer.
>
> Thanks,
>
> Brian
>
> On Aug 28, 2017, at 12:43 AM, Nishit Jain <nishit.jain at oracle.com 
> <mailto:nishit.jain at oracle.com>> wrote:
>
>> >L178-195: Is a list necessary here?
>> The list approach is used to clearly mention that these two points 
>> are under "Implementation Requirements". If it does not look good, 
>> another way could be to use @implSpec tag for individual points, 
>> separated by <p>. Please check if this is fine.
>>
>> Also updated the webrev with other suggested changes.
>>
>> http://cr.openjdk.java.net/~nishjain/8186713/webrev.01/
>



More information about the i18n-dev mailing list