[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-
From: |
Dmitry Gutov |
Subject: |
bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results |
Date: |
Sun, 08 May 2016 22:06:37 +0300 |
1. Do a search that has several matches in one buffer, e.g. using
dired-do-find-regexp.
2. Press `r' in the *xref* buffer, to initiate replacement. Input `.*'
and `abcd', for instance. This value of FROM is important.
3. Replace a couple then undo that (press y, y, u, u).
4. Try agreeing with the subsequent prompts. The replacements performed
then will be wrong.
I've tried to come up with a patch but stopped short of really delving
into the code of perform-replace. Which seems really necessary at this
point. Attaching what I already have.
The big problem is that perform-replace does not consistently use
replace-re-search-function. With the new undo code, it's became worse.
After you press `u', it seems to switch to moving around the saved match
data stack and using a plain looking-at, without checking with
isearch-filter-predicate.
xref-undo-fix-attempt.diff
Description: Text Data
In GNU Emacs 25.1.50.5 (x86_64-unknown-linux-gnu, GTK+ Version 3.18.9)
of 2016-05-08 built on axl
Repository revision: 2eb6817ba971184cc109f8530f4b3b38f65650ea
Windowing system distributor 'The X.Org Foundation', version 11.0.11803000
System Description: Ubuntu 16.04 LTS
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results,
Dmitry Gutov <=
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Juri Linkov, 2016/05/08
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Dmitry Gutov, 2016/05/08
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Juri Linkov, 2016/05/09
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Dmitry Gutov, 2016/05/09
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Dmitry Gutov, 2016/05/09
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Juri Linkov, 2016/05/10
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Dmitry Gutov, 2016/05/10
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Juri Linkov, 2016/05/11
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Dmitry Gutov, 2016/05/11
- bug#23484: 25.1.50; undo doesn't work properly in xref-query-replace-in-results, Juri Linkov, 2016/05/12