[MPlayer-cvslog] r35250 - trunk/configure

[Ingo Brückl]

Carl Eugen Hoyos wrote on Thu, 18 Oct 2012 21:18:53 +0000 (UTC):

> Ingo Brückl <ib <at> wupperonline.de> writes:

>> > Log:
>> > Detect direct.h and io.h on Windows.
>>
>> > Patch by Stephen Sheldon, sfsheldo gmail
>>
>> This breaks Wine compilation:
>>
>> libdvdcss/libdvdcss.c:124:23: error: direct.h: No such file or directory

> Please elaborate, [...]

The HAVE_ definitions will be declared by configure either like

  if test "$_shm" = yes ; then
    def_shm='#define HAVE_SHM 1'
  else
    def_shm='#undef HAVE_SHM'
  fi

or

  if [ $_direct_h = yes ]; then
    def_direct_h='#define HAVE_DIRECT_H 1'
  else
    def_direct_h='#define HAVE_DIRECT_H 0'
  fi

So there doesn't seem to be a common rule for it.

Unfortunately, HAVE_DIRECT_H (and I'm not sure it's the only one) is checked
both #if and #ifdef in ffmpeg and mplayer sources, which is why it isn't a
good idea to #define HAVE_DIRECT_H 0 or check #ifdef HAVE_DIRECT_H in
libdvdcss/libdvdcss.c.

Ingo