From 5495d73c35a88f8e2549d641f39d89625e054e11 Mon Sep 17 00:00:00 2001 From: Kyle Evans Date: Mon, 26 Feb 2018 04:12:54 +0000 Subject: [PATCH] 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. --- stand/lua/screen.lua | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/stand/lua/screen.lua b/stand/lua/screen.lua index 360306e8c27..7f883af8b44 100644 --- a/stand/lua/screen.lua +++ b/stand/lua/screen.lua @@ -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()