Re: isearch-resume-enabled

Re: isearch-resume-enabled

[Kim F. Storm]

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

Re: isearch-resume-enabled

[Miles Bader]

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.

Re: isearch-resume-enabled

[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

Re: isearch-resume-enabled

[Richard Stallman]

    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.

Re: isearch-resume-enabled

[Kim F. Storm]

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