diff options
author | Mathias Gumz <akira at fluxbox dot org> | 2013-01-12 08:24:11 (GMT) |
---|---|---|
committer | Mathias Gumz <akira at fluxbox dot org> | 2013-01-12 08:24:11 (GMT) |
commit | 4d307dcd10af9d817ff5c05fc40ae7487564cb31 (patch) | |
tree | 97d4e32d99c90ee5fe1e5036ad15dedd280a09b4 /src/ToolbarItem.cc | |
parent | 06655f6d7ff2af0626d37f083b927af2af1be529 (diff) | |
download | fluxbox-4d307dcd10af9d817ff5c05fc40ae7487564cb31.zip fluxbox-4d307dcd10af9d817ff5c05fc40ae7487564cb31.tar.bz2 |
Fix bug: handle the list of Timers not in-place
With commit 541c8c4 we switched from an (manually) ordered list to a
std::set<> to handle the active timers. The code which checks for overdue
timers now traverses and modifies the std::set<> in place. This might
lead to an infinite loop. Examples of such bad behavior are "flickering of
the tooltip" (bug #3590078) or crashes (bug #3600143) or just insanely high
cpu load when autoraising windows or submenus.
We now make a copy of the std::set<> traverse this instead of the original.
Diffstat (limited to 'src/ToolbarItem.cc')
0 files changed, 0 insertions, 0 deletions