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

[Stephen Colebourne]

Looks good
Stephen


On 21 Dec 2016 6:31 a.m., "Abhijit Roy" <abhijit.r.roy at oracle.com> 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
>>>
>>
>>
>