[FFmpeg-cvslog] r25979 - trunk/libavcodec/ac3enc.c

jbr subversion
Tue Dec 14 15:52:21 CET 2010


Author: jbr
Date: Tue Dec 14 15:52:21 2010
New Revision: 25979

Log:
Remove some duplicate local copies of avctx fields.
This is an av_cold function, and we don't need to duplicate variables just to
save a few characters.

Modified:
   trunk/libavcodec/ac3enc.c

Modified: trunk/libavcodec/ac3enc.c
==============================================================================
--- trunk/libavcodec/ac3enc.c	Tue Dec 14 15:52:18 2010	(r25978)
+++ trunk/libavcodec/ac3enc.c	Tue Dec 14 15:52:21 2010	(r25979)
@@ -1252,8 +1252,6 @@ static av_cold int set_channel_info(AC3E
 
 static av_cold int validate_options(AVCodecContext *avctx, AC3EncodeContext *s)
 {
-    int freq = avctx->sample_rate;
-    int bitrate = avctx->bit_rate;
     int i, j;
 
     if (!avctx->channel_layout) {
@@ -1269,12 +1267,12 @@ static av_cold int validate_options(AVCo
     /* frequency */
     for (i = 0; i < 3; i++) {
         for (j = 0; j < 3; j++)
-            if ((ff_ac3_sample_rate_tab[j] >> i) == freq)
+            if ((ff_ac3_sample_rate_tab[j] >> i) == avctx->sample_rate)
                 goto found;
     }
     return -1;
  found:
-    s->sample_rate        = freq;
+    s->sample_rate        = avctx->sample_rate;
     s->bit_alloc.sr_shift = i;
     s->bit_alloc.sr_code  = j;
     s->bitstream_id       = 8 + s->bit_alloc.sr_shift;
@@ -1282,12 +1280,12 @@ static av_cold int validate_options(AVCo
 
     /* bitrate & frame size */
     for (i = 0; i < 19; i++) {
-        if ((ff_ac3_bitrate_tab[i] >> s->bit_alloc.sr_shift)*1000 == bitrate)
+        if ((ff_ac3_bitrate_tab[i] >> s->bit_alloc.sr_shift)*1000 == avctx->bit_rate)
             break;
     }
     if (i == 19)
         return -1;
-    s->bit_rate        = bitrate;
+    s->bit_rate        = avctx->bit_rate;
     s->frame_size_code = i << 1;
 
     return 0;



More information about the ffmpeg-cvslog mailing list