svn commit: r331259 - head/stand/lua

Kyle Evans kevans at freebsd.org
Wed Mar 21 15:12:35 UTC 2018


On Tue, Mar 20, 2018 at 6:04 PM, Devin Teske <dteske at freebsd.org> wrote:
>
> Minor nit ...
>
> While resetf is unlikely to be mistaken for anything other than "reset foreground", ...
>
> "resetb" could easily be mistaken for "reset bold".
>
> I would like to suggest, in tradition, that resetf/resetb be renamed to resetfg/resetbg
>
> You know... should you want to add a "resetb" later that actually resets bold (e.g., if you have to use CSI 22m in more than one place).
> --
> Devin

This set of functions has been renamed in r331304. The choice of 'f'
and 'b' always kind of bothered me, but not enough to actually fix it
until someone else complained about it. =)


More information about the svn-src-all mailing list