From f21b88ef08d6f6d64e783f55819fc8b86c3b370e Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Sat, 4 Aug 2018 01:51:46 +0000 Subject: Do more CBA_FontMap initialization in the ctor and header. Change-Id: I705a49198a91b67aa403beea4446095a6a359259 Reviewed-on: https://pdfium-review.googlesource.com/39415 Reviewed-by: Ryan Harrison Commit-Queue: Lei Zhang --- fpdfsdk/formfiller/cba_fontmap.cpp | 10 ++-------- fpdfsdk/formfiller/cba_fontmap.h | 6 +++--- 2 files changed, 5 insertions(+), 11 deletions(-) (limited to 'fpdfsdk/formfiller') diff --git a/fpdfsdk/formfiller/cba_fontmap.cpp b/fpdfsdk/formfiller/cba_fontmap.cpp index ae0cc80bf2..98464cb6d6 100644 --- a/fpdfsdk/formfiller/cba_fontmap.cpp +++ b/fpdfsdk/formfiller/cba_fontmap.cpp @@ -23,14 +23,8 @@ CBA_FontMap::CBA_FontMap(CPDFSDK_Annot* pAnnot, CFX_SystemHandler* pSystemHandler) : CPWL_FontMap(pSystemHandler), - m_pDocument(nullptr), - m_pAnnotDict(nullptr), - m_pDefaultFont(nullptr), - m_sAPType("N") { - CPDF_Page* pPage = pAnnot->GetPDFPage(); - - m_pDocument = pPage->GetDocument(); - m_pAnnotDict = pAnnot->GetPDFAnnot()->GetAnnotDict(); + m_pDocument(pAnnot->GetPDFPage()->GetDocument()), + m_pAnnotDict(pAnnot->GetPDFAnnot()->GetAnnotDict()) { Initialize(); } diff --git a/fpdfsdk/formfiller/cba_fontmap.h b/fpdfsdk/formfiller/cba_fontmap.h index 9d9f2e002a..0ca07eb984 100644 --- a/fpdfsdk/formfiller/cba_fontmap.h +++ b/fpdfsdk/formfiller/cba_fontmap.h @@ -36,11 +36,11 @@ class CBA_FontMap : public CPWL_FontMap { CPDF_Font* GetAnnotDefaultFont(ByteString* csNameTag); void AddFontToAnnotDict(CPDF_Font* pFont, const ByteString& sAlias); - UnownedPtr m_pDocument; - UnownedPtr m_pAnnotDict; + UnownedPtr const m_pDocument; + UnownedPtr const m_pAnnotDict; UnownedPtr m_pDefaultFont; ByteString m_sDefaultFontName; - ByteString m_sAPType; + ByteString m_sAPType = "N"; }; #endif // FPDFSDK_FORMFILLER_CBA_FONTMAP_H_ -- cgit v1.2.3