Merge commit 'a7da517f6a5c472f46f67dd33bb6b95ccc919923'
authorDerek Buitenhuis <derek.buitenhuis@gmail.com>
Sun, 24 Apr 2016 10:54:53 +0000 (11:54 +0100)
committerDerek Buitenhuis <derek.buitenhuis@gmail.com>
Sun, 24 Apr 2016 10:54:53 +0000 (11:54 +0100)
* commit 'a7da517f6a5c472f46f67dd33bb6b95ccc919923':
  h264data: Move all data tables from a header to a .c file

Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
1  2 
libavcodec/Makefile
libavcodec/dxva2_h264.c
libavcodec/h264_cabac.c
libavcodec/h264_cavlc.c
libavcodec/h264_parser.c
libavcodec/h264_slice.c
libavcodec/h264data.c
libavcodec/h264data.h
libavcodec/svq3.c

@@@ -308,10 -251,10 +308,10 @@@ OBJS-$(CONFIG_H263_ENCODER)            
  OBJS-$(CONFIG_H264_DECODER)            += h264.o h264_cabac.o h264_cavlc.o \
                                            h264_direct.o h264_loopfilter.o  \
                                            h264_mb.o h264_picture.o h264_ps.o \
-                                           h264_refs.o h264_sei.o h264_slice.o
 -                                          h264_refs.o h264_sei.o \
 -                                          h264_slice.o h264data.o
++                                          h264_refs.o h264_sei.o h264_slice.o h264data.o
 +OBJS-$(CONFIG_H264_MEDIACODEC_DECODER) += mediacodecdec_h264.o
  OBJS-$(CONFIG_H264_MMAL_DECODER)       += mmaldec.o
 -OBJS-$(CONFIG_H264_NVENC_ENCODER)      += nvenc_h264.o
 +OBJS-$(CONFIG_H264_VDA_DECODER)        += vda_h264_dec.o
  OBJS-$(CONFIG_H264_QSV_DECODER)        += qsvdec_h2645.o
  OBJS-$(CONFIG_H264_QSV_ENCODER)        += qsvenc_h264.o
  OBJS-$(CONFIG_HAP_DECODER)             += hapdec.o hap.o
Simple merge
@@@ -2014,12 -2008,12 +2014,12 @@@ int ff_h264_decode_mb_cabac(const H264C
          mb_type = decode_cabac_intra_mb_type(sl, 3, 1);
          if (sl->slice_type == AV_PICTURE_TYPE_SI && mb_type)
              mb_type--;
 -        assert(sl->slice_type_nos == AV_PICTURE_TYPE_I);
 +        av_assert2(sl->slice_type_nos == AV_PICTURE_TYPE_I);
  decode_intra_mb:
          partition_count = 0;
-         cbpi_mb_type_info[mb_type].cbp;
-         sl->intra16x16_pred_mode = i_mb_type_info[mb_type].pred_mode;
-         mb_typei_mb_type_info[mb_type].type;
+         cbp                      = ff_h264_i_mb_type_info[mb_type].cbp;
+         sl->intra16x16_pred_mode = ff_h264_i_mb_type_info[mb_type].pred_mode;
+         mb_type                  = ff_h264_i_mb_type_info[mb_type].type;
      }
      if (MB_FIELD(sl))
          mb_type |= MB_TYPE_INTERLACED;
  #include "internal.h"
  #include "avcodec.h"
  #include "h264.h"
- #include "h264data.h" // FIXME FIXME FIXME
  #include "h264_mvpred.h"
