Fix minor typo in wxDV_VERT_RULES documentation.

[wxWidgets.git] / src / unix / timerunx.cpp

diff --git a/src/unix/timerunx.cpp b/src/unix/timerunx.cpp
index d6ee503df5cf7210437ef27227140b9479f5a706..7ee1177c89adabebc16d56d8b0ecfae742c9cc65 100644 (file)
--- a/src/unix/timerunx.cpp
+++ b/src/unix/timerunx.cpp
@@ -31,6 +31,8 @@
 
 #include "wx/apptrait.h"
 #include "wx/longlong.h"
 
 #include "wx/apptrait.h"
 #include "wx/longlong.h"

+#include "wx/time.h"
+#include "wx/vector.h"

 
 #include <sys/time.h>
 #include <signal.h>
 
 #include <sys/time.h>
 #include <signal.h>


@@ -53,7 +55,7 @@ static inline wxString wxUsecClockAsString(wxUsecClock_t usec)
     #if wxUSE_LONGLONG
         return usec.ToString();
     #else // wxUsecClock_t == double
     #if wxUSE_LONGLONG
         return usec.ToString();
     #else // wxUsecClock_t == double

-        return wxString::Format(_T("%.0f"), usec);
+        return wxString::Format(wxT("%.0f"), usec);

     #endif
 }
 
     #endif
 }
 


@@ -85,7 +87,7 @@ void wxTimerScheduler::DoAddTimer(wxTimerSchedule *s)
     for ( node = m_timers.begin(); node != m_timers.end(); ++node )
     {
         wxASSERT_MSG( (*node)->m_timer != s->m_timer,
     for ( node = m_timers.begin(); node != m_timers.end(); ++node )
     {
         wxASSERT_MSG( (*node)->m_timer != s->m_timer,

-                      _T("adding the same timer twice?") );
+                      wxT("adding the same timer twice?") );

 
         if ( (*node)->m_expiration > s->m_expiration )
             break;
 
         if ( (*node)->m_expiration > s->m_expiration )
             break;


@@ -114,7 +116,7 @@ void wxTimerScheduler::RemoveTimer(wxUnixTimerImpl *timer)
         }
     }
 
         }
     }
 

-    wxFAIL_MSG( _T("removing inexistent timer?") );
+    wxFAIL_MSG( wxT("removing inexistent timer?") );

 }
 
 bool wxTimerScheduler::GetNext(wxUsecClock_t *remaining) const
 }
 
 bool wxTimerScheduler::GetNext(wxUsecClock_t *remaining) const


@@ -122,9 +124,9 @@ bool wxTimerScheduler::GetNext(wxUsecClock_t *remaining) const
     if ( m_timers.empty() )
       return false;
 
     if ( m_timers.empty() )
       return false;
 

-    wxCHECK_MSG( remaining, false, _T("NULL pointer") );
+    wxCHECK_MSG( remaining, false, wxT("NULL pointer") );

 
 

-    *remaining = (*m_timers.begin())->m_expiration - wxGetLocalTimeUsec();
+    *remaining = (*m_timers.begin())->m_expiration - wxGetUTCTimeUSec();

     if ( *remaining < 0 )
     {
         // timer already expired, don't wait at all before notifying it
     if ( *remaining < 0 )
     {
         // timer already expired, don't wait at all before notifying it


@@ -139,13 +141,12 @@ bool wxTimerScheduler::NotifyExpired()
     if ( m_timers.empty() )
       return false;
 
     if ( m_timers.empty() )
       return false;
 

-    bool notified = false;
+    const wxUsecClock_t now = wxGetUTCTimeUSec();

 
 

-    const wxUsecClock_t now = wxGetLocalTimeUsec();
-
-    wxTimerList::iterator cur,
-                          next;
-    for ( cur = m_timers.begin(); cur != m_timers.end(); cur = next )
+    typedef wxVector<wxUnixTimerImpl *> TimerImpls;
+    TimerImpls toNotify;
+    for ( wxTimerList::iterator next,
+            cur = m_timers.begin(); cur != m_timers.end(); cur = next )

     {
         wxTimerSchedule * const s = *cur;
         if ( s->m_expiration > now )
     {
         wxTimerSchedule * const s = *cur;
         if ( s->m_expiration > now )


@@ -182,12 +183,25 @@ bool wxTimerScheduler::NotifyExpired()
             DoAddTimer(s);
         }
 
             DoAddTimer(s);
         }
 

-        // and finally notify the timer
-        timer->Notify();
-        notified = true;
+        // we can't notify the timer from this loop as the timer event handler
+        // could modify m_timers (for example, but not only, by stopping this
+        // timer) which would render our iterators invalid, so do it after the
+        // loop end
+        toNotify.push_back(timer);
+    }
+
+    if ( toNotify.empty() )
+        return false;
+
+    for ( TimerImpls::const_iterator i = toNotify.begin(),
+                                     end = toNotify.end();
+          i != end;
+          ++i )
+    {
+        (*i)->Notify();

     }
 
     }
 

-    return notified;
+    return true;

 }
 
 // ============================================================================
 }
 
 // ============================================================================


@@ -205,7 +219,7 @@ bool wxUnixTimerImpl::Start(int milliseconds, bool oneShot)
     // notice that this will stop an already running timer
     wxTimerImpl::Start(milliseconds, oneShot);
 
     // notice that this will stop an already running timer
     wxTimerImpl::Start(milliseconds, oneShot);
 

-    wxTimerScheduler::Get().AddTimer(this, wxGetLocalTimeUsec() + m_milli*1000);
+    wxTimerScheduler::Get().AddTimer(this, wxGetUTCTimeUSec() + m_milli*1000);

     m_isRunning = true;
 
     return true;
     m_isRunning = true;
 
     return true;


@@ -228,7 +242,7 @@ bool wxUnixTimerImpl::IsRunning() const
 
 wxUnixTimerImpl::~wxUnixTimerImpl()
 {
 
 wxUnixTimerImpl::~wxUnixTimerImpl()
 {

-    wxASSERT_MSG( !m_isRunning, _T("must have been stopped before") );
+    wxASSERT_MSG( !m_isRunning, wxT("must have been stopped before") );

 }
 
 // ============================================================================
 }
 
 // ============================================================================


@@ -251,21 +265,6 @@ IMPLEMENT_DYNAMIC_CLASS(wxTimerUnixModule, wxModule)
 // global functions
 // ============================================================================
 
 // global functions
 // ============================================================================
 

-wxUsecClock_t wxGetLocalTimeUsec()
-{
-#ifdef HAVE_GETTIMEOFDAY
-    struct timeval tv;
-    if ( wxGetTimeOfDay(&tv) != -1 )
-    {
-        wxUsecClock_t val = 1000000L; // usec/sec
-        val *= tv.tv_sec;
-        return val + tv.tv_usec;
-    }
-#endif // HAVE_GETTIMEOFDAY
-
-    return wxGetLocalTimeMillis() * 1000L;
-}
-

 wxTimerImpl *wxConsoleAppTraits::CreateTimerImpl(wxTimer *timer)
 {
     return new wxUnixTimerImpl(timer);
 wxTimerImpl *wxConsoleAppTraits::CreateTimerImpl(wxTimer *timer)
 {
     return new wxUnixTimerImpl(timer);