RFR:JDK-8071919 :Add java.time.Clock.tickMillis(ZoneId zone) method

[Daniel Fuchs]

Hi Nadeesh,

This looks good to me.

Best regards,

-- daniel

On 11/13/15 8:42 PM, nadeesh tv wrote:
> Hi ,
>
> Please see the updated webrev 
> http://cr.openjdk.java.net/~ntv/8071919/webrev.02/
>
> Thanks and Regards,
> Nadeesh
> On 11/13/2015 9:23 PM, Roger Riggs wrote:
>> Hi Nadeesh,
>>
>> One suggestion:
>>
>> Replace "This clock will always have the other than milli part of 
>> nano-of-second field set to zero."
>> Though similar to the other methods, the "other than milli part" is 
>> awkward.
>>
>> With:
>>
>> "This clock will always have the nano-of-second field truncated to 
>> milliseconds"
>>
>> The rest looks fine.
>>
>> Thanks, Roger
>>
>>
>> On 11/13/2015 6:00 AM, nadeesh tv wrote:
>>> Hi all,
>>>
>>> Please review a fix for Bug Id 
>>> -https://bugs.openjdk.java.net/browse/JDK-8071919 
>>> <https://bugs.openjdk.java.net/browse/JDK-8071919>
>>>
>>> Issue - Add java.time.Clock.tickMillis(ZoneId zone) method
>>>
>>> webrev - http://cr.openjdk.java.net/~ntv/8071919/webrev.01/ 
>>> <http://cr.openjdk.java.net/%7Entv/8071919/webrev.01/>
>>>
>>> -- 
>>> Thanks and Regards,
>>> Nadeesh TV
>>>
>>
>