diff options
| author | zeertzjq <zeertzjq@outlook.com> | 2024-09-06 16:24:41 +0200 |
|---|---|---|
| committer | Christian Brabandt <cb@256bit.org> | 2024-09-06 16:24:41 +0200 |
| commit | 66f65a46c5d169f20f780721d4f74d4729855b96 (patch) | |
| tree | 084673458eb7fd66b0d9613212c02d1894a54299 | |
| parent | 82f6134b1b46414fdbb97060276a5687f46abf89 (diff) | |
patch 9.1.0719: Resetting cell widths can make 'listchars' or 'fillchars' invalidv9.1.0719
Problem: Resetting cell widths can make 'listchars' or 'fillchars'
invalid.
Solution: Check for conflicts when resetting cell widths (zeertzjq).
closes: #15629
Signed-off-by: zeertzjq <zeertzjq@outlook.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
| -rw-r--r-- | src/mbyte.c | 24 | ||||
| -rw-r--r-- | src/testdir/test_utf8.vim | 12 | ||||
| -rw-r--r-- | src/version.c | 2 |
3 files changed, 22 insertions, 16 deletions
diff --git a/src/mbyte.c b/src/mbyte.c index fa67032a6f..a2d36f151f 100644 --- a/src/mbyte.c +++ b/src/mbyte.c @@ -5630,7 +5630,8 @@ f_setcellwidths(typval_T *argvars, typval_T *rettv UNUSED) int item; int i; listitem_T **ptrs; - cw_interval_T *table; + cw_interval_T *table = NULL; + size_t table_size; cw_interval_T *cw_table_save; size_t cw_table_size_save; char *error = NULL; @@ -5639,15 +5640,12 @@ f_setcellwidths(typval_T *argvars, typval_T *rettv UNUSED) return; l = argvars[0].vval.v_list; - if (l->lv_len == 0) - { + table_size = (size_t)l->lv_len; + if (table_size == 0) // Clearing the table. - VIM_CLEAR(cw_table); - cw_table_size = 0; - goto done; - } + goto update; - ptrs = ALLOC_MULT(listitem_T *, l->lv_len); + ptrs = ALLOC_MULT(listitem_T *, table_size); if (ptrs == NULL) return; @@ -5706,9 +5704,9 @@ f_setcellwidths(typval_T *argvars, typval_T *rettv UNUSED) } // Sort the list on the first number. - qsort((void *)ptrs, (size_t)l->lv_len, sizeof(listitem_T *), tv_nr_compare); + qsort((void *)ptrs, table_size, sizeof(listitem_T *), tv_nr_compare); - table = ALLOC_MULT(cw_interval_T, l->lv_len); + table = ALLOC_MULT(cw_interval_T, table_size); if (table == NULL) { vim_free(ptrs); @@ -5716,7 +5714,7 @@ f_setcellwidths(typval_T *argvars, typval_T *rettv UNUSED) } // Store the items in the new table. - for (item = 0; item < l->lv_len; ++item) + for (item = 0; (size_t)item < table_size; ++item) { listitem_T *lili = ptrs[item]; varnumber_T n1; @@ -5738,10 +5736,11 @@ f_setcellwidths(typval_T *argvars, typval_T *rettv UNUSED) vim_free(ptrs); +update: cw_table_save = cw_table; cw_table_size_save = cw_table_size; cw_table = table; - cw_table_size = l->lv_len; + cw_table_size = table_size; // Check that the new value does not conflict with 'listchars' or // 'fillchars'. @@ -5756,7 +5755,6 @@ f_setcellwidths(typval_T *argvars, typval_T *rettv UNUSED) } vim_free(cw_table_save); -done: changed_window_setting_all(); redraw_all_later(UPD_CLEAR); } diff --git a/src/testdir/test_utf8.vim b/src/testdir/test_utf8.vim index 0314fcdfd2..991a095380 100644 --- a/src/testdir/test_utf8.vim +++ b/src/testdir/test_utf8.vim @@ -255,15 +255,21 @@ func Test_setcellwidths() call assert_fails('call setcellwidths([[0x33, 0x44, 2]])', 'E1114:') - set listchars=tab:--\\u2192 + set listchars=tab:--\\u2192 fillchars=stl:\\u2501 call assert_fails('call setcellwidths([[0x2192, 0x2192, 2]])', 'E834:') - - set fillchars=stl:\\u2501 call assert_fails('call setcellwidths([[0x2501, 0x2501, 2]])', 'E835:') + call setcellwidths([[0x201c, 0x201d, 1]]) + set listchars& fillchars& ambiwidth=double + + set listchars=nbsp:\\u201c fillchars=vert:\\u201d + call assert_fails('call setcellwidths([])', 'E834:') set listchars& + call assert_fails('call setcellwidths([])', 'E835:') set fillchars& + call setcellwidths([]) + set ambiwidth& bwipe! endfunc diff --git a/src/version.c b/src/version.c index d7f012c641..ddbf4eb491 100644 --- a/src/version.c +++ b/src/version.c @@ -705,6 +705,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 719, +/**/ 718, /**/ 717, |