From: Thien-Thi Nguyen <ttn@gnuvola.org>
Date: Wed, 8 Sep 2004 11:21:16 +0000 (+0000)
Subject: (grep-mode): Use normal autoload cookie.
X-Git-Tag: ttn-vms-21-2-B4~5009
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=de4c2de7611201b1debf700a8d4bd50701651bad;p=emacs.git

(grep-mode): Use normal autoload cookie.
---

diff --git a/lisp/progmodes/grep.el b/lisp/progmodes/grep.el
index ddfca946a78..3c3ba4c0366 100644
--- a/lisp/progmodes/grep.el
+++ b/lisp/progmodes/grep.el
@@ -500,7 +500,17 @@ temporarily highlight in visited source lines."
 			 command-args)
 		       'grep-mode nil highlight-regexp)))
 
-;;;###autoload (autoload 'grep-mode "grep" nil t)
+;;; This doesn't work:
+;;;      ###autoload (autoload 'grep-mode "grep" nil t)
+;;; The ostensibly correct result is nonetheless opaque to the accounting
+;;; done in `generate-file-autoloads'; in loaddefs.el, the generated elisp is
+;;; correct but the generated header comment for grep.el lacks `grep-mode'.
+;;; This approach also doesn't help other users of `define-compilation-mode'
+;;; who wish to autoload.
+;;;
+;;; Better to extend autoload.el to grok `define-compilation-mode'.
+
+;;;###autoload
 (define-compilation-mode grep-mode "Grep"
   "Sets `grep-last-buffer' and `compilation-window-height'."
   (setq grep-last-buffer (current-buffer))