Re: [coreboot] Tinycurses depends on active video console.

[Patrick Georgi]

Mats Erik Andersson schrieb:
> Index: libpayload/curses/tinycurses.c
>
> --- libpayload/curses/tinycurses.c	(revision 3623)
> +++ libpayload/curses/tinycurses.c	(arbetskopia)
> @@ -736,8 +736,10 @@
>    	for (x = win->_line[y].firstchar; x <= win->_line[y].lastchar; x++) {
>    		attr_t attr = win->_line[y].text[x].attr;
>
> +#ifdef CONFIG_VIDEO_CONSOLE
>    		unsigned int c =
>    			((int)color_pairs[PAIR_NUMBER(attr)]) << 8;
> +#endif
>
> #ifdef CONFIG_SERIAL_CONSOLE
>    		if (curses_flags & F_ENABLE_SERIAL) {

how about just moving that variable down to the #ifdef block that uses 
it? color_pairs[] or attr aren't changed inbetween, afaics.