[MPlayer-cvslog] r27240 - trunk/configure

[Diego Biurrun]

On Mon, Jul 14, 2008 at 02:36:59AM +0200, Michael Niedermayer wrote:
> On Sun, Jul 13, 2008 at 12:54:37PM +0200, Reimar Döffinger wrote:
> > On Sun, Jul 13, 2008 at 12:49:16PM +0200, Diego Biurrun wrote:
> > > > Could you please post a few of the warnings you where trying to fix. There
> > > > may be a more correct and clean way to get rid of them.
> > > 
> > > Warnings like the following, there was a handful of them:
> > > 
> > > libmpcodecs/ad_libvorbis.c: In function 'read_vorbis_comment':
> > > libmpcodecs/ad_libvorbis.c:48: warning: implicit declaration of function 'vsscanf'
> > 
> > Well, at least my man page suggests these solutions to that:
> > "_XOPEN_SOURCE >= 600 || _ISOC99_SOURCE; or cc -std=c99"
> > at least strictly speaking, -std=gnu99 is not among the recommended
> > solutions.
> 
> Ohh lord, this must have been written before the birth of the saviour uoti.
> 
> > In addition, -D_ISOC99_SOURCE would work in all compilers,
> > not just gcc (though ICC probably supports it anyway and no other
> > compiler is supported. Still...).
> 
> Heretic, dont you know that all compilers must use the --std=gnu99 switch
> to enable ISO C99 mode, how can you dare to even mention a standard
> compliant solution. --std=gnu99 must be used the saviour has decided
> and he knows what is best.
> Dont be blinded by the mention of these defines in the POSIX spec, they
> have been placed by the devil there to confuse you. --std=gnu99 is the
> only and best way to compile C99 programs!

Stop flaming already.

-std=gnu99 is closer to C99 than -std=gnu89, period.  This change
already helped find and fix some non-C99 constructs in FFmpeg.  If you
want to see -std=c99 used, then make FFmpeg compilable with that flag,
which is a prerequisite for MPlayer to use it.

Diego