[FFmpeg-cvslog] r23182 - in branches/0.6: . libavcodec/aaccoder.c

siretart subversion
Wed May 19 13:38:07 CEST 2010


Author: siretart
Date: Wed May 19 13:38:07 2010
New Revision: 23182

Log:
Make the faac inspired quantizer search make sense for a slightly narrower definition of "make sense."

backport r23035 by alexc

Modified:
   branches/0.6/   (props changed)
   branches/0.6/libavcodec/aaccoder.c

Modified: branches/0.6/libavcodec/aaccoder.c
==============================================================================
--- branches/0.6/libavcodec/aaccoder.c	Wed May 19 11:46:29 2010	(r23181)
+++ branches/0.6/libavcodec/aaccoder.c	Wed May 19 13:38:07 2010	(r23182)
@@ -763,7 +763,7 @@ static void search_for_quantizers_faac(A
             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 @@ static void search_for_quantizers_faac(A
                 }
                 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;
         }



More information about the ffmpeg-cvslog mailing list