2011-04-15 Kenneth Russell <kbr@google.com>

kenrussell · kenrussell · commit 7d22f504d95c · 2011-04-15T23:03:02.000Z
Unreviewed. Chromium Linux Release build fix due to unused variables. * platform/audio/mkl/FFTFrameMKL.cpp: (WebCore::FFTFrame::doFFT): (WebCore::FFTFrame::doInverseFFT): (WebCore::FFTFrame::cleanup): Canonical link: https://commits.webkit.org/73787@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@84046 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
@@ -1,3 +1,12 @@
+2011-04-15  Kenneth Russell  <kbr@google.com>
+
+        Unreviewed. Chromium Linux Release build fix due to unused variables.
+
+        * platform/audio/mkl/FFTFrameMKL.cpp:
+        (WebCore::FFTFrame::doFFT):
+        (WebCore::FFTFrame::doInverseFFT):
+        (WebCore::FFTFrame::cleanup):
+
 2011-04-15  Brent Fulgham  <bfulgham@webkit.org>
 
         Unreviewed build change after r83945.
diff --git a/Source/WebCore/platform/audio/mkl/FFTFrameMKL.cpp b/Source/WebCore/platform/audio/mkl/FFTFrameMKL.cpp
@@ -174,7 +174,7 @@ void FFTFrame::doFFT(float* data)
 {
     // Compute Forward transform.
     MKL_LONG status = DftiComputeForward(m_handle, data, m_complexData.data());
-    ASSERT(DftiErrorClass(status, DFTI_NO_ERROR));
+    ASSERT_UNUSED(status, DftiErrorClass(status, DFTI_NO_ERROR));
 
     // De-interleave to separate real and complex arrays. FIXME:
     // figure out if it's possible to get MKL to use split-complex
@@ -199,7 +199,7 @@ void FFTFrame::doInverseFFT(float* data)
 
     // Compute backward transform.
     MKL_LONG status = DftiComputeBackward(m_handle, interleavedData, data);
-    ASSERT(DftiErrorClass(status, DFTI_NO_ERROR));
+    ASSERT_UNUSED(status, DftiErrorClass(status, DFTI_NO_ERROR));
 }
 
 void FFTFrame::initialize()
@@ -214,7 +214,7 @@ void FFTFrame::cleanup()
     for (int i = 0; i < kMaxFFTPow2Size; ++i) {
         if (descriptorHandles[i]) {
             MKL_LONG status = DftiFreeDescriptor(&descriptorHandles[i]);
-            ASSERT(DftiErrorClass(status, DFTI_NO_ERROR));
+            ASSERT_UNUSED(status, DftiErrorClass(status, DFTI_NO_ERROR));
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ void FFTFrame::doFFT(float* data)`
`174`	`174`	`{`
`175`	`175`	`// Compute Forward transform.`
`176`	`176`	`MKL_LONG status = DftiComputeForward(m_handle, data, m_complexData.data());`
`177`		`- ASSERT(DftiErrorClass(status, DFTI_NO_ERROR));`
	`177`	`+ ASSERT_UNUSED(status, DftiErrorClass(status, DFTI_NO_ERROR));`
`178`	`178`
`179`	`179`	`// De-interleave to separate real and complex arrays. FIXME:`
`180`	`180`	`// figure out if it's possible to get MKL to use split-complex`
`@@ -199,7 +199,7 @@ void FFTFrame::doInverseFFT(float* data)`
`199`	`199`
`200`	`200`	`// Compute backward transform.`
`201`	`201`	`MKL_LONG status = DftiComputeBackward(m_handle, interleavedData, data);`
`202`		`- ASSERT(DftiErrorClass(status, DFTI_NO_ERROR));`
	`202`	`+ ASSERT_UNUSED(status, DftiErrorClass(status, DFTI_NO_ERROR));`
`203`	`203`	`}`
`204`	`204`
`205`	`205`	`void FFTFrame::initialize()`
`@@ -214,7 +214,7 @@ void FFTFrame::cleanup()`
`214`	`214`	`for (int i = 0; i < kMaxFFTPow2Size; ++i) {`
`215`	`215`	`if (descriptorHandles[i]) {`
`216`	`216`	`MKL_LONG status = DftiFreeDescriptor(&descriptorHandles[i]);`
`217`		`- ASSERT(DftiErrorClass(status, DFTI_NO_ERROR));`
	`217`	`+ ASSERT_UNUSED(status, DftiErrorClass(status, DFTI_NO_ERROR));`
`218`	`218`	`}`
`219`	`219`	`}`
`220`	`220`