From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Juri Linkov Newsgroups: gmane.emacs.bugs Subject: bug#17453: Isearch doesn't work properly with Follow Mode. Date: Mon, 02 Nov 2015 02:14:47 +0200 Organization: LINKOV.NET Message-ID: <87bnbddzpk.fsf__42448.5682546676$1446423443$gmane$org@mail.linkov.net> References: <20140509224458.GA4205@acm.acm> <20151029232302.GB3812@acm.fritz.box> <87h9l6627a.fsf@mail.linkov.net> <20151031235651.GE1853@acm.fritz.box> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1446423443 16271 80.91.229.3 (2 Nov 2015 00:17:23 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 2 Nov 2015 00:17:23 +0000 (UTC) Cc: 17453@debbugs.gnu.org, Stefan Monnier , emacs-devel@gnu.org To: Alan Mackenzie Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Mon Nov 02 01:17:12 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 1Zt2oA-0003By-P2 for geb-bug-gnu-emacs@m.gmane.org; Mon, 02 Nov 2015 01:17:11 +0100 Original-Received: from localhost ([::1]:39415 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zt2oA-0000Nz-47 for geb-bug-gnu-emacs@m.gmane.org; Sun, 01 Nov 2015 19:17:10 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:52210) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zt2o7-0000Nt-2c for bug-gnu-emacs@gnu.org; Sun, 01 Nov 2015 19:17:07 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zt2o3-0002ga-0u for bug-gnu-emacs@gnu.org; Sun, 01 Nov 2015 19:17:06 -0500 Original-Received: from debbugs.gnu.org ([208.118.235.43]:57514) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zt2o2-0002gW-Th for bug-gnu-emacs@gnu.org; Sun, 01 Nov 2015 19:17:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1Zt2o2-0002Rm-Ea for bug-gnu-emacs@gnu.org; Sun, 01 Nov 2015 19:17:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Juri Linkov Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 02 Nov 2015 00:17:02 +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.14464233659332 (code B ref 17453); Mon, 02 Nov 2015 00:17:02 +0000 Original-Received: (at 17453) by debbugs.gnu.org; 2 Nov 2015 00:16:05 +0000 Original-Received: from localhost ([127.0.0.1]:48222 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1Zt2n7-0002QR-0g for submit@debbugs.gnu.org; Sun, 01 Nov 2015 19:16:05 -0500 Original-Received: from sub3.mail.dreamhost.com ([69.163.253.7]:46317 helo=homiemail-a100.g.dreamhost.com) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1Zt2mn-0002Pf-6V for 17453@debbugs.gnu.org; Sun, 01 Nov 2015 19:16:04 -0500 Original-Received: from homiemail-a100.g.dreamhost.com (localhost [127.0.0.1]) by homiemail-a100.g.dreamhost.com (Postfix) with ESMTP id 4596231A073; Sun, 1 Nov 2015 16:15:44 -0800 (PST) Original-Received: from localhost.linkov.net (82.131.11.41.cable.starman.ee [82.131.11.41]) (Authenticated sender: jurta@jurta.org) by homiemail-a100.g.dreamhost.com (Postfix) with ESMTPA id 0160131A070; Sun, 1 Nov 2015 16:15:42 -0800 (PST) In-Reply-To: <20151031235651.GE1853@acm.fritz.box> (Alan Mackenzie's message of "Sat, 31 Oct 2015 23:56:51 +0000") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (x86_64-pc-linux-gnu) 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:108285 Archived-At: >> >>From another perspective, settings lazy-highlight-buffer to t >> (implemented in bug#21092) and removing the current restriction of >> (overlay-put ov 'window (selected-window)) will lazy-highlight matches >> in all follow windows with no effort. > > I wasn't actually aware of that fix. > > There were three main problems my patch fixed: > 1) Searching commands were restricted to a single follow window. This > was caused by the lazy highlighting mechanism, as you say. > 2) Lazy highlighting was only being done in a single window. > 3) In scrolling commands, point was restricted to the singled window, > rather than being able to move freely throughout all the windows. IIUC, lazy highlighting is not a problem anymore, so the remaining problem is how to switch windows when the next search position happens to be in an adjacent window. Since there is no other core library that require so much changes to adapt to follow-mode, and there are already some mode-specific hooks in follow.el like compilation-filter-hook, indicates that it's the responsibility of follow.el to support isearch in follow-mode. So I believe the right way to do this is like Artur presented in a short patch, and maybe it's possible to simplify it even more by using isearch-update-post-hook with a follow-align-compilation-windows like hook. I mean something like (add-hook 'isearch-update-post-hook follow-align-isearch-windows t t)