[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#17453: Isearch doesn't work properly with Follow Mode.
From: |
Alan Mackenzie |
Subject: |
bug#17453: Isearch doesn't work properly with Follow Mode. |
Date: |
Tue, 3 Nov 2015 08:35:02 +0000 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, Nov 02, 2015 at 07:49:43PM +0200, Eli Zaretskii wrote:
> > Date: Mon, 2 Nov 2015 16:09:17 +0000
> > Cc: bruce.connor.am@gmail.com, juri@linkov.net, 17453@debbugs.gnu.org,
> > monnier@iro.umontreal.ca
> > From: Alan Mackenzie <acm@muc.de>
> > > If this means that sit*-for does something other than redisplay and
> > > wait, like switch to another window, I'd really suggest to rethink
> > > that. It is entirely counter-intuitive to have a sit-for family of
> > > functions do anything other than some kind of redisplay and some kind
> > > of waiting.
> > sit*-for's synchronising windows, switching to the appropriate window,
> > etc., is conceptually an extension of redisplay's scrolling to get point
> > on screen.
> No, redisplay doesn't necessarily scroll. It does so only if needed.
> So sit-for should not be thought as a way to scroll the window, even
> if sometimes it does.
Redisplay scrolls sometimes, and it moves point sometimes. But it does
those things only when needed for the actual display. Similarly the
sit*-for, invoking display would do other things strictly necessary for
successful redisplay. That sometimes would include resynchronising the
windows, for example.
> > But I think sit*-for could well be not needed here, anyway.
> That's okay, but my point is: please do not add to established
> functions code that does on behalf of Follow mode something that
> conceptually doesn't belong to those functions. OK?
Fine!
--
Alan Mackenzie (Nuremberg, Germany).
- bug#17453: Isearch doesn't work properly with Follow Mode., (continued)
- bug#17453: Isearch doesn't work properly with Follow Mode., Alan Mackenzie, 2015/11/03
- bug#17453: Isearch doesn't work properly with Follow Mode., Juri Linkov, 2015/11/02
- bug#17453: Isearch doesn't work properly with Follow Mode., Eli Zaretskii, 2015/11/02
- bug#17453: Isearch doesn't work properly with Follow Mode., Drew Adams, 2015/11/02
- bug#17453: Isearch doesn't work properly with Follow Mode., Alan Mackenzie, 2015/11/02
- bug#17453: Isearch doesn't work properly with Follow Mode., Eli Zaretskii, 2015/11/02
- bug#17453: Isearch doesn't work properly with Follow Mode., John Wiegley, 2015/11/02
- bug#17453: Isearch doesn't work properly with Follow Mode.,
Alan Mackenzie <=
- bug#17453: Isearch doesn't work properly with Follow Mode., Juri Linkov, 2015/11/02