Re: Search minibuffer history

[Richard Stallman]

    Below is a new patch with added comprehensive comments necessary to
    understand how this feature works (I suppose that docstrings are not needed,
    since new minibuffer-specific functions and variables are purely internal
    and are not intended for calling directly by the user).

It is best to give every function and variable a doc string, even if
it is for internal use only.  Your new functions, including
`minibuffer-history-isearch-setup', `minibuffer-history-isearch-end',
`minibuffer-history-isearch-search' and the rest need overall
documentation.  It has to be either a comment or a doc string, but it
may as well be a doc string.