RFR 8162106 Remove VarHandle.addAndGet

Aleksey Shipilev aleksey.shipilev at gmail.com
Wed Aug 10 15:29:59 UTC 2016


On 08/10/2016 05:37 PM, Paul Sandoz wrote:
> Please review the following webrev which removes
> VarHandle.addAndGet:
> 
> http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8162106-vhs-rm-addAndGet/webrev/

The removal looks good. +1 from me.

> The previous incarnation of those tests was awkwardly designed to
> test a combination of getAndAdd and addAndGet rather than test
> specifically for overflow. With the removal of addAndGet i simplified
> it to be the same as the acquire/release variants.

Yes, and the overflow tests should really be more complex anyway: e.g.
for sub-long getAndAdd we should check that we did not accidentally
overflow into the adjacent element.

Thanks,
-Aleksey



More information about the core-libs-dev mailing list