[FFmpeg-cvslog] g722dec: remove the use of lowres for half-rate decoding.

[Justin Ruggles]

ffmpeg | branch: master | Justin Ruggles <justin.ruggles at gmail.com> | Sat Oct 22 18:29:14 2011 -0400| [d0a196962a82f26a06357efd843488ae62fabea7] | committer: Justin Ruggles

g722dec: remove the use of lowres for half-rate decoding.

It is broken because an AVCodecContext can be opened/closed multiple
times, and sample_rate is getting divided by 2 each time that happens.

This removes the only use of lowres for audio.

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

 avconv.c             |    5 +----
 libavcodec/g722dec.c |   18 +++++-------------
 2 files changed, 6 insertions(+), 17 deletions(-)

diff --git a/avconv.c b/avconv.c
index 96f6059..f93c9fa 100644
--- a/avconv.c
+++ b/avconv.c
@@ -2074,11 +2074,8 @@ static int transcode_init(OutputFile *output_files,
                     return AVERROR(ENOMEM);
                 }
                 ost->reformat_pair = MAKE_SFMT_PAIR(AV_SAMPLE_FMT_NONE,AV_SAMPLE_FMT_NONE);
-                if (!codec->sample_rate) {
+                if (!codec->sample_rate)
                     codec->sample_rate = icodec->sample_rate;
-                    if (icodec->lowres)
-                        codec->sample_rate >>= icodec->lowres;
-                }
                 choose_sample_rate(ost->st, ost->enc);
                 codec->time_base = (AVRational){1, codec->sample_rate};
                 if (codec->sample_fmt == AV_SAMPLE_FMT_NONE)
diff --git a/libavcodec/g722dec.c b/libavcodec/g722dec.c
index 51f87c9..d2d2a48 100644
--- a/libavcodec/g722dec.c
+++ b/libavcodec/g722dec.c
@@ -66,9 +66,6 @@ static av_cold int g722_decode_init(AVCodecContext * avctx)
     c->band[1].scale_factor = 2;
     c->prev_samples_pos = 22;
 
-    if (avctx->lowres)
-        avctx->sample_rate /= 2;
-
     return 0;
 }
 
@@ -96,7 +93,8 @@ static int g722_decode_frame(AVCodecContext *avctx, void *data,
     init_get_bits(&gb, avpkt->data, avpkt->size * 8);
 
     for (j = 0; j < avpkt->size; j++) {
-        int ilow, ihigh, rlow;
+        int ilow, ihigh, rlow, rhigh, dhigh;
+        int xout1, xout2;
 
         ihigh = get_bits(&gb, 2);
         ilow = get_bits(&gb, 6 - skip);
@@ -107,12 +105,9 @@ static int g722_decode_frame(AVCodecContext *avctx, void *data,
 
         ff_g722_update_low_predictor(&c->band[0], ilow >> (2 - skip));
 
-        if (!avctx->lowres) {
-            const int dhigh = c->band[1].scale_factor *
-                              ff_g722_high_inv_quant[ihigh] >> 10;
-            const int rhigh = av_clip(dhigh + c->band[1].s_predictor,
-                                      -16384, 16383);
-            int xout1, xout2;
+            dhigh = c->band[1].scale_factor *
+                    ff_g722_high_inv_quant[ihigh] >> 10;
+            rhigh = av_clip(dhigh + c->band[1].s_predictor, -16384, 16383);
 
             ff_g722_update_high_predictor(&c->band[1], dhigh, ihigh);
 
@@ -128,8 +123,6 @@ static int g722_decode_frame(AVCodecContext *avctx, void *data,
                         22 * sizeof(c->prev_samples[0]));
                 c->prev_samples_pos = 22;
             }
-        } else
-            out_buf[out_len++] = rlow;
     }
     *data_size = out_len << 1;
     return avpkt->size;
@@ -143,5 +136,4 @@ AVCodec ff_adpcm_g722_decoder = {
     .init           = g722_decode_init,
     .decode         = g722_decode_frame,
     .long_name      = NULL_IF_CONFIG_SMALL("G.722 ADPCM"),
-    .max_lowres     = 1,
 };