pgrondek/polybar-dwm
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(xworkspaces): Never swallow updates (#2275)

Browse Source 
If two WM events arrive withing 25ms of one-another, the second one does
not trigger a bar update.
The module state is still correct, it is just not reflected in the bar.

This somehow caused updates being swallowed in fluxbox, but only after
PR #882 was merged, even though that 25ms restriction existed long
before that.

Fixes #2272
This commit is contained in:
Patrick Ziegler 2020-12-05 23:28:48 +01:00 committed by GitHub
parent 82ebad5e7a
commit 037f6e6c45
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
include/modules/xworkspaces.hpp
View File

@ -110,8 +110,6 @@ namespace modules {
// The following mutex is here to protect the data of this modules. // The following mutex is here to protect the data of this modules.
// This can't be achieved using m_buildlock since we "CRTP override" get_output(). // This can't be achieved using m_buildlock since we "CRTP override" get_output().
mutable mutex m_workspace_mutex; mutable mutex m_workspace_mutex;
event_timer m_timer{0L, 25L};
}; };
} // namespace modules } // namespace modules

2
src/modules/xworkspaces.cpp
View File

@ -126,10 +126,8 @@ namespace modules {
return; return;
} }
if (m_timer.allow(evt->time)) {
broadcast(); broadcast();
} }
}
/** /**
* Rebuild the list of managed clients * Rebuild the list of managed clients