On Fri, 23 May 2014 07:37:18 -0700, Diego Biurrun <di...@biurrun.de> wrote:
> Also shorten HuffYUV context member names to avoid clutter.
> @@ -503,9 +505,15 @@ static int encode_frame(AVCodecContext *avctx, AVPacket 
> *pkt,
>              lefttopy = p->data[0][3];
>              lefttopu = p->data[1][1];
>              lefttopv = p->data[2][1];
> -            s->dsp.sub_hfyu_median_prediction(s->temp[0], p->data[0]+4, 
> p->data[0] + fake_ystride + 4, width - 4 , &lefty, &lefttopy);
> -            s->dsp.sub_hfyu_median_prediction(s->temp[1], p->data[1]+2, 
> p->data[1] + fake_ustride + 2, width2 - 2, &leftu, &lefttopu);
> -            s->dsp.sub_hfyu_median_prediction(s->temp[2], p->data[2]+2, 
> p->data[2] + fake_vstride + 2, width2 - 2, &leftv, &lefttopv);
> +            s->hencdsp.sub_hfyu_median_pred(s->temp[0], p->data[0] + 4,
> +                                            p->data[0] + fake_ystride + 4,
> +                                            width - 4, &lefty, &lefttopy);
> +            s->hencdsp.sub_hfyu_median_pred(s->temp[1], p->data[1] + 2,
> +                                            p->data[1] + fake_ustride + 2,
> +                                            width2 - 2, &leftu, &lefttopu);
> +            s->hencdsp.sub_hfyu_median_pred(s->temp[2], p->data[2] + 2,
> +                                            p->data[2] + fake_vstride + 2,
> +                                            width2 - 2, &leftv, &lefttopv);

nit: i think it was more readable with the long lines.

looks fine otherwise

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to