RFR JDK-8171348: Incorrect documentation for DateTimeFormatter letter 'k'

[Ivan Gerasimov]

Hi Abhijt!

As you're changing the description of 'F' pattern in 
DateTimeFormatterBuilder, it makes sense to do the same in 
DateTimeFormatter.

With kind regards,
Ivan


On 21.12.2016 9:30, Abhijit Roy wrote:
> Hi Roger,
>
> I have fixed the same error in DateTimeFormatterBuiler. Please see the 
> updated webrev below.
>
> Webrev: http://cr.openjdk.java.net/~rpatil/8171348/webrev.01/
>
> Thanks
> Abhijit
>
>
> On 12/16/2016 8:01 PM, Roger Riggs wrote:
>> Hi,
>>
>> Sorry, I meant DateTimeFormatterBuilder.
>>
>> Roger
>>
>>
>> On 12/16/2016 9:28 AM, Roger Riggs wrote:
>>> Hi Abhijit,
>>>
>>> Please also fix the same error in DateTimeFormatter; line 300.
>>>
>>> I would use '24' as the example of the hour of day.
>>> It would emphasize that the range is 1-24.
>>>
>>> Roger
>>>
>>>
>>> On 12/16/2016 6:19 AM, Abhijit Roy wrote:
>>>> Hi all,
>>>>
>>>>
>>>> Please review the java doc fix for the below bug:
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8171348
>>>>
>>>> Description: Incorrect documentation for DateTimeFormatter letter 'k'
>>>>
>>>> Webrev: http://cr.openjdk.java.net/~rpatil/8171348/webrev.00/
>>>>
>>>>
>>>> I have just rectified and modified those errors. And moving forward 
>>>> it for review.
>>>>
>>>>
>>>> Regards,
>>>>
>>>> Abhijit
>>>>
>>>>
>>>>
>>>> P.S.  It will be merged with RFR: JDK-8164923, JDK-8170566, 
>>>> JDK-8169482, JDK-8170653
>>>
>>
>
>