Changeset 149763 in webkit

Timestamp:

May 8, 2013 1:33:43 PM (11 years ago)

Author:

rwlbuis@webkit.org

Message:

Fix some compiler warnings (miscellaneous)
​https://bugs.webkit.org/show_bug.cgi?id=80790

Patch by Rob Buis <​rbuis@rim.com> on 2013-05-08
Reviewed by Philip Rogers.

Get rid of the following warning for BlackBerry:

BackingStoreClient.cpp:54:21: warning: unused parameter 'parentFrame' [-Wunused-parameter]

by using ASSERT_UNUSED instead of ASSERT.

• WebKitSupport/BackingStoreClient.cpp:

(BlackBerry::WebKit::BackingStoreClient::create):

Location:

trunk/Source/WebKit/blackberry

Files:

• ChangeLog (modified) (1 diff)
• WebKitSupport/BackingStoreClient.cpp (modified) (1 diff)

trunk/Source/WebKit/blackberry/ChangeLog

@@ +1 @@
+2013-05-08  Rob Buis  <rbuis@rim.com>
+
+        Fix some compiler warnings (miscellaneous)
+        https://bugs.webkit.org/show_bug.cgi?id=80790
+
+        Reviewed by Philip Rogers.
+
+        Get rid of the following warning for BlackBerry:
+
+        BackingStoreClient.cpp:54:21: warning: unused parameter 'parentFrame' [-Wunused-parameter]
+
+        by using ASSERT_UNUSED instead of ASSERT.
+
+        * WebKitSupport/BackingStoreClient.cpp:
+        (BlackBerry::WebKit::BackingStoreClient::create):
+
 2013-05-07  Xuefei Ren  <xren@blackberry.com>
 

trunk/Source/WebKit/blackberry/WebKitSupport/BackingStoreClient.cpp

@@ -54 +54 @@
 BackingStoreClient* BackingStoreClient::create(Frame* frame, Frame* parentFrame, WebPage* parentPage)
 {
-    ASSERT(!parentFrame);
+    ASSERT_UNUSED(parentFrame, !parentFrame);
     BackingStoreClient* it = new BackingStoreClient(frame, parentPage);
     return it;