[MPlayer-dev-eng] [PATCH] VCD support for OS/2

Diego Biurrun diego at biurrun.de
Fri Feb 19 14:48:21 CET 2010


On Fri, Feb 19, 2010 at 10:26:26PM +0900, KO Myung-Hun wrote:
> Diego Biurrun wrote:
>> On Fri, Feb 19, 2010 at 12:23:32PM +0900, KO Myung-Hun wrote:
>>>
>>> This patch adds VCD support for OS/2.
>>>
>>> --- stream/stream_vcd.c	(revision 30605)
>>> +++ stream/stream_vcd.c	(working copy)
>>> @@ -22,6 +22,12 @@
>>>
>>> +#if defined(__OS2__)
>>> +#define INCL_DOS
>>> +#define INCL_DOSDEVIOCTL
>>> +#include<os2.h>
>>> +#endif
>>
>> Somehow this seems to be in every other patch you post.
>> Maybe it could be in a more central place...
>
> At least, removed one from vcd_read_os2.h.

I'm not sure this is the way to go.  A wild idea: Why not have a header
(platform.h or similar) in some central place, like os_support.h in
libavformat?  OS-specific hacks and #ifdefs could be hidden there...

Diego



More information about the MPlayer-dev-eng mailing list