diff options
author | Will Dietz <wdietz2@illinois.edu> | 2012-08-30 00:30:21 +0000 |
---|---|---|
committer | Will Dietz <wdietz2@illinois.edu> | 2012-08-30 00:30:21 +0000 |
commit | 55a4b514faad3c7e91b1cb0ffd3313724c7efc03 (patch) | |
tree | 89cc59e9182c016396ebf88d9b88d86db48b5548 /include | |
parent | faa1159a6915992d7f035ce06caf952fd4a4e96a (diff) | |
download | external_llvm-55a4b514faad3c7e91b1cb0ffd3313724c7efc03.zip external_llvm-55a4b514faad3c7e91b1cb0ffd3313724c7efc03.tar.gz external_llvm-55a4b514faad3c7e91b1cb0ffd3313724c7efc03.tar.bz2 |
Fix HashString's Bernstein hash to use unsigned chars, as is usually done.
Changes the hash result for strings containing characters
with values >= 128, such as UTF8 strings (not normal ASCII).
Changed mostly so we match other implementations.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@162882 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include')
-rw-r--r-- | include/llvm/ADT/StringExtras.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/llvm/ADT/StringExtras.h b/include/llvm/ADT/StringExtras.h index 655d884..36df5ac 100644 --- a/include/llvm/ADT/StringExtras.h +++ b/include/llvm/ADT/StringExtras.h @@ -125,7 +125,7 @@ void SplitString(StringRef Source, // X*33+c -> X*33^c static inline unsigned HashString(StringRef Str, unsigned Result = 0) { for (unsigned i = 0, e = Str.size(); i != e; ++i) - Result = Result * 33 + Str[i]; + Result = Result * 33 + (unsigned char)Str[i]; return Result; } |