[MPlayer-dev-eng] [PATCH 2/x] Fix stereo3d stride align

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Nov 14 11:57:05 CET 2010


On Wed, Oct 27, 2010 at 01:36:36AM +0200, Endre Kollár wrote:
> -        vf->priv->width         = width / 2;
> -        vf->priv->in.off_right  = vf->priv->width * 3;
> -        vf->priv->in.stride     = vf->priv->width * 6;
> +        vf->priv->width           = width / 2;
> +        vf->priv->in.off_right_x  = vf->priv->width * 3;
> +        vf->priv->in.stride       = vf->priv->width * 6;

Those reindentations make it near impossible to figure out
what changed, that's not ok.
You can also further reduce the size by keeping the
"off_right" name for what you here call off_right_x
and add a row_right instead of off_right_y for example.
(I don't really care so much about the specific names).

> -        dmpi = vf_get_image(vf->next, IMGFMT_RGB24, MP_IMGTYPE_TEMP, 0,
> +        dmpi = vf_get_image(vf->next, IMGFMT_RGB24,
> +                            MP_IMGTYPE_TEMP, MP_IMGFLAG_ACCEPT_STRIDE,

This seems unrelated.

> @@ -344,7 +360,6 @@ static int vf_open(vf_instance_t *vf, ch
>      vf->uninit          = uninit;
>      vf->put_image       = put_image;
>      vf->query_format    = query_format;
> -    vf->default_reqs    = VFCAP_ACCEPT_STRIDE;

Why?


More information about the MPlayer-dev-eng mailing list