[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