Re: query-replace in isearch (was Re: should search ring contain duplicates?)

[Richard Stallman]

    This solution has a significant drawback: this argument is available only
    to one reading function `read-from-minibuffer' and is not available to
    all other minibuffer reading functions (like `read-file-name' etc.).
    Adding this argument to every reading function would be a clumsy solution.

That is a good point.

    I'm really inclined to bind a global variable `history-length' to
    a special value before calling a reading function (and to access this
    value in `read_minibuf' where all reading functions eventually go).

I agree that a global variable seems like a good solution,
but calling it `history-length' seems counterintuitive.
I suggest calling it `history-dont-add'.

Would you like to implement this?  Meanwhile, would you please
install the previous patch to delete KEEP_ALL, aside from the
parts that change the behavior of command-history, as I said before?