[Patch] JNI support for direct buffers
Andrew Haley
aph@redhat.com
Fri Jan 7 11:16:00 GMT 2005
Michael Koch writes:
> After a discussion on irc with Bryce McKinley I added two additional
> checks into the implementations of GetDirectBufferAddress and
> GetDirectBufferLength. New patch attached.
>
> Ok to commit ?
>
> 2005-01-07 Michael Koch <konqueror@gmx.de>
>
> * java/nio/Buffer.java
> (address): New field.
> * java/nio/DirectByteBufferImpl.java
> (address): Removed.
> * java/nio/MappedByteBufferImpl.java
> (address): Likewise..
> * java/nio/CharViewBufferImpl.java
> (CharViewBufferImpl): Explicitely initialize Buffer.address if
> needed.
explicitly
> * java/nio/DoubleViewBufferImpl.java
> (DoubleViewBufferImpl): Likewise.
> * java/nio/FloatViewBufferImpl.java
> (FloatViewBufferImpl): Likewise.
> * java/nio/IntViewBufferImpl.java
> (IntViewBufferImpl): Likewise.
> * java/nio/LongViewBufferImpl.java
> (LongViewBufferImpl): Likewise.
> * java/nio/ShortViewBufferImpl.java
> (ShortViewBufferImpl): Likewise.
> * jni.cc (_Jv_JNI_GetDirectBufferAddress): Don't assume buffer is a
> DirectByteBufferImpl object.
> (_Jv_JNI_GetDirectBufferCapacity): Likewise.
> * testsuite/libjava.jni/directbuffer.c,
> testsuite/libjava.jni/directbuffer.java,
> testsuite/libjava.jni/directbuffer.out,
> testsuite/libjava.jni/bytebuffer.c,
> testsuite/libjava.jni/bytebuffer.java,
> testsuite/libjava.jni/bytebuffer.out: New files.
OK, but please add the PR when you check in the ChangeLog.
BTW, something bad happened to your ChangeLog formatting.
Andrew.
More information about the Java-patches
mailing list