On 2016-02-04 04:22, Andrew Haley wrote:
On 02/02/2016 07:25 PM, Mikael Vidstedt wrote:
Please review this change which introduces a Copy::conjoint_swap and an
Unsafe.copySwapMemory method to call it from Java, along with the
necessary changes to have java.nio.Bits call it instead of the Bits.c code.
http://cr.openjdk.java.net/~mikael/webrevs/8141491/webrev.03/hotspot/webrev/
http://cr.openjdk.java.net/~mikael/webrevs/8141491/webrev.03/jdk/webrev/
One other little thing: why are the byte-swapping methods in class nio.Bits
not called copySwapSomething? e.g.:
That sure would be a better name, wouldn't it? I'm not going to be
changing the Bits method names as part of this change, but it does seem
like a very reasonable follow-up enhancement.
Cheers,
Mikael
