Get rid of NULLs in xfa/

xfa/fde/css/fde_cssstylesheet.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 "xfa/fde/css/fde_cssstylesheet.h"
 
 #include <memory>
 
@@ skipping 37 matching lines @@
 }
 
 IFDE_CSSStyleSheet* IFDE_CSSStyleSheet::LoadFromStream(
     const CFX_WideString& szUrl,
     IFX_Stream* pStream,
     uint16_t wCodePage,
     uint32_t dwMediaList) {
   CFDE_CSSStyleSheet* pStyleSheet = new CFDE_CSSStyleSheet(dwMediaList);
   if (!pStyleSheet->LoadFromStream(szUrl, pStream, wCodePage)) {
     pStyleSheet->Release();
-    pStyleSheet = NULL;
+    pStyleSheet = nullptr;
   }
   return pStyleSheet;
 }
 IFDE_CSSStyleSheet* IFDE_CSSStyleSheet::LoadFromBuffer(
     const CFX_WideString& szUrl,
     const FX_WCHAR* pBuffer,
     int32_t iBufSize,
     uint16_t wCodePage,
     uint32_t dwMediaList) {
   CFDE_CSSStyleSheet* pStyleSheet = new CFDE_CSSStyleSheet(dwMediaList);
   if (!pStyleSheet->LoadFromBuffer(szUrl, pBuffer, iBufSize, wCodePage)) {
     pStyleSheet->Release();
-    pStyleSheet = NULL;
+    pStyleSheet = nullptr;
   }
   return pStyleSheet;
 }
 CFDE_CSSStyleSheet::CFDE_CSSStyleSheet(uint32_t dwMediaList)
     : m_wCodePage(FX_CODEPAGE_UTF8),
       m_wRefCount(1),
       m_dwMediaList(dwMediaList),
