From f74ad998d2e8d2636fb25e94823946a3b151e34e Mon Sep 17 00:00:00 2001 From: tsepez Date: Wed, 11 May 2016 10:26:05 -0700 Subject: Replace some calls to Release() with direct delete, part 1. Searching for the anti-pattern: void Release() { delete this; } We must be explicit on the ownership model. Add unique_ptrs as a result. Review-Url: https://codereview.chromium.org/1960673003 --- xfa/fde/css/fde_csssyntax.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'xfa/fde/css/fde_csssyntax.h') diff --git a/xfa/fde/css/fde_csssyntax.h b/xfa/fde/css/fde_csssyntax.h index d14f81ea7a..a4e9249055 100644 --- a/xfa/fde/css/fde_csssyntax.h +++ b/xfa/fde/css/fde_csssyntax.h @@ -77,9 +77,8 @@ enum FDE_CSSSYNTAXMODE { class CFDE_CSSSyntaxParser : public CFX_Target { public: CFDE_CSSSyntaxParser(); - ~CFDE_CSSSyntaxParser(); + ~CFDE_CSSSyntaxParser() override; - void Release() { delete this; } FX_BOOL Init(IFX_Stream* pStream, int32_t iCSSPlaneSize, int32_t iTextDataSize = 32, -- cgit v1.2.3