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

KO Myung-Hun komh at chollian.net
Thu Feb 25 10:17:16 CET 2010


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.

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

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: cddbos2.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100225/6f9ecd2a/attachment.txt>


More information about the MPlayer-dev-eng mailing list