[MPlayer-dev-eng] [PATCH] internal libdvdcss for OS/2

Diego Biurrun diego at biurrun.de
Fri Jan 30 22:18:26 CET 2009


On Thu, Jan 29, 2009 at 04:40:45PM +0900, KO Myung-Hun wrote:
> 
> I already sent this patch to the upstream, but I've not got any response
> from them.
> 
> So I send this here directly.
> 
> Any comments ?

I pinged some of the videolan guys on IRC, xtophe reacted.  Try pinging
them again, possibly with an updated patch, over on libdvdcss-devel.

That said, I will not accept libdvdcss patches.  We got rid of all our
local patches, I plan to keep it that way.

> --- libdvdcss/device.c  (revision 28306)
> +++ libdvdcss/device.c  (working copy)
> @@ -93,6 +99,14 @@
>  static int aspi_read_internal  ( int, void *, int );
>  #endif
>  
> +#ifdef SYS_OS2

You can save two lines here with #elif, not that it changes much..

> @@ -270,6 +314,33 @@
>  
> +        rc = DosDevIOCtl(
> +                ( HFILE )-1, IOCTL_DISK, DSK_GETDEVICEPARAMS,
> +                &param, sizeof( param ), &ulParamLen,
> +                &data, sizeof( data ), &ulDataLen );

This looks like one of your weird indentation ideas and does not match
surrounding code style.

Diego



More information about the MPlayer-dev-eng mailing list