]> code.delx.au - gnu-emacs/blobdiff - lisp/map-ynp.el
Comment change.
[gnu-emacs] / lisp / map-ynp.el
index f4e22ecbb244ed17f08cfa469ea47bc4ad063631..bfe2ee7af4c39a903aa5e739967b4d8a6d0ed05c 100644 (file)
@@ -1,38 +1,39 @@
 ;;; map-ynp.el --- General-purpose boolean question-asker.
 
-;;; Copyright (C) 1991, 1992, 1993, 1994 Free Software Foundation, Inc.
+;; Copyright (C) 1991, 1992, 1993, 1994, 1995 Free Software Foundation, Inc.
 
 ;; Author: Roland McGrath <roland@gnu.ai.mit.edu>
 ;; Keywords: lisp, extensions
 
-;;; This program is free software; you can redistribute it and/or modify
-;;; it under the terms of the GNU General Public License as published by
-;;; the Free Software Foundation; either version 2, or (at your option)
-;;; any later version.
-;;;
-;;; This program is distributed in the hope that it will be useful,
-;;; but WITHOUT ANY WARRANTY; without even the implied warranty of
-;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-;;; GNU General Public License for more details.
-;;;
-;;; A copy of the GNU General Public License can be obtained from this
-;;; program's author (send electronic mail to roland@ai.mit.edu) or from
-;;; the Free Software Foundation, Inc., 675 Mass Ave, Cambridge, MA
-;;; 02139, USA.
+;; This file is part of GNU Emacs.
+
+;; GNU Emacs is free software; you can redistribute it and/or modify
+;; it under the terms of the GNU General Public License as published by
+;; the Free Software Foundation; either version 2, or (at your option)
+;; any later version.
+
+;; GNU Emacs is distributed in the hope that it will be useful,
+;; but WITHOUT ANY WARRANTY; without even the implied warranty of
+;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;; GNU General Public License for more details.
+
+;; You should have received a copy of the GNU General Public License
+;; along with GNU Emacs; see the file COPYING.  If not, write to the
+;; Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+;; Boston, MA 02111-1307, USA.
 
 ;;; Commentary:
 
-;;; map-y-or-n-p is a general-purpose question-asking function.
-;;; It asks a series of y/n questions (a la y-or-n-p), and decides to
-;;; applies an action to each element of a list based on the answer.
-;;; The nice thing is that you also get some other possible answers
-;;; to use, reminiscent of query-replace: ! to answer y to all remaining
-;;; questions; ESC or q to answer n to all remaining questions; . to answer
-;;; y once and then n for the remainder; and you can get help with C-h.
+;; map-y-or-n-p is a general-purpose question-asking function.
+;; It asks a series of y/n questions (a la y-or-n-p), and decides to
+;; applies an action to each element of a list based on the answer.
+;; The nice thing is that you also get some other possible answers
+;; to use, reminiscent of query-replace: ! to answer y to all remaining
+;; questions; ESC or q to answer n to all remaining questions; . to answer
+;; y once and then n for the remainder; and you can get help with C-h.
 
 ;;; Code:
 
