[FFmpeg-devel] [PATCH] aacenc: avoid double in quantize_bands.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Tue Mar 1 22:55:08 CET 2016


I cannot see any point whatsoever to use
double here instead of float.
Using float allows for use of SIMD.
---
 libavcodec/aacenc_utils.h | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/libavcodec/aacenc_utils.h b/libavcodec/aacenc_utils.h
index cb5bc8d..571b1e6 100644
--- a/libavcodec/aacenc_utils.h
+++ b/libavcodec/aacenc_utils.h
@@ -66,10 +66,9 @@ static inline void quantize_bands(int *out, const float *in, const float *scaled
                                   const float rounding)
 {
     int i;
-    double qc;
     for (i = 0; i < size; i++) {
-        qc = scaled[i] * Q34;
-        out[i] = (int)FFMIN(qc + rounding, (double)maxval);
+        float qc = scaled[i] * Q34;
+        out[i] = (int)FFMIN(qc + rounding, (float)maxval);
         if (is_signed && in[i] < 0.0f) {
             out[i] = -out[i];
         }
-- 
2.7.0



More information about the ffmpeg-devel mailing list