emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

master 7ba76dc9c3e: lisp/help-fns.el: Revert commit c12838c73ef161850a08


From: Stefan Monnier
Subject: master 7ba76dc9c3e: lisp/help-fns.el: Revert commit c12838c73ef161850a081f9ccea6e375b7c2f93b
Date: Tue, 15 Aug 2023 10:22:00 -0400 (EDT)

branch: master
commit 7ba76dc9c3efbbcdbeb47529744ec4da5dae6ada
Author: Stefan Monnier <monnier@iro.umontreal.ca>
Commit: Stefan Monnier <monnier@iro.umontreal.ca>

    lisp/help-fns.el: Revert commit c12838c73ef161850a081f9ccea6e375b7c2f93b
    
    * lisp/help-fns.el (help-fns--describe-function-or-command-prompt):
    Don't allow specifying a non-existing function.
---
 lisp/help-fns.el | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/lisp/help-fns.el b/lisp/help-fns.el
index bedc5a9e49b..81c58ba1998 100644
--- a/lisp/help-fns.el
+++ b/lisp/help-fns.el
@@ -229,11 +229,11 @@ interactive command."
                (lambda (f) (if want-command
                           (commandp f)
                         (or (fboundp f) (get f 'function-documentation))))
-               ;; We use 'confirm' here, unlike in other describe-*
-               ;; commands, for cases like a function that is advised
-               ;; but not yet defined (e.g., if 'advice-add' is called
-               ;; before defining the function).
-               'confirm nil nil
+               ;; We used `confirm' for a while because we may want to see the
+               ;; meta-info about a function even if the function itself is not
+               ;; defined, but this use case is too marginal and rarely tested,
+               ;; not worth the trouble (bug#64902).
+               t nil nil
                (and fn (symbol-name fn)))))
     (unless (equal val "")
       (setq fn (intern val)))



reply via email to

[Prev in Thread] Current Thread [Next in Thread]