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

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

diff --git a/widgets/maildir.lua b/widgets/maildir.lua
index bd792210530346d92ae92de247ee6310ab817bff..eed613840921ecb955ee427489323be5db989a84 100644 (file)
--- a/widgets/maildir.lua
+++ b/widgets/maildir.lua
@@ -8,12 +8,13 @@
 --]]
 
 local newtimer        = require("lain.helpers").newtimer
 --]]
 
 local newtimer        = require("lain.helpers").newtimer

+local read_pipe       = require("lain.helpers").read_pipe

 
 local wibox           = require("wibox")
 
 local util            = require("lain.util")
 
 
 local wibox           = require("wibox")
 
 local util            = require("lain.util")
 

-local io              = io
+local io              = { popen  = io.popen }

 local os              = { getenv = os.getenv }
 local pairs           = pairs
 local string          = { len    = string.len,
 local os              = { getenv = os.getenv }
 local pairs           = pairs
 local string          = { len    = string.len,


@@ -50,10 +51,9 @@ local function worker(args)
                 -- match files that begin with a dot.
                 -- Afterwards the length of this string is the number of
                 -- new mails in that box.
                 -- match files that begin with a dot.
                 -- Afterwards the length of this string is the number of
                 -- new mails in that box.

-                local np = io.popen("find " .. line ..
+                local mailstring = read_pipe("find " .. line ..

                                     "/new -mindepth 1 -type f " ..
                                     "-not -name '.*' -printf a")
                                     "/new -mindepth 1 -type f " ..
                                     "-not -name '.*' -printf a")

-                local mailstring = np:read("*a")

 
                 -- Strip off leading mailpath.
                 local box = string.match(line, mailpath .. "/*([^/]+)")
 
                 -- Strip off leading mailpath.
                 local box = string.match(line, mailpath .. "/*([^/]+)")


@@ -65,10 +65,11 @@ local function worker(args)
             end
         until line == nil
 
             end
         until line == nil
 

+       p:close()

         table.sort(boxes)
 
         newmail = "no mail"
         table.sort(boxes)
 
         newmail = "no mail"

-        --Count the total number of mails irrespective of where it was found
+        -- Count the total number of mails irrespective of where it was found

         total = 0
 
         for box, number in pairs(boxes)
         total = 0
 
         for box, number in pairs(boxes)