]> git.ipfire.org Git - thirdparty/vim.git/commitdiff
patch 9.0.1813: linebreak incorrect drawn with breakindent v9.0.1813
authorzeertzjq <zeertzjq@outlook.com>
Mon, 28 Aug 2023 19:20:16 +0000 (21:20 +0200)
committerChristian Brabandt <cb@256bit.org>
Mon, 28 Aug 2023 19:21:46 +0000 (21:21 +0200)
Problem: 'linebreak' is incorrectly drawn after 'breakindent'.
Solution: Don't include 'breakindent' size when already after it.

closes: #12937
closes: #12940

Signed-off-by: Christian Brabandt <cb@256bit.org>
Co-authored-by: zeertzjq <zeertzjq@outlook.com>
src/charset.c
src/drawline.c
src/testdir/test_display.vim
src/version.c

index 1c7f7d7913a4a846f5bd4044093eeb808b024b73..03ef3fd1c6bddc3f371435ecc5c9316a94151241 100644 (file)
@@ -1319,6 +1319,9 @@ win_lbr_chartabsize(
        vcol -= wp->w_virtcol_first_char;
 #endif
        colnr_T wcol = vcol + col_off_prev;
+       colnr_T max_head_vcol = cts->cts_max_head_vcol;
+       int     added = 0;
+
        // cells taken by 'showbreak'/'breakindent' before current char
        int     head_prev = 0;
        if (wcol >= wp->w_width)
@@ -1332,23 +1335,18 @@ win_lbr_chartabsize(
            if (wp->w_p_bri)
                head_prev += get_breakindent_win(wp, line);
            if (wcol < head_prev)
-               wcol = head_prev;
-           wcol += col_off_prev;
-       }
-
-       if ((vcol > 0 && wcol == col_off_prev + head_prev)
-                                                 || wcol + size > wp->w_width)
-       {
-           int added = 0;
-           colnr_T max_head_vcol = cts->cts_max_head_vcol;
-
-           if (vcol > 0 && wcol == col_off_prev + head_prev)
            {
+               head_prev -= wcol;
+               wcol += head_prev;
                added += head_prev;
                if (max_head_vcol <= 0 || vcol < max_head_vcol)
                    head += head_prev;
            }
+           wcol += col_off_prev;
+       }
 
+       if (wcol + size > wp->w_width)
+       {
            // cells taken by 'showbreak'/'breakindent' halfway current char
            int head_mid = 0;
            if (*sbr != NUL)
@@ -1384,9 +1382,9 @@ win_lbr_chartabsize(
                }
 #endif
            }
-
-           size += added;
        }
+
+       size += added;
     }
     if (headp != NULL)
        *headp = head;
index 5b5fbf2383ea091ee96e0ae1031367d531e95f9b..63bc96c4da08170b033d403ec182e70a2fb49be9 100644 (file)
@@ -2887,14 +2887,6 @@ win_line(
                    wlv.n_extra = win_lbr_chartabsize(&cts, NULL) - 1;
                    clear_chartabsize_arg(&cts);
 
-                   // We have just drawn the showbreak value, no need to add
-                   // space for it again.
-                   if (wlv.vcol == wlv.vcol_sbr)
-                   {
-                       wlv.n_extra -= MB_CHARLEN(get_showbreak_value(wp));
-                       if (wlv.n_extra < 0)
-                           wlv.n_extra = 0;
-                   }
                    if (on_last_col && c != TAB)
                        // Do not continue search/match highlighting over the
                        // line break, but for TABs the highlighting should
index f817a8502d785ae21265688c52e5fe1600cc66f8..55764ba160f2d5d69f39a7e8ef6cd6aa6eec80de 100644 (file)
@@ -410,12 +410,19 @@ func Test_display_linebreak_breakat()
   new
   vert resize 25
   let _breakat = &breakat
-  setl signcolumn=yes linebreak breakat=) showbreak=+
+  setl signcolumn=yes linebreak breakat=) showbreak=++
   call setline(1, repeat('x', winwidth(0) - 2) .. ')abc')
   let lines = ScreenLines([1, 2], 25)
   let expected = [
           \ '  xxxxxxxxxxxxxxxxxxxxxxx',
-          \ '  + )abc                 '
+          \ '  ++)abc                 ',
+          \ ]
+  call assert_equal(expected, lines)
+  setl breakindent breakindentopt=shift:2
+  let lines = ScreenLines([1, 2], 25)
+  let expected = [
+          \ '  xxxxxxxxxxxxxxxxxxxxxxx',
+          \ '    ++)abc               ',
           \ ]
   call assert_equal(expected, lines)
   %bw!
index 46f2676a103da5e8fe9acb0b7890e35976b728a6..7b8683e7cbe6c2f684812dd4e35a9c244fda81df 100644 (file)
@@ -699,6 +699,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1813,
 /**/
     1812,
 /**/