Re: Search minibuffer history

[Richard Stallman]

It is getting pretty good now, but some of these doc strings need
clarification.

    + (defun minibuffer-history-isearch-end ()
    +   "Clean up the minibuffer after terminating an incremental search."

That doc string needs to identify the usage more precisely.
This is not used after just any isearch, only after a specific kind.

    + (defun minibuffer-history-isearch-search ()
    +   "Return the function for isearch that searches in the minibuffer 
history."

I can't understand that sentence.

    + (defun minibuffer-history-isearch-message (&optional c-q-hack ellipsis)
    +   "Display the minibuffer history search prompt.
    + It is either the standard isearch prompt displayed by `isearch-message'
    + in case of a search error, or an overlay with the isearch prompt displayed
    + over the original minibuffer prompt in case of successful search."

I don't understand "it is either" in that doc string.
Are you talking about _how_ this function does its job?
If so, please say that explicitly.