[MPlayer-dev-eng] add -Wall to default CFLAGS

Michael Niedermayer michaelni at gmx.at
Wed May 2 14:59:16 CEST 2007


Hi

On Wed, May 02, 2007 at 11:00:12AM +0200, Diego Biurrun wrote:
> I propose adding -Wall (back) into the set of default CFLAGS.  I think
> it was removed by an icc patch ages ago.  Having those extra warnings
> cannot hurt, on the contrary, hopefully some more warnings will get
> fixed.
> 
> Opinions?  Objections?

quoting ffmpeg/configure
---
check_cflags -Wdeclaration-after-statement
check_cflags -Wall
check_cflags -Wno-switch
check_cflags -Wdisabled-optimization
check_cflags -Wpointer-arith
check_cflags -Wredundant-decls
check_cflags -Wno-pointer-sign
---

i havnt checked which if any of these mplayer already uses but i think
that they all might be a good idea

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070502/a1b38351/attachment.pgp>


More information about the MPlayer-dev-eng mailing list