From 3ac07ffafea1ee4d3cf985e32692849dd3fcfdfb Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Thu, 2 Nov 2017 19:09:58 +0000 Subject: Cleanup params to CJX_Node::TryCData This CL removes the bProto param as it was never set and inlines the value where needed in the methods. The default value was removed from bUseDefault and inlined into callsites as needed. Change-Id: I773261d19aa3799bc607e7df482b5f5e5217bee6 Reviewed-on: https://pdfium-review.googlesource.com/17533 Commit-Queue: dsinclair Reviewed-by: Tom Sepez --- fxjs/cjx_node.h | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) (limited to 'fxjs/cjx_node.h') diff --git a/fxjs/cjx_node.h b/fxjs/cjx_node.h index 863dcc53af..df360011a4 100644 --- a/fxjs/cjx_node.h +++ b/fxjs/cjx_node.h @@ -84,14 +84,8 @@ class CJX_Node : public CJX_Object { bool SetInteger(XFA_ATTRIBUTE eAttr, int32_t iValue, bool bNotify); int32_t GetInteger(XFA_ATTRIBUTE eAttr); - bool TryCData(XFA_ATTRIBUTE eAttr, - WideStringView& wsValue, - bool bUseDefault = true, - bool bProto = true); - bool TryCData(XFA_ATTRIBUTE eAttr, - WideString& wsValue, - bool bUseDefault = true, - bool bProto = true); + bool TryCData(XFA_ATTRIBUTE eAttr, WideStringView& wsValue, bool bUseDefault); + bool TryCData(XFA_ATTRIBUTE eAttr, WideString& wsValue, bool bUseDefault); bool SetCData(XFA_ATTRIBUTE eAttr, const WideString& wsValue, bool bNotify = false, -- cgit v1.2.3