GNU bug report logs - #17453
Isearch doesn't work properly with Follow Mode.

Previous Next

Full log

View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alan Mackenzie <acm <at> muc.de>
Cc: 17453 <at> debbugs.gnu.org, monnier <at> iro.umontreal.ca, bruce.connor.am <at> gmail.com, juri <at> linkov.net
Subject: bug#17453: Isearch doesn't work properly with Follow Mode.
Date: Mon, 02 Nov 2015 19:49:43 +0200

> Date: Mon, 2 Nov 2015 16:09:17 +0000
> Cc: bruce.connor.am <at> gmail.com, juri <at> linkov.net, 17453 <at> debbugs.gnu.org,
>   monnier <at> iro.umontreal.ca
> From: Alan Mackenzie <acm <at> 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.

> 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?

Previous Next

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