[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#55110: 29.0.50; Regression in query-replace prompt
From: |
Juri Linkov |
Subject: |
bug#55110: 29.0.50; Regression in query-replace prompt |
Date: |
Wed, 27 Apr 2022 10:44:56 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (x86_64-pc-linux-gnu) |
>> Maybe this is caused by minibuffer-lazy-highlight-setup
>> that sets filter to replace--region-filter in the minibuffer
>> instead of the original buffer?
>
> Most likely, yes. `replace--region-filter' is modified globally, so a
> similar problem should happen if you temporarily leave the minibuffer
> and do Isearch in any other buffer.
>
> If that's the case, I think we would have two options:
>
> 1) Add a quick fix for the minibuffer Isearch only.
>
> 2) A more complicated change that solves the issue generally by saving
> the region filter in the fashion of isearch-lazy-highlight-regexp et
> alii.
>
> WDYT?
Recently we fixed a similar problem in `perform-replace'
by creating a dynamically bound value in `let':
(let ((opos (point-marker))
;; Restore original isearch filter to allow
;; using isearch in a recursive edit even
;; when perform-replace was started from
;; `xref--query-replace-1' that let-binds
;; `isearch-filter-predicate' (bug#53758).
(isearch-filter-predicate #'isearch-filter-visible))
So maybe a buffer-local value of `isearch-filter-predicate'
in the minibuffer would help.
Also I recommend to make all hooks in `minibuffer-lazy-highlight-setup'
local by adding the argument LOCAL to add-hook/remove-hook.