[FFmpeg-devel] [PATCH 1/2] avcodec/mpegvideo: Use intra_scantable in dct_unquantize_h263_intra_c()

[Kieran Kunhya]

On Tue, 20 Jun 2017, 00:38 Michael Niedermayer, <michael at niedermayer.cc>
wrote:

> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/mpegvideo.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c
> index 63a30b93ce..e29558b3a2 100644
> --- a/libavcodec/mpegvideo.c
> +++ b/libavcodec/mpegvideo.c
> @@ -225,7 +225,7 @@ static void dct_unquantize_h263_intra_c(MpegEncContext
> *s,
>      if(s->ac_pred)
>          nCoeffs=63;
>      else
> -        nCoeffs= s->inter_scantable.raster_end[ s->block_last_index[n] ];
> +        nCoeffs= s->intra_scantable.raster_end[ s->block_last_index[n] ];
>
>      for(i=1; i<=nCoeffs; i++) {
>          level = block[i];
> --
> 2.13.0
>

Are you able to document some of this, Ronald and James spent a lot of time
trying to understand this undocumented code and this patch set doesn't
explain anything better.

Kieran

>