Blame SOURCES/7.4.353

d6ba96
To: vim_dev@googlegroups.com
d6ba96
Subject: Patch 7.4.353
d6ba96
Fcc: outbox
d6ba96
From: Bram Moolenaar <Bram@moolenaar.net>
d6ba96
Mime-Version: 1.0
d6ba96
Content-Type: text/plain; charset=UTF-8
d6ba96
Content-Transfer-Encoding: 8bit
d6ba96
------------
d6ba96
d6ba96
Patch 7.4.353
d6ba96
Problem:    'linebreak' doesn't work with the 'list' option.
d6ba96
Solution:   Make it work. (Christian Brabandt)
d6ba96
Files:	    src/charset.c, src/screen.c,
d6ba96
	    src/testdir/Make_amiga.mak, src/testdir/Make_dos.mak,
d6ba96
	    src/testdir/Make_ming.mak, src/testdir/Make_os2.mak,
d6ba96
	    src/testdir/Make_vms.mms, src/testdir/Makefile,
d6ba96
	    src/testdir/test_listlbr.in, src/testdir/test_listlbr.ok
d6ba96
d6ba96
d6ba96
*** ../vim-7.4.352/src/charset.c	2014-07-02 19:37:38.462354956 +0200
d6ba96
--- src/charset.c	2014-07-02 19:47:21.602363892 +0200
d6ba96
***************
d6ba96
*** 1120,1126 ****
d6ba96
      if (wp->w_p_lbr
d6ba96
  	    && vim_isbreak(c)
d6ba96
  	    && !vim_isbreak(s[1])
d6ba96
- 	    && !wp->w_p_list
d6ba96
  	    && wp->w_p_wrap
d6ba96
  # ifdef FEAT_VERTSPLIT
d6ba96
  	    && wp->w_width != 0
d6ba96
--- 1120,1125 ----
d6ba96
*** ../vim-7.4.352/src/screen.c	2014-07-02 17:16:51.330225522 +0200
d6ba96
--- src/screen.c	2014-07-02 19:51:42.082367883 +0200
d6ba96
***************
d6ba96
*** 2843,2848 ****
d6ba96
--- 2843,2849 ----
d6ba96
      char_u	extra[18];		/* "%ld" and 'fdc' must fit in here */
d6ba96
      int		n_extra = 0;		/* number of extra chars */
d6ba96
      char_u	*p_extra = NULL;	/* string of extra chars, plus NUL */
d6ba96
+     char_u	*p_extra_free = NULL;   /* p_extra needs to be freed */
d6ba96
      int		c_extra = NUL;		/* extra chars, all the same */
d6ba96
      int		extra_attr = 0;		/* attributes when n_extra != 0 */
d6ba96
      static char_u *at_end_str = (char_u *)""; /* used for p_extra when
d6ba96
***************
d6ba96
*** 4053,4058 ****
d6ba96
--- 4054,4064 ----
d6ba96
  	}
d6ba96
  	else
d6ba96
  	{
d6ba96
+ 	    if (p_extra_free != NULL)
d6ba96
+ 	    {
d6ba96
+ 		vim_free(p_extra_free);
d6ba96
+ 		p_extra_free = NULL;
d6ba96
+ 	    }
d6ba96
  	    /*
d6ba96
  	     * Get a character from the line itself.
d6ba96
  	     */
d6ba96
***************
d6ba96
*** 4424,4431 ****
d6ba96
  		/*
d6ba96
  		 * Found last space before word: check for line break.
d6ba96
  		 */
d6ba96
! 		if (wp->w_p_lbr && vim_isbreak(c) && !vim_isbreak(*ptr)
d6ba96
! 							     && !wp->w_p_list)
d6ba96
  		{
d6ba96
  		    char_u *p = ptr - (
d6ba96
  # ifdef FEAT_MBYTE
d6ba96
--- 4430,4436 ----
d6ba96
  		/*
d6ba96
  		 * Found last space before word: check for line break.
d6ba96
  		 */
d6ba96
! 		if (wp->w_p_lbr && vim_isbreak(c) && !vim_isbreak(*ptr))
d6ba96
  		{
d6ba96
  		    char_u *p = ptr - (
d6ba96
  # ifdef FEAT_MBYTE
d6ba96
***************
d6ba96
*** 4433,4439 ****
d6ba96
  # endif
d6ba96
  				1);
d6ba96
  		    /* TODO: is passing p for start of the line OK? */
d6ba96
! 		    n_extra = win_lbr_chartabsize(wp, p, p, (colnr_T)vcol,
d6ba96
  								    NULL) - 1;
d6ba96
  		    c_extra = ' ';
d6ba96
  		    if (vim_iswhite(c))
d6ba96
--- 4438,4444 ----
d6ba96
  # endif
d6ba96
  				1);
d6ba96
  		    /* TODO: is passing p for start of the line OK? */
d6ba96
! 		    n_extra = win_lbr_chartabsize(wp, line, p, (colnr_T)vcol,
d6ba96
  								    NULL) - 1;
d6ba96
  		    c_extra = ' ';
d6ba96
  		    if (vim_iswhite(c))
d6ba96
***************
d6ba96
*** 4443,4449 ****
d6ba96
  			    /* See "Tab alignment" below. */
d6ba96
  			    FIX_FOR_BOGUSCOLS;
d6ba96
  #endif
d6ba96
! 			c = ' ';
d6ba96
  		    }
d6ba96
  		}
d6ba96
  #endif
d6ba96
--- 4448,4455 ----
d6ba96
  			    /* See "Tab alignment" below. */
d6ba96
  			    FIX_FOR_BOGUSCOLS;
d6ba96
  #endif
d6ba96
! 			if (!wp->w_p_list)
d6ba96
! 			    c = ' ';
d6ba96
  		    }
