Merge pull request #110 from dnedbaylo/master

[etc/awesome.git] / widgets / alsa.lua

diff --git a/widgets/alsa.lua b/widgets/alsa.lua
index 8571ec600834d6799a1a1063648779a3095902ef..7d250962374a2df2c78180bae4fbadbf092c183c 100644 (file)
--- a/widgets/alsa.lua
+++ b/widgets/alsa.lua
@@ -1,10 +1,10 @@
 
 --[[
 
 --[[

-                                                      
-     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 newtimer        = require("lain.helpers").newtimer


@@ -12,7 +12,8 @@ local newtimer        = require("lain.helpers").newtimer
 local wibox           = require("wibox")
 
 local io              = { popen  = io.popen }
 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
 
 
 local setmetatable    = setmetatable
 


@@ -23,44 +24,45 @@ local alsa = {}
 local function worker(args)
     local args     = args or {}
     local timeout  = args.timeout or 5
 local function worker(args)
     local args     = args or {}
     local timeout  = args.timeout or 5

-    local channel  = args.channel or "Master"

     local settings = args.settings or function() end
 
     local settings = args.settings or function() end
 

-    widget = wibox.widget.textbox('')
+    alsa.card    = args.card or "0"
+    alsa.channel = args.channel or "Master"

 
 

-    function update()
-        local f = io.popen('amixer get ' .. channel)
-        local mixer = f:read("*all")
+    alsa.widget = wibox.widget.textbox('')
+
+    function alsa.update()
+        local f = assert(io.popen(string.format("amixer -c %s -M get %s", alsa.card, alsa.channel)))
+        local mixer = f:read("*a")

         f:close()
 
         f:close()
 

-        volume = {}
+        volume_now = {}

 
 

-        volume.level, volume.status = string.match(mixer, "([%d]+)%%.*%[([%l]*)")
+        volume_now.level, volume_now.status = string.match(mixer, "([%d]+)%%.*%[([%l]*)")

 
 

-        if volume.level == nil
+        if volume_now.level == nil

         then
         then

-            volume.level  = 0
-            volume.status = "off"
+            volume_now.level  = "0"
+            volume_now.status = "off"

         end
 
         end
 

-        if volume.status == ""
+        if volume_now.status == ""

         then
         then

-            if volume.level == 0
+            if volume_now.level == "0"

             then
             then

-                volume.status = "off"
+                volume_now.status = "off"

             else
             else

-                volume.status = "on"
+                volume_now.status = "on"

             end
         end
 
             end
         end
 

+        widget = alsa.widget

         settings()
     end
 
         settings()
     end
 

-    newtimer("alsa", timeout, update)
-
-    output = { widget = widget, notify = update }
+    newtimer("alsa", timeout, alsa.update)

 
 

-    return setmetatable(output, { __index = output.widget })
+    return setmetatable(alsa, { __index = alsa.widget })

 end
 
 return setmetatable(alsa, { __call = function(_, ...) return worker(...) end })
 end
 
 return setmetatable(alsa, { __call = function(_, ...) return worker(...) end })