Fix a warning in TestWebKitAPI's MediaTime test-suite
authorpeter@chromium.org <peter@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 15 Aug 2012 17:59:08 +0000 (17:59 +0000)
committerpeter@chromium.org <peter@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 15 Aug 2012 17:59:08 +0000 (17:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=94096

Reviewed by Adam Barth.

This warning is visible when building the file with certain GCC versions,
including the one used by Chromium for Android. The warning is visible in
the cr-android build bot output:

MediaTime.cpp:152: warning: this decimal constant is unsigned only in ISO C90

* TestWebKitAPI/Tests/WTF/MediaTime.cpp:
(TestWebKitAPI::TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@125683 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/TestWebKitAPI/Tests/WTF/MediaTime.cpp

index 50366de..ffd7782 100644 (file)
@@ -1,3 +1,19 @@
+2012-08-15  Peter Beverloo  <peter@chromium.org>
+
+        Fix a warning in TestWebKitAPI's MediaTime test-suite
+        https://bugs.webkit.org/show_bug.cgi?id=94096
+
+        Reviewed by Adam Barth.
+
+        This warning is visible when building the file with certain GCC versions,
+        including the one used by Chromium for Android. The warning is visible in
+        the cr-android build bot output:
+
+        MediaTime.cpp:152: warning: this decimal constant is unsigned only in ISO C90
+
+        * TestWebKitAPI/Tests/WTF/MediaTime.cpp:
+        (TestWebKitAPI::TEST):
+
 2012-08-15  Kevin Funk  <kevin.funk@kdab.com>
 
         Fix the 'git log' call in VCSUtils.pm for Windows
index 3f02a0f..d6d48fc 100644 (file)
@@ -149,7 +149,7 @@ TEST(WTF, MediaTime)
     EXPECT_EQ(MediaTime(3, 2).toDouble(), 1.5);
     EXPECT_EQ(MediaTime(1, 1 << 16).toFloat(), 1 / pow(2.0f, 16.0f));
     EXPECT_EQ(MediaTime(1, 1 << 30).toDouble(), 1 / pow(2.0, 30.0));
-    EXPECT_EQ(MediaTime::createWithDouble(M_PI, 1 << 30), MediaTime(3373259426, 1 << 30));
+    EXPECT_EQ(MediaTime::createWithDouble(M_PI, 1 << 30), MediaTime(3373259426U, 1 << 30));
     EXPECT_EQ(MediaTime::createWithFloat(INFINITY), MediaTime::positiveInfiniteTime());
     EXPECT_EQ(MediaTime::createWithFloat(-INFINITY), MediaTime::negativeInfiniteTime());
     EXPECT_EQ(MediaTime::createWithFloat(NAN), MediaTime::invalidTime());