diff options
| author | Mathias Gumz <akira at fluxbox dot org> | 2013-01-15 20:40:33 (GMT) |
|---|---|---|
| committer | Mathias Gumz <akira at fluxbox dot org> | 2013-01-15 20:40:33 (GMT) |
| commit | 33161595f482d0dad950ce127b6016eefe8ea691 (patch) | |
| tree | 27fa5588b2ddd7410eb083d7bf0c8811d72f8270 /src/FbTk/MenuSeparator.cc | |
| parent | 239e895826b2f843bc50cc6fef8108db174c33e8 (diff) | |
| download | fluxbox-33161595f482d0dad950ce127b6016eefe8ea691.zip fluxbox-33161595f482d0dad950ce127b6016eefe8ea691.tar.bz2 | |
Simplifies and fix bugs in FbTk::Timer
* Calling Timer::setTimeout() from within Timer::start() might lead to ugly
behavior (as experienced in bugs #3590078, #3600143, etc; see commit
4d307dcd10af9d817ff5c05fc40ae7487564cb31, fixes the problem partially).
* Stop a timer first, then call the handler (via Timer::fireTimeout()). A
given handler might call Timer::start() again, which (re)adds the Timer
to the control list .. the following Timer::stop() would remove it again.
* Use 'm_start' as indicator if timer is running.
* Move the (now quite short) code of ::addTimer / ::removeTimer
into the Timer::start() and Timer::stop() functions.
Diffstat (limited to 'src/FbTk/MenuSeparator.cc')
0 files changed, 0 insertions, 0 deletions