Blame SOURCES/7.4.207

d6ba96
To: vim_dev@googlegroups.com
d6ba96
Subject: Patch 7.4.207
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.207
d6ba96
Problem:    The cursor report sequence is sometimes not recognized and results
d6ba96
	    in entering replace mode.
d6ba96
Solution:   Also check for the cursor report when not asked for.
d6ba96
Files:	    src/term.c
d6ba96
d6ba96
d6ba96
*** ../vim-7.4.206/src/term.c	2013-09-29 16:27:42.000000000 +0200
d6ba96
--- src/term.c	2014-03-19 13:30:23.589874866 +0100
d6ba96
***************
d6ba96
*** 3379,3385 ****
d6ba96
  	 out_str(buf);
d6ba96
  	 out_str(T_U7);
d6ba96
  	 u7_status = U7_SENT;
d6ba96
! 	 term_windgoto(0, 0);
d6ba96
  	 out_str((char_u *)"  ");
d6ba96
  	 term_windgoto(0, 0);
d6ba96
  	 /* check for the characters now, otherwise they might be eaten by
d6ba96
--- 3379,3386 ----
d6ba96
  	 out_str(buf);
d6ba96
  	 out_str(T_U7);
d6ba96
  	 u7_status = U7_SENT;
d6ba96
! 	 out_flush();
d6ba96
! 	 term_windgoto(1, 0);
d6ba96
  	 out_str((char_u *)"  ");
d6ba96
  	 term_windgoto(0, 0);
d6ba96
  	 /* check for the characters now, otherwise they might be eaten by
d6ba96
***************
d6ba96
*** 4185,4208 ****
d6ba96
  			    || (tp[0] == CSI && len >= 2))
d6ba96
  			&& (VIM_ISDIGIT(*p) || *p == '>' || *p == '?'))
d6ba96
  	    {
d6ba96
  		j = 0;
d6ba96
  		extra = 0;
d6ba96
  		for (i = 2 + (tp[0] != CSI); i < len
d6ba96
  				&& !(tp[i] >= '{' && tp[i] <= '~')
d6ba96
  				&& !ASCII_ISALPHA(tp[i]); ++i)
d6ba96
  		    if (tp[i] == ';' && ++j == 1)
d6ba96
  			extra = i + 1;
d6ba96
  		if (i == len)
d6ba96
  		{
d6ba96
  		    LOG_TR("Not enough characters for CRV");
d6ba96
  		    return -1;
d6ba96
  		}
d6ba96
- 
d6ba96
  #ifdef FEAT_MBYTE
d6ba96
! 		/* Eat it when it has 2 arguments and ends in 'R'. Ignore it
d6ba96
! 		 * when u7_status is not "sent", <S-F3> sends something
d6ba96
! 		 * similar. */
d6ba96
! 		if (j == 1 && tp[i] == 'R' && u7_status == U7_SENT)
d6ba96
  		{
d6ba96
  		    char *aw = NULL;
d6ba96
  
d6ba96
--- 4186,4223 ----
d6ba96
  			    || (tp[0] == CSI && len >= 2))
d6ba96
  			&& (VIM_ISDIGIT(*p) || *p == '>' || *p == '?'))
