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

[KO Myung-Hun]

Hi/2.

Dave Yeo wrote:
> 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.
>   

Ok, I confirmed that ffplay worked well with MorphToPM().

Thanks. ^^

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 1.1.4
Under OS/2 Warp 4 for Korean with FixPak #15
On AMD ThunderBird 750 MHz with 512 MB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr