Fix more signedness comparisons in SBuf

diff --git a/src/SBuf.cc b/src/SBuf.cc
index 46faa6bc57a50d79fa4c788ca60ddadbb1ec8fe0..db745a1672f64238a6a9af1fb3a52aa3f13cb4c4 100644 (file)
--- a/src/SBuf.cc
+++ b/src/SBuf.cc
@@ -558,10 +558,6 @@ SBuf::find(char c, size_type startPos) const
     if (startPos > length())
         return npos;
 
-    // ignore invalid startPos
-    if (startPos < 0)
-        startPos = 0;
-
     const void *i = memchr(buf()+startPos, (int)c, (size_type)length()-startPos);
 
     if (i == NULL)
@@ -578,9 +574,6 @@ SBuf::find(const SBuf &needle, size_type startPos) const
         return npos;
     }
 
-    if (startPos < 0)
-        startPos = 0;
-
     // std::string allows needle to overhang hay but not start outside
     if (startPos > length()) {
         ++stats.find;
@@ -673,8 +666,6 @@ SBuf::rfind(char c, SBuf::size_type endPos) const
     // on npos input std::string compares last octet of hay
     if (endPos == npos || endPos >= length()) {
         endPos = length();
-    } else if (endPos < 0) {
-        return npos;
     } else {
         // NP: off-by-one weirdness:
         // endPos is an offset ... 0-based
@@ -709,9 +700,6 @@ SBuf::find_first_of(const SBuf &set, size_type startPos) const
     if (startPos >= length())
         return npos;
 
-    if (startPos < 0)
-        startPos = 0;
-
     if (set.length() == 0)
         return npos;
 
@@ -813,8 +801,6 @@ SBuf::toUpper() const
 void
 SBuf::checkAccessBounds(size_type pos) const
 {
-    if (pos < 0)
-        throw OutOfBoundsException(*this, pos, __FILE__, __LINE__);
     if (pos >= length())
         throw OutOfBoundsException(*this, pos, __FILE__, __LINE__);
 }