[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Patch: Make HEAD compile with standard C compiler

From: Daniel Berlin <dberlin_at_dberlin.org>
Date: 2006-02-09 03:37:27 CET

On Wed, 2006-02-08 at 18:18 -0800, Greg Stein wrote:
> 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?

GCC actually uses -pedantic -Wno-long-long -Wno-variadic-macros
this sounds different than what it is:

# We want to use -pedantic, but we don't want warnings about
# * 'long long'
# * variadic macros
# So, we only use -pedantic if we can disable those warnings.

(IE it -Wno-<etc> disables those warnings)

>
> Cheers,
> -g
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Feb 9 03:37:53 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.