# HG changeset patch
# User alexc
# Date 1273177116 0
# Node ID 41a083a2527ac8837d1d5d154e5c967b646258ba
# Parent  d3098fb8ec051902cf70c7c366eae4254b7e49cd
Make the faac inspired quantizer search make sense for a slightly narrower definition of "make sense."

diff -r d3098fb8ec05 -r 41a083a2527a aaccoder.c
--- a/aaccoder.c	Wed May 05 21:38:02 2010 +0000
+++ b/aaccoder.c	Thu May 06 20:18:36 2010 +0000
@@ -763,7 +763,7 @@
             const float *scaled = s->scoefs   + start;
             const int size      = sce->ics.swb_sizes[g];
             int scf, prev_scf, step;
-            int min_scf = 0, max_scf = 255;
+            int min_scf = -1, max_scf = 256;
             float curdiff;
             if (maxq[w*16+g] < 21.544) {
                 sce->zeroes[w*16+g] = 1;
@@ -797,21 +797,23 @@
                 }
                 prev_scf = scf;
                 curdiff = fabsf(dist - uplim[w*16+g]);
-                if (curdiff == 0.0f)
+                if (curdiff <= 1.0f)
                     step = 0;
                 else
-                    step = fabsf(log2(curdiff));
+                    step = log2(curdiff);
                 if (dist > uplim[w*16+g])
                     step = -step;
+                scf += step;
+                av_clip_uint8(scf);
+                step = scf - prev_scf;
                 if (FFABS(step) <= 1 || (step > 0 && scf >= max_scf) || (step < 0 && scf <= min_scf)) {
-                    sce->sf_idx[w*16+g] = scf;
+                    sce->sf_idx[w*16+g] = av_clip(scf, min_scf, max_scf);
                     break;
                 }
-                scf += step;
                 if (step > 0)
-                    min_scf = scf;
+                    min_scf = prev_scf;
                 else
-                    max_scf = scf;
+                    max_scf = prev_scf;
             }
             start += size;
         }