Vim Developer list

Bram Moolenaar | 15 Aug 2012 16:21

Patch 7.3.632


Patch 7.3.632
Problem:    Cannot select beyond 222 columns with the mouse in xterm.
Solution:   Add support for SGR mouse tracking. (Hayaki Saito)
Files:      runtime/doc/options.txt, src/feature.h, src/keymap.h, src/misc2.c,
            src/option.h, src/os_unix.c, src/term.c, src/version.c

*** ../vim-7.3.631/runtime/doc/options.txt	2012-07-10 16:49:08.000000000 +0200
--- runtime/doc/options.txt	2012-08-15 16:04:26.000000000 +0200
***************
*** 7383,7388 ****
--- 7424,7435 ----
  	   jsbterm	JSB term mouse handling.
  							*pterm-mouse*
  	   pterm	QNX pterm mouse handling.
+ 							*urxvt-mouse*
+ 	   urxvt	Mouse handling for the urxvt (rxvt-unicode) terminal.
+ 							*sgr-mouse*
+ 	   sgr		Mouse handling for the terminal that emits SGR-styled
+ 			mouse reporting. Works with xterm version 277 or
+ 			later.

  	The mouse handling must be enabled at compile time |+mouse_xterm|
  	|+mouse_dec| |+mouse_netterm|.
***************
*** 7395,7400 ****
--- 7442,7448 ----
  	or "xterm2" already.  The main use of this option is to set it to
  	"xterm", when the terminal name doesn't start with "xterm", but it can
  	handle xterm mouse codes.

(Continue reading)

headers

Tony Mechelynck | 16 Aug 2012 02:13

Re: Patch 7.3.632

On 15/08/12 16:21, Bram Moolenaar wrote:
>
> Patch 7.3.632
> Problem:    Cannot select beyond 222 columns with the mouse in xterm.
> Solution:   Add support for SGR mouse tracking. (Hayaki Saito)
> Files:      runtime/doc/options.txt, src/feature.h, src/keymap.h, src/misc2.c,
>              src/option.h, src/os_unix.c, src/term.c, src/version.c

In the output of :version, the mouse features are currently in the 
following sequence:

+mouse
+mouseshape
+mouse_dec
+mouse_gpm
-mouse_jsbterm
+mouse_netterm
-mouse_sysmouse
+mouse_xterm
+mouse_urxvt
+mouse_sgr

Shouldn't they be reordered alphabetically (i.e. sgr before sysmouse and 
urxvt before xterm)?

Best regards,
Tony.
--

-- 
		Gimmie That Old Time Religion
We will follow Zarathustra,		We will worship like the Druids,

(Continue reading)

headers

Hayaki Saito | 16 Aug 2012 04:03

Re: Patch 7.3.632

Thanks for accepting.

In addition, I noticed a small mismatch.

According to src/feature.h:1056, +mouse_urxvt seems to be included in the BIG feature set.

># ifdef FEAT_BIG
>#  define FEAT_MOUSE_URXVT
># endif

But in runtime/doc/various.txt, it marks as a NORMAL feature.
so I think it should be changed as follows:

diff -r 536aa8b0c934 runtime/doc/various.txt
--- a/runtime/doc/various.txt	Wed Aug 15 17:43:31 2012 +0200
+++ b/runtime/doc/various.txt	Thu Aug 16 10:43:07 2012 +0900
 <at>  <at>  -356,7 +356,7  <at>  <at> 
 N  *+mouse_pterm*		QNX only: pterm mouse handling |qnx-terminal|
 N  *+mouse_sysmouse*	Unix only: *BSD console mouse handling |sysmouse|
 B  *+mouse_sgr*		Unix only: sgr mouse handling |sgr-mouse|
-N  *+mouse_urxvt*		Unix only: urxvt mouse handling |urxvt-mouse|
+B  *+mouse_urxvt*		Unix only: urxvt mouse handling |urxvt-mouse|
 N  *+mouse_xterm*		Unix only: xterm mouse handling |xterm-mouse|
 B  *+multi_byte*		16 and 32 bit characters |multibyte|
    *+multi_byte_ime*	Win32 input method for multibyte chars |multibyte-ime|

On 2012/08/16, at 9:13, Tony Mechelynck wrote:

> On 15/08/12 16:21, Bram Moolenaar wrote:
>>

(Continue reading)