Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1231)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: xfa/fde/css/fde_cssstyleselector.cpp

Issue 2031873003: Get rid of NULLs in xfa/ (Closed) Base URL: https://pdfium.googlesource.com/pdfium@nullptr_fpdfsdk
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « xfa/fde/css/fde_cssstyleselector.h ('k') | xfa/fde/css/fde_cssstylesheet.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: xfa/fde/css/fde_cssstyleselector.cpp
diff --git a/xfa/fde/css/fde_cssstyleselector.cpp b/xfa/fde/css/fde_cssstyleselector.cpp
index c684d0bece6da1e8fc1fb8cc5ac4446584fcdfb5..a6846a2c67a4fc357437c0f7148c285a228a0643 100644
--- a/xfa/fde/css/fde_cssstyleselector.cpp
+++ b/xfa/fde/css/fde_cssstyleselector.cpp
@@ -28,9 +28,9 @@ int32_t CFDE_CSSCounterStyle::FindIndex(const FX_WCHAR* pszIdentifier) {
return -1;
}
void CFDE_CSSCounterStyle::DoUpdateIndex(IFDE_CSSValueList* pList) {
- if (pList == NULL) {
+ if (!pList)
return;
- }
+
int32_t iCount = pList->CountValues();
FX_FLOAT fDefValue = 1.0;
FX_BOOL bDefIncrement = TRUE;
@@ -84,10 +84,10 @@ void CFDE_CSSCounterStyle::UpdateIndex() {
FDE_CSSRuleData::FDE_CSSRuleData(CFDE_CSSSelector* pSel,
CFDE_CSSDeclaration* pDecl,
uint32_t dwPos)
- : pSelector(pSel), pDeclaration(pDecl), dwPriority(dwPos), pNext(NULL) {
+ : pSelector(pSel), pDeclaration(pDecl), dwPriority(dwPos), pNext(nullptr) {
static const uint32_t s_Specific[5] = {0x00010000, 0x00010000, 0x00100000,
0x00100000, 0x01000000};
- for (; pSel != NULL; pSel = pSel->GetNextSelector()) {
+ for (; pSel; pSel = pSel->GetNextSelector()) {
FDE_CSSSELECTORTYPE eType = pSel->GetType();
if (eType > FDE_CSSSELECTORTYPE_Descendant ||
pSel->GetNameHash() != FDE_CSSUNIVERSALHASH) {
@@ -99,8 +99,8 @@ void CFDE_CSSRuleCollection::Clear() {
m_IDRules.RemoveAll();
m_TagRules.RemoveAll();
m_ClassRules.RemoveAll();
- m_pUniversalRules = NULL;
- m_pStaticStore = NULL;
+ m_pUniversalRules = nullptr;
+ m_pStaticStore = nullptr;
m_iSelectors = 0;
}
void CFDE_CSSRuleCollection::AddRulesFrom(const CFDE_CSSStyleSheetArray& sheets,
@@ -182,7 +182,7 @@ void CFDE_CSSRuleCollection::AddRuleTo(CFX_MapPtrToPtr& map,
CFDE_CSSDeclaration* pDecl) {
void* pKey = (void*)(uintptr_t)dwKey;
FDE_CSSRuleData* pData = NewRuleData(pSel, pDecl);
- FDE_CSSRuleData* pList = NULL;
+ FDE_CSSRuleData* pList = nullptr;
if (!map.Lookup(pKey, (void*&)pList)) {
map.SetAt(pKey, pData);
} else if (AddRuleTo(pList, pData)) {
@@ -210,12 +210,12 @@ FDE_CSSRuleData* CFDE_CSSRuleCollection::NewRuleData(
}
CFDE_CSSStyleSelector::CFDE_CSSStyleSelector()
- : m_pFontMgr(NULL),
+ : m_pFontMgr(nullptr),
m_fDefFontSize(12.0f),
- m_pRuleDataStore(NULL),
- m_pInlineStyleStore(NULL),
- m_pFixedStyleStore(NULL),
- m_pAccelerator(NULL) {
+ m_pRuleDataStore(nullptr),
+ m_pInlineStyleStore(nullptr),
+ m_pFixedStyleStore(nullptr),
+ m_pAccelerator(nullptr) {
m_ePriorities[FDE_CSSSTYLESHEETPRIORITY_High] = FDE_CSSSTYLESHEETGROUP_Author;
m_ePriorities[FDE_CSSSTYLESHEETPRIORITY_Mid] = FDE_CSSSTYLESHEETGROUP_User;
m_ePriorities[FDE_CSSSTYLESHEETPRIORITY_Low] =
@@ -266,9 +266,8 @@ FX_BOOL CFDE_CSSStyleSelector::SetStyleSheet(FDE_CSSSTYLESHEETGROUP eType,
ASSERT(eType < FDE_CSSSTYLESHEETGROUP_MAX);
CFDE_CSSStyleSheetArray& dest = m_SheetGroups[eType];
dest.RemoveAt(0, dest.GetSize());
- if (pSheet != NULL) {
+ if (pSheet)
dest.Add(pSheet);
- }
return TRUE;
}
FX_BOOL CFDE_CSSStyleSelector::SetStyleSheets(
@@ -276,11 +275,10 @@ FX_BOOL CFDE_CSSStyleSelector::SetStyleSheets(
const CFDE_CSSStyleSheetArray* pArray) {
ASSERT(eType < FDE_CSSSTYLESHEETGROUP_MAX);
CFDE_CSSStyleSheetArray& dest = m_SheetGroups[eType];
- if (pArray == NULL) {
- dest.RemoveAt(0, dest.GetSize());
- } else {
+ if (pArray)
dest.Copy(*pArray);
- }
+ else
+ dest.RemoveAt(0, dest.GetSize());
return TRUE;
}
void CFDE_CSSStyleSelector::SetStylePriority(
@@ -362,11 +360,11 @@ FX_BOOL CFDE_CSSStyleSelector::MatchSelector(FDE_CSSTagCache* pCache,
CFDE_CSSSelector* pSel,
FDE_CSSPERSUDO ePersudoType) {
uint32_t dwHash;
- while (pSel != NULL && pCache != NULL) {
+ while (pSel && pCache) {
switch (pSel->GetType()) {
case FDE_CSSSELECTORTYPE_Descendant:
dwHash = pSel->GetNameHash();
- while ((pCache = pCache->GetParent()) != NULL) {
+ while ((pCache = pCache->GetParent()) != nullptr) {
if (dwHash != FDE_CSSUNIVERSALHASH && dwHash != pCache->HashTag()) {
continue;
}
@@ -405,7 +403,7 @@ FX_BOOL CFDE_CSSStyleSelector::MatchSelector(FDE_CSSTagCache* pCache,
}
pSel = pSel->GetNextSelector();
}
- return pSel == NULL && pCache != NULL;
+ return !pSel && pCache;
}
void CFDE_CSSStyleSelector::ComputeStyle(
@@ -480,16 +478,16 @@ void CFDE_CSSStyleSelector::ApplyDeclarations(
FX_BOOL bImportant;
int32_t i;
if (bPriority) {
- IFDE_CSSValue *pLastest = NULL, *pImportant = NULL;
+ IFDE_CSSValue *pLastest = nullptr, *pImportant = nullptr;
for (i = 0; i < iDeclCount; ++i) {
pVal = ppDeclArray[i]->GetProperty(FDE_CSSPROPERTY_FontSize, bImportant);
- if (pVal == NULL) {
+ if (!pVal)
continue;
- } else if (bImportant) {
+
+ if (bImportant)
pImportant = pVal;
- } else {
+ else
pLastest = pVal;
- }
}
if (pImportant) {
ApplyProperty(FDE_CSSPROPERTY_FontSize, pImportant, pComputedStyle);
@@ -504,7 +502,7 @@ void CFDE_CSSStyleSelector::ApplyDeclarations(
for (i = 0; i < iDeclCount; ++i) {
pDecl = ppDeclArray[i];
pos = pDecl->GetStartPosition();
- while (pos != NULL) {
+ while (pos) {
pDecl->GetNextProperty(pos, eProp, pVal, bImportant);
if (eProp == FDE_CSSPROPERTY_FontSize) {
continue;
@@ -520,7 +518,7 @@ void CFDE_CSSStyleSelector::ApplyDeclarations(
for (i = 0; i < iDeclCount; ++i) {
pDecl = importants[i];
pos = pDecl->GetStartPosition();
- while (pos != NULL) {
+ while (pos) {
pDecl->GetNextProperty(pos, eProp, pVal, bImportant);
if (bImportant && eProp != FDE_CSSPROPERTY_FontSize) {
ApplyProperty(eProp, pVal, pComputedStyle);
@@ -955,7 +953,7 @@ void CFDE_CSSStyleSelector::ApplyProperty(
break;
case FDE_CSSPROPERTY_BackgroundImage:
if (eType == FDE_CSSPRIMITIVETYPE_Enum) {
- FDE_CSSNONINHERITS.m_pszBKGImage = NULL;
+ FDE_CSSNONINHERITS.m_pszBKGImage = nullptr;
} else if (eType == FDE_CSSPRIMITIVETYPE_URI) {
int32_t iLength;
FDE_CSSNONINHERITS.m_pszBKGImage = pPrimitive->GetString(iLength);
@@ -984,7 +982,7 @@ void CFDE_CSSStyleSelector::ApplyProperty(
break;
case FDE_CSSPROPERTY_ListStyleImage:
if (eType == FDE_CSSPRIMITIVETYPE_Enum) {
- FDE_CSSINHERITS.m_pszListStyleImage = NULL;
+ FDE_CSSINHERITS.m_pszListStyleImage = nullptr;
} else if (eType == FDE_CSSPRIMITIVETYPE_URI) {
int32_t iLength;
FDE_CSSINHERITS.m_pszListStyleImage = pPrimitive->GetString(iLength);
@@ -1041,15 +1039,13 @@ void CFDE_CSSStyleSelector::ApplyProperty(
FDE_CSSNONINHERITS.m_dwTextDecoration = ToTextDecoration(pList);
break;
case FDE_CSSPROPERTY_CounterIncrement: {
- if (FDE_CSSNONINHERITS.m_pCounterStyle == NULL) {
+ if (!FDE_CSSNONINHERITS.m_pCounterStyle)
FDE_CSSNONINHERITS.m_pCounterStyle = new CFDE_CSSCounterStyle;
- }
FDE_CSSNONINHERITS.m_pCounterStyle->SetCounterIncrementList(pList);
} break;
case FDE_CSSPROPERTY_CounterReset: {
- if (FDE_CSSNONINHERITS.m_pCounterStyle == NULL) {
+ if (!FDE_CSSNONINHERITS.m_pCounterStyle)
FDE_CSSNONINHERITS.m_pCounterStyle = new CFDE_CSSCounterStyle;
- }
FDE_CSSNONINHERITS.m_pCounterStyle->SetCounterResetList(pList);
} break;
case FDE_CSSPROPERTY_Content:
« no previous file with comments | « xfa/fde/css/fde_cssstyleselector.h ('k') | xfa/fde/css/fde_cssstylesheet.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698