[MPlayer-dev-eng] [PATCH] Base patches for OS/2 support

Rich Felker dalias at aerifal.cx
Mon Feb 18 18:45:05 CET 2008


On Mon, Feb 18, 2008 at 10:14:04AM +0100, Diego Biurrun wrote:
> On Mon, Feb 18, 2008 at 12:47:06PM +0900, KO Myung-Hun wrote:
> > 
> > I try another patch for OS/2 support. This is a base patch for others.
> 
> Basically looks good except for
> 
> > --- configure	(revision 26012)
> > +++ configure	(working copy)
> > @@ -7544,6 +7628,9 @@
> >  if cygwin ; then
> >    CFLAGS="$CFLAGS -D__CYGWIN__"
> >  fi
> > +if os2 ; then
> > +  CFLAGS="$CFLAGS -Zomf -DSYS_OS2"
> > +fi
> 
> What do you use SYS_OS2 for?  It seems to be just one extra level of
> indirection for __OS2__.  Just use __OS2__ directly then.

FWIW, I'm against ANY direct use of __-prefixed preprocessor symbols.
It hardcodes lots of nasty implementation-specific knowledge in the C
source files and makes it very hard to use a different compiler or
build environment on the same (or similar) system. Instead, the
implementation-specific knowledge (gathered from the __ symbols)
should be tucked away in the configure script and the actual C source
files should use preprocessor symbols reserved to the application.

Rich



More information about the MPlayer-dev-eng mailing list