Changeset 95333 in webkit

Timestamp:

Sep 16, 2011 4:24:34 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

Set but not used variables cleanup (gcc 4.6)
​https://bugs.webkit.org/show_bug.cgi?id=68157

Patch by Max Perepelitsyn <​pph34r@gmail.com> on 2011-09-16
Reviewed by Darin Adler.

• platform/audio/ReverbConvolverStage.cpp:

(WebCore::ReverbConvolverStage::process):

• rendering/RenderMediaControlsChromium.cpp:

(WebCore::paintMediaButton):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/audio/ReverbConvolverStage.cpp (modified) (2 diffs)
• rendering/RenderMediaControlsChromium.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-09-16  Max Perepelitsyn  <pph34r@gmail.com>
+
+        Set but not used variables cleanup (gcc 4.6)
+        https://bugs.webkit.org/show_bug.cgi?id=68157
+
+        Reviewed by Darin Adler.
+
+        * platform/audio/ReverbConvolverStage.cpp:
+        (WebCore::ReverbConvolverStage::process):
+        * rendering/RenderMediaControlsChromium.cpp:
+        (WebCore::paintMediaButton):
+
 2011-09-16  Geoffrey Garen  <ggaren@apple.com>
 

trunk/Source/WebCore/platform/audio/ReverbConvolverStage.cpp

@@ -123 +123 @@
         return;
 
-    int writeIndex = 0;
-
     if (m_framesProcessed < m_preDelayLength) {
         // For the first m_preDelayLength frames don't process the convolver, instead simply buffer in the pre-delay.
@@ -136 +134 @@
 
         // Now accumulate into reverb's accumulation buffer.
-        writeIndex = m_accumulationBuffer->accumulate(temporaryBuffer, framesToProcess, &m_accumulationReadIndex, m_postDelayLength);
+        m_accumulationBuffer->accumulate(temporaryBuffer, framesToProcess, &m_accumulationReadIndex, m_postDelayLength);
     }
 

trunk/Source/WebCore/rendering/RenderMediaControlsChromium.cpp

@@ -64 +64 @@
 static bool paintMediaButton(GraphicsContext* context, const IntRect& rect, Image* image)
 {
-    IntRect imageRect = image->rect();
     context->drawImage(image, ColorSpaceDeviceRGB, rect);
     return true;