xmobar/src/Xmobar/System, branch 0.38

StdinReader: Improve exception handling

2020-11-19T13:38:15Z

This corrects a misleading comment "EOF check is necessary for certain systems" which was added without complete understanding of the root cause of #442. That issue was in fact caused by old xmobars not being terminated on early EOF, and is thus necessary on _all_ systems that rely on EOF to terminate old xmobar before starting a new one. (To trigger the bug, one additionally needs to close the xmobar pipe before sending any input to it, which is unusual, but incorrectly configured xmonad might trigger that.) Furthermore, this fixes another execution path that could lead to xmobar not being terminated on EOF: `echo -e '\xff' | xmobar -c '[Run StdinReader]' -t '%StdinReader%'` would terminate the StdinReader thread upon catching the "invalid argument (invalid byte sequence)" so there'd be no thread to detect the subsequent EOF and xmobar would get stuck. Additionally, I believe that terminating either the thread or the entire xmobar upon receiving a single miscoded byte isn't desirable, as this might be an intermittent issue and another input line can be perfectly okay. Therefore I suggest that the original issue @psibi was trying to fix by b7a3d6745817 is worked around by introducing a throttling delay instead of terminating the thread, as I assume that exceptions other than async and EOF are recoverable. Fixes: b7a3d6745817 ("Avoid busy looping by not catching all exceptions") Fixes: 68ac4d3ae6f3 ("Update stderr and the bar on receiving exception") Fixes: ed0663aac942 ("Add EOF check before getLine operation from stdin") Fixes: https://github.com/jaor/xmobar/issues/442 Related: https://github.com/jaor/xmobar/pull/439 Related: https://github.com/jaor/xmobar/pull/448

Redundant imports (mostly <$>) removed

2020-08-07T02:33:15Z

more hlinting

2020-05-11T18:32:06Z

hlinting

2020-05-11T18:22:41Z

Fix crashes/busy looping happening via index

2020-05-10T14:13:56Z

Right now, with the `StdinReader` plugin enabled - you can crash/cause busy looping of xmobar if the following html file is opened: ``` hello <fn=1>string</fn> ``` More details about this bug is here: https://github.com/jaor/xmobar/issues/442#issuecomment-625706001 This MR also fixes another bug which produces a crash in xmobar if you pass non integer items between fn:

Update stderr and the bar on receiving exception

2020-05-02T15:34:52Z

Spurious import and hlinting

2020-04-30T17:06:11Z

Refactor the usage of hGetLineSafe

2020-04-30T15:45:31Z

hGetLineSafe is always hGetLine and hence we can directly use it.

Xmobar.System.Utils, Xmobar.X11.Events

2018-11-25T23:43:41Z

Back to app/src, since it seems they're the default convention for stack

2018-11-25T15:10:29Z