Re: -Woverride

[Richard Dawe]

Hello.

Alexandre Duret-Lutz wrote:
> Here is my proposal for -Woverride.  This has caught a few
> issues in the test-suite or Automake itself, so this sounds
> useful.

Cool.

[snip]
> Index: lib/Automake/ChannelDefs.pm
> ===================================================================
> RCS file: /cvs/automake/automake/lib/Automake/ChannelDefs.pm,v
> retrieving revision 1.1
> diff -u -r1.1 ChannelDefs.pm
> --- lib/Automake/ChannelDefs.pm 31 Dec 2002 21:31:51 -0000      1.1
> +++ lib/Automake/ChannelDefs.pm 13 Apr 2003 22:41:04 -0000
> @@ -1,4 +1,4 @@
> -# Copyright (C) 2002 Free Software Foundation, Inc.
> +# Copyright (C) 2002, 2003 Free Software Foundation, Inc.
> 
>  # This program is free software; you can redistribute it and/or modify
>  # it under the terms of the GNU General Public License as published by
> @@ -98,6 +98,11 @@
> 
>  Warnings about obsolete features (silent by default).
> 
> +=item C<override>
> +
> +Warnings about user redefinitions of Automake rules or
> +variables (silent by default).
> +
>  =item C<portability>
> 
>  Warnings about non-portable constructs.
[snip]

I noticed that C<syntax> did not have "Warnings about" in front of its
description. Maybe you could fix that too?

 =item C<syntax>

-Weird syntax, unused variables, typos...
+Warnings about weird syntax, unused variables, typos...

The Typo Police didn't find anything, on reviewing the patch. ;)

Bye, Rich =]

-- 
Richard Dawe [ http://www.phekda.freeserve.co.uk/richdawe/ ]