unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Juri Linkov <juri@jurta.org>
Cc: rms@gnu.org, emacs-devel@gnu.org
Subject: Re: isearch-query-replace-regexp and stuff
Date: Tue, 06 Jul 2004 19:36:38 +0300	[thread overview]
Message-ID: <87zn6dqei0.fsf@mail.jurta.org> (raw)
In-Reply-To: <x5k6xhpekl.fsf@lola.goethe.zz> (David Kastrup's message of "06 Jul 2004 13:20:26 +0200")

David Kastrup <dak@gnu.org> writes:
> Juri Linkov <juri@jurta.org> writes:
>> C-M-% is a standard key binding to start regexp replacement,
>> so it should be available in isearch too.
>
> Well, C-s is a standard key binding to start literal searches, so the
> same argument would apply for its meaning of continuing a regexp
> search.

This analogy is appropriate (even C-M-s can continue a non-regexp
search), but at the same time I think no one would have an incentive
to type C-M-% in a non-regexp search to start a non-regexp replacement.
C-M-% is quite hard to type, and if someone still decided to type it,
this means only that it was typed with the intention to start a regexp
replacement.

> Well, if I do
> C-s e[e] M-r
> on this mail, then a "failed isearch" turns into a "failed regexp
> isearch" until I press C-s again. While this has the advantage that
> M-r M-r is a noop, it seems counterintuitive.  Maybe switching with
> M-r should do something like the following?
>
> If looking-at (or its non-regexp equivalent) returns non-nil after the
> change, remove a potentially previous "failed" tag.  If looking-at
> returns nil, call the search "pending" instead of "failed" until C-s
> is pressed again.

Maybe it's better to always clear the failed flag after switching
with M-r, because repeating the failed search after switching the
search type has such undesirable behavior as wrapping the search and
starting from the beginning of the buffer.  After switching the search
type it should continue from the same point even if the previous search
type failed.

-- 
Juri Linkov
http://www.jurta.org/emacs/

  reply	other threads:[~2004-07-06 16:36 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-01 23:18 isearch-query-replace-regexp and stuff David Kastrup
2004-07-02  0:05 ` David Kastrup
2004-07-02 17:51   ` Richard Stallman
2004-07-02  6:55 ` Juri Linkov
2004-07-02  7:55   ` David Kastrup
2004-07-02 13:44     ` Stefan
2004-07-03  6:59     ` Juri Linkov
2004-07-03 18:20 ` Richard Stallman
2004-07-05 19:07   ` David Kastrup
2004-07-06  9:59     ` Juri Linkov
2004-07-06 11:20       ` David Kastrup
2004-07-06 16:36         ` Juri Linkov [this message]
2004-07-06 17:29           ` David Kastrup
2004-07-07  5:08             ` Juri Linkov
2004-07-07  9:29               ` David Kastrup
2004-07-07 18:29                 ` Juri Linkov
2004-07-07 19:33                   ` David Kastrup
2004-07-08 16:45                     ` Juri Linkov
2004-07-08 18:12                       ` David Kastrup
2004-07-09 20:54                         ` Juri Linkov
2004-07-08 23:17                 ` Richard Stallman
2004-07-06 22:00         ` Richard Stallman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87zn6dqei0.fsf@mail.jurta.org \
    --to=juri@jurta.org \
    --cc=emacs-devel@gnu.org \
    --cc=rms@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).