From: zeertzjq Date: Sat, 29 Mar 2025 08:05:52 +0000 (+0100) Subject: patch 9.1.1257: Mixing vim_strsize() with mb_ptr2cells() in pum_redraw() X-Git-Tag: v9.1.1257^0 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=90e52490b39f0052fb5313d67728eff77191aaae;p=thirdparty%2Fvim.git patch 9.1.1257: Mixing vim_strsize() with mb_ptr2cells() in pum_redraw() Problem: Mixing vim_strsize() with mb_ptr2cells() in pum_redraw(). Solution: Change vim_strsize() to mb_string2cells() (zeertzjq). Since vim_strsize() uses ptr2cells() for the cell width of each char, it is strange to mix it with mb_ptr2cells(), which is used both just below and in pum_screen_puts_with_attr(), and screen_puts_len() also uses something similar. Meanwhile mb_string2cells() uses mb_ptr2cells() for the cell width of each char. Note that the vim_strsize() and mb_string2cells() actually return the same value here, as the transstr() above makes sure the string only contains printable chars, and ptr2cells() and mb_ptr2cells() only return different values for unprintable chars. closes: #17003 Signed-off-by: zeertzjq Signed-off-by: Christian Brabandt --- diff --git a/src/popupmenu.c b/src/popupmenu.c index 5a2f070313..71bb499845 100644 --- a/src/popupmenu.c +++ b/src/popupmenu.c @@ -731,7 +731,7 @@ pum_redraw(void) char_u *old_rt = NULL; char_u *orig_rt = NULL; - cells = vim_strsize(rt); + cells = mb_string2cells(rt, -1); need_ellipsis = p_pmw > ellipsis_width && pum_width == p_pmw && cells > pum_width; diff --git a/src/version.c b/src/version.c index 2a83869a49..7d6c7e3f9b 100644 --- a/src/version.c +++ b/src/version.c @@ -704,6 +704,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 1257, /**/ 1256, /**/