-;;;###autoload
 (defun map-y-or-n-p (prompter actor list &optional help action-alist
                              no-cursor-in-echo-area)
   "Ask a series of boolean questions.
@@ -44,9 +45,8 @@ object or nil.
 If PROMPTER is a string, the prompt is \(format PROMPTER OBJECT\).  If not
 a string, PROMPTER is a function of one arg (an object from LIST), which
 returns a string to be used as the prompt for that object.  If the return
-value is not a string, it is eval'd to get the answer; it may be nil to
-ignore the object, t to act on the object without asking the user, or a
-form to do a more complex prompt.
+value is not a string, it may be nil to ignore the object or non-nil to act
+on the object without asking the user.
 
 ACTOR is a function of one arg (an object from LIST),
 which gets called with each object that the user answers `yes' for.
@@ -78,6 +78,8 @@ are meaningful here.
 Returns the number of actions taken."
   (let* ((actions 0)
         user-keys mouse-event map prompt char elt tail def
+        ;; Non-nil means we should use mouse menus to ask.
+        use-menus
         delayed-switch-frame
         (next (if (or (and list (symbolp list))
                       (subrp list)
@@ -104,10 +106,10 @@ Returns the number of actions taken."
                        ((, (if help (concat action " All " objects)
                              "Do All")) . automatic)
                        (,@ (mapcar (lambda (elt)
-                                     (cons (concat (capitalize (nth 2 elt))
-                                                   " " object)
+                                     (cons (capitalize (nth 2 elt))
                                            (vector (nth 1 elt))))
                                    action-alist))))
+               use-menus t
                mouse-event last-nonmenu-event))                               
       (setq user-keys (if action-alist
                          (concat (mapconcat (function
@@ -131,117 +133,122 @@ Returns the number of actions taken."
                                  (format (, prompter) object)))))
          (while (funcall next)
            (setq prompt (funcall prompter elt))
-           (if (stringp prompt)
-               (progn
-                 (setq quit-flag nil)
-                 ;; Prompt the user about this object.
-                 (if mouse-event
-                     (setq def (or (x-popup-dialog mouse-event
-                                                   (cons prompt map))
-                                   'quit))
-                   ;; Prompt in the echo area.
-                   (let ((cursor-in-echo-area (not no-cursor-in-echo-area)))
-                     (message "%s(y, n, !, ., q, %sor %s) "
-                              prompt user-keys
-                              (key-description (vector help-char)))
-                     (setq char (read-event)))
-                   ;; Show the answer to the question.
-                   (message "%s(y, n, !, ., q, %sor %s) %s"
-                            prompt user-keys
-                            (key-description (vector help-char))
-                            (single-key-description char))
-                   (setq def (lookup-key map (vector char))))
-                 (cond ((eq def 'exit)
-                        (setq next (function (lambda () nil))))
-                       ((eq def 'act)
-                        ;; Act on the object.
-                        (funcall actor elt)
-                        (setq actions (1+ actions)))
-                       ((eq def 'skip)
-                        ;; Skip the object.
-                        )
-                       ((eq def 'act-and-exit)
-                        ;; Act on the object and then exit.
-                        (funcall actor elt)
-                        (setq actions (1+ actions)
-                              next (function (lambda () nil))))
-                       ((eq def 'quit)
-                        (setq quit-flag t)
-                        (setq next (` (lambda ()
-                                        (setq next '(, next))
-                                        '(, elt)))))
-                       ((eq def 'automatic)
-                        ;; Act on this and all following objects.
-                        (if (eval (funcall prompter elt))
-                            (progn
-                              (funcall actor elt)
-                              (setq actions (1+ actions))))
-                        (while (funcall next)
-                          (if (eval (funcall prompter elt))
-                              (progn
-                                (funcall actor elt)
-                                (setq actions (1+ actions))))))
-                       ((eq def 'help)
-                        (with-output-to-temp-buffer "*Help*"
-                          (princ
-                           (let ((object (if help (nth 0 help) "object"))
-                                 (objects (if help (nth 1 help) "objects"))
-                                 (action (if help (nth 2 help) "act on")))
-                             (concat
-                              (format "Type SPC or `y' to %s the current %s;
+           (cond ((stringp prompt)
+                  ;; Prompt the user about this object.
+                  (setq quit-flag nil)
+                  (if use-menus
+                      (setq def (or (x-popup-dialog (or mouse-event use-menus)
+                                                    (cons prompt map))
+                                    'quit))
+                    ;; Prompt in the echo area.
+                    (let ((cursor-in-echo-area (not no-cursor-in-echo-area))
+                          (message-log-max nil))
+                      (message "%s(y, n, !, ., q, %sor %s) "
+                               prompt user-keys
+                               (key-description (vector help-char)))
+                      (if minibuffer-auto-raise
+                          (raise-frame (window-frame (minibuffer-window))))
+                      (setq char (read-event))
+                      ;; Show the answer to the question.
+                      (message "%s(y, n, !, ., q, %sor %s) %s"
+                               prompt user-keys
+                               (key-description (vector help-char))
+                               (single-key-description char)))
+                    (setq def (lookup-key map (vector char))))
+                  (cond ((eq def 'exit)
+                         (setq next (function (lambda () nil))))
+                        ((eq def 'act)
+                         ;; Act on the object.
+                         (funcall actor elt)
+                         (setq actions (1+ actions)))
+                        ((eq def 'skip)
+                         ;; Skip the object.
+                         )
+                        ((eq def 'act-and-exit)
+                         ;; Act on the object and then exit.
+                         (funcall actor elt)
+                         (setq actions (1+ actions)
+                               next (function (lambda () nil))))
+                        ((or (eq def 'quit) (eq def 'exit-prefix))
+                         (setq quit-flag t)
+                         (setq next (` (lambda ()
+                                         (setq next '(, next))
+                                         '(, elt)))))
+                        ((eq def 'automatic)
+                         ;; Act on this and all following objects.
+                         (if (funcall prompter elt)
+                             (progn
+                               (funcall actor elt)
+                               (setq actions (1+ actions))))
+                         (while (funcall next)
+                           (if (funcall prompter elt)
+                               (progn
+                                 (funcall actor elt)
+                                 (setq actions (1+ actions))))))
+                        ((eq def 'help)
+                         (with-output-to-temp-buffer "*Help*"
+                           (princ
+                            (let ((object (if help (nth 0 help) "object"))
+                                  (objects (if help (nth 1 help) "objects"))
+                                  (action (if help (nth 2 help) "act on")))
+                              (concat
+                               (format "Type SPC or `y' to %s the current %s;
 DEL or `n' to skip the current %s;
 ! to %s all remaining %s;
 ESC or `q' to exit;\n"
-                                      action object object action objects)
-                              (mapconcat (function
-                                          (lambda (elt)
-                                            (format "%c to %s"
-                                                    (nth 0 elt)
-                                                    (nth 2 elt))))
-                                         action-alist
-                                         ";\n")
-                              (if action-alist ";\n")
-                              (format "or . (period) to %s \
+                                       action object object action objects)
+                               (mapconcat (function
+                                           (lambda (elt)
+                                             (format "%c to %s"
+                                                     (nth 0 elt)
+                                                     (nth 2 elt))))
+                                          action-alist
+                                          ";\n")
+                               (if action-alist ";\n")
+                               (format "or . (period) to %s \
 the current %s and exit."
-                                      action object)))))
-
-                        (setq next (` (lambda ()
-                                        (setq next '(, next))
-                                        '(, elt)))))
-                       ((vectorp def)
-                        ;; A user-defined key.
-                        (if (funcall (aref def 0) elt) ;Call its function.
-                            ;; The function has eaten this object.
-                            (setq actions (1+ actions))
-                          ;; Regurgitated; try again.
-                          (setq next (` (lambda ()
-                                          (setq next '(, next))
-                                          '(, elt))))))
-                       ((and (consp char)
-                             (eq (car char) 'switch-frame))
-                        ;; switch-frame event.  Put it off until we're done.
-                        (setq delayed-switch-frame char)
-                        (setq next (` (lambda ()
-                                        (setq next '(, next))
-                                        '(, elt)))))
-                       (t
-                        ;; Random char.
-                        (message "Type %s for help."
-                                 (key-description (vector help-char)))
-                        (beep)
-                        (sit-for 1)
-                        (setq next (` (lambda ()
-                                        (setq next '(, next))
-                                        '(, elt)))))))
-             (if (eval prompt)
-                 (progn
-                   (funcall actor elt)
-                   (setq actions (1+ actions)))))))
+                                       action object))))
+                           (save-excursion
+                             (set-buffer standard-output)
+                             (help-mode)))
+
+                         (setq next (` (lambda ()
+                                         (setq next '(, next))
+                                         '(, elt)))))
+                        ((vectorp def)
+                         ;; A user-defined key.
+                         (if (funcall (aref def 0) elt) ;Call its function.
+                             ;; The function has eaten this object.
+                             (setq actions (1+ actions))
+                           ;; Regurgitated; try again.
+                           (setq next (` (lambda ()
+                                           (setq next '(, next))
+                                           '(, elt))))))
+                        ((and (consp char)
+                              (eq (car char) 'switch-frame))
+                         ;; switch-frame event.  Put it off until we're done.
+                         (setq delayed-switch-frame char)
+                         (setq next (` (lambda ()
+                                         (setq next '(, next))
+                                         '(, elt)))))
+                        (t
+                         ;; Random char.
+                         (message "Type %s for help."
+                                  (key-description (vector help-char)))
+                         (beep)
+                         (sit-for 1)
+                         (setq next (` (lambda ()
+                                         (setq next '(, next))
+                                         '(, elt)))))))
+                 (prompt
+                  (funcall actor elt)
+                  (setq actions (1+ actions))))))
       (if delayed-switch-frame
          (setq unread-command-events
                (cons delayed-switch-frame unread-command-events))))
     ;; Clear the last prompt from the minibuffer.
-    (message "")
+    (let ((message-log-max nil))
+      (message ""))
     ;; Return the number of actions that were taken.
     actions))