#288 trying metatables

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

diff --git a/widgets/cpu.lua b/widgets/cpu.lua
index 15544433b91dc9c6ad751b58589072a3ff0f823e..f61e2ccf794ae7974abb5fdbf38d8f7f0b5bff5e 100644 (file)
--- a/widgets/cpu.lua
+++ b/widgets/cpu.lua
@@ -9,7 +9,6 @@
 
 local helpers      = require("lain.helpers")
 local wibox        = require("wibox")
 
 local helpers      = require("lain.helpers")
 local wibox        = require("wibox")

-

 local math         = { ceil   = math.ceil }
 local string       = { format = string.format,
                        gmatch = string.gmatch }
 local math         = { ceil   = math.ceil }
 local string       = { format = string.format,
                        gmatch = string.gmatch }


@@ -18,63 +17,66 @@ local setmetatable = setmetatable
 
 -- CPU usage
 -- lain.widgets.cpu
 
 -- CPU usage
 -- lain.widgets.cpu

-local cpu = {
-    last_total = 0,
-    last_active = 0
-}
+local cpu = helpers.make_widget_textbox()

 
 local function worker(args)
     local args     = args or {}
 
 local function worker(args)
     local args     = args or {}

-    local timeout  = args.timeout or 1
+    local timeout  = args.timeout or 2

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

-    cpu.widget = wibox.widget.textbox('')
-    helpers.set_map("cpuactive", 0)
-    helpers.set_map("cputotal", 0)
+    cpu.core = {}

 
     function update()
         -- Read the amount of time the CPUs have spent performing
         -- different kinds of work. Read the first line of /proc/stat
         -- which is the sum of all CPUs.
 
     function update()
         -- Read the amount of time the CPUs have spent performing
         -- different kinds of work. Read the first line of /proc/stat
         -- which is the sum of all CPUs.

-        local times = helpers.first_line("/proc/stat")
-        local at = 1
-        local idle = 0
-        local total = 0
-        for field in string.gmatch(times, "[%s]+([^%s]+)")
-        do
-            -- 4 = idle, 5 = ioWait. Essentially, the CPUs have done
-            -- nothing during these times.
-            if at == 4 or at == 5
-            then
-                idle = idle + field
+        local times = helpers.lines_match("cpu","/proc/stat")
+
+        for index,time in pairs(times) do
+            local coreid = index - 1
+            local core   = cpu.core[coreid] or
+                           { last_active = 0 , last_total = 0, usage = 0 }
+            local at     = 1
+            local idle   = 0
+            local total  = 0
+
+            for field in string.gmatch(time, "[%s]+([^%s]+)") do
+                -- 4 = idle, 5 = ioWait. Essentially, the CPUs have done
+                -- nothing during these times.
+                if at == 4 or at == 5 then
+                    idle = idle + field
+                end
+                total = total + field
+                at = at + 1

             end
             end

-            total = total + field
-            at = at + 1
-        end
-        local active = total - idle

 
 

-        if helpers.get_map("cpuactive") ~= active
-           or helpers.get_map("cputotal") ~= total
-        then
-            -- Read current data and calculate relative values.
-            local dactive = active - cpu.last_active
-            local dtotal = total - cpu.last_total
+            local active = total - idle

 
 

-            cpu_now = {}
-            cpu_now.usage = tostring(math.ceil((dactive / dtotal) * 100))
+            if core.last_active ~= active or core.last_total ~= total then
+                -- Read current data and calculate relative values.
+                local dactive = active - core.last_active
+                local dtotal  = total - core.last_total
+                local usage   = math.ceil((dactive / dtotal) * 100)

 
 

-            widget = cpu.widget
-            settings()
+                core.last_active = active
+                core.last_total  = total
+                core.usage       = usage

 
 

-            -- Save current data for the next run.
-            helpers.set_map("cpuactive", active)
-            helpers.set_map("cputotal", total)
+                -- Save current data for the next run.
+                cpu.core[coreid] = core
+            end

         end
         end

+
+        widget = cpu.widget
+        cpu_now = cpu.core
+        cpu_now.usage = cpu_now[0].usage
+
+        settings()

     end
 
     helpers.newtimer("cpu", timeout, update)
 
     end
 
     helpers.newtimer("cpu", timeout, update)
 

-    return cpu.widget
+    return cpu

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