# HG changeset patch # User Christian Brabandt # Date 1509211804 -7200 # Node ID 25f7d8ee04c79972bee7113c4a33b921ef5986a4 # Parent de225126b342fedccbdfa3cf876e32bb5e7ddfda patch 8.0.1234: MS-Windows: composing chars are not shown properly commit https://github.com/vim/vim/commit/a6ce1ccf5c10baa5c2a25897c46961d751a21dda Author: Bram Moolenaar Date: Sat Oct 28 19:23:11 2017 +0200 patch 8.0.1234: MS-Windows: composing chars are not shown properly Problem: MS-Windows: composing characters are not shown properly. Solution: Pass base character and composing characters to the renderer at once. (Ken Takata, closes #2206) diff --git a/src/gui.c b/src/gui.c --- a/src/gui.c +++ b/src/gui.c @@ -2429,9 +2429,14 @@ gui_outstr_nowrap( int cl; /* byte length of current char */ int comping; /* current char is composing */ int scol = col; /* screen column */ - int curr_wide; /* use 'guifontwide' */ + int curr_wide = FALSE; /* use 'guifontwide' */ int prev_wide = FALSE; int wide_changed; +# ifdef WIN3264 + int sep_comp = FALSE; /* Don't separate composing chars. */ +# else + int sep_comp = TRUE; /* Separate composing chars. */ +# endif /* Break the string at a composing character, it has to be drawn on * top of the previous character. */ @@ -2441,17 +2446,20 @@ gui_outstr_nowrap( { c = utf_ptr2char(s + i); cn = utf_char2cells(c); - if (cn > 1 -# ifdef FEAT_XFONTSET - && fontset == NOFONTSET -# endif - && wide_font != NOFONT) - curr_wide = TRUE; - else - curr_wide = FALSE; comping = utf_iscomposing(c); if (!comping) /* count cells from non-composing chars */ cells += cn; + if (!comping || sep_comp) + { + if (cn > 1 +# ifdef FEAT_XFONTSET + && fontset == NOFONTSET +# endif + && wide_font != NOFONT) + curr_wide = TRUE; + else + curr_wide = FALSE; + } cl = utf_ptr2len(s + i); if (cl == 0) /* hit end of string */ len = i + cl; /* len must be wrong "cannot happen" */ @@ -2460,7 +2468,8 @@ gui_outstr_nowrap( /* Print the string so far if it's the last character or there is * a composing character. */ - if (i + cl >= len || (comping && i > start) || wide_changed + if (i + cl >= len || (comping && sep_comp && i > start) + || wide_changed # if defined(FEAT_GUI_X11) || (cn > 1 # ifdef FEAT_XFONTSET @@ -2472,7 +2481,7 @@ gui_outstr_nowrap( # endif ) { - if (comping || wide_changed) + if ((comping && sep_comp) || wide_changed) thislen = i - start; else thislen = i - start + cl; @@ -2490,7 +2499,7 @@ gui_outstr_nowrap( cells = 0; /* Adjust to not draw a character which width is changed * against with last one. */ - if (wide_changed && !comping) + if (wide_changed && !(comping && sep_comp)) { scol -= cn; cl = 0; @@ -2509,7 +2518,7 @@ gui_outstr_nowrap( # endif } /* Draw a composing char on top of the previous char. */ - if (comping) + if (comping && sep_comp) { # if (defined(__APPLE_CC__) || defined(__MRC__)) && TARGET_API_MAC_CARBON /* Carbon ATSUI autodraws composing char over previous char */ diff --git a/src/gui_w32.c b/src/gui_w32.c --- a/src/gui_w32.c +++ b/src/gui_w32.c @@ -6295,8 +6295,8 @@ gui_mch_draw_string( if (enc_utf8 && n < len && unicodebuf != NULL) { - /* Output UTF-8 characters. Caller has already separated - * composing characters. */ + /* Output UTF-8 characters. Composing characters should be + * handled here. */ int i; int wlen; /* string length in words */ int clen; /* string length in characters */ @@ -6320,9 +6320,16 @@ gui_mch_draw_string( { unicodebuf[wlen++] = c; } - cw = utf_char2cells(c); - if (cw > 2) /* don't use 4 for unprintable char */ - cw = 1; + + if (utf_iscomposing(c)) + cw = 0; + else + { + cw = utf_char2cells(c); + if (cw > 2) /* don't use 4 for unprintable char */ + cw = 1; + } + if (unicodepdy != NULL) { /* Use unicodepdy to make characters fit as we expect, even @@ -6337,7 +6344,7 @@ gui_mch_draw_string( unicodepdy[wlen - 1] = cw * gui.char_width; } cells += cw; - i += utfc_ptr2len_len(text + i, len - i); + i += utf_ptr2len_len(text + i, len - i); ++clen; } #if defined(FEAT_DIRECTX) diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -762,6 +762,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1234, +/**/ 1233, /**/ 1232,