Mercurial > prosody-modules
comparison mod_firewall/actions.lib.lua @ 2070:2356114ff505
mod_firewall: Optimize string match operations, string.find is faster than .match since no string is returned
| author | Kim Alvefur <zash@zash.se> |
|---|---|
| date | Fri, 11 Mar 2016 18:13:53 +0100 |
| parents | 7dbde05b48a9 |
| children | de6b95d5e01b |
comparison
equal
deleted
inserted
replaced
| 2069:cf9cd666ba00 | 2070:2356114ff505 |
|---|---|
| 27 if #attr_str == 0 then | 27 if #attr_str == 0 then |
| 28 table.insert(attr_str, ", { "); | 28 table.insert(attr_str, ", { "); |
| 29 else | 29 else |
| 30 table.insert(attr_str, ", "); | 30 table.insert(attr_str, ", "); |
| 31 end | 31 end |
| 32 if k:match("^%a%w*$") then | 32 if k:find("^%a%w*$") then |
| 33 table.insert(attr_str, string.format("%s = %q", k, v)); | 33 table.insert(attr_str, string.format("%s = %q", k, v)); |
| 34 else | 34 else |
| 35 table.insert(attr_str, string.format("[%q] = %q", k, v)); | 35 table.insert(attr_str, string.format("[%q] = %q", k, v)); |
| 36 end | 36 end |
| 37 end | 37 end |
| 43 first = nil; | 43 first = nil; |
| 44 else | 44 else |
| 45 code[#code+1] = (string.format(":tag(%q%s)", name, table.concat(attr_str))); | 45 code[#code+1] = (string.format(":tag(%q%s)", name, table.concat(attr_str))); |
| 46 end | 46 end |
| 47 end | 47 end |
| 48 if text and text:match("%S") then | 48 if text and text:find("%S") then |
| 49 code[#code+1] = (string.format(":text(%q)", text)); | 49 code[#code+1] = (string.format(":text(%q)", text)); |
| 50 elseif short_close then | 50 elseif short_close then |
| 51 short_close = nil; | 51 short_close = nil; |
| 52 code[#code+1] = (":up()"); | 52 code[#code+1] = (":up()"); |
| 53 end | 53 end |
