On Mon, Jan 25, 2016 at 12:56:12PM -0500, Vittorio Giovara wrote:
> --- a/libavcodec/Makefile
> +++ b/libavcodec/Makefile
> @@ -208,7 +208,7 @@ OBJS-$(CONFIG_EAMAD_DECODER)           += eamad.o 
> eaidct.o mpeg12.o \
>  OBJS-$(CONFIG_EATGQ_DECODER)           += eatgq.o eaidct.o
>  OBJS-$(CONFIG_EATGV_DECODER)           += eatgv.o
> -OBJS-$(CONFIG_EATQI_DECODER)           += eatqi.o eaidct.o
> +OBJS-$(CONFIG_EATQI_DECODER)           += eatqi.o eaidct.o mpeg12.o 
> mpeg12data.o

Note to self: These object dependencies should be refactored some day.

> --- a/libavcodec/eatqi.c
> +++ b/libavcodec/eatqi.c
> @@ -35,42 +35,51 @@
>  
> -static int tqi_decode_mb(MpegEncContext *s, int16_t (*block)[64])
> +static int tqi_decode_mb(AVCodecContext *avctx, int16_t (*block)[64])
>  {
> +    TqiContext *t = avctx->priv_data;
>      int n;

This seems silly, just pass the TqiContext directly.

> @@ -78,31 +87,35 @@ static int tqi_decode_mb(MpegEncContext *s, int16_t 
> (*block)[64])
>      return 0;
>  }
>  
> -static inline void tqi_idct_put(TqiContext *t, AVFrame *frame, int16_t 
> (*block)[64])
> +static inline void tqi_idct_put(AVCodecContext *avctx, AVFrame *frame,
> +                                int16_t (*block)[64])
>  {
> -    MpegEncContext *s = &t->s;
> +    TqiContext *t = avctx->priv_data;

same

> -static void tqi_calculate_qtable(MpegEncContext *s, int quant)
> +static void tqi_calculate_qtable(AVCodecContext *avctx, int quant)
>  {
> +    TqiContext *t = avctx->priv_data;

same

> @@ -137,15 +149,15 @@ static int tqi_decode_frame(AVCodecContext *avctx,
> -    s->last_dc[0] = s->last_dc[1] = s->last_dc[2] = 0;
> +    memset(t->last_dc, 0, sizeof(t->last_dc));

This is unrelated, please don't.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to