d6ba96
  	    {
d6ba96
+ #ifdef FEAT_MBYTE
d6ba96
+ 		int col;
d6ba96
+ 		int row_char;
d6ba96
+ #endif
d6ba96
  		j = 0;
d6ba96
  		extra = 0;
d6ba96
  		for (i = 2 + (tp[0] != CSI); i < len
d6ba96
  				&& !(tp[i] >= '{' && tp[i] <= '~')
d6ba96
  				&& !ASCII_ISALPHA(tp[i]); ++i)
d6ba96
  		    if (tp[i] == ';' && ++j == 1)
d6ba96
+ 		    {
d6ba96
  			extra = i + 1;
d6ba96
+ #ifdef FEAT_MBYTE
d6ba96
+ 			row_char = tp[i - 1];
d6ba96
+ #endif
d6ba96
+ 		    }
d6ba96
  		if (i == len)
d6ba96
  		{
d6ba96
  		    LOG_TR("Not enough characters for CRV");
d6ba96
  		    return -1;
d6ba96
  		}
d6ba96
  #ifdef FEAT_MBYTE
d6ba96
! 		if (extra > 0)
d6ba96
! 		    col = atoi((char *)tp + extra);
d6ba96
! 		else
d6ba96
! 		    col = 0;
d6ba96
! 
d6ba96
! 		/* Eat it when it has 2 arguments and ends in 'R'. Also when
d6ba96
! 		 * u7_status is not "sent", it may be from a previous Vim that
d6ba96
! 		 * just exited.  But not for <S-F3>, it sends something
d6ba96
! 		 * similar, check for row and column to make sense. */
d6ba96
! 		if (j == 1 && tp[i] == 'R' && row_char == '2' && col >= 2)
d6ba96
  		{
d6ba96
  		    char *aw = NULL;
d6ba96
  
d6ba96
***************
d6ba96
*** 4211,4228 ****
d6ba96
  # ifdef FEAT_AUTOCMD
d6ba96
  		    did_cursorhold = TRUE;
d6ba96
  # endif
d6ba96
! 		    if (extra > 0)
d6ba96
! 			extra = atoi((char *)tp + extra);
d6ba96
! 		    if (extra == 2)
d6ba96
  			aw = "single";
d6ba96
! 		    else if (extra == 3)
d6ba96
  			aw = "double";
d6ba96
  		    if (aw != NULL && STRCMP(aw, p_ambw) != 0)
d6ba96
  		    {
d6ba96
  			/* Setting the option causes a screen redraw. Do that
d6ba96
  			 * right away if possible, keeping any messages. */
d6ba96
  			set_option_value((char_u *)"ambw", 0L, (char_u *)aw, 0);
d6ba96
! #ifdef DEBUG_TERMRESPONSE
d6ba96
  			{
d6ba96
  			    char buf[100];
d6ba96
  			    int  r = redraw_asap(CLEAR);
d6ba96
--- 4226,4241 ----
d6ba96
  # ifdef FEAT_AUTOCMD
d6ba96
  		    did_cursorhold = TRUE;
d6ba96
  # endif
d6ba96
! 		    if (col == 2)
d6ba96
  			aw = "single";
d6ba96
! 		    else if (col == 3)
d6ba96
  			aw = "double";
d6ba96
  		    if (aw != NULL && STRCMP(aw, p_ambw) != 0)
d6ba96
  		    {
d6ba96
  			/* Setting the option causes a screen redraw. Do that
d6ba96
  			 * right away if possible, keeping any messages. */
d6ba96
  			set_option_value((char_u *)"ambw", 0L, (char_u *)aw, 0);
d6ba96
! # ifdef DEBUG_TERMRESPONSE
d6ba96
  			{
d6ba96
  			    char buf[100];
d6ba96
  			    int  r = redraw_asap(CLEAR);
d6ba96
***************
d6ba96
*** 4231,4239 ****
d6ba96
  									   r);
d6ba96
  			    log_tr(buf);
d6ba96
  			}
d6ba96
! #else
d6ba96
  			redraw_asap(CLEAR);
d6ba96
! #endif
d6ba96
  		    }
d6ba96
  		    key_name[0] = (int)KS_EXTRA;
d6ba96
  		    key_name[1] = (int)KE_IGNORE;
d6ba96
--- 4244,4252 ----
d6ba96
  									   r);
d6ba96
  			    log_tr(buf);
d6ba96
  			}
d6ba96
! # else
d6ba96
  			redraw_asap(CLEAR);
d6ba96
! # endif
d6ba96
  		    }
d6ba96
  		    key_name[0] = (int)KS_EXTRA;
d6ba96
  		    key_name[1] = (int)KE_IGNORE;
d6ba96
*** ../vim-7.4.206/src/version.c	2014-03-19 12:37:18.537826062 +0100
d6ba96
--- src/version.c	2014-03-19 12:55:42.249842974 +0100
d6ba96
***************
d6ba96
*** 740,741 ****
d6ba96
--- 740,743 ----
d6ba96
  {   /* Add new patch number below this line */
d6ba96
+ /**/
d6ba96
+     207,
d6ba96
  /**/
d6ba96
d6ba96
-- 
d6ba96
How come wrong numbers are never busy?
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    ///