Re: jni-h-fix-warning-1.patch

[Tom Tromey]

>>>>> "Steven" == Steven Augart <address@hidden> writes:

Steven> * include/jni.h.in: Fix a warning you otherwise get when compiling
Steven> with gcc (not g++) with the -Wstrict-prototypes flag.

_Jv_func is used only as a placeholder for reserved functions in
various structs.  Sun uses "void *" here, but I think that is
pedantically incorrect, since there is no guarantee that a void* and a
function pointer have the same size.  Of course that brings up the
problem: perhaps we're being incompatible (with a buggy
implementation), and we should just use `void *' ourselves.

Another option would be to unconditionally use `(void)' for both C
and C++.  That would be a little cleaner, one less ifdef.

Tom