Re: master 3b721dd39af: Let 'query-replace-show-preview' be a function (bug#81583)
Sean Whitton <[email protected]>
| Newsgroups | gmane.emacs.devel |
|---|---|
| Message-ID | <[email protected]> |
Juri Linkov [02/Sep 3:03pm -04] wrote:
> If you set @code{query-replace-show-preview} to a non-@code{nil}
> +value, the replace commands show a preview of the replacement while you
> +type it
This sentence isn't true because the value can be any function,
including one that does nothing.
I think this would be much better if the options were: the symbol
`replacement-only', the symbol `both', or a custom function. Most users
will not need to write a custom function and these shorter symbols are
easier to handle. And in addition, the documentation could more
truthfully say that setting the user option a non-nil value enables
previews.
Could one of you prepare a patch along those lines, please?
--
Sean Whitton