diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-02-07 17:10:22 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-07 17:10:22 +0000 |
commit | c0c32b0a3af11624a1f3eaeb9d940d525b54cd4d (patch) | |
tree | 327d1df5334ac3893d5714ca023ab2d81b0ff151 /xfa/fxfa/parser/cxfa_attachnodelist.cpp | |
parent | b3a3eaab0471ed4fe22299c02b51b16bec813773 (diff) | |
download | pdfium-c0c32b0a3af11624a1f3eaeb9d940d525b54cd4d.tar.xz |
Remove unused return values from CXFA_Node
This CL removes the unused return values from InsertChild and
RemoveChild methods in CXFA_Node.
Bug: chromium:807863
Change-Id: Iac468afc5c48f51e7df3ea12d11b128a0ac124ea
Reviewed-on: https://pdfium-review.googlesource.com/25670
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_attachnodelist.cpp')
-rw-r--r-- | xfa/fxfa/parser/cxfa_attachnodelist.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/xfa/fxfa/parser/cxfa_attachnodelist.cpp b/xfa/fxfa/parser/cxfa_attachnodelist.cpp index f1fbfa784b..4e6a678162 100644 --- a/xfa/fxfa/parser/cxfa_attachnodelist.cpp +++ b/xfa/fxfa/parser/cxfa_attachnodelist.cpp @@ -21,24 +21,24 @@ size_t CXFA_AttachNodeList::GetLength() { m_pAttachNode->GetElementType() == XFA_Element::Subform); } -bool CXFA_AttachNodeList::Append(CXFA_Node* pNode) { +void CXFA_AttachNodeList::Append(CXFA_Node* pNode) { CXFA_Node* pParent = pNode->GetParent(); if (pParent) pParent->RemoveChild(pNode, true); - return m_pAttachNode->InsertChild(pNode, nullptr); + m_pAttachNode->InsertChild(pNode, nullptr); } -bool CXFA_AttachNodeList::Insert(CXFA_Node* pNewNode, CXFA_Node* pBeforeNode) { +void CXFA_AttachNodeList::Insert(CXFA_Node* pNewNode, CXFA_Node* pBeforeNode) { CXFA_Node* pParent = pNewNode->GetParent(); if (pParent) pParent->RemoveChild(pNewNode, true); - return m_pAttachNode->InsertChild(pNewNode, pBeforeNode); + m_pAttachNode->InsertChild(pNewNode, pBeforeNode); } -bool CXFA_AttachNodeList::Remove(CXFA_Node* pNode) { - return m_pAttachNode->RemoveChild(pNode, true); +void CXFA_AttachNodeList::Remove(CXFA_Node* pNode) { + m_pAttachNode->RemoveChild(pNode, true); } CXFA_Node* CXFA_AttachNodeList::Item(size_t index) { |