From: Jens Schmidt <jschmidt4gnu@vodafonemail.de>
Date: Tue, 16 May 2023 21:31:19 +0000 (+0200)
Subject: Clarify misleading comment in isearch.el
X-Git-Tag: emacs-29.0.92~136
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=dc1a437d460d5e100563eb7dde3e0fdc0e8e53d9;p=emacs.git

Clarify misleading comment in isearch.el

Clarify a misleading comment in isearch.el as to whether frame events
should exit an isearch or not (Bug#62032, Bug#41338 for background
information).
* lisp/isearch.el (isearch-mode-map): Replace the misleading comment.
(Bug#62032)

Copyright-paperwork-exempt: yes
---

diff --git a/lisp/isearch.el b/lisp/isearch.el
index 094e02d605e..3d2bbda4975 100644
--- a/lisp/isearch.el
+++ b/lisp/isearch.el
@@ -633,9 +633,9 @@ This is like `describe-bindings', but displays only Isearch keys."
 
     (define-key map "\M-\t" 'isearch-complete)
 
-    ;; Pass frame events transparently so they won't exit the search.
-    ;; In particular, if we have more than one display open, then a
-    ;; switch-frame might be generated by someone typing at another keyboard.
+    ;; Frame events should exit the search, because such frame events
+    ;; as `switch-frame’ and `delete-frame’ change the current buffer.
+    ;; See Bug#41338 for details.
     (define-key map [switch-frame] nil)
     (define-key map [delete-frame] nil)
     (define-key map [iconify-frame] nil)