GNU bug report logs - #56535
28.1; `no' and `no-ding' values for isearch-wrap-pause option (since Emacs 28.1) not working as documented

Previous Next

Package: emacs;

Reported by: Arun V <arunlalv04 <at> gmail.com>

Date: Wed, 13 Jul 2022 11:54:01 UTC

Severity: normal

Tags: moreinfo

Found in version 28.1

Fixed in version 29.0.50

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Juri Linkov <juri <at> linkov.net>
To: Arun V <arunlalv04 <at> gmail.com>
Cc: 56535 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>, visuweshm <at> gmail.com
Subject: bug#56535: 28.1; `no' and `no-ding' values for isearch-wrap-pause option (since Emacs 28.1) not working as documented
Date: Wed, 13 Jul 2022 22:50:08 +0300
[Message part 1 (text/plain, inline)]
> But when I set the value to `no' or `no-ding', I was expecting the
> point to automatically move to the third line immediately after typing
> "t" as mentioned in the above use case.

It was never intended to wrap around on typing a letter or yanking a text.
If such a behavior is a good idea, it could be added as a new value
to this variable.

Please try the following patch after setting isearch-wrap-pause to the
new value 'auto-repeat':

[isearch-wrap-pause-auto-repeat.patch (text/x-diff, inline)]
diff --git a/lisp/isearch.el b/lisp/isearch.el
index cedbfdc2e8..6e14003614 100644
--- a/lisp/isearch.el
+++ b/lisp/isearch.el
@@ -2838,6 +2838,9 @@ isearch-search-and-update
 			    isearch-barrier
 			    (1+ isearch-other-end)))))
       (isearch-search)
+      (when (and (eq isearch-wrap-pause 'auto-repeat)
+                 (not isearch-success))
+        (isearch-repeat (if isearch-forward 'forward 'backward)))
       ))
   (isearch-push-state)
   (if isearch-op-fun (funcall isearch-op-fun))

This bug report was last modified 2 years and 345 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.