fixed compiler warnings
authorVictor Kirhenshtein <victor@netxms.org>
Mon, 9 Oct 2017 09:38:42 +0000 (12:38 +0300)
committerVictor Kirhenshtein <victor@netxms.org>
Mon, 9 Oct 2017 09:38:42 +0000 (12:38 +0300)
src/libnetxms/diff.cpp
src/libnetxms/string.cpp

index 6a6c9d7..5228db0 100644 (file)
@@ -1427,6 +1427,9 @@ String DiffEngine::diff_generateLineDiff(ObjectArray<Diff> *diffs)
          case DIFF_DELETE:
             AppendLines(out, aDiff->text, _T('-'));
             break;
+         case DIFF_EQUAL:
+            /* nothing to do - added to avoid compiler warning */
+            break;
       }
    }
    return out;
index 0fc561d..8404a89 100644 (file)
@@ -428,7 +428,7 @@ void String::replace(const TCHAR *pszSrc, const TCHAR *pszDst)
 String String::substring(size_t start, ssize_t len) const
 {
    String s;
-   if ((start < m_length) && (start >= 0))
+   if (start < m_length)
    {
       size_t count;
       if (len == -1)
@@ -450,7 +450,7 @@ String String::substring(size_t start, ssize_t len) const
 TCHAR *String::substring(size_t start, ssize_t len, TCHAR *buffer) const
 {
        TCHAR *s;
-       if ((start < m_length) && (start >= 0))
+       if (start < m_length)
        {
           size_t count;
                if (len == -1)
@@ -478,7 +478,7 @@ TCHAR *String::substring(size_t start, ssize_t len, TCHAR *buffer) const
  */
 int String::find(const TCHAR *str, size_t start) const
 {
-       if ((start >= m_length) || (start < 0))
+       if (start >= m_length)
                return npos;
 
        TCHAR *p = _tcsstr(&m_buffer[start], str);