vim: src/term.c comparison

comparison src/term.c @ 20778:6d5e233bac9c v8.2.0941

patch 8.2.0941: detecting terminal properties is unstructured Commit: https://github.com/vim/vim/commit/517f00f78893087d2f9d45768a5ba4dc48cb6856 Author: Bram Moolenaar <Bram@vim.org> Date: Wed Jun 10 12:15:51 2020 +0200 patch 8.2.0941: detecting terminal properties is unstructured Problem: Detecting terminal properties is unstructured. Solution: Add a table with terminal properties. Set properties when a terminal is detected.

author	Bram Moolenaar <Bram@vim.org>
date	Wed, 10 Jun 2020 12:30:04 +0200
parents	7728e309e013
children	9064044fd4f6

comparison

equal deleted inserted replaced

-:9e7672e41fba
+:6d5e233bac9c
 static int	need_gather = FALSE;	    // need to fill termleader[]
 static char_u	termleader[256 + 1];	    // for check_termcode()
 #ifdef FEAT_TERMRESPONSE
 static int	check_for_codes = FALSE;    // check for key code response
-static int	is_not_xterm = FALSE;	    // recognized not-really-xterm
-static int	xcc_test_failed = FALSE;    // xcc_status check failed
+/*
+* Structure and table to store terminal features that can be detected by
+* querying the terminal.  Either by inspecting the termresponse or a more
+* specific request.  Besides this there are:
+* t_colors - number of colors supported
+*/
+typedef struct {
+char    *tpr_name;
+int	    tpr_set_by_termresponse;
+int	    tpr_status;
+} termprop_T;
+// Values for tpr_status.
+#define TPR_UNKNOWN	    'u'
+#define TPR_YES		    'y'
+#define TPR_NO		    'n'
+#define TPR_MOUSE_XTERM     'x'	// use "xterm" for 'ttymouse'
+#define TPR_MOUSE_XTERM2    '2'	// use "xterm2" for 'ttymouse'
+#define TPR_MOUSE_SGR	    's'	// use "sgr" for 'ttymouse'
+// can request the cursor style without messing up the display
+#define TPR_CURSOR_STYLE	    0
+// can request the cursor blink mode without messing up the display
+#define TPR_CURSOR_BLINK	    1
+// can set the underline color with t_8u without resetting other colors
+#define TPR_UNDERLINE_RGB	    2
+// mouse support - TPR_MOUSE_XTERM, TPR_MOUSE_XTERM2 or TPR_MOUSE_SGR
+#define TPR_MOUSE		    3
+// table size
+#define TPR_COUNT		    4
+static termprop_T term_props[TPR_COUNT];
+/*
+* Initialize the term_props table.
+* When "all" is FALSE only set those that are detected from the version
+* response.
+*/
+static void
+init_term_props(int all)
+{
+int i;
+term_props[TPR_CURSOR_STYLE].tpr_name = "cursor_style";
+term_props[TPR_CURSOR_STYLE].tpr_set_by_termresponse = FALSE;
+term_props[TPR_CURSOR_BLINK].tpr_name = "cursor_blink_mode";
+term_props[TPR_CURSOR_BLINK].tpr_set_by_termresponse = FALSE;
+term_props[TPR_UNDERLINE_RGB].tpr_name = "underline_rgb";
+term_props[TPR_UNDERLINE_RGB].tpr_set_by_termresponse = TRUE;
+term_props[TPR_MOUSE].tpr_name = "mouse";
+term_props[TPR_MOUSE].tpr_set_by_termresponse = TRUE;
+for (i = 0; i < TPR_COUNT; ++i)
+	if (all || term_props[i].tpr_set_by_termresponse)
+	    term_props[i].tpr_status = TPR_UNKNOWN;
+}
 #endif
 static struct builtin_term *
 find_builtin_term(char_u *term)
 {
 #if defined(UNIX) || defined(VMS)
 term_is_xterm = vim_is_xterm(term);
 #endif
 #ifdef FEAT_TERMRESPONSE
-is_not_xterm = FALSE;
+// Reset terminal properties that are set based on the termresponse, which
-is_mac_terminal = FALSE;
+// will be sent out soon.
+init_term_props(FALSE);
 #endif
 #if defined(UNIX) || defined(VMS)
 /*
 * For Unix, set the 'ttymouse' option to the type of mouse to be used.
 }
 /*
 * Send sequences to the terminal and check with t_u7 how the cursor moves, to
 * find out properties of the terminal.
+* Note that this goes out before T_CRV, so that the result can be used when
+* the termresponse arrives.
 */
 void
 check_terminal_behavior(void)
 {
 int	    did_send = FALSE;
+init_term_props(TRUE);
 if (!can_get_termresponse() || starting != 0 || *T_U7 == NUL)
 	return;
 if (u7_status.tr_progress == STATUS_GET
 # endif
 	}
 }
 else if (arg[0] == 3)
 {
+	int value;
 	LOG_TR(("Received compatibility test result: %s", tp));
+	xcc_status.tr_progress = STATUS_GOT;
 	// Third row: xterm compatibility test.
-	// If the cursor is not on the first column then the
+	// If the cursor is on the first column then the terminal can handle
-	// terminal is not xterm compatible.
+	// the request for cursor style and blinking.
-	if (arg[1] != 1)
+	value = arg[1] == 1 ? TPR_YES : TPR_NO;
-	    xcc_test_failed = TRUE;
+	term_props[TPR_CURSOR_STYLE].tpr_status = value;
-	xcc_status.tr_progress = STATUS_GOT;
+	term_props[TPR_CURSOR_BLINK].tpr_status = value;
 }
 }
 /*
-* Handle a response to T_CRV.
+* Handle a response to T_CRV: {lead}{first}{x};{vers};{y}c
+* Xterm and alikes use '>' for {first}.
+* Rxvt sends "{lead}?1;2c".
 */
 static void
 handle_version_response(int first, int *arg, int argc, char_u *tp)
 {
+// The xterm version.  It is set to zero when it can't be an actual xterm
+// version.
 int version = arg[1];
 LOG_TR(("Received CRV response: %s", tp));
 crv_status.tr_progress = STATUS_GOT;
 did_cursorhold = TRUE;
+// Reset terminal properties that are set based on the termresponse.
+// Mainly useful for tests that send the termresponse multiple times.
+init_term_props(FALSE);
 // If this code starts with CSI, you can bet that the
 // terminal uses 8-bit codes.
 if (tp[0] == CSI)
 	switch_to_8bit();
 // Ignore it for when the user has set 'term' to xterm,
 // even though it's an rxvt.
 if (version > 20000)
 	version = 0;
+// Figure out more if the reeponse is CSI > 99 ; 99 ; 99 c
 if (first == '>' && argc == 3)
 {
 	int need_flush = FALSE;
-	int is_iterm2 = FALSE;
-	int is_mintty = FALSE;
-	int is_screen = FALSE;
 	// mintty 2.9.5 sends 77;20905;0c.
 	// (77 is ASCII 'M' for mintty.)
 	if (arg[0] == 77)
-	    is_mintty = TRUE;
+	{
+	    // mintty can do SGR mouse reporting
-	// if xterm version >= 141 try to get termcap codes
+	    term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_SGR;
+	}
+	// If xterm version >= 141 try to get termcap codes.  For other
+	// terminals the request should be ignored.
 	if (version >= 141)
 	{
 	    LOG_TR(("Enable checking for XT codes"));
 	    check_for_codes = TRUE;
 	    need_gather = TRUE;
 	    // colors the terminal supports.  Otherwise assume
 	    // 256, libvterm supports even more.
 	    if (mch_getenv((char_u *)"COLORS") == NULL)
 		may_adjust_color_count(256);
 	    // Libvterm can handle SGR mouse reporting.
-	    if (!option_was_set((char_u *)"ttym"))
+	    term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_SGR;
-		set_option_value((char_u *)"ttym", 0L,
-					   (char_u *)"sgr", 0);
 	}
 	if (version == 95)
 	{
 	    // Mac Terminal.app sends 1;95;0
 	    if (arg[0] == 1 && arg[2] == 0)
 	    {
-		is_not_xterm = TRUE;
+		term_props[TPR_UNDERLINE_RGB].tpr_status = TPR_YES;
-		is_mac_terminal = TRUE;
+		term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_SGR;
 	    }
 	    // iTerm2 sends 0;95;0
 	    else if (arg[0] == 0 && arg[2] == 0)
-		is_iterm2 = TRUE;
+	    {
+		// iTerm2 can do SGR mouse reporting
+		term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_SGR;
+	    }
 	    // old iTerm2 sends 0;95;
 	    else if (arg[0] == 0 && arg[2] == -1)
-		is_not_xterm = TRUE;
+		term_props[TPR_UNDERLINE_RGB].tpr_status = TPR_YES;
 	}
 	// screen sends 83;40500;0 83 is 'S' in ASCII.
 	if (arg[0] == 83)
-	    is_screen = TRUE;
+	{
+	    // screen supports SGR mouse codes since 4.7.0
-	// Only set 'ttymouse' automatically if it was not set
+	    if (arg[1] >= 40700)
-	// by the user already.
+		term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_SGR;
-	if (!option_was_set((char_u *)"ttym"))
+	    else
-	{
+		term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_XTERM;
-	    // Xterm version 277 supports SGR.  Also support
+	}
-	    // Terminal.app, iTerm2, mintty, and screen 4.7+.
-	    if ((!is_screen && version >= 277)
+	// If no recognized terminal has set mouse behavior, assume xterm.
-		    || is_iterm2
+	if (term_props[TPR_MOUSE].tpr_status == TPR_UNKNOWN)
-		    || is_mac_terminal
+	{
-		    || is_mintty
+	    // Xterm version 277 supports SGR.
-		    || (is_screen && arg[1] >= 40700))
+	    // Xterm version >= 95 supports mouse dragging.
-		set_option_value((char_u *)"ttym", 0L,
+	    if (version >= 277)
-					  (char_u *)"sgr", 0);
+		term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_SGR;
-	    // For xterm version >= 95 mouse dragging works.
 	    else if (version >= 95)
-		set_option_value((char_u *)"ttym", 0L,
+		term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_XTERM2;
-				       (char_u *)"xterm2", 0);
 	}
 	// Detect terminals that set $TERM to something like
 	// "xterm-256color" but are not fully xterm compatible.
+	//
 	// Gnome terminal sends 1;3801;0, 1;4402;0 or 1;2501;0.
 	// xfce4-terminal sends 1;2802;0.
 	// screen sends 83;40500;0
 	// Assuming any version number over 2500 is not an
 	// xterm (without the limit for rxvt and screen).
 	if (arg[1] >= 2500)
-	    is_not_xterm = TRUE;
+	    term_props[TPR_UNDERLINE_RGB].tpr_status = TPR_YES;
-	// PuTTY sends 0;136;0
-	// vandyke SecureCRT sends 1;136;0
 	else if (version == 136 && arg[2] == 0)
 	{
-	    is_not_xterm = TRUE;
+	    term_props[TPR_UNDERLINE_RGB].tpr_status = TPR_YES;
-	    // PuTTY supports sgr-like mouse reporting, but
+	    // PuTTY sends 0;136;0
-	    // only set 'ttymouse' if it was not set by the
+	    if (arg[0] == 0)
-	    // user already.
+	    {
-	    if (arg[0] == 0
+		// supports sgr-like mouse reporting.
-			  && !option_was_set((char_u *)"ttym"))
+		term_props[TPR_MOUSE].tpr_status = TPR_MOUSE_SGR;
-		set_option_value((char_u *)"ttym", 0L,
+	    }
-					(char_u *)"sgr", 0);
+	    // vandyke SecureCRT sends 1;136;0
 	}
 	// Konsole sends 0;115;0
 	else if (version == 115 && arg[0] == 0 && arg[2] == 0)
-	    is_not_xterm = TRUE;
+	    term_props[TPR_UNDERLINE_RGB].tpr_status = TPR_YES;
 	// GNU screen sends 83;30600;0, 83;40500;0, etc.
-	// 30600/40500 is a version number of GNU screen. DA2
+	// 30600/40500 is a version number of GNU screen. DA2 support is added
-	// support is added on 3.6.  DCS string has a special
+	// on 3.6.  DCS string has a special meaning to GNU screen, but xterm
-	// meaning to GNU screen, but xterm compatibility
+	// compatibility checking does not detect GNU screen.
-	// checking does not detect GNU screen.
+	if (arg[0] == 83 && arg[1] >= 30600)
-	if (version >= 30600 && arg[0] == 83)
+	{
-	    xcc_test_failed = TRUE;
+	    term_props[TPR_CURSOR_STYLE].tpr_status = TPR_NO;
+	    term_props[TPR_CURSOR_BLINK].tpr_status = TPR_NO;
+	}
 	// Xterm first responded to this request at patch level
-	// 95, so assume anything below 95 is not xterm.
+	// 95, so assume anything below 95 is not xterm and hopefully supports
+	// the underline RGB color sequence.
 	if (version < 95)
-	    is_not_xterm = TRUE;
+	    term_props[TPR_UNDERLINE_RGB].tpr_status = TPR_YES;
-	// With the real Xterm setting the underline RGB color
+	// Getting the cursor style is only supported properly by xterm since
-	// clears the background color, disable "t_8u".
+	// version 279 (otherwise it returns 0x18).
-	if (!is_not_xterm && *T_8U != NUL)
+	if (version < 279)
+	    term_props[TPR_CURSOR_STYLE].tpr_status = TPR_NO;
+	/*
+	 * Take action on the detected properties.
+	 */
+	// Unless the underline RGB color is expected to work, disable "t_8u".
+	// It does not work for the real Xterm, it resets the background color.
+	if (term_props[TPR_UNDERLINE_RGB].tpr_status == TPR_YES && *T_8U != NUL)
 	    T_8U = empty_option;
+	// Only set 'ttymouse' automatically if it was not set
+	// by the user already.
+	if (!option_was_set((char_u *)"ttym")
+		&& (term_props[TPR_MOUSE].tpr_status == TPR_MOUSE_XTERM2
+		    || term_props[TPR_MOUSE].tpr_status == TPR_MOUSE_SGR))
+	{
+	    set_option_value((char_u *)"ttym", 0L,
+		    term_props[TPR_MOUSE].tpr_status == TPR_MOUSE_SGR
+				    ? (char_u *)"sgr" : (char_u *)"xterm2", 0);
+	}
 	// Only request the cursor style if t_SH and t_RS are
 	// set. Only supported properly by xterm since version
 	// 279 (otherwise it returns 0x18).
-	// Only when the xcc_status was set, the test finished,
+	// Only when getting the cursor style was detected to work.
-	// and xcc_test_failed is FALSE;
 	// Not for Terminal.app, it can't handle t_RS, it
 	// echoes the characters to the screen.
 	if (rcs_status.tr_progress == STATUS_GET
-		&& xcc_status.tr_progress == STATUS_GOT
+		&& term_props[TPR_CURSOR_STYLE].tpr_status == TPR_YES
-		&& !xcc_test_failed
-		&& version >= 279
 		&& *T_CSH != NUL
 		&& *T_CRS != NUL)
 	{
 	    LOG_TR(("Sending cursor style request"));
 	    out_str(T_CRS);
 	}
 	// Only request the cursor blink mode if t_RC set. Not
 	// for Gnome terminal, it can't handle t_RC, it
 	// echoes the characters to the screen.
-	// Only when the xcc_status was set, the test finished,
+	// Only when getting the cursor style was detected to work.
-	// and xcc_test_failed is FALSE;
 	if (rbm_status.tr_progress == STATUS_GET
-		&& xcc_status.tr_progress == STATUS_GOT
+		&& term_props[TPR_CURSOR_BLINK].tpr_status == TPR_YES
-		&& !xcc_test_failed
 		&& *T_CRC != NUL)
 	{
 	    LOG_TR(("Sending cursor blink mode request"));
 	    out_str(T_CRC);
 	    termrequest_sent(&rbm_status);
 return new_slen - csi_len + offset;
 }
 /*
 * Handle a CSI escape sequence.
-* - Xterm version string: {lead}>{x};{vers};{y}c
+* - Xterm version string.
-*   Also eat other possible responses to t_RV, rxvt returns
-*   "{lead}?1;2c".
 *
 * - Cursor position report: {lead}{row};{col}R
 *   The final byte must be 'R'. It is used for checking the
 *   ambiguous-width character state.
 *

Mercurial > vim

comparison src/term.c @ 20778:6d5e233bac9c v8.2.0941