Title: [225730] trunk/Source/WTF
Revision
225730
Author
[email protected]
Date
2017-12-10 05:00:24 -0800 (Sun, 10 Dec 2017)

Log Message

Unreviewed, follow-up patch after r225726
https://bugs.webkit.org/show_bug.cgi?id=180622

Suggested by Darin. We should take care of signed-extension of `char`.

* wtf/text/StringHasher.h:
(WTF::StringHasher::defaultConverter):

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (225729 => 225730)


--- trunk/Source/WTF/ChangeLog	2017-12-10 02:53:29 UTC (rev 225729)
+++ trunk/Source/WTF/ChangeLog	2017-12-10 13:00:24 UTC (rev 225730)
@@ -1,3 +1,13 @@
+2017-12-10  Yusuke Suzuki  <[email protected]>
+
+        Unreviewed, follow-up patch after r225726
+        https://bugs.webkit.org/show_bug.cgi?id=180622
+
+        Suggested by Darin. We should take care of signed-extension of `char`.
+
+        * wtf/text/StringHasher.h:
+        (WTF::StringHasher::defaultConverter):
+
 2017-12-09  Yusuke Suzuki  <[email protected]>
 
         [WTF] Remove RELAXED_CONSTEXPR

Modified: trunk/Source/WTF/wtf/text/StringHasher.h (225729 => 225730)


--- trunk/Source/WTF/wtf/text/StringHasher.h	2017-12-10 02:53:29 UTC (rev 225729)
+++ trunk/Source/WTF/wtf/text/StringHasher.h	2017-12-10 13:00:24 UTC (rev 225730)
@@ -246,7 +246,7 @@
 
     static constexpr UChar defaultConverter(char character)
     {
-        return character;
+        return static_cast<LChar>(character);
     }
 
     static constexpr UChar defaultConverter(char16_t character)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to