Re: Patch: Add JNICALL/JNIEXPORT attributes to JNI functions in jni.h/jni.cc

[Ranjit Mathew <rmathew at hotmail dot com>]

Hi,

    I'm attaching a revised version of the patch with this mail
(9KB, GZIP). Changes from the last time include:

1. (jni.cc) All "retType _Jv_JNI_FooBar( ...)" functions have been
   consistently redefined as "retType (JNICALL _Jv_JNI_FooBar)(...)".
   This was done to avoid a weird bug (IMHO) in the GCC 3.2 C++
   front-end - see my other mail in this thread for more details.

2. (jni.h) A new definition JNIIMPEXP has been defined that maps
   to JNIEXPORT if __GCJ_JNI_IMPL__ has been defined and to
   JNIIMPORT if not. The JNI_FooBar methods have been declared using
   this.

The revised ChangeLog is:

2002-11-19  Ranjit Mathew <rmathew@hotmail.com>

    * include/jni.h: Add missing JNICALL and JNIEXPORT attributes
    to function and function pointer declarations in accordance with
    Sun's JDKs. Define JNIIMPEXP to either JNIEXPORT or JNIIMPORT
    based on whether __GCJ_JNI_IMPL__ has been defined or not.

    * jni.cc: Add missing JNICALL and JNIEXPORT attributes to
    JNI function definitions.

Sincerely Yours,
Ranjit.

--
Ranjit Mathew        Email: rmathew AT hotmail DOT com
Bangalore,
INDIA.               Web: http://ranjitmathew.tripod.com/

Attachment: newjnipatches.gz
Description: application/gzip