[vlc-devel] [PATCH 11/41] Implement cdrom tools to access audio/video CD on OS/2

[KO Myung-Hun]

Rémi Denis-Courmont wrote:
> Le lundi 10 octobre 2011 14:43:50 KO Myung-Hun, vous avez écrit :
>> ---
>>  configure.ac                         |    2 +-
>>  modules/access/vcd/cdrom.c           |  131
>> +++++++++++++++++++++++++++++++++- modules/access/vcd/cdrom_internals.h | 
>>  53 ++++++++++++++
>>  3 files changed, 183 insertions(+), 3 deletions(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 02d7c72..b6fb801 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -2042,7 +2042,7 @@ then
>>      AC_MSG_RESULT(no)
>>    ])
>>
>> -  if test "${SYS}" = "bsdi" -o "${SYS}" = "mingw32"
>> +  if test "${SYS}" = "bsdi" -o "${SYS}" = "mingw32" -o "${SYS}" = "os2"
>>    then
>>      VLC_ADD_PLUGIN([vcd cdda])
>>    fi
>> diff --git a/modules/access/vcd/cdrom.c b/modules/access/vcd/cdrom.c
>> index 3f260a2..d95c8fd 100644
>> --- a/modules/access/vcd/cdrom.c
>> +++ b/modules/access/vcd/cdrom.c
>> @@ -30,6 +30,10 @@
>>  # include "config.h"
>>  #endif
>>
>> +#ifdef __OS2__
>> +#   define INCL_DOSDEVIOCTL
>> +#endif
> 
> Any reason why this is not #include <os2.h> ?
> 

To prevent os2.h from being included twice.

Otherwise, some stuffs of os2.h will not be included due to its
inclusion guards.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.0.14
Under OS/2 Warp 4 for Korean with FixPak #15
On AMD ThunderBird 1GHz with 512 MB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr