qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2] ui/curses: Fix color attribute of monitor fo


From: Gerd Hoffmann
Subject: Re: [Qemu-devel] [PATCH v2] ui/curses: Fix color attribute of monitor for curses
Date: Tue, 15 Dec 2015 11:37:19 +0100

On Mi, 2015-12-02 at 03:44 +0900, OGAWA Hirofumi wrote:
> Current text_console_update() writes totally broken color attributes
> to console_write_ch(). The format now is writing,
> 
> [WRONG]
>       bold << 21 | fg << 12 | bg << 8 | char
>       fg == 3bits curses color number
>       bg == 3bits curses color number
> 
> I can't see this format is where come from. Anyway, this doesn't work
> at all.
> 
> What curses expects is actually (and vga.c is using),
> 
> [RIGHT]
>       bold << 21 | bg << 11 | fg << 8 | char
>       fg == 3bits vga color number
>       bg == 3bits vga color number
> 
> And curses set COLOR_PAIR() up to match this format, and curses's
> chtype. I.e,
> 
>       bold | color_pair | char
>       color_pair == (bg << 3 | fg)
> 
> To fix, this simply uses VGA color number everywhere except curses.c
> internal. Then, convert it to above [RIGHT] format to write by
> console_write_ch(). And as bonus, this reduces to expose curses define
> to other parts (removes COLOR_* from console.c).
> 
> [Tested the first line is displayed as white on blue back for monitor
> in curses console]

Added to ui patch queue.

thanks,
  Gerd




reply via email to

[Prev in Thread] Current Thread [Next in Thread]