[kaffe] More amd64, warning fixes, cleanup
Dalibor Topic
robilad at kaffe.org
Fri Oct 22 16:42:00 PDT 2004
Eric Anholt wrote:
Hi Eric,
thanks a lot for the new load of patches!
> AMD64:
> 1) kaffe-support-cpp-2.diff: Fixup for the commit to avoid undefined
> macro warnings for PROMOTE_* an NO_HOLES, because defined(MACRO) is true
> even if MACRO is 0. Set the macros to 0 when they're unset rather than
> using defined().
> 2) kaffe-promote-holes.diff: Move PROMOTE_* / NO_HOLES to common.h
> instead of sysdepCallMethod.h (sysdepCallMethod isn't always included by
> common.h)
I've CC:ed Michael & Gwenole, so that they can give the patches you sent
a spin on x86_64-linux as well.
> Warnings:
> 3) kaffe-stackTrace-proto.diff: kaffe/kaffevm/exception.c uses
> printStackTrace, so prototype it publically.
> 4) kaffe-support.h-replacechar.diff: it's defined only statically, so
> remove from public header
> 5) kaffe-verify3a.c.diff: Fixes some prototype-related warnings by
> including relevant headers.
Thanks, all checked in now.
> Cleanup:
> 6) kaffe-internalthreadsdefs.diff: USE_INTERNAL_THREADS appears to be
> unused, so gc it from config/
Nice catch! Checked in as well.
> With no patches or all but #2, I have 12/144 regression test failures.
> With #2 thrown in the mix, I'm down to 4 errors and I can run my
> classes. (no printf diffs were needed, since fixing arg passing means
> printf doesn't get absurd arguments on a regular basis).
That sounds quite good. What are the remaining failures you get?
keep up the good work,
dalibor topic
More information about the kaffe
mailing list