RFR(S), SA, warnings from b116 for hotspot.agent.src.share.native: JNI exception pending

[Dmitry Samersoff]

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.samersoff at 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.