Hi Vyom,

I agree with you but it’s already pushed and I doubt it is worth an issue just 
to make this small change.

Thanks,

Brian

On May 18, 2017, at 9:05 PM, Vyom Tewari <vyom.tew...@oracle.com> wrote:

> looks good to me, although i am not JDK10 reviewer, one minor comment i think 
> you can combine the below two statements to one
> 
> rv = (jlong)sb.st_mtimespec.tv_sec * 1000;
> rv += (jlong)sb.st_mtimespec.tv_nsec / 1000000; rv= 
> (jlong)sb.st_mtimespec.tv_sec * 1000  +(jlong)sb.st_mtimespec.tv_nsec / 
> 1000000;

Reply via email to