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

[Naoto Sato]

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
>>>>>
>>>
>