From f8d8ac4fca2774550f3fbd3a21bbc1e326028c18 Mon Sep 17 00:00:00 2001 From: dsinclair Date: Thu, 19 May 2016 11:37:20 -0700 Subject: Expand some typedefs to the original types Sevearal typedefs are only used once or twice. Expand and remove the typedef. Review-Url: https://codereview.chromium.org/1995033002 --- xfa/fde/css/fde_csscache.h | 2 +- xfa/fgas/crt/fgas_utils.h | 2 -- xfa/fgas/font/fgas_gefont.cpp | 18 +++++++----------- xfa/fgas/font/fgas_gefont.h | 2 +- xfa/fgas/font/fgas_stdfontmgr.h | 21 ++++++--------------- 5 files changed, 15 insertions(+), 30 deletions(-) diff --git a/xfa/fde/css/fde_csscache.h b/xfa/fde/css/fde_csscache.h index 2cf007db79..28962e45eb 100644 --- a/xfa/fde/css/fde_csscache.h +++ b/xfa/fde/css/fde_csscache.h @@ -46,7 +46,7 @@ class FDE_CSSTagCache : public CFX_Target { uint32_t dwIDHash; uint32_t dwTagHash; int32_t iClassIndex; - CFDE_DWordArray dwClassHashs; + CFX_BaseArrayTemplate dwClassHashs; }; typedef CFX_ObjectStackTemplate CFDE_CSSTagStack; diff --git a/xfa/fgas/crt/fgas_utils.h b/xfa/fgas/crt/fgas_utils.h index 56fa018ad4..6fef23d968 100644 --- a/xfa/fgas/crt/fgas_utils.h +++ b/xfa/fgas/crt/fgas_utils.h @@ -80,7 +80,6 @@ class CFX_BaseArrayTemplate : public CFX_BaseArray { CFX_BaseArray::RemoveAll(bLeaveMemory); } }; -typedef CFX_BaseArrayTemplate CFDE_DWordArray; template class CFX_ObjectBaseArrayTemplate : public CFX_BaseArray { @@ -275,7 +274,6 @@ class CFX_MassArrayTemplate : public CFX_BaseMassArray { CFX_BaseMassArray::RemoveAll(bLeaveMemory); } }; -typedef CFX_MassArrayTemplate CFX_RectMassArray; template class CFX_ObjectMassArrayTemplate : public CFX_BaseMassArray { diff --git a/xfa/fgas/font/fgas_gefont.cpp b/xfa/fgas/font/fgas_gefont.cpp index 5dfdb70018..c877080ec1 100644 --- a/xfa/fgas/font/fgas_gefont.cpp +++ b/xfa/fgas/font/fgas_gefont.cpp @@ -299,24 +299,20 @@ FX_BOOL CFX_GEFont::LoadFont(CFX_Font* pExtFont, FX_BOOL bTakeOver) { return bRet; } FX_BOOL CFX_GEFont::InitFont() { - if (!m_pFont) { + if (!m_pFont) return FALSE; - } if (!m_pFontEncoding) { m_pFontEncoding = FX_CreateFontEncodingEx(m_pFont); - if (!m_pFontEncoding) { + if (!m_pFontEncoding) return FALSE; - } } - if (!m_pCharWidthMap) { + if (!m_pCharWidthMap) m_pCharWidthMap = new CFX_WordDiscreteArray(1024); - } - if (!m_pRectArray) { - m_pRectArray = new CFX_RectMassArray(16); - } - if (!m_pBBoxMap) { + if (!m_pRectArray) + m_pRectArray = new CFX_MassArrayTemplate(16); + if (!m_pBBoxMap) m_pBBoxMap = new CFX_MapPtrToPtr(16); - } + return TRUE; } IFX_Font* CFX_GEFont::Derive(uint32_t dwFontStyles, uint16_t wCodePage) { diff --git a/xfa/fgas/font/fgas_gefont.h b/xfa/fgas/font/fgas_gefont.h index 491f38bc89..8595782dc3 100644 --- a/xfa/fgas/font/fgas_gefont.h +++ b/xfa/fgas/font/fgas_gefont.h @@ -70,7 +70,7 @@ class CFX_GEFont : public IFX_Font { IFX_FileRead* m_pFileRead; CFX_UnicodeEncoding* m_pFontEncoding; CFX_WordDiscreteArray* m_pCharWidthMap; - CFX_RectMassArray* m_pRectArray; + CFX_MassArrayTemplate* m_pRectArray; CFX_MapPtrToPtr* m_pBBoxMap; CXFA_PDFFontMgr* m_pProvider; uint16_t m_wCharSet; diff --git a/xfa/fgas/font/fgas_stdfontmgr.h b/xfa/fgas/font/fgas_stdfontmgr.h index 268c61f6ea..bbfa875e5e 100644 --- a/xfa/fgas/font/fgas_stdfontmgr.h +++ b/xfa/fgas/font/fgas_stdfontmgr.h @@ -142,15 +142,6 @@ class CFX_FontSourceEnum_File { CFX_ByteStringArray m_FolderPaths; }; -typedef CFX_MapPtrTemplate CFX_HashFileMap; -typedef CFX_MapPtrTemplate CFX_HashFontMap; -typedef CFX_MapPtrTemplate - CFX_HashFontDescsMap; -typedef CFX_MapPtrTemplate*> - CFX_HashFontsMap; -typedef CFX_MapPtrTemplate CFX_UnicodeFontMap; -typedef CFX_MapPtrTemplate CFX_FonStreamtMap; - class CFX_FontMgrImp : public IFX_FontMgr { public: CFX_FontMgrImp(CFX_FontSourceEnum_File* pFontEnum); @@ -235,12 +226,12 @@ class CFX_FontMgrImp : public IFX_FontMgr { uint32_t index); IFX_FileRead* CreateFontStream(const CFX_ByteString& bsFaceName); - CFX_HashFontDescsMap m_Hash2CandidateList; - CFX_HashFontsMap m_Hash2Fonts; - CFX_HashFileMap m_Hash2FileAccess; - CFX_HashFontMap m_FileAccess2IFXFont; - CFX_FonStreamtMap m_IFXFont2FileRead; - CFX_UnicodeFontMap m_FailedUnicodes2NULL; + CFX_MapPtrTemplate m_Hash2CandidateList; + CFX_MapPtrTemplate*> m_Hash2Fonts; + CFX_MapPtrTemplate m_Hash2FileAccess; + CFX_MapPtrTemplate m_FileAccess2IFXFont; + CFX_MapPtrTemplate m_IFXFont2FileRead; + CFX_MapPtrTemplate m_FailedUnicodes2NULL; CFX_FontSourceEnum_File* m_pFontSource; }; #endif -- cgit v1.2.3