+ #include "h264data.h"
  #include "golomb.h"
  #include "mpegutils.h"
 +#include "libavutil/avassert.h"
  
 -#include <assert.h>
  
  static const uint8_t golomb_to_inter_cbp_gray[16]={
   0, 1, 2, 4, 8, 3, 5,10,12,15, 7,11,13,14, 6, 9,
@@@ -1142,9 -1130,8 +1144,9 @@@ decode_intra_mb
              if(cbp&0x30){
                  for(chroma_idx=0; chroma_idx<2; chroma_idx++)
                      if (decode_residual(h, sl, gb, sl->mb + ((256 + 16*16*chroma_idx) << pixel_shift),
-                                         CHROMA_DC_BLOCK_INDEX+chroma_idx,
-                                         CHROMA422(h) ? chroma422_dc_scan : chroma_dc_scan,
-                                         NULL, 4*num_c8x8) < 0) {
 -                                        CHROMA_DC_BLOCK_INDEX + chroma_idx, ff_h264_chroma422_dc_scan,
 -                                        NULL, 8) < 0) {
++                                        CHROMA_DC_BLOCK_INDEX + chroma_idx,
++                                        CHROMA422(h) ? ff_h264_chroma422_dc_scan : ff_h264_chroma_dc_scan,
++                                        NULL, 4 * num_c8x8) < 0) {
                          return -1;
                      }
              }
