diff options
author | Tomas Janousek <tomi@nomi.cz> | 2020-11-18 18:49:00 +0000 |
---|---|---|
committer | jao <jao@gnu.org> | 2020-11-19 13:38:15 +0000 |
commit | 0cb56eb14e4e7c9257b290a8e22331d90145105c (patch) | |
tree | d599587abf1c5840a7279e1bbe5d116419915dfb /test/Xmobar/Plugins/Monitors | |
parent | b23fc95a3bc2c0fefba7a44d6f1a4e8e0941d77c (diff) | |
download | xmobar-0cb56eb14e4e7c9257b290a8e22331d90145105c.tar.gz xmobar-0cb56eb14e4e7c9257b290a8e22331d90145105c.tar.bz2 |
StdinReader: Remove throttling on exception
Introducing the throttling unfortunately has a negative side-effect: it
delays all stdin processing, including EOF detection, which can cause
confusion the previous commit tries to fix.
The only benefit of the throttling is to prevent 100% CPU usage when a
lot of garbage is provided on xmobar stdin. We still don't know where
that garbage comes from in https://github.com/jaor/xmobar/issues/438, or
rather why there's more than a handful of lines of such garbage. @psibi
has since fixed his setup to not produce that garbage, and no one else
ever reported such a problem, so it's probably safe to ignore it for
now.
Should anyone ever encounter that again, feel free to ping me, even in
the middle of the night, to help debug this.
Fixes: 7759df11f746 ("StdinReader: Improve exception handling")
Fixes: b7a3d6745817 ("Avoid busy looping by not catching all exceptions")
Diffstat (limited to 'test/Xmobar/Plugins/Monitors')
0 files changed, 0 insertions, 0 deletions