On 29/04/2016 7:50 PM, Aleksey Shipilev wrote:
On 04/29/2016 02:09 AM, David Holmes wrote:
bug: https://bugs.openjdk.java.net/browse/JDK-8154710
webrev: http://cr.openjdk.java.net/~dholmes/8154710/webrev/
Looks good.
Is hrtime_t always integral, so you can "(hrtime_t)now /
NANOSECS_PER_MILLISEC" it?
Yes it is a 64-bit (long long) signed integer.
This change is small in nature but somewhat broad in scope. It "affects"
the implementation of System.currentTimeMillis() in the Java space, and
os::javaTimeMillis() in the VM. But on Solaris only.
I say "affects" but the change will be unobservable other than in terms
of performance.
Observable enough to me.
:) Any apps you can think of that might show benefit from this?
Thanks,
David
Thanks,
-Aleksey