Merge commit '9752d07d33d5370f7819865fbb5e582b982aad06'
authorMichael Niedermayer <michaelni@gmx.at>
Tue, 9 Sep 2014 10:12:29 +0000 (12:12 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Tue, 9 Sep 2014 10:12:29 +0000 (12:12 +0200)
* commit '9752d07d33d5370f7819865fbb5e582b982aad06':
  dirac: K&R formatting cosmetics

Conflicts:
libavcodec/dirac.c
libavcodec/dirac_parser.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavcodec/dirac.c
libavcodec/dirac_parser.c

  #include "internal.h"
  #include "mpeg12data.h"
  
 -// defaults for source parameters
 +/* defaults for source parameters */
  static const dirac_source_params dirac_source_parameters_defaults[] = {
-     { 640,  480,  2, 0, 0, 1,  1, 640,  480,  0, 0, 1, 0 },
-     { 176,  120,  2, 0, 0, 9,  2, 176,  120,  0, 0, 1, 1 },
-     { 176,  144,  2, 0, 1, 10, 3, 176,  144,  0, 0, 1, 2 },
-     { 352,  240,  2, 0, 0, 9,  2, 352,  240,  0, 0, 1, 1 },
-     { 352,  288,  2, 0, 1, 10, 3, 352,  288,  0, 0, 1, 2 },
-     { 704,  480,  2, 0, 0, 9,  2, 704,  480,  0, 0, 1, 1 },
-     { 704,  576,  2, 0, 1, 10, 3, 704,  576,  0, 0, 1, 2 },
-     { 720,  480,  1, 1, 0, 4,  2, 704,  480,  8, 0, 3, 1 },
-     { 720,  576,  1, 1, 1, 3,  3, 704,  576,  8, 0, 3, 2 },
-     { 1280, 720,  1, 0, 1, 7,  1, 1280, 720,  0, 0, 3, 3 },
-     { 1280, 720,  1, 0, 1, 6,  1, 1280, 720,  0, 0, 3, 3 },
-     { 1920, 1080, 1, 1, 1, 4,  1, 1920, 1080, 0, 0, 3, 3 },
-     { 1920, 1080, 1, 1, 1, 3,  1, 1920, 1080, 0, 0, 3, 3 },
-     { 1920, 1080, 1, 0, 1, 7,  1, 1920, 1080, 0, 0, 3, 3 },
-     { 1920, 1080, 1, 0, 1, 6,  1, 1920, 1080, 0, 0, 3, 3 },
-     { 2048, 1080, 0, 0, 1, 2,  1, 2048, 1080, 0, 0, 4, 4 },
-     { 4096, 2160, 0, 0, 1, 2,  1, 4096, 2160, 0, 0, 4, 4 },
-     { 3840, 2160, 1, 0, 1, 7,  1, 3840, 2160, 0, 0, 3, 3 },
-     { 3840, 2160, 1, 0, 1, 6,  1, 3840, 2160, 0, 0, 3, 3 },
-     { 7680, 4320, 1, 0, 1, 7,  1, 3840, 2160, 0, 0, 3, 3 },
-     { 7680, 4320, 1, 0, 1, 6,  1, 3840, 2160, 0, 0, 3, 3 },
+     {  640,  480, 2, 0, 0,  1, 1,  640,  480, 0, 0, 1, 0 },
+     {  176,  120, 2, 0, 0,  9, 2,  176,  120, 0, 0, 1, 1 },
+     {  176,  144, 2, 0, 1, 10, 3,  176,  144, 0, 0, 1, 2 },
+     {  352,  240, 2, 0, 0,  9, 2,  352,  240, 0, 0, 1, 1 },
+     {  352,  288, 2, 0, 1, 10, 3,  352,  288, 0, 0, 1, 2 },
+     {  704,  480, 2, 0, 0,  9, 2,  704,  480, 0, 0, 1, 1 },
+     {  704,  576, 2, 0, 1, 10, 3,  704,  576, 0, 0, 1, 2 },
+     {  720,  480, 1, 1, 0,  4, 2,  704,  480, 8, 0, 3, 1 },
+     {  720,  576, 1, 1, 1,  3, 3,  704,  576, 8, 0, 3, 2 },
+     { 1280,  720, 1, 0, 1,  7, 1, 1280,  720, 0, 0, 3, 3 },
+     { 1280,  720, 1, 0, 1,  6, 1, 1280,  720, 0, 0, 3, 3 },
+     { 1920, 1080, 1, 1, 1,  4, 1, 1920, 1080, 0, 0, 3, 3 },
+     { 1920, 1080, 1, 1, 1,  3, 1, 1920, 1080, 0, 0, 3, 3 },
+     { 1920, 1080, 1, 0, 1,  7, 1, 1920, 1080, 0, 0, 3, 3 },
+     { 1920, 1080, 1, 0, 1,  6, 1, 1920, 1080, 0, 0, 3, 3 },
+     { 2048, 1080, 0, 0, 1,  2, 1, 2048, 1080, 0, 0, 4, 4 },
+     { 4096, 2160, 0, 0, 1,  2, 1, 4096, 2160, 0, 0, 4, 4 },
+     { 3840, 2160, 1, 0, 1,  7, 1, 3840, 2160, 0, 0, 3, 3 },
+     { 3840, 2160, 1, 0, 1,  6, 1, 3840, 2160, 0, 0, 3, 3 },
+     { 7680, 4320, 1, 0, 1,  7, 1, 3840, 2160, 0, 0, 3, 3 },
+     { 7680, 4320, 1, 0, 1,  6, 1, 3840, 2160, 0, 0, 3, 3 },
  };
  
  /* [DIRAC_STD] Table 10.4 - Available preset pixel aspect ratio values */
@@@ -117,10 -119,9 +120,10 @@@ static const enum AVPixelFormat dirac_p
  static int parse_source_parameters(AVCodecContext *avctx, GetBitContext *gb,
                                     dirac_source_params *source)
  {
-     AVRational frame_rate = {0,0};
+     AVRational frame_rate = { 0, 0 };
      unsigned luma_depth = 8, luma_offset = 16;
      int idx;
 +    int chroma_x_shift, chroma_y_shift;
  
      /* [DIRAC_STD] 10.3.2 Frame size. frame_size(video_params) */
      /* [DIRAC_STD] custom_dimensions_flag */
      /* [DIRAC_STD] 10.3.6 Pixel Aspect Ratio.
       * pixel_aspect_ratio(video_params) */
      if (get_bits1(gb)) { /* [DIRAC_STD] custom_pixel_aspect_ratio_flag */
-          /* [DIRAC_STD] index */
+         /* [DIRAC_STD] index */
          source->aspect_ratio_index = svq3_get_ue_golomb(gb);
  
 -        if (source->aspect_ratio_index > 6)
 +        if (source->aspect_ratio_index > 6U)
              return AVERROR_INVALIDDATA;
  
          if (!source->aspect_ratio_index) {
          /* [DIRAC_STD] index */
          source->pixel_range_index = svq3_get_ue_golomb(gb);
  
 -        if (source->pixel_range_index > 4)
 +        if (source->pixel_range_index > 4U)
              return AVERROR_INVALIDDATA;
  
 -        // This assumes either fullrange or MPEG levels only
 +        /* This assumes either fullrange or MPEG levels only */
          if (!source->pixel_range_index) {
              luma_offset = svq3_get_ue_golomb(gb);
-             luma_depth  = av_log2(svq3_get_ue_golomb(gb))+1;
+             luma_depth  = av_log2(svq3_get_ue_golomb(gb)) + 1;
              svq3_get_ue_golomb(gb); /* chroma offset    */
              svq3_get_ue_golomb(gb); /* chroma excursion */
-             avctx->color_range = luma_offset ? AVCOL_RANGE_MPEG : AVCOL_RANGE_JPEG;
+             avctx->color_range = luma_offset ? AVCOL_RANGE_MPEG
+                                              : AVCOL_RANGE_JPEG;
          }
      }
      /* [DIRAC_STD] Table 10.5
  
      /* [DIRAC_STD] 10.3.9 Colour specification. colour_spec(video_params) */
      if (get_bits1(gb)) { /* [DIRAC_STD] custom_colour_spec_flag */
-          /* [DIRAC_STD] index */
+         /* [DIRAC_STD] index */
          idx = source->color_spec_index = svq3_get_ue_golomb(gb);
  
 -        if (source->color_spec_index > 4)
 +        if (source->color_spec_index > 4U)
              return AVERROR_INVALIDDATA;
  
          avctx->color_primaries = dirac_color_presets[idx].color_primaries;
@@@ -161,11 -161,9 +161,11 @@@ static int dirac_combine_frame(AVCodecP
           * we can be pretty sure that we have a valid parse unit */
          if (!unpack_parse_unit(&pu1, pc, pc->index - 13)                     ||
              !unpack_parse_unit(&pu, pc, pc->index - 13 - pu1.prev_pu_offset) ||
 -            pu.next_pu_offset != pu1.prev_pu_offset) {
 +            pu.next_pu_offset != pu1.prev_pu_offset                          ||
 +            pc->index < pc->dirac_unit_size + 13LL + pu1.prev_pu_offset
 +        ) {
-             pc->index -= 9;
-             *buf_size = next-9;
+             pc->index              -= 9;
+             *buf_size               = next - 9;
              pc->header_bytes_needed = 9;
              return -1;
          }