From 8e557bd9e7c09f9ce75fbea42f826b74d5cdc54b Mon Sep 17 00:00:00 2001 From: weili Date: Mon, 15 Aug 2016 12:25:29 -0700 Subject: Clean up XFA locale and locale manager Return unique_ptr for GetLocale(), directly use destructors instead of Release() functions, use vectors to manage arrays. Review-Url: https://codereview.chromium.org/2241863002 --- xfa/fgas/localization/fgas_locale.h | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) (limited to 'xfa/fgas') diff --git a/xfa/fgas/localization/fgas_locale.h b/xfa/fgas/localization/fgas_locale.h index 3401d6ca5a..cc565a6d77 100644 --- a/xfa/fgas/localization/fgas_locale.h +++ b/xfa/fgas/localization/fgas_locale.h @@ -54,12 +54,9 @@ enum FX_DATETIMETYPE { class IFX_Locale { public: - virtual ~IFX_Locale() {} - virtual void Release() = 0; - - virtual CFX_WideString GetName() = 0; + virtual CFX_WideString GetName() const = 0; virtual void GetNumbericSymbol(FX_LOCALENUMSYMBOL eType, CFX_WideString& wsNumSymbol) const = 0; virtual void GetDateTimeSymbols(CFX_WideString& wsDtSymbol) const = 0; @@ -85,11 +82,13 @@ class IFX_Locale { class IFX_LocaleMgr { public: virtual ~IFX_LocaleMgr() {} - virtual void Release() = 0; - virtual uint16_t GetDefLocaleID() = 0; + + virtual uint16_t GetDefLocaleID() const = 0; virtual IFX_Locale* GetDefLocale() = 0; - virtual IFX_Locale* GetLocale(uint16_t lcid) = 0; virtual IFX_Locale* GetLocaleByName(const CFX_WideString& wsLocaleName) = 0; + + protected: + virtual std::unique_ptr GetLocale(uint16_t lcid) = 0; }; FX_BOOL FX_DateFromCanonical(const CFX_WideString& wsDate, -- cgit v1.2.3