@@@ -310,9 -259,9 +310,9 @@@ static inline int parse_nal_units(AVCod
          /* fall through */
          case NAL_SLICE:
              init_get_bits(&sl->gb, ptr, 8 * dst_length);
 -            get_ue_golomb(&sl->gb);  // skip first_mb_in_slice
 +            get_ue_golomb_long(&sl->gb);  // skip first_mb_in_slice
              slice_type   = get_ue_golomb_31(&sl->gb);
-             s->pict_type = golomb_to_pict_type[slice_type % 5];
+             s->pict_type = ff_h264_golomb_to_pict_type[slice_type % 5];
              if (h->sei_recovery_frame_cnt >= 0) {
                  /* key frame, since recovery_frame_cnt is set */
                  s->key_frame = 1;
Simple merge
index 0000000,f7aed48..35b0a07
mode 000000,100644..100644
--- /dev/null
@@@ -1,0 -1,149 +1,149 @@@
 - * This file is part of Libav.
+ /*
+  * H26L/H264/AVC/JVT/14496-10/... encoder/decoder
+  * Copyright (c) 2003 Michael Niedermayer <michaelni@gmx.at>
+  *
 - * Libav is free software; you can redistribute it and/or
++ * This file is part of FFmpeg.
+  *
 - * Libav is distributed in the hope that it will be useful,
++ * FFmpeg is free software; you can redistribute it and/or
+  * modify it under the terms of the GNU Lesser General Public
+  * License as published by the Free Software Foundation; either
+  * version 2.1 of the License, or (at your option) any later version.
+  *
 - * License along with Libav; if not, write to the Free Software
++ * FFmpeg is distributed in the hope that it will be useful,
+  * but WITHOUT ANY WARRANTY; without even the implied warranty of
+  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+  * Lesser General Public License for more details.
+  *
+  * You should have received a copy of the GNU Lesser General Public
++ * License along with FFmpeg; if not, write to the Free Software
+  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+  */
+ /**
+  * @file
+  * @brief
+  *     H264 / AVC / MPEG4 part10 codec data table
+  * @author Michael Niedermayer <michaelni@gmx.at>
+  */
+ #include <stdint.h>
+ #include "libavutil/avutil.h"
+ #include "avcodec.h"
+ #include "h264data.h"
+ const uint8_t ff_h264_golomb_to_pict_type[5] = {
+     AV_PICTURE_TYPE_P, AV_PICTURE_TYPE_B, AV_PICTURE_TYPE_I,
+     AV_PICTURE_TYPE_SP, AV_PICTURE_TYPE_SI
+ };
+ const uint8_t ff_h264_golomb_to_intra4x4_cbp[48] = {
+     47, 31, 15, 0,  23, 27, 29, 30, 7,  11, 13, 14, 39, 43, 45, 46,
+     16, 3,  5,  10, 12, 19, 21, 26, 28, 35, 37, 42, 44, 1,  2,  4,
+     8,  17, 18, 20, 24, 6,  9,  22, 25, 32, 33, 34, 36, 40, 38, 41
+ };
+ const uint8_t ff_h264_golomb_to_inter_cbp[48] = {
+     0,  16, 1,  2,  4,  8,  32, 3,  5,  10, 12, 15, 47, 7,  11, 13,
+     14, 6,  9,  31, 35, 37, 42, 44, 33, 34, 36, 40, 39, 43, 45, 46,
+     17, 18, 20, 24, 19, 21, 26, 28, 23, 27, 29, 30, 22, 25, 38, 41
+ };
+ const uint8_t ff_h264_chroma_dc_scan[4] = {
+     (0 + 0 * 2) * 16, (1 + 0 * 2) * 16,
+     (0 + 1 * 2) * 16, (1 + 1 * 2) * 16,
+ };
+ const uint8_t ff_h264_chroma422_dc_scan[8] = {
+     (0 + 0 * 2) * 16, (0 + 1 * 2) * 16,
+     (1 + 0 * 2) * 16, (0 + 2 * 2) * 16,
+     (0 + 3 * 2) * 16, (1 + 1 * 2) * 16,
+     (1 + 2 * 2) * 16, (1 + 3 * 2) * 16,
+ };
+ const IMbInfo ff_h264_i_mb_type_info[26] = {
+     { MB_TYPE_INTRA4x4,  -1,  -1 },
+     { MB_TYPE_INTRA16x16, 2,   0 },
+     { MB_TYPE_INTRA16x16, 1,   0 },
+     { MB_TYPE_INTRA16x16, 0,   0 },
+     { MB_TYPE_INTRA16x16, 3,   0 },
+     { MB_TYPE_INTRA16x16, 2,  16 },
+     { MB_TYPE_INTRA16x16, 1,  16 },
+     { MB_TYPE_INTRA16x16, 0,  16 },
+     { MB_TYPE_INTRA16x16, 3,  16 },
+     { MB_TYPE_INTRA16x16, 2,  32 },
+     { MB_TYPE_INTRA16x16, 1,  32 },
+     { MB_TYPE_INTRA16x16, 0,  32 },
+     { MB_TYPE_INTRA16x16, 3,  32 },
+     { MB_TYPE_INTRA16x16, 2,  15 +  0 },
+     { MB_TYPE_INTRA16x16, 1,  15 +  0 },
+     { MB_TYPE_INTRA16x16, 0,  15 +  0 },
+     { MB_TYPE_INTRA16x16, 3,  15 +  0 },
+     { MB_TYPE_INTRA16x16, 2,  15 + 16 },
+     { MB_TYPE_INTRA16x16, 1,  15 + 16 },
+     { MB_TYPE_INTRA16x16, 0,  15 + 16 },
+     { MB_TYPE_INTRA16x16, 3,  15 + 16 },
+     { MB_TYPE_INTRA16x16, 2,  15 + 32 },
+     { MB_TYPE_INTRA16x16, 1,  15 + 32 },
+     { MB_TYPE_INTRA16x16, 0,  15 + 32 },
+     { MB_TYPE_INTRA16x16, 3,  15 + 32 },
+     { MB_TYPE_INTRA_PCM,  -1, -1 },
+ };
+ const PMbInfo ff_h264_p_mb_type_info[5] = {
+     { MB_TYPE_16x16 | MB_TYPE_P0L0,                               1 },
+     { MB_TYPE_16x8  | MB_TYPE_P0L0 | MB_TYPE_P1L0,                2 },
+     { MB_TYPE_8x16  | MB_TYPE_P0L0 | MB_TYPE_P1L0,                2 },
+     { MB_TYPE_8x8   | MB_TYPE_P0L0 | MB_TYPE_P1L0,                4 },
+     { MB_TYPE_8x8   | MB_TYPE_P0L0 | MB_TYPE_P1L0 | MB_TYPE_REF0, 4 },
+ };
+ const PMbInfo ff_h264_p_sub_mb_type_info[4] = {
+     { MB_TYPE_16x16 | MB_TYPE_P0L0, 1 },
+     { MB_TYPE_16x8  | MB_TYPE_P0L0, 2 },
+     { MB_TYPE_8x16  | MB_TYPE_P0L0, 2 },
+     { MB_TYPE_8x8   | MB_TYPE_P0L0, 4 },
+ };
+ const PMbInfo ff_h264_b_mb_type_info[23] = {
+     { MB_TYPE_DIRECT2 | MB_TYPE_L0L1,                                              1, },
+     { MB_TYPE_16x16   | MB_TYPE_P0L0,                                              1, },
+     { MB_TYPE_16x16   | MB_TYPE_P0L1,                                              1, },
+     { MB_TYPE_16x16   | MB_TYPE_P0L0 | MB_TYPE_P0L1,                               1, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               2, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               2, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P1L1,                               2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P1L1,                               2, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L1 | MB_TYPE_P1L0,                               2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L1 | MB_TYPE_P1L0,                               2, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P1L0 | MB_TYPE_P1L1,                2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P1L0 | MB_TYPE_P1L1,                2, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1,                2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1,                2, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0,                2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0,                2, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L1,                2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L1,                2, },
+     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 2, },
+     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 2, },
+     { MB_TYPE_8x8     | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 4, },
+ };
+ const PMbInfo ff_h264_b_sub_mb_type_info[13] = {
+     { MB_TYPE_DIRECT2,                                                           1, },
+     { MB_TYPE_16x16 | MB_TYPE_P0L0,                                              1, },
+     { MB_TYPE_16x16 | MB_TYPE_P0L1,                                              1, },
+     { MB_TYPE_16x16 | MB_TYPE_P0L0 | MB_TYPE_P0L1,                               1, },
+     { MB_TYPE_16x8  | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               2, },
+     { MB_TYPE_8x16  | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               2, },
+     { MB_TYPE_16x8  | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               2, },
+     { MB_TYPE_8x16  | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               2, },
+     { MB_TYPE_16x8  | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 2, },
+     { MB_TYPE_8x16  | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 2, },
+     { MB_TYPE_8x8   | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               4, },
+     { MB_TYPE_8x8   | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               4, },
+     { MB_TYPE_8x8   | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 4, },
+ };
@@@ -1,10 -1,7 +1,7 @@@
  /*
-  * H26L/H264/AVC/JVT/14496-10/... encoder/decoder
-  * Copyright (c) 2003 Michael Niedermayer <michaelni@gmx.at>
-  *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
@@@ -103,80 -43,9 +43,28 @@@ typedef struct PMbInfo 
      uint8_t partition_count;
  } PMbInfo;
  
- static const PMbInfo p_mb_type_info[5] = {
-     { MB_TYPE_16x16 | MB_TYPE_P0L0,                               1 },
-     { MB_TYPE_16x8  | MB_TYPE_P0L0 | MB_TYPE_P1L0,                2 },
-     { MB_TYPE_8x16  | MB_TYPE_P0L0 | MB_TYPE_P1L0,                2 },
-     { MB_TYPE_8x8   | MB_TYPE_P0L0 | MB_TYPE_P1L0,                4 },
-     { MB_TYPE_8x8   | MB_TYPE_P0L0 | MB_TYPE_P1L0 | MB_TYPE_REF0, 4 },
- };
- static const PMbInfo p_sub_mb_type_info[4] = {
-     { MB_TYPE_16x16 | MB_TYPE_P0L0, 1 },
-     { MB_TYPE_16x8  | MB_TYPE_P0L0, 2 },
-     { MB_TYPE_8x16  | MB_TYPE_P0L0, 2 },
-     { MB_TYPE_8x8   | MB_TYPE_P0L0, 4 },
- };
- static const PMbInfo b_mb_type_info[23] = {
-     { MB_TYPE_DIRECT2 | MB_TYPE_L0L1,                                              1, },
-     { MB_TYPE_16x16   | MB_TYPE_P0L0,                                              1, },
-     { MB_TYPE_16x16   | MB_TYPE_P0L1,                                              1, },
-     { MB_TYPE_16x16   | MB_TYPE_P0L0 | MB_TYPE_P0L1,                               1, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               2, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               2, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P1L1,                               2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P1L1,                               2, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L1 | MB_TYPE_P1L0,                               2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L1 | MB_TYPE_P1L0,                               2, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P1L0 | MB_TYPE_P1L1,                2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P1L0 | MB_TYPE_P1L1,                2, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1,                2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1,                2, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0,                2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0,                2, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L1,                2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L1,                2, },
-     { MB_TYPE_16x8    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 2, },
-     { MB_TYPE_8x16    | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 2, },
-     { MB_TYPE_8x8     | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 4, },
- };
- static const PMbInfo b_sub_mb_type_info[13] = {
-     { MB_TYPE_DIRECT2,                                                           1, },
-     { MB_TYPE_16x16 | MB_TYPE_P0L0,                                              1, },
-     { MB_TYPE_16x16 | MB_TYPE_P0L1,                                              1, },
-     { MB_TYPE_16x16 | MB_TYPE_P0L0 | MB_TYPE_P0L1,                               1, },
-     { MB_TYPE_16x8  | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               2, },
-     { MB_TYPE_8x16  | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               2, },
-     { MB_TYPE_16x8  | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               2, },
-     { MB_TYPE_8x16  | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               2, },
-     { MB_TYPE_16x8  | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 2, },
-     { MB_TYPE_8x16  | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 2, },
-     { MB_TYPE_8x8   | MB_TYPE_P0L0 | MB_TYPE_P1L0,                               4, },
-     { MB_TYPE_8x8   | MB_TYPE_P0L1 | MB_TYPE_P1L1,                               4, },
-     { MB_TYPE_8x8   | MB_TYPE_P0L0 | MB_TYPE_P0L1 | MB_TYPE_P1L0 | MB_TYPE_P1L1, 4, },
- };
+ extern const PMbInfo ff_h264_p_mb_type_info[5];
+ extern const PMbInfo ff_h264_p_sub_mb_type_info[4];
+ extern const PMbInfo ff_h264_b_mb_type_info[23];
+ extern const PMbInfo ff_h264_b_sub_mb_type_info[13];
  
 +static const AVRational ff_h264_pixel_aspect[17] = {
 +    {   0,  1 },
 +    {   1,  1 },
 +    {  12, 11 },
 +    {  10, 11 },
 +    {  16, 11 },
 +    {  40, 33 },
 +    {  24, 11 },
 +    {  20, 11 },
 +    {  32, 11 },
 +    {  80, 33 },
 +    {  18, 11 },
 +    {  15, 11 },
 +    {  64, 33 },
 +    { 160, 99 },
 +    {   4,  3 },
 +    {   3,  2 },
 +    {   2,  1 },
 +};
  #endif /* AVCODEC_H264DATA_H */
@@@ -246,10 -239,10 +244,10 @@@ static inline int svq3_decode_block(Get
                                      int index, const int type)
  {
      static const uint8_t *const scan_patterns[4] = {
-         luma_dc_zigzag_scan, ff_zigzag_scan, svq3_scan, chroma_dc_scan
+         luma_dc_zigzag_scan, ff_zigzag_scan, svq3_scan, ff_h264_chroma_dc_scan
      };
  
 -    int run, level, limit;
 +    int run, level, sign, limit;
      unsigned vlc;
      const int intra           = 3 * type >> 2;
      const uint8_t *const scan = scan_patterns[type];