[FFmpeg-devel] [PATCH] Re-add OS/2 support

[Dave Yeo]

KO Myung-Hun wrote:
> Hi/2.
> 
> Michael Niedermayer wrote:
>> Hi
>>
>> On Sat, Sep 08, 2007 at 04:37:44PM +0900, KO Myung-Hun wrote:
>>   
>>> Hi/2.
>>>
>>> This patch re-add OS/2 support.
>>>     
>> please explain why precissely the changes to ffplay.c are needed?
>> also trailing whitespace is forbidden in ffmpeg svn
>>   
> 
> It is a trick to enable VIO(text mode) app to use PM(GUI) APIs. So if it
> is missing, VIO apps cannot use PM APIs.
> 
...

It also keeps the console window around so stdout and stderr get displayed.
KO are you still needing this even with the newest SDL? 
(ftp://ftp.netlabs.org/pub/sdl/sdl-1.2.10-bin-20070618.zip)
Looking in the SDL source (SDL_os2fslib.c) I see
static void os2fslib_MorphToPM()
{
   PPIB pib;
   PTIB tib;

   DosGetInfoBlocks(&tib, &pib);

   // Change flag from VIO to PM:
   if (pib->pib_ultype==2) pib->pib_ultype = 3;
}

and similar in SDL_os2dive.c which I'd think should take care of it. At 
least it does here.
Not sure about the unbuffering of stdout and stderr though Doodle does 
do it in SDL for debug builds.
Dave