Looks good Stephen
On 21 Dec 2016 6:31 a.m., "Abhijit Roy" <abhijit.r....@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 >>> >> >> >
