summaryrefslogtreecommitdiff
path: root/fxjs/cfxjse_resolveprocessor.cpp
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2017-12-04 18:08:58 +0000
committerChromium commit bot <commit-bot@chromium.org>2017-12-04 18:08:58 +0000
commit76e336d26e78f6c6b5cdede6bb4b94ebddf3a2b1 (patch)
tree07bc69b943a29840f947e97e26d3173130e51a91 /fxjs/cfxjse_resolveprocessor.cpp
parent0ddd5dc7969676f0c661c859512be50e379a2260 (diff)
downloadpdfium-76e336d26e78f6c6b5cdede6bb4b94ebddf3a2b1.tar.xz
Cleanup CFXJSE engine and resolve processor
Change-Id: Ibb7411a7c90f723a19fd5d41552988bc8943e4ea Reviewed-on: https://pdfium-review.googlesource.com/20330 Reviewed-by: Henrique Nakashima <hnakashima@chromium.org> Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs/cfxjse_resolveprocessor.cpp')
-rw-r--r--fxjs/cfxjse_resolveprocessor.cpp280
1 files changed, 125 insertions, 155 deletions
diff --git a/fxjs/cfxjse_resolveprocessor.cpp b/fxjs/cfxjse_resolveprocessor.cpp
index 5dba39b25d..fb7bc555c6 100644
--- a/fxjs/cfxjse_resolveprocessor.cpp
+++ b/fxjs/cfxjse_resolveprocessor.cpp
@@ -27,20 +27,20 @@ CFXJSE_ResolveProcessor::CFXJSE_ResolveProcessor()
CFXJSE_ResolveProcessor::~CFXJSE_ResolveProcessor() {}
-int32_t CFXJSE_ResolveProcessor::Resolve(CFXJSE_ResolveNodeData& rnd) {
+bool CFXJSE_ResolveProcessor::Resolve(CFXJSE_ResolveNodeData& rnd) {
if (!rnd.m_CurObject)
- return -1;
+ return false;
if (!rnd.m_CurObject->IsNode()) {
if (rnd.m_dwStyles & XFA_RESOLVENODE_Attributes) {
return ResolveForAttributeRs(rnd.m_CurObject, rnd,
rnd.m_wsName.AsStringView());
}
- return 0;
+ return false;
}
- if (rnd.m_dwStyles & XFA_RESOLVENODE_AnyChild) {
+ if (rnd.m_dwStyles & XFA_RESOLVENODE_AnyChild)
return ResolveAnyChild(rnd);
- }
+
if (rnd.m_wsName.GetLength()) {
wchar_t wch = rnd.m_wsName[0];
switch (wch) {
@@ -61,7 +61,7 @@ int32_t CFXJSE_ResolveProcessor::Resolve(CFXJSE_ResolveNodeData& rnd) {
}
if (rnd.m_uHashName == XFA_HASHCODE_This && rnd.m_nLevel == 0) {
rnd.m_Objects.push_back(rnd.m_pSC->GetThisObject());
- return 1;
+ return true;
}
if (rnd.m_CurObject->GetElementType() == XFA_Element::Xfa) {
CXFA_Object* pObjNode =
@@ -73,20 +73,20 @@ int32_t CFXJSE_ResolveProcessor::Resolve(CFXJSE_ResolveNodeData& rnd) {
} else if ((rnd.m_dwStyles & XFA_RESOLVENODE_Attributes) &&
ResolveForAttributeRs(rnd.m_CurObject, rnd,
rnd.m_wsName.AsStringView())) {
- return 1;
+ return true;
}
if (!rnd.m_Objects.empty())
FilterCondition(rnd, rnd.m_wsCondition);
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
+ return !rnd.m_Objects.empty();
}
- if (ResolveNormal(rnd) < 1 && rnd.m_uHashName == XFA_HASHCODE_Xfa)
+ if (!ResolveNormal(rnd) && rnd.m_uHashName == XFA_HASHCODE_Xfa)
rnd.m_Objects.push_back(rnd.m_pSC->GetDocument()->GetRoot());
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
+ return !rnd.m_Objects.empty();
}
-int32_t CFXJSE_ResolveProcessor::ResolveAnyChild(CFXJSE_ResolveNodeData& rnd) {
+bool CFXJSE_ResolveProcessor::ResolveAnyChild(CFXJSE_ResolveNodeData& rnd) {
WideString wsName = rnd.m_wsName;
WideString wsCondition = rnd.m_wsCondition;
CXFA_Node* findNode = nullptr;
@@ -97,13 +97,14 @@ int32_t CFXJSE_ResolveProcessor::ResolveAnyChild(CFXJSE_ResolveNodeData& rnd) {
}
findNode = m_pNodeHelper->ResolveNodes_GetOneChild(
ToNode(rnd.m_CurObject), wsName.c_str(), bClassName);
- if (!findNode) {
- return 0;
- }
+ if (!findNode)
+ return false;
+
if (wsCondition.IsEmpty()) {
rnd.m_Objects.push_back(findNode);
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
+ return !rnd.m_Objects.empty();
}
+
std::vector<CXFA_Node*> tempNodes;
for (auto* pObject : rnd.m_Objects)
tempNodes.push_back(pObject->AsNode());
@@ -111,20 +112,20 @@ int32_t CFXJSE_ResolveProcessor::ResolveAnyChild(CFXJSE_ResolveNodeData& rnd) {
bClassName);
rnd.m_Objects = std::vector<CXFA_Object*>(tempNodes.begin(), tempNodes.end());
FilterCondition(rnd, wsCondition);
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
+ return !rnd.m_Objects.empty();
}
-int32_t CFXJSE_ResolveProcessor::ResolveDollar(CFXJSE_ResolveNodeData& rnd) {
+bool CFXJSE_ResolveProcessor::ResolveDollar(CFXJSE_ResolveNodeData& rnd) {
WideString wsName = rnd.m_wsName;
WideString wsCondition = rnd.m_wsCondition;
int32_t iNameLen = wsName.GetLength();
if (iNameLen == 1) {
rnd.m_Objects.push_back(rnd.m_CurObject);
- return 1;
- }
- if (rnd.m_nLevel > 0) {
- return -1;
+ return true;
}
+ if (rnd.m_nLevel > 0)
+ return false;
+
XFA_HashCode dwNameHash = static_cast<XFA_HashCode>(FX_HashCode_GetW(
WideStringView(wsName.c_str() + 1, iNameLen - 1), false));
if (dwNameHash == XFA_HASHCODE_Xfa) {
@@ -137,18 +138,17 @@ int32_t CFXJSE_ResolveProcessor::ResolveDollar(CFXJSE_ResolveNodeData& rnd) {
if (!rnd.m_Objects.empty())
FilterCondition(rnd, wsCondition);
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
+ return !rnd.m_Objects.empty();
}
-int32_t CFXJSE_ResolveProcessor::ResolveExcalmatory(
- CFXJSE_ResolveNodeData& rnd) {
+bool CFXJSE_ResolveProcessor::ResolveExcalmatory(CFXJSE_ResolveNodeData& rnd) {
if (rnd.m_nLevel > 0)
- return 0;
+ return false;
CXFA_Node* datasets =
ToNode(rnd.m_pSC->GetDocument()->GetXFAObject(XFA_HASHCODE_Datasets));
if (!datasets)
- return 0;
+ return false;
CFXJSE_ResolveNodeData rndFind(rnd.m_pSC);
rndFind.m_CurObject = datasets;
@@ -159,18 +159,18 @@ int32_t CFXJSE_ResolveProcessor::ResolveExcalmatory(
rndFind.m_dwStyles = XFA_RESOLVENODE_Children;
rndFind.m_wsCondition = rnd.m_wsCondition;
Resolve(rndFind);
+
rnd.m_Objects.insert(rnd.m_Objects.end(), rndFind.m_Objects.begin(),
rndFind.m_Objects.end());
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
+ return !rnd.m_Objects.empty();
}
-int32_t CFXJSE_ResolveProcessor::ResolveNumberSign(
- CFXJSE_ResolveNodeData& rnd) {
+bool CFXJSE_ResolveProcessor::ResolveNumberSign(CFXJSE_ResolveNodeData& rnd) {
WideString wsName = rnd.m_wsName.Right(rnd.m_wsName.GetLength() - 1);
WideString wsCondition = rnd.m_wsCondition;
CXFA_Node* curNode = ToNode(rnd.m_CurObject);
if (ResolveForAttributeRs(curNode, rnd, wsName.AsStringView()))
- return 1;
+ return true;
CFXJSE_ResolveNodeData rndFind(rnd.m_pSC);
rndFind.m_nLevel = rnd.m_nLevel + 1;
@@ -184,7 +184,7 @@ int32_t CFXJSE_ResolveProcessor::ResolveNumberSign(
rndFind.m_CurObject = curNode;
ResolveNormal(rndFind);
if (rndFind.m_Objects.empty())
- return 0;
+ return false;
if (wsCondition.GetLength() == 0 &&
pdfium::ContainsValue(rndFind.m_Objects, curNode)) {
@@ -193,27 +193,27 @@ int32_t CFXJSE_ResolveProcessor::ResolveNumberSign(
rnd.m_Objects.insert(rnd.m_Objects.end(), rndFind.m_Objects.begin(),
rndFind.m_Objects.end());
}
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
+ return !rnd.m_Objects.empty();
}
-int32_t CFXJSE_ResolveProcessor::ResolveForAttributeRs(
+bool CFXJSE_ResolveProcessor::ResolveForAttributeRs(
CXFA_Object* curNode,
CFXJSE_ResolveNodeData& rnd,
const WideStringView& strAttr) {
const XFA_SCRIPTATTRIBUTEINFO* lpScriptAttribute =
XFA_GetScriptAttributeByName(curNode->GetElementType(), strAttr);
if (!lpScriptAttribute)
- return 0;
+ return false;
rnd.m_pScriptAttribute = lpScriptAttribute;
rnd.m_Objects.push_back(curNode);
rnd.m_dwFlag = XFA_RESOLVENODE_RSTYPE_Attribute;
- return 1;
+ return true;
}
-int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
+bool CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
if (rnd.m_nLevel > 32 || !rnd.m_CurObject->IsNode())
- return 0;
+ return false;
CXFA_Node* curNode = rnd.m_CurObject->AsNode();
size_t nNum = rnd.m_Objects.size();
@@ -248,8 +248,7 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
children.push_back(pChild);
}
if ((dwStyles & XFA_RESOLVENODE_Properties) && pVariablesNode) {
- uint32_t uPropHash = pVariablesNode->GetClassHashCode();
- if (uPropHash == uNameHash) {
+ if (pVariablesNode->GetClassHashCode() == uNameHash) {
rnd.m_Objects.push_back(pVariablesNode);
} else {
rndFind.m_CurObject = pVariablesNode;
@@ -264,9 +263,10 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
}
if (rnd.m_Objects.size() > nNum) {
FilterCondition(rnd, wsCondition);
- return !rnd.m_Objects.empty() ? 1 : 0;
+ return !rnd.m_Objects.empty();
}
}
+
if (dwStyles & XFA_RESOLVENODE_Children) {
bool bSetFlag = false;
if (pPageSetNode && (dwStyles & XFA_RESOLVENODE_Properties))
@@ -279,6 +279,7 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
} else if (child->GetNameHash() == uNameHash) {
rnd.m_Objects.push_back(child);
}
+
if (m_pNodeHelper->NodeIsTransparent(child) &&
child->GetElementType() != XFA_Element::PageSet) {
if (!bSetFlag) {
@@ -286,9 +287,11 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
bSetFlag = true;
}
rndFind.m_CurObject = child;
+
WideString wsSaveCondition = rndFind.m_wsCondition;
rndFind.m_wsCondition.clear();
ResolveNormal(rndFind);
+
rndFind.m_wsCondition = wsSaveCondition;
rnd.m_Objects.insert(rnd.m_Objects.end(), rndFind.m_Objects.begin(),
rndFind.m_Objects.end());
@@ -311,7 +314,7 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
}
}
FilterCondition(rnd, wsCondition);
- return !rnd.m_Objects.empty() ? 1 : 0;
+ return !rnd.m_Objects.empty();
}
}
if (dwStyles & XFA_RESOLVENODE_Attributes) {
@@ -333,8 +336,9 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
}
if (rnd.m_Objects.size() > nNum) {
FilterCondition(rnd, wsCondition);
- return !rnd.m_Objects.empty() ? 1 : 0;
+ return !rnd.m_Objects.empty();
}
+
CXFA_Node* pProp = nullptr;
if (XFA_Element::Subform == curNode->GetElementType() &&
XFA_HASHCODE_Occur == uNameHash) {
@@ -353,9 +357,10 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
}
if (pProp) {
rnd.m_Objects.push_back(pProp);
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
+ return !rnd.m_Objects.empty();
}
}
+
CXFA_Node* parentNode = m_pNodeHelper->ResolveNodes_GetParent(
curNode->AsNode(), XFA_LOGIC_NoTransparent);
uint32_t uCurClassHash = curNode->GetClassHashCode();
@@ -364,10 +369,11 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
rnd.m_Objects.push_back(curNode->AsNode());
FilterCondition(rnd, wsCondition);
if (!rnd.m_Objects.empty())
- return 1;
+ return true;
}
- return 0;
+ return false;
}
+
if (dwStyles & XFA_RESOLVENODE_Siblings) {
CXFA_Node* child = parentNode->GetNodeItem(XFA_NODEITEM_FirstChild);
uint32_t dwSubStyles =
@@ -376,26 +382,27 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
dwSubStyles |= XFA_RESOLVENODE_TagName;
if (dwStyles & XFA_RESOLVENODE_ALL)
dwSubStyles |= XFA_RESOLVENODE_ALL;
+
rndFind.m_dwStyles = dwSubStyles;
while (child) {
if (child == curNode) {
if (dwStyles & XFA_RESOLVENODE_TagName) {
- if (uCurClassHash == uNameHash) {
+ if (uCurClassHash == uNameHash)
rnd.m_Objects.push_back(curNode);
- }
} else {
if (child->GetNameHash() == uNameHash) {
rnd.m_Objects.push_back(curNode);
if (rnd.m_nLevel == 0 && wsCondition.GetLength() == 0) {
rnd.m_Objects.clear();
rnd.m_Objects.push_back(curNode);
- return 1;
+ return true;
}
}
}
child = child->GetNodeItem(XFA_NODEITEM_NextSibling);
continue;
}
+
if (dwStyles & XFA_RESOLVENODE_TagName) {
if (child->GetClassHashCode() == uNameHash)
rnd.m_Objects.push_back(child);
@@ -409,18 +416,18 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
child->GetElementType() == XFA_Element::PageSet)) {
bInnerSearch = true;
}
- } else {
- if (m_pNodeHelper->NodeIsTransparent(child)) {
- bInnerSearch = true;
- }
+ } else if (m_pNodeHelper->NodeIsTransparent(child)) {
+ bInnerSearch = true;
}
if (bInnerSearch) {
rndFind.m_CurObject = child;
WideString wsOriginCondition = rndFind.m_wsCondition;
rndFind.m_wsCondition.clear();
+
uint32_t dwOriginStyle = rndFind.m_dwStyles;
rndFind.m_dwStyles = dwOriginStyle | XFA_RESOLVENODE_ALL;
ResolveNormal(rndFind);
+
rndFind.m_dwStyles = dwOriginStyle;
rndFind.m_wsCondition = wsOriginCondition;
rnd.m_Objects.insert(rnd.m_Objects.end(), rndFind.m_Objects.begin(),
@@ -443,18 +450,18 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
}
}
FilterCondition(rnd, wsCondition);
- return !rnd.m_Objects.empty() ? 1 : 0;
+ return !rnd.m_Objects.empty();
}
}
+
if (dwStyles & XFA_RESOLVENODE_Parent) {
uint32_t dwSubStyles = XFA_RESOLVENODE_Siblings | XFA_RESOLVENODE_Parent |
XFA_RESOLVENODE_Properties;
- if (dwStyles & XFA_RESOLVENODE_TagName) {
+ if (dwStyles & XFA_RESOLVENODE_TagName)
dwSubStyles |= XFA_RESOLVENODE_TagName;
- }
- if (dwStyles & XFA_RESOLVENODE_ALL) {
+ if (dwStyles & XFA_RESOLVENODE_ALL)
dwSubStyles |= XFA_RESOLVENODE_ALL;
- }
+
rndFind.m_dwStyles = dwSubStyles;
rndFind.m_CurObject = parentNode;
rnd.m_pSC->GetUpObjectArray()->push_back(parentNode);
@@ -463,37 +470,29 @@ int32_t CFXJSE_ResolveProcessor::ResolveNormal(CFXJSE_ResolveNodeData& rnd) {
rndFind.m_Objects.end());
rndFind.m_Objects.clear();
if (rnd.m_Objects.size() > nNum)
- return 1;
+ return true;
}
- return 0;
+ return false;
}
-int32_t CFXJSE_ResolveProcessor::ResolveAsterisk(CFXJSE_ResolveNodeData& rnd) {
+bool CFXJSE_ResolveProcessor::ResolveAsterisk(CFXJSE_ResolveNodeData& rnd) {
CXFA_Node* curNode = ToNode(rnd.m_CurObject);
std::vector<CXFA_Node*> array =
curNode->GetNodeList(XFA_NODEFILTER_Children | XFA_NODEFILTER_Properties,
XFA_Element::Unknown);
rnd.m_Objects.insert(rnd.m_Objects.end(), array.begin(), array.end());
- return pdfium::CollectionSize<int32_t>(rnd.m_Objects);
-}
-
-int32_t CFXJSE_ResolveProcessor::ResolvePopStack(std::vector<int32_t>* stack) {
- if (stack->empty())
- return -1;
-
- int32_t nType = stack->back();
- stack->pop_back();
- return nType;
+ return !rnd.m_Objects.empty();
}
int32_t CFXJSE_ResolveProcessor::GetFilter(const WideStringView& wsExpression,
int32_t nStart,
CFXJSE_ResolveNodeData& rnd) {
ASSERT(nStart > -1);
+
int32_t iLength = wsExpression.GetLength();
- if (nStart >= iLength) {
+ if (nStart >= iLength)
return 0;
- }
+
WideString& wsName = rnd.m_wsName;
WideString& wsCondition = rnd.m_wsCondition;
wchar_t* pNameBuf = wsName.GetBuffer(iLength - nStart);
@@ -503,7 +502,8 @@ int32_t CFXJSE_ResolveProcessor::GetFilter(const WideStringView& wsExpression,
std::vector<int32_t> stack;
int32_t nType = -1;
const wchar_t* pSrc = wsExpression.unterminated_c_str();
- wchar_t wPrev = 0, wCur;
+ wchar_t wPrev = 0;
+ wchar_t wCur;
bool bIsCondition = false;
while (nStart < iLength) {
wCur = pSrc[nStart++];
@@ -516,12 +516,10 @@ int32_t CFXJSE_ResolveProcessor::GetFilter(const WideStringView& wsExpression,
rnd.m_dwStyles |= XFA_RESOLVENODE_AnyChild;
continue;
}
+
wchar_t wLookahead = nStart < iLength ? pSrc[nStart] : 0;
- if (wLookahead != '[' && wLookahead != '(') {
- if (nType < 0) {
- break;
- }
- }
+ if (wLookahead != '[' && wLookahead != '(' && nType < 0)
+ break;
}
if (wCur == '[' || wCur == '(') {
bIsCondition = true;
@@ -529,47 +527,25 @@ int32_t CFXJSE_ResolveProcessor::GetFilter(const WideStringView& wsExpression,
(pSrc[nStart] == '[' || pSrc[nStart] == '(')) {
bIsCondition = true;
}
- if (bIsCondition) {
+ if (bIsCondition)
pConditionBuf[nConditionCount++] = wCur;
- } else {
+ else
pNameBuf[nNameCount++] = wCur;
- }
- bool bRecursive = true;
- switch (nType) {
- case 0:
- if (wCur == ']') {
- nType = ResolvePopStack(&stack);
- bRecursive = false;
- }
- break;
- case 1:
- if (wCur == ')') {
- nType = ResolvePopStack(&stack);
- bRecursive = false;
- }
- break;
- case 2:
- if (wCur == '"') {
- nType = ResolvePopStack(&stack);
- bRecursive = false;
- }
- break;
- }
- if (bRecursive) {
- switch (wCur) {
- case '[':
- stack.push_back(nType);
- nType = 0;
- break;
- case '(':
- stack.push_back(nType);
- nType = 1;
- break;
- case '"':
- stack.push_back(nType);
- nType = 2;
- break;
- }
+
+ if ((nType == 0 && wCur == ']') || (nType == 1 && wCur == ')') ||
+ (nType == 2 && wCur == '"')) {
+ nType = stack.empty() ? -1 : stack.back();
+ if (!stack.empty())
+ stack.pop_back();
+ } else if (wCur == '[') {
+ stack.push_back(nType);
+ nType = 0;
+ } else if (wCur == '(') {
+ stack.push_back(nType);
+ nType = 1;
+ } else if (wCur == '"') {
+ stack.push_back(nType);
+ nType = 2;
}
wPrev = wCur;
}
@@ -597,11 +573,11 @@ void CFXJSE_ResolveProcessor::ConditionArray(int32_t iCurIndex,
wchar_t ch = wsCondition[i];
if (ch == ' ')
continue;
- if (ch == '+' || ch == '-') {
+ if (ch == '+' || ch == '-')
bRelative = true;
- } else if (ch == '*') {
+ else if (ch == '*')
bAll = true;
- }
+
break;
}
if (bAll) {
@@ -612,28 +588,23 @@ void CFXJSE_ResolveProcessor::ConditionArray(int32_t iCurIndex,
rnd.m_Objects.clear();
m_pNodeHelper->m_iCurAllStart = -1;
m_pNodeHelper->m_pAllStartParent = nullptr;
- } else {
- if (m_pNodeHelper->m_iCurAllStart == -1) {
- m_pNodeHelper->m_iCurAllStart = m_iCurStart;
- m_pNodeHelper->m_pAllStartParent = ToNode(rnd.m_CurObject);
- }
+ } else if (m_pNodeHelper->m_iCurAllStart == -1) {
+ m_pNodeHelper->m_iCurAllStart = m_iCurStart;
+ m_pNodeHelper->m_pAllStartParent = ToNode(rnd.m_CurObject);
}
} else if (rnd.m_dwStyles & XFA_RESOLVENODE_BindNew) {
- if (m_pNodeHelper->m_iCurAllStart == -1) {
+ if (m_pNodeHelper->m_iCurAllStart == -1)
m_pNodeHelper->m_iCurAllStart = m_iCurStart;
- }
}
return;
}
- if (iFoundCount == 1 && !iLen) {
+ if (iFoundCount == 1 && !iLen)
return;
- }
- WideString wsIndex;
- wsIndex = wsCondition.Mid(i, iLen - 1 - i);
- int32_t iIndex = wsIndex.GetInteger();
- if (bRelative) {
+
+ int32_t iIndex = wsCondition.Mid(i, iLen - 1 - i).GetInteger();
+ if (bRelative)
iIndex += iCurIndex;
- }
+
if (iFoundCount <= iIndex || iIndex < 0) {
if (rnd.m_dwStyles & XFA_RESOLVENODE_CreateNode) {
m_pNodeHelper->m_pCreateParent = ToNode(rnd.m_CurObject);
@@ -654,13 +625,12 @@ void CFXJSE_ResolveProcessor::DoPredicateFilter(int32_t iCurIndex,
ASSERT(iFoundCount == pdfium::CollectionSize<int32_t>(rnd.m_Objects));
WideString wsExpression;
CXFA_ScriptData::Type eLangType = CXFA_ScriptData::Type::Unknown;
- if (wsCondition.Left(2) == L".[" && wsCondition.Last() == L']') {
+ if (wsCondition.Left(2) == L".[" && wsCondition.Last() == L']')
eLangType = CXFA_ScriptData::Type::Formcalc;
- } else if (wsCondition.Left(2) == L".(" && wsCondition.Last() == L')') {
+ else if (wsCondition.Left(2) == L".(" && wsCondition.Last() == L')')
eLangType = CXFA_ScriptData::Type::Javascript;
- } else {
+ else
return;
- }
CFXJSE_Engine* pContext = rnd.m_pSC;
wsExpression = wsCondition.Mid(2, wsCondition.GetLength() - 3);
@@ -689,16 +659,17 @@ void CFXJSE_ResolveProcessor::FilterCondition(CFXJSE_ResolveNodeData& rnd,
bIsProperty, false);
}
}
+
int32_t iFoundCount = pdfium::CollectionSize<int32_t>(rnd.m_Objects);
wsCondition.Trim();
+
int32_t iLen = wsCondition.GetLength();
if (!iLen) {
- if (rnd.m_dwStyles & XFA_RESOLVENODE_ALL) {
+ if (rnd.m_dwStyles & XFA_RESOLVENODE_ALL)
return;
- }
- if (iFoundCount == 1) {
+ if (iFoundCount == 1)
return;
- }
+
if (iFoundCount <= iCurrIndex) {
if (rnd.m_dwStyles & XFA_RESOLVENODE_CreateNode) {
m_pNodeHelper->m_pCreateParent = ToNode(rnd.m_CurObject);
@@ -706,26 +677,25 @@ void CFXJSE_ResolveProcessor::FilterCondition(CFXJSE_ResolveNodeData& rnd,
}
rnd.m_Objects.clear();
return;
- } else {
- CXFA_Object* ret = rnd.m_Objects[iCurrIndex];
- rnd.m_Objects.clear();
- rnd.m_Objects.push_back(ret);
- return;
}
+
+ CXFA_Object* ret = rnd.m_Objects[iCurrIndex];
+ rnd.m_Objects.clear();
+ rnd.m_Objects.push_back(ret);
+ return;
}
+
wchar_t wTypeChar = wsCondition[0];
switch (wTypeChar) {
case '[':
ConditionArray(iCurrIndex, wsCondition, iFoundCount, rnd);
return;
- case '(':
- return;
- case '"':
- return;
case '.':
- if (iLen > 1 && (wsCondition[1] == '[' || wsCondition[1] == '(')) {
+ if (iLen > 1 && (wsCondition[1] == '[' || wsCondition[1] == '('))
DoPredicateFilter(iCurrIndex, wsCondition, iFoundCount, rnd);
- }
+ return;
+ case '(':
+ case '"':
default:
return;
}
@@ -733,9 +703,9 @@ void CFXJSE_ResolveProcessor::FilterCondition(CFXJSE_ResolveNodeData& rnd,
void CFXJSE_ResolveProcessor::SetStylesForChild(uint32_t dwParentStyles,
CFXJSE_ResolveNodeData& rnd) {
uint32_t dwSubStyles = XFA_RESOLVENODE_Children;
- if (dwParentStyles & XFA_RESOLVENODE_TagName) {
+ if (dwParentStyles & XFA_RESOLVENODE_TagName)
dwSubStyles |= XFA_RESOLVENODE_TagName;
- }
+
dwSubStyles &= ~XFA_RESOLVENODE_Parent;
dwSubStyles &= ~XFA_RESOLVENODE_Siblings;
dwSubStyles &= ~XFA_RESOLVENODE_Properties;