Hi All,
Please review the fix for JNI pending exception issues reported in
jdk-8029007. Thanks!
Webrev: http://cr.openjdk.java.net/~dxu/8029007/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8029007
-Dan
Looks ok to me.
I presume getThreadStateValues is simply no longer needed.
-Chris.
> On 10 Jan 2014, at 06:31, Dan Xu wrote:
>
> Hi All,
>
> Please review the fix for JNI pending exception issues reported in
> jdk-8029007. Thanks!
>
> Webrev: http://cr.openjdk.java.net/~dxu/8029007/webrev.0
On 10/01/2014 06:31, Dan Xu wrote:
Hi All,
Please review the fix for JNI pending exception issues reported in
jdk-8029007. Thanks!
Webrev: http://cr.openjdk.java.net/~dxu/8029007/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8029007
This looks good, the only one that isn't clear (t
Hi Dan,
Just pushed are macros in jni_util.h to do the same function as your new
macros.
Please update to use the common macros instead of introducing new ones.
Style wise, I would avoid mixing binary operators (!) with pointers.
it is clearer to compare with NULL. (The CHECK_NULL macro will
Thanks, Chris.
Right, I don't find any usage of getThreadStateValues, so it is simpler
to just remove it.
-Dan
On 01/09/2014 11:49 PM, Chris Hegarty wrote:
Looks ok to me.
I presume getThreadStateValues is simply no longer needed.
-Chris.
On 10 Jan 2014, at 06:31, Dan Xu wrote:
Hi All
On 01/10/2014 02:32 AM, Alan Bateman wrote:
On 10/01/2014 06:31, Dan Xu wrote:
Hi All,
Please review the fix for JNI pending exception issues reported in
jdk-8029007. Thanks!
Webrev: http://cr.openjdk.java.net/~dxu/8029007/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8029007
Thi
Hi Dan,
One other comment, instead of changing the return type of the
setStaticIntField
just return and the caller should check for exceptions and return.
See jni.h: CHECK_EXCEPTION(env)
Roger
On 1/10/2014 11:40 AM, roger riggs wrote:
Hi Dan,
Just pushed are macros in jni_util.h to do the
Hi Roger,
My macro is a little different from yours, which compares with -1
instead of NULL. I also see CHECK_EXCEPTION macro. Thanks for adding
them, which are useful when I cannot decide the pending exception state
by just using return values.
As for the style, actually I prefer the (!poin
On 10 Jan 2014, at 18:05, Dan Xu wrote:
> Hi Roger,
>
> My macro is a little different from yours, which compares with -1 instead of
> NULL. I also see CHECK_EXCEPTION macro. Thanks for adding them, which are
> useful when I cannot decide the pending exception state by just using return
> val
Yes, you are right. I just found this macro. It looks very handy to use.
Thanks!
-Dan
On 01/10/2014 09:59 AM, roger riggs wrote:
Hi Dan,
One other comment, instead of changing the return type of the
setStaticIntField
just return and the caller should check for exceptions and return.
See jni
On Jan 10 2014, at 10:09 , Chris Hegarty wrote:
> On 10 Jan 2014, at 18:05, Dan Xu wrote:
>
>> Hi Roger,
>>
>> My macro is a little different from yours, which compares with -1 instead of
>> NULL. I also see CHECK_EXCEPTION macro. Thanks for adding them, which are
>> useful when I cannot de
Thank you for all the feedback. I have updated my changes to use
"CHECK_EXCEPTION_RETURN" and "CHECK_EXCEPTION" macro recently added into
jni_util.h. I also removed else block in function setStaticIntField() in
Version.c since (*env)->GetStaticFieldID will throw a same exception if
the field ca
> On 10 Jan 2014, at 19:40, Dan Xu wrote:
>
> Thank you for all the feedback. I have updated my changes to use
> "CHECK_EXCEPTION_RETURN" and "CHECK_EXCEPTION" macro recently added into
> jni_util.h. I also removed else block in function setStaticIntField() in
> Version.c since (*env)->GetSta
13 matches
Mail list logo