Replace "Maintainer: FSF" with the emacs-devel mailing address
[bpt/emacs.git] / lisp / novice.el
index bbb7fa6..46727d3 100644 (file)
@@ -1,17 +1,17 @@
 ;;; novice.el --- handling of disabled commands ("novice mode") for Emacs
 
-;; Copyright (C) 1985, 1986, 1987, 1994, 2001, 2002, 2003, 2004,
-;;   2005, 2006, 2007 Free Software Foundation, Inc.
+;; Copyright (C) 1985-1987, 1994, 2001-2014 Free Software Foundation,
+;; Inc.
 
-;; Maintainer: FSF
+;; Maintainer: emacs-devel@gnu.org
 ;; Keywords: internal, help
 
 ;; This file is part of GNU Emacs.
 
-;; GNU Emacs is free software; you can redistribute it and/or modify
+;; 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 3, or (at your option)
-;; any later version.
+;; the Free Software Foundation, either version 3 of the License, 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
@@ -19,9 +19,7 @@
 ;; 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., 51 Franklin Street, Fifth Floor,
-;; Boston, MA 02110-1301, USA.
+;; along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.
 
 ;;; Commentary:
 
 ;; The command is found in this-command
 ;; and the keys are returned by (this-command-keys).
 
+;;;###autoload
+(define-obsolete-variable-alias 'disabled-command-hook
+  'disabled-command-function "22.1")
 ;;;###autoload
 (defvar disabled-command-function 'disabled-command-function
   "Function to call to handle disabled commands.
 If nil, the feature is disabled, i.e., all commands work normally.")
 
-;;;###autoload
-(define-obsolete-variable-alias 'disabled-command-hook 'disabled-command-function "22.1")
-
 ;; It is ok here to assume that this-command is a symbol
 ;; because we won't get called otherwise.
 ;;;###autoload
-(defun disabled-command-function (&rest ignore)
+(defun disabled-command-function (&optional cmd keys)
+  (unless cmd (setq cmd this-command))
+  (unless keys (setq keys (this-command-keys)))
   (let (char)
     (save-window-excursion
-     (with-output-to-temp-buffer "*Disabled Command*"
-       (let ((keys (this-command-keys)))
+      (with-output-to-temp-buffer "*Disabled Command*" ;; (help-buffer)
         (if (or (eq (aref keys 0)
                     (if (stringp keys)
                         (aref "\M-x" 0)
@@ -59,22 +58,21 @@ If nil, the feature is disabled, i.e., all commands work normally.")
                 (and (>= (length keys) 2)
                      (eq (aref keys 0) meta-prefix-char)
                      (eq (aref keys 1) ?x)))
-            (princ (format "You have invoked the disabled command %s.\n"
-                           (symbol-name this-command)))
+           (princ (format "You have invoked the disabled command %s.\n" cmd))
           (princ (format "You have typed %s, invoking disabled command %s.\n"
-                         (key-description keys) (symbol-name this-command)))))
+                        (key-description keys) cmd)))
        ;; Print any special message saying why the command is disabled.
-       (if (stringp (get this-command 'disabled))
-          (princ (get this-command 'disabled))
+       (if (stringp (get cmd 'disabled))
+           (princ (get cmd 'disabled))
         (princ "It is disabled because new users often find it confusing.\n")
         (princ "Here's the first part of its description:\n\n")
         ;; Keep only the first paragraph of the documentation.
-        (with-current-buffer "*Disabled Command*"
+          (with-current-buffer "*Disabled Command*" ;; standard-output
           (goto-char (point-max))
           (let ((start (point)))
             (save-excursion
               (princ (or (condition-case ()
-                             (documentation this-command)
+                              (documentation cmd)
                            (error nil))
                          "<< not documented >>")))
             (if (search-forward "\n\n" nil t)
@@ -87,9 +85,10 @@ y   to try it and enable it (no questions if you use it again).
 n   to cancel--don't try the command, and it remains disabled.
 SPC to try the command just this once, but leave it disabled.
 !   to try it, and enable all disabled commands for this session only.")
-       (save-excursion
-        (set-buffer standard-output)
-        (help-mode)))
+        ;; Redundant since with-output-to-temp-buffer will do it anyway.
+        ;; (with-current-buffer standard-output
+        ;;   (help-mode))
+        )
      (fit-window-to-buffer (get-buffer-window "*Disabled Command*"))
      (message "Type y, n, ! or SPC (the space bar): ")
      (let ((cursor-in-echo-area t))
@@ -100,31 +99,26 @@ SPC to try the command just this once, but leave it disabled.
         (ding)
         (message "Please type y, n, ! or SPC (the space bar): "))))
     (setq char (downcase char))
-    (if (= char ?\C-g)
-       (setq quit-flag t))
-    (if (= char ?!)
-       (setq disabled-command-function nil))
-    (if (= char ?y)
+    (pcase char
+     (?\C-g (setq quit-flag t))
+     (?! (setq disabled-command-function nil))
+     (?y
        (if (and user-init-file
                 (not (string= "" user-init-file))
                 (y-or-n-p "Enable command for future editing sessions also? "))
-           (enable-command this-command)
-         (put this-command 'disabled nil)))
-    (if (/= char ?n)
-       (call-interactively this-command))))
-
-;;;###autoload
-(defun enable-command (command)
-  "Allow COMMAND to be executed without special confirmation from now on.
-COMMAND must be a symbol.
-This command alters the user's .emacs file so that this will apply
-to future sessions."
-  (interactive "CEnable command: ")
-  (put command 'disabled nil)
+         (enable-command cmd)
+       (put cmd 'disabled nil))))
+    (or (char-equal char ?n)
+        (call-interactively cmd))))
+
+(defun en/disable-command (command disable)
+  (unless (commandp command)
+    (error "Invalid command name `%s'" command))
+  (put command 'disabled disable)
   (let ((init-file user-init-file)
        (default-init-file
          (if (eq system-type 'ms-dos) "~/_emacs" "~/.emacs")))
-    (when (null init-file)
+    (unless init-file
       (if (or (file-exists-p default-init-file)
              (and (eq system-type 'windows-nt)
                   (file-exists-p "~/_emacs")))
@@ -137,9 +131,8 @@ to future sessions."
               (eq system-type 'windows-nt)
               (file-exists-p "~/_emacs"))
          (setq init-file "~/_emacs")))
-    (save-excursion
-      (set-buffer (find-file-noselect
-                  (substitute-in-file-name init-file)))
+    (with-current-buffer (find-file-noselect
+                          (substitute-in-file-name init-file))
       (goto-char (point-min))
       (if (search-forward (concat "(put '" (symbol-name command) " ") nil t)
          (delete-region
@@ -148,49 +141,29 @@ to future sessions."
       ;; Explicitly enable, in case this command is disabled by default
       ;; or in case the code we deleted was actually a comment.
       (goto-char (point-max))
-      (insert "\n(put '" (symbol-name command) " 'disabled nil)\n")
+      (unless (bolp) (newline))
+      (insert "(put '" (symbol-name command) " 'disabled "
+             (symbol-name disable) ")\n")
       (save-buffer))))
 
 ;;;###autoload
-(defun disable-command (command)
-  "Require special confirmation to execute COMMAND from now on.
+(defun enable-command (command)
+  "Allow COMMAND to be executed without special confirmation from now on.
 COMMAND must be a symbol.
 This command alters the user's .emacs file so that this will apply
 to future sessions."
+  (interactive "CEnable command: ")
+  (en/disable-command command nil))
+
+;;;###autoload
+(defun disable-command (command)
+  "Require special confirmation to execute COMMAND from now on.
+COMMAND must be a symbol.
+This command alters your init file so that this choice applies to
+future sessions."
   (interactive "CDisable command: ")
-  (if (not (commandp command))
-      (error "Invalid command name `%s'" command))
-  (put command 'disabled t)
-  (let ((init-file user-init-file)
-       (default-init-file
-         (if (eq system-type 'ms-dos) "~/_emacs" "~/.emacs")))
-    (when (null init-file)
-      (if (or (file-exists-p default-init-file)
-             (and (eq system-type 'windows-nt)
-                  (file-exists-p "~/_emacs")))
-         ;; Started with -q, i.e. the file containing
-         ;; enabled/disabled commands hasn't been read.  Saving
-         ;; settings there would overwrite other settings.
-         (error "Saving settings from \"emacs -q\" would overwrite existing customizations"))
-      (setq init-file default-init-file)
-      (if (and (not (file-exists-p init-file))
-              (eq system-type 'windows-nt)
-              (file-exists-p "~/_emacs"))
-         (setq init-file "~/_emacs")))
-    (save-excursion
-      (set-buffer (find-file-noselect
-                  (substitute-in-file-name init-file)))
-      (goto-char (point-min))
-      (if (search-forward (concat "(put '" (symbol-name command) " ") nil t)
-         (delete-region
-          (progn (beginning-of-line) (point))
-          (progn (forward-line 1) (point)))
-       (goto-char (point-max))
-       (insert ?\n))
-      (insert "(put '" (symbol-name command) " 'disabled t)\n")
-      (save-buffer))))
+  (en/disable-command command t))
 
 (provide 'novice)
 
-;; arch-tag: f83c0f96-497e-4db6-a430-8703716c6dd9
 ;;; novice.el ends here