all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: isearch-resume-enabled
       [not found] <E1C5pO0-0007bv-QA@fencepost.gnu.org>
@ 2004-09-11 19:21 ` Kim F. Storm
  2004-09-11 19:32   ` isearch-resume-enabled Miles Bader
  2004-09-13  7:01   ` isearch-resume-enabled Richard Stallman
  0 siblings, 2 replies; 5+ messages in thread
From: Kim F. Storm @ 2004-09-11 19:21 UTC (permalink / raw)
  Cc: emacs-devel

Richard Stallman <rms@gnu.org> writes:

> I see you added the variable, described this way in etc/NEWS:
>
>       +++
>     + ** New user option `isearch-resume-enabled'.
>     + This option can be disabled, to avoid the normal behavior of isearch
>     + which puts calls to `isearch-resume' in the command history.
>
> This doesn't clearly explain the user-level behavior
> either in the default case or in the other case.
> The doc string doesn't, either.
>
> Could you explain to me what they are?

With the default setting, when you do C-s SOMETHING RET, a command
     (isearch-resume "SOMETHING" nil nil t "SOMETHING" (quote t))
is added to the command history.

IMO, this is very annoying.

C-s C-s already resumes the last isearch, and
C-s M-p M-p resumes earlier isearches.

So there's no reason to clutter C-x ESC ESC with this.

Setting isearch-resume-enabled to nil avoid cluttering the command
history with those (superfluous) isearch-resume commands.

I would recommend using isearch-resume-enabled = nil the default.

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: isearch-resume-enabled
  2004-09-11 19:21 ` isearch-resume-enabled Kim F. Storm
@ 2004-09-11 19:32   ` Miles Bader
  2004-09-11 20:54     ` isearch-resume-enabled Kim F. Storm
  2004-09-13  7:01   ` isearch-resume-enabled Richard Stallman
  1 sibling, 1 reply; 5+ messages in thread
From: Miles Bader @ 2004-09-11 19:32 UTC (permalink / raw)
  Cc: rms, emacs-devel

On Sat, Sep 11, 2004 at 09:21:51PM +0200, Kim F. Storm wrote:
> Setting isearch-resume-enabled to nil avoid cluttering the command
> history with those (superfluous) isearch-resume commands.
> 
> I would recommend using isearch-resume-enabled = nil the default.

The name is bizarre though; how about something more obvious like
`isearch-resume-in-command-history' or whatever?

-Miles
-- 
P.S.  All information contained in the above letter is false,
      for reasons of military security.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: isearch-resume-enabled
  2004-09-11 19:32   ` isearch-resume-enabled Miles Bader
@ 2004-09-11 20:54     ` Kim F. Storm
  0 siblings, 0 replies; 5+ messages in thread
From: Kim F. Storm @ 2004-09-11 20:54 UTC (permalink / raw)
  Cc: emacs-devel, rms, Kim F. Storm

Miles Bader <miles@gnu.org> writes:

> On Sat, Sep 11, 2004 at 09:21:51PM +0200, Kim F. Storm wrote:
>> Setting isearch-resume-enabled to nil avoid cluttering the command
>> history with those (superfluous) isearch-resume commands.
>> 
>> I would recommend using isearch-resume-enabled = nil the default.
>
> The name is bizarre though; how about something more obvious like
> `isearch-resume-in-command-history' or whatever?

That would be better, yes.

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: isearch-resume-enabled
  2004-09-11 19:21 ` isearch-resume-enabled Kim F. Storm
  2004-09-11 19:32   ` isearch-resume-enabled Miles Bader
@ 2004-09-13  7:01   ` Richard Stallman
  2004-09-13  8:19     ` isearch-resume-enabled Kim F. Storm
  1 sibling, 1 reply; 5+ messages in thread
From: Richard Stallman @ 2004-09-13  7:01 UTC (permalink / raw)
  Cc: emacs-devel

    With the default setting, when you do C-s SOMETHING RET, a command
	 (isearch-resume "SOMETHING" nil nil t "SOMETHING" (quote t))
    is added to the command history.

    IMO, this is very annoying.

I guess I agree.  I see Gerd added that feature in 2000; I am not
sure why.  I would not mind turning it off by default.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: isearch-resume-enabled
  2004-09-13  7:01   ` isearch-resume-enabled Richard Stallman
@ 2004-09-13  8:19     ` Kim F. Storm
  0 siblings, 0 replies; 5+ messages in thread
From: Kim F. Storm @ 2004-09-13  8:19 UTC (permalink / raw)
  Cc: emacs-devel

Richard Stallman <rms@gnu.org> writes:

>     With the default setting, when you do C-s SOMETHING RET, a command
> 	 (isearch-resume "SOMETHING" nil nil t "SOMETHING" (quote t))
>     is added to the command history.
>
>     IMO, this is very annoying.
>
> I guess I agree.  I see Gerd added that feature in 2000; I am not
> sure why.  I would not mind turning it off by default.

I have renamed it to isearch-resume-in-command-history for clarity,
and changed the default to nil.  Updated NEWS and mini.texi.

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2004-09-13  8:19 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <E1C5pO0-0007bv-QA@fencepost.gnu.org>
2004-09-11 19:21 ` isearch-resume-enabled Kim F. Storm
2004-09-11 19:32   ` isearch-resume-enabled Miles Bader
2004-09-11 20:54     ` isearch-resume-enabled Kim F. Storm
2004-09-13  7:01   ` isearch-resume-enabled Richard Stallman
2004-09-13  8:19     ` isearch-resume-enabled Kim F. Storm

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.