commit 6fa9707e5db5454ab5ea1a8349d9e1d7cf2b175c
author Dan Sinclair <dsinclair@chromium.org> Sat Nov 04 01:17:27 2017 +0000
committer Chromium commit bot <commit-bot@chromium.org> Sat Nov 04 01:17:27 2017 +0000
tree 16fde30271b76d4a012f4b67d29141e62a653869
parent 83ae996e5860edb31c061e12abdbf7d4a955a0b3

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 <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>

fxjs/cjx_node.cpp

diff --git a/fxjs/cjx_node.cpp b/fxjs/cjx_node.cpp
index c14e6f0..17553c9 100644
--- a/fxjs/cjx_node.cpp
+++ b/fxjs/cjx_node.cpp
@@ -3894,9 +3894,9 @@
                                  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);

fxjs/cjx_node.h

diff --git a/fxjs/cjx_node.h b/fxjs/cjx_node.h
index 8a67d3f..02091b9 100644
--- a/fxjs/cjx_node.h
+++ b/fxjs/cjx_node.h
@@ -449,7 +449,7 @@
   void MoveBufferMapData(CXFA_Node* pSrcModule,
                          CXFA_Node* pDstModule,
                          void* pKey,
-                         bool bRecursive = false);
+                         bool bRecursive);
 
   int32_t execSingleEventByName(const WideStringView& wsEventName,
                                 XFA_Element eType);