GNU bug report logs - #45628
28.0.50; Scrolling on echo area with pixel-scroll-mode enabled locks up Emacs

Previous Next

Full log

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Tak Kunihiro <tkk <at> misasa.okayama-u.ac.jp>
Cc: 45628-done <at> debbugs.gnu.org, bandali <at> gnu.org
Subject: Re: bug#45628: 28.0.50; Scrolling on echo area with pixel-scroll-mode
 enabled locks up Emacs
Date: Sat, 09 Jan 2021 11:22:35 +0200

> From: Tak Kunihiro <tkk <at> misasa.okayama-u.ac.jp>
> Date: Wed, 6 Jan 2021 09:47:26 +0900
> Cc: 国広卓也 <tkk <at> misasa.okayama-u.ac.jp>,
>  Amin Bandali <bandali <at> gnu.org>,
>  45628 <at> debbugs.gnu.org
> 
> > How about a normal while loop with catch before it and throw inside?
> > 
> > +          (while (and (pixel-point-at-top-p amt) ; prevent too late (multi tries)
> > +                      (equal (vertical-motion 1) 1))) ; move point downward
> > 
> > +        (while (and (pixel-point-at-bottom-p amt) ; prevent too late (multi tries)
> > +                    (equal (vertical-motion -1) -1))) ; move point upward
> > 
> > I think equality to 1 or -1 is too stringent.  vertical-motion could
> > move more if you have overlay strings or display strings at point that
> > include embedded newlines.  So I think you should test >= 1 and <= -1
> > respectively.
> 
> I revised the patch and is attached on this message.
> 
> 
> > Also, I believe this problem exists on the emacs-27 branch as well,
> > right?  Then we should install there first.
> 
> Yes.  Can you install the patch with the commit message to emacs-27 branch?

Thanks, I installed this on the emacs-27 branch, and I'm closing this
bug report.

Previous Next

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