]> git.proxmox.com Git - mirror_qemu.git/commit - ui/console.c
ui/curses: Fix color attribute of monitor for curses
authorOGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Sun, 29 Nov 2015 13:28:24 +0000 (22:28 +0900)
committerGerd Hoffmann <kraxel@redhat.com>
Fri, 8 Jan 2016 11:20:07 +0000 (12:20 +0100)
commit4083733db5e4120939acee57019ff52db1f45b9d
tree8f268767181decdd790235ae5aa30d773eb4e891
parenta7e00e2536941a6e570b45b7ab4afec4505ff67e
ui/curses: Fix color attribute of monitor for curses

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]

Signed-off-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Message-id: 87r3j95407.fsf@mail.parknet.co.jp
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
hw/display/jazz_led.c
hw/display/vga.c
include/ui/console.h
ui/console.c
ui/curses.c