Patch 9.0.0636

4 views
Skip to first unread message

Bram Moolenaar

unread,
Oct 1, 2022, 4:22:59 PM10/1/22
to vim...@googlegroups.com

Patch 9.0.0636
Problem: Underline color does not work in terminals that don't send a
termresponse.
Solution: Do output t_8u if it was set explicitly. (closes #11253)
Files: src/term.c


*** ../vim-9.0.0635/src/term.c 2022-08-26 17:52:47.890406156 +0100
--- src/term.c 2022-10-01 21:13:25.301399966 +0100
***************
*** 153,159 ****

// The t_8u code may default to a value but get reset when the term response is
// received. To avoid redrawing too often, only redraw when t_8u is not reset
! // and it was supposed to be written.
// FALSE -> don't output t_8u yet
// MAYBE -> tried outputing t_8u while FALSE
// OK -> can write t_8u
--- 153,159 ----

// The t_8u code may default to a value but get reset when the term response is
// received. To avoid redrawing too often, only redraw when t_8u is not reset
! // and it was supposed to be written. Unless t_8u was set explicitly.
// FALSE -> don't output t_8u yet
// MAYBE -> tried outputing t_8u while FALSE
// OK -> can write t_8u
***************
*** 3011,3017 ****
term_ul_rgb_color(guicolor_T rgb)
{
# ifdef FEAT_TERMRESPONSE
! if (write_t_8u_state != OK)
write_t_8u_state = MAYBE;
else
# endif
--- 3011,3020 ----
term_ul_rgb_color(guicolor_T rgb)
{
# ifdef FEAT_TERMRESPONSE
! // If the user explicitly sets t_8u then use it. Otherwise wait for
! // termresponse to be received, which is when t_8u would be set and a
! // redraw is needed if it was used.
! if (!option_was_set((char_u *)"t_8u") && write_t_8u_state != OK)
write_t_8u_state = MAYBE;
else
# endif
*** ../vim-9.0.0635/src/version.c 2022-10-01 20:17:13.820291832 +0100
--- src/version.c 2022-10-01 21:15:07.493355747 +0100
***************
*** 701,702 ****
--- 701,704 ----
{ /* Add new patch number below this line */
+ /**/
+ 636,
/**/

--
"Marriage is when a man and woman become as one; the trouble starts
when they try to decide which one"

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
Reply all
Reply to author
Forward
0 new messages