vim: src/term.c comparison

comparison src/term.c @ 5090:8b7baf39a345 v7.3.1288

updated for version 7.3.1288 Problem: The first ":echo 'hello'" command output doesn't show. Mapping for <S-F3> gets triggered during startup. Solution: Add debugging code for the termresponse. When receiving the "Co" entry and when setting 'ambiwidth' redraw right away if possible. Add redraw_asap(). Don't set 'ambiwidth' if it already had the right value. Do the 'ambiwidth' check in the second row to avoid confusion with <S-F3>.

author	Bram Moolenaar <bram@vim.org>
date	Wed, 03 Jul 2013 12:45:31 +0200
parents	19ed30f7cef7
children	6d11572e2c8b

comparison

equal deleted inserted replaced

-:941318315ace
+:8b7baf39a345
 * files have been disabled by the define at the start of this file.
 */
 char		*tgetstr __ARGS((char *, char **));
 # ifdef FEAT_TERMRESPONSE
+/* Change this to "if 1" to debug what happens with termresponse. */
+#  if 0
+#   define DEBUG_TERMRESPONSE
+static void log_tr(char *msg);
+#   define LOG_TR(msg) log_tr(msg)
+#  else
+#   define LOG_TR(msg)
+#  endif
 /* Request Terminal Version status: */
 #  define CRV_GET	1	/* send T_CRV when switched to RAW mode */
 #  define CRV_SENT	2	/* did send T_CRV, waiting for answer */
 #  define CRV_GOT	3	/* received T_CRV response */
 static int crv_status = CRV_GET;
 ttest(TRUE);	/* make sure we have a valid set of terminal codes */
 full_screen = TRUE;		/* we can use termcap codes from now on */
 set_term_defaults();	/* use current values as defaults */
 #ifdef FEAT_TERMRESPONSE
+LOG_TR("setting crv_status to CRV_GET");
 crv_status = CRV_GET;	/* Get terminal version later */
 #endif
 /*
 * Initialize the terminal with the appropriate termcap codes.
 	    && isatty(1)
 	    && isatty(read_cmd_fd)
 # endif
 	    && *T_CRV != NUL)
 {
+	LOG_TR("Sending CRV");
 	out_str(T_CRV);
 	crv_status = CRV_SENT;
 	/* check for the characters now, otherwise they might be eaten by
 	 * get_keystroke() */
 	out_flush();
 }
 # if defined(FEAT_MBYTE) || defined(PROTO)
 /*
 * Check how the terminal treats ambiguous character width (UAX #11).
-* First, we move the cursor to (0, 0) and print a test ambiguous character
+* First, we move the cursor to (1, 0) and print a test ambiguous character
 * \u25bd (WHITE DOWN-POINTING TRIANGLE) and query current cursor position.
-* If the terminal treats \u25bd as single width, the position is (0, 1),
+* If the terminal treats \u25bd as single width, the position is (1, 1),
-* or if it is treated as double width, that will be (0, 2).
+* or if it is treated as double width, that will be (1, 2).
 * This function has the side effect that changes cursor position, so
 * it must be called immediately after entering termcap mode.
 */
 void
 may_req_ambiguous_character_width()
 	    && *T_U7 != NUL
 	    && !option_was_set((char_u *)"ambiwidth"))
 {
 	 char_u	buf[16];
-	 term_windgoto(0, 0);
+	 LOG_TR("Sending U7 request");
+	 /* Do this in the second row.  In the first row the returned sequence
+	  * may be CSI 1;2R, which is the same as <S-F3>. */
+	 term_windgoto(1, 0);
 	 buf[mb_char2bytes(0x25bd, buf)] = 0;
 	 out_str(buf);
 	 out_str(T_U7);
 	 u7_status = U7_SENT;
 	 term_windgoto(0, 0);
 	 /* check for the characters now, otherwise they might be eaten by
 	  * get_keystroke() */
 	 out_flush();
 	 (void)vpeekc_nomap();
 }
+}
+# endif
+# ifdef DEBUG_TERMRESPONSE
+static void
+log_tr(char *msg)
+{
+static FILE *fd_tr = NULL;
+static proftime_T start;
+proftime_T now;
+if (fd_tr == NULL)
+{
+	fd_tr = fopen("termresponse.log", "w");
+	profile_start(&start);
+}
+now = start;
+profile_end(&now);
+fprintf(fd_tr, "%s: %s %s\n",
+	    profile_msg(&now),
+	    must_redraw == NOT_VALID ? "NV"
+					 : must_redraw == CLEAR ? "CL" : "  ",
+	    msg);
 }
 # endif
 #endif
 /*
 	    }
 	}
 	need_gather = TRUE;		/* need to fill termleader[] */
 }
 detected_8bit = TRUE;
+LOG_TR("Switching to 8 bit");
 }
 #endif
 #ifdef CHECK_DOUBLE_CLICK
 static linenr_T orig_topline = 0;
 				&& !(tp[i] >= '{' && tp[i] <= '~')
 				&& !ASCII_ISALPHA(tp[i]); ++i)
 		    if (tp[i] == ';' && ++j == 1)
 			extra = i + 1;
 		if (i == len)
