Staffan, Thanks!
Will fix it. -Dmitry On 2014-12-02 22:51, Staffan Larsen wrote: > These two lines are wrong. The "!= NULL” should not be there: > > 241 if (!(*env)->ExceptionOccurred(env) != NULL) { > 255 if (!(*env)->ExceptionOccurred(env) != NULL) { > > Otherwise looks good. > > /Staffan > >> On 2 dec 2014, at 20:14, Dmitry Samersoff <dmitry.samers...@oracle.com> >> wrote: >> >> Please review the small fix. >> >> http://cr.openjdk.java.net/~dsamersoff/JDK-8028773/webrev.01/ >> >> Added more missed exception checks to sadis.c >> >> -- >> Dmitry Samersoff >> Oracle Java development team, Saint Petersburg, Russia >> * I would love to change the world, but they won't give me the sources. > -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.