RFR: 6900441 PlatformEvent.park(millis) on Linux could still be affected by changes to the time-of-day clock
david.holmes at oracle.com
Fri Sep 13 16:42:27 PDT 2013
On 14/09/2013 3:49 AM, Aleksey Shipilev wrote:
> On 09/13/2013 09:03 PM, Daniel D. Daugherty wrote:
>> I think you want to get some other eyes on this change. This is
>> my fourth review and I'm not catching everything. Or I'm catching
>> something, but not for exactly the right reasons.
> (Granted, I have mediocre experience with Linux APIs).
> Nothing pops out for me. Thumbs up.
> "++absTime->tv_sec;" bothers me. There seems to be no reason to not to
> use post-increment rather than let the readers infer what is being
> incremented, and if the precedence rules play dirty here.
Noted for future cleanup :)
More information about the hotspot-dev