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

[KO Myung-Hun]

Hi/2.

Diego Biurrun wrote:
> On Tue, Feb 09, 2010 at 11:12:06PM +0900, KO Myung-Hun wrote:
>>
>> Reimar Döffinger wrote:
>>> On Fri, Feb 05, 2010 at 08:53:25PM +0900, KO Myung-Hun wrote:
>>>> This patch adds Win32 loader support for OS/2.
>>>
>>>> +#ifdef DEBUG_QTX_API
>>>> +    printf("addr = %p, cb = %d, fl = %08X : ", p, cb, fl);
>>>> +    if(fl&   0x10)
>>>> +        printf("COMMIT ");
>>>> +    if(fl&   0x2000 )
>>>> +        printf("SHARED ");
>>>> +    if(fl&   0x4000)
>>>> +        printf("FREE ");
>>>> +    if(fl&   0x10000)
>>>> +        printf("BASE ");
>>>> +    if(fl&   0x01)
>>>> +        printf("READ ");
>>>> +    if(fl&   0x02)
>>>> +        printf("WRITE ");
>>>> +    if(fl&   0x04)
>>>> +        printf("EXEC ");
>>>> +    if(fl&   0x08)
>>>> +        printf("GUARD ");
>>>> +    printf("\n");
>>>> +#endif
>>>
>>> is that necessary?
>>
>> Of course for debugging.
>
> Probably you should just drop that hunk then.


Would you mind explaining the reason ?

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.0
Under OS/2 Warp 4 for Korean with FixPak #15
On Intel Core2Duo T5500 1.66 GHz with 1 GB RAM

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