[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/5] curses: add option to specify VGA font enco
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 4/5] curses: add option to specify VGA font encoding |
Date: |
Sat, 15 Oct 2016 17:07:23 -0400 (EDT) |
> +#ifdef CONFIG_ICONV
> + if (font_charset) {
> + unsigned char ch;
> + wchar_t wch;
> + char *pch, *pwch;
> + size_t sch, swch;
> + iconv_t conv;
> +
> + conv = iconv_open("WCHAR_T", font_charset);
Is this portable? It seems safer to convert to UTF-8 and parse the
resulting character (assuming that wchar_t is Unicode). There is
already mod_utf8_codepoint for this task.
Paolo
- [Qemu-devel] [PATCH 0/5] curses: wide character support, Samuel Thibault, 2016/10/15
- [Qemu-devel] [PATCH 5/5] curses: support wide input, Samuel Thibault, 2016/10/15
- [Qemu-devel] [PATCH 1/5] curses: fix left/right arrow translation, Samuel Thibault, 2016/10/15
- [Qemu-devel] [PATCH 4/5] curses: add option to specify VGA font encoding, Samuel Thibault, 2016/10/15
- [Qemu-devel] [PATCH 2/5] curses: Use cursesw instead of curses, Samuel Thibault, 2016/10/15
- [Qemu-devel] [PATCH 3/5] curses: use wide output functions, Samuel Thibault, 2016/10/15
- Re: [Qemu-devel] [PATCH 0/5] curses: wide character support, Samuel Thibault, 2016/10/25