Blob Blame History Raw
To: vim_dev@googlegroups.com
Subject: Patch 7.4.675
Fcc: outbox
From: Bram Moolenaar <Bram@moolenaar.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
------------

Patch 7.4.675
Problem:    When a FileReadPost autocommand moves the cursor inside a line it
	    gets moved back.
Solution:   When checking whether an autocommand moved the cursor store the
	    column as well. (Christian Brabandt)
Files:	    src/ex_cmds.c


*** ../vim-7.4.674/src/ex_cmds.c	2015-02-27 19:34:51.464777369 +0100
--- src/ex_cmds.c	2015-03-24 11:42:38.796239766 +0100
***************
*** 3185,3191 ****
  #endif
      int		retval = FAIL;
      long	n;
!     linenr_T	lnum;
      linenr_T	topline = 0;
      int		newcol = -1;
      int		solcol = -1;
--- 3185,3191 ----
  #endif
      int		retval = FAIL;
      long	n;
!     pos_T	orig_pos;
      linenr_T	topline = 0;
      int		newcol = -1;
      int		solcol = -1;
***************
*** 3678,3684 ****
  	 * Careful: open_buffer() and apply_autocmds() may change the current
  	 * buffer and window.
  	 */
! 	lnum = curwin->w_cursor.lnum;
  	topline = curwin->w_topline;
  	if (!oldbuf)			    /* need to read the file */
  	{
--- 3678,3684 ----
  	 * Careful: open_buffer() and apply_autocmds() may change the current
  	 * buffer and window.
  	 */
! 	orig_pos = curwin->w_cursor;
  	topline = curwin->w_topline;
  	if (!oldbuf)			    /* need to read the file */
  	{
***************
*** 3719,3729 ****
  	check_arg_idx(curwin);
  #endif
  
! 	/*
! 	 * If autocommands change the cursor position or topline, we should
! 	 * keep it.
! 	 */
! 	if (curwin->w_cursor.lnum != lnum)
  	{
  	    newlnum = curwin->w_cursor.lnum;
  	    newcol = curwin->w_cursor.col;
--- 3719,3727 ----
  	check_arg_idx(curwin);
  #endif
  
! 	/* If autocommands change the cursor position or topline, we should
! 	 * keep it.  Also when it moves within a line. */
! 	if (!equalpos(curwin->w_cursor, orig_pos))
  	{
  	    newlnum = curwin->w_cursor.lnum;
  	    newcol = curwin->w_cursor.col;
*** ../vim-7.4.674/src/version.c	2015-03-21 22:18:37.808371766 +0100
--- src/version.c	2015-03-24 11:44:14.240327255 +0100
***************
*** 743,744 ****
--- 743,746 ----
  {   /* Add new patch number below this line */
+ /**/
+     675,
  /**/

-- 
There are three kinds of people: Those who can count & those who can't.

 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///