[SCM] libav/experimental: Change output_frame_end() to return void. The frame size is already known.

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 17:20:14 UTC 2013


The following commit has been merged in the experimental branch:
commit 1971ab6ed64b6626903d830628d142b6e2e063a7
Author: Justin Ruggles <justin.ruggles at gmail.com>
Date:   Tue Dec 14 14:52:08 2010 +0000

    Change output_frame_end() to return void. The frame size is already known.
    
    Originally committed as revision 25975 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavcodec/ac3enc.c b/libavcodec/ac3enc.c
index a7211c1..71b62c5 100644
--- a/libavcodec/ac3enc.c
+++ b/libavcodec/ac3enc.c
@@ -1038,7 +1038,7 @@ static unsigned int pow_poly(unsigned int a, unsigned int n, unsigned int poly)
 /**
  * Fill the end of the frame with 0's and compute the two CRCs.
  */
-static int output_frame_end(AC3EncodeContext *s)
+static void output_frame_end(AC3EncodeContext *s)
 {
     int frame_size, frame_size_58, n, crc1, crc2, crc_inv;
     uint8_t *frame;
@@ -1069,8 +1069,6 @@ static int output_frame_end(AC3EncodeContext *s)
                              frame + 2 * frame_size_58,
                              (frame_size - frame_size_58) * 2 - 2));
     AV_WB16(frame + 2*frame_size - 2, crc2);
-
-    return frame_size * 2;
 }
 
 
@@ -1188,7 +1186,9 @@ static int ac3_encode_frame(AVCodecContext *avctx,
         output_audio_block(s, exp_strategy[blk], encoded_exp[blk],
                            bap[blk], mdct_coef[blk], exp_shift[blk], blk);
     }
-    return output_frame_end(s);
+    output_frame_end(s);
+
+    return s->frame_size * 2;
 }
 
 

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list