awesome/mpc: use a keep alive timer instead of ping on error

[dotfiles.git] / awesome / widgets / mpd_widget.lua

diff --git a/awesome/widgets/mpd_widget.lua b/awesome/widgets/mpd_widget.lua
index 9da5152e69a38bf1c6aed974f17d6d4f954d6959..2e4b215a0ad5ccf9989075c2e290f816f5f5bd8a 100644 (file)
--- a/awesome/widgets/mpd_widget.lua
+++ b/awesome/widgets/mpd_widget.lua
@@ -137,9 +137,7 @@ end
 
 function widget:error_handler(err)
     self._textbox:set_text("Error: " .. tostring(err))
-    timer.start_new(10, function()
-        self._connection:send("ping")
-    end)
+    self._status = {}
 end
 
 function widget:run()
@@ -150,9 +148,13 @@ function widget:run()
         "player", function(conn)
             conn:send("status", function(err, status)
                 local songchanged = self._status.songid ~= status.songid
+                local statechanged = self._status.state ~= status.state
                 self._status = status
                 if not songchanged then
                     self:update_widget()
+                    if statechanged then
+                        self:popup_oneshot(5)
+                    end
                     return
                 end
 
@@ -168,6 +170,13 @@ function widget:run()
         end
     )
 
+    self._keep_alive_timer = timer {
+      timeout = 1
+    , autostart = true
+    , callback = function() self._connection:connect() end
+    , call_now = true
+    }
+
     self._timer = timer({ timeout = 1 })
     self._timer:connect_signal("timeout", function() self:popup_update() end)