[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