[14] RFR: 8215181: Accounting currency format support

naoto.sato at oracle.com naoto.sato at oracle.com
Wed Aug 7 21:31:58 UTC 2019


Thanks, Roger.

I will replace the string length check as you suggested before push.

Naoto

On 8/7/19 2:14 PM, Roger Riggs wrote:
> +1,
> 
> Bundle.java:226:  there is a slight preference for String.isEmpty() 
> over  String.length() == 0
> 
> Thanks, Roger
> 
> On 8/7/19 4:26 PM, Lance Andersen wrote:
>> awesome, I must have missed it thinking it was a bug when I reviewed 
>> the bug itself (long day :-) ) ….
>>
>>> On Aug 7, 2019, at 4:24 PM, naoto.sato at oracle.com wrote:
>>>
>>> Thanks for the review, Lance.
>>>
>>> Yes, CSR is needed and it is already approved:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8218770
>>>
>>> Naoto
>>>
>>> On 8/7/19 1:20 PM, Lance Andersen wrote:
>>>> Hi again Naoto,
>>>> I meant to ask is there a CSR due to the change in the javadoc for 
>>>> NumberFormat?  If not, there probably should
>>>>> On Aug 7, 2019, at 4:17 PM, Lance Andersen 
>>>>> <lance.andersen at oracle.com <mailto:lance.andersen at oracle.com>> wrote:
>>>>>
>>>>> Hi Naoto,
>>>>>
>>>>> I looked over the proposed changes and the bug.  They seem 
>>>>> reasonable to me.
>>>>>
>>>>> Best
>>>>> Lance
>>>>>> On Aug 7, 2019, at 4:12 PM, naoto.sato at oracle.com 
>>>>>> <mailto:naoto.sato at oracle.com> wrote:
>>>>>>
>>>>>> Ping.
>>>>>>
>>>>>> Naoto
>>>>>>
>>>>>> On 7/31/19 5:57 PM, naoto.sato at oracle.com 
>>>>>> <mailto:naoto.sato at oracle.com> wrote:
>>>>>>> Hi,
>>>>>>> Please review the fix to the following issue:
>>>>>>> https://bugs.openjdk.java.net/browse/JDK-8215181
>>>>>>> The proposed changeset is located at:
>>>>>>> https://cr.openjdk.java.net/~naoto/8215181/webrev.00/
>>>>>>> This is to enable "accounting" style currency formatting 
>>>>>>> (negative values in parentheses) with CLDR.
>>>>>>> Naoto
>>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif> 
>>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>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 <mailto:Lance.Andersen at oracle.com> 
>>>>> <mailto:Lance.Andersen at oracle.com>
>>>>>
>>>>>
>>>>>
>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif> 
>>>>
>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>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 <mailto:Lance.Andersen at oracle.com>
>>   <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>   <http://oracle.com/us/design/oracle-email-sig-198324.gif> 
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>   <http://oracle.com/us/design/oracle-email-sig-198324.gif>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 <mailto:Lance.Andersen at oracle.com>
>>
>>
>>
> 


More information about the core-libs-dev mailing list