Title: [140168] trunk/Source/WebCore
Revision
140168
Author
commit-qu...@webkit.org
Date
2013-01-18 09:51:44 -0800 (Fri, 18 Jan 2013)

Log Message

[BlackBerry] fix compile warnings in NetworkJob
https://bugs.webkit.org/show_bug.cgi?id=107293

Patch by Joe Mason <jma...@rim.com> on 2013-01-18
Reviewed by Yong Li.

Internal PR: 283096

* platform/network/blackberry/NetworkJob.cpp:
(WebCore):
(WebCore::isAppendableHeader):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (140167 => 140168)


--- trunk/Source/WebCore/ChangeLog	2013-01-18 17:50:33 UTC (rev 140167)
+++ trunk/Source/WebCore/ChangeLog	2013-01-18 17:51:44 UTC (rev 140168)
@@ -1,3 +1,16 @@
+2013-01-18  Joe Mason  <jma...@rim.com>
+
+        [BlackBerry] fix compile warnings in NetworkJob
+        https://bugs.webkit.org/show_bug.cgi?id=107293
+
+        Reviewed by Yong Li.
+
+        Internal PR: 283096
+
+        * platform/network/blackberry/NetworkJob.cpp:
+        (WebCore):
+        (WebCore::isAppendableHeader):
+
 2013-01-18  Dominic Mazzoni  <dmazz...@google.com>
 
         REGRESSION (r132699): Crashes in WebCore::TextIterator::handleTextNodeFirstLetter

Modified: trunk/Source/WebCore/platform/network/blackberry/NetworkJob.cpp (140167 => 140168)


--- trunk/Source/WebCore/platform/network/blackberry/NetworkJob.cpp	2013-01-18 17:50:33 UTC (rev 140167)
+++ trunk/Source/WebCore/platform/network/blackberry/NetworkJob.cpp	2013-01-18 17:51:44 UTC (rev 140168)
@@ -63,7 +63,7 @@
     return statusCode == 401;
 }
 
-static char* const appendableHeaders[] = {"access-control-allow-origin", "allow",
+static const char* const appendableHeaders[] = {"access-control-allow-origin", "allow",
     "set-cookie", "set-cookie2", "vary", "via", "warning"};
 
 static bool isAppendableHeader(const String& key)
@@ -72,7 +72,7 @@
     if (key.startsWith("x-"))
         return true;
 
-    for (int i = 0; i < sizeof(appendableHeaders) /sizeof(char*); i++)
+    for (size_t i = 0; i < sizeof(appendableHeaders) /sizeof(char*); i++)
         if (key == appendableHeaders[i])
             return true;
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to