Get rid of most instance of 'foo == NULL'

core/src/fxcrt/fx_basic_bstring.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include <stddef.h>  // For offsetof().
 #include <cctype>
 
 #include "core/include/fxcrt/fx_basic.h"
@@ skipping 90 matching lines @@
     m_pData = NULL;
   }
 }
 CFX_ByteString::CFX_ByteString(char ch) {
   m_pData = StringData::Create(1);
   if (m_pData) {
     m_pData->m_String[0] = ch;
   }
 }
 CFX_ByteString::CFX_ByteString(const CFX_ByteString& stringSrc) {
-  if (stringSrc.m_pData == NULL) {
+  if (!stringSrc.m_pData) {
     m_pData = NULL;
     return;
   }
   if (stringSrc.m_pData->m_nRefs >= 0) {
     m_pData = stringSrc.m_pData;
     m_pData->Retain();
   } else {
     m_pData = NULL;
     *this = stringSrc;
   }
@@ skipping 14 matching lines @@
     return;
   }
   m_pData = StringData::Create(nNewLen);
   if (m_pData) {
     FXSYS_memcpy(m_pData->m_String, str1.GetCStr(), str1.GetLength());
     FXSYS_memcpy(m_pData->m_String + str1.GetLength(), str2.GetCStr(),
                  str2.GetLength());
   }
 }
 const CFX_ByteString& CFX_ByteString::operator=(const FX_CHAR* lpsz) {
-  if (lpsz == NULL || lpsz[0] == 0) {
+  if (!lpsz || lpsz[0] == 0) {
     Empty();
   } else {
     AssignCopy(FXSYS_strlen(lpsz), lpsz);
   }
   return *this;
 }
 const CFX_ByteString& CFX_ByteString::operator=(const CFX_ByteStringC& str) {
   if (str.IsEmpty()) {
     Empty();
   } else {
@@ skipping 39 matching lines @@
   if (lpsz) {
     ConcatInPlace(FXSYS_strlen(lpsz), lpsz);
   }
   return *this;
 }
 const CFX_ByteString& CFX_ByteString::operator+=(char ch) {
   ConcatInPlace(1, &ch);
   return *this;
 }
 const CFX_ByteString& CFX_ByteString::operator+=(const CFX_ByteString& string) {
-  if (string.m_pData == NULL) {
+  if (!string.m_pData) {
     return *this;
   }
   ConcatInPlace(string.m_pData->m_nDataLength, string.m_pData->m_String);
   return *this;
 }
 const CFX_ByteString& CFX_ByteString::operator+=(
     const CFX_ByteStringC& string) {
   if (string.IsEmpty()) {
     return *this;
   }
   ConcatInPlace(string.GetLength(), string.GetCStr());
   return *this;
 }
 bool CFX_ByteString::Equal(const char* ptr) const {
   if (!m_pData) {
     return !ptr || ptr[0] == '\0';
   }
   if (!ptr) {
     return m_pData->m_nDataLength == 0;
   }
   return FXSYS_strlen(ptr) == m_pData->m_nDataLength &&
          FXSYS_memcmp(ptr, m_pData->m_String, m_pData->m_nDataLength) == 0;
 }
 bool CFX_ByteString::Equal(const CFX_ByteStringC& str) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return str.IsEmpty();
   }
   return m_pData->m_nDataLength == str.GetLength() &&
          FXSYS_memcmp(m_pData->m_String, str.GetCStr(), str.GetLength()) == 0;
 }
 bool CFX_ByteString::Equal(const CFX_ByteString& other) const {
   if (IsEmpty()) {
     return other.IsEmpty();
   }
   if (other.IsEmpty()) {
     return false;
   }
   return other.m_pData->m_nDataLength == m_pData->m_nDataLength &&
          FXSYS_memcmp(other.m_pData->m_String, m_pData->m_String,
                       m_pData->m_nDataLength) == 0;
 }
 void CFX_ByteString::Empty() {
   if (m_pData) {
     m_pData->Release();
     m_pData = NULL;
   }
 }
 bool CFX_ByteString::EqualNoCase(const CFX_ByteStringC& str) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return str.IsEmpty();
   }
   FX_STRSIZE len = str.GetLength();
   if (m_pData->m_nDataLength != len) {
     return false;
   }
   const uint8_t* pThis = (const uint8_t*)m_pData->m_String;
   const uint8_t* pThat = str.GetPtr();
   for (FX_STRSIZE i = 0; i < len; i++) {
     if ((*pThis) != (*pThat)) {
@@ skipping 15 matching lines @@
   return true;
 }
 void CFX_ByteString::AssignCopy(FX_STRSIZE nSrcLen,
                                 const FX_CHAR* lpszSrcData) {
   AllocBeforeWrite(nSrcLen);
   FXSYS_memcpy(m_pData->m_String, lpszSrcData, nSrcLen);
   m_pData->m_nDataLength = nSrcLen;
   m_pData->m_String[nSrcLen] = 0;
 }
 void CFX_ByteString::CopyBeforeWrite() {
-  if (m_pData == NULL || m_pData->m_nRefs <= 1) {
+  if (!m_pData || m_pData->m_nRefs <= 1) {
     return;
   }
   StringData* pData = m_pData;
   m_pData->Release();
   FX_STRSIZE nDataLength = pData->m_nDataLength;
   m_pData = StringData::Create(nDataLength);
   if (m_pData) {
     FXSYS_memcpy(m_pData->m_String, pData->m_String, nDataLength + 1);
   }
 }
 void CFX_ByteString::AllocBeforeWrite(FX_STRSIZE nLen) {
   if (m_pData && m_pData->m_nRefs <= 1 && m_pData->m_nAllocLength >= nLen) {
     return;
   }
   Empty();
   m_pData = StringData::Create(nLen);
 }
 void CFX_ByteString::ReleaseBuffer(FX_STRSIZE nNewLength) {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return;
   }
   CopyBeforeWrite();
   if (nNewLength == -1) {
     nNewLength = FXSYS_strlen((const FX_CHAR*)m_pData->m_String);
   }
   if (nNewLength == 0) {
     Empty();
     return;
   }
   FXSYS_assert(nNewLength <= m_pData->m_nAllocLength);
   m_pData->m_nDataLength = nNewLength;
   m_pData->m_String[nNewLength] = 0;
 }
 void CFX_ByteString::Reserve(FX_STRSIZE len) {
   GetBuffer(len);
   ReleaseBuffer(GetLength());
 }
 FX_CHAR* CFX_ByteString::GetBuffer(FX_STRSIZE nMinBufLength) {
-  if (m_pData == NULL && nMinBufLength == 0) {
+  if (!m_pData && nMinBufLength == 0) {
     return NULL;
   }
   if (m_pData && m_pData->m_nRefs <= 1 &&
       m_pData->m_nAllocLength >= nMinBufLength) {
     return m_pData->m_String;
   }
-  if (m_pData == NULL) {
+  if (!m_pData) {
     m_pData = StringData::Create(nMinBufLength);
     if (!m_pData) {
       return NULL;
     }
     m_pData->m_nDataLength = 0;
     m_pData->m_String[0] = 0;
     return m_pData->m_String;
   }
   StringData* pOldData = m_pData;
   FX_STRSIZE nOldLen = pOldData->m_nDataLength;
   if (nMinBufLength < nOldLen) {
     nMinBufLength = nOldLen;
   }
   m_pData = StringData::Create(nMinBufLength);
   if (!m_pData) {
     return NULL;
   }
   FXSYS_memcpy(m_pData->m_String, pOldData->m_String, (nOldLen + 1));
   m_pData->m_nDataLength = nOldLen;
   pOldData->Release();
   return m_pData->m_String;
 }
 FX_STRSIZE CFX_ByteString::Delete(FX_STRSIZE nIndex, FX_STRSIZE nCount) {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return 0;
   }
   if (nIndex < 0) {
     nIndex = 0;
   }
   FX_STRSIZE nOldLength = m_pData->m_nDataLength;
   if (nCount > 0 && nIndex < nOldLength) {
     FX_STRSIZE mLength = nIndex + nCount;
     if (mLength >= nOldLength) {
       m_pData->m_nDataLength = nIndex;
       return m_pData->m_nDataLength;
     }
     CopyBeforeWrite();
     int nBytesToCopy = nOldLength - mLength + 1;
     FXSYS_memmove(m_pData->m_String + nIndex, m_pData->m_String + mLength,
                   nBytesToCopy);
     m_pData->m_nDataLength = nOldLength - nCount;
   }
   return m_pData->m_nDataLength;
 }
 void CFX_ByteString::ConcatInPlace(FX_STRSIZE nSrcLen,
                                    const FX_CHAR* lpszSrcData) {
-  if (nSrcLen == 0 || lpszSrcData == NULL) {
+  if (nSrcLen == 0 || !lpszSrcData) {
     return;
   }
-  if (m_pData == NULL) {
+  if (!m_pData) {
     m_pData = StringData::Create(nSrcLen);
     if (!m_pData) {
       return;
     }
     FXSYS_memcpy(m_pData->m_String, lpszSrcData, nSrcLen);
     return;
   }
   if (m_pData->m_nRefs > 1 ||
       m_pData->m_nDataLength + nSrcLen > m_pData->m_nAllocLength) {
     ConcatCopy(m_pData->m_nDataLength, m_pData->m_String, nSrcLen, lpszSrcData);
@@ skipping 15 matching lines @@
   // Don't release until done copying, might be one of the arguments.
   StringData* pOldData = m_pData;
   m_pData = StringData::Create(nNewLen);
   if (m_pData) {
     memcpy(m_pData->m_String, lpszSrc1Data, nSrc1Len);
     memcpy(m_pData->m_String + nSrc1Len, lpszSrc2Data, nSrc2Len);
   }
   pOldData->Release();
 }
 CFX_ByteString CFX_ByteString::Mid(FX_STRSIZE nFirst) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return CFX_ByteString();
   }
   return Mid(nFirst, m_pData->m_nDataLength - nFirst);
 }
 CFX_ByteString CFX_ByteString::Mid(FX_STRSIZE nFirst, FX_STRSIZE nCount) const {
   if (nFirst < 0) {
     nFirst = 0;
   }
   if (nCount < 0) {
     nCount = 0;
@@ skipping 12 matching lines @@
   return dest;
 }
 void CFX_ByteString::AllocCopy(CFX_ByteString& dest,
                                FX_STRSIZE nCopyLen,
                                FX_STRSIZE nCopyIndex) const {
   // |FX_STRSIZE| is currently typedef'd as in |int|. TODO(palmer): It
   // should be a |size_t|, or at least unsigned.
   if (nCopyLen == 0 || nCopyLen < 0) {
     return;
   }
-  ASSERT(dest.m_pData == NULL);
+  ASSERT(!dest.m_pData);
   dest.m_pData = StringData::Create(nCopyLen);
   if (dest.m_pData) {
     FXSYS_memcpy(dest.m_pData->m_String, m_pData->m_String + nCopyIndex,
                  nCopyLen);
   }
 }
 #define FORCE_ANSI 0x10000
 #define FORCE_UNICODE 0x20000
 #define FORCE_INT64 0x40000
 void CFX_ByteString::FormatV(const FX_CHAR* lpszFormat, va_list argList) {
@@ skipping 84 matching lines @@
         nItemLen = 2;
         va_arg(argList, int);
         break;
       case 'c' | FORCE_UNICODE:
       case 'C' | FORCE_UNICODE:
         nItemLen = 2;
         va_arg(argList, int);
         break;
       case 's': {
         const FX_CHAR* pstrNextArg = va_arg(argList, const FX_CHAR*);
-        if (pstrNextArg == NULL) {
-          nItemLen = 6;
-        } else {
+        if (pstrNextArg) {
           nItemLen = FXSYS_strlen(pstrNextArg);
           if (nItemLen < 1) {
             nItemLen = 1;
           }
+        } else {
+          nItemLen = 6;
         }
       } break;
       case 'S': {
         FX_WCHAR* pstrNextArg = va_arg(argList, FX_WCHAR*);
-        if (pstrNextArg == NULL) {
-          nItemLen = 6;
-        } else {
+        if (pstrNextArg) {
           nItemLen = FXSYS_wcslen(pstrNextArg);
           if (nItemLen < 1) {
             nItemLen = 1;
           }
+        } else {
+          nItemLen = 6;
         }
       } break;
       case 's' | FORCE_ANSI:
       case 'S' | FORCE_ANSI: {
         const FX_CHAR* pstrNextArg = va_arg(argList, const FX_CHAR*);
-        if (pstrNextArg == NULL) {
-          nItemLen = 6;
-        } else {
+        if (pstrNextArg) {
           nItemLen = FXSYS_strlen(pstrNextArg);
           if (nItemLen < 1) {
             nItemLen = 1;
           }
+        } else {
+          nItemLen = 6;
         }
       } break;
       case 's' | FORCE_UNICODE:
       case 'S' | FORCE_UNICODE: {
         FX_WCHAR* pstrNextArg = va_arg(argList, FX_WCHAR*);
-        if (pstrNextArg == NULL) {
-          nItemLen = 6;
-        } else {
+        if (pstrNextArg) {
           nItemLen = FXSYS_wcslen(pstrNextArg);
           if (nItemLen < 1) {
             nItemLen = 1;
           }
+        } else {
+          nItemLen = 6;
         }
       } break;
     }
     if (nItemLen != 0) {
       if (nPrecision != 0 && nItemLen > nPrecision) {
         nItemLen = nPrecision;
       }
       if (nItemLen < nWidth) {
         nItemLen = nWidth;
       }
@@ skipping 71 matching lines @@
 FX_STRSIZE CFX_ByteString::Insert(FX_STRSIZE nIndex, FX_CHAR ch) {
   CopyBeforeWrite();
   if (nIndex < 0) {
     nIndex = 0;
   }
   FX_STRSIZE nNewLength = m_pData ? m_pData->m_nDataLength : 0;
   if (nIndex > nNewLength) {
     nIndex = nNewLength;
   }
   nNewLength++;
-  if (m_pData == NULL || m_pData->m_nAllocLength < nNewLength) {
+  if (!m_pData || m_pData->m_nAllocLength < nNewLength) {
     StringData* pOldData = m_pData;
     const FX_CHAR* pstr = m_pData->m_String;
     m_pData = StringData::Create(nNewLength);
     if (!m_pData) {
       return 0;
     }
     if (pOldData) {
       FXSYS_memmove(m_pData->m_String, pstr, (pOldData->m_nDataLength + 1));
       pOldData->Release();
     } else {
       m_pData->m_String[0] = 0;
     }
   }
   FXSYS_memmove(m_pData->m_String + nIndex + 1, m_pData->m_String + nIndex,
                 (nNewLength - nIndex));
   m_pData->m_String[nIndex] = ch;
   m_pData->m_nDataLength = nNewLength;
   return nNewLength;
 }
 CFX_ByteString CFX_ByteString::Right(FX_STRSIZE nCount) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return CFX_ByteString();
   }
   if (nCount < 0) {
     nCount = 0;
   }
   if (nCount >= m_pData->m_nDataLength) {
     return *this;
   }
   CFX_ByteString dest;
   AllocCopy(dest, nCount, m_pData->m_nDataLength - nCount);
   return dest;
 }
 CFX_ByteString CFX_ByteString::Left(FX_STRSIZE nCount) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return CFX_ByteString();
   }
   if (nCount < 0) {
     nCount = 0;
   }
   if (nCount >= m_pData->m_nDataLength) {
     return *this;
   }
   CFX_ByteString dest;
   AllocCopy(dest, nCount, 0);
   return dest;
 }
 FX_STRSIZE CFX_ByteString::Find(FX_CHAR ch, FX_STRSIZE nStart) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return -1;
   }
   FX_STRSIZE nLength = m_pData->m_nDataLength;
   if (nStart >= nLength) {
     return -1;
   }
   const FX_CHAR* lpsz = FXSYS_strchr(m_pData->m_String + nStart, ch);
-  return (lpsz == NULL) ? -1 : (int)(lpsz - m_pData->m_String);
+  return lpsz ? (int)(lpsz - m_pData->m_String) : -1;
 }
 FX_STRSIZE CFX_ByteString::ReverseFind(FX_CHAR ch) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return -1;
   }
   FX_STRSIZE nLength = m_pData->m_nDataLength;
   while (nLength) {
     if (m_pData->m_String[nLength - 1] == ch) {
       return nLength - 1;
     }
     nLength--;
   }
   return -1;
@@ skipping 16 matching lines @@
       if (i == len2) {
         return str1;
       }
     }
     str1++;
   }
   return NULL;
 }
 FX_STRSIZE CFX_ByteString::Find(const CFX_ByteStringC& lpszSub,
                                 FX_STRSIZE nStart) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return -1;
   }
   FX_STRSIZE nLength = m_pData->m_nDataLength;
   if (nStart > nLength) {
     return -1;
   }
   const FX_CHAR* lpsz =
       FX_strstr(m_pData->m_String + nStart, m_pData->m_nDataLength - nStart,
                 lpszSub.GetCStr(), lpszSub.GetLength());
