[MPlayer-dev-eng] [PATCH] VCD support for OS/2
KO Myung-Hun
komh at chollian.net
Tue Jan 13 04:24:25 CET 2009
Hi/2.
Diego Biurrun wrote:
> On Tue, Jan 13, 2009 at 02:13:31AM +0900, KO Myung-Hun wrote:
>
>> This patch enables a VCD support on OS/2.
>>
>> --- stream/vcd_read_os2.h (revision 0)
>> +++ stream/vcd_read_os2.h (revision 0)
>> @@ -0,0 +1,275 @@
>> +#ifndef MPLAYER_VCD_READ_OS2_H
>> +#define MPLAYER_VCD_READ_OS2_H
>>
>
> Please add our standard license header.
>
>
Ok.
>> +typedef struct
>> +{
>> + BYTE bFrame;
>> + BYTE bSecond;
>> + BYTE bMinute;
>> + BYTE bReserved;
>> +} msf_t;
>> +
>> +typedef struct
>> +{
>> + HFILE hcd;
>> + msf_t msfCurrent;
>> + int iFirstTrack;
>> + int iLastTrack;
>> + msf_t msfLeadOut;
>> +} mp_vcd_priv_t;
>>
>
> The _t namespace is reserved by POSIX. mp_vcd_priv_t is already used
> elsewhere, but let's not add more violations where avoidable.
>
> This is a new file, so please use K&R style.
>
>
Fixed.
>> --- stream/stream_vcd.c (revision 28306)
>> +++ stream/stream_vcd.c (working copy)
>> @@ -108,6 +122,13 @@
>> hd = CreateFile(device, GENERIC_READ, FILE_SHARE_READ, NULL,
>> OPEN_EXISTING, FILE_FLAG_SEQUENTIAL_SCAN, NULL);
>> f = _open_osfhandle((long)hd, _O_RDONLY);
>> +#elif defined(__OS2__)
>> + device[ 0 ] = p->device[ 0 ];
>> + rc = DosOpen( device, &hcd, &ulAction, 0, FILE_NORMAL,
>> + OPEN_ACTION_OPEN_IF_EXISTS | OPEN_ACTION_FAIL_IF_NEW,
>> + OPEN_ACCESS_READONLY | OPEN_SHARE_DENYNONE | OPEN_FLAGS_DASD,
>> + NULL );
>>
>
> Respect the local coding style, drop the extra space within the
> parentheses.
>
>
Fixed.
I've attached the updated patches as your suggestion.
--
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
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: vcd.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20090113/b92e1514/attachment.txt>
More information about the MPlayer-dev-eng
mailing list