[FFmpeg-cvslog] r19862 - in trunk/libavcodec: dsputil.h mdct.c twinvq.c wma.c wmaprodec.c

[vitor]

Author: vitor
Date: Tue Sep 15 18:18:16 2009
New Revision: 19862

Log:
Add two more sizes to ff_sine_windows[] and also pad it with NULLs so 
that FF_ELEMS(ff_sine_windows[x]) == 1 << x.

Fix issue 1384.

Modified:
   trunk/libavcodec/dsputil.h
   trunk/libavcodec/mdct.c
   trunk/libavcodec/twinvq.c
   trunk/libavcodec/wma.c
   trunk/libavcodec/wmaprodec.c

Modified: trunk/libavcodec/dsputil.h
==============================================================================
--- trunk/libavcodec/dsputil.h	Tue Sep 15 17:28:11 2009	(r19861)
+++ trunk/libavcodec/dsputil.h	Tue Sep 15 18:18:16 2009	(r19862)
@@ -760,13 +760,15 @@ void ff_kbd_window_init(float *window, f
  * @param   n       size of half window
  */
 void ff_sine_window_init(float *window, int n);
+extern float ff_sine_32  [  32];
+extern float ff_sine_64  [  64];
 extern float ff_sine_128 [ 128];
 extern float ff_sine_256 [ 256];
 extern float ff_sine_512 [ 512];
 extern float ff_sine_1024[1024];
 extern float ff_sine_2048[2048];
 extern float ff_sine_4096[4096];
-extern float * const ff_sine_windows[6];
+extern float * const ff_sine_windows[13];
 
 int ff_mdct_init(MDCTContext *s, int nbits, int inverse, double scale);
 void ff_imdct_calc_c(MDCTContext *s, FFTSample *output, const FFTSample *input);

Modified: trunk/libavcodec/mdct.c
==============================================================================
--- trunk/libavcodec/mdct.c	Tue Sep 15 17:28:11 2009	(r19861)
+++ trunk/libavcodec/mdct.c	Tue Sep 15 18:18:16 2009	(r19862)
@@ -48,13 +48,17 @@ av_cold void ff_kbd_window_init(float *w
        window[i] = sqrt(local_window[i] / sum);
 }
 
+DECLARE_ALIGNED(16, float, ff_sine_32  [  32]);
+DECLARE_ALIGNED(16, float, ff_sine_64  [  64]);
 DECLARE_ALIGNED(16, float, ff_sine_128 [ 128]);
 DECLARE_ALIGNED(16, float, ff_sine_256 [ 256]);
 DECLARE_ALIGNED(16, float, ff_sine_512 [ 512]);
 DECLARE_ALIGNED(16, float, ff_sine_1024[1024]);
 DECLARE_ALIGNED(16, float, ff_sine_2048[2048]);
 DECLARE_ALIGNED(16, float, ff_sine_4096[4096]);
-float * const ff_sine_windows[6] = {
+float * const ff_sine_windows[] = {
+    NULL, NULL, NULL, NULL, NULL, // unused
+    ff_sine_32 , ff_sine_64 ,
     ff_sine_128, ff_sine_256, ff_sine_512, ff_sine_1024, ff_sine_2048, ff_sine_4096
 };
 

Modified: trunk/libavcodec/twinvq.c
==============================================================================
--- trunk/libavcodec/twinvq.c	Tue Sep 15 17:28:11 2009	(r19861)
+++ trunk/libavcodec/twinvq.c	Tue Sep 15 18:18:16 2009	(r19862)
@@ -648,7 +648,7 @@ static void imdct_and_window(TwinContext
         tctx->dsp.vector_fmul_window(out2,
                                      prev_buf + (bsize-wsize)/2,
                                      buf1 + bsize*j,
-                                     ff_sine_windows[av_log2(wsize) - 7],
+                                     ff_sine_windows[av_log2(wsize)],
                                      0.0,
                                      wsize/2);
         out2 += wsize;
@@ -903,9 +903,9 @@ static av_cold void init_mdct_win(TwinCo
     }
 
 
-    ff_sine_window_init(ff_sine_windows[av_log2(size_m)     - 7], size_m  );
-    ff_sine_window_init(ff_sine_windows[av_log2(size_s/2)   - 7], size_s/2);
-    ff_sine_window_init(ff_sine_windows[av_log2(mtab->size) - 7], mtab->size);
+    ff_sine_window_init(ff_sine_windows[av_log2(size_m)    ], size_m  );
+    ff_sine_window_init(ff_sine_windows[av_log2(size_s/2)  ], size_s/2);
+    ff_sine_window_init(ff_sine_windows[av_log2(mtab->size)], mtab->size);
 }
 
 /**

Modified: trunk/libavcodec/wma.c
==============================================================================
--- trunk/libavcodec/wma.c	Tue Sep 15 17:28:11 2009	(r19861)
+++ trunk/libavcodec/wma.c	Tue Sep 15 18:18:16 2009	(r19862)
@@ -341,8 +341,8 @@ int ff_wma_init(AVCodecContext *avctx, i
     for (i = 0; i < s->nb_block_sizes; i++) {
         int n;
         n = 1 << (s->frame_len_bits - i);
-        ff_sine_window_init(ff_sine_windows[s->frame_len_bits - i - 7], n);
-        s->windows[i] = ff_sine_windows[s->frame_len_bits - i - 7];
+        ff_sine_window_init(ff_sine_windows[s->frame_len_bits - i], n);
+        s->windows[i] = ff_sine_windows[s->frame_len_bits - i];
     }
 
     s->reset_block_lengths = 1;

Modified: trunk/libavcodec/wmaprodec.c
==============================================================================
--- trunk/libavcodec/wmaprodec.c	Tue Sep 15 17:28:11 2009	(r19861)
+++ trunk/libavcodec/wmaprodec.c	Tue Sep 15 18:18:16 2009	(r19862)
@@ -426,7 +426,7 @@ static av_cold int decode_init(AVCodecCo
     /** init MDCT windows: simple sinus window */
     for (i = 0; i < WMAPRO_BLOCK_SIZES; i++) {
         const int n       = 1 << (WMAPRO_BLOCK_MAX_BITS - i);
-        const int win_idx = WMAPRO_BLOCK_MAX_BITS - i - 7;
+        const int win_idx = WMAPRO_BLOCK_MAX_BITS - i;
         ff_sine_window_init(ff_sine_windows[win_idx], n);
         s->windows[WMAPRO_BLOCK_SIZES - i - 1] = ff_sine_windows[win_idx];
     }