From 0d73f6d8ae32f1cd48ce9f089b902eb0877605e1 Mon Sep 17 00:00:00 2001 From: "Adrian C. (anrxc)" Date: Mon, 26 Oct 2009 20:32:48 +0100 Subject: 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... --- batat.lua | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'batat.lua') diff --git a/batat.lua b/batat.lua index 860eb26..bf84101 100644 --- a/batat.lua +++ b/batat.lua @@ -4,6 +4,7 @@ --------------------------------------------------- -- {{{ Grab environment +local tonumber = tonumber local io = { popen = io.popen } local setmetatable = setmetatable local table = { insert = table.insert } @@ -34,11 +35,11 @@ local function worker(format) if string.match(line, "^[%s]+Battery.*") then -- Store state and charge information table.insert(battery_info, (battery_state[string.match(line, "([%a]*),") or "unknown"])) - table.insert(battery_info, (string.match(line, "([%d]?[%d]?[%d])%.") or "0")) + table.insert(battery_info, (tonumber(string.match(line, "([%d]?[%d]?[%d])%.")) or 0)) -- Store remaining time information table.insert(battery_info, (string.match(line, "%%,%s(.*)") or "N/A")) else - return {battery_state["unknown"], "0", "N/A"} + return {battery_state["unknown"], 0, "N/A"} end end f:close() -- cgit v1.2.3