-      m_pAllocator(NULL) {
+      m_pAllocator(nullptr) {
   ASSERT(m_dwMediaList > 0);
 }
 CFDE_CSSStyleSheet::~CFDE_CSSStyleSheet() {
   Reset();
 }
 void CFDE_CSSStyleSheet::Reset() {
   for (int32_t i = m_RuleArray.GetSize() - 1; i >= 0; --i) {
     IFDE_CSSRule* pRule = m_RuleArray.GetAt(i);
     switch (pRule->GetType()) {
       case FDE_CSSRULETYPE_Style:
@@ skipping 84 matching lines @@
         break;
     }
   } while (eStatus >= FDE_CSSSYNTAXSTATUS_None);
   m_Selectors.RemoveAll();
   m_StringCache.clear();
   return eStatus != FDE_CSSSYNTAXSTATUS_Error;
 }
 FDE_CSSSYNTAXSTATUS CFDE_CSSStyleSheet::LoadMediaRule(
     CFDE_CSSSyntaxParser* pSyntax) {
   uint32_t dwMediaList = 0;
-  CFDE_CSSMediaRule* pMediaRule = NULL;
+  CFDE_CSSMediaRule* pMediaRule = nullptr;
   for (;;) {
     switch (pSyntax->DoSyntaxParse()) {
       case FDE_CSSSYNTAXSTATUS_MediaType: {
         int32_t iLen;
         const FX_WCHAR* psz = pSyntax->GetCurrentString(iLen);
         FDE_LPCCSSMEDIATYPETABLE pMediaType =
             FDE_GetCSSMediaTypeByName(CFX_WideStringC(psz, iLen));
-        if (pMediaType != NULL) {
+        if (pMediaType)
           dwMediaList |= pMediaType->wValue;
-        }
       } break;
       case FDE_CSSSYNTAXSTATUS_StyleRule:
-        if (pMediaRule == NULL) {
-          SkipRuleSet(pSyntax);
-        } else {
+        if (pMediaRule) {
           FDE_CSSSYNTAXSTATUS eStatus =
               LoadStyleRule(pSyntax, pMediaRule->GetArray());
           if (eStatus < FDE_CSSSYNTAXSTATUS_None) {
             return eStatus;
           }
+        } else {
+          SkipRuleSet(pSyntax);
         }
         break;
       case FDE_CSSSYNTAXSTATUS_DeclOpen:
-        if ((dwMediaList & m_dwMediaList) > 0 && pMediaRule == NULL) {
+        if ((dwMediaList & m_dwMediaList) > 0 && !pMediaRule) {
           pMediaRule =
               FXTARGET_NewWith(m_pAllocator) CFDE_CSSMediaRule(dwMediaList);
           m_RuleArray.Add(pMediaRule);
         }
         break;
       case FDE_CSSSYNTAXSTATUS_DeclClose:
         return FDE_CSSSYNTAXSTATUS_None;
         FDE_CSSSWITCHDEFAULTS();
     }
   }
 }
 FDE_CSSSYNTAXSTATUS CFDE_CSSStyleSheet::LoadStyleRule(
     CFDE_CSSSyntaxParser* pSyntax,
     CFDE_CSSRuleArray& ruleArray) {
   m_Selectors.RemoveAt(0, m_Selectors.GetSize());
-  CFDE_CSSStyleRule* pStyleRule = NULL;
-  const FX_WCHAR* pszValue = NULL;
+  CFDE_CSSStyleRule* pStyleRule = nullptr;
+  const FX_WCHAR* pszValue = nullptr;
   int32_t iValueLen = 0;
   FDE_CSSPROPERTYARGS propertyArgs;
   propertyArgs.pStaticStore = m_pAllocator;
   propertyArgs.pStringCache = &m_StringCache;
-  propertyArgs.pProperty = NULL;
+  propertyArgs.pProperty = nullptr;
   CFX_WideString wsName;
   for (;;) {
     switch (pSyntax->DoSyntaxParse()) {
       case FDE_CSSSYNTAXSTATUS_Selector: {
         pszValue = pSyntax->GetCurrentString(iValueLen);
         CFDE_CSSSelector* pSelector =
             CFDE_CSSSelector::FromString(m_pAllocator, pszValue, iValueLen);
-        if (pSelector != NULL) {
+        if (pSelector)
           m_Selectors.Add(pSelector);
-        }
       } break;
       case FDE_CSSSYNTAXSTATUS_PropertyName:
         pszValue = pSyntax->GetCurrentString(iValueLen);
         propertyArgs.pProperty =
             FDE_GetCSSPropertyByName(CFX_WideStringC(pszValue, iValueLen));
-        if (propertyArgs.pProperty == NULL) {
+        if (!propertyArgs.pProperty)
           wsName = CFX_WideStringC(pszValue, iValueLen);
-        }
         break;
       case FDE_CSSSYNTAXSTATUS_PropertyValue:
-        if (propertyArgs.pProperty != NULL) {
+        if (propertyArgs.pProperty) {
           pszValue = pSyntax->GetCurrentString(iValueLen);
           if (iValueLen > 0) {
             pStyleRule->GetDeclImp().AddProperty(&propertyArgs, pszValue,
                                                  iValueLen);
           }
         } else if (iValueLen > 0) {
           pszValue = pSyntax->GetCurrentString(iValueLen);
           if (iValueLen > 0) {
             pStyleRule->GetDeclImp().AddProperty(&propertyArgs, wsName.c_str(),
                                                  wsName.GetLength(), pszValue,
                                                  iValueLen);
           }
         }
         break;
       case FDE_CSSSYNTAXSTATUS_DeclOpen:
-        if (pStyleRule == NULL && m_Selectors.GetSize() > 0) {
+        if (!pStyleRule && m_Selectors.GetSize() > 0) {
           pStyleRule = FXTARGET_NewWith(m_pAllocator) CFDE_CSSStyleRule;
           pStyleRule->SetSelector(m_pAllocator, m_Selectors);
           ruleArray.Add(pStyleRule);
         } else {
           SkipRuleSet(pSyntax);
           return FDE_CSSSYNTAXSTATUS_None;
         }
         break;
       case FDE_CSSSYNTAXSTATUS_DeclClose:
-        if (pStyleRule != NULL &&
-            pStyleRule->GetDeclImp().GetStartPosition() == NULL) {
+        if (pStyleRule && !pStyleRule->GetDeclImp().GetStartPosition()) {
           pStyleRule->~CFDE_CSSStyleRule();
           ruleArray.RemoveLast(1);
         }
         return FDE_CSSSYNTAXSTATUS_None;
         FDE_CSSSWITCHDEFAULTS();
     }
   }
 }
 FDE_CSSSYNTAXSTATUS CFDE_CSSStyleSheet::LoadFontFaceRule(
     CFDE_CSSSyntaxParser* pSyntax,
     CFDE_CSSRuleArray& ruleArray) {
-  CFDE_CSSFontFaceRule* pFontFaceRule = NULL;
-  const FX_WCHAR* pszValue = NULL;
+  CFDE_CSSFontFaceRule* pFontFaceRule = nullptr;
+  const FX_WCHAR* pszValue = nullptr;
   int32_t iValueLen = 0;
   FDE_CSSPROPERTYARGS propertyArgs;
   propertyArgs.pStaticStore = m_pAllocator;
   propertyArgs.pStringCache = &m_StringCache;
-  propertyArgs.pProperty = NULL;
+  propertyArgs.pProperty = nullptr;
   for (;;) {
     switch (pSyntax->DoSyntaxParse()) {
       case FDE_CSSSYNTAXSTATUS_PropertyName:
         pszValue = pSyntax->GetCurrentString(iValueLen);
         propertyArgs.pProperty =
             FDE_GetCSSPropertyByName(CFX_WideStringC(pszValue, iValueLen));
         break;
       case FDE_CSSSYNTAXSTATUS_PropertyValue:
-        if (propertyArgs.pProperty != NULL) {
+        if (propertyArgs.pProperty) {
           pszValue = pSyntax->GetCurrentString(iValueLen);
           if (iValueLen > 0) {
             pFontFaceRule->GetDeclImp().AddProperty(&propertyArgs, pszValue,
                                                     iValueLen);
           }
         }
         break;
       case FDE_CSSSYNTAXSTATUS_DeclOpen:
-        if (pFontFaceRule == NULL) {
+        if (!pFontFaceRule) {
           pFontFaceRule = FXTARGET_NewWith(m_pAllocator) CFDE_CSSFontFaceRule;
           ruleArray.Add(pFontFaceRule);
         }
         break;
       case FDE_CSSSYNTAXSTATUS_DeclClose:
         return FDE_CSSSYNTAXSTATUS_None;
         FDE_CSSSWITCHDEFAULTS();
     }
   }
 }
@@ skipping 23 matching lines @@
       case FDE_CSSSYNTAXSTATUS_PropertyValue:
         break;
       case FDE_CSSSYNTAXSTATUS_DeclClose:
         return FDE_CSSSYNTAXSTATUS_None;
         FDE_CSSSWITCHDEFAULTS();
     }
   }
 }
 void CFDE_CSSStyleRule::SetSelector(IFX_MemoryAllocator* pStaticStore,
                                     const CFDE_CSSSelectorArray& list) {
-  ASSERT(m_ppSelector == NULL);
+  ASSERT(!m_ppSelector);
   m_iSelectors = list.GetSize();
   m_ppSelector = static_cast<CFDE_CSSSelector**>(
       pStaticStore->Alloc(m_iSelectors * sizeof(CFDE_CSSSelector*)));
   for (int32_t i = 0; i < m_iSelectors; ++i) {
     m_ppSelector[i] = list.GetAt(i);
   }
 }
 CFDE_CSSMediaRule::~CFDE_CSSMediaRule() {
   for (int32_t i = m_RuleArray.GetSize() - 1; i >= 0; --i) {
     IFDE_CSSRule* pRule = m_RuleArray.GetAt(i);
@@ skipping 33 matching lines @@
     } else {
       break;
     }
   }
   return psz - pStart;
 }
 CFDE_CSSSelector* CFDE_CSSSelector::FromString(
     IFX_MemoryAllocator* pStaticStore,
     const FX_WCHAR* psz,
     int32_t iLen) {
-  ASSERT(pStaticStore != NULL && psz != NULL && iLen > 0);
+  ASSERT(pStaticStore && psz && iLen > 0);
   const FX_WCHAR* pStart = psz;
   const FX_WCHAR* pEnd = psz + iLen;
   for (; psz < pEnd; ++psz) {
     switch (*psz) {
       case '>':
       case '[':
       case '+':
-        return NULL;
+        return nullptr;
     }
   }
-  CFDE_CSSSelector *pFirst = NULL, *pLast = NULL;
-  CFDE_CSSSelector *pPersudoFirst = NULL, *pPersudoLast = NULL;
+  CFDE_CSSSelector *pFirst = nullptr, *pLast = nullptr;
+  CFDE_CSSSelector *pPersudoFirst = nullptr, *pPersudoLast = nullptr;
   for (psz = pStart; psz < pEnd;) {
     FX_WCHAR wch = *psz;
     if (wch == '.' || wch == '#') {
       if (psz == pStart || psz[-1] == ' ') {
         CFDE_CSSSelector* p = FXTARGET_NewWith(pStaticStore)
             CFDE_CSSSelector(FDE_CSSSELECTORTYPE_Element, L"*", 1, true);
-        if (p == NULL) {
-          return NULL;
-        }
-        if (pFirst != NULL) {
+        if (!p)
+          return nullptr;
+
+        if (pFirst) {
           pFirst->SetType(FDE_CSSSELECTORTYPE_Descendant);
           p->SetNext(pFirst);
         }
         pFirst = pLast = p;
       }
-      ASSERT(pLast != NULL);
+      ASSERT(pLast);
       int32_t iNameLen = FDE_GetCSSNameLen(++psz, pEnd);
       if (iNameLen == 0) {
-        return NULL;
+        return nullptr;
       }
       FDE_CSSSELECTORTYPE eType =
           wch == '.' ? FDE_CSSSELECTORTYPE_Class : FDE_CSSSELECTORTYPE_ID;
       CFDE_CSSSelector* p = FXTARGET_NewWith(pStaticStore)
           CFDE_CSSSelector(eType, psz, iNameLen, false);
-      if (p == NULL) {
-        return NULL;
-      }
+      if (!p)
+        return nullptr;
+
       p->SetNext(pLast->GetNextSelector());
       pLast->SetNext(p);
       pLast = p;
       psz += iNameLen;
     } else if (FDE_IsCSSChar(wch) || wch == '*') {
       int32_t iNameLen = wch == '*' ? 1 : FDE_GetCSSNameLen(psz, pEnd);
       if (iNameLen == 0) {
-        return NULL;
+        return nullptr;
       }
       CFDE_CSSSelector* p = FXTARGET_NewWith(pStaticStore)
           CFDE_CSSSelector(FDE_CSSSELECTORTYPE_Element, psz, iNameLen, true);
-      if (p == NULL) {
-        return NULL;
-      }
-      if (pFirst == NULL) {
-        pFirst = pLast = p;
-      } else {
+      if (!p)
+        return nullptr;
+
+      if (pFirst) {
         pFirst->SetType(FDE_CSSSELECTORTYPE_Descendant);
         p->SetNext(pFirst);
-        pFirst = pLast = p;
       }
+      pFirst = p;
+      pLast = p;
       psz += iNameLen;
     } else if (wch == ':') {
       int32_t iNameLen = FDE_GetCSSPersudoLen(psz, pEnd);
       if (iNameLen == 0) {
-        return NULL;
+        return nullptr;
       }
       CFDE_CSSSelector* p = FXTARGET_NewWith(pStaticStore)
           CFDE_CSSSelector(FDE_CSSSELECTORTYPE_Persudo, psz, iNameLen, true);
-      if (p == NULL) {
-        return NULL;
-      }
-      if (pPersudoFirst == NULL) {
-        pPersudoFirst = pPersudoLast = p;
-      } else {
+      if (!p)
+        return nullptr;
+
+      if (pPersudoFirst)
         pPersudoLast->SetNext(p);
-        pPersudoLast = p;
-      }
+      else
+        pPersudoFirst = p;
+      pPersudoLast = p;
       psz += iNameLen;
     } else if (wch == ' ') {
       psz++;
     } else {
-      return NULL;
+      return nullptr;
     }
   }
-  if (pPersudoFirst == NULL) {
+  if (!pPersudoFirst)
     return pFirst;
-  } else {
-    pPersudoLast->SetNext(pFirst);
-    return pPersudoFirst;
-  }
+
+  pPersudoLast->SetNext(pFirst);
+  return pPersudoFirst;
 }