Re: Ncursesw bug in refreshing wide character (with patch)

[Thomas Dickey]

On Tue, 11 Jan 2005, Eungkyu Song wrote:

> On Sat, Jan 08, 2005 at 01:54:40PM -0500, Thomas Dickey wrote:
> This patch work correctly.
> Thank you very much.
>
> > If that is what you are talking about, this is simpler:
> >
> > ===================================================================
> > RCS file: tty/RCS/tty_update.c,v
> > retrieving revision 1.213
> > diff -u -r1.213 tty/tty_update.c
> > --- tty/tty_update.c    2005/01/02 01:33:32     1.213
> > +++ tty/tty_update.c    2005/01/08 18:52:53
> > @@ -1298,8 +1298,10 @@
> >             if (oLastChar < nLastChar) {
> >                 int m = max(nLastNonblank, oLastNonblank);
> >                 GoTo(lineno, n + 1);
> > -               if (InsCharCost(nLastChar - oLastChar)
> > -                   > (m - n)) {
> > +               if (InsCharCost(nLastChar - oLastChar) > (m - n)
> > +#if USE_WIDEC_SUPPORT
> > +                   || isWidecExt(newLine[n + 1])
   +#endif
> > +               ) {
> >                     PutRange(oldLine, newLine, lineno, n + 1, m);
> >                 } else {
> >                     InsStr(&newLine[n + 1], nLastChar - oLastChar);
>
> This patch has no #endif line :(

sorry - I didn't compile it, of course.

> Anyway first patch is perfect.
>
> Simple program is attached to test this patch.
> You may need Korean (EUC-KR) locale, terminal and font.

thanks - I assume mlterm will handle that.

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