Review for 8130344
Alan Bateman
Alan.Bateman at oracle.com
Fri Jul 10 16:34:28 UTC 2015
On 10/07/2015 16:46, Jean-Francois Denise wrote:
> Hi,
> asking review for this bug fix.
> http://cr.openjdk.java.net/~jfdenise/JDK-8130344/
>
I assume if JNI's NewByteArray returns NULL then it does so with a
pending OOME. You might want to double check the JNI spec on this. If it
guarantee it then you might need not need the null check in getStringBytes.
The change to Image.c looks okay. In other areas of the libraries we
have macros to do this.
-Alan.
More information about the jigsaw-dev
mailing list