diff options
| author | Ryan Harrison <rharrison@chromium.org> | 2018-01-04 13:57:02 -0500 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-04 19:15:31 +0000 |
| commit | cb6c8d4ec4781c7db8448f2b4d9a8af879cf014e (patch) | |
| tree | de752e161b1196c6b26b2f34418b7dff7ee39b64 /xfa/fxfa/parser/cxfa_node.cpp | |
| parent | 981a2908d7a213363772d1846a7c63c2cfd73c7e (diff) | |
| download | pdfium-cb6c8d4ec4781c7db8448f2b4d9a8af879cf014e.tar.xz | |
Make CXFA_List::GetLength return a size_t
This requires a number of other minor cleanups to get sizes and
signedness agreeing everywhere.
Change-Id: Ie25318a23d458be5a8a2a91927c3b37eb198f5ce
Reviewed-on: https://pdfium-review.googlesource.com/22256
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_node.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_node.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/xfa/fxfa/parser/cxfa_node.cpp b/xfa/fxfa/parser/cxfa_node.cpp index d7d187d168..f6f2675c29 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -784,17 +784,17 @@ CXFA_Node* CXFA_Node::GetModelNode() { } } -int32_t CXFA_Node::CountChildren(XFA_Element eType, bool bOnlyChild) { - int32_t iCount = 0; +size_t CXFA_Node::CountChildren(XFA_Element eType, bool bOnlyChild) { + size_t count = 0; for (CXFA_Node* pNode = m_pChild; pNode; pNode = pNode->GetNodeItem(XFA_NODEITEM_NextSibling)) { if (pNode->GetElementType() != eType && eType != XFA_Element::Unknown) continue; if (bOnlyChild && HasProperty(pNode->GetElementType())) continue; - ++iCount; + ++count; } - return iCount; + return count; } CXFA_Node* CXFA_Node::GetChildInternal(int32_t index, |