Not handling zero length strings.
authorJim Laskey <jlaskey@mac.com>
Sun, 29 Oct 2006 08:27:07 +0000 (08:27 +0000)
committerJim Laskey <jlaskey@mac.com>
Sun, 29 Oct 2006 08:27:07 +0000 (08:27 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@31277 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Support/FoldingSet.cpp

index 2ae987605871ca350078181b3435c5cf090d2037..4fe67d7895eaa616eadda3df862bea8b59cadf03 100644 (file)
@@ -61,7 +61,7 @@ void FoldingSetImpl::NodeID::AddString(const std::string &String) {
     Pos = Units * sizeof(unsigned);
   } else {
     // Otherwise do it the hard way.
-    for ( Pos += 4; Pos < Size; Pos += 4) {
+    for ( Pos += 4; Pos <= Size; Pos += 4) {
       unsigned V = ((unsigned char)String[Pos - 4] << 24) |
                    ((unsigned char)String[Pos - 3] << 16) |
                    ((unsigned char)String[Pos - 2] << 8) |
@@ -77,7 +77,7 @@ void FoldingSetImpl::NodeID::AddString(const std::string &String) {
   case 1: V = (V << 8) | (unsigned char)String[Size - 3]; // Fall thru.
   case 2: V = (V << 8) | (unsigned char)String[Size - 2]; // Fall thru.
   case 3: V = (V << 8) | (unsigned char)String[Size - 1]; break;
-  case 0: return; // Nothing left.
+  default: return; // Nothing left.
   }
 
   Bits.push_back(V);