Some cosmetic fixes.

I tried to scrap the hide/reveal boilerplate, but that didn't work out
due different functions ({show,hide}Window) and signaltypes. Got almost
as ugly. Maybe a pattern matching function instead of the "case typ of"
would be nicer. But that's just code golfing.

1 files changed, 14 insertions, 16 deletions

diff --git a/src/Xmobar.hs b/src/Xmobar.hs
index d2d29ad..9bec6bc 100644
--- a/src/Xmobar.hs
+++ b/src/Xmobar.hs
@@ -156,24 +156,22 @@ eventLoop tv xc@(XConf d r w fs cfg) signal = do
     where
         isPersistent = not $ persistent cfg
 
-        hide t | t == 0    = do
-            when isPersistent $ hideWindow d w
-            eventLoop tv xc signal
-               | otherwise = do
-            void $ forkIO
-                 $ threadDelay t >> atomically (putTMVar signal $ Hide 0)
-            eventLoop tv xc signal
+        hide t
+            | t == 0 =
+                when isPersistent (hideWindow d w) >> eventLoop tv xc signal
+            | otherwise = do
+                void $ forkIO
+                     $ threadDelay t >> atomically (putTMVar signal $ Hide 0)
+                eventLoop tv xc signal
 
-        reveal t | t == 0 =
-            if isPersistent
-                then do
-                showWindow r cfg d w
+        reveal t
+            | t == 0 = do
+                when isPersistent (showWindow r cfg d w)
+                eventLoop tv xc signal
+            | otherwise = do
+                void $ forkIO
+                     $ threadDelay t >> atomically (putTMVar signal $ Reveal 0)
                 eventLoop tv xc signal
-            else eventLoop tv xc signal
-                 | otherwise = do
-            void $ forkIO
-                 $ threadDelay t >> atomically (putTMVar signal $ Reveal 0)
-            eventLoop tv xc signal
 
         toggle t = do
             ismapped <- isMapped d w