From 6fdfe387645217cdd77e8e4230a0d8348d831e31 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Fri, 10 Aug 2018 23:43:26 +0000 Subject: Kill some optional parameters that are always supplied. No need to even bring any .cpp files in line with these headers. Change-Id: I934169d77ae09adc11f02e5ea92b1f8b078c9477 Reviewed-on: https://pdfium-review.googlesource.com/39876 Commit-Queue: Tom Sepez Reviewed-by: Lei Zhang --- xfa/fgas/font/cfgas_fontmgr.h | 4 ++-- xfa/fwl/theme/cfwl_carettp.h | 2 +- xfa/fwl/theme/cfwl_listboxtp.h | 4 ++-- xfa/fwl/theme/cfwl_scrollbartp.h | 10 +++++----- xfa/fwl/theme/cfwl_widgettp.h | 14 +++++++------- xfa/fxfa/cxfa_ffwidgethandler.h | 2 +- xfa/fxfa/fxfa.h | 12 ++++++------ xfa/fxfa/parser/cxfa_layoutpagemgr.h | 22 +++++++++++----------- xfa/fxfa/parser/cxfa_nodehelper.h | 12 ++++++------ 9 files changed, 41 insertions(+), 41 deletions(-) (limited to 'xfa') diff --git a/xfa/fgas/font/cfgas_fontmgr.h b/xfa/fgas/font/cfgas_fontmgr.h index 5a1e9babf6..72578be744 100644 --- a/xfa/fgas/font/cfgas_fontmgr.h +++ b/xfa/fgas/font/cfgas_fontmgr.h @@ -165,12 +165,12 @@ class CFGAS_FontMgr : public Observable { uint16_t wCodePage, uint32_t dwFontStyles, const WideString& FontName, - wchar_t wcUnicode = 0xFFFE); + wchar_t wcUnicode); int32_t CalcPenalty(CFX_FontDescriptor* pInstalled, uint16_t wCodePage, uint32_t dwFontStyles, const WideString& FontName, - wchar_t wcUnicode = 0xFFFE); + wchar_t wcUnicode); RetainPtr LoadFont(const WideString& wsFaceName, int32_t iFaceIndex, int32_t* pFaceCount); diff --git a/xfa/fwl/theme/cfwl_carettp.h b/xfa/fwl/theme/cfwl_carettp.h index 8b88feeda3..64bf51b20a 100644 --- a/xfa/fwl/theme/cfwl_carettp.h +++ b/xfa/fwl/theme/cfwl_carettp.h @@ -22,7 +22,7 @@ class CFWL_CaretTP : public CFWL_WidgetTP { uint32_t dwStates, const CFX_RectF* pRect, CXFA_GEColor* crFill, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); }; #endif // XFA_FWL_THEME_CFWL_CARETTP_H_ diff --git a/xfa/fwl/theme/cfwl_listboxtp.h b/xfa/fwl/theme/cfwl_listboxtp.h index 5dc45af4fe..cc4691cce0 100644 --- a/xfa/fwl/theme/cfwl_listboxtp.h +++ b/xfa/fwl/theme/cfwl_listboxtp.h @@ -21,8 +21,8 @@ class CFWL_ListBoxTP : public CFWL_WidgetTP { void DrawListBoxItem(CXFA_Graphics* pGraphics, uint32_t dwStates, const CFX_RectF* prtItem, - void* pData = nullptr, - CFX_Matrix* pMatrix = nullptr); + void* pData, + CFX_Matrix* pMatrix); }; #endif // XFA_FWL_THEME_CFWL_LISTBOXTP_H_ diff --git a/xfa/fwl/theme/cfwl_scrollbartp.h b/xfa/fwl/theme/cfwl_scrollbartp.h index bae4e1902c..8398706458 100644 --- a/xfa/fwl/theme/cfwl_scrollbartp.h +++ b/xfa/fwl/theme/cfwl_scrollbartp.h @@ -33,24 +33,24 @@ class CFWL_ScrollBarTP : public CFWL_WidgetTP { const CFX_RectF* pRect, bool bVert, FWLTHEME_STATE eState, - bool bPawButton = true, - CFX_Matrix* pMatrix = nullptr); + bool bPawButton, + CFX_Matrix* pMatrix); void DrawTrack(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, bool bVert, FWLTHEME_STATE eState, bool bLowerTrack, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); void DrawMaxMinBtn(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, FWLTHEME_DIRECTION eDict, FWLTHEME_STATE eState, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); void DrawPaw(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, bool bVert, FWLTHEME_STATE eState, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); std::unique_ptr m_pThemeData; diff --git a/xfa/fwl/theme/cfwl_widgettp.h b/xfa/fwl/theme/cfwl_widgettp.h index f62098d4ef..0facb1ee62 100644 --- a/xfa/fwl/theme/cfwl_widgettp.h +++ b/xfa/fwl/theme/cfwl_widgettp.h @@ -56,31 +56,31 @@ class CFWL_WidgetTP { void DrawBorder(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); void FillBackground(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); void FillSolidRect(CXFA_Graphics* pGraphics, FX_ARGB fillColor, const CFX_RectF* pRect, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); void DrawFocus(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); void DrawArrow(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, FWLTHEME_DIRECTION eDict, FX_ARGB argSign, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); void DrawBtn(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, FWLTHEME_STATE eState, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); void DrawArrowBtn(CXFA_Graphics* pGraphics, const CFX_RectF* pRect, FWLTHEME_DIRECTION eDict, FWLTHEME_STATE eState, - CFX_Matrix* pMatrix = nullptr); + CFX_Matrix* pMatrix); uint32_t m_dwRefCount; std::unique_ptr m_pTextOut; diff --git a/xfa/fxfa/cxfa_ffwidgethandler.h b/xfa/fxfa/cxfa_ffwidgethandler.h index f2b6ae0d1c..58e531f098 100644 --- a/xfa/fxfa/cxfa_ffwidgethandler.h +++ b/xfa/fxfa/cxfa_ffwidgethandler.h @@ -25,7 +25,7 @@ class CXFA_FFWidgetHandler { CXFA_FFWidget* CreateWidget(CXFA_FFWidget* hParent, XFA_WIDGETTYPE eType, - CXFA_FFWidget* hBefore = nullptr); + CXFA_FFWidget* hBefore); bool OnMouseEnter(CXFA_FFWidget* hWidget); bool OnMouseExit(CXFA_FFWidget* hWidget); diff --git a/xfa/fxfa/fxfa.h b/xfa/fxfa/fxfa.h index 6de20f0fbc..254d99f924 100644 --- a/xfa/fxfa/fxfa.h +++ b/xfa/fxfa/fxfa.h @@ -158,9 +158,9 @@ class IXFA_AppProvider { * user, refer to XFA_ID. */ virtual int32_t MsgBox(const WideString& wsMessage, - const WideString& wsTitle = L"", - uint32_t dwIconType = 0, - uint32_t dwButtonType = 0) = 0; + const WideString& wsTitle, + uint32_t dwIconType, + uint32_t dwButtonType) = 0; /** * Get a response from the user. @@ -171,9 +171,9 @@ class IXFA_AppProvider { * @return A string containing the user's response. */ virtual WideString Response(const WideString& wsQuestion, - const WideString& wsTitle = L"", - const WideString& wsDefaultAnswer = L"", - bool bMask = true) = 0; + const WideString& wsTitle, + const WideString& wsDefaultAnswer, + bool bMask) = 0; /** * Download something from somewhere. diff --git a/xfa/fxfa/parser/cxfa_layoutpagemgr.h b/xfa/fxfa/parser/cxfa_layoutpagemgr.h index 42fa4e6cd9..7c9055dcf8 100644 --- a/xfa/fxfa/parser/cxfa_layoutpagemgr.h +++ b/xfa/fxfa/parser/cxfa_layoutpagemgr.h @@ -45,8 +45,8 @@ class CXFA_LayoutPageMgr { bool ProcessOverflow(CXFA_Node* pFormNode, CXFA_Node*& pLeaderNode, CXFA_Node*& pTrailerNode, - bool bDataMerge = false, - bool bCreatePage = true); + bool bDataMerge, + bool bCreatePage); CXFA_Node* QueryOverflow(CXFA_Node* pFormNode); bool ProcessBookendLeaderOrTrailer(CXFA_Node* pBookendNode, bool bLeader, @@ -78,7 +78,7 @@ class CXFA_LayoutPageMgr { CXFA_Node* BreakOverflow(CXFA_Node* pOverflowNode, CXFA_Node*& pLeaderTemplate, CXFA_Node*& pTrailerTemplate, - bool bCreatePage = true); + bool bCreatePage); bool ResolveBookendLeaderOrTrailer(CXFA_Node* pBookendNode, bool bLeader, CXFA_Node*& pBookendAppendTemplate); @@ -94,16 +94,16 @@ class CXFA_LayoutPageMgr { void CreateNextMinRecord(CXFA_Node* pRecordNode); bool FindPageAreaFromPageSet(CXFA_Node* pPageSet, CXFA_Node* pStartChild, - CXFA_Node* pTargetPageArea = nullptr, - CXFA_Node* pTargetContentArea = nullptr, - bool bNewPage = false, - bool bQuery = false); + CXFA_Node* pTargetPageArea, + CXFA_Node* pTargetContentArea, + bool bNewPage, + bool bQuery); bool FindPageAreaFromPageSet_Ordered(CXFA_Node* pPageSet, CXFA_Node* pStartChild, - CXFA_Node* pTargetPageArea = nullptr, - CXFA_Node* pTargetContentArea = nullptr, - bool bNewPage = false, - bool bQuery = false); + CXFA_Node* pTargetPageArea, + CXFA_Node* pTargetContentArea, + bool bNewPage, + bool bQuery); bool FindPageAreaFromPageSet_SimplexDuplex( CXFA_Node* pPageSet, CXFA_Node* pStartChild, diff --git a/xfa/fxfa/parser/cxfa_nodehelper.h b/xfa/fxfa/parser/cxfa_nodehelper.h index b53e1f4063..49e537ccaa 100644 --- a/xfa/fxfa/parser/cxfa_nodehelper.h +++ b/xfa/fxfa/parser/cxfa_nodehelper.h @@ -25,7 +25,7 @@ class CXFA_NodeHelper { CXFA_Node* ResolveNodes_GetOneChild(CXFA_Node* parent, const wchar_t* pwsName, - bool bIsClassName = false); + bool bIsClassName); CXFA_Node* ResolveNodes_GetParent( CXFA_Node* pNode, XFA_LOGIC_TYPE eLogicType = XFA_LOGIC_NoTransparent); @@ -39,15 +39,15 @@ class CXFA_NodeHelper { int32_t NodeAcc_TraverseAnySiblings(CXFA_Node* parent, uint32_t dNameHash, std::vector* pSiblings, - bool bIsClassName = false); + bool bIsClassName); int32_t CountSiblings(CXFA_Node* pNode, XFA_LOGIC_TYPE eLogicType, std::vector* pSiblings, - bool bIsClassName = false); + bool bIsClassName); int32_t GetIndex(CXFA_Node* pNode, - XFA_LOGIC_TYPE eLogicType = XFA_LOGIC_NoTransparent, - bool bIsProperty = false, - bool bIsClassIndex = false); + XFA_LOGIC_TYPE eLogicType, + bool bIsProperty, + bool bIsClassIndex); WideString GetNameExpression( CXFA_Node* refNode, bool bIsAllPath, -- cgit v1.2.3