# HG changeset patch # User banan # Date 1253324700 0 # Node ID de34f74fb75890395ec3c6fdf23ee9adae145b00 # Parent 404026d9adb50b743d7c28b908726d512a75cca4 Fix the short block transform for atrac1. diff -r 404026d9adb5 -r de34f74fb758 atrac1.c --- a/atrac1.c Fri Sep 18 19:45:09 2009 +0000 +++ b/atrac1.c Sat Sep 19 01:45:00 2009 +0000 @@ -73,7 +73,6 @@ typedef struct { AT1SUCtx SUs[AT1_MAX_CHANNELS]; ///< channel sound unit DECLARE_ALIGNED_16(float,spec[AT1_SU_SAMPLES]); ///< the mdct spectrum buffer - DECLARE_ALIGNED_16(float,short_buf[512]); ///< buffer for the short mode DECLARE_ALIGNED_16(float, low[256]); DECLARE_ALIGNED_16(float, mid[256]); @@ -133,6 +132,7 @@ return -1; if (num_blocks == 1) { + /* long blocks */ at1_imdct(q, &q->spec[pos], &su->spectrum[0][ref_pos], nbits, band_num); pos += block_size; // move to the next mdct block in the spectrum @@ -142,29 +142,21 @@ memcpy(q->bands[band_num]+32, &su->spectrum[0][ref_pos+16], 240 * sizeof(float)); } else { - /* calc start position for the 1st short block: 96(128) or 112(256) */ - int short_pos = 32; + /* short blocks */ float *prev_buf; - start_pos = (band_samples * (num_blocks - 1)) >> (log2_block_count + 1); - memset(&su->spectrum[0][ref_pos], 0, sizeof(float) * (band_samples * 2)); - + start_pos = 0; prev_buf = &su->spectrum[1][ref_pos+band_samples-16]; for (; num_blocks!=0 ; num_blocks--) { - /* use hardcoded nbits for the short mode */ - at1_imdct(q, &q->spec[pos], &q->short_buf[short_pos], 5, band_num); + at1_imdct(q, &q->spec[pos], &su->spectrum[0][ref_pos+start_pos], 5, band_num); /* overlap and window between short blocks */ - q->dsp.vector_fmul_window(&su->spectrum[0][ref_pos+start_pos], - &q->short_buf[short_pos-16], - &q->short_buf[short_pos],short_window, 0, 16); + q->dsp.vector_fmul_window(&q->bands[band_num][start_pos], prev_buf, + &su->spectrum[0][ref_pos+start_pos], short_window, 0, 16); - prev_buf = &q->short_buf[short_pos+16]; - + prev_buf = &su->spectrum[0][ref_pos+start_pos+16]; start_pos += 32; // use hardcoded block_size pos += 32; - short_pos +=32; } - memcpy(q->bands[band_num], &su->spectrum[0][ref_pos], band_samples*sizeof(float)); } ref_pos += band_samples; }