Hi Alan,
The macros are generally useful even without being used on a method that
involves jni. An overly aggressive find caught the uses in
java/util/jar/pack.
But yes, it might be better to limit their scope to functions that
involve jni.
Roger
On 1/16/2014 11:41 AM, Alan Bateman wrote:
On 16/01/2014 16:26, roger riggs wrote:
Please review:
The native macros for checking and returning when exceptions
are thrown have been renamed to include the "JNU_" prefix
consistent with other functions in jni_util.h.
The macros have been renamed and existing uses in the jdk repository
for networking, pack200, and have been updated. A jprt run has passed
(except for unrelated failures).
webrev:
http://cr.openjdk.java.net/~rriggs/webrev-jnu-check-rename-8031737/
Did you mean to rename the CHECK_NULL and CHECK_NULL_RETURN macros?
They don't require a JNIEnv so I'm wondering if JNU_* really make
sense here.
-Alan.