aboutsummaryrefslogtreecommitdiff
path: root/gmail.lua
diff options
context:
space:
mode:
authorAdrian C. (anrxc) <anrxc@sysphere.org>2009-10-26 20:32:48 +0100
committerAdrian C. (anrxc) <anrxc@sysphere.org>2009-10-26 20:32:48 +0100
commit0d73f6d8ae32f1cd48ce9f089b902eb0877605e1 (patch)
tree80d41d5fb6095610c781d47fa1a34e91cf184ce3 /gmail.lua
parentb105ae21cd47682a4e426604cb15ee6c11aea201 (diff)
downloadvicious-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.lua5
1 files changed, 3 insertions, 2 deletions
diff --git a/gmail.lua b/gmail.lua
index 345f6a6..2cdc9b9 100644
--- a/gmail.lua
+++ b/gmail.lua
@@ -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>")