From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: =?UTF-8?Q?J=C3=A9r=C3=A9my?= Compostella Newsgroups: gmane.emacs.bugs Subject: bug#10614: [EXPERIMENTAL PATCH] Extending Isearch-repeat-forward/backward to support a prefix argument following suggesion by Juri Linkov Date: Thu, 26 Jan 2012 20:28:02 +0100 Message-ID: References: <87lioufh0t.fsf@thinkpad.tsdh.de> <871uqmffee.fsf@thinkpad.tsdh.de> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: multipart/alternative; boundary=f46d044471a32d2e2004b77362c9 X-Trace: dough.gmane.org 1327606206 12106 80.91.229.12 (26 Jan 2012 19:30:06 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Thu, 26 Jan 2012 19:30:06 +0000 (UTC) Cc: Gideon Stupp , emacs-devel@gnu.org, 10614@debbugs.gnu.org To: Tassilo Horn Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Thu Jan 26 20:30:01 2012 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from [140.186.70.17] (helo=lists.gnu.org) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1RqV1E-0000AI-Sc for geb-bug-gnu-emacs@m.gmane.org; Thu, 26 Jan 2012 20:30:01 +0100 Original-Received: from localhost ([::1]:36103 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RqV11-0002e2-Ug for geb-bug-gnu-emacs@m.gmane.org; Thu, 26 Jan 2012 14:29:47 -0500 Original-Received: from [140.186.70.92] (port=57854 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RqV0v-0002cQ-Gg for bug-gnu-emacs@gnu.org; Thu, 26 Jan 2012 14:29:45 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RqV0p-00049y-Ux for bug-gnu-emacs@gnu.org; Thu, 26 Jan 2012 14:29:37 -0500 Original-Received: from [140.186.70.43] (port=40472 helo=debbugs.gnu.org) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RqV0p-00049e-Op for bug-gnu-emacs@gnu.org; Thu, 26 Jan 2012 14:29:35 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.72) (envelope-from ) id 1RqV1G-0005XH-II for bug-gnu-emacs@gnu.org; Thu, 26 Jan 2012 14:30:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: =?UTF-8?Q?J=C3=A9r=C3=A9my?= Compostella Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Thu, 26 Jan 2012 19:30:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 10614 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 10614-submit@debbugs.gnu.org id=B10614.132760614621187 (code B ref 10614); Thu, 26 Jan 2012 19:30:02 +0000 Original-Received: (at 10614) by debbugs.gnu.org; 26 Jan 2012 19:29:06 +0000 Original-Received: from localhost ([127.0.0.1]:45859 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1RqV0M-0005Vf-4h for submit@debbugs.gnu.org; Thu, 26 Jan 2012 14:29:06 -0500 Original-Received: from mail-tul01m020-f172.google.com ([209.85.214.172]:36757) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1RqV0I-0005V9-AX for 10614@debbugs.gnu.org; Thu, 26 Jan 2012 14:29:03 -0500 Original-Received: by obbwc12 with SMTP id wc12so906034obb.3 for <10614@debbugs.gnu.org>; Thu, 26 Jan 2012 11:28:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=t/qXnR9UhlM6SWOG1s97U6iZdRGQ051hKTvULli7okk=; b=xlM01bqIAas6ZjbNL+RVsAmbn+HF+mbVdBfAMibGnaockMisrxTMi6aYnGilR4BynH ZFibBn9XdRz9Q6aaAiOuXirw1r+hQGsifwZv7IZK+eWikw8juTQ73+KMfJlVbo0Xxucj ECpNc8HTSgN1c1/xwBbDEAFe5jxxsnwavZBNY= Original-Received: by 10.182.74.102 with SMTP id s6mr3565868obv.46.1327606103092; Thu, 26 Jan 2012 11:28:23 -0800 (PST) Original-Received: by 10.60.50.37 with HTTP; Thu, 26 Jan 2012 11:28:02 -0800 (PST) In-Reply-To: <871uqmffee.fsf@thinkpad.tsdh.de> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.13 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) X-Received-From: 140.186.70.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:56055 Archived-At: --f46d044471a32d2e2004b77362c9 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 2012/1/26 Tassilo Horn > Tassilo Horn writes: > > > But how the heck is the usage of the prefix arg supposed to work? > > When I do C-s foo C-3, isearch is exited... > > That was my fault. Of course, loading the patched isearch.el didn't > make the changed isearch-mode-map active (defvar). It works fine, now. > > Usability-wise, C-s foo C- C-s makes only sense if that > keystroke is shorter to type than C-s multiple times. So maybe the > numbering should start with some offset, say, with the 5th next match? > > And the typing of C-s after the number is a bit inconvenient, so maybe, > C-7 could jump directly to the match hinted with 7, and only for > two-digit matches C-s was needed, i.e., C-u 1 2 C-s to jump to the 12th > match? > > I just played play with it too and I really like it. Here my notes: - I do agree with Tassilo, C-7 should directly jump to match hinted 7. - When more than lazy-highlight-max-at-a-time are visible in the window, the prefix number is restarted from 1 on the next match hinted. It's confusing and I think it should not appear on these items. - A defcustom variable should be added to enable/disable the match hinted prefix number. Anyway, it's an very interesting improvement. J=E9r=E9my --f46d044471a32d2e2004b77362c9 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
2012/1/26 Tassilo Horn <tassilo@member.fsf.org>
Tassilo Horn <tassilo@member.fsf.org> writes:

> But how the heck is the usage of the prefix arg supposed to work?
> When I do C-s foo C-3, isearch is exited...

That was my fault. =A0Of course, loading the patched isearch.el didn&= #39;t
make the changed isearch-mode-map active (defvar). =A0It works fine, now.
Usability-wise, C-s foo C-<number> C-s makes only sense if that
keystroke is shorter to type than C-s multiple times. =A0So maybe the
numbering should start with some offset, say, with the 5th next match?

And the typing of C-s after the number is a bit inconvenient, so maybe,
C-7 could jump directly to the match hinted with 7, and only for
two-digit matches C-s was needed, i.e., C-u 1 2 C-s to jump to the 12th
match?


I just played play with it too and I r= eally like it. Here my notes:
- I do agree with Tassilo, C-7 shou= ld directly jump to match hinted 7.
- When more than=A0lazy-highl= ight-max-at-a-time are visible in the window,
=A0=A0the prefix number is restarted from 1 on the next match hinted. = It's confusing
=A0=A0and I think it should not appear on thes= e items.
- A defcustom variable should be added to enable/disable= the match hinted prefix
=A0=A0number.

Anyway, it's an very=A0inte= resting=A0improvement.

J=E9r=E9my

--f46d044471a32d2e2004b77362c9--