Index: xfa/fxfa/parser/xfa_layout_pagemgr_new.cpp |
diff --git a/xfa/fxfa/parser/xfa_layout_pagemgr_new.cpp b/xfa/fxfa/parser/xfa_layout_pagemgr_new.cpp |
index 02bc0a846988b417019ffc66d1d8071d2091036f..4b36bc8e4ca098406767ff4466e64788df6150e8 100644 |
--- a/xfa/fxfa/parser/xfa_layout_pagemgr_new.cpp |
+++ b/xfa/fxfa/parser/xfa_layout_pagemgr_new.cpp |
@@ -140,30 +140,29 @@ CXFA_Node* ResolveBreakTarget(CXFA_Node* pPageSetRoot, |
int32_t iSpliteIndex = 0; |
FX_BOOL bTargetAllFind = TRUE; |
while (iSpliteIndex != -1) { |
- CFX_WideString wsTargetExpr; |
+ CFX_WideString wsExpr; |
int32_t iSpliteNextIndex = 0; |
if (!bTargetAllFind) { |
iSpliteNextIndex = wsTargetAll.Find(' ', iSpliteIndex); |
- wsTargetExpr = |
- wsTargetAll.Mid(iSpliteIndex, iSpliteNextIndex - iSpliteIndex); |
+ wsExpr = wsTargetAll.Mid(iSpliteIndex, iSpliteNextIndex - iSpliteIndex); |
} else { |
- wsTargetExpr = wsTargetAll; |
+ wsExpr = wsTargetAll; |
} |
- if (wsTargetExpr.IsEmpty()) |
+ if (wsExpr.IsEmpty()) |
return nullptr; |
bTargetAllFind = FALSE; |
- if (wsTargetExpr.GetAt(0) == '#') { |
+ if (wsExpr.GetAt(0) == '#') { |
CXFA_Node* pNode = pDocument->GetNodeByID( |
ToNode(pDocument->GetXFAObject(XFA_HASHCODE_Template)), |
- wsTargetExpr.Mid(1).AsStringC()); |
+ wsExpr.Mid(1).AsStringC()); |
if (pNode) |
return pNode; |
} else if (bNewExprStyle) { |
- CFX_WideString wsProcessedTarget = wsTargetExpr; |
- if (wsTargetExpr.Left(4) == FX_WSTRC(L"som(") && |
- wsTargetExpr.Right(1) == FX_WSTRC(L")")) { |
- wsProcessedTarget = wsTargetExpr.Mid(4, wsTargetExpr.GetLength() - 5); |
+ CFX_WideString wsProcessedTarget = wsExpr; |
+ if (wsExpr.Left(4) == FX_WSTRC(L"som(") && |
+ wsExpr.Right(1) == FX_WSTRC(L")")) { |
+ wsProcessedTarget = wsExpr.Mid(4, wsExpr.GetLength() - 5); |
} |
XFA_RESOLVENODE_RS rs; |
int32_t iCount = pDocument->GetScriptContext()->ResolveObjects( |
@@ -893,7 +892,6 @@ CXFA_Node* CXFA_LayoutPageMgr::BreakOverflow(CXFA_Node* pOverflowNode, |
CXFA_Node*& pLeaderTemplate, |
CXFA_Node*& pTrailerTemplate, |
FX_BOOL bCreatePage) { |
- CFX_WideStringC wsOverflowLeader, wsOverflowTrailer; |
CXFA_Node* pContainer = |
pOverflowNode->GetNodeItem(XFA_NODEITEM_Parent, |
XFA_OBJECTTYPE_ContainerNode) |
@@ -937,6 +935,8 @@ CXFA_Node* CXFA_LayoutPageMgr::BreakOverflow(CXFA_Node* pOverflowNode, |
} |
return NULL; |
} else if (pOverflowNode->GetClassID() == XFA_ELEMENT_Overflow) { |
+ CFX_WideStringC wsOverflowLeader; |
+ CFX_WideStringC wsOverflowTrailer; |
CFX_WideStringC wsOverflowTarget; |
pOverflowNode->TryCData(XFA_ATTRIBUTE_Leader, wsOverflowLeader); |
pOverflowNode->TryCData(XFA_ATTRIBUTE_Trailer, wsOverflowTrailer); |
@@ -967,8 +967,9 @@ CXFA_Node* CXFA_LayoutPageMgr::BreakOverflow(CXFA_Node* pOverflowNode, |
} |
return pOverflowNode; |
} |
- return NULL; |
+ return nullptr; |
} |
+ |
FX_BOOL CXFA_LayoutPageMgr::ProcessOverflow(CXFA_Node* pFormNode, |
CXFA_Node*& pLeaderNode, |
CXFA_Node*& pTrailerNode, |
@@ -1718,11 +1719,11 @@ void CXFA_LayoutPageMgr::MergePageSetContents() { |
pDocument, pContainerItem->m_pFormNode->GetClassID(), |
pContainerItem->m_pFormNode->GetNameHash(), pParentNode); |
CXFA_ContainerIterator sIterator(pExistingNode); |
- for (CXFA_Node* pNode = sIterator.GetCurrent(); pNode; |
- pNode = sIterator.MoveToNext()) { |
- if (pNode->GetClassID() != XFA_ELEMENT_ContentArea) { |
+ for (CXFA_Node* pIter = sIterator.GetCurrent(); pIter; |
+ pIter = sIterator.MoveToNext()) { |
+ if (pIter->GetClassID() != XFA_ELEMENT_ContentArea) { |
CXFA_LayoutItem* pLayoutItem = static_cast<CXFA_LayoutItem*>( |
- pNode->GetUserData(XFA_LAYOUTITEMKEY)); |
+ pIter->GetUserData(XFA_LAYOUTITEMKEY)); |
if (pLayoutItem) { |
pNotify->OnLayoutItemRemoving(pDocLayout, pLayoutItem); |
delete pLayoutItem; |