Re: Odd behavior of vim 8.0.1203 in a wide xterm -- solution found

[Christian Brabandt]

On Do, 02 Nov 2017, Gary Johnson wrote:

> On 2017-11-02, Gary Johnson wrote:
> 
> > So the funny behavior does seem related to the old addressing
> > style, but why is it happening on an xterm-318 with 'ttymouse'
> > automatically set to "sgr"?
> 
> I tried finding the problem by bisecting the repository and found
> that my build of 7.4.160 failed, too.  That led me to discover that
> my normal build was missing the mouse_sgr feature.  I added
> -DFEAT_MOUSE_SGR to CFLAGS and rebuilt and all is good.
> 
> Thanks to James McCoy for directing my attention to SGR.

So you could `:set ttymouse=sgr` but it didn't work because your vim was 
build without the FEAT_MOUSE_SGR feature? We should need an error 
message then, right? Perhaps the attached patch should be included.

Christian
-- 
Zeig mir deine Uhr, und ich sage dir, wie spät es ist.

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

diff --git a/src/os_unix.c b/src/os_unix.c
index a39caffe4..765cbaa32 100644
--- a/src/os_unix.c
+++ b/src/os_unix.c
@@ -3567,8 +3567,8 @@ mch_setmouse(int on)
 
     xterm_mouse_vers = use_xterm_mouse();
 
-# ifdef FEAT_MOUSE_URXVT
     if (ttym_flags == TTYM_URXVT)
+# ifdef FEAT_MOUSE_URXVT
     {
 	out_str_nf((char_u *)
 		   (on
@@ -3576,10 +3576,12 @@ mch_setmouse(int on)
 		   : IF_EB("\033[?1015l", ESC_STR "[?1015l")));
 	ison = on;
     }
+# else
+	emsg((char_u *)"Exxx: Option value \"urxvt\" for 'ttymouse' not compiled in");
 # endif
 
-# ifdef FEAT_MOUSE_SGR
     if (ttym_flags == TTYM_SGR)
+# ifdef FEAT_MOUSE_SGR
     {
 	out_str_nf((char_u *)
 		   (on
@@ -3587,6 +3589,8 @@ mch_setmouse(int on)
 		   : IF_EB("\033[?1006l", ESC_STR "[?1006l")));
 	ison = on;
     }
+# else
+	emsg((char_u *)"Exxx: Option value \"sgr\" for 'ttymouse' not compiled in");
 # endif
 
     if (xterm_mouse_vers > 0)
@@ -3604,8 +3608,8 @@ mch_setmouse(int on)
 	ison = on;
     }
 
-# ifdef FEAT_MOUSE_DEC
     else if (ttym_flags == TTYM_DEC)
+# ifdef FEAT_MOUSE_DEC
     {
 	if (on)	/* enable mouse events */
 	    out_str_nf((char_u *)"\033[1;2'z\033[1;3'{");
@@ -3613,6 +3617,8 @@ mch_setmouse(int on)
 	    out_str_nf((char_u *)"\033['z");
 	ison = on;
     }
+# else
+	emsg((char_u *)"Exxx: Option value \"dec\" for 'ttymouse' not compiled in");
 # endif
 
 # ifdef FEAT_MOUSE_GPM
@@ -3647,8 +3653,8 @@ mch_setmouse(int on)
     }
 # endif
 
+    if (ttym_flags == TTYM_JSBTERM)
 # ifdef FEAT_MOUSE_JSB
-    else
     {
 	if (on)
 	{
@@ -3684,9 +3690,11 @@ mch_setmouse(int on)
 	    ison = FALSE;
 	}
     }
+# else
+	emsg((char_u *)"Exxx: Option value \"jsbterm\" for 'ttymouse' not compiled in");
 # endif
+    else if (ttym_flags == TTYM_PTERM)
 # ifdef FEAT_MOUSE_PTERM
-    else
     {
 	/* 1 = button press, 6 = release, 7 = drag, 1h...9l = right button */
 	if (on)
@@ -3695,6 +3703,8 @@ mch_setmouse(int on)
 	    out_str_nf("\033[>1l\033[>6l\033[>7l\033[>1l\033[>9h");
 	ison = on;
     }
+# else
+	emsg((char_u *)"Exxx: Option value \"pterm\" for 'ttymouse' not compiled in");
 # endif
 }