Re: Patch: Make HEAD compile with standard C compiler

On Wed, Feb 08, 2006 at 03:38:42PM -0500, Daniel Berlin wrote:
> On Wed, 2006-02-08 at 17:00 +0000, Julian Foad wrote:
>...
> > May I recommend that developers using recent version of GCC add
> >
> > -Wdeclaration-after-statement
> >
> > to their CFLAGS.
>
> I'm used to bootstrapping gcc, where we have -Werror, -Wunused, etc on
> by default, so it catches this stuff for me :(
>
> I'm not sure why we don't do the same. We could rip the configure magic
> that detects whether it is supported or not by the compiler directly
> from gcc's configure.

We already detect GCC and add some special flags, so putting that one
in should be a no-brainer. (line 489 of configure.in)

Probably a little more difficult if we have to test GCC's version...

Any others that would be good?

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org