diff options
author | Adrian C. (anrxc) <anrxc@sysphere.org> | 2009-10-26 20:32:48 +0100 |
---|---|---|
committer | Adrian C. (anrxc) <anrxc@sysphere.org> | 2009-10-26 20:32:48 +0100 |
commit | 0d73f6d8ae32f1cd48ce9f089b902eb0877605e1 (patch) | |
tree | 80d41d5fb6095610c781d47fa1a34e91cf184ce3 /gmail.lua | |
parent | b105ae21cd47682a4e426604cb15ee6c11aea201 (diff) | |
download | vicious-legacy-0d73f6d8ae32f1cd48ce9f089b902eb0877605e1.tar.xz |
Ensure returned numbers are of type number
Thanks to Felix for bringing this to my attention. Obviously there was
already a safety net for feeding progressbars and graphs... and while
this makes for a good coding practice it's not a big deal. We have
widgets of type textbox for one, and a lot of string concatenation
happens. Strings are formatted, markup is applied...
Diffstat (limited to 'gmail.lua')
-rw-r--r-- | gmail.lua | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -4,6 +4,7 @@ --------------------------------------------------- -- {{{ Grab environment +local tonumber = tonumber local io = { popen = io.popen } local setmetatable = setmetatable local string = { match = string.match } @@ -24,7 +25,7 @@ local function worker(format, feed) local auth = user .. ":" .. pass local feed = feed or "https://mail.google.com/mail/feed/atom/unread" local mail = { - ["{count}"] = "0", + ["{count}"] = 0, ["{subject}"] = "N/A" } @@ -34,7 +35,7 @@ local function worker(format, feed) -- Could be huge don't read it all at once, info we are after is at the top for line in f:lines() do mail["{count}"] = -- Count comes before messages and matches at least 0 - string.match(line, "<fullcount>([%d]+)</fullcount>") or mail["{count}"] + tonumber(string.match(line, "<fullcount>([%d]+)</fullcount>")) or mail["{count}"] -- Find subject tags local title = string.match(line, "<title>(.*)</title>") |