From 6fa9707e5db5454ab5ea1a8349d9e1d7cf2b175c Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Sat, 4 Nov 2017 01:17:27 +0000 Subject: Remove default from CJX_Node::MoveBufferMapData This CL removes the default value from the bRecursive flag. Change-Id: I88882fd1206d82facd977f97bda4e585571ce5e0 Reviewed-on: https://pdfium-review.googlesource.com/17571 Reviewed-by: Tom Sepez Commit-Queue: dsinclair --- fxjs/cjx_node.cpp | 4 ++-- fxjs/cjx_node.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/fxjs/cjx_node.cpp b/fxjs/cjx_node.cpp index c14e6f092c..17553c940a 100644 --- a/fxjs/cjx_node.cpp +++ b/fxjs/cjx_node.cpp @@ -3894,9 +3894,9 @@ void CJX_Node::MoveBufferMapData(CXFA_Node* pSrcModule, CXFA_Node* pDstModule, void* pKey, bool bRecursive) { - if (!pSrcModule || !pDstModule || !pKey) { + if (!pSrcModule || !pDstModule || !pKey) return; - } + if (bRecursive) { CXFA_Node* pSrcChild = pSrcModule->GetNodeItem(XFA_NODEITEM_FirstChild); CXFA_Node* pDstChild = pDstModule->GetNodeItem(XFA_NODEITEM_FirstChild); diff --git a/fxjs/cjx_node.h b/fxjs/cjx_node.h index 8a67d3f0fc..02091b924b 100644 --- a/fxjs/cjx_node.h +++ b/fxjs/cjx_node.h @@ -449,7 +449,7 @@ class CJX_Node : public CJX_Object { void MoveBufferMapData(CXFA_Node* pSrcModule, CXFA_Node* pDstModule, void* pKey, - bool bRecursive = false); + bool bRecursive); int32_t execSingleEventByName(const WideStringView& wsEventName, XFA_Element eType); -- cgit v1.2.3