[FFmpeg-cvslog] dirac: K&R formatting cosmetics

Gabriel Dume git at videolan.org
Tue Sep 9 12:23:51 CEST 2014


ffmpeg | branch: master | Gabriel Dume <gabriel.ddx84 at gmail.com> | Mon Sep  8 18:32:34 2014 -0400| [9752d07d33d5370f7819865fbb5e582b982aad06] | committer: Diego Biurrun

dirac: K&R formatting cosmetics

Signed-off-by: Diego Biurrun <diego at biurrun.de>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=9752d07d33d5370f7819865fbb5e582b982aad06
---

 libavcodec/dirac.c        |  108 +++++++++++++++++++++++----------------------
 libavcodec/dirac_parser.c |   58 ++++++++++++------------
 2 files changed, 83 insertions(+), 83 deletions(-)

diff --git a/libavcodec/dirac.c b/libavcodec/dirac.c
index ed0ea9f..10c29d7 100644
--- a/libavcodec/dirac.c
+++ b/libavcodec/dirac.c
@@ -26,55 +26,56 @@
  */
 
 #include "libavutil/imgutils.h"
-#include "dirac.h"
+
 #include "avcodec.h"
+#include "dirac.h"
 #include "golomb.h"
 #include "internal.h"
 #include "mpeg12data.h"
 
 // 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 */
 static const AVRational dirac_preset_aspect_ratios[] = {
-    {1, 1},
-    {10, 11},
-    {12, 11},
-    {40, 33},
-    {16, 11},
-    {4, 3},
+    {  1,  1 },
+    { 10, 11 },
+    { 12, 11 },
+    { 40, 33 },
+    { 16, 11 },
+    {  4,  3 },
 };
 
 /* [DIRAC_STD] Values 9,10 of 10.3.5 Frame Rate.
  * Table 10.3 Available preset frame rate values
  */
 static const AVRational dirac_frame_rate[] = {
-    {15000, 1001},
-    {25, 2},
+    { 15000, 1001 },
+    {    25,    2 },
 };
 
 /* [DIRAC_STD] This should be equivalent to Table 10.5 Available signal
@@ -83,10 +84,10 @@ static const struct {
     uint8_t             bitdepth;
     enum AVColorRange   color_range;
 } pixel_range_presets[] = {
-    {8,  AVCOL_RANGE_JPEG},
-    {8,  AVCOL_RANGE_MPEG},
-    {10, AVCOL_RANGE_MPEG},
-    {12, AVCOL_RANGE_MPEG},
+    {  8, AVCOL_RANGE_JPEG },
+    {  8, AVCOL_RANGE_MPEG },
+    { 10, AVCOL_RANGE_MPEG },
+    { 12, AVCOL_RANGE_MPEG },
 };
 
 static const enum AVColorPrimaries dirac_primaries[] = {
@@ -118,7 +119,7 @@ static const enum AVPixelFormat dirac_pix_fmt[2][3] = {
 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;
 
@@ -168,16 +169,16 @@ static int parse_source_parameters(AVCodecContext *avctx, GetBitContext *gb,
         if (source->frame_rate_index <= 8)
             frame_rate = ff_mpeg12_frame_rate_tab[source->frame_rate_index];
         else
-             /* [DIRAC_STD] Table 10.3 values 9-10 */
-            frame_rate = dirac_frame_rate[source->frame_rate_index-9];
+            /* [DIRAC_STD] Table 10.3 values 9-10 */
+            frame_rate = dirac_frame_rate[source->frame_rate_index - 9];
     }
     av_reduce(&avctx->time_base.num, &avctx->time_base.den,
-              frame_rate.den, frame_rate.num, 1<<30);
+              frame_rate.den, frame_rate.num, 1 << 30);
 
     /* [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)
@@ -192,16 +193,16 @@ static int parse_source_parameters(AVCodecContext *avctx, GetBitContext *gb,
      *  aspect ratio values */
     if (source->aspect_ratio_index > 0)
         avctx->sample_aspect_ratio =
-                dirac_preset_aspect_ratios[source->aspect_ratio_index-1];
+            dirac_preset_aspect_ratios[source->aspect_ratio_index - 1];
 
     /* [DIRAC_STD] 10.3.7 Clean area. clean_area(video_params) */
     if (get_bits1(gb)) { /* [DIRAC_STD] custom_clean_area_flag */
         /* [DIRAC_STD] CLEAN_WIDTH */
-        source->clean_width        = svq3_get_ue_golomb(gb);
-         /* [DIRAC_STD] CLEAN_HEIGHT */
-        source->clean_height       = svq3_get_ue_golomb(gb);
-         /* [DIRAC_STD] CLEAN_LEFT_OFFSET */
-        source->clean_left_offset  = svq3_get_ue_golomb(gb);
+        source->clean_width = svq3_get_ue_golomb(gb);
+        /* [DIRAC_STD] CLEAN_HEIGHT */
+        source->clean_height = svq3_get_ue_golomb(gb);
+        /* [DIRAC_STD] CLEAN_LEFT_OFFSET */
+        source->clean_left_offset = svq3_get_ue_golomb(gb);
         /* [DIRAC_STD] CLEAN_RIGHT_OFFSET */
         source->clean_right_offset = svq3_get_ue_golomb(gb);
     }
