[kaffe] warnings cleanup for amd64

Guilhem Lavaux guilhem at kaffe.org
Sun Apr 17 09:17:06 PDT 2005


On Sat, 2005-04-16 at 13:13 -0700, Eric Anholt wrote:
> Seeing only one regression failure on FreeBSD/amd64 (GCTest, which fails
> 100% of the time), I went on another warnings cleanup spree.  No
> improvement in regressions, but fewer warnings as a result.  It's mostly
> type cleanups, but there's one uninitialized variable fix to jni.c which
> looks important, and I think I set the correct initial value for it.
> 
> _______________________________________________
> kaffe mailing list
> kaffe at kaffe.org
> http://kaffe.org/cgi-bin/mailman/listinfo/kaffe


Hi Eric !

I wonder why this patch is needed ? It obscures the type we really need.

--- kaffe/kaffevm/intrp/icode.h 5 Feb 2005 09:22:42 -0000       1.23
+++ kaffe/kaffevm/intrp/icode.h 17 Apr 2005 16:20:33 -0000
@@ -16,7 +16,7 @@
 #include "soft.h"

 #if defined(KAFFE_VMDEBUG)
-static const int32 UNINITIALIZED_STACK_SLOT = 0x00c0ffee;
+static const long UNINITIALIZED_STACK_SLOT = 0x00c0ffee;
 #endif

I'm ok for the rest.

Cheers,

Guilhem.





More information about the kaffe mailing list