autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: Turn on compiler warnings by default for AC_PROG_CC, AC_PROG_CXX & A


From: Dale Visser
Subject: RE: Turn on compiler warnings by default for AC_PROG_CC, AC_PROG_CXX & AC_PROG_FC
Date: Mon, 14 Apr 2014 11:55:25 -0400

I've attached a new diff that fixes that error in the code comments. Emoji
 
> From: address@hidden
> To: address@hidden
> Subject: Re: Turn on compiler warnings by default for AC_PROG_CC, AC_PROG_CXX & AC_PROG_FC
> Date: Fri, 11 Apr 2014 15:23:33 -0400
> CC: address@hidden; address@hidden
>
> On Fri, 11 Apr 2014 08:33:01 -0400, Dale Visser <address@hidden> wrote:
> > You found a mistake in the code comments that were brought over with
> > this code from Autoconf Archive.
>
> Okay, hope you'll fix that!
>
> > To answer your specific question about GCC, only '-Wall' gets
> > used to compile.
>
> Good, that makes way more sense.
>
> I like to also use "-Wextra" when using gcc.
> If I understand patch correctly, I could then use:
> AC_APPEND_CFLAG_IFVALID([-Wextra])
> It will then add "-Wextra"... but only if the compiler *accepts* that flag.
>
> BTW, I recently got a patch from someone on a project that
> force-added "-Wall" to configure.ac. Dale Visser's addition to autoconf is way better,
> because it doesn't presume that "all the world is gcc".
>
> --- David A. Wheeler
>

Attachment: autoconf.diff
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]