[MPlayer-dev-eng] [PATHC] define O_BINARY in stream.h
Diego Biurrun
diego at biurrun.de
Wed Mar 3 18:44:07 CET 2010
On Thu, Mar 04, 2010 at 02:02:30AM +0900, KO Myung-Hun wrote:
> 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 ?
Forget about my comment.
Diego
More information about the MPlayer-dev-eng
mailing list