David, Agree, It's better to set resultList to NULL explicitly in case of exception. I'll change it.
Thank you for the pointer. -Dmitry On 2014-10-15 04:21, David Holmes wrote: > Hi Dmitry, > > On 15/10/2014 3:46 AM, Dmitry Samersoff wrote: >> Please review a small fix: >> >> http://cr.openjdk.java.net/~dsamersoff/JDK-8029465/webrev.01/ >> >> Added couple of missed exception checks. > > Added checks look fine. > > Am wondering about: > > 104 resultList = (*env)->CallStaticObjectMethod(env, arraysCls, mid, > result); > 105 return resultList; > > If there is an exception pending due to the call what will resultList be > set to? Hopefully NULL but the JNI spec says nothing. > > Thanks, > David > -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the source code.
