Changeset 149794 in webkit

Timestamp:

May 8, 2013 11:39:22 PM (11 years ago)

Author:

commit-queue@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 Brent Fulgham.

Get rid of the following warning for BlackBerry:

NavigatorContentUtils.cpp:60:78: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]

by not compiling any of the code to add protocols to the protocol whitelist.

• Modules/navigatorcontentutils/NavigatorContentUtils.cpp:

(WebCore::initProtocolHandlerWhitelist):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• Modules/navigatorcontentutils/NavigatorContentUtils.cpp (modified) (2 diffs)

trunk/Source/WebCore/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 Brent Fulgham.
+
+        Get rid of the following warning for BlackBerry:
+
+        NavigatorContentUtils.cpp:60:78: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
+
+        by not compiling any of the code to add protocols to the protocol whitelist.
+
+        * Modules/navigatorcontentutils/NavigatorContentUtils.cpp:
+        (WebCore::initProtocolHandlerWhitelist):
+
 2013-05-08  peavo@outlook.com  <peavo@outlook.com>
 

trunk/Source/WebCore/Modules/navigatorcontentutils/NavigatorContentUtils.cpp

@@ -44 +44 @@
 {
     protocolWhitelist = new HashSet<String>;
+#if !PLATFORM(BLACKBERRY)
     static const char* protocols[] = {
-#if !PLATFORM(BLACKBERRY)
         "irc",
         "mailto",
@@ -56 +56 @@
         "urn",
         "webcal",
-#endif
     };
     for (size_t i = 0; i < WTF_ARRAY_LENGTH(protocols); ++i)
         protocolWhitelist->add(protocols[i]);
+#endif
 }