[vlc-devel] [PATCH 1/3] OS/2: Include sys/types.h in vlc_common.h

[KO Myung-Hun]

Hi/2.

Måns Rullgård wrote:
> KO Myung-Hun <komh78 at gmail.com> writes:
> 
>> This fixes the following error.
>>
>> -----
>> make.exe[1]: Entering directory `F:/lang/work/vlc/vlc.git/modules/demux/mkv'
>>   CXX    libmkv_plugin_la-virtual_segment.lo
>> In file included from ../../../include/vlc_common.h:938,
>>                  from mkv.hpp:45,
>>                  from demux.hpp:28,
>>                  from virtual_segment.cpp:27:
>> ../../../include/vlc_configuration.h:104: error: 'ssize_t' does not name a type
>> ../../../include/vlc_configuration.h:106: error: 'ssize_t' does not name a type
>> In file included from mkv.hpp:53,
>>                  from demux.hpp:28,
>>                  from virtual_segment.cpp:27:
>> ../../../include/vlc_charset.h:153: warning: 'warn_unused_result' attribute ignored
>> make.exe[1]: *** [libmkv_plugin_la-virtual_segment.lo] Error 1
>> make.exe[1]: Leaving directory `F:/lang/work/vlc/vlc.git/modules/demux/mkv'
>> make.exe: *** [all] Error 2
>> -----
>> ---
>>  include/vlc_common.h |    4 ++++
>>  1 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/include/vlc_common.h b/include/vlc_common.h
>> index 6514a94..a24dd1a 100644
>> --- a/include/vlc_common.h
>> +++ b/include/vlc_common.h
>> @@ -151,6 +151,10 @@
>>   #include <sys/syslimits.h>
>>  #endif
>>  
>> +#ifdef __OS2__
>> +#   include <sys/types.h>
>> +#endif
> 
> This should be included unconditionally, or if ssize_t is used
> conditionally, under the same condition.
> 

Ok, include unconditionally.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.7.2
Under OS/2 Warp 4 for Korean with FixPak #15
In VirtualBox v4.1.18 on Intel Core i7-3615QM 2.30GHz with 8GB RAM

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

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-OS-2-Include-sys-types.h-in-vlc_common.h.patch
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120824/945b2827/attachment.ksh>