[13] RFR: 8216969: ParseException thrown for certain months with russian locale

[Roger Riggs]

Looks good.

Thanks, Roger


On 1/22/19 3:31 PM, Naoto Sato wrote:
> Hi Roger,
>
> Thanks. Modified as suggested:
>
> http://cr.openjdk.java.net/~naoto/8216969/webrev.02/
>
> Naoto
>
> On 1/19/19 10:03 AM, Roger Riggs wrote:
>> Hi Naoto,
>>
>> SimpleDateFormat: 2454, perhaps add javadoc about what it is doing 
>> and why.
>>    Use only the official style fields and avoid using the 0x8000 that 
>> is supposed to be internal to Calendar.
>>
>> 2457: (forceStandaloneForm ? Calendar.SHORT_STANDALONE ? 
>> SHORT_FORMAT) 2460: (forceStandaloneForm ? Calendar.LONG_STANDALONE ? 
>> LONG_FORMAT)
>>
>> Thanks, Roger
>>
>> On 01/18/2019 06:45 PM, naoto.sato at oracle.com wrote:
>>> Gentle reminder. Still waiting for reviews from OpenJDK Reviewers.
>>>
>>> Naoto
>>>
>>> On 1/18/19 1:57 AM, Nishit Jain wrote:
>>>> Looks Good.
>>>>
>>>> Regards,
>>>> Nishit Jain
>>>> On 17-01-2019 22:07, Naoto Sato wrote:
>>>>> Hi Nishit,
>>>>>
>>>>> Thanks. Updated:
>>>>>
>>>>> http://cr.openjdk.java.net/~naoto/8216969/webrev.01/
>>>>>
>>>>> Naoto
>>>>>
>>>>> On 1/17/19 2:57 AM, Nishit Jain wrote:
>>>>>> Hi Naoto,
>>>>>>
>>>>>> Looks good to me. Just a small suggestion.
>>>>>>
>>>>>> - To improve readability, can we declare "standalone mask" 
>>>>>> (0x8000) as a static field and use that at all the places?
>>>>>>
>>>>>> Regards,
>>>>>> Nishit Jain
>>>>>> On 17-01-2019 05:50, naoto.sato at oracle.com wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> Please review the fix to the following issue:
>>>>>>>
>>>>>>> https://bugs.openjdk.java.net/browse/JDK-8216969
>>>>>>>
>>>>>>> The proposed changeset is located at:
>>>>>>>
>>>>>>> http://cr.openjdk.java.net/~naoto/8216969/webrev.00/
>>>>>>>
>>>>>>> For parsing the context sensitive month 'M', the logic was to 
>>>>>>> look for the best match for the short name regardless of the 
>>>>>>> styles. Changed the parsing code to take the context into account.
>>>>>>>
>>>>>>> Naoto
>>>>>>
>>>>
>>