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

KO Myung-Hun komh at chollian.net
Sun Feb 28 04:31:44 CET 2010


KO Myung-Hun wrote:
> 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.
>

Applied.

-- 
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




More information about the MPlayer-dev-eng mailing list