Title: [128848] trunk/Source/WebCore
Revision
128848
Author
[email protected]
Date
2012-09-17 21:37:50 -0700 (Mon, 17 Sep 2012)

Log Message

[Gtk] Remove the unused variable warning in GamepadsGtk.cpp using ASSERT_UNUSED macro
https://bugs.webkit.org/show_bug.cgi?id=96975

Patch by Vivek Galatage <[email protected]> on 2012-09-17
Reviewed by Kentaro Hara.

Replacing a simple ASSERT with ASSERT_UNUSED to avoid the warning.

No new tests as refactoring done.

* platform/gtk/GamepadsGtk.cpp:
(WebCore::GamepadDeviceGtk::readCallback):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (128847 => 128848)


--- trunk/Source/WebCore/ChangeLog	2012-09-18 04:16:06 UTC (rev 128847)
+++ trunk/Source/WebCore/ChangeLog	2012-09-18 04:37:50 UTC (rev 128848)
@@ -1,3 +1,17 @@
+2012-09-17  Vivek Galatage  <[email protected]>
+
+        [Gtk] Remove the unused variable warning in GamepadsGtk.cpp using ASSERT_UNUSED macro
+        https://bugs.webkit.org/show_bug.cgi?id=96975
+
+        Reviewed by Kentaro Hara.
+
+        Replacing a simple ASSERT with ASSERT_UNUSED to avoid the warning.
+
+        No new tests as refactoring done.
+
+        * platform/gtk/GamepadsGtk.cpp:
+        (WebCore::GamepadDeviceGtk::readCallback):
+
 2012-09-17  Roger Fong  <[email protected]>
 
         [Win] Null check timing function received from CoreAnimation when calling CACFAnimationGetTimingFunction.

Modified: trunk/Source/WebCore/platform/gtk/GamepadsGtk.cpp (128847 => 128848)


--- trunk/Source/WebCore/platform/gtk/GamepadsGtk.cpp	2012-09-18 04:16:06 UTC (rev 128847)
+++ trunk/Source/WebCore/platform/gtk/GamepadsGtk.cpp	2012-09-18 04:37:50 UTC (rev 128848)
@@ -91,7 +91,7 @@
     if (error)
         return g_error_matches(error.get(), G_IO_ERROR, G_IO_ERROR_WOULD_BLOCK);
 
-    ASSERT(len == sizeof(event));
+    ASSERT_UNUSED(len, len == sizeof(event));
     gamepadDevice->updateForEvent(event);
     return TRUE;
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to