Title: [121797] trunk/Source/WebCore
Revision
121797
Author
[email protected]
Date
2012-07-03 12:12:13 -0700 (Tue, 03 Jul 2012)

Log Message

[Gtk][Gamepads] 'warning: comparison is always true due to limited range of data type [-Wtype-limits]' in GamepadsGtk.cpp
https://bugs.webkit.org/show_bug.cgi?id=90477

Reviewed by Martin Robinson.

Vector::find() returns value of type size_t, springing errors when
assigning the value to a variable of type unsigned. Use size_t type
for that variable instead.

No new tests - no changed functionality.

* platform/gtk/GamepadsGtk.cpp:
(WebCore::GamepadsGtk::unregisterDevice):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (121796 => 121797)


--- trunk/Source/WebCore/ChangeLog	2012-07-03 19:08:04 UTC (rev 121796)
+++ trunk/Source/WebCore/ChangeLog	2012-07-03 19:12:13 UTC (rev 121797)
@@ -1,3 +1,19 @@
+2012-07-03  Zan Dobersek  <[email protected]>
+
+        [Gtk][Gamepads] 'warning: comparison is always true due to limited range of data type [-Wtype-limits]' in GamepadsGtk.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=90477
+
+        Reviewed by Martin Robinson.
+
+        Vector::find() returns value of type size_t, springing errors when
+        assigning the value to a variable of type unsigned. Use size_t type
+        for that variable instead.
+
+        No new tests - no changed functionality.
+
+        * platform/gtk/GamepadsGtk.cpp:
+        (WebCore::GamepadsGtk::unregisterDevice):
+
 2012-07-03  Dominik Röttsches  <[email protected]>
 
         Fix LayoutUnit usage in RenderImage::imageDimensionsChanged

Modified: trunk/Source/WebCore/platform/gtk/GamepadsGtk.cpp (121796 => 121797)


--- trunk/Source/WebCore/platform/gtk/GamepadsGtk.cpp	2012-07-03 19:08:04 UTC (rev 121796)
+++ trunk/Source/WebCore/platform/gtk/GamepadsGtk.cpp	2012-07-03 19:12:13 UTC (rev 121797)
@@ -154,7 +154,7 @@
     ASSERT(m_deviceMap.contains(deviceFile));
 
     GamepadDeviceGtk* gamepadDevice = m_deviceMap.take(deviceFile);
-    unsigned index = m_slots.find(gamepadDevice);
+    size_t index = m_slots.find(gamepadDevice);
     ASSERT(index != notFound);
 
     m_slots[index].clear();
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to