Re: master 49f3888c5dc: Add new options to 'query-replace-show-preview' (bug#81583)

Juri Linkov <[email protected]>
Newsgroups gmane.emacs.devel
Organization LINKOV.NET
Message-ID <[email protected]>
>>     * lisp/replace.el (query-replace-show-preview): Accept 'both'
>>     and a function of two arguments as values.  Doc fix.
>
> With these user options with multiple possibilities, where there isn't a
> backwards compatibility issue to consider (as there isn't here), it's
> nice to make it so that `t' turns on all the features and the other
> values turn them on partially.  Then there is a natural ordering
> nil->something->other->t.
>
> So could we perhaps make `t' mean `both' and use another symbol for when
> only the result is displayed?

Another variant is to support either nil or a value with
a function also for the case when only the result is displayed
with e.g. 'replace-preview-only-replacement' instead of t.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.