[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: propose adding Icicles to Emacs
From: |
Stefan Monnier |
Subject: |
Re: propose adding Icicles to Emacs |
Date: |
Mon, 11 Jun 2007 15:17:12 -0400 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/22.1.50 (gnu/linux) |
> The `completing-read' code is here:
> http://www.emacswiki.org/cgi-bin/wiki/icicles-fn.el. Perhaps others more
> familiar with Emacs implementation can advise whether some of this
> redefinition could or should be moved to the vanilla definition of
> `completing-read' or what might be a "cleaner way" to do what is done now
> this way. I would advise just leaving this code as is, at least for now.
Redefining completing-read is a no-no in my book. Please try to rewrite
your code so as not to do that (e.g. using minibuffer-setup-hook). Also, if
it turns out to be impossible, or very inconvenient, then post a request for
enhancement to make it possible/easier.
>> it can replace some top-level commands with Icicles
>> versions (e.g. `find-file' with `icicle-find-file').
> I should be clear that this is not about redefining commands; it is about
> remapping their bindings: the bindings for `find-file' are mapped to
> `icicle-find-file'.
I.e. find-file is not changed. You just remap C-x C-f to your
command, right? That's perfectly fine.
Stefan
- Re: address@hidden: RE: cannot find :enable inElispmanualindex], (continued)
- Re: address@hidden: RE: cannot find :enable inElispmanualindex], Richard Stallman, 2007/06/08
- Re: address@hidden: RE: cannot find :enable inElispmanualindex], Juri Linkov, 2007/06/08
- Re: address@hidden: RE: cannot find :enable inElispmanualindex], Stefan Monnier, 2007/06/08
- propose adding Icicles to Emacs, Drew Adams, 2007/06/08
- Re: propose adding Icicles to Emacs, Richard Stallman, 2007/06/10
- RE: propose adding Icicles to Emacs, Drew Adams, 2007/06/10
- Re: propose adding Icicles to Emacs, Sean O'Rourke, 2007/06/10
- RE: propose adding Icicles to Emacs, Drew Adams, 2007/06/10
- Re: propose adding Icicles to Emacs, Richard Stallman, 2007/06/11
- RE: propose adding Icicles to Emacs, Drew Adams, 2007/06/11
- Re: propose adding Icicles to Emacs,
Stefan Monnier <=
- RE: propose adding Icicles to Emacs, Drew Adams, 2007/06/11
- Re: propose adding Icicles to Emacs, Stefan Monnier, 2007/06/11
- RE: propose adding Icicles to Emacs, Drew Adams, 2007/06/11
- Re: propose adding Icicles to Emacs, Richard Stallman, 2007/06/13
- Re: propose adding Icicles to Emacs, Richard Stallman, 2007/06/14
- RE: propose adding Icicles to Emacs, Drew Adams, 2007/06/14
- Re: propose adding Icicles to Emacs, Stefan Monnier, 2007/06/14
- RE: propose adding Icicles to Emacs, Drew Adams, 2007/06/14
- Re: propose adding Icicles to Emacs, Richard Stallman, 2007/06/15
- RE: propose adding Icicles to Emacs, Drew Adams, 2007/06/15