Changeset 225730 in webkit

Timestamp:

Dec 10, 2017 5:00:24 AM (6 years ago)

Author:

Yusuke Suzuki

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):

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/text/StringHasher.h (modified) (1 diff)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2017-12-10  Yusuke Suzuki  <utatane.tea@gmail.com>
+
+        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  <utatane.tea@gmail.com>
 

trunk/Source/WTF/wtf/text/StringHasher.h

@@ -247 +247 @@
     static constexpr UChar defaultConverter(char character)
     {
-        return character;
+        return static_cast<LChar>(character);
     }