Re: ncurses-5.4-20040918.patch.gz

[Thomas Dickey]

On Mon, 27 Sep 2004, Marcin 'Qrczak' Kowalczyk wrote:

> Thomas Dickey <address@hidden> writes:
>
> >     + remove check in wget_wch() added to fix an infinite loop, appears to
> >       have been working around a transitory glibc bug, and interferes
> >       with normal operation (report by Marcin 'Qrczak' Kowalczyk).
> >     + correct wadd_wch() and wecho_wch(), which did not pass the rendition
> >       information (report by Marcin 'Qrczak' Kowalczyk).
>
> Thanks. Here is a change needed to make setcchar/getcchar actually
> work with colors:
>
> --- ncurses-5.4/ncurses/widechar/lib_cchar.c.orig     2004-09-27 
> 23:10:47.113467830 +0200
> +++ ncurses-5.4/ncurses/widechar/lib_cchar.c  2004-09-27 23:14:13.220833629 
> +0200
> @@ -79,7 +79,7 @@
>       memset(wcval, 0, sizeof(*wcval));
>
>       if (len != 0) {
> -         SetAttr(*wcval, attrs | color_pair);
> +         SetAttr(*wcval, attrs | COLOR_PAIR(color_pair));

thanks (I seem to recall making this error more than once - might have
noticed it if I'd implemented my to-do item to augment text-color mixing
in the ncurses b/B test screens).

-- 
Thomas E. Dickey
http://invisible-island.net
ftp://invisible-island.net