[MPlayer-dev-eng] [PATCH] OS/2 - stream
KO Myung-Hun
komh at chollian.net
Tue Aug 21 14:05:09 CEST 2007
Hi/2.
Diego Biurrun wrote:
> On Mon, Aug 20, 2007 at 10:33:06PM +0900, KO Myung-Hun wrote:
>
>> Diego Biurrun wrote:
>>
>>> On Sat, Aug 18, 2007 at 05:55:15PM +0900, KO Myung-Hun wrote:
>>>
>>>
>>>> This is a patch of 'stream' for OS/2.
>>>>
>>>> --- stream/udp.c (revision 24089)
>>>> +++ stream/udp.c (working copy)
>>>> @@ -43,6 +43,10 @@
>>>>
>>>> +#ifndef HAVE_SOCKLEN_T
>>>> +typedef int socklen_t;
>>>> +#endif
>>>>
>>>> --- stream/tcp.c (revision 24089)
>>>> +++ stream/tcp.c (working copy)
>>>> @@ -34,6 +34,10 @@
>>>>
>>>> +#ifndef HAVE_SOCKLEN_T
>>>> +typedef int socklen_t;
>>>> +#endif
>>>>
>>>> --- stream/librtsp/rtsp_rtp.c (revision 24089)
>>>> +++ stream/librtsp/rtsp_rtp.c (working copy)
>>>> @@ -45,6 +45,10 @@
>>>>
>>>> +#ifndef HAVE_SOCKLEN_T
>>>> +typedef int socklen_t;
>>>> +#endif
>>>>
>>> I'm sure there is a way to have this in just one common place.
>>>
>>>
>> I also thought about that at first. First candidate was 'config.h'. But
>> 'libavformat/os_support.h' also had it because of the same reason as
>> stream part. So gcc complained about 'redefinition'. Of course, I can
>> modify libavformat/os_support.h, but it is a part of FFmpeg. And these
>> require another work, right ? So I decided to do that way.
>>
>> Anyway, those files seem not to have common user header files. Would you
>> mind advicing to me ?
>>
>
> I don't see such a warning as a big problem ..
>
>
Hmm... It's a big problem because it is not a warning, but an error
making compiler to halt.
--
KO Myung-Hun
Using Mozilla SeaMonkey 1.1.2
Under OS/2 Warp 4 for Korean with FixPak #15
On AMD ThunderBird 750 MHz with 512 MB RAM
Korean OS/2 User Community : http://www.ecomstation.co.kr
More information about the MPlayer-dev-eng
mailing list