-  return (lpsz == NULL) ? -1 : (int)(lpsz - m_pData->m_String);
+  return lpsz ? (int)(lpsz - m_pData->m_String) : -1;
 }
 void CFX_ByteString::MakeLower() {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return;
   }
   CopyBeforeWrite();
   if (GetLength() < 1) {
     return;
   }
   FXSYS_strlwr(m_pData->m_String);
 }
 void CFX_ByteString::MakeUpper() {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return;
   }
   CopyBeforeWrite();
   if (GetLength() < 1) {
     return;
   }
   FXSYS_strupr(m_pData->m_String);
 }
 FX_STRSIZE CFX_ByteString::Remove(FX_CHAR chRemove) {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return 0;
   }
   CopyBeforeWrite();
   if (GetLength() < 1) {
     return 0;
   }
   FX_CHAR* pstrSource = m_pData->m_String;
   FX_CHAR* pstrDest = m_pData->m_String;
   FX_CHAR* pstrEnd = m_pData->m_String + m_pData->m_nDataLength;
   while (pstrSource < pstrEnd) {
     if (*pstrSource != chRemove) {
       *pstrDest = *pstrSource;
       pstrDest++;
     }
     pstrSource++;
   }
   *pstrDest = 0;
   FX_STRSIZE nCount = (FX_STRSIZE)(pstrSource - pstrDest);
   m_pData->m_nDataLength -= nCount;
   return nCount;
 }
 FX_STRSIZE CFX_ByteString::Replace(const CFX_ByteStringC& lpszOld,
                                    const CFX_ByteStringC& lpszNew) {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return 0;
   }
   if (lpszOld.IsEmpty()) {
     return 0;
   }
   FX_STRSIZE nSourceLen = lpszOld.GetLength();
   FX_STRSIZE nReplacementLen = lpszNew.GetLength();
   FX_STRSIZE nCount = 0;
   const FX_CHAR* pStart = m_pData->m_String;
   FX_CHAR* pEnd = m_pData->m_String + m_pData->m_nDataLength;
   while (1) {
     const FX_CHAR* pTarget = FX_strstr(pStart, (FX_STRSIZE)(pEnd - pStart),
                                        lpszOld.GetCStr(), nSourceLen);
-    if (pTarget == NULL) {
+    if (!pTarget) {
       break;
     }
     nCount++;
     pStart = pTarget + nSourceLen;
   }
   if (nCount == 0) {
     return 0;
   }
   FX_STRSIZE nNewLength =
       m_pData->m_nDataLength + (nReplacementLen - nSourceLen) * nCount;
@@ skipping 15 matching lines @@
     FXSYS_memcpy(pDest, lpszNew.GetCStr(), lpszNew.GetLength());
     pDest += lpszNew.GetLength();
     pStart = pTarget + nSourceLen;
   }
   FXSYS_memcpy(pDest, pStart, pEnd - pStart);
   m_pData->Release();
   m_pData = pNewData;
   return nCount;
 }
 void CFX_ByteString::SetAt(FX_STRSIZE nIndex, FX_CHAR ch) {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return;
   }
   FXSYS_assert(nIndex >= 0);
   FXSYS_assert(nIndex < m_pData->m_nDataLength);
   CopyBeforeWrite();
   m_pData->m_String[nIndex] = ch;
 }
 CFX_WideString CFX_ByteString::UTF8Decode() const {
   CFX_UTF8Decoder decoder;
   for (FX_STRSIZE i = 0; i < GetLength(); i++) {
     decoder.Input((uint8_t)m_pData->m_String[i]);
   }
   return decoder.GetResult();
 }
 CFX_ByteString CFX_ByteString::FromUnicode(const FX_WCHAR* str,
                                            FX_STRSIZE len) {
   if (len < 0) {
     len = FXSYS_wcslen(str);
   }
   CFX_ByteString bstr;
   bstr.ConvertFrom(CFX_WideString(str, len));
   return bstr;
 }
 CFX_ByteString CFX_ByteString::FromUnicode(const CFX_WideString& str) {
   return FromUnicode(str.c_str(), str.GetLength());
 }
 void CFX_ByteString::ConvertFrom(const CFX_WideString& str,
                                  CFX_CharMap* pCharMap) {
-  if (pCharMap == NULL) {
+  if (!pCharMap) {
     pCharMap = CFX_CharMap::GetDefaultMapper();
   }
   *this = (*pCharMap->m_GetByteString)(pCharMap, str);
 }
 int CFX_ByteString::Compare(const CFX_ByteStringC& str) const {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return str.IsEmpty() ? 0 : -1;
   }
   int this_len = m_pData->m_nDataLength;
   int that_len = str.GetLength();
   int min_len = this_len < that_len ? this_len : that_len;
   for (int i = 0; i < min_len; i++) {
     if ((uint8_t)m_pData->m_String[i] < str.GetAt(i)) {
       return -1;
     }
     if ((uint8_t)m_pData->m_String[i] > str.GetAt(i)) {
       return 1;
     }
   }
   if (this_len < that_len) {
     return -1;
   }
   if (this_len > that_len) {
     return 1;
   }
   return 0;
 }
 void CFX_ByteString::TrimRight(const CFX_ByteStringC& lpszTargets) {
-  if (m_pData == NULL || lpszTargets.IsEmpty()) {
+  if (!m_pData || lpszTargets.IsEmpty()) {
     return;
   }
   CopyBeforeWrite();
   FX_STRSIZE pos = GetLength();
   if (pos < 1) {
     return;
   }
   while (pos) {
     FX_STRSIZE i = 0;
     while (i < lpszTargets.GetLength() &&
@@ skipping 10 matching lines @@
     m_pData->m_nDataLength = pos;
   }
 }
 void CFX_ByteString::TrimRight(FX_CHAR chTarget) {
   TrimRight(CFX_ByteStringC(chTarget));
 }
 void CFX_ByteString::TrimRight() {
   TrimRight("\x09\x0a\x0b\x0c\x0d\x20");
 }
 void CFX_ByteString::TrimLeft(const CFX_ByteStringC& lpszTargets) {
-  if (m_pData == NULL) {
+  if (!m_pData) {
     return;
   }
   if (lpszTargets.IsEmpty()) {
     return;
   }
   CopyBeforeWrite();
   FX_STRSIZE len = GetLength();
   if (len < 1) {
     return;
   }
@@ skipping 89 matching lines @@
     fraction %= scale;
     scale /= 10;
   }
   return buf_size;
 }
 CFX_ByteString CFX_ByteString::FormatFloat(FX_FLOAT d, int precision) {
   FX_CHAR buf[32];
   FX_STRSIZE len = FX_ftoa(d, buf);
   return CFX_ByteString(buf, len);
 }