[MPlayer-dev-eng] [PATCH] CDDB support for OS/2

[KO Myung-Hun]

KO Myung-Hun wrote:
> Diego Biurrun wrote:
>> On Wed, Feb 24, 2010 at 11:02:50PM +0900, KO Myung-Hun wrote:
>>> --- stream/stream_cddb.c (revision 30696)
>>> +++ stream/stream_cddb.c (working copy)
>>> @@ -68,12 +68,18 @@
>>> #include "network.h"
>>> #include "libavutil/common.h"
>>>
>>> +#ifndef O_BINARY
>>> +#define O_BINARY 0
>>> +#endif
>>> +
>>> #define DEFAULT_FREEDB_SERVER "freedb.freedb.org"
>>> #define DEFAULT_CACHE_DIR "/.cddb/"
>>>
>>> @@ -349,11 +427,7 @@
>>>
>>> - file_fd = open(file_name, O_RDONLY
>>> -#if defined(__MINGW32__) || defined(__CYGWIN__)
>>> - | O_BINARY
>>> -#endif
>>> - );
>>> + file_fd = open(file_name, O_RDONLY | O_BINARY);
>>> if (file_fd< 0) {
>>
>> This is unrelated and I think Reimar already OKed it. Please commit
>> right away.
>
> I think it's a part of OS/2 port. Anyway I've commited.
>
> And another patch without it.
>

I'll commit in tomorrow.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.0
Under OS/2 Warp 4 for Korean with FixPak #15
On Intel Core2Duo T5500 1.66 GHz with 1 GB RAM

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