Merge branch 'master' of github.com:copycat-killer/lain

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

diff --git a/widgets/net.lua b/widgets/net.lua
index d859d911123ad9dc575ac77a7697739c01baea56..983da41da0655e5a45ef366833045cda12d0fc80 100644 (file)
--- a/widgets/net.lua
+++ b/widgets/net.lua
@@ -13,7 +13,6 @@ local notify_fg    = require("beautiful").fg_focus
 local naughty      = require("naughty")
 local wibox        = require("wibox")
 
 local naughty      = require("naughty")
 local wibox        = require("wibox")
 

-local io           = { popen  = io.popen }

 local string       = { format = string.format,
                        gsub   = string.gsub,
                        match  = string.match }
 local string       = { format = string.format,
                        gsub   = string.gsub,
                        match  = string.match }


@@ -22,15 +21,10 @@ local setmetatable = setmetatable
 
 -- Network infos
 -- lain.widgets.net
 
 -- Network infos
 -- lain.widgets.net

-local net = {
-    last_t = 0,
-    last_r = 0
-}
+local net = {}

 
 function net.get_device()
 
 function net.get_device()

-    f = io.popen("ip link show | cut -d' ' -f2,9")
-    ws = f:read("*all")
-    f:close()
+    local ws = helpers.read_pipe("ip link show | cut -d' ' -f2,9")

     ws = ws:match("%w+: UP") or ws:match("ppp%w+: UNKNOWN")
     if ws ~= nil then
         return ws:match("(%w+):")
     ws = ws:match("%w+: UP") or ws:match("ppp%w+: UNKNOWN")
     if ws ~= nil then
         return ws:match("(%w+):")


@@ -41,7 +35,7 @@ end
 
 local function worker(args)
     local args = args or {}
 
 local function worker(args)
     local args = args or {}

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

     local units = args.units or 1024 --kb
     local notify = args.notify or "on"
     local screen = args.screen or 1
     local units = args.units or 1024 --kb
     local notify = args.notify or "on"
     local screen = args.screen or 1


@@ -52,35 +46,44 @@ local function worker(args)
     net.widget = wibox.widget.textbox('')
 
     helpers.set_map(iface, true)
     net.widget = wibox.widget.textbox('')
 
     helpers.set_map(iface, true)

+    helpers.set_map("net_t", 0)
+    helpers.set_map("net_r", 0)

 
     function update()
 
     function update()

-        net_now = {}
+        net_now = {
+            sent     = "0.0",
+            received = "0.0"
+        }

 
         if iface == "" or string.match(iface, "network off")
         then
             iface = net.get_device()
         end
 
 
         if iface == "" or string.match(iface, "network off")
         then
             iface = net.get_device()
         end
 

-        net_now.carrier = helpers.first_line('/sys/class/net/' .. iface ..
-                                           '/carrier') or "0"
-        net_now.state = helpers.first_line('/sys/class/net/' .. iface ..
-                                           '/operstate') or "down"

         local now_t = helpers.first_line('/sys/class/net/' .. iface ..
                                            '/statistics/tx_bytes') or 0
         local now_r = helpers.first_line('/sys/class/net/' .. iface ..
                                            '/statistics/rx_bytes') or 0
 
         local now_t = helpers.first_line('/sys/class/net/' .. iface ..
                                            '/statistics/tx_bytes') or 0
         local now_r = helpers.first_line('/sys/class/net/' .. iface ..
                                            '/statistics/rx_bytes') or 0
 

-        net_now.sent = (now_t - net.last_t) / timeout / units
-        net_now.sent = string.gsub(string.format('%.1f', net_now.sent), ",", ".")
+        if now_t ~= helpers.get_map("net_t")
+           or now_r ~= helpers.get_map("net_r") then
+            net_now.carrier = helpers.first_line('/sys/class/net/' .. iface ..
+                                           '/carrier') or "0"
+            net_now.state = helpers.first_line('/sys/class/net/' .. iface ..
+                                           '/operstate') or "down"

 
 

-        net_now.received = (now_r - net.last_r) / timeout / units
-        net_now.received = string.gsub(string.format('%.1f', net_now.received), ",", ".")
+            net_now.sent = (now_t - net.last_t) / timeout / units
+            net_now.sent = string.gsub(string.format('%.1f', net_now.sent), ",", ".")

 
 

-        widget = net.widget
-        settings()
+            net_now.received = (now_r - net.last_r) / timeout / units
+            net_now.received = string.gsub(string.format('%.1f', net_now.received), ",", ".")

 
 

-        net.last_t = now_t
-        net.last_r = now_r
+            widget = net.widget
+            settings()
+
+            helpers.set_map("net_t", now_t)
+            helpers.set_map("net_r", now_r)
+        end

 
         if net_now.carrier ~= "1" and notify == "on"
         then
 
         if net_now.carrier ~= "1" and notify == "on"
         then


@@ -102,7 +105,8 @@ local function worker(args)
         end
     end
 
         end
     end
 

-    helpers.newtimer(iface, timeout, update)
+    helpers.newtimer(iface, timeout, update, false)
+

     return net.widget
 end
 
     return net.widget
 end