all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: Juri Linkov <juri@linkov.net>
Cc: 20687@debbugs.gnu.org
Subject: bug#20687: 25.0.50; `perform-replace' should invoke a key that you have bound in `query-replace-map'
Date: Mon, 1 Jun 2015 14:11:42 -0700 (PDT)	[thread overview]
Message-ID: <890ac68c-1c71-49c7-9e60-45afba90d74a@default> (raw)
In-Reply-To: <87pp5fb172.fsf@mail.linkov.net>

> Could you please send an example of your custom keybindings in
> `query-replace-map' that currently don't work.

I don't have any custom keybindings in `query-replace-map' that
don't work (in fact, I don't have any custom bindings in that
map at all).

This bug report came from this emacs.StackExchange answer - see
the discussion in the comments.
http://emacs.stackexchange.com/a/12781/105.

The aim here was to add `C' to `query-replace-map', to have it
toggle `case-fold-search'.  But it doesn't matter what key a
user might want to bind to what command during q-r.

The point is that a user can do that (that's what keymaps and
key bindings are for), but currently `perform-replace' refuses to
recognize such a key and its command.

There is no good reason for this, AFAICT.  It should be OK for
a user to do this.  Of course, that doesn't update the doc
string to reflect the new key and its action, but that's all.
At the user level, this should be something that users can do
easily, without needing to perform surgery.





  reply	other threads:[~2015-06-01 21:11 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-28 21:12 bug#20687: 25.0.50; `perform-replace' should invoke a key that you have bound in `query-replace-map' Drew Adams
2015-06-01 20:53 ` Juri Linkov
2015-06-01 21:11   ` Drew Adams [this message]
2015-06-02 22:01     ` Juri Linkov
2015-06-02 22:12       ` Drew Adams
2020-09-17 18:11       ` Lars Ingebrigtsen
2015-06-02 13:08 ` Kaushal
2015-06-02 22:02   ` Juri Linkov
2015-06-02 22:50     ` Drew Adams
2015-06-03  3:35       ` Kaushal
2015-06-03  4:39         ` Drew Adams
2015-06-03  5:10           ` Kaushal

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

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

  git send-email \
    --in-reply-to=890ac68c-1c71-49c7-9e60-45afba90d74a@default \
    --to=drew.adams@oracle.com \
    --cc=20687@debbugs.gnu.org \
    --cc=juri@linkov.net \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.