Index: xfa/fxfa/app/xfa_ffdocview.cpp |
diff --git a/xfa/fxfa/app/xfa_ffdocview.cpp b/xfa/fxfa/app/xfa_ffdocview.cpp |
index 6d38cf589c745e90715fd8cefef6d69b7fee5c20..7c7136b1ca8eefbaedf458e04a5e35d604faeb63 100644 |
--- a/xfa/fxfa/app/xfa_ffdocview.cpp |
+++ b/xfa/fxfa/app/xfa_ffdocview.cpp |
@@ -199,7 +199,7 @@ int32_t CXFA_FFDocView::CountPageViews() { |
} |
CXFA_FFPageView* CXFA_FFDocView::GetPageView(int32_t nIndex) { |
if (!m_pXFADocLayout) { |
- return NULL; |
+ return nullptr; |
} |
return static_cast<CXFA_FFPageView*>(m_pXFADocLayout->GetPage(nIndex)); |
} |
@@ -224,7 +224,7 @@ FX_BOOL CXFA_FFDocView::ResetSingleWidgetAccData(CXFA_WidgetAcc* pWidgetAcc) { |
void CXFA_FFDocView::ResetWidgetData(CXFA_WidgetAcc* pWidgetAcc) { |
m_bLayoutEvent = TRUE; |
FX_BOOL bChanged = FALSE; |
- CXFA_Node* pFormNode = NULL; |
+ CXFA_Node* pFormNode = nullptr; |
if (pWidgetAcc) { |
bChanged = ResetSingleWidgetAccData(pWidgetAcc); |
pFormNode = pWidgetAcc->GetNode(); |
@@ -315,10 +315,7 @@ CXFA_FFWidgetHandler* CXFA_FFDocView::GetWidgetHandler() { |
CXFA_WidgetAccIterator* CXFA_FFDocView::CreateWidgetAccIterator( |
XFA_WIDGETORDER eOrder) { |
CXFA_Node* pFormRoot = GetRootSubform(); |
- if (!pFormRoot) { |
- return NULL; |
- } |
- return new CXFA_WidgetAccIterator(this, pFormRoot); |
+ return pFormRoot ? new CXFA_WidgetAccIterator(this, pFormRoot) : nullptr; |
} |
CXFA_FFWidget* CXFA_FFDocView::GetFocusWidget() { |
return m_pFocusWidget; |
@@ -326,11 +323,11 @@ CXFA_FFWidget* CXFA_FFDocView::GetFocusWidget() { |
void CXFA_FFDocView::KillFocus() { |
if (m_pFocusWidget && |
(m_pFocusWidget->GetStatus() & XFA_WidgetStatus_Focused)) { |
- (m_pFocusWidget)->OnKillFocus(NULL); |
+ (m_pFocusWidget)->OnKillFocus(nullptr); |
} |
- m_pFocusAcc = NULL; |
- m_pFocusWidget = NULL; |
- m_pOldFocusWidget = NULL; |
+ m_pFocusAcc = nullptr; |
+ m_pFocusWidget = nullptr; |
+ m_pOldFocusWidget = nullptr; |
} |
FX_BOOL CXFA_FFDocView::SetFocus(CXFA_FFWidget* hWidget) { |
CXFA_FFWidget* pNewFocus = hWidget; |
@@ -358,10 +355,10 @@ FX_BOOL CXFA_FFDocView::SetFocus(CXFA_FFWidget* hWidget) { |
} |
pNewFocus = m_pOldFocusWidget; |
if (m_pListFocusWidget && pNewFocus == m_pListFocusWidget) { |
- m_pFocusAcc = NULL; |
- m_pFocusWidget = NULL; |
- m_pListFocusWidget = NULL; |
- m_pOldFocusWidget = NULL; |
+ m_pFocusAcc = nullptr; |
+ m_pFocusWidget = nullptr; |
+ m_pListFocusWidget = nullptr; |
+ m_pOldFocusWidget = nullptr; |
return FALSE; |
} |
if (pNewFocus && (pNewFocus->GetStatus() & XFA_WidgetStatus_Visible)) { |
@@ -370,7 +367,7 @@ FX_BOOL CXFA_FFDocView::SetFocus(CXFA_FFWidget* hWidget) { |
} |
pNewFocus->OnSetFocus(m_pFocusWidget); |
} |
- m_pFocusAcc = pNewFocus ? pNewFocus->GetDataAcc() : NULL; |
+ m_pFocusAcc = pNewFocus ? pNewFocus->GetDataAcc() : nullptr; |
m_pFocusWidget = pNewFocus; |
m_pOldFocusWidget = m_pFocusWidget; |
return TRUE; |
@@ -380,7 +377,7 @@ CXFA_WidgetAcc* CXFA_FFDocView::GetFocusWidgetAcc() { |
} |
void CXFA_FFDocView::SetFocusWidgetAcc(CXFA_WidgetAcc* pWidgetAcc) { |
CXFA_FFWidget* pNewFocus = |
- pWidgetAcc ? pWidgetAcc->GetNextWidget(NULL) : NULL; |
+ pWidgetAcc ? pWidgetAcc->GetNextWidget(nullptr) : nullptr; |
if (SetFocus(pNewFocus)) { |
m_pFocusAcc = pWidgetAcc; |
if (m_iStatus == XFA_DOCVIEW_LAYOUTSTATUS_End) { |
@@ -390,9 +387,9 @@ void CXFA_FFDocView::SetFocusWidgetAcc(CXFA_WidgetAcc* pWidgetAcc) { |
} |
void CXFA_FFDocView::DeleteLayoutItem(CXFA_FFWidget* pWidget) { |
if (m_pFocusAcc == pWidget->GetDataAcc()) { |
- m_pFocusAcc = NULL; |
- m_pFocusWidget = NULL; |
- m_pOldFocusWidget = NULL; |
+ m_pFocusAcc = nullptr; |
+ m_pFocusWidget = nullptr; |
+ m_pOldFocusWidget = nullptr; |
} |
} |
static int32_t XFA_ProcessEvent(CXFA_FFDocView* pDocView, |
@@ -447,7 +444,7 @@ int32_t CXFA_FFDocView::ExecEventActivityByDeepFirst(CXFA_Node* pFormNode, |
return iRet; |
} |
CXFA_WidgetAcc* pWidgetAcc = (CXFA_WidgetAcc*)pFormNode->GetWidgetData(); |
- if (pWidgetAcc == NULL) { |
+ if (!pWidgetAcc) { |
return iRet; |
} |
CXFA_EventParam eParam; |
@@ -470,7 +467,7 @@ int32_t CXFA_FFDocView::ExecEventActivityByDeepFirst(CXFA_Node* pFormNode, |
} |
} |
CXFA_WidgetAcc* pWidgetAcc = (CXFA_WidgetAcc*)pFormNode->GetWidgetData(); |
- if (pWidgetAcc == NULL) { |
+ if (!pWidgetAcc) { |
return iRet; |
} |
CXFA_EventParam eParam; |
@@ -770,11 +767,7 @@ void CXFA_FFDocView::RunBindItems() { |
wsValue = refNode->GetContent(); |
} else { |
CXFA_Node* nodeValue = refNode->GetFirstChildByName(uValueHash); |
- if (nodeValue == NULL) { |
- wsValue = refNode->GetContent(); |
- } else { |
- wsValue = nodeValue->GetContent(); |
- } |
+ wsValue = nodeValue ? nodeValue->GetContent() : refNode->GetContent(); |
} |
if (!bUseValue) { |
if (bLabelUseContent) { |
@@ -802,7 +795,7 @@ CXFA_Node* CXFA_FFDocView::GetRootSubform() { |
CXFA_Node* pFormPacketNode = |
ToNode(m_pDoc->GetXFADoc()->GetXFAObject(XFA_HASHCODE_Form)); |
if (!pFormPacketNode) { |
- return NULL; |
+ return nullptr; |
} |
return pFormPacketNode->GetFirstChildByClass(XFA_Element::Subform); |
} |
@@ -811,18 +804,18 @@ CXFA_WidgetAccIterator::CXFA_WidgetAccIterator(CXFA_FFDocView* pDocView, |
CXFA_Node* pTravelRoot) |
: m_ContentIterator(pTravelRoot) { |
m_pDocView = pDocView; |
- m_pCurWidgetAcc = NULL; |
+ m_pCurWidgetAcc = nullptr; |
} |
CXFA_WidgetAccIterator::~CXFA_WidgetAccIterator() {} |
void CXFA_WidgetAccIterator::Reset() { |
- m_pCurWidgetAcc = NULL; |
+ m_pCurWidgetAcc = nullptr; |
m_ContentIterator.Reset(); |
} |
CXFA_WidgetAcc* CXFA_WidgetAccIterator::MoveToFirst() { |
- return NULL; |
+ return nullptr; |
} |
CXFA_WidgetAcc* CXFA_WidgetAccIterator::MoveToLast() { |
- return NULL; |
+ return nullptr; |
} |
CXFA_WidgetAcc* CXFA_WidgetAccIterator::MoveToNext() { |
CXFA_Node* pItem = m_pCurWidgetAcc ? m_ContentIterator.MoveToNext() |
@@ -833,18 +826,18 @@ CXFA_WidgetAcc* CXFA_WidgetAccIterator::MoveToNext() { |
return m_pCurWidgetAcc; |
pItem = m_ContentIterator.MoveToNext(); |
} |
- return NULL; |
+ return nullptr; |
} |
CXFA_WidgetAcc* CXFA_WidgetAccIterator::MoveToPrevious() { |
- return NULL; |
+ return nullptr; |
} |
CXFA_WidgetAcc* CXFA_WidgetAccIterator::GetCurrentWidgetAcc() { |
- return NULL; |
+ return nullptr; |
} |
FX_BOOL CXFA_WidgetAccIterator::SetCurrentWidgetAcc(CXFA_WidgetAcc* hWidget) { |
return FALSE; |
} |
void CXFA_WidgetAccIterator::SkipTree() { |
m_ContentIterator.SkipChildrenAndMoveToNext(); |
- m_pCurWidgetAcc = NULL; |
+ m_pCurWidgetAcc = nullptr; |
} |