Title: [153611] trunk/Source/WebKit2
Revision
153611
Author
[email protected]
Date
2013-08-01 14:50:58 -0700 (Thu, 01 Aug 2013)

Log Message

Remove return statement in void function
https://bugs.webkit.org/show_bug.cgi?id=119379

Reviewed by Darin Adler.

It does not make sense to return a value in void function.

* WebProcess/WebProcess.cpp:
(WebKit::addCaseFoldedCharacters):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (153610 => 153611)


--- trunk/Source/WebKit2/ChangeLog	2013-08-01 21:48:07 UTC (rev 153610)
+++ trunk/Source/WebKit2/ChangeLog	2013-08-01 21:50:58 UTC (rev 153611)
@@ -1,5 +1,17 @@
 2013-08-01  Kwang Yul Seo  <[email protected]>
 
+        Remove return statement in void function
+        https://bugs.webkit.org/show_bug.cgi?id=119379
+
+        Reviewed by Darin Adler.
+
+        It does not make sense to return a value in void function.
+
+        * WebProcess/WebProcess.cpp:
+        (WebKit::addCaseFoldedCharacters):
+
+2013-08-01  Kwang Yul Seo  <[email protected]>
+
         [WK2] Add USE(SOUP) guard in WebProcess::destroyPrivateBrowsingSession
         https://bugs.webkit.org/show_bug.cgi?id=119381
 

Modified: trunk/Source/WebKit2/WebProcess/WebProcess.cpp (153610 => 153611)


--- trunk/Source/WebKit2/WebProcess/WebProcess.cpp	2013-08-01 21:48:07 UTC (rev 153610)
+++ trunk/Source/WebKit2/WebProcess/WebProcess.cpp	2013-08-01 21:50:58 UTC (rev 153611)
@@ -811,9 +811,11 @@
 {
     if (string.isEmpty())
         return;
-    if (string.is8Bit())
-        return hasher.addCharacters<LChar, CaseFoldingHash::foldCase<LChar>>(string.characters8(), string.length());
-    return hasher.addCharacters<UChar, CaseFoldingHash::foldCase<UChar>>(string.characters16(), string.length());
+    if (string.is8Bit()) {
+        hasher.addCharacters<LChar, CaseFoldingHash::foldCase<LChar>>(string.characters8(), string.length());
+        return;
+    }
+    hasher.addCharacters<UChar, CaseFoldingHash::foldCase<UChar>>(string.characters16(), string.length());
 }
 
 static unsigned hashForPlugInOrigin(const String& pageOrigin, const String& pluginOrigin, const String& mimeType)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to