comparison layer/SpectrogramLayer.cpp @ 102:8ce53683d0d7 sv1-v0.9rc2

* I wonder if the fftw_ functions are not as thread-safe as I'd hoped?

                  MatrixFile::ReadOnly);
 
 	    m_layer.setColourmap();
 //!!!	    m_layer.m_writeCache->reset();
 
+	    double *input = (double *)
+		fftw_malloc(windowSize * sizeof(double));
+
+	    fftw_complex *output = (fftw_complex *)
+		fftw_malloc(windowSize * sizeof(fftw_complex));
+
+            float *workbuffer = (float *)
+                fftw_malloc(windowSize * sizeof(float));
+
+	    fftw_plan plan = fftw_plan_dft_r2c_1d(windowSize, input,
+						  output, FFTW_ESTIMATE);
+
+	    if (!plan) {
+		std::cerr << "WARNING: fftw_plan_dft_r2c_1d(" << windowSize << ") failed!" << std::endl;
+		fftw_free(input);
+		fftw_free(output);
+                fftw_free(workbuffer);
+		continue;
+	    }
+
 	    // We don't need a lock when writing to or reading from
 	    // the pixels in the cache.  We do need to ensure we have
 	    // the width and height of the cache and the FFT
 	    // parameters known before we unlock, in case they change
 	    // in the model while we aren't holding a lock.  It's safe

 	    // happens, because they will continue to match the
 	    // dimensions of the actual cache (which this thread
 	    // manages, not the layer's).
 	    m_layer.m_mutex.unlock();
 
-	    double *input = (double *)
-		fftw_malloc(windowSize * sizeof(double));
-
-	    fftw_complex *output = (fftw_complex *)
-		fftw_malloc(windowSize * sizeof(fftw_complex));
-
-            float *workbuffer = (float *)
-                fftw_malloc(windowSize * sizeof(float));
-
-	    fftw_plan plan = fftw_plan_dft_r2c_1d(windowSize, input,
-						  output, FFTW_ESTIMATE);
-
 	    Window<double> windower(windowType, windowSize);
-
-	    if (!plan) {
-		std::cerr << "WARNING: fftw_plan_dft_r2c_1d(" << windowSize << ") failed!" << std::endl;
-		fftw_free(input);
-		fftw_free(output);
-                fftw_free(workbuffer);
-		m_layer.m_mutex.lock();
-		continue;
-	    }
 
 	    int counter = 0;
 	    int updateAt = (end / windowIncrement) / 20;
 	    if (updateAt < 100) updateAt = 100;