Looks good to me. How did you spot this ? The error code path is
unlikely to be taken
so checking for JNI warnings presumably won't report it.
-phil
On 9/9/19 7:49 AM, Baesken, Matthias wrote:
Hello, please review the following webrev .
I noticed that in function BufImg_SetupICM, in case of initCubemap
returning NULL we currently miss a ReleasePrimitiveArrayCritical call;
this should be added.
Bug/webrev :
https://bugs.openjdk.java.net/browse/JDK-8230769
http://cr.openjdk.java.net/~mbaesken/webrevs/8230769.0/
Thanks, Matthias
