From c611941c606328740b6e9acc1191a5d97ace13e2 Mon Sep 17 00:00:00 2001 From: rbtnn <naru123456789@gmail.com> Date: Sat, 7 Aug 2021 13:08:45 +0200 Subject: [PATCH] patch 8.2.3304: popup window title with wide characters is truncated Problem: Popup window title with wide characters is truncated. Solution: Use vim_strsize() instead of MB_CHARLEN(). (Naruhiko Nishino, closes #8721) --- src/popupwin.c | 2 +- src/testdir/dumps/Test_popupwin_multibytetitle.dump | 2 +- src/testdir/test_popupwin.vim | 2 +- src/version.c | 2 ++ 4 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/popupwin.c b/src/popupwin.c index 470bd70150..d86c6c0ef1 100644 --- a/src/popupwin.c +++ b/src/popupwin.c @@ -3896,7 +3896,7 @@ update_popups(void (*win_update)(win_T *wp)) title_wincol = wp->w_wincol + 1; if (wp->w_popup_title != NULL) { - title_len = (int)MB_CHARLEN(wp->w_popup_title); + title_len = vim_strsize(wp->w_popup_title); // truncate the title if too long if (title_len > total_width - 2) diff --git a/src/testdir/dumps/Test_popupwin_multibytetitle.dump b/src/testdir/dumps/Test_popupwin_multibytetitle.dump index e21c0cf2bb..95b08717ec 100644 --- a/src/testdir/dumps/Test_popupwin_multibytetitle.dump +++ b/src/testdir/dumps/Test_popupwin_multibytetitle.dump @@ -1,7 +1,7 @@ >1+0&#ffffff0| @73 |2| @73 |3| @73 -|4| @25|â•”+0#0000001#ffd7ff255|â–¶|Ä|Ö|Ü|â—€|â•@12|â•—| +0#0000000#ffffff0@27 +|4| @25|â•”+0#0000001#ffd7ff255|â–¶|Ä|ã‚*&|ã„|ã†|ãˆ|ãŠ|â—€+&|â•@4|â•—| +0#0000000#ffffff0@27 |5| @25|â•‘+0#0000001#ffd7ff255| |T+0&#e0e0e08|h|i|s| |i|s| |a| |l|i|n|e| @1| +0&#ffd7ff255|â•‘| +0#0000000#ffffff0@27 |6| @25|â•‘+0#0000001#ffd7ff255| |a|n|d| |a|n|o|t|h|e|r| |l|i|n|e| |â•‘| +0#0000000#ffffff0@27 |7| @25|╚+0#0000001#ffd7ff255|â•@17|â•| +0#0000000#ffffff0@27 diff --git a/src/testdir/test_popupwin.vim b/src/testdir/test_popupwin.vim index 62e19fa5d2..83d632eeed 100644 --- a/src/testdir/test_popupwin.vim +++ b/src/testdir/test_popupwin.vim @@ -1817,7 +1817,7 @@ func Test_popup_title() call VerifyScreenDump(buf, 'Test_popupwin_longtitle_4', {}) call term_sendkeys(buf, ":call popup_clear()\<CR>") - call term_sendkeys(buf, ":call popup_menu(['This is a line', 'and another line'], #{title: '▶ÄÖÜ◀', })\<CR>") + call term_sendkeys(buf, ":call popup_menu(['This is a line', 'and another line'], #{title: '▶Äã‚ã„ã†ãˆãŠâ—€', })\<CR>") call VerifyScreenDump(buf, 'Test_popupwin_multibytetitle', {}) call term_sendkeys(buf, "x") diff --git a/src/version.c b/src/version.c index fedc23befe..206234d9b9 100644 --- a/src/version.c +++ b/src/version.c @@ -755,6 +755,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 3304, /**/ 3303, /**/ -- GitLab