GNU bug report logs - #36269
26.2; mouse-drag-and-drop-region conflicts with mouse-avoidance

Previous Next

Full log

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

From: martin rudalics <rudalics <at> gmx.at>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: rswgnu <at> gmail.com, scotto <at> sharpleaf.org, 28620 <at> debbugs.gnu.org,
 36269 <at> debbugs.gnu.org
Subject: Re: bug#28620: Mouse drag event records wrong window for release when
 crossing frames
Date: Sun, 28 Jul 2019 09:34:00 +0200

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

> Thanks, a few minor comments below.

Thanks for the comments.  I hopefully applied all changes you
proposed.

Please keep in mind that I wrote the code almost two years ago and
then forgot about it.  It's only in the context of Bug#36269 that I
resurrected it - with all its inadequacies.

>> @@ -3995,7 +3992,7 @@ kbd_buffer_get_event (KBOARD **kbp,
>>         }
>>       }
>>     /* Try generating a mouse motion event.  */
>> -  else if (!NILP (do_mouse_tracking) && some_mouse_moved ())
>> +  else if (some_mouse_moved ())
>
> Can't we have mouse motion events outside track-mouse?

Not to my knowledge.  In either case, I wouldn't want to change
anything in this regard here.

> There's too much of whitespace changes in the rest of the patch,
> making it very hard to review.  Can you show the patch without
> whitespace differences?

I attach a patch which re-adds some extraneous braces to w32term.c.
If you want me to restore more of the old code, please tell me.

Also, I do not intend to push the changes until someone tells me that
they work.  I neither use the mouse to drop nor mouse avoidance mode
and so have never given it any serious testing.

Thanks, martin

[track-mouse-with-braces.diff (text/plain, attachment)]

Previous Next

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