From 104fa7c879398c67cdaab3f7064ca9f6da07b072 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Thu, 25 Feb 2016 14:42:01 -0800 Subject: Fix static initializers. R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1725303003 . --- fpdfsdk/include/fpdfxfa/fpdfxfa_util.h | 2 +- fpdfsdk/src/fpdfxfa/fpdfxfa_util.cpp | 24 ++++++++++++++++-------- 2 files changed, 17 insertions(+), 9 deletions(-) (limited to 'fpdfsdk') diff --git a/fpdfsdk/include/fpdfxfa/fpdfxfa_util.h b/fpdfsdk/include/fpdfxfa/fpdfxfa_util.h index 9678453db6..af69812471 100644 --- a/fpdfsdk/include/fpdfxfa/fpdfxfa_util.h +++ b/fpdfsdk/include/fpdfxfa/fpdfxfa_util.h @@ -31,7 +31,7 @@ class CXFA_FWLAdapterTimerMgr : public IFWL_AdapterTimerMgr { protected: static void TimerProc(int32_t idEvent); - static std::vector s_TimerArray; + static std::vector* s_TimerArray; CPDFDoc_Environment* const m_pEnv; }; diff --git a/fpdfsdk/src/fpdfxfa/fpdfxfa_util.cpp b/fpdfsdk/src/fpdfxfa/fpdfxfa_util.cpp index 03f1cfc358..c2a0db022c 100644 --- a/fpdfsdk/src/fpdfxfa/fpdfxfa_util.cpp +++ b/fpdfsdk/src/fpdfxfa/fpdfxfa_util.cpp @@ -8,7 +8,7 @@ #include "fpdfsdk/include/fsdk_define.h" #include "fpdfsdk/include/fsdk_mgr.h" -std::vector CXFA_FWLAdapterTimerMgr::s_TimerArray; +std::vector* CXFA_FWLAdapterTimerMgr::s_TimerArray = nullptr; FWL_ERR CXFA_FWLAdapterTimerMgr::Start(IFWL_Timer* pTimer, FX_DWORD dwElapse, @@ -18,8 +18,10 @@ FWL_ERR CXFA_FWLAdapterTimerMgr::Start(IFWL_Timer* pTimer, return FWL_ERR_Indefinite; uint32_t uIDEvent = m_pEnv->FFI_SetTimer(dwElapse, TimerProc); - s_TimerArray.push_back(new CFWL_TimerInfo(uIDEvent, pTimer)); - hTimer = reinterpret_cast(s_TimerArray.back()); + if (!s_TimerArray) + s_TimerArray = new std::vector; + s_TimerArray->push_back(new CFWL_TimerInfo(uIDEvent, pTimer)); + hTimer = reinterpret_cast(s_TimerArray->back()); return FWL_ERR_Succeeded; } @@ -29,16 +31,22 @@ FWL_ERR CXFA_FWLAdapterTimerMgr::Stop(FWL_HTIMER hTimer) { CFWL_TimerInfo* pInfo = reinterpret_cast(hTimer); m_pEnv->FFI_KillTimer(pInfo->uIDEvent); - auto it = std::find(s_TimerArray.begin(), s_TimerArray.end(), pInfo); - if (it != s_TimerArray.end()) { - s_TimerArray.erase(it); - delete pInfo; + if (s_TimerArray) { + auto it = std::find(s_TimerArray->begin(), s_TimerArray->end(), pInfo); + if (it != s_TimerArray->end()) { + s_TimerArray->erase(it); + delete pInfo; + } } return FWL_ERR_Succeeded; } +// static void CXFA_FWLAdapterTimerMgr::TimerProc(int32_t idEvent) { - for (CFWL_TimerInfo* pInfo : s_TimerArray) { + if (!s_TimerArray) + return; + + for (CFWL_TimerInfo* pInfo : *s_TimerArray) { if (pInfo->uIDEvent == idEvent) { pInfo->pTimer->Run(reinterpret_cast(pInfo)); break; -- cgit v1.2.3