X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/04cd30dea218e6a604521d55e3057b5ca9d3e9d2..b5435dccd098ae153c3b7b3b8fca8dd0d763986c:/src/msw/timer.cpp diff --git a/src/msw/timer.cpp b/src/msw/timer.cpp index b339a8f040..2c9f4d79e3 100644 --- a/src/msw/timer.cpp +++ b/src/msw/timer.cpp @@ -5,8 +5,8 @@ // Modified by: // Created: 04/01/98 // RCS-ID: $Id$ -// Copyright: (c) Julian Smart and Markus Holzem -// Licence: wxWindows license +// Copyright: (c) Julian Smart +// Licence: wxWindows licence ///////////////////////////////////////////////////////////////////////////// #ifdef __GNUG__ @@ -23,7 +23,6 @@ #if wxUSE_TIMER #ifndef WX_PRECOMP - #include "wx/setup.h" #include "wx/window.h" #include "wx/list.h" #include "wx/event.h" @@ -32,6 +31,8 @@ #include "wx/log.h" #endif +#include "wx/hashmap.h" + #include "wx/timer.h" #include "wx/msw/private.h" @@ -40,8 +41,15 @@ // private functions // ---------------------------------------------------------------------------- -wxList wxTimerList(wxKEY_INTEGER); -UINT WINAPI _EXPORT wxTimerProc(HWND hwnd, WORD, int idTimer, DWORD); +WX_DECLARE_HASH_MAP( long, + wxTimer *, + wxIntegerHash, + wxIntegerEqual, + wxTimerMap ); + +wxTimerMap wxTimerList; + +void WINAPI _EXPORT wxTimerProc(HWND hwnd, WORD, int idTimer, DWORD); // ---------------------------------------------------------------------------- // macros @@ -75,9 +83,11 @@ void wxTimer::Init() wxTimer::~wxTimer() { + long id = m_id; + wxTimer::Stop(); - wxTimerList.DeleteObject(this); + wxTimerList.erase(id); } bool wxTimer::Start(int milliseconds, bool oneShot) @@ -86,15 +96,20 @@ bool wxTimer::Start(int milliseconds, bool oneShot) wxCHECK_MSG( m_milli > 0, false, wxT("invalid value for timer timeour") ); +#ifdef __WXWINCE__ + m_id = ::SetTimer(NULL, (UINT)(m_id ? m_id : 1), + (UINT)m_milli, (TIMERPROC) wxTimerProc); +#else TIMERPROC wxTimerProcInst = (TIMERPROC) MakeProcInstance((FARPROC)wxTimerProc, wxGetInstance()); m_id = ::SetTimer(NULL, (UINT)(m_id ? m_id : 1), (UINT)m_milli, wxTimerProcInst); +#endif if ( m_id > 0 ) { - wxTimerList.Append(m_id, this); + wxTimerList[m_id] = this; return true; } @@ -112,7 +127,7 @@ void wxTimer::Stop() { ::KillTimer(NULL, (UINT)m_id); - wxTimerList.DeleteObject(this); + wxTimerList.erase(m_id); } m_id = 0; @@ -134,15 +149,16 @@ void wxProcessTimer(wxTimer& timer) timer.Notify(); } -UINT WINAPI _EXPORT wxTimerProc(HWND WXUNUSED(hwnd), WORD, int idTimer, DWORD) +void WINAPI _EXPORT wxTimerProc(HWND WXUNUSED(hwnd), WORD, int idTimer, DWORD) { - wxNode *node = wxTimerList.Find((long)idTimer); + + wxTimerMap::iterator node = wxTimerList.find((long)idTimer); - wxCHECK_MSG( node, 0, wxT("bogus timer id in wxTimerProc") ); + wxASSERT_MSG( node != wxTimerList.end(), wxT("bogus timer id in wxTimerProc") ); - wxProcessTimer(*(wxTimer *)node->GetData()); + wxProcessTimer(*(node->second)); - return 0; + // return 0; } #endif // wxUSE_TIMER