[Mplayer-cvslog] CVS: main/libvo vosub_vidix.c,1.40,1.41

[Nick Kurshev]

Hello, Arpi!

On Wed, 27 Mar 2002 22:49:21 +0100 you wrote:

> Update of /cvsroot/mplayer/main/libvo
> In directory mplayer:/var/tmp.root/cvs-serv2149
> 
> Modified Files:
> 	vosub_vidix.c 
> Log Message:
> 10l to nick - ACCEPT_WIDTH fixed
> 
> Index: vosub_vidix.c
> ===================================================================
> RCS file: /cvsroot/mplayer/main/libvo/vosub_vidix.c,v
> retrieving revision 1.40
> retrieving revision 1.41
> diff -u -r1.40 -r1.41
> --- vosub_vidix.c	24 Mar 2002 21:17:16 -0000	1.40
> +++ vosub_vidix.c	27 Mar 2002 21:49:19 -0000	1.41
> @@ -636,18 +636,28 @@
>      if((is_422_planes_eq || (mpi->flags&(MP_IMGFLAG_ACCEPT_STRIDE|MP_IMGFLAG_ACCEPT_WIDTH)) && 
>         !forced_fourcc && !(vidix_play.flags & VID_PLAY_INTERLEAVED_UV)))
>      {
> +	if(mpi->flags&MP_IMGFLAG_ACCEPT_WIDTH){
> +	    // check if only width is enough to represent strides:
> +	    if(mpi->flags&MP_IMGFLAG_PLANAR){
> +		if((dstrides.y>>1)!=dstrides.v || dstrides.v!=dstrides.u) return VO_FALSE;
> +	    } else {
> +		if(dstrides.y % (mpi->bpp/8)) return VO_FALSE;
> +	    }
> +	}
>  	mpi->planes[0]=vidix_mem+vidix_play.offsets[next_frame]+vidix_play.offset.y;
> -	mpi->stride[0]=dstrides.y;
> +	mpi->width=mpi->stride[0]=dstrides.y;
Perfectly stupid idea: STRIDE != WIDTH in general case so it's your mess!
And IMHO mplayerxp has more correct version of such fix.

Best regards! Nick
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20020328/8aef8001/attachment.pgp>