d6ba96
  		}
d6ba96
  #endif
d6ba96
***************
d6ba96
*** 4483,4491 ****
d6ba96
  		 */
d6ba96
  		if (c == TAB && (!wp->w_p_list || lcs_tab1))
d6ba96
  		{
d6ba96
  		    /* tab amount depends on current column */
d6ba96
! 		    n_extra = (int)wp->w_buffer->b_p_ts
d6ba96
  					- vcol % (int)wp->w_buffer->b_p_ts - 1;
d6ba96
  #ifdef FEAT_CONCEAL
d6ba96
  		    /* Tab alignment should be identical regardless of
d6ba96
  		     * 'conceallevel' value. So tab compensates of all
d6ba96
--- 4489,4538 ----
d6ba96
  		 */
d6ba96
  		if (c == TAB && (!wp->w_p_list || lcs_tab1))
d6ba96
  		{
d6ba96
+ 		    int tab_len = 0;
d6ba96
  		    /* tab amount depends on current column */
d6ba96
! 		    tab_len = (int)wp->w_buffer->b_p_ts
d6ba96
  					- vcol % (int)wp->w_buffer->b_p_ts - 1;
d6ba96
+ #ifdef FEAT_LINEBREAK
d6ba96
+ 		    if (!wp->w_p_lbr)
d6ba96
+ #endif
d6ba96
+ 		    /* tab amount depends on current column */
d6ba96
+ 			n_extra = tab_len;
d6ba96
+ #ifdef FEAT_LINEBREAK
d6ba96
+ 		    else
d6ba96
+ 		    {
d6ba96
+ 			char_u *p;
d6ba96
+ 			int	len = n_extra;
d6ba96
+ 			int	i;
d6ba96
+ 			int	saved_nextra = n_extra;
d6ba96
+ 
d6ba96
+ 			/* if n_extra > 0, it gives the number of chars, to
d6ba96
+ 			 * use for a tab, else we need to calculate the width
d6ba96
+ 			 * for a tab */
d6ba96
+ #ifdef FEAT_MBYTE
d6ba96
+ 			len = (tab_len * mb_char2len(lcs_tab2));
d6ba96
+ 			if (n_extra > 0)
d6ba96
+ 			    len += n_extra - tab_len;
d6ba96
+ #endif
d6ba96
+ 			c = lcs_tab1;
d6ba96
+ 			p = alloc((unsigned)(len + 1));
d6ba96
+ 			vim_memset(p, ' ', len);
d6ba96
+ 			p[len] = NUL;
d6ba96
+ 			p_extra_free = p;
d6ba96
+ 			for (i = 0; i < tab_len; i++)
d6ba96
+ 			{
d6ba96
+ #ifdef FEAT_MBYTE
d6ba96
+ 			    mb_char2bytes(lcs_tab2, p);
d6ba96
+ 			    p += mb_char2len(lcs_tab2);
d6ba96
+ 			    n_extra += mb_char2len(lcs_tab2)
d6ba96
+ 						 - (saved_nextra > 0 ? 1 : 0);
d6ba96
+ #else
d6ba96
+ 			    p[i] = lcs_tab2;
d6ba96
+ #endif
d6ba96
+ 			}
d6ba96
+ 			p_extra = p_extra_free;
d6ba96
+ 		    }
d6ba96
+ #endif
d6ba96
  #ifdef FEAT_CONCEAL
d6ba96
  		    /* Tab alignment should be identical regardless of
d6ba96
  		     * 'conceallevel' value. So tab compensates of all
d6ba96
***************
d6ba96
*** 4501,4508 ****
d6ba96
  		    if (wp->w_p_list)
d6ba96
  		    {
d6ba96
  			c = lcs_tab1;
d6ba96
! 			c_extra = lcs_tab2;
d6ba96
! 			n_attr = n_extra + 1;
d6ba96
  			extra_attr = hl_attr(HLF_8);
d6ba96
  			saved_attr2 = char_attr; /* save current attr */
d6ba96
  #ifdef FEAT_MBYTE
d6ba96
--- 4548,4560 ----
d6ba96
  		    if (wp->w_p_list)
d6ba96
  		    {
d6ba96
  			c = lcs_tab1;
d6ba96
! #ifdef FEAT_LINEBREAK
d6ba96
! 			if (wp->w_p_lbr)
d6ba96
! 			    c_extra = NUL; /* using p_extra from above */
d6ba96
! 			else
d6ba96
! #endif
d6ba96
! 			    c_extra = lcs_tab2;
d6ba96
! 			n_attr = tab_len + 1;
d6ba96
  			extra_attr = hl_attr(HLF_8);
d6ba96
  			saved_attr2 = char_attr; /* save current attr */
d6ba96
  #ifdef FEAT_MBYTE
d6ba96
***************
d6ba96
*** 4598,4606 ****
d6ba96
  		    if ((dy_flags & DY_UHEX) && wp->w_p_rl)
d6ba96
  			rl_mirror(p_extra);	/* reverse "<12>" */
d6ba96
  #endif
d6ba96
- 		    n_extra = byte2cells(c) - 1;
d6ba96
  		    c_extra = NUL;
d6ba96
! 		    c = *p_extra++;
d6ba96
  		    if (!attr_pri)
d6ba96
  		    {
d6ba96
  			n_attr = n_extra + 1;
d6ba96
--- 4650,4674 ----
d6ba96
  		    if ((dy_flags & DY_UHEX) && wp->w_p_rl)
d6ba96
  			rl_mirror(p_extra);	/* reverse "<12>" */
d6ba96
  #endif
d6ba96
  		    c_extra = NUL;
d6ba96
! #ifdef FEAT_LINEBREAK
d6ba96
! 		    if (wp->w_p_lbr)
d6ba96
! 		    {
d6ba96
! 			char_u *p;
d6ba96
! 
d6ba96
! 			c = *p_extra;
d6ba96
! 			p = alloc((unsigned)n_extra + 1);
d6ba96
! 			vim_memset(p, ' ', n_extra);
d6ba96
! 			STRNCPY(p, p_extra + 1, STRLEN(p_extra) - 1);
d6ba96
! 			p[n_extra] = NUL;
d6ba96
! 			p_extra_free = p_extra = p;
d6ba96
! 		    }
d6ba96
! 		    else
d6ba96
! #endif
d6ba96
! 		    {
d6ba96
! 			n_extra = byte2cells(c) - 1;
d6ba96
! 			c = *p_extra++;
d6ba96
! 		    }
d6ba96
  		    if (!attr_pri)
d6ba96
  		    {
d6ba96
  			n_attr = n_extra + 1;
d6ba96
*** ../vim-7.4.352/src/testdir/Make_amiga.mak	2014-06-25 14:39:35.110348584 +0200
d6ba96
--- src/testdir/Make_amiga.mak	2014-07-02 19:47:21.610363892 +0200
d6ba96
***************
d6ba96
*** 38,43 ****
d6ba96
--- 38,44 ----
d6ba96
  		test104.out test105.out test106.out test107.out \
d6ba96
  		test_autoformat_join.out \
d6ba96
  		test_breakindent.out \
d6ba96
+ 		test_listlbr.out \
d6ba96
  		test_eval.out \
d6ba96
  		test_options.out
d6ba96
  
d6ba96
***************
d6ba96
*** 165,169 ****
d6ba96
--- 166,171 ----
d6ba96
  test107.out: test107.in
d6ba96
  test_autoformat_join.out: test_autoformat_join.in
d6ba96
  test_breakindent.out: test_breakindent.in
d6ba96
+ test_listlbr.out: test_listlbr.in
d6ba96
  test_eval.out: test_eval.in
d6ba96
  test_options.out: test_options.in
d6ba96
*** ../vim-7.4.352/src/testdir/Make_dos.mak	2014-06-25 14:39:35.110348584 +0200
d6ba96
--- src/testdir/Make_dos.mak	2014-07-02 19:47:21.610363892 +0200
d6ba96
***************
d6ba96
*** 37,42 ****
d6ba96
--- 37,43 ----
d6ba96
  		test105.out test106.out  test107.out\
d6ba96
  		test_autoformat_join.out \
d6ba96
  		test_breakindent.out \
d6ba96
+ 		test_listlbr \
d6ba96
  		test_eval.out \
d6ba96
  		test_options.out
d6ba96
  
d6ba96
*** ../vim-7.4.352/src/testdir/Make_ming.mak	2014-06-25 14:39:35.110348584 +0200
d6ba96
--- src/testdir/Make_ming.mak	2014-07-02 19:47:21.610363892 +0200
d6ba96
***************
d6ba96
*** 57,62 ****
d6ba96
--- 57,63 ----
d6ba96
  		test105.out test106.out test107.out \
d6ba96
  		test_autoformat_join.out \
d6ba96
  		test_breakindent.out \
d6ba96
+ 		test_listlbr.out \
d6ba96
  		test_eval.out \
d6ba96
  		test_options.out
d6ba96
  
d6ba96
*** ../vim-7.4.352/src/testdir/Make_os2.mak	2014-06-25 14:39:35.114348584 +0200
d6ba96
--- src/testdir/Make_os2.mak	2014-07-02 19:47:21.610363892 +0200
d6ba96
***************
d6ba96
*** 40,45 ****
d6ba96
--- 40,46 ----
d6ba96
  		test_autoformat_join.out \
d6ba96
  		test_eval.out \
d6ba96
  		test_breakindent.out \
d6ba96
+ 		test_listlbr.out \
d6ba96
  		test_options.out
d6ba96
  
d6ba96
  .SUFFIXES: .in .out
d6ba96
*** ../vim-7.4.352/src/testdir/Make_vms.mms	2014-06-25 14:39:35.114348584 +0200
d6ba96
--- src/testdir/Make_vms.mms	2014-07-02 19:47:21.610363892 +0200
d6ba96
***************
d6ba96
*** 98,103 ****
d6ba96
--- 98,104 ----
d6ba96
  	 test105.out test106.out test107.out \
d6ba96
  	 test_autoformat_join.out \
d6ba96
  	 test_breakindent.out \
d6ba96
+ 	 test_listlbr.out \
d6ba96
  	 test_eval.out \
d6ba96
  	 test_options.out
d6ba96
  
d6ba96
*** ../vim-7.4.352/src/testdir/Makefile	2014-06-25 14:39:35.114348584 +0200
d6ba96
--- src/testdir/Makefile	2014-07-02 19:47:21.610363892 +0200
d6ba96
***************
d6ba96
*** 35,40 ****
d6ba96
--- 35,41 ----
d6ba96
  		test104.out test105.out test106.out test107.out \
d6ba96
  		test_autoformat_join.out \
d6ba96
  		test_breakindent.out \
d6ba96
+ 		test_listlbr.out \
d6ba96
  		test_eval.out \
d6ba96
  		test_options.out
d6ba96
  
d6ba96
*** ../vim-7.4.352/src/testdir/test_listlbr.in	2014-07-02 19:58:25.642374067 +0200
d6ba96
--- src/testdir/test_listlbr.in	2014-07-02 19:47:21.610363892 +0200
d6ba96
***************
d6ba96
*** 0 ****
d6ba96
--- 1,62 ----
d6ba96
+ Test for linebreak and list option
d6ba96
+ 
d6ba96
+ STARTTEST
d6ba96
+ :so small.vim
d6ba96
+ :if !exists("+linebreak") | e! test.ok | w! test.out | qa! | endif
d6ba96
+ :10new|:vsp|:vert resize 20
d6ba96
+ :put =\"\tabcdef hijklmn\tpqrstuvwxyz\u00a01060ABCDEFGHIJKLMNOP \"
d6ba96
+ :norm! zt
d6ba96
+ :set ts=4 sw=4 sts=4 linebreak sbr=+ wrap
d6ba96
+ :fu! ScreenChar(width)
d6ba96
+ :	let c=''
d6ba96
+ :	for j in range(1,4)
d6ba96
+ :	    for i in range(1,a:width)
d6ba96
+ :	    	let c.=nr2char(screenchar(j, i))
d6ba96
+ :	    endfor
d6ba96
+ :           let c.="\n"
d6ba96
+ :	endfor
d6ba96
+ :	return c
d6ba96
+ :endfu
d6ba96
+ :fu! DoRecordScreen()
d6ba96
+ :	wincmd l
d6ba96
+ :	$put =printf(\"\n%s\", g:test)
d6ba96
+ :	$put =g:line
d6ba96
+ :	wincmd p
d6ba96
+ :endfu
d6ba96
+ :let g:test="Test 1: set linebreak"
d6ba96
+ :redraw!
d6ba96
+ :let line=ScreenChar(winwidth(0))
d6ba96
+ :call DoRecordScreen()
d6ba96
+ :let g:test="Test 2: set linebreak + set list"
d6ba96
+ :set linebreak list listchars=
d6ba96
+ :redraw!
d6ba96
+ :let line=ScreenChar(winwidth(0))
d6ba96
+ :call DoRecordScreen()
d6ba96
+ :let g:test ="Test 3: set linebreak + set list + fancy listchars"
d6ba96
+ :exe "set linebreak list listchars=nbsp:\u2423,tab:\u2595\u2014,trail:\u02d1,eol:\ub6"
d6ba96
+ :redraw!
d6ba96
+ :let line=ScreenChar(winwidth(0))
d6ba96
+ :call DoRecordScreen()
d6ba96
+ :let g:test ="Test 4: set linebreak nolist"
d6ba96
+ :set nolist linebreak
d6ba96
+ :redraw!
d6ba96
+ :let line=ScreenChar(winwidth(0))
d6ba96
+ :call DoRecordScreen()
d6ba96
+ :let g:test ="Test 5: set nolinebreak list"
d6ba96
+ :set list nolinebreak
d6ba96
+ :redraw!
d6ba96
+ :let line=ScreenChar(winwidth(0))
d6ba96
+ :call DoRecordScreen()
d6ba96
+ :let g:test ="Test 6: set linebreak with tab and 1 line as long as screen: should break!"
d6ba96
+ :set nolist linebreak ts=8
d6ba96
+ :let line="1\t".repeat('a', winwidth(0)-2)
d6ba96
+ :$put =line
d6ba96
+ :$
d6ba96
+ :norm! zt
d6ba96
+ :redraw!
d6ba96
+ :let line=ScreenChar(winwidth(0))
d6ba96
+ :call DoRecordScreen()
d6ba96
+ :%w! test.out
d6ba96
+ :qa!
d6ba96
+ ENDTEST
d6ba96
+ dummy text
d6ba96
*** ../vim-7.4.352/src/testdir/test_listlbr.ok	2014-07-02 19:58:25.646374067 +0200
d6ba96
--- src/testdir/test_listlbr.ok	2014-07-02 19:47:21.610363892 +0200
d6ba96
***************
d6ba96
*** 0 ****
d6ba96
--- 1,39 ----
d6ba96
+ 
d6ba96
+ 	abcdef hijklmn	pqrstuvwxyz 1060ABCDEFGHIJKLMNOP 
d6ba96
+ 
d6ba96
+ Test 1: set linebreak
d6ba96
+     abcdef          
d6ba96
+ +hijklmn            
d6ba96
+ +pqrstuvwxyz 1060ABC
d6ba96
+ +DEFGHIJKLMNOP      
d6ba96
+ 
d6ba96
+ Test 2: set linebreak + set list
d6ba96
+ ^Iabcdef hijklmn^I  
d6ba96
+ +pqrstuvwxyz 1060ABC
d6ba96
+ +DEFGHIJKLMNOP      
d6ba96
+                     
d6ba96
+ 
d6ba96
+ Test 3: set linebreak + set list + fancy listchars
d6ba96
+ ▕———abcdef          
d6ba96
+ +hijklmn▕———        
d6ba96
+ +pqrstuvwxyz␣1060ABC
d6ba96
+ +DEFGHIJKLMNOPˑ¶    
d6ba96
+ 
d6ba96
+ Test 4: set linebreak nolist
d6ba96
+     abcdef          
d6ba96
+ +hijklmn            
d6ba96
+ +pqrstuvwxyz 1060ABC
d6ba96
+ +DEFGHIJKLMNOP      
d6ba96
+ 
d6ba96
+ Test 5: set nolinebreak list
d6ba96
+ ▕———abcdef hijklmn▕—
d6ba96
+ +pqrstuvwxyz␣1060ABC
d6ba96
+ +DEFGHIJKLMNOPˑ¶    
d6ba96
+ ¶                   
d6ba96
+ 1	aaaaaaaaaaaaaaaaaa
d6ba96
+ 
d6ba96
+ Test 6: set linebreak with tab and 1 line as long as screen: should break!
d6ba96
+ 1                   
d6ba96
+ +aaaaaaaaaaaaaaaaaa 
d6ba96
+ ~                   
d6ba96
+ ~                   
d6ba96
*** ../vim-7.4.352/src/version.c	2014-07-02 19:37:38.462354956 +0200
d6ba96
--- src/version.c	2014-07-02 19:57:44.066373430 +0200
d6ba96
***************
d6ba96
*** 736,737 ****
d6ba96
--- 736,739 ----
d6ba96
  {   /* Add new patch number below this line */
d6ba96
+ /**/
d6ba96
+     353,
d6ba96
  /**/
d6ba96
d6ba96
-- 
d6ba96
FATHER:    You only killed the bride's father - that's all -
d6ba96
LAUNCELOT: Oh dear, I didn't really mean to...
d6ba96
FATHER:    Didn't mean to?  You put your sword right through his head!
d6ba96
LAUNCELOT: Gosh - Is he all right?
d6ba96
                 "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
d6ba96
d6ba96
 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
d6ba96
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
d6ba96
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
d6ba96
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///