From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Juri Linkov Newsgroups: gmane.emacs.devel Subject: Re: isearch-allow-prefix [Was: [PATCH] Make `C-x {' and `C-x }' repeatable] Date: Tue, 04 Jun 2013 21:03:07 +0300 Organization: JURTA Message-ID: <87sj0xn484.fsf@mail.jurta.org> References: <87txlts7so.fsf@mail.jurta.org> <20130524093858.GA2754@acm.acm> <87hahstd47.fsf@mail.jurta.org> <20130525200103.GA3451@acm.acm> <878v32aj3c.fsf@mail.jurta.org> <20130602210512.GC2765@acm.acm> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1370369271 16344 80.91.229.3 (4 Jun 2013 18:07:51 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 4 Jun 2013 18:07:51 +0000 (UTC) Cc: Stefan Monnier , Drew Adams , emacs-devel@gnu.org To: Alan Mackenzie Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Jun 04 20:07:52 2013 Return-path: Envelope-to: ged-emacs-devel@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 1Ujve7-0005x0-DZ for ged-emacs-devel@m.gmane.org; Tue, 04 Jun 2013 20:07:47 +0200 Original-Received: from localhost ([::1]:47509 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ujve6-0004SX-Qd for ged-emacs-devel@m.gmane.org; Tue, 04 Jun 2013 14:07:46 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:42473) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ujve0-0004SQ-3r for emacs-devel@gnu.org; Tue, 04 Jun 2013 14:07:44 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ujvdp-0000Xr-Gt for emacs-devel@gnu.org; Tue, 04 Jun 2013 14:07:40 -0400 Original-Received: from ps18281.dreamhost.com ([69.163.218.105]:39561 helo=ps18281.dreamhostps.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ujvdp-0000Uo-CA for emacs-devel@gnu.org; Tue, 04 Jun 2013 14:07:29 -0400 Original-Received: from localhost (ps18281.dreamhostps.com [69.163.218.105]) by ps18281.dreamhostps.com (Postfix) with ESMTP id 52409258B9E91C; Tue, 4 Jun 2013 11:07:26 -0700 (PDT) In-Reply-To: <20130602210512.GC2765@acm.acm> (Alan Mackenzie's message of "Sun, 2 Jun 2013 21:05:12 +0000") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (x86_64-pc-linux-gnu) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x (no timestamps) [generic] X-Received-From: 69.163.218.105 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:160082 Archived-At: > OK. Here is a patch for isearch.el, search.texi and NEWS. Thanks, I tested your patch and it correctly passes a prefix argument to the next command. > I'm not terribly impressed by the documentation patch, > but it was the best I could manage. Could you try adding new text to the existing node (info "(emacs) Isearch Scroll") and renaming it to something more general that would fit both variables? I know that it's not trivial to find such a name, but it would be better to have related variables in the same node. What about the node name "Exiting Isearch" to document in one node all options that affect commands exiting Isearch? > Do you have any comments on the patch, or should I just commit it? I have no more comments on your patch, and just want to note that after committing we will be able to finally implement http://debbugs.gnu.org/10614 to support a prefix argument in `isearch-repeat-forward' and `isearch-repeat-backward' and other commands.