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

KO Myung-Hun komh at chollian.net
Tue Jan 13 17:08:00 CET 2009


Hi/2.

Diego Biurrun wrote:
> On Tue, Jan 13, 2009 at 11:01:24PM +0900, KO Myung-Hun wrote:
>   
>> Diego Biurrun wrote:
>>     
>>> On Tue, Jan 13, 2009 at 12:24:25PM +0900, KO Myung-Hun wrote:
>>>  
>>>       
>>>> I've attached the updated patches as your suggestion.
>>>>
>>>> --- stream/vcd_read_os2.h   (revision 0)
>>>> +++ stream/vcd_read_os2.h   (revision 0)
>>>> +typedef struct {
>>>> +    BYTE bFrame;
>>>> +    BYTE bSecond;
>>>> +    BYTE bMinute;
>>>> +    BYTE bReserved;
>>>> +} msf;
>>>>         
>>> Do you really need a typedef here?
>>>       
>> Any problem ?
>>     
>
> IMO the typedef is just an obfuscation.  And yes, it's used
> in this way everywhere.
>
>   

If there is no other problem than that, I hope to let it be.

>>>> +            mp_msg(MSGT_OPEN, MSGL_INFO, "track %02d:  adr=%d  ctrl=%d  %02d:%02d:%02d\n",
>>>> +                i,
>>>> +                sDataTrack.bControlInfo & 0x0F,
>>>> +                (sDataTrack.bControlInfo >> 4) & 0x0F,
>>>> +                sDataTrack.msfStart.bMinute,
>>>> +                sDataTrack.msfStart.bSecond,
>>>> +                sDataTrack.msfStart.bFrame);
>>>>         
>>> weird indentation
>>>       
>> Fixed, but why is it weird ?
>>     
>
> It's more sensible to align the function arguments with each other.
>
>   

Ok, I see. ^^

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 1.1.14
Under OS/2 Warp 4 for Korean with FixPak #15
On AMD ThunderBird 1 GHz with 512 MB RAM

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





More information about the MPlayer-dev-eng mailing list