X-Git-Url: https://git.madduck.net/etc/awesome.git/blobdiff_plain/8d37b9a1e4521a220f20ce2e1414f656aa3a5015..912bd26ede8901c98fd8d7a35f8493ee448109ad:/widgets/alsa.lua

diff --git a/widgets/alsa.lua b/widgets/alsa.lua
index 8571ec6..0bb2059 100644
--- a/widgets/alsa.lua
+++ b/widgets/alsa.lua
@@ -1,66 +1,77 @@
 
 --[[
-                                                      
-     Licensed under GNU General Public License v2     
-      * (c) 2013, Luke Bonham                         
-      * (c) 2010, Adrian C. <anrxc@sysphere.org>      
-                                                      
+                                                  
+     Licensed under GNU General Public License v2 
+      * (c) 2013, Luke Bonham                     
+      * (c) 2010, Adrian C. <anrxc@sysphere.org>  
+                                                  
 --]]
 
 local newtimer        = require("lain.helpers").newtimer
+local read_pipe       = require("lain.helpers").read_pipe
 
 local wibox           = require("wibox")
 
-local io              = { popen  = io.popen }
-local string          = { match  = string.match }
+local string          = { match  = string.match,
+                          format = string.format }
 
 local setmetatable    = setmetatable
 
 -- ALSA volume
 -- lain.widgets.alsa
-local alsa = {}
+local alsa = {
+    level  = "0",
+    status = "off",
+}
 
 local function worker(args)
     local args     = args or {}
-    local timeout  = args.timeout or 5
-    local channel  = args.channel or "Master"
+    local timeout  = args.timeout or 1
     local settings = args.settings or function() end
 
-    widget = wibox.widget.textbox('')
+    alsa.cmd     = args.cmd or "amixer"
+    alsa.channel = args.channel or "Master"
 
-    function update()
-        local f = io.popen('amixer get ' .. channel)
-        local mixer = f:read("*all")
-        f:close()
+    alsa.widget = wibox.widget.textbox('')
 
-        volume = {}
+    function alsa.update()
+        local mixer = read_pipe(string.format("%s get %s", alsa.cmd, alsa.channel))
 
-        volume.level, volume.status = string.match(mixer, "([%d]+)%%.*%[([%l]*)")
+        volume_now = {}
 
-        if volume.level == nil
+        volume_now.level, volume_now.status = string.match(mixer, "([%d]+)%%.*%[([%l]*)")
+
+        if volume_now.level == nil
         then
-            volume.level  = 0
-            volume.status = "off"
+            volume_now.level  = "0"
+            volume_now.status = "off"
         end
 
-        if volume.status == ""
+        if volume_now.status == ""
         then
-            if volume.level == 0
+            if volume_now.level == "0"
             then
-                volume.status = "off"
+                volume_now.status = "off"
             else
-                volume.status = "on"
+                volume_now.status = "on"
             end
         end
 
-        settings()
+        if alsa.level ~= volume_now.level or alsa.status ~= volume_now.status
+        then
+            widget = alsa.widget
+            settings()
+
+            alsa.level = volume_now.level
+            alsa.status = volume_now.status
+        end
     end
 
-    newtimer("alsa", timeout, update)
+    timer_id = string.format("alsa-%s-%s", alsa.cmd, alsa.channel)
 
-    output = { widget = widget, notify = update }
+    newtimer(timer_id, timeout, alsa.update)
 
-    return setmetatable(output, { __index = output.widget })
+    return setmetatable(alsa, { __index = alsa.widget })
 end
 
 return setmetatable(alsa, { __call = function(_, ...) return worker(...) end })