@@ -219,16 +220,17 @@ static int parse_source_parameters(AVCodecContext *avctx, GetBitContext *gb,
         // 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
      * Available signal range presets <--> pixel_range_presets */
     if (source->pixel_range_index > 0) {
-        idx                = source->pixel_range_index-1;
+        idx                = source->pixel_range_index - 1;
         luma_depth         = pixel_range_presets[idx].bitdepth;
         avctx->color_range = pixel_range_presets[idx].color_range;
     }
@@ -240,7 +242,7 @@ static int parse_source_parameters(AVCodecContext *avctx, GetBitContext *gb,
 
     /* [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)
@@ -270,7 +272,7 @@ static int parse_source_parameters(AVCodecContext *avctx, GetBitContext *gb,
                 avctx->color_trc = AVCOL_TRC_BT709;
         }
     } else {
-        idx = source->color_spec_index;
+        idx                    = source->color_spec_index;
         avctx->color_primaries = dirac_color_presets[idx].color_primaries;
         avctx->colorspace      = dirac_color_presets[idx].colorspace;
         avctx->color_trc       = dirac_color_presets[idx].color_trc;
@@ -281,7 +283,7 @@ static int parse_source_parameters(AVCodecContext *avctx, GetBitContext *gb,
 
 /* [DIRAC_STD] 10. Sequence Header. sequence_header() */
 int avpriv_dirac_parse_sequence_header(AVCodecContext *avctx, GetBitContext *gb,
-                                   dirac_source_params *source)
+                                       dirac_source_params *source)
 {
     unsigned version_major;
     unsigned video_format, picture_coding_mode;
diff --git a/libavcodec/dirac_parser.c b/libavcodec/dirac_parser.c
index 46f5430..5c9d266 100644
--- a/libavcodec/dirac_parser.c
+++ b/libavcodec/dirac_parser.c
@@ -31,6 +31,7 @@
 
 #include "libavutil/intreadwrite.h"
 #include "libavutil/mem.h"
+
 #include "parser.h"
 
 #define DIRAC_PARSE_INFO_PREFIX 0x42424344
@@ -75,23 +76,22 @@ static int find_frame_end(DiracParseContext *pc,
         pc->sync_offset = 0;
         for (; i < buf_size; i++) {
             if (state == DIRAC_PARSE_INFO_PREFIX) {
-                if ((buf_size-i) >= pc->header_bytes_needed) {
+                if ((buf_size - i) >= pc->header_bytes_needed) {
                     pc->state = -1;
                     return i + pc->header_bytes_needed;
                 } else {
-                    pc->header_bytes_needed = 9-(buf_size-i);
+                    pc->header_bytes_needed = 9 - (buf_size - i);
                     break;
                 }
             } else
-              state = (state << 8) | buf[i];
+                state = (state << 8) | buf[i];
         }
     }
     pc->state = state;
     return -1;
 }
 
-typedef struct DiracParseUnit
-{
+typedef struct DiracParseUnit {
     int next_pu_offset;
     int prev_pu_offset;
     uint8_t pu_type;
@@ -102,12 +102,12 @@ static int unpack_parse_unit(DiracParseUnit *pu, DiracParseContext *pc,
 {
     uint8_t *start = pc->buffer + offset;
     uint8_t *end   = pc->buffer + pc->index;
-    if (start < pc->buffer || (start+13 > end))
+    if (start < pc->buffer || (start + 13 > end))
         return 0;
     pu->pu_type = start[4];
 
-    pu->next_pu_offset = AV_RB32(start+5);
-    pu->prev_pu_offset = AV_RB32(start+9);
+    pu->next_pu_offset = AV_RB32(start + 5);
+    pu->prev_pu_offset = AV_RB32(start + 9);
 
     if (pu->pu_type == 0x10 && pu->next_pu_offset == 0)
         pu->next_pu_offset = 13;
@@ -125,7 +125,7 @@ static int dirac_combine_frame(AVCodecParserContext *s, AVCodecContext *avctx,
     if (pc->overread_index) {
         memcpy(pc->buffer, pc->buffer + pc->overread_index,
                pc->index - pc->overread_index);
-        pc->index -= pc->overread_index;
+        pc->index         -= pc->overread_index;
         pc->overread_index = 0;
         if (*buf_size == 0 && pc->buffer[4] == 0x10) {
             *buf      = pc->buffer;
@@ -134,13 +134,13 @@ static int dirac_combine_frame(AVCodecParserContext *s, AVCodecContext *avctx,
         }
     }
 
-    if ( next == -1) {
+    if (next == -1) {
         /* Found a possible frame start but not a frame end */
-        void *new_buffer = av_fast_realloc(pc->buffer, &pc->buffer_size,
-                                           pc->index + (*buf_size -
-                                                        pc->sync_offset));
+        void *new_buffer =
+            av_fast_realloc(pc->buffer, &pc->buffer_size,
+                            pc->index + (*buf_size - pc->sync_offset));
         pc->buffer = new_buffer;
-        memcpy(pc->buffer+pc->index, (*buf + pc->sync_offset),
+        memcpy(pc->buffer + pc->index, (*buf + pc->sync_offset),
                *buf_size - pc->sync_offset);
         pc->index += *buf_size - pc->sync_offset;
         return -1;
@@ -162,8 +162,8 @@ static int dirac_combine_frame(AVCodecParserContext *s, AVCodecContext *avctx,
         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) {
-            pc->index -= 9;
-            *buf_size = next-9;
+            pc->index              -= 9;
+            *buf_size               = next - 9;
             pc->header_bytes_needed = 9;
             return -1;
         }
@@ -177,9 +177,9 @@ static int dirac_combine_frame(AVCodecParserContext *s, AVCodecContext *avctx,
 
         pc->dirac_unit_size += pu.next_pu_offset;
 
-        if ((pu.pu_type&0x08) != 0x08) {
+        if ((pu.pu_type & 0x08) != 0x08) {
             pc->header_bytes_needed = 9;
-            *buf_size = next;
+            *buf_size               = next;
             return -1;
         }
 
@@ -187,24 +187,24 @@ static int dirac_combine_frame(AVCodecParserContext *s, AVCodecContext *avctx,
         if (parse_timing_info) {
             uint8_t *cur_pu = pc->buffer +
                               pc->index - 13 - pu1.prev_pu_offset;
-            int pts =  AV_RB32(cur_pu + 13);
+            int pts = AV_RB32(cur_pu + 13);
             if (s->last_pts == 0 && s->last_dts == 0)
                 s->dts = pts - 1;
             else
-                s->dts = s->last_dts+1;
+                s->dts = s->last_dts + 1;
             s->pts = pts;
             if (!avctx->has_b_frames && (cur_pu[4] & 0x03))
                 avctx->has_b_frames = 1;
         }
         if (avctx->has_b_frames && s->pts == s->dts)
-             s->pict_type = AV_PICTURE_TYPE_B;
+            s->pict_type = AV_PICTURE_TYPE_B;
 
         /* Finally have a complete Dirac data unit */
         *buf      = pc->dirac_unit;
         *buf_size = pc->dirac_unit_size;
 
         pc->dirac_unit_size     = 0;
-        pc->overread_index      = pc->index-13;
+        pc->overread_index      = pc->index - 13;
         pc->header_bytes_needed = 9;
     }
     return next;
@@ -217,27 +217,25 @@ static int dirac_parse(AVCodecParserContext *s, AVCodecContext *avctx,
     DiracParseContext *pc = s->priv_data;
     int next;
 
-    *poutbuf = NULL;
+    *poutbuf      = NULL;
     *poutbuf_size = 0;
 
     if (s->flags & PARSER_FLAG_COMPLETE_FRAMES) {
-        next = buf_size;
-        *poutbuf = buf;
+        next          = buf_size;
+        *poutbuf      = buf;
         *poutbuf_size = buf_size;
         /* Assume that data has been packetized into an encapsulation unit. */
     } else {
         next = find_frame_end(pc, buf, buf_size);
-        if (!pc->is_synced && next == -1) {
+        if (!pc->is_synced && next == -1)
             /* No frame start found yet. So throw away the entire buffer. */
             return buf_size;
-        }
 
-        if (dirac_combine_frame(s, avctx, next, &buf, &buf_size) < 0) {
+        if (dirac_combine_frame(s, avctx, next, &buf, &buf_size) < 0)
             return buf_size;
-        }
     }
 
-    *poutbuf = buf;
+    *poutbuf      = buf;
     *poutbuf_size = buf_size;
     return next;
 }



More information about the ffmpeg-cvslog mailing list