diff options
author | Philipp Gesang <phg42.2a@gmail.com> | 2013-05-04 14:32:09 +0200 |
---|---|---|
committer | Philipp Gesang <phg42.2a@gmail.com> | 2013-05-04 14:32:09 +0200 |
commit | 2b455d3b8d0d08374f21c99fb1f7d6bad75df4ca (patch) | |
tree | 0c164c655e40de63664ca3542e28736ea7d1a717 | |
parent | ed0b0f5b4f9cb26c12069146d2c9c3ed990e8478 (diff) | |
download | luaotfload-2b455d3b8d0d08374f21c99fb1f7d6bad75df4ca.tar.gz |
refactor color expression validation
-rw-r--r-- | luaotfload-colors.lua | 57 |
1 files changed, 42 insertions, 15 deletions
diff --git a/luaotfload-colors.lua b/luaotfload-colors.lua index 0bc3f9c..58089dc 100644 --- a/luaotfload-colors.lua +++ b/luaotfload-colors.lua @@ -15,6 +15,7 @@ local insert_node_after = node.insert_after local stringformat = string.format local stringgsub = string.gsub local stringfind = string.find +local stringsub = string.sub local otffeatures = fonts.constructors.newfeatures("otf") local identifiers = fonts.hashes.identifiers @@ -22,6 +23,43 @@ local registerotffeature = otffeatures.register local add_color_callback --[[ this used to be a global‽ ]] +local lpeg = require"lpeg" +local lpegmatch = lpeg.match +local C, Cg, Ct, P, R, S = lpeg.C, lpeg.Cg, lpeg.Ct, lpeg.P, lpeg.R, lpeg.S + +local digit16 = R("09", "af", "AF") +local octet = C(digit16 * digit16) +local p_rgb = Cg(octet, "red") + * Cg(octet, "green") + * Cg(octet, "blue") +local p_rgba = p_rgb * Cg(octet, "alpha") +local p_color = Ct(p_rgba + p_rgb) + +local old_sanitize_color_expression = function (value) + local sanitized + if value then + value = tostring(value) + local n_value = #value + if n_value == 6 or n_value == 8 then + sanitized = value + elseif n_value == 7 then --> take first six bytes + _, _, sanitized = stringsub(value, 1, 6) + elseif n_value > 8 then --> take first eight bytes + _, _, sanitized = stringsub(value, 1, 8) + else + -- broken color code ignored, issue a warning? + luaotfload.warning( + "“%s” is not a valid rgb[a] color expression", value) + end + end + return sanitized +end + +local sanitize_color_expression = function (digits) + local old = old_sanitize_color_expression(digits) + return old +end + --[[doc-- ``setcolor`` modifies tfmdata.properties.color in place --doc]]-- @@ -32,23 +70,12 @@ local add_color_callback --[[ this used to be a global‽ ]] --- hexadecimal, with an optional fourth transparency --- value) --- -local function setcolor(tfmdata,value) - local sanitized +local function setcolor (tfmdata, value) + print("~~~~~~~~~~~~~") + print(value) + local sanitized = sanitize_color_expression(value) local properties = tfmdata.properties - if value then - value = tostring(value) - if #value == 6 or #value == 8 then - sanitized = value - elseif #value == 7 then - _, _, sanitized = stringfind(value, "(......)") - elseif #value > 8 then - _, _, sanitized = stringfind(value, "(........)") - else - -- broken color code ignored, issue a warning? - end - end - if sanitized then tfmdata.properties.color = sanitized add_color_callback() |