comparison layer/SpectrogramLayer.cpp @ 483:18f38f564d7c

* reorganise function for faster data retrieval

     if (minbin < 0) minbin = 0;
     if (maxbin < 0) maxbin = minbin+1;
 
     int psx = -1;
     float values[maxbin - minbin + 1];
+    float peaks[h];
 
     for (int x = 0; x < w; ++x) {
         
         if (binforx[x] < 0) continue;
 
-        for (int y = 0; y < h; ++y) {
-
-            unsigned char peakpix = 0;
-
-            int sx0 = binforx[x];
-            int sx1 = sx0;
-            if (x+1 < w) sx1 = binforx[x+1];
-            if (sx0 < 0) sx0 = sx1 - 1;
-            if (sx0 < 0) continue;
-            if (sx1 <= sx0) sx1 = sx0 + 1;
+        int sx0 = binforx[x];
+        int sx1 = sx0;
+        if (x+1 < w) sx1 = binforx[x+1];
+        if (sx0 < 0) sx0 = sx1 - 1;
+        if (sx0 < 0) continue;
+        if (sx1 <= sx0) sx1 = sx0 + 1;
+
+        for (int y = 0; y < h; ++y) peaks[y] = 0.f;
             
-            for (int sx = sx0; sx < sx1; ++sx) {
-
-                if (sx < 0 || sx >= int(fft->getWidth())) continue;
-
-                if (!m_synchronous) {
-                    if (!fft->isColumnAvailable(sx)) {
-#ifdef DEBUG_SPECTROGRAM_REPAINT
-                        std::cerr << "Met unavailable column at col " << sx << std::endl;
-#endif
-                        return false;
-                    }
+        for (int sx = sx0; sx < sx1; ++sx) {
+
+            if (sx < 0 || sx >= int(fft->getWidth())) continue;
+
+            if (!m_synchronous) {
+                if (!fft->isColumnAvailable(sx)) {
+#ifdef DEBUG_SPECTROGRAM_REPAINT
+                    std::cerr << "Met unavailable column at col " << sx << std::endl;
+#endif
+                    return false;
                 }
-
-                if (sx != psx) {
-                    fft->getMagnitudesAt(sx, values, minbin, maxbin - minbin + 1);
-                    psx = sx;
-                }
+            }
+
+            if (sx != psx) {
+                cerr << "Retrieving column " << sx << endl;
+                fft->getMagnitudesAt(sx, values, minbin, maxbin - minbin + 1);
+                psx = sx;
+            }
+
+            for (int y = 0; y < h; ++y) {
 
                 int sy0 = binfory[y];
                 int sy1 = sy0;
                 if (y+1 < h) sy1 = binfory[y+1];
                 if (sy0 < 0) sy0 = sy1 - 1;

 
                 //!!! review -- if we know we're dealing with
                 //!!! magnitudes here, we can just use peak of the
                 //!!! float values
 
-                float peak = 0.f;
-                
                 for (int sy = sy0; sy < sy1; ++sy) {
 
                     float value = values[sy - minbin];
 /*
                     if (m_colourScale != PhaseColourScale) {

 
                     unsigned char pix = getDisplayValue(v, value);
                     if (pix > peakpix) peakpix = pix;
 //                    cerr <<x<<","<<y<<" -> "<<sx<<","<<sy<<" -> "<<values[sy]<<" -> "<<(int)pix<< endl;
 */
-                    if (value > peak) peak = value; //!!! not right for phase!
+                    if (value > peaks[y]) peaks[y] = value; //!!! not right for phase!
                 }
-
-                if (m_colourScale != PhaseColourScale) {
-                    if (!m_normalizeColumns) {
-                        peak /= (m_fftSize/2.f);
-                    }
+            }
+        }
+
+        for (int y = 0; y < h; ++y) {
+
+            float peak = peaks[y];
+            
+            if (m_colourScale != PhaseColourScale) {
+                if (!m_normalizeColumns) {
+                    peak /= (m_fftSize/2.f);
+                }
 //!!!                        mag.sample(value);
-                    peak *= m_gain;
-                }
-
-                peakpix = getDisplayValue(v, peak);
-            }
+                peak *= m_gain;
+            }
+            
+            unsigned char peakpix = getDisplayValue(v, peak);
 
             m_drawBuffer.setPixel(x, h-y-1, peakpix);
         }
     }