[MPlayer-dev-eng] [PATCH] More warnings

[Diego Biurrun]

On Wed, Feb 02, 2011 at 02:50:28PM +0100, Ingo Brückl wrote:
> Objections?
> 
> --- Makefile	(revision 32837)
> +++ Makefile	(working copy)
> @@ -856,7 +856,7 @@
>  
> -gui/%: CFLAGS += -Wno-strict-prototypes
> +gui/%: CFLAGS += -Wall -Wextra -Wno-strict-prototypes

AFAIR -Wall is in global CFLAGS.  You can get -Wextra if you add
--enable-debug to configure flags.  I have had good experience with
enabling -Wextra at work, but in the past this option has not been
terribly popular around here.

Restructuring configure to enable -Wextra and -Wwhatnot with an option
like --enable-extra-warnings or so could be an alternative.

Diego