Subject: Re: bug#32990: 26.1.50; isearch-forward + t-m-m/mark-active doc

From: Eli Zaretskii <[email-masked]>
Date: Sat, 20 Oct 2018 12:59:48 +0300

> Date: Tue, 16 Oct 2018 20:57:13 +0200
> From: [email-masked] (Charles A. Roelli)
> CC: [email-masked]
> 
> Ok, how about the following?

LGTM, with 2 comments:

>  (defun isearch-delete-char ()
> -  "Discard last input item and move point back.
> -Last input means the last character or the last isearch command
> -that added or deleted characters from the search string,
> -moved point, toggled regexp mode or case-sensitivity, etc.
> -If no previous match was done, just beep."
> +  "Undo last input item during a search.
> +
> +An input item is a command that pushes a new state of isearch (as

I think you meant to say that "An input item is the result of a
command that pushes a new state ...".  Right?

> +Info node `Basic Isearch' explains when Emacs pushes a new
> +isearch state.

It is better to use `(emacs)Basic Search' as a link to the manual;
that's what we do elsewhere.

Thanks.