Merge commit '64bb3fa6b7f5b2db22b7577585d5861be24eebd8'
authorMichael Niedermayer <michaelni@gmx.at>
Mon, 31 Dec 2012 12:14:06 +0000 (13:14 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Mon, 31 Dec 2012 12:14:06 +0000 (13:14 +0100)
* commit '64bb3fa6b7f5b2db22b7577585d5861be24eebd8':
  cavs: cosmetics, reformat top part
  cavsdec: cosmetics, reformat
  libavcodec/utils: Add braces to shut up gcc warnings

Conflicts:
libavcodec/cavsdec.c
libavcodec/utils.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavcodec/cavs.c
libavcodec/cavsdec.c
libavcodec/utils.c

Simple merge
@@@ -544,34 -546,34 +547,35 @@@ static inline int dequant(AVSContext *h
   */
  static int decode_residual_block(AVSContext *h, GetBitContext *gb,
                                   const struct dec_2dvlc *r, int esc_golomb_order,
-                                  int qp, uint8_t *dst, int stride) {
+                                  int qp, uint8_t *dst, int stride)
+ {
 -    int i, level_code, esc_code, level, run, mask;
 +    int i, esc_code, level, mask;
 +    unsigned int level_code, run;
      DCTELEM level_buf[65];
      uint8_t run_buf[65];
      DCTELEM *block = h->block;
  
-     for(i=0;i<65;i++) {
-         level_code = get_ue_code(gb,r->golomb_order);
-         if(level_code >= ESCAPE_CODE) {
-             run = ((level_code - ESCAPE_CODE) >> 1) + 1;
 -    for (i = 0;i < 65; i++) {
++    for (i = 0; i < 65; i++) {
+         level_code = get_ue_code(gb, r->golomb_order);
+         if (level_code >= ESCAPE_CODE) {
+             run      = ((level_code - ESCAPE_CODE) >> 1) + 1;
 +            if(run > 64)
 +                return -1;
-             esc_code = get_ue_code(gb,esc_golomb_order);
-             level = esc_code + (run > r->max_run ? 1 : r->level_add[run]);
-             while(level > r->inc_limit)
+             esc_code = get_ue_code(gb, esc_golomb_order);
+             level    = esc_code + (run > r->max_run ? 1 : r->level_add[run]);
+             while (level > r->inc_limit)
                  r++;
-             mask = -(level_code & 1);
-             level = (level^mask) - mask;
+             mask  = -(level_code & 1);
+             level = (level ^ mask) - mask;
 -        } else if (level_code >= 0) {
 +        } else {
              level = r->rltab[level_code][0];
-             if(!level) //end of block signal
+             if (!level) //end of block signal
                  break;
-             run   = r->rltab[level_code][1];
-             r += r->rltab[level_code][2];
+             run = r->rltab[level_code][1];
+             r  += r->rltab[level_code][2];
 -        } else {
 -            break;
          }
          level_buf[i] = level;
-         run_buf[i] = run;
+         run_buf[i]   = run;
      }
      if (dequant(h, level_buf, run_buf, block, dequant_mul[qp],
                  dequant_shift[qp], i))
@@@ -595,8 -599,8 +601,8 @@@ static inline int decode_residual_inter
      int block;
  
      /* get coded block pattern */
-     int cbp= get_ue_golomb(&h->s.gb);
-     if(cbp > 63U){
+     int cbp = get_ue_golomb(&h->s.gb);
 -    if (cbp > 63) {
++    if (cbp > 63U) {
          av_log(h->s.avctx, AV_LOG_ERROR, "illegal inter cbp\n");
          return -1;
      }
@@@ -663,9 -669,9 +671,9 @@@ static int decode_mb_i(AVSContext *h, i
      ff_cavs_modify_mb_i(h, &pred_mode_uv);
  
      /* get coded block pattern */
-     if(h->pic_type == AV_PICTURE_TYPE_I)
+     if (h->pic_type == AV_PICTURE_TYPE_I)
          cbp_code = get_ue_golomb(gb);
-     if(cbp_code > 63U){
 -    if (cbp_code > 63) {
++    if (cbp_code > 63U) {
          av_log(h->s.avctx, AV_LOG_ERROR, "illegal intra cbp\n");
          return -1;
      }
@@@ -750,7 -758,8 +760,8 @@@ static void decode_mb_p(AVSContext *h, 
      h->col_type_base[h->mbidx] = mb_type;
  }
  
- static int decode_mb_b(AVSContext *h, enum cavs_mb mb_type) {
 -static void decode_mb_b(AVSContext *h, enum cavs_mb mb_type)
++static int decode_mb_b(AVSContext *h, enum cavs_mb mb_type)
+ {
      int block;
      enum cavs_sub_mb sub_type[4];
      int flags;
          }
          break;
      default:
 -        assert((mb_type > B_SYM_16X16) && (mb_type < B_8X8));
 +        if (mb_type <= B_SYM_16X16) {
 +            av_log(h->s.avctx, AV_LOG_ERROR, "Invalid mb_type %d in B frame\n", mb_type);
 +            return AVERROR_INVALIDDATA;
 +        }
 +        av_assert2(mb_type < B_8X8);
          flags = ff_cavs_partition_flags[mb_type];
-         if(mb_type & 1) { /* 16x8 macroblock types */
-             if(flags & FWD0)
+         if (mb_type & 1) { /* 16x8 macroblock types */
+             if (flags & FWD0)
                  ff_cavs_mv(h, MV_FWD_X0, MV_FWD_C2, MV_PRED_TOP,  BLK_16X8, 1);
-             if(flags & SYM0)
+             if (flags & SYM0)
                  mv_pred_sym(h, &h->mv[MV_FWD_X0], BLK_16X8);
-             if(flags & FWD1)
+             if (flags & FWD1)
                  ff_cavs_mv(h, MV_FWD_X2, MV_FWD_A1, MV_PRED_LEFT, BLK_16X8, 1);
-             if(flags & SYM1)
+             if (flags & SYM1)
                  mv_pred_sym(h, &h->mv[MV_FWD_X2], BLK_16X8);
-             if(flags & BWD0)
+             if (flags & BWD0)
                  ff_cavs_mv(h, MV_BWD_X0, MV_BWD_C2, MV_PRED_TOP,  BLK_16X8, 0);
-             if(flags & BWD1)
+             if (flags & BWD1)
                  ff_cavs_mv(h, MV_BWD_X2, MV_BWD_A1, MV_PRED_LEFT, BLK_16X8, 0);
          } else {          /* 8x16 macroblock types */
-             if(flags & FWD0)
+             if (flags & FWD0)
                  ff_cavs_mv(h, MV_FWD_X0, MV_FWD_B3, MV_PRED_LEFT, BLK_8X16, 1);
-             if(flags & SYM0)
+             if (flags & SYM0)
                  mv_pred_sym(h, &h->mv[MV_FWD_X0], BLK_8X16);
-             if(flags & FWD1)
-                 ff_cavs_mv(h,MV_FWD_X1,MV_FWD_C2,MV_PRED_TOPRIGHT,BLK_8X16,1);
-             if(flags & SYM1)
+             if (flags & FWD1)
+                 ff_cavs_mv(h, MV_FWD_X1, MV_FWD_C2, MV_PRED_TOPRIGHT, BLK_8X16, 1);
+             if (flags & SYM1)
                  mv_pred_sym(h, &h->mv[MV_FWD_X1], BLK_8X16);
-             if(flags & BWD0)
+             if (flags & BWD0)
                  ff_cavs_mv(h, MV_BWD_X0, MV_BWD_B3, MV_PRED_LEFT, BLK_8X16, 0);
-             if(flags & BWD1)
-                 ff_cavs_mv(h,MV_BWD_X1,MV_BWD_C2,MV_PRED_TOPRIGHT,BLK_8X16,0);
+             if (flags & BWD1)
+                 ff_cavs_mv(h, MV_BWD_X1, MV_BWD_C2, MV_PRED_TOPRIGHT, BLK_8X16, 0);
          }
      }
      ff_cavs_inter(h, mb_type);
      set_intra_mode_default(h);
-     if(mb_type != B_SKIP)
+     if (mb_type != B_SKIP)
          decode_residual_inter(h);
-     ff_cavs_filter(h,mb_type);
+     ff_cavs_filter(h, mb_type);
 +
 +    return 0;
  }
  
  /*****************************************************************************
   *
   ****************************************************************************/
  
- static inline int decode_slice_header(AVSContext *h, GetBitContext *gb) {
-     if(h->stc > 0xAF)
+ static inline int decode_slice_header(AVSContext *h, GetBitContext *gb)
+ {
+     if (h->stc > 0xAF)
          av_log(h->s.avctx, AV_LOG_ERROR, "unexpected start code 0x%02x\n", h->stc);
-     h->mby = h->stc;
-     h->mbidx = h->mby*h->mb_width;
 +
 +    if (h->stc >= h->mb_height)
 +        return -1;
 +
+     h->mby   = h->stc;
+     h->mbidx = h->mby * h->mb_width;
  
      /* mark top macroblocks as unavailable */
-     h->flags &= ~(B_AVAIL|C_AVAIL);
-     if((h->mby == 0) && (!h->qp_fixed)){
+     h->flags &= ~(B_AVAIL | C_AVAIL);
+     if ((h->mby == 0) && (!h->qp_fixed)) {
          h->qp_fixed = get_bits1(gb);
-         h->qp = get_bits(gb,6);
+         h->qp       = get_bits(gb, 6);
      }
      /* inter frame or second slice can have weighting params */
-     if((h->pic_type != AV_PICTURE_TYPE_I) || (!h->pic_structure && h->mby >= h->mb_width/2))
-         if(get_bits1(gb)) { //slice_weighting_flag
+     if ((h->pic_type != AV_PICTURE_TYPE_I) ||
+         (!h->pic_structure && h->mby >= h->mb_width / 2))
+         if (get_bits1(gb)) { //slice_weighting_flag
              av_log(h->s.avctx, AV_LOG_ERROR,
                     "weighted prediction not yet supported\n");
          }
@@@ -922,10 -924,10 +936,11 @@@ static inline int check_for_slice(AVSCo
   *
   ****************************************************************************/
  
- static int decode_pic(AVSContext *h) {
+ static int decode_pic(AVSContext *h)
+ {
      MpegEncContext *s = &h->s;
-     int skip_count = -1;
+     int skip_count    = -1;
 +    int ret;
      enum cavs_mb mb_type;
  
      if (!s->context_initialized) {
              return -1;
      } else {
          h->pic_type = AV_PICTURE_TYPE_I;
-         if(get_bits1(&s->gb))
-             skip_bits(&s->gb,24);//time_code
+         if (get_bits1(&s->gb))
+             skip_bits(&s->gb, 24);//time_code
          /* old sample clips were all progressive and no low_delay,
             bump stream revision if detected otherwise */
-         if (s->low_delay || !(show_bits(&s->gb,9) & 1))
+         if (s->low_delay || !(show_bits(&s->gb, 9) & 1))
              h->stream_revision = 1;
          /* similarly test top_field_first and repeat_first_field */
-         else if(show_bits(&s->gb,11) & 3)
+         else if (show_bits(&s->gb, 11) & 3)
              h->stream_revision = 1;
-         if(h->stream_revision > 0)
-             skip_bits(&s->gb,1); //marker_bit
+         if (h->stream_revision > 0)
+             skip_bits(&s->gb, 1); //marker_bit
      }
      /* release last B frame */
-     if(h->picture.f.data[0])
+     if (h->picture.f.data[0])
          s->avctx->release_buffer(s->avctx, &h->picture.f);
  
 -    ff_get_buffer(s->avctx, &h->picture.f);
 -    ff_cavs_init_pic(h);
 +    if ((ret = ff_get_buffer(s->avctx, &h->picture.f)) < 0)
 +        return ret;
 +    if ((ret = ff_cavs_init_pic(h)) < 0)
 +        return ret;
-     h->picture.poc = get_bits(&s->gb,8)*2;
+     h->picture.poc = get_bits(&s->gb, 8) * 2;
  
      /* get temporal distances and MV scaling factors */
-     if(h->pic_type != AV_PICTURE_TYPE_B) {
+     if (h->pic_type != AV_PICTURE_TYPE_B) {
          h->dist[0] = (h->picture.poc - h->DPB[0].poc  + 512) % 512;
      } else {
          h->dist[0] = (h->DPB[0].poc  - h->picture.poc + 512) % 512;
@@@ -1160,17 -1159,15 +1178,17 @@@ static int cavs_decode_frame(AVCodecCon
              }
          case PIC_PB_START_CODE:
              *got_frame = 0;
-             if(!h->got_keyframe)
+             if (!h->got_keyframe)
                  break;
 +            if(!h->top_qp)
 +                break;
              init_get_bits(&s->gb, buf_ptr, input_size);
              h->stc = stc;
-             if(decode_pic(h))
+             if (decode_pic(h))
                  break;
              *got_frame = 1;
-             if(h->pic_type != AV_PICTURE_TYPE_B) {
-                 if(h->DPB[1].f.data[0]) {
+             if (h->pic_type != AV_PICTURE_TYPE_B) {
+                 if (h->DPB[1].f.data[0]) {
                      *picture = h->DPB[1].f;
                  } else {
                      *got_frame = 0;
Simple merge