From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Alan Mackenzie Newsgroups: gmane.emacs.bugs Subject: bug#17453: Isearch doesn't work properly with Follow Mode. Date: Sat, 31 Oct 2015 23:32:25 +0000 Message-ID: <20151031233225.GD1853__41818.6620147482$1446334284$gmane$org@acm.fritz.box> References: <20140509224458.GA4205@acm.acm> <20151029232302.GB3812@acm.fritz.box> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Trace: ger.gmane.org 1446334284 4939 80.91.229.3 (31 Oct 2015 23:31:24 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sat, 31 Oct 2015 23:31:24 +0000 (UTC) Cc: 17453@debbugs.gnu.org, Stefan Monnier , emacs-devel To: Artur Malabarba Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Sun Nov 01 00:31:13 2015 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Zsfc8-0005DW-8g for geb-bug-gnu-emacs@m.gmane.org; Sun, 01 Nov 2015 00:31:12 +0100 Original-Received: from localhost ([::1]:57346 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zsfc7-0004H1-F2 for geb-bug-gnu-emacs@m.gmane.org; Sat, 31 Oct 2015 19:31:11 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:56560) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zsfc1-0004AM-80 for bug-gnu-emacs@gnu.org; Sat, 31 Oct 2015 19:31:05 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zsfby-0003eO-4w for bug-gnu-emacs@gnu.org; Sat, 31 Oct 2015 19:31:05 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:56670) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zsfby-0003eA-28 for bug-gnu-emacs@gnu.org; Sat, 31 Oct 2015 19:31:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1Zsfbx-0002av-Kx for bug-gnu-emacs@gnu.org; Sat, 31 Oct 2015 19:31:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Alan Mackenzie Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sat, 31 Oct 2015 23:31:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 17453 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 17453-submit@debbugs.gnu.org id=B17453.14463342439949 (code B ref 17453); Sat, 31 Oct 2015 23:31:01 +0000 Original-Received: (at 17453) by debbugs.gnu.org; 31 Oct 2015 23:30:43 +0000 Original-Received: from localhost ([127.0.0.1]:47378 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1Zsfbe-0002aP-OC for submit@debbugs.gnu.org; Sat, 31 Oct 2015 19:30:43 -0400 Original-Received: from mail.muc.de ([193.149.48.3]:46708) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1Zsfbc-0002aG-5J for 17453@debbugs.gnu.org; Sat, 31 Oct 2015 19:30:40 -0400 Original-Received: (qmail 93842 invoked by uid 3782); 31 Oct 2015 23:30:39 -0000 Original-Received: from acm.muc.de (p579E9C22.dip0.t-ipconnect.de [87.158.156.34]) by colin.muc.de (tmda-ofmipd) with ESMTP; Sun, 01 Nov 2015 00:30:37 +0100 Original-Received: (qmail 26432 invoked by uid 1000); 31 Oct 2015 23:32:25 -0000 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) X-Delivery-Agent: TMDA/1.1.12 (Macallan) X-Primary-Address: acm@muc.de X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:108251 Archived-At: Hello, Artur On Sat, Oct 31, 2015 at 11:13:34PM +0000, Artur Malabarba wrote: > On 29 Oct 2015 11:23 pm, "Alan Mackenzie" wrote: > > Hello, Stefan, hello, Emacs. > > Resuming a conversation from a year and a half ago, in it I had proposed > > a patch which made isearch work properly in a Follow Mode set of > > windows. In the end, you rejected my patch (in effect), because... > > On Fri, May 09, 2014 at 10:40:07PM -0400, Stefan Monnier wrote: > > > I must say I really dislike this hard-coding of follow-mode support in > > > isearch.el. Isearch should not know so much about follow-mode and > > > follow-mode should not know so much about Isearch either. > > > IOW we should try harder to come up with more general hooks. > What if isearch just took into account all windows displaying > current-buffer, instead of just the selected one? > This wouldn’t involve anything specific to follow mode, and I believe it > would solve the issue, no? I don't think so, really. What exactly does "took into account" mean? With Follow Mode active, on a forward search you explicitly want point to move into the next window when the search target is visible there. When FM is not active, you most definitely don't want this to happen. I have working code which make isearch and FM work together nicely. I think it's a sufficiently "nice" implementation to commit. -- Alan Mackenzie (Nuremberg, Germany).