On Tue, Jul 10, 2007 at 09:23:54AM -0300, Davi Arnaut wrote: > Joe Orton wrote: > > On Mon, Jul 09, 2007 at 07:31:01PM -0300, Davi Arnaut wrote: > >> Updated patch, now it only compiles (should work on cross-compilers now > >> too). I also removed the OS X overrides (not necessary anymore). > > > > +1, nice work! Probably the s390*linux overrides are unnecessary now > > too? > > What worries me is "breaking" stuff for non-gcc builds. I think I'll > move the overrides to the top (default values) and let the type checks > override.
Oh, I was presuming the code would only be run for GCC anyway, I missed the fact that it wasn't; I'd just surround the calls to the new macro with: if test "$GCC" = "yes"; then
