RFR: (8031737) rename jni_util.h macros for checking and returning on exceptions
Kumar Srinivasan
kumar.x.srinivasan at oracle.com
Thu Jan 16 17:30:38 UTC 2014
Hi Roger,
Its confusing to use a JNU_ prefixed macro on a method not involvng jni,
why not rename these to modulo JNU_ ?
I am cc'ing Alex as he has a related bug fix in his queue for pack's jni
code.
Kumar
> 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.
>
More information about the core-libs-dev
mailing list