X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/a81258bee4e20d3fa9b3d2f6bf3bcd1200d1e5e0..8e54aaedb5329a49ffd9c5d50071f9c8cdd5c59e:/src/gtk1/timer.cpp diff --git a/src/gtk1/timer.cpp b/src/gtk1/timer.cpp index 61de7eeb11..0353bede8b 100644 --- a/src/gtk1/timer.cpp +++ b/src/gtk1/timer.cpp @@ -1,67 +1,91 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: timer.cpp -// Purpose: +// Name: gtk/timer.cpp +// Purpose: wxTimer implementation // Author: Robert Roebling // Id: $Id$ // Copyright: (c) 1998 Robert Roebling -// Licence: wxWindows licence +// Licence: wxWindows licence ///////////////////////////////////////////////////////////////////////////// -#ifdef __GNUG__ +#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA) #pragma implementation "timer.h" #endif +// For compilers that support precompilation, includes "wx.h". +#include "wx/wxprec.h" + +#if wxUSE_TIMER + #include "wx/timer.h" -//----------------------------------------------------------------------------- +#include "gtk/gtk.h" + +// ---------------------------------------------------------------------------- // wxTimer -//----------------------------------------------------------------------------- +// ---------------------------------------------------------------------------- -IMPLEMENT_ABSTRACT_CLASS(wxTimer,wxObject) +IMPLEMENT_ABSTRACT_CLASS(wxTimer, wxObject) -gint timeout_callback( gpointer data ) +extern "C" gint timeout_callback( gpointer data ) { - wxTimer *timer = (wxTimer*)data; - timer->Notify(); - - if ( timer->OneShot() ) - timer->Stop(); - - return TRUE; + wxTimer *timer = (wxTimer*)data; + + // Don't change the order of anything in this callback! + + if (timer->IsOneShot()) + { + // This sets m_tag to -1 + timer->Stop(); + } + + // When getting called from GDK's timer handler we + // are no longer within GDK's grab on the GUI + // thread so we must lock it here ourselves. + gdk_threads_enter(); + + timer->Notify(); + + // Release lock again. + gdk_threads_leave(); + + if (timer->IsOneShot()) + return FALSE; + + return TRUE; } -wxTimer::wxTimer() +void wxTimer::Init() { - m_tag = -1; - m_time = 1000; - m_oneShot = FALSE; + m_tag = -1; + m_milli = 1000; } wxTimer::~wxTimer() { - Stop(); + wxTimer::Stop(); } bool wxTimer::Start( int millisecs, bool oneShot ) { - if ( millisecs != -1 ) - m_time = millisecs; + (void)wxTimerBase::Start(millisecs, oneShot); - m_oneShot = oneShot; + if (m_tag != -1) + gtk_timeout_remove( m_tag ); - m_tag = gtk_timeout_add( millisecs, timeout_callback, this ); + m_tag = gtk_timeout_add( m_milli, timeout_callback, this ); - return TRUE; + return TRUE; } void wxTimer::Stop() { - if ( m_tag != -1 ) - { - gtk_timeout_remove( m_tag ); - - m_tag = -1; - } + if (m_tag != -1) + { + gtk_timeout_remove( m_tag ); + m_tag = -1; + } } +#endif // wxUSE_TIMER +