diff options
author | Martin Sustrik <sustrik@250bpm.com> | 2010-09-26 21:42:23 +0200 |
---|---|---|
committer | Martin Sustrik <sustrik@250bpm.com> | 2010-09-26 21:42:23 +0200 |
commit | 238640a526c419392bf2df95de196db89ea6eb73 (patch) | |
tree | 8434dfc49117739f1d344a3d96a4957f6206d9d0 /src/poll.cpp | |
parent | 8d7bf6684cbb9625ec7c963b8867e2411b49eb57 (diff) |
timers properly implemented
Diffstat (limited to 'src/poll.cpp')
-rw-r--r-- | src/poll.cpp | 33 |
1 files changed, 7 insertions, 26 deletions
diff --git a/src/poll.cpp b/src/poll.cpp index 9afa6b5..1cab5bb 100644 --- a/src/poll.cpp +++ b/src/poll.cpp @@ -109,18 +109,6 @@ void zmq::poll_t::reset_pollout (handle_t handle_) pollset [index].events &= ~((short) POLLOUT); } -void zmq::poll_t::add_timer (int timeout_, i_poll_events *events_, int id_) -{ - timers.push_back (events_); -} - -void zmq::poll_t::cancel_timer (i_poll_events *events_, int id_) -{ - timers_t::iterator it = std::find (timers.begin (), timers.end (), events_); - if (it != timers.end ()) - timers.erase (it); -} - void zmq::poll_t::start () { worker.start (worker_routine, this); @@ -135,27 +123,20 @@ void zmq::poll_t::loop () { while (!stopping) { + // Execute any due timers. + uint64_t timeout = execute_timers (); + // Wait for events. - int rc = poll (&pollset [0], pollset.size (), - timers.empty () ? -1 : max_timer_period); + int rc = poll (&pollset [0], pollset.size (), timeout ? timeout : -1); if (rc == -1 && errno == EINTR) continue; errno_assert (rc != -1); - // Handle timer. - if (!rc) { - - // Use local list of timers as timer handlers may fill new timers - // into the original array. - timers_t t; - std::swap (timers, t); - - // Trigger all the timers. - for (timers_t::iterator it = t.begin (); it != t.end (); it ++) - (*it)->timer_event (-1); + // If there are no events (i.e. it's a timeout) there's no point + // in checking the pollset. + if (rc == 0) continue; - } for (pollset_t::size_type i = 0; i != pollset.size (); i++) { |