[MPlayer-dev-eng] [PATHC] define O_BINARY in stream.h
KO Myung-Hun
komh at chollian.net
Wed Mar 3 18:02:30 CET 2010
Diego Biurrun wrote:
> On Thu, Mar 04, 2010 at 01:09:36AM +0900, KO Myung-Hun wrote:
>>
>> Define O_BINARY in stream/stream.h unless it is defined yet, and use it
>> in other places.
>>
>> This removes platform specific checks and prevents repeated definitions
>> of O_BINARY.
>>
>> --- stream/stream_file.c (revision 30819)
>> +++ stream/stream_file.c (working copy)
>> @@ -136,9 +136,7 @@
>>
>> -#if defined(__CYGWIN__)|| defined(__MINGW32__) || defined(__OS2__)
>> m |= O_BINARY;
>> -#endif
>
> This looks wrong since previously it was only done for DOS platforms..
>
If so, do you know the platforms with O_BINARY other than DOS ones ?
I know, BeOS does. But as Reimar said in other thread, it has no effect.
In addition, you accepted the same patch for stream_cddb.c. Nevertheless
why do you say that it is wrong this time ?
--
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