073263
To: vim_dev@googlegroups.com
073263
Subject: Patch 7.4.594
073263
Fcc: outbox
073263
From: Bram Moolenaar <Bram@moolenaar.net>
073263
Mime-Version: 1.0
073263
Content-Type: text/plain; charset=UTF-8
073263
Content-Transfer-Encoding: 8bit
073263
------------
073263
073263
Patch 7.4.594
073263
Problem:    Using a block delete while 'breakindent' is set does not work
073263
	    properly.
073263
Solution:   Use "line" instead of "prev_pend" as the first argument to
073263
	    lbr_chartabsize_adv().  (Hirohito Higashi)
073263
Files:	    src/ops.c, src/testdir/test_breakindent.in,
073263
	    src/testdir/test_breakindent.ok
073263
073263
073263
*** ../vim-7.4.593/src/ops.c	2015-01-18 14:08:52.699436994 +0100
073263
--- src/ops.c	2015-01-27 13:07:11.518790582 +0100
073263
***************
073263
*** 5308,5317 ****
073263
  	    {
073263
  		/* Count a tab for what it's worth (if list mode not on) */
073263
  		prev_pend = pend;
073263
! 		/* TODO: is passing prev_pend for start of the line OK?
073263
! 		 * perhaps it should be "line". */
073263
! 		incr = lbr_chartabsize_adv(prev_pend, &pend,
073263
! 						      (colnr_T)bdp->end_vcol);
073263
  		bdp->end_vcol += incr;
073263
  	    }
073263
  	    if (bdp->end_vcol <= oap->end_vcol
073263
--- 5308,5314 ----
073263
  	    {
073263
  		/* Count a tab for what it's worth (if list mode not on) */
073263
  		prev_pend = pend;
073263
! 		incr = lbr_chartabsize_adv(line, &pend, (colnr_T)bdp->end_vcol);
073263
  		bdp->end_vcol += incr;
073263
  	    }
073263
  	    if (bdp->end_vcol <= oap->end_vcol
073263
*** ../vim-7.4.593/src/testdir/test_breakindent.in	2014-11-27 14:09:09.490354943 +0100
073263
--- src/testdir/test_breakindent.in	2015-01-27 13:06:51.067014258 +0100
073263
***************
073263
*** 99,104 ****
073263
--- 99,121 ----
073263
  :$put =line1
073263
  :$put =line2
073263
  :"
073263
+ :let g:test="Test 14: breakindent + visual blockwise delete #1"
073263
+ :set all& breakindent
073263
+ :30vnew
073263
+ :normal! 3a1234567890
073263
+ :normal! a    abcde
073263
+ :exec "normal! 0\<C-V>tex"
073263
+ :let line1=ScreenChar(line('.'),8)
073263
+ :call DoRecordScreen()
073263
+ :"
073263
+ :let g:test="Test 15: breakindent + visual blockwise delete #2"
073263
+ :%d
073263
+ :normal! 4a1234567890
073263
+ :exec "normal! >>\<C-V>3f0x"
073263
+ :let line1=ScreenChar(line('.'),20)
073263
+ :call DoRecordScreen()
073263
+ :quit!
073263
+ :"
073263
  :%w! test.out
073263
  :qa!
073263
  ENDTEST
073263
*** ../vim-7.4.593/src/testdir/test_breakindent.ok	2014-11-27 14:09:09.490354943 +0100
073263
--- src/testdir/test_breakindent.ok	2015-01-27 13:06:51.067014258 +0100
073263
***************
073263
*** 62,64 ****
073263
--- 62,74 ----
073263
  Test 13: breakindent with wrapping Tab
073263
  d
073263
  w
073263
+ 
073263
+ Test 14: breakindent + visual blockwise delete #1
073263
+ e       
073263
+ ~       
073263
+ ~       
073263
+ 
073263
+ Test 15: breakindent + visual blockwise delete #2
073263
+         1234567890  
073263
+ ~                   
073263
+ ~                   
073263
*** ../vim-7.4.593/src/version.c	2015-01-27 12:59:51.859602392 +0100
073263
--- src/version.c	2015-01-27 13:10:18.260748209 +0100
073263
***************
073263
*** 743,744 ****
073263
--- 743,746 ----
073263
  {   /* Add new patch number below this line */
073263
+ /**/
073263
+     594,
073263
  /**/
073263
073263
-- 
073263
hundred-and-one symptoms of being an internet addict:
073263
122. You ask if the Netaholics Anonymous t-shirt you ordered can be
073263
     sent to you via e-mail.
073263
073263
 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
073263
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
073263
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
073263
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///