[vlc-devel] [PATCH 19/41] Implement thread support for OS/2
komh78 at gmail.com
Wed Oct 19 14:16:32 CEST 2011
Rémi Denis-Courmont wrote:
> On Tue, 18 Oct 2011 21:27:02 +0900, KO Myung-Hun <komh78 at gmail.com> wrote:
>> Rémi Denis-Courmont wrote:
>>> Comments inline...
>>> Le lundi 10 octobre 2011 14:43:58 KO Myung-Hun, vous avez écrit :
>>>> @@ -149,6 +152,11 @@
>>>> #include <sys/syslimits.h>
>>>> +#ifdef __OS2__
>>>> +# include <unistd.h>
>>>> +# include <time.h>
>>> Those two headers should not be needed in <vlc_common.h>. Normally,
>>> files that
>>> dependent on them already include them explicitly.
>> You missed the patch for langfromtelx.c.
>> I modified the patch slightly.
>> This is required for off_t of vlc_sout.h.
> Err? ssize_t and off_t are in <sys/types.h> not in <unistd.h>.
On OS/2, they are defined in unistd.h as well.
> I don't like to promiscuously include headers from other headers. It
> always ends up a mess when reorganizing header files.
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
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the vlc-devel