* include/jni.h: Added FIXME.

From-SVN: r90370

* include/jni.h: Added FIXME.
From-SVN: r90370
cdf87ecb · Tom Tromey · Tom Tromey · d34aacdf · cdf87ecb · cdf87ecb
Commit cdf87ecb authored Nov 09, 2004 by Tom Tromey Committed by Tom Tromey Nov 09, 2004
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletions

libjava/ChangeLog
+4 -0

libjava/include/jni.h
+3 -1

No files found.
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
+2004-11-09  Tom Tromey  <tromey@redhat.com>
+
+	* include/jni.h: Added FIXME.
+
 2004-11-08  Tom Tromey  <tromey@redhat.com>

 	* java/util/GregorianCalendar.java: Removed unused imports.

--- a/libjava/include/jni.h
+++ b/libjava/include/jni.h
@@ -44,7 +44,9 @@ typedef struct _Jv_JavaVM JavaVM;
 # ifdef __GNUC__

 /* If we're using gcc, we can use a platform-independent scheme to get
-   the right integer types.  */
+   the right integer types.  FIXME: this is not always correct, for
+   instance on the c4x it will be wrong -- it depends on whether
+   QImode is 8 bits.  */
 typedef int    jbyte  __attribute__((__mode__(__QI__)));
 typedef int    jshort __attribute__((__mode__(__HI__)));
 typedef int    jint   __attribute__((__mode__(__SI__)));