From 4de3d095c9d9e961f93750cf1ebd489fd515be12 Mon Sep 17 00:00:00 2001 From: tsepez Date: Thu, 3 Nov 2016 17:05:07 -0700 Subject: Remove CPDF_Object::Release() in favor of direct delete Follow-on once we prove Release always deletes in previous CL. Review-Url: https://codereview.chromium.org/2384883003 --- core/fpdfapi/parser/cpdf_dictionary.h | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) (limited to 'core/fpdfapi/parser/cpdf_dictionary.h') diff --git a/core/fpdfapi/parser/cpdf_dictionary.h b/core/fpdfapi/parser/cpdf_dictionary.h index fb8200f78c..4ef2f96ce7 100644 --- a/core/fpdfapi/parser/cpdf_dictionary.h +++ b/core/fpdfapi/parser/cpdf_dictionary.h @@ -26,6 +26,7 @@ class CPDF_Dictionary : public CPDF_Object { CPDF_Dictionary(); explicit CPDF_Dictionary(const CFX_WeakPtr& pPool); + ~CPDF_Dictionary() override; // CPDF_Object. Type GetType() const override; @@ -88,8 +89,6 @@ class CPDF_Dictionary : public CPDF_Object { CFX_WeakPtr GetByteStringPool() const { return m_pPool; } protected: - ~CPDF_Dictionary() override; - CFX_ByteString MaybeIntern(const CFX_ByteString& str); CPDF_Object* CloneNonCyclic( bool bDirect, @@ -99,9 +98,6 @@ class CPDF_Dictionary : public CPDF_Object { std::map m_Map; }; -using UniqueDictionary = - std::unique_ptr>; - inline CPDF_Dictionary* ToDictionary(CPDF_Object* obj) { return obj ? obj->AsDictionary() : nullptr; } @@ -110,12 +106,13 @@ inline const CPDF_Dictionary* ToDictionary(const CPDF_Object* obj) { return obj ? obj->AsDictionary() : nullptr; } -inline UniqueDictionary ToDictionary(UniqueObject obj) { +inline std::unique_ptr ToDictionary( + std::unique_ptr obj) { CPDF_Dictionary* pDict = ToDictionary(obj.get()); if (!pDict) return nullptr; obj.release(); - return UniqueDictionary(pDict); + return std::unique_ptr(pDict); } #endif // CORE_FPDFAPI_PARSER_CPDF_DICTIONARY_H_ -- cgit v1.2.3