diff --git a/src/main.c b/src/main.c index e0d0e7e37f..cac5db145e 100644 --- a/src/main.c +++ b/src/main.c @@ -892,10 +892,6 @@ vim_main2(void) may_req_termresponse(); may_req_bg_color(); - - // Same reason as for termresponse: don't want the terminal sending out - // the DECRPM response after Vim has exited. - may_req_decrqm(); # endif // start in insert mode diff --git a/src/proto/term.pro b/src/proto/term.pro index 5ec909f46c..9653a35af3 100644 --- a/src/proto/term.pro +++ b/src/proto/term.pro @@ -60,7 +60,6 @@ void stoptermcap(void); void may_req_termresponse(void); void check_terminal_behavior(void); void may_req_bg_color(void); -void may_req_decrqm(void); int swapping_screen(void); void scroll_start(void); void cursor_on_force(void); diff --git a/src/term.c b/src/term.c index 876e15d950..07eb5974b7 100644 --- a/src/term.c +++ b/src/term.c @@ -690,15 +690,17 @@ static tcap_entry_T builtin_sync_output[] = { }; #endif +#ifdef FEAT_TERMRESPONSE /* * List of DECRQM modes that Vim supports */ static const int dec_modes[] = { 2026, // Synchronized output -#ifdef UNIX +# ifdef UNIX 2048 // In-band terminal resize events -#endif +# endif }; +#endif #ifdef FEAT_TERMGUICOLORS /* @@ -4312,36 +4314,6 @@ may_req_bg_color(void) } } -/* - * Query the settings for the DEC modes we support via DECRQM. - * Only sent once, and only when the terminal is known not to dislike it - * (i.e. TPR_DECRQM is TPR_YES, or still TPR_UNKNOWN when the version response - * has not yet been received). - * The DECRPM responses are caught in handle_csi(). - */ - void -may_req_decrqm(void) -{ - if (decrqm_status.tr_progress == STATUS_GET - && term_props[TPR_DECRQM].tpr_status != TPR_NO - && can_get_termresponse() - && starting == 0) - { - MAY_WANT_TO_LOG_THIS; - LOG_TR1("Sending DECRQM requests"); - for (int i = 0; i < (int)ARRAY_LENGTH(dec_modes); i++) - { - vim_snprintf((char *)IObuff, IOSIZE, "\033[?%d$p", dec_modes[i]); - out_str(IObuff); - } - termrequest_sent(&decrqm_status); - // check for the characters now, otherwise they might be eaten by - // get_keystroke() - out_flush(); - (void)vpeekc_nomap(); - } -} - #endif /* @@ -5398,6 +5370,23 @@ handle_version_response(int first, int *arg, int argc, char_u *tp) need_flush = TRUE; } + // Only request DEC modes via DECRQM when the terminal is known to + // handle it. Not for Apple Terminal.app or GNU screen, they echo + // the trailing "p" to the screen. See issue #19852. + if (decrqm_status.tr_progress == STATUS_GET + && term_props[TPR_DECRQM].tpr_status == TPR_YES) + { + MAY_WANT_TO_LOG_THIS; + LOG_TR1("Sending DECRQM requests"); + for (int i = 0; i < (int)ARRAY_LENGTH(dec_modes); i++) + { + vim_snprintf((char *)IObuff, IOSIZE, "\033[?%d$p", dec_modes[i]); + out_str(IObuff); + } + termrequest_sent(&decrqm_status); + need_flush = TRUE; + } + if (need_flush) out_flush(); #endif diff --git a/src/version.c b/src/version.c index 95f20bc670..bfd90a13fc 100644 --- a/src/version.c +++ b/src/version.c @@ -729,6 +729,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 389, /**/ 388, /**/