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