From c757d9a6f90bb879ce2cdd4a756b7b0e1885eb25 Mon Sep 17 00:00:00 2001 From: tsepez Date: Mon, 23 Jan 2017 11:01:42 -0800 Subject: Remove some |void Release() { delete this; }| anti-pattern. m_pSyntaxParser was unused. Review-Url: https://codereview.chromium.org/2646203002 --- xfa/fxfa/parser/cxfa_simple_parser.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'xfa/fxfa/parser/cxfa_simple_parser.cpp') diff --git a/xfa/fxfa/parser/cxfa_simple_parser.cpp b/xfa/fxfa/parser/cxfa_simple_parser.cpp index 9f6fef9c1f..2417afaf1e 100644 --- a/xfa/fxfa/parser/cxfa_simple_parser.cpp +++ b/xfa/fxfa/parser/cxfa_simple_parser.cpp @@ -747,7 +747,7 @@ CXFA_Node* CXFA_SimpleParser::ParseAsXDPPacket_Data( m_pFactory->CreateNode(XFA_XDPPACKET_Datasets, XFA_Element::DataGroup); if (!pNode) { if (pDataXMLNode != pXMLDocumentNode) - pDataXMLNode->Release(); + delete pDataXMLNode; return nullptr; } CFX_WideString wsLocalName; -- cgit v1.2.3