Merge commit 'ee0ebd3c1412fdd9d80aa97c98d1a20b893f1f47'
authorMichael Niedermayer <michaelni@gmx.at>
Tue, 9 Sep 2014 17:28:59 +0000 (19:28 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Tue, 9 Sep 2014 17:30:55 +0000 (19:30 +0200)
* commit 'ee0ebd3c1412fdd9d80aa97c98d1a20b893f1f47':
  dv: K&R formatting cosmetics

Conflicts:
libavcodec/dv.c
libavcodec/dv.h
libavcodec/dv_profile.c
libavcodec/dvdec.c
libavcodec/dvenc.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavcodec/dv.c
libavcodec/dv.h
libavcodec/dv_profile.c
libavcodec/dv_profile.h
libavcodec/dv_tablegen.h
libavcodec/dvdata.c
libavcodec/dvdec.c
libavcodec/dvenc.c

diff --cc libavcodec/dv.c
@@@ -267,12 -274,12 +274,12 @@@ av_cold int ff_dvvideo_init(AVCodecCont
          }
  
          /* NOTE: as a trick, we use the fact the no codes are unused
-            to accelerate the parsing of partial codes */
-         init_vlc(&dv_vlc, TEX_VLC_BITS, j,
-                  new_dv_vlc_len, 1, 1, new_dv_vlc_bits, 2, 2, 0);
+          * to accelerate the parsing of partial codes */
+         init_vlc(&dv_vlc, TEX_VLC_BITS, j, new_dv_vlc_len,
+                  1, 1, new_dv_vlc_bits, 2, 2, 0);
 -        assert(dv_vlc.table_size == 1184);
 +        av_assert1(dv_vlc.table_size == 1184);
  
-         for (i = 0; i < dv_vlc.table_size; i++){
+         for (i = 0; i < dv_vlc.table_size; i++) {
              int code = dv_vlc.table[i][0];
              int len  = dv_vlc.table[i][1];
              int level, run;
diff --cc libavcodec/dv.h
@@@ -43,9 -43,9 +43,9 @@@ typedef struct DVVideoContext 
      AVCodecContext  *avctx;
      uint8_t         *buf;
  
-     uint8_t  dv_zigzag[2][64];
+     uint8_t dv_zigzag[2][64];
  
 -    void (*get_pixels)(int16_t *block, const uint8_t *pixels, int line_size);
 +    void (*get_pixels)(int16_t *block, const uint8_t *pixels, ptrdiff_t line_size);
      void (*fdct[2])(int16_t *block);
      void (*idct_put[2])(uint8_t *dest, int line_size, int16_t *block);
      me_cmp_func ildct_cmp;
@@@ -80,7 -80,10 +80,6 @@@ enum dv_pack_type 
  #define DV_PROFILE_IS_1080i50(p) (((p)->video_stype == 0x14) && ((p)->dsf == 1))
  #define DV_PROFILE_IS_720p50(p)  (((p)->video_stype == 0x18) && ((p)->dsf == 1))
  
 -/* minimum number of bytes to read from a DV stream in order to
 - * determine the profile */
 -#define DV_PROFILE_BYTES (6 * 80) /* 6 DIF blocks */
--
  /**
   * largest possible DV frame, in bytes (1080i50)
   */
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
- #include <stdint.h>
  #include "config.h"
  
+ #include <stdint.h>
  #include "libavutil/common.h"
 +#include "libavutil/intreadwrite.h"
  #include "libavutil/log.h"
  #include "libavutil/pixdesc.h"
  #include "avcodec.h"
  #include "dv_profile.h"
  #include "dv_profile_internal.h"
@@@ -265,31 -256,22 +257,30 @@@ void ff_dv_print_profiles(void *logctx
  
  #endif /* CONFIG_DVPROFILE */
  
 -const AVDVProfile *av_dv_frame_profile(const AVDVProfile *sys,
 +const AVDVProfile* avpriv_dv_frame_profile2(AVCodecContext* codec, const AVDVProfile *sys,
-                                        const uint8_tframe, unsigned buf_size)
+                                        const uint8_t *frame, unsigned buf_size)
  {
  #if CONFIG_DVPROFILE
      int i, dsf, stype;
  
 -    if (buf_size < 80 * 5 + 48 + 4)
 +    if(buf_size < DV_PROFILE_BYTES)
          return NULL;
  
-     dsf = (frame[3] & 0x80) >> 7;
+     dsf   = (frame[3] & 0x80) >> 7;
      stype = frame[80 * 5 + 48 + 3] & 0x1f;
  
      /* 576i50 25Mbps 4:1:1 is a special case */
 -    if (dsf == 1 && stype == 0 && frame[4] & 0x07 /* the APT field */)
 +    if ((dsf == 1 && stype == 0 && frame[4] & 0x07 /* the APT field */) ||
-         (stype == 31 && codec && codec->codec_tag==AV_RL32("SL25") && codec->coded_width==720 && codec->coded_height==576)) {
++        (stype == 31 && codec && codec->codec_tag==AV_RL32("SL25") && codec->coded_width==720 && codec->coded_height==576))
          return &dv_profiles[2];
-     }
  
 +    if(   stype == 0
 +       && codec
 +       && (codec->codec_tag==AV_RL32("dvsd") || codec->codec_tag==AV_RL32("CDVC"))
 +       && codec->coded_width ==720
 +       && codec->coded_height==576)
 +        return &dv_profiles[1];
 +
      for (i = 0; i < FF_ARRAY_ELEMS(dv_profiles); i++)
          if (dsf == dv_profiles[i].dsf && stype == dv_profiles[i].video_stype)
              return &dv_profiles[i];
      return NULL;
  }
  
-                                        const uint8_t* frame, unsigned buf_size)
 +const AVDVProfile *av_dv_frame_profile(const AVDVProfile *sys,
++                                       const uint8_t *frame, unsigned buf_size)
 +{
 +    return avpriv_dv_frame_profile2(NULL, sys, frame, buf_size);
 +}
 +
  const AVDVProfile *av_dv_codec_profile(int width, int height,
                                         enum AVPixelFormat pix_fmt)
  {
  #include "libavutil/rational.h"
  #include "avcodec.h"
  
-    determine the profile */
- #define DV_PROFILE_BYTES (6*80) /* 6 DIF blocks */
 +/* minimum number of bytes to read from a DV stream in order to
++ * determine the profile */
++#define DV_PROFILE_BYTES (6 * 80) /* 6 DIF blocks */
++
 +
  /*
   * AVDVProfile is used to express the differences between various
   * DV flavors. For now it's primarily used for differentiating
Simple merge
Simple merge
   * DV decoder
   */
  
- #include "libavutil/internal.h"
 +#include "libavutil/avassert.h"
  #include "libavutil/imgutils.h"
+ #include "libavutil/internal.h"
  #include "libavutil/pixdesc.h"
  #include "avcodec.h"
+ #include "dv.h"
+ #include "dvdata.h"
+ #include "get_bits.h"
  #include "idctdsp.h"
  #include "internal.h"
- #include "get_bits.h"
  #include "put_bits.h"
  #include "simple_idct.h"
- #include "dvdata.h"
- #include "dv.h"
  
  typedef struct BlockInfo {
      const uint32_t *factor_table;
@@@ -70,15 -69,9 +71,15 @@@ static av_cold int dvvideo_decode_init(
      ff_idctdsp_init(&idsp, avctx);
  
      for (i = 0; i < 64; i++)
-        s->dv_zigzag[0][i] = idsp.idct_permutation[ff_zigzag_direct[i]];
+         s->dv_zigzag[0][i] = idsp.idct_permutation[ff_zigzag_direct[i]];
  
 -    memcpy(s->dv_zigzag[1], ff_dv_zigzag248_direct, sizeof(s->dv_zigzag[1]));
 +    if (avctx->lowres){
 +        for (i = 0; i < 64; i++){
 +            int j = ff_dv_zigzag248_direct[i];
 +            s->dv_zigzag[1][i] = idsp.idct_permutation[(j & 7) + (j & 8) * 4 + (j & 48) / 2];
 +        }
 +    }else
 +        memcpy(s->dv_zigzag[1], ff_dv_zigzag248_direct, sizeof(s->dv_zigzag[1]));
  
      s->idct_put[0] = idsp.idct_put;
      s->idct_put[1] = ff_simple_idct248_put;
@@@ -172,19 -166,19 +174,19 @@@ static int dv_decode_video_segment(AVCo
      PutBitContext pb, vs_pb;
      GetBitContext gb;
      BlockInfo mb_data[5 * DV_MAX_BPM], *mb, *mb1;
-     LOCAL_ALIGNED_16(int16_t, sblock, [5*DV_MAX_BPM], [64]);
-     LOCAL_ALIGNED_16(uint8_t, mb_bit_buffer, [  80 + FF_INPUT_BUFFER_PADDING_SIZE]); /* allow some slack */
-     LOCAL_ALIGNED_16(uint8_t, vs_bit_buffer, [5*80 + FF_INPUT_BUFFER_PADDING_SIZE]); /* allow some slack */
+     LOCAL_ALIGNED_16(int16_t, sblock, [5 * DV_MAX_BPM], [64]);
+     LOCAL_ALIGNED_16(uint8_t, mb_bit_buffer, [80     + FF_INPUT_BUFFER_PADDING_SIZE]); /* allow some slack */
+     LOCAL_ALIGNED_16(uint8_t, vs_bit_buffer, [80 * 5 + FF_INPUT_BUFFER_PADDING_SIZE]); /* allow some slack */
 -    const int log2_blocksize = 3;
 +    const int log2_blocksize = 3-s->avctx->lowres;
      int is_field_mode[5];
  
-     av_assert1((((int)mb_bit_buffer) & 7) == 0);
-     av_assert1((((int)vs_bit_buffer) & 7) == 0);
 -    assert((((int) mb_bit_buffer) & 7) == 0);
 -    assert((((int) vs_bit_buffer) & 7) == 0);
++    av_assert1((((int) mb_bit_buffer) & 7) == 0);
++    av_assert1((((int) vs_bit_buffer) & 7) == 0);
  
-     memset(sblock, 0, 5*DV_MAX_BPM*sizeof(*sblock));
+     memset(sblock, 0, 5 * DV_MAX_BPM * sizeof(*sblock));
  
      /* pass 1: read DC and AC coefficients in blocks */
-     buf_ptr = &s->buf[work_chunk->buf_offset*80];
+     buf_ptr = &s->buf[work_chunk->buf_offset * 80];
      block1  = &sblock[0][0];
      mb1     = mb_data;
      init_put_bits(&vs_pb, vs_bit_buffer, 5 * 80);
          for (j = 2; j; j--) {
              uint8_t *c_ptr = s->frame->data[j] + c_offset;
              if (s->sys->pix_fmt == AV_PIX_FMT_YUV411P && mb_x >= (704 / 8)) {
-                   uint64_t aligned_pixels[64/8];
-                   uint8_t *pixels = (uint8_t*)aligned_pixels;
-                   uint8_t *c_ptr1, *ptr1;
-                   int x, y;
-                   mb->idct_put(pixels, 8, block);
-                   for (y = 0; y < (1 << log2_blocksize); y++, c_ptr += s->frame->linesize[j], pixels += 8) {
-                       ptr1   = pixels + ((1 << (log2_blocksize))>>1);
-                       c_ptr1 = c_ptr + (s->frame->linesize[j] << log2_blocksize);
-                       for (x = 0; x < (1 << FFMAX(log2_blocksize - 1, 0)); x++) {
-                           c_ptr[x]  = pixels[x];
-                           c_ptr1[x] = ptr1[x];
-                       }
-                   }
-                   block += 64; mb++;
+                 uint64_t aligned_pixels[64 / 8];
+                 uint8_t *pixels = (uint8_t *) aligned_pixels;
+                 uint8_t *c_ptr1, *ptr1;
+                 int x, y;
+                 mb->idct_put(pixels, 8, block);
+                 for (y = 0; y < (1 << log2_blocksize); y++, c_ptr += s->frame->linesize[j], pixels += 8) {
 -                    ptr1   = pixels + (1 << (log2_blocksize - 1));
++                    ptr1   = pixels + ((1 << (log2_blocksize))>>1);
+                     c_ptr1 = c_ptr + (s->frame->linesize[j] << log2_blocksize);
 -                    for (x = 0; x < (1 << (log2_blocksize - 1)); x++) {
++                    for (x = 0; x < (1 << FFMAX(log2_blocksize - 1, 0)); x++) {
+                         c_ptr[x]  = pixels[x];
+                         c_ptr1[x] = ptr1[x];
+                     }
+                 }
+                 block += 64;
+                 mb++;
              } else {
-                   y_stride = (mb_y == 134) ? (1 << log2_blocksize) :
-                                              s->frame->linesize[j] << ((!is_field_mode[mb_index]) * log2_blocksize);
-                   linesize = s->frame->linesize[j] << is_field_mode[mb_index];
-                   (mb++)->    idct_put(c_ptr           , linesize, block); block += 64;
-                   if (s->sys->bpm == 8) {
-                       (mb++)->idct_put(c_ptr + y_stride, linesize, block); block += 64;
-                   }
+                 y_stride = (mb_y == 134) ? (1 << log2_blocksize) :
+                            s->frame->linesize[j] << ((!is_field_mode[mb_index]) * log2_blocksize);
+                 linesize = s->frame->linesize[j] << is_field_mode[mb_index];
+                 (mb++)->idct_put(c_ptr, linesize, block);
+                 block += 64;
+                 if (s->sys->bpm == 8) {
+                     (mb++)->idct_put(c_ptr + y_stride, linesize, block);
+                     block += 64;
+                 }
              }
          }
      }
@@@ -377,10 -380,11 +388,11 @@@ static int dvvideo_decode_frame(AVCodec
          return ret;
  
      /* Determine the codec's sample_aspect ratio from the packet */
-     vsc_pack = buf + 80*5 + 48 + 5;
-     if ( *vsc_pack == dv_video_control ) {
-         apt = buf[4] & 0x07;
-         is16_9 = (vsc_pack[2] & 0x07) == 0x02 || (!apt && (vsc_pack[2] & 0x07) == 0x07);
+     vsc_pack = buf + 80 * 5 + 48 + 5;
+     if (*vsc_pack == dv_video_control) {
+         apt    = buf[4] & 0x07;
 -        is16_9 = (vsc_pack && ((vsc_pack[2] & 0x07) == 0x02 ||
 -                               (!apt && (vsc_pack[2] & 0x07) == 0x07)));
++        is16_9 = (vsc_pack[2] & 0x07) == 0x02 ||
++                               (!apt && (vsc_pack[2] & 0x07) == 0x07);
          ff_set_sar(avctx, s->sys->sar[is16_9]);
      }
  
@@@ -244,29 -244,32 +251,32 @@@ static av_always_inline int dv_init_enc
      LOCAL_ALIGNED_16(int16_t, blk, [64]);
      int i, area;
      /* We offer two different methods for class number assignment: the
-        method suggested in SMPTE 314M Table 22, and an improved
-        method. The SMPTE method is very conservative; it assigns class
-        3 (i.e. severe quantization) to any block where the largest AC
-        component is greater than 36. FFmpeg's DV encoder tracks AC bit
-        consumption precisely, so there is no need to bias most blocks
-        towards strongly lossy compression. Instead, we assign class 2
-        to most blocks, and use class 3 only when strictly necessary
-        (for blocks whose largest AC component exceeds 255). */
+      * method suggested in SMPTE 314M Table 22, and an improved
+      * method. The SMPTE method is very conservative; it assigns class
+      * 3 (i.e. severe quantization) to any block where the largest AC
 -     * component is greater than 36. Libav's DV encoder tracks AC bit
++     * component is greater than 36. FFmpeg's DV encoder tracks AC bit
+      * consumption precisely, so there is no need to bias most blocks
+      * towards strongly lossy compression. Instead, we assign class 2
+      * to most blocks, and use class 3 only when strictly necessary
+      * (for blocks whose largest AC component exceeds 255). */
  
  #if 0 /* SMPTE spec method */
-     static const int classes[] = {12, 24, 36, 0xffff};
+     static const int classes[] = { 12, 24, 36, 0xffff };
 -#else /* improved Libav method */
 +#else /* improved FFmpeg method */
-     static const int classes[] = {-1, -1, 255, 0xffff};
+     static const int classes[] = { -1, -1, 255, 0xffff };
  #endif
      int max  = classes[0];
      int prev = 0;
  
-     av_assert2((((int)blk) & 15) == 0);
 -    assert((((int) blk) & 15) == 0);
++    av_assert2((((int) blk) & 15) == 0);
  
-     bi->area_q[0] = bi->area_q[1] = bi->area_q[2] = bi->area_q[3] = 0;
-     bi->partial_bit_count = 0;
+     bi->area_q[0]          =
+     bi->area_q[1]          =
+     bi->area_q[2]          =
+     bi->area_q[3]          = 0;
+     bi->partial_bit_count  = 0;
      bi->partial_bit_buffer = 0;
-     bi->cur_ac = 0;
+     bi->cur_ac             = 0;
      if (data) {
          bi->dct_mode = dv_guess_dct_mode(s, data, linesize);
          s->get_pixels(blk, data, linesize);
      bi->mb[0] = blk[0];
  
      zigzag_scan = bi->dct_mode ? ff_dv_zigzag248_direct : ff_zigzag_direct;
-     weight = bi->dct_mode ? dv_weight_248 : dv_weight_88;
+     weight      = bi->dct_mode ? dv_weight_248 : dv_weight_88;
  
      for (area = 0; area < 4; area++) {
-        bi->prev[area]     = prev;
-        bi->bit_size[area] = 1; // 4 areas 4 bits for EOB :)
-        for (i = mb_area_start[area]; i < mb_area_start[area+1]; i++) {
-           int level = blk[zigzag_scan[i]];
-           if (level + 15 > 30U) {
-               bi->sign[i] = (level >> 31) & 1;
-               /* weight it and shift down into range, adding for rounding */
-               /* the extra division by a factor of 2^4 reverses the 8x expansion of the DCT
-                  AND the 2x doubling of the weights */
-               level = (FFABS(level) * weight[i] + (1 << (dv_weight_bits+3))) >> (dv_weight_bits+4);
-               bi->mb[i] = level;
-               if (level > max)
-                   max = level;
-               bi->bit_size[area] += dv_rl2vlc_size(i - prev  - 1, level);
-               bi->next[prev]= i;
-               prev = i;
-           }
-        }
+         bi->prev[area]     = prev;
+         bi->bit_size[area] = 1; // 4 areas 4 bits for EOB :)
+         for (i = mb_area_start[area]; i < mb_area_start[area + 1]; i++) {
+             int level = blk[zigzag_scan[i]];
+             if (level + 15 > 30U) {
+                 bi->sign[i] = (level >> 31) & 1;
 -                /* Weight it and and shift down into range, adding for rounding.
++                /* Weight it and shift down into range, adding for rounding.
+                  * The extra division by a factor of 2^4 reverses the 8x
+                  * expansion of the DCT AND the 2x doubling of the weights. */
+                 level     = (FFABS(level) * weight[i] + (1 << (dv_weight_bits + 3))) >>
+                             (dv_weight_bits + 4);
+                 bi->mb[i] = level;
+                 if (level > max)
+                     max = level;
+                 bi->bit_size[area] += dv_rl2vlc_size(i - prev - 1, level);
+                 bi->next[prev]      = i;
+                 prev                = i;
+             }
+         }
      }
-     bi->next[prev]= i;
-     for (bi->cno = 0; max > classes[bi->cno]; bi->cno++);
+     bi->next[prev] = i;
+     for (bi->cno = 0; max > classes[bi->cno]; bi->cno++)
+         ;
  
      bi->cno += bias;
  
@@@ -335,65 -342,68 +349,68 @@@ static inline void dv_guess_qnos(EncBlo
  {
      int size[5];
      int i, j, k, a, prev, a2;
-     EncBlockInfob;
+     EncBlockInfo *b;
  
-     size[0] = size[1] = size[2] = size[3] = size[4] = 1 << 24;
+     size[0] =
+     size[1] =
+     size[2] =
+     size[3] =
+     size[4] = 1 << 24;
      do {
-        b = blks;
-        for (i = 0; i < 5; i++) {
-           if (!qnos[i])
-               continue;
-           qnos[i]--;
-           size[i] = 0;
-           for (j = 0; j < 6; j++, b++) {
-              for (a = 0; a < 4; a++) {
-                 if (b->area_q[a] != ff_dv_quant_shifts[qnos[i] + ff_dv_quant_offset[b->cno]][a]) {
-                     b->bit_size[a] = 1; // 4 areas 4 bits for EOB :)
-                     b->area_q[a]++;
-                     prev = b->prev[a];
-                     av_assert2(b->next[prev] >= mb_area_start[a+1] || b->mb[prev]);
-                     for (k = b->next[prev] ; k < mb_area_start[a+1]; k = b->next[k]) {
-                        b->mb[k] >>= 1;
-                        if (b->mb[k]) {
-                            b->bit_size[a] += dv_rl2vlc_size(k - prev - 1, b->mb[k]);
-                            prev = k;
-                        } else {
-                            if (b->next[k] >= mb_area_start[a+1] && b->next[k]<64){
-                                 for (a2 = a + 1; b->next[k] >= mb_area_start[a2+1]; a2++)
+         b = blks;
+         for (i = 0; i < 5; i++) {
+             if (!qnos[i])
+                 continue;
+             qnos[i]--;
+             size[i] = 0;
+             for (j = 0; j < 6; j++, b++) {
+                 for (a = 0; a < 4; a++) {
+                     if (b->area_q[a] != ff_dv_quant_shifts[qnos[i] + ff_dv_quant_offset[b->cno]][a]) {
+                         b->bit_size[a] = 1; // 4 areas 4 bits for EOB :)
+                         b->area_q[a]++;
+                         prev = b->prev[a];
 -                        assert(b->next[prev] >= mb_area_start[a + 1] || b->mb[prev]);
++                        av_assert2(b->next[prev] >= mb_area_start[a + 1] || b->mb[prev]);
+                         for (k = b->next[prev]; k < mb_area_start[a + 1]; k = b->next[k]) {
+                             b->mb[k] >>= 1;
+                             if (b->mb[k]) {
+                                 b->bit_size[a] += dv_rl2vlc_size(k - prev - 1, b->mb[k]);
+                                 prev            = k;
+                             } else {
+                                 if (b->next[k] >= mb_area_start[a + 1] && b->next[k] < 64) {
+                                     for (a2 = a + 1; b->next[k] >= mb_area_start[a2 + 1]; a2++)
+                                         b->prev[a2] = prev;
 -                                    assert(a2 < 4);
 -                                    assert(b->mb[b->next[k]]);
++                                    av_assert2(a2 < 4);
++                                    av_assert2(b->mb[b->next[k]]);
+                                     b->bit_size[a2] += dv_rl2vlc_size(b->next[k] - prev - 1, b->mb[b->next[k]]) -
+                                                        dv_rl2vlc_size(b->next[k] - k    - 1, b->mb[b->next[k]]);
 -                                    assert(b->prev[a2] == k && (a2 + 1 >= 4 || b->prev[a2 + 1] != k));
++                                    av_assert2(b->prev[a2] == k && (a2 + 1 >= 4 || b->prev[a2 + 1] != k));
                                      b->prev[a2] = prev;
-                                 av_assert2(a2 < 4);
-                                 av_assert2(b->mb[b->next[k]]);
-                                 b->bit_size[a2] += dv_rl2vlc_size(b->next[k] - prev - 1, b->mb[b->next[k]])
-                                                   -dv_rl2vlc_size(b->next[k] -    k - 1, b->mb[b->next[k]]);
-                                 av_assert2(b->prev[a2] == k && (a2 + 1 >= 4 || b->prev[a2+1] != k));
-                                 b->prev[a2] = prev;
-                            }
-                            b->next[prev] = b->next[k];
-                        }
+                                 }
+                                 b->next[prev] = b->next[k];
+                             }
+                         }
+                         b->prev[a + 1] = prev;
                      }
-                     b->prev[a+1]= prev;
+                     size[i] += b->bit_size[a];
                  }
-                 size[i] += b->bit_size[a];
-              }
-           }
-           if (vs_total_ac_bits >= size[0] + size[1] + size[2] + size[3] + size[4])
+             }
+             if (vs_total_ac_bits >= size[0] + size[1] + size[2] + size[3] + size[4])
                  return;
-        }
-     } while (qnos[0]|qnos[1]|qnos[2]|qnos[3]|qnos[4]);
+         }
+     } while (qnos[0] | qnos[1] | qnos[2] | qnos[3] | qnos[4]);
  
-     for (a = 2; a == 2 || vs_total_ac_bits < size[0]; a += a){
-         b = blks;
-         size[0] = 5 * 6 * 4; //EOB
-         for (j = 0; j < 6 *5; j++, b++) {
+     for (a = 2; a == 2 || vs_total_ac_bits < size[0]; a += a) {
+         b       = blks;
+         size[0] = 5 * 6 * 4; // EOB
+         for (j = 0; j < 6 * 5; j++, b++) {
              prev = b->prev[0];
              for (k = b->next[prev]; k < 64; k = b->next[k]) {
-                 if (b->mb[k] < a && b->mb[k] > -a){
+                 if (b->mb[k] < a && b->mb[k] > -a) {
                      b->next[prev] = b->next[k];
-                 }else{
+                 } else {
                      size[0] += dv_rl2vlc_size(k - prev - 1, b->mb[k]);
-                     prev = k;
+                     prev     = k;
                  }
              }
          }
@@@ -552,54 -570,57 +577,58 @@@ static inline int dv_write_pack(enum dv
       *   2. It is not at all clear what STYPE is used for 4:2:0 PAL
       *      compression scheme (if any).
       */
-     int apt   = (c->sys->pix_fmt == AV_PIX_FMT_YUV420P ? 0 : 1);
-     int fs    = c->frame->top_field_first ? 0x00 : 0x40;
+     int apt = (c->sys->pix_fmt == AV_PIX_FMT_YUV420P ? 0 : 1);
++    int fs  = c->frame->top_field_first ? 0x00 : 0x40;
  
      uint8_t aspect = 0;
-     if ((int)(av_q2d(c->avctx->sample_aspect_ratio) * c->avctx->width / c->avctx->height * 10) >= 17) /* 16:9 */
+     if ((int) (av_q2d(c->avctx->sample_aspect_ratio) *
+                c->avctx->width / c->avctx->height * 10) >= 17) /* 16:9 */
          aspect = 0x02;
  
-     buf[0] = (uint8_t)pack_id;
+     buf[0] = (uint8_t) pack_id;
      switch (pack_id) {
      case dv_header525: /* I can't imagine why these two weren't defined as real */
      case dv_header625: /* packs in SMPTE314M -- they definitely look like ones */
-           buf[1] = 0xf8 |        /* reserved -- always 1 */
-                    (apt & 0x07); /* APT: Track application ID */
-           buf[2] = (0    << 7) | /* TF1: audio data is 0 - valid; 1 - invalid */
-                    (0x0f << 3) | /* reserved -- always 1 */
-                    (apt & 0x07); /* AP1: Audio application ID */
-           buf[3] = (0    << 7) | /* TF2: video data is 0 - valid; 1 - invalid */
-                    (0x0f << 3) | /* reserved -- always 1 */
-                    (apt & 0x07); /* AP2: Video application ID */
-           buf[4] = (0    << 7) | /* TF3: subcode(SSYB) is 0 - valid; 1 - invalid */
-                    (0x0f << 3) | /* reserved -- always 1 */
-                    (apt & 0x07); /* AP3: Subcode application ID */
-           break;
+         buf[1] =  0xf8       | /* reserved -- always 1 */
+                  (apt & 0x07); /* APT: Track application ID */
+         buf[2] = (0    << 7) | /* TF1: audio data is 0 - valid; 1 - invalid */
+                  (0x0f << 3) | /* reserved -- always 1 */
+                  (apt & 0x07); /* AP1: Audio application ID */
+         buf[3] = (0    << 7) | /* TF2: video data is 0 - valid; 1 - invalid */
+                  (0x0f << 3) | /* reserved -- always 1 */
+                  (apt & 0x07); /* AP2: Video application ID */
+         buf[4] = (0    << 7) | /* TF3: subcode(SSYB) is 0 - valid; 1 - invalid */
+                  (0x0f << 3) | /* reserved -- always 1 */
+                  (apt & 0x07); /* AP3: Subcode application ID */
+         break;
      case dv_video_source:
-           buf[1] = 0xff;      /* reserved -- always 1 */
-           buf[2] = (1 << 7) | /* B/W: 0 - b/w, 1 - color */
-                    (1 << 6) | /* following CLF is valid - 0, invalid - 1 */
-                    (3 << 4) | /* CLF: color frames ID (see ITU-R BT.470-4) */
-                    0xf;       /* reserved -- always 1 */
-           buf[3] = (3 << 6) | /* reserved -- always 1 */
-                    (c->sys->dsf << 5) | /*  system: 60fields/50fields */
-                    c->sys->video_stype; /* signal type video compression */
-           buf[4] = 0xff;      /* VISC: 0xff -- no information */
-           break;
+         buf[1] = 0xff;         /* reserved -- always 1 */
+         buf[2] = (1 << 7) |    /* B/W: 0 - b/w, 1 - color */
+                  (1 << 6) |    /* following CLF is valid - 0, invalid - 1 */
+                  (3 << 4) |    /* CLF: color frames ID (see ITU-R BT.470-4) */
+                  0xf;          /* reserved -- always 1 */
+         buf[3] = (3 << 6)           | /* reserved -- always 1 */
+                  (c->sys->dsf << 5) | /*  system: 60fields/50fields */
+                  c->sys->video_stype; /* signal type video compression */
+         buf[4] = 0xff;         /* VISC: 0xff -- no information */
+         break;
      case dv_video_control:
-           buf[1] = (0 << 6) | /* Copy generation management (CGMS) 0 -- free */
-                    0x3f;      /* reserved -- always 1 */
-           buf[2] = 0xc8 |     /* reserved -- always b11001xxx */
-                    aspect;
-           buf[3] = (1 << 7) | /* frame/field flag 1 -- frame, 0 -- field */
-                    fs       | /* first/second field flag 0 -- field 2, 1 -- field 1 */
-                    (1 << 5) | /* frame change flag 0 -- same picture as before, 1 -- different */
-                    (1 << 4) | /* 1 - interlaced, 0 - noninterlaced */
-                    0xc;       /* reserved -- always b1100 */
-           buf[4] = 0xff;      /* reserved -- always 1 */
-           break;
+         buf[1] = (0 << 6) |    /* Copy generation management (CGMS) 0 -- free */
+                  0x3f;         /* reserved -- always 1 */
+         buf[2] = 0xc8 |        /* reserved -- always b11001xxx */
+                  aspect;
+         buf[3] = (1 << 7) |    /* frame/field flag 1 -- frame, 0 -- field */
 -                 (1 << 6) |    /* first/second field flag 0 -- field 2, 1 -- field 1 */
++                 fs       |    /* first/second field flag 0 -- field 2, 1 -- field 1 */
+                  (1 << 5) |    /* frame change flag 0 -- same picture as before, 1 -- different */
+                  (1 << 4) |    /* 1 - interlaced, 0 - noninterlaced */
+                  0xc;          /* reserved -- always b1100 */
+         buf[4] = 0xff;         /* reserved -- always 1 */
+         break;
      default:
-           buf[1] = buf[2] = buf[3] = buf[4] = 0xff;
+         buf[1] =
+         buf[2] =
+         buf[3] =
+         buf[4] = 0xff;
      }
      return 5;
  }