[SCM] libav/experimental: Remove st1 and st2 temporary buffers in backward_filter() and use instead RA288Context.pr{1, 2}. Note that the pr{1, 2} buffers are one unity smaller than the st{1, 2} buffers. My guess is that the original coder decided to add one to the array sizes "just to be sure".

[siretart at users.alioth.debian.org]

The following commit has been merged in the experimental branch:
commit 59da0390dc2bd2722665305d1d22c4ea10cfe049
Author: Vitor Sessak <vitor1001 at gmail.com>
Date:   Sun Jul 27 20:55:27 2008 +0000

    Remove st1 and st2 temporary buffers in backward_filter() and use instead
    RA288Context.pr{1,2}. Note that the pr{1,2} buffers are one unity smaller
    than the st{1,2} buffers. My guess is that the original coder decided to
    add one to the array sizes "just to be sure".
    
    Originally committed as revision 14435 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavcodec/ra288.c b/libavcodec/ra288.c
index 824fd55..f5018cf 100644
--- a/libavcodec/ra288.c
+++ b/libavcodec/ra288.c
@@ -197,20 +197,18 @@ static void backward_filter(RA288Context *ractx)
 {
     float temp1[37];
     float temp2[11];
-    float st1[37];
-    float st2[11];
 
     do_hybrid_window(36, 40, 35, ractx->output, temp1, ractx->sp_hist, ractx->sp_rec,
                      syn_window);
 
-    if (!eval_lpc_coeffs(temp1, st1, 36))
-        colmult(ractx->pr1, st1, syn_bw_tab, 36);
+    if (!eval_lpc_coeffs(temp1, ractx->pr1, 36))
+        colmult(ractx->pr1, ractx->pr1, syn_bw_tab, 36);
 
     do_hybrid_window(10, 8, 20, ractx->history, temp2, ractx->gain_hist, ractx->gain_rec,
                      gain_window);
 
-    if (!eval_lpc_coeffs(temp2, st2, 10))
-        colmult(ractx->pr2, st2, gain_bw_tab, 10);
+    if (!eval_lpc_coeffs(temp2, ractx->pr2, 10))
+        colmult(ractx->pr2, ractx->pr2, gain_bw_tab, 10);
 }
 
 /* Decode a block (celp) */

-- 
Libav/FFmpeg packaging