lualoader: screen argument fixes

screen was also guilty of not-so-great argument names, but it was also
guilty of handling color sequences on its own. Change those bits to using
the color module instead.

As a side note, between color and screen, I'm not 100% sure that returning
the color_value is the right thing to do if we won't generate the escape
sequences. This should be re-evaluated at a later time, and they should
likely return nil instead.
This commit is contained in:
kevans 2018-02-26 04:12:54 +00:00
parent d4894b0aac
commit 0af574732d

View File

@ -49,18 +49,18 @@ function screen.setcursor(x, y)
loader.printc("\027[" .. y .. ";" .. x .. "H")
end
function screen.setforeground(c)
function screen.setforeground(color_value)
if color.disabled then
return c
return color_value
end
loader.printc("\027[3" .. c .. "m")
loader.printc(color.escapef(color_value))
end
function screen.setbackground(c)
function screen.setbackground(color_value)
if color.disabled then
return c
return color_value
end
loader.printc("\027[4" .. c .. "m")
loader.printc(color.escapeb(color_value))
end
function screen.defcolor()