RFR JDK-8141491: Unaligned memory access in Bits.c

[Paul Sandoz]

> On 10 Feb 2016, at 04:42, Mikael Vidstedt <mikael.vidstedt at oracle.com> wrote:
> 
> 
> Can I please get a quick review of these updated webrevs:
> 
> hotspot: http://cr.openjdk.java.net/~mikael/webrevs/8141491/webrev.05/hotspot/webrev/
> jdk: http://cr.openjdk.java.net/~mikael/webrevs/8141491/webrev.05/jdk/webrev/
> 
> incremental webrevs:
> 
> hotspot: http://cr.openjdk.java.net/~mikael/webrevs/8141491/webrev.05.incr/hotspot/webrev/
> jdk: http://cr.openjdk.java.net/~mikael/webrevs/8141491/webrev.05.incr/jdk/webrev/
> 

+1

I agree with David on the JavaDoc, but that could be followed up with any future changes, including potentially the removal of wrapping methods in Bits.java, since buffers any way use Unsafe the wrappers now appear to offer little value.

Paul.


> Changes:
> 
> * Added asserts in copy.cpp/conjoint_swap
> * Correctness: Moved offset sign checks to only be performed if corresponding base object is null, and added corresponding tests
> 
> I'm about to make additional changes in this same area, so unless this last change is horribly broken I'm planning on pushing this and doing any additional cleanup in the upcoming change(s).
> 
> Cheers,
> Mikael
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 841 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20160210/05ae5746/signature.asc>