emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: isearch-allow-prefix [Was: [PATCH] Make `C-x {' and `C-x }' repeatab


From: Juri Linkov
Subject: Re: isearch-allow-prefix [Was: [PATCH] Make `C-x {' and `C-x }' repeatable]
Date: Tue, 04 Jun 2013 21:03:07 +0300
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (x86_64-pc-linux-gnu)

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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]