comparison data/fileio/CodedAudioFileReader.cpp @ 322:7623ecfd5891

* correct frame count in cache-in-memory
* fix clipping avoidance code (was only working for 1/channelCount of the samples)

 }
 
 void
 CodedAudioFileReader::pushBuffer(float *buffer, size_t sz, bool final)
 {
-    for (size_t i = 0; i < sz; ++i) {
-        if (buffer[i] >  1.f) buffer[i] =  1.f;
-    }
-    for (size_t i = 0; i < sz; ++i) {
-        if (buffer[i] < -1.f) buffer[i] = -1.f;
-    }
+    float max = 1.0;
+    size_t count = sz * m_channelCount;
 
     if (m_resampler) {
         
         float ratio = float(m_sampleRate) / float(m_fileRate);
 
         if (ratio != 1.f) {
+
             size_t out = m_resampler->resampleInterleaved
                 (buffer,
                  m_resampleBuffer,
                  sz,
                  ratio,
                  final);
 
             buffer = m_resampleBuffer;
             sz = out;
-        }
-
-        for (size_t i = 0; i < sz; ++i) {
-            if (buffer[i] >  1.f) buffer[i] =  1.f;
-        }
-        for (size_t i = 0; i < sz; ++i) {
-            if (buffer[i] < -1.f) buffer[i] = -1.f;
-        }
+            count = sz * m_channelCount;
+        }
+    }
+
+    for (size_t i = 0; i < count; ++i) {
+        if (buffer[i] >  max) buffer[i] =  max;
+    }
+    for (size_t i = 0; i < count; ++i) {
+        if (buffer[i] < -max) buffer[i] = -max;
     }
 
     m_frameCount += sz;
 
     switch (m_cacheMode) {

         //!!! check for return value! out of disk space, etc!
         sf_writef_float(m_cacheFileWritePtr, buffer, sz);
         break;
 
     case CacheInMemory:
-        for (size_t s = 0; s < sz; ++s) {
-            m_data.push_back(buffer[sz]);
+        for (size_t s = 0; s < count; ++s) {
+            m_data.push_back(buffer[count]);
         }
 	MUNLOCK_SAMPLEBLOCK(m_data);
         break;
     }
 }