[MPlayer-cvslog] r35670 - trunk/libmpcodecs/vf_ass.c

[Diego Biurrun]

On Wed, Dec 12, 2012 at 06:18:01PM +0100, upsuper wrote:
> --- trunk/libmpcodecs/vf_ass.c	Wed Dec 12 18:13:44 2012	(r35669)
> +++ trunk/libmpcodecs/vf_ass.c	Wed Dec 12 18:18:01 2012	(r35670)
> @@ -392,6 +414,152 @@ static void render_frame_yuv420p(vf_inst
>  
> +    CLEAN_XMM(7);
> +
> +#define CHECK_16_ALPHA \
> +            "cmpl   $-1,     0(%[alpha], %[j], 1) \n\t" \
> +            "jne    2f \n\t"                            \
> +            "cmpl   $-1,     4(%[alpha], %[j], 1) \n\t" \
> +            "jne    2f \n\t"                            \
> +            "cmpl   $-1,     8(%[alpha], %[j], 1) \n\t" \
> +            "jne    2f \n\t"                            \
> +            "cmpl   $-1,    12(%[alpha], %[j], 1) \n\t" \
> +            "jne    2f \n\t"                            \
> +            "jmp    3f \n\t"

Just use 4 spaces for indentation, not some arbitrary amount.

> +    // u & v
> +    alpha = vf->priv->alphas[1];
> +    stride = vf->dmpi->stride[1];
> +    for (i = 0; i < outh / 2; i++) {
> +        size_t xmin = FFMIN(dr[i * 2].xmin, dr[i * 2 + 1].xmin) & ~31,
> +               xmax = FFMAX(dr[i * 2].xmax, dr[i * 2 + 1].xmax);
> +        __asm__ volatile (
> +                "jmp    4f \n\t"

same

This also affects some other inline asm you wrote.

Diego