[FFmpeg-cvslog] adx: define and use 2 new macro constants BLOCK_SIZE and BLOCK_SAMPLES
Justin Ruggles
git at videolan.org
Sun Nov 27 00:39:15 CET 2011
ffmpeg | branch: master | Justin Ruggles <justin.ruggles at gmail.com> | Sun Nov 20 17:50:46 2011 -0500| [05c1f11b56a76d04bb77fb0a31e783ac1d435ad0] | committer: Justin Ruggles
adx: define and use 2 new macro constants BLOCK_SIZE and BLOCK_SAMPLES
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=05c1f11b56a76d04bb77fb0a31e783ac1d435ad0
---
libavcodec/adxdec.c | 31 +++++++++++++++++--------------
1 files changed, 17 insertions(+), 14 deletions(-)
diff --git a/libavcodec/adxdec.c b/libavcodec/adxdec.c
index ee1f1b0..5989233 100644
--- a/libavcodec/adxdec.c
+++ b/libavcodec/adxdec.c
@@ -33,6 +33,9 @@
* adx2wav & wav2adx http://www.geocities.co.jp/Playtown/2004/
*/
+#define BLOCK_SIZE 18
+#define BLOCK_SAMPLES 32
+
static av_cold int adx_decode_init(AVCodecContext *avctx)
{
avctx->sample_fmt = AV_SAMPLE_FMT_S16;
@@ -54,10 +57,10 @@ static void adx_decode(ADXContext *c, int16_t *out, const uint8_t *in, int ch)
int i;
int s0, s1, s2, d;
- init_get_bits(&gb, in + 2, (18 - 2) * 8);
+ init_get_bits(&gb, in + 2, (BLOCK_SIZE - 2) * 8);
s1 = prev->s1;
s2 = prev->s2;
- for (i = 0; i < 32; i++) {
+ for (i = 0; i < BLOCK_SAMPLES; i++) {
d = get_sbits(&gb, 4);
s0 = ((d << COEFF_BITS) * scale + c->coeff[0] * s1 + c->coeff[1] * s2) >> COEFF_BITS;
s2 = s1;
@@ -100,9 +103,9 @@ static int adx_decode_header(AVCodecContext *avctx, const uint8_t *buf,
return AVERROR_INVALIDDATA;
avctx->sample_rate = AV_RB32(buf + 8);
if (avctx->sample_rate < 1 ||
- avctx->sample_rate > INT_MAX / (avctx->channels * 18 * 8))
+ avctx->sample_rate > INT_MAX / (avctx->channels * BLOCK_SIZE * 8))
return AVERROR_INVALIDDATA;
- avctx->bit_rate = avctx->sample_rate * avctx->channels * 18 * 8 / 32;
+ avctx->bit_rate = avctx->sample_rate * avctx->channels * BLOCK_SIZE * 8 / BLOCK_SAMPLES;
cutoff = AV_RB16(buf + 16);
ff_adx_calculate_coeffs(cutoff, avctx->sample_rate, COEFF_BITS, c->coeff);
@@ -133,27 +136,27 @@ static int adx_decode_frame(AVCodecContext *avctx, void *data, int *data_size,
/* 18 bytes of data are expanded into 32*2 bytes of audio,
so guard against buffer overflows */
- if (rest / 18 > *data_size / 64)
- rest = (*data_size / 64) * 18;
+ if (rest / (BLOCK_SIZE * c->channels) > *data_size / (BLOCK_SAMPLES * c->channels))
+ rest = (*data_size / (BLOCK_SAMPLES * c->channels)) * BLOCK_SIZE;
if (c->in_temp) {
- int copysize = 18 * avctx->channels - c->in_temp;
+ int copysize = BLOCK_SIZE * avctx->channels - c->in_temp;
memcpy(c->dec_temp + c->in_temp, buf, copysize);
rest -= copysize;
buf += copysize;
adx_decode(c, samples, c->dec_temp, 0);
if (avctx->channels == 2)
- adx_decode(c, samples + 1, c->dec_temp + 18, 1);
- samples += 32 * c->channels;
+ adx_decode(c, samples + 1, c->dec_temp + BLOCK_SIZE, 1);
+ samples += BLOCK_SAMPLES * c->channels;
}
- while (rest >= 18 * c->channels) {
+ while (rest >= BLOCK_SIZE * c->channels) {
adx_decode(c, samples, buf, 0);
if (c->channels == 2)
- adx_decode(c, samples + 1, buf + 18, 1);
- rest -= 18 * c->channels;
- buf += 18 * c->channels;
- samples += 32 * c->channels;
+ adx_decode(c, samples + 1, buf + BLOCK_SIZE, 1);
+ rest -= BLOCK_SIZE * c->channels;
+ buf += BLOCK_SIZE * c->channels;
+ samples += BLOCK_SAMPLES * c->channels;
}
c->in_temp = rest;
More information about the ffmpeg-cvslog
mailing list