Merge branch 'master' of http://git.sysphere.org/vicious

TODO

@@ -5,11 +5,11 @@
 
 
 * Vicious
+** TODO Consider commiting power drain support to bat.lua
+** TODO Try replacing dio.lua with io.lua
+*** TODO First io.lua should grab data for all devices
 ** TODO Document contrib widgets in contrib/README
 ** TODO Consider multigraph, graph stacking, support
-** TODO Consider including this code in format helper
-   - return format and format:gsub("${[^}]+}", "")
-   - Note: We should then also replace just $1/$2...
 ** TODO Complete the hddtemp fix
    - In certain setups regexp does not match all devices
      - The regexp catches the first device name, but last stats

helpers.lua

@@ -9,6 +9,7 @@
 
 -- {{{ Grab environment
 local pairs = pairs
+local tonumber = tonumber
 local io = { open = io.open }
 local setmetatable = setmetatable
 local getmetatable = getmetatable
@@ -53,7 +54,9 @@ end
 -- {{{ Format a string with args
 function format(format, args)
     for var, val in pairs(args) do
-        format = format:gsub("$" .. var, val)
+        format = format:gsub("$" .. (tonumber(var) and var or
+            var:gsub("[-+?*]", function(i) return "%"..i end)),
+        val)
     end
 
     return format

widgets/bat.lua

@@ -57,21 +57,26 @@ local function worker(format, warg)
     -- Get charge information
     if battery.current_now then
         rate = battery.current_now
-    else -- Todo: other rate sources, as with capacity?
+    elseif battery.power_now then
+        rate = battery.power_now
+    else
         return {state, percent, "N/A"}
     end
 
     -- Calculate remaining (charging or discharging) time
-    if state == "+" then
+    local time = "N/A"
-        timeleft = (tonumber(capacity) - tonumber(remaining)) / tonumber(rate)
+    if rate ~= nil then
-    elseif state == "-" then
+        if state == "+" then
-        timeleft = tonumber(remaining) / tonumber(rate)
+            timeleft = (tonumber(capacity) - tonumber(remaining)) / tonumber(rate)
-    else
+        elseif state == "-" then
-        return {state, percent, "N/A"}
+            timeleft = tonumber(remaining) / tonumber(rate)
+        else
+            return {state, percent, time}
+        end
+        local hoursleft = math.floor(timeleft)
+        local minutesleft = math.floor((timeleft - hoursleft) * 60 )
+        time = string.format("%02d:%02d", hoursleft, minutesleft)
     end
-    local hoursleft = math.floor(timeleft)
-    local minutesleft = math.floor((timeleft - hoursleft) * 60 )
-    local time = string.format("%02d:%02d", hoursleft, minutesleft)
 
     return {state, percent, time}
 end