From b97784706ec898ff6b1c36f1564c0c66f9419b17 Mon Sep 17 00:00:00 2001 From: dsinclair Date: Thu, 23 Jun 2016 13:34:10 -0700 Subject: Pass needed value to CXFA_Node constructor Currently the CXFA_Node constructor will call out to lookup the object type for a given element type. There is only one called of this constructor and it already has the object type so just pass it through instead of getting the element data a second time. Review-Url: https://codereview.chromium.org/2092853002 --- xfa/fxfa/parser/xfa_object_imp.cpp | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'xfa/fxfa/parser/xfa_object_imp.cpp') diff --git a/xfa/fxfa/parser/xfa_object_imp.cpp b/xfa/fxfa/parser/xfa_object_imp.cpp index 28c56ce0f3..04b33b47c9 100644 --- a/xfa/fxfa/parser/xfa_object_imp.cpp +++ b/xfa/fxfa/parser/xfa_object_imp.cpp @@ -43,10 +43,6 @@ void XFA_CopyWideString(void*& pData) { XFA_MAPDATABLOCKCALLBACKINFO deleteWideStringCallBack = {XFA_DeleteWideString, XFA_CopyWideString}; -XFA_ObjectType XFA_GetElementObjectType(XFA_Element eType) { - return XFA_GetElementByID(eType)->eObjectType; -} - void XFA_DataNodeDeleteBindItem(void* pData) { delete static_cast(pData); } @@ -108,8 +104,11 @@ XFA_MAPMODULEDATA::XFA_MAPMODULEDATA() {} XFA_MAPMODULEDATA::~XFA_MAPMODULEDATA() {} -CXFA_Node::CXFA_Node(CXFA_Document* pDoc, uint16_t ePacket, XFA_Element eType) - : CXFA_Object(pDoc, XFA_GetElementObjectType(eType), eType), +CXFA_Node::CXFA_Node(CXFA_Document* pDoc, + uint16_t ePacket, + XFA_ObjectType oType, + XFA_Element eType) + : CXFA_Object(pDoc, oType, eType), m_pNext(nullptr), m_pChild(nullptr), m_pLastChild(nullptr), -- cgit v1.2.3