-		    return -1;		/* not enough characters */
+		{
+		    LOG_TR("Not enough characters for CRV");
+		    return -1;
+		}
 #ifdef FEAT_MBYTE
-		/* eat it when it has 2 arguments and ends in 'R' */
+		/* Eat it when it has 2 arguments and ends in 'R'. Ignore it
-		if (j == 1 && tp[i] == 'R')
+		 * when u7_status is not "sent", <S-F3> sends something
+		 * similar. */
+		if (j == 1 && tp[i] == 'R' && u7_status == U7_SENT)
 		{
 		    char *aw = NULL;
+		    LOG_TR("Received U7 status");
 		    u7_status = U7_GOT;
 # ifdef FEAT_AUTOCMD
 		    did_cursorhold = TRUE;
 # endif
 		    if (extra > 0)
 			extra = atoi((char *)tp + extra);
 		    if (extra == 2)
 			aw = "single";
 		    else if (extra == 3)
 			aw = "double";
-		    if (aw != NULL)
+		    if (aw != NULL && STRCMP(aw, p_ambw) != 0)
+		    {
+			/* Setting the option causes a screen redraw. Do that
+			 * right away if possible, keeping any messages. */
 			set_option_value((char_u *)"ambw", 0L, (char_u *)aw, 0);
+#ifdef DEBUG_TERMRESPONSE
+			{
+			    char buf[100];
+			    int  r = redraw_asap(CLEAR);
+			    sprintf(buf, "set 'ambiwidth', redraw_asap(): %d",
+									   r);
+			    log_tr(buf);
+			}
+#else
+			redraw_asap(CLEAR);
+#endif
+		    }
 		    key_name[0] = (int)KS_EXTRA;
 		    key_name[1] = (int)KE_IGNORE;
 		    slen = i + 1;
 		}
 		else
 #endif
 		/* eat it when at least one digit and ending in 'c' */
 		if (*T_CRV != NUL && i > 2 + (tp[0] != CSI) && tp[i] == 'c')
 		{
+		    LOG_TR("Received CRV");
 		    crv_status = CRV_GOT;
 # ifdef FEAT_AUTOCMD
 		    did_cursorhold = TRUE;
 # endif
 			}
 			/* if xterm version >= 141 try to get termcap codes */
 			if (extra >= 141)
 			{
+			    LOG_TR("Enable checking for XT codes");
 			    check_for_codes = TRUE;
 			    need_gather = TRUE;
 			    req_codes_from_term();
 			}
 		    }
 			slen = i + 1 + (tp[i] == ESC);
 			break;
 		    }
 		if (i == len)
+		{
+		    LOG_TR("not enough characters for XT");
 		    return -1;		/* not enough characters */
+		}
 	    }
 	}
 #endif
 	if (key_name[0] == NUL)
 	    *buflen = *buflen + extra + new_slen;
 	}
 	return retval == 0 ? (len + extra + offset) : retval;
 }
+#ifdef FEAT_TERMRESPONSE
+LOG_TR("normal character");
+#endif
 return 0;			    /* no match found */
 }
 /*
 * Replace any terminal code strings in from[] with the equivalent internal
 /* Send up to 10 more requests out than we received.  Avoid sending too
 * many, there can be a buffer overflow somewhere. */
 while (xt_index_out < xt_index_in + 10 && key_names[xt_index_out] != NULL)
 {
+# ifdef DEBUG_TERMRESPONSE
+	char dbuf[100];
+	sprintf(dbuf, "Requesting XT %d: %s",
+				       xt_index_out, key_names[xt_index_out]);
+	log_tr(dbuf);
+# endif
 	sprintf(buf, "\033P+q%02x%02x\033\\",
 		      key_names[xt_index_out][0], key_names[xt_index_out][1]);
 	out_str_nf((char_u *)buf);
 	++xt_index_out;
 }
 	    {
 		xt_index_in = i;
 		break;
 	    }
 	}
+# ifdef DEBUG_TERMRESPONSE
+	{
+	    char buf[100];
+	    sprintf(buf, "Received XT %d: %s", xt_index_in, (char *)name);
+	    log_tr(buf);
+	}
+# endif
 	if (key_names[i] != NULL)
 	{
 	    for (i = 8; (c = hexhex2nr(code + i)) >= 0; i += 2)
 		str[j++] = c;
 	    str[j] = NUL;
 		     * clears the message.  Try keeping the message if it
 		     * might work. */
 		    set_keep_msg_from_hist();
 		    set_color_count(i);
 		    init_highlight(TRUE, FALSE);
-		    redraw_later(CLEAR);
+#ifdef DEBUG_TERMRESPONSE
+		    {
+			char buf[100];
+			int  r = redraw_asap(CLEAR);
+			sprintf(buf, "Received t_Co, redraw_asap(): %d", r);
+			log_tr(buf);
+		    }
+#else
+		    redraw_asap(CLEAR);
+#endif
 		}
 	    }
 	    else
 	    {
 		/* First delete any existing entry with the same code. */

Mercurial > vim

comparison src/term.c @ 5090:8b7baf39a345 v7.3.1288