[MPlayer-dev-eng] [PATCH] mmap support for OS/2
Diego Biurrun
diego at biurrun.de
Mon Feb 25 10:31:46 CET 2008
On Mon, Feb 25, 2008 at 10:41:08AM +0900, KO Myung-Hun wrote:
>
> Diego Biurrun wrote:
>> On Mon, Feb 25, 2008 at 01:12:56AM +0900, KO Myung-Hun wrote:
>>
>>> This patch add a simple mmap support for OS/2.
>>>
>>> --- osdep/mmap.h (revision 0)
>>> +++ osdep/mmap.h (revision 0)
>>> @@ -0,0 +1,73 @@
>>> +
>>> +/* MAP_FAILED is defined in config.h */
>>> +#if 0
>>> +/*
>>> + * Error return from mmap()
>>> + */
>>> +#ifndef MAP_FAILED
>>> +#define MAP_FAILED ((void *)-1)
>>> +#endif
>>> +#endif
>>
>> I don't like #if 0 code...
>
> Instead, commented it out.
That's no better. I meant that I don't like committing code that is off
by default. Why is it there in the first place?
Diego
More information about the MPlayer-dev-eng
mailing list