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