[kaffe] warnings cleanup for amd64
Eric Anholt
eta at lclark.edu
Sun Apr 17 07:00:07 PDT 2005
On Sun, 2005-04-17 at 18:17 +0200, Guilhem Lavaux wrote:
> 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.
It was being compared to a pointer value at icode.h:341, and I thought
it was trying to be a relatively-unique pointer (though there would
probably be a better way of going about that).
--
Eric Anholt eta at lclark.edu
http://people.freebsd.org/~anholt/ anholt at FreeBSD.org
More information about the kaffe
mailing list