[MPlayer-cvslog] r25535 - trunk/configure

[Rich Felker]

On Sat, Dec 29, 2007 at 06:27:43PM +0200, Uoti Urpala wrote:
> On Sat, 2007-12-29 at 11:08 -0500, Rich Felker wrote:
> > On Sat, Dec 29, 2007 at 09:30:15AM +0100, Jindrich Makovicka wrote:
> > > It enables C99 features and bunch of other stuff,
> > > see /usr/include/features.h. -D_ISOC99_SOURCE could be used as well in
> > > this case.
> > > 
> > > With newer GCCs, -std=c99 is preferred.
> 
> For MPlayer we probably want -std=gnu99 instead. -std=c99 means strict
> standard mode without compiler extensions like "asm", which also doesn't
> work well with some current kernel headers (at least).

Would -std=c99 -Dasm=__asm__ work? If so this seems like a very clean
solution. BTW there should also be a configure check for the ability
to compile gcc-style inline asm so that build does not fail for
non-gcc-compatible compilers.

> POSIX specifies the compiler name "c99" which is expected to compile in
> C99 mode. We could try c99 before cc as a "generic" compiler name.
> Unfortunately at least on Debian c99 invokes gcc in the strict standard
> mode -std=c99 instead of -std=gnu99, so configure should still test for
> gcc first and add a "-std=gnu99" flag when it is used.

Hmm....

Rich