[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