Mercurial > vim
changeset 20850:bf7453768034 v8.2.0977
patch 8.2.0977: t_8u is made empty for the wrong terminals
Commit: https://github.com/vim/vim/commit/8dff4cbf6d917e20322821e8e866ffc1c1175258
Author: Bram Moolenaar <Bram@vim.org>
Date: Sun Jun 14 14:34:16 2020 +0200
patch 8.2.0977: t_8u is made empty for the wrong terminals
Problem: t_8u is made empty for the wrong terminals. (Dominique Pelle)
Solution: Invert the check for TPR_YES. (closes https://github.com/vim/vim/issues/6254)
| author | Bram Moolenaar <Bram@vim.org> |
|---|---|
| date | Sun, 14 Jun 2020 14:45:03 +0200 |
| parents | a13abf01e8cd |
| children | f4c1966646f3 |
| files | src/term.c src/testdir/test_termcodes.vim src/version.c |
| diffstat | 3 files changed, 10 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/src/term.c +++ b/src/term.c @@ -4637,6 +4637,7 @@ handle_version_response(int first, int * // "xterm-256color" but are not fully xterm compatible. // // Gnome terminal sends 1;3801;0, 1;4402;0 or 1;2501;0. + // Newer Gnome-terminal sends 65;6001;1. // xfce4-terminal sends 1;2802;0. // screen sends 83;40500;0 // Assuming any version number over 2500 is not an @@ -4688,7 +4689,7 @@ handle_version_response(int first, int * // 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) + 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
--- a/src/testdir/test_termcodes.vim +++ b/src/testdir/test_termcodes.vim @@ -1013,6 +1013,8 @@ func Test_xx04_Mac_Terminal_response() call test_override('term_props', 1) set ttymouse=xterm + " t_8u is not reset + let &t_8u = "\<Esc>[58;2;%lu;%lu;%lum" call test_option_not_set('ttymouse') let seq = "\<Esc>[>1;95;0c" call feedkeys(seq, 'Lx!') @@ -1025,6 +1027,7 @@ func Test_xx04_Mac_Terminal_response() \ underline_rgb: 'y', \ mouse: 's' \ }, terminalprops()) + call assert_equal("\<Esc>[58;2;%lu;%lu;%lum", &t_8u) " Reset is_not_xterm and is_mac_terminal. set t_RV= @@ -1158,9 +1161,10 @@ func Test_xx07_xterm_response() \ mouse: 's' \ }, terminalprops()) - " xterm >= 279: "sgr" and cursor_style not reset + " xterm >= 279: "sgr" and cursor_style not reset; also check t_8u reset set ttymouse=xterm call test_option_not_set('ttymouse') + let &t_8u = "\<Esc>[58;2;%lu;%lu;%lum" let seq = "\<Esc>[>0;279;0c" call feedkeys(seq, 'Lx!') call assert_equal(seq, v:termresponse) @@ -1172,6 +1176,7 @@ func Test_xx07_xterm_response() \ underline_rgb: 'u', \ mouse: 's' \ }, terminalprops()) + call assert_equal('', &t_8u) set t_RV= call test_override('term_props', 0)