GNU bug report logs - #28658
27.0.50; [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect mouse position

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 28658 in the body.
You can then email your comments to 28658 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Alex <agrambot <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Sat, 30 Sep 2017 16:06:34 -0600

[Message part 1 (text/plain, inline)]

1. emacs -Q -nw
2. M-x xterm-mouse-mode RET
3. Click once, then quickly move the cursor to a different word and click again.
4. Notice that the 2nd click was registered as a double click even
though the cursor moved.
5. Click once, and quickly perform step 3 again. Notice that the 3rd
click was registered as a triple click even though the cursor moved.

I've attached a patch that fixes this behaviour. It would be nice if it
worked with pixel positions rather than character positions, but I'm not
sure how to do that in a terminal Emacs.

[0001-Increase-xterm-click-count-only-with-unchanged-point.patch (text/x-diff, attachment)]

Message #8 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Alex <agrambot <at> gmail.com>
To: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Sat, 30 Sep 2017 21:56:58 -0600

[Message part 1 (text/plain, inline)]

I also found another issue, but it doesn't really warrant a new report.
If you double click right after entering xterm-mouse-mode, then you'll
see:

xterm-mouse-translate-1: Wrong type argument: number-or-marker-p, nil

This is because click-count isn't set to 1 when `xterm-mouse-last-click'
is first set in `xterm-mouse-event'.

Here's a patch:

[0001-Set-xterm-click-count-to-1-even-with-no-last-click.patch (text/x-diff, attachment)]

[Message part 3 (text/plain, inline)]

P.S. Why is the error prefixed by `xterm-mouse-translate-1' instead of
`xterm-mouse-event'?

Message #11 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: martin rudalics <rudalics <at> gmx.at>
To: Alex <agrambot <at> gmail.com>, 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode
 doesn't respect mouse position
Date: Sun, 01 Oct 2017 10:23:50 +0200

+           ((and last-time
+                 (> double-click-time (* 1000 (- this-time last-time)))

Note that here and elsewhere coders assume that ‘double-click-time’ is a
number.  However, it may be also nil or t and we should fix those
instances eventually (see Bug#23419).  In new or changed code, at least.

Thanks, martin

Message #14 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Alex <agrambot <at> gmail.com>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Sun, 01 Oct 2017 12:30:23 -0600

[Message part 1 (text/plain, inline)]

martin rudalics <rudalics <at> gmx.at> writes:

> +           ((and last-time
> +                 (> double-click-time (* 1000 (- this-time last-time)))
>
> Note that here and elsewhere coders assume that ‘double-click-time’ is a
> number.  However, it may be also nil or t and we should fix those
> instances eventually (see Bug#23419).  In new or changed code, at least.

Sure, here's a third patch to deal with that:

[0001-lisp-xt-mouse.el-xterm-mouse-event-Handle-boolean-do.patch (text/x-diff, attachment)]

Message #17 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex <agrambot <at> gmail.com>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Thu, 05 Oct 2017 13:36:00 +0300

> From: Alex <agrambot <at> gmail.com>
> Date: Sat, 30 Sep 2017 16:06:34 -0600
> 
> 1. emacs -Q -nw
> 2. M-x xterm-mouse-mode RET
> 3. Click once, then quickly move the cursor to a different word and click again.
> 4. Notice that the 2nd click was registered as a double click even
> though the cursor moved.
> 5. Click once, and quickly perform step 3 again. Notice that the 3rd
> click was registered as a triple click even though the cursor moved.
> 
> I've attached a patch that fixes this behaviour.

Thanks, I have a comment to this, but in general this is the right
fix, IMO.

> It would be nice if it worked with pixel positions rather than
> character positions, but I'm not sure how to do that in a terminal
> Emacs.

You can't: TTY frames cannot discern screen positions with resolution
of more than 1 character.

> @@ -290,12 +292,14 @@ xterm-mouse-event
>                (xterm-mouse--set-click-count event click-count)))
>             ((not last-time) nil)
>             ((and (> double-click-time (* 1000 (- this-time last-time)))
> +                 (eq x last-x)
> +                 (eq y last-y)

IMO, 'eq' is not right here: this test should obey the value of
double-click-fuzz, whose units on TTY frames are 1/8 of a character.

Message #20 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex <agrambot <at> gmail.com>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Thu, 05 Oct 2017 13:52:13 +0300

> From: Alex <agrambot <at> gmail.com>
> Date: Sat, 30 Sep 2017 21:56:58 -0600
> 
> I also found another issue, but it doesn't really warrant a new report.
> If you double click right after entering xterm-mouse-mode, then you'll
> see:
> 
> xterm-mouse-translate-1: Wrong type argument: number-or-marker-p, nil
> 
> This is because click-count isn't set to 1 when `xterm-mouse-last-click'
> is first set in `xterm-mouse-event'.
> 
> Here's a patch:

Thanks, pushed to the release branch.

> P.S. Why is the error prefixed by `xterm-mouse-translate-1' instead of
> `xterm-mouse-event'?

Can you show the full backtrace?

Message #23 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex <agrambot <at> gmail.com>
Cc: rudalics <at> gmx.at, 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Thu, 05 Oct 2017 13:52:46 +0300

> From: Alex <agrambot <at> gmail.com>
> Date: Sun, 01 Oct 2017 12:30:23 -0600
> Cc: 28658 <at> debbugs.gnu.org
> 
> martin rudalics <rudalics <at> gmx.at> writes:
> 
> > +           ((and last-time
> > +                 (> double-click-time (* 1000 (- this-time last-time)))
> >
> > Note that here and elsewhere coders assume that ‘double-click-time’ is a
> > number.  However, it may be also nil or t and we should fix those
> > instances eventually (see Bug#23419).  In new or changed code, at least.
> 
> Sure, here's a third patch to deal with that:

Pushed to emacs-26, thanks.

Message #26 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Alex <agrambot <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Thu, 05 Oct 2017 18:03:31 -0600

Eli Zaretskii <eliz <at> gnu.org> writes:

>> P.S. Why is the error prefixed by `xterm-mouse-translate-1' instead of
>> `xterm-mouse-event'?
>
> Can you show the full backtrace?

  Debugger entered--Lisp error: (wrong-type-argument number-or-marker-p nil)
    xterm-mouse-event(1006)
    xterm-mouse-translate-1(1006)
    xterm-mouse-translate-extended(nil)

So the backtrace acknowledges xterm-mouse-event is the culprit, but
without `debug-on-error' set the error message blames
xterm-mouse-translate-1.

If I don't use the bytecode versions of the procedures, though, then the
error message becomes:

  setq: Wrong type argument: number-or-marker-p, nil

which is better, but I believe it should message:

  1+: Wrong type argument: number-or-marker-p, nil

Which is what the debugger shows if I don't use the bytecode versions:

  Debugger entered--Lisp error: (wrong-type-argument number-or-marker-p nil)
    1+(nil)
    (setq click-count (1+ click-count))
    ...
    xterm-mouse-event(1006)
    (let* ((event (xterm-mouse-event extension)) (ev-command (nth 0 event)) (ev-data (nth 1 event)) (ev-where (nth 1 ev-$
    (save-excursion (let* ((event (xterm-mouse-event extension)) (ev-command (nth 0 event)) (ev-data (nth 1 event)) (ev-$
    xterm-mouse-translate-1(1006)
    xterm-mouse-translate-extended(nil)

Message #29 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Alex <agrambot <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Thu, 05 Oct 2017 18:14:39 -0600

Eli Zaretskii <eliz <at> gnu.org> writes:


>> It would be nice if it worked with pixel positions rather than
>> character positions, but I'm not sure how to do that in a terminal
>> Emacs.
>
> You can't: TTY frames cannot discern screen positions with resolution
> of more than 1 character.
>
>> @@ -290,12 +292,14 @@ xterm-mouse-event
>>                (xterm-mouse--set-click-count event click-count)))
>>             ((not last-time) nil)
>>             ((and (> double-click-time (* 1000 (- this-time last-time)))
>> +                 (eq x last-x)
>> +                 (eq y last-y)
>
> IMO, 'eq' is not right here: this test should obey the value of
> double-click-fuzz, whose units on TTY frames are 1/8 of a character.

I don't understand how to use double-click-fuzz in TTY frames. You said
that TTY frames can't discern screen position differences of less than a
character, so then why are the units 1/8th of a character?

Is there a way to get the mouse coordinate position in 1/8ths of a
character to use double-click-fuzz?

Message #32 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex <agrambot <at> gmail.com>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Fri, 06 Oct 2017 10:14:13 +0300

> From: Alex <agrambot <at> gmail.com>
> Cc: 28658 <at> debbugs.gnu.org
> Date: Thu, 05 Oct 2017 18:14:39 -0600
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> 
> >> @@ -290,12 +292,14 @@ xterm-mouse-event
> >>                (xterm-mouse--set-click-count event click-count)))
> >>             ((not last-time) nil)
> >>             ((and (> double-click-time (* 1000 (- this-time last-time)))
> >> +                 (eq x last-x)
> >> +                 (eq y last-y)
> >
> > IMO, 'eq' is not right here: this test should obey the value of
> > double-click-fuzz, whose units on TTY frames are 1/8 of a character.
> 
> I don't understand how to use double-click-fuzz in TTY frames. You said
> that TTY frames can't discern screen position differences of less than a
> character, so then why are the units 1/8th of a character?

I don't know why the units are 1/8th of a character, perhaps so that a
user could set the same value for both GUI and TTY frames.  In any
case, dividing the value of double-click-fuzz by 8 before comparing
with coordinate differences is easy enough, no?

> Is there a way to get the mouse coordinate position in 1/8ths of a
> character to use double-click-fuzz?

I don't think you can get sub-character resolution, but that doesn't
mean double-click-fuzz cannot be obeyed: a user could set the value to
16, in which case a 2-character distance should still be okay.  With
the default value of 3, the second click should indeed be on the same
character, so it will be equivalent to your eq test, but that's only
the default.

Message #35 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex <agrambot <at> gmail.com>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Fri, 06 Oct 2017 10:44:22 +0300

> From: Alex <agrambot <at> gmail.com>
> Cc: 28658 <at> debbugs.gnu.org
> Date: Thu, 05 Oct 2017 18:03:31 -0600
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> >> P.S. Why is the error prefixed by `xterm-mouse-translate-1' instead of
> >> `xterm-mouse-event'?
> >
> > Can you show the full backtrace?
> 
>   Debugger entered--Lisp error: (wrong-type-argument number-or-marker-p nil)
>     xterm-mouse-event(1006)
>     xterm-mouse-translate-1(1006)
>     xterm-mouse-translate-extended(nil)
> 
> So the backtrace acknowledges xterm-mouse-event is the culprit, but
> without `debug-on-error' set the error message blames
> xterm-mouse-translate-1.
> 
> If I don't use the bytecode versions of the procedures, though, then the
> error message becomes:
> 
>   setq: Wrong type argument: number-or-marker-p, nil

So this is something related to the byte compiler, I presume.  Did you
look at the byte code?

Message #38 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Alex <agrambot <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Sat, 07 Oct 2017 15:57:04 -0600

[Message part 1 (text/plain, inline)]

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Alex <agrambot <at> gmail.com>
>> Cc: 28658 <at> debbugs.gnu.org
>> Date: Thu, 05 Oct 2017 18:14:39 -0600
>> 
>> Eli Zaretskii <eliz <at> gnu.org> writes:
>> 
>> 
>> >> @@ -290,12 +292,14 @@ xterm-mouse-event
>> >>                (xterm-mouse--set-click-count event click-count)))
>> >>             ((not last-time) nil)
>> >>             ((and (> double-click-time (* 1000 (- this-time last-time)))
>> >> +                 (eq x last-x)
>> >> +                 (eq y last-y)
>> >
>> > IMO, 'eq' is not right here: this test should obey the value of
>> > double-click-fuzz, whose units on TTY frames are 1/8 of a character.
>> 
>> I don't understand how to use double-click-fuzz in TTY frames. You said
>> that TTY frames can't discern screen position differences of less than a
>> character, so then why are the units 1/8th of a character?
>
> I don't know why the units are 1/8th of a character, perhaps so that a
> user could set the same value for both GUI and TTY frames.  In any
> case, dividing the value of double-click-fuzz by 8 before comparing
> with coordinate differences is easy enough, no?

Yes, I was just confused about the units, but that makes sense. Though
in my GTK Emacs, (frame-char-width) returns 9 instead of 8 for me...

Anyway, here's the patch:

[0001-Increase-xterm-click-count-only-within-double-click-.patch (text/x-diff, attachment)]

Message #41 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex <agrambot <at> gmail.com>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Sun, 08 Oct 2017 10:42:20 +0300

> From: Alex <agrambot <at> gmail.com>
> Cc: 28658 <at> debbugs.gnu.org
> Date: Sat, 07 Oct 2017 15:57:04 -0600
> 
> > I don't know why the units are 1/8th of a character, perhaps so that a
> > user could set the same value for both GUI and TTY frames.  In any
> > case, dividing the value of double-click-fuzz by 8 before comparing
> > with coordinate differences is easy enough, no?
> 
> Yes, I was just confused about the units, but that makes sense. Though
> in my GTK Emacs, (frame-char-width) returns 9 instead of 8 for me...

Yes, the 8 thing cannot be anything but an approximation.

> Anyway, here's the patch:

LGTM, thanks.  Please mention the bug number in the log message when
you push (to the emacs-26 branch).

Message #46 received at 28658-done <at> debbugs.gnu.org (full text, mbox):

From: Alex <agrambot <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 28658-done <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Sun, 08 Oct 2017 17:44:52 -0600

Eli Zaretskii <eliz <at> gnu.org> writes:

> LGTM, thanks.  Please mention the bug number in the log message when
> you push (to the emacs-26 branch).

Pushed.

Message #49 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Alex <agrambot <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Sun, 08 Oct 2017 20:37:20 -0600

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Alex <agrambot <at> gmail.com>
>> Cc: 28658 <at> debbugs.gnu.org
>> Date: Thu, 05 Oct 2017 18:03:31 -0600
>> 
>>   Debugger entered--Lisp error: (wrong-type-argument number-or-marker-p nil)
>>     xterm-mouse-event(1006)
>>     xterm-mouse-translate-1(1006)
>>     xterm-mouse-translate-extended(nil)
>> 
>> So the backtrace acknowledges xterm-mouse-event is the culprit, but
>> without `debug-on-error' set the error message blames
>> xterm-mouse-translate-1.
>> 
>> If I don't use the bytecode versions of the procedures, though, then the
>> error message becomes:
>> 
>>   setq: Wrong type argument: number-or-marker-p, nil
>
> So this is something related to the byte compiler, I presume.

For the most part, yeah. Though without the byte compiler the error
message is prefixed by `setq' instead of `1+' as it should be (and is
when debugging).

> Did you look at the byte code?

No, I'm not sure what I should be looking for. I thought perhaps this
was a known limitation; should I report this as a bug?

Message #52 received at 28658 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex <agrambot <at> gmail.com>
Cc: 28658 <at> debbugs.gnu.org
Subject: Re: bug#28658: 27.0.50;
 [PATCH] double/triple clicking in xterm-mouse-mode doesn't respect
 mouse position
Date: Mon, 09 Oct 2017 10:11:18 +0300

> From: Alex <agrambot <at> gmail.com>
> Cc: 28658 <at> debbugs.gnu.org
> Date: Sun, 08 Oct 2017 20:37:20 -0600
> 
> No, I'm not sure what I should be looking for. I thought perhaps this
> was a known limitation; should I report this as a bug?

Maybe first mention this on emacs-devel, perhaps someone already knows
about this.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.