(define-generic-mode): Add debug declaration. Add defcustom for the

mode hook.
(generic-mode-internal): Use run-mode-hooks.

2 files changed, 40 insertions, 19 deletions

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index b9779b18bfd..cd2043e3f3a 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,12 @@
+2005-03-31  Lute Kamstra  <lute@gnu.org>
+
+        * emacs-lisp/easy-mmode.el (easy-mmode-define-global-mode)
+        (define-minor-mode): Call custom-current-group at load-time.
+
+        * generic.el (define-generic-mode): Add debug declaration.
+        Add defcustom for the mode hook.
+        (generic-mode-internal): Use run-mode-hooks.
+
 2005-03-31  Kim F. Storm  <storm@cua.dk>
 
         * mouse.el (mouse-1-click-follows-link): Increase to 450 ms.
diff --git a/lisp/generic.el b/lisp/generic.el
index e20f73688c7..22fc8b012f3 100644
--- a/lisp/generic.el
+++ b/lisp/generic.el
@@ -217,58 +217,70 @@ FUNCTION-LIST is a list of functions to call to do some
 additional setup.
 
 See the file generic-x.el for some examples of `define-generic-mode'."
-  (let* ((name-unquoted (if (eq (car-safe mode) 'quote) ; Backward compatibility.
-                            (eval mode)
-                          mode))
-         (name-string (symbol-name name-unquoted))
+  (declare (debug (sexp def-form def-form def-form form def-form &optional stringp)))
+
+  ;; Backward compatibility.
+  (when (eq (car-safe mode) 'quote)
+    (setq mode (eval mode)))
+  (let* ((mode-name (symbol-name mode))
          (pretty-name (capitalize (replace-regexp-in-string
-                                   "-mode\\'" "" name-string))))
+                                   "-mode\\'" "" mode-name)))
+         (mode-hook (intern (concat mode-name "-hook"))))
 
     `(progn
        ;; Add a new entry.
-       (add-to-list 'generic-mode-list ,name-string)
+       (add-to-list 'generic-mode-list ,mode-name)
 
        ;; Add it to auto-mode-alist
        (dolist (re ,auto-mode-list)
-         (add-to-list 'auto-mode-alist (cons re ',name-unquoted)))
+         (add-to-list 'auto-mode-alist (cons re ',mode)))
+
+       (defcustom ,mode-hook nil
+         ,(concat "Hook run when entering " pretty-name " mode.")
+         :type 'hook
+         :group (or (custom-current-group)
+                    ',(intern (replace-regexp-in-string
+                               "-mode\\'" "" mode-name))))
 
-       (defun ,name-unquoted ()
+       (defun ,mode ()
          ,(or docstring
               (concat pretty-name " mode.\n"
                       "This a generic mode defined with `define-generic-mode'."))
          (interactive)
-         (generic-mode-internal ',name-unquoted ,comment-list ,keyword-list
+         (generic-mode-internal ',mode ,comment-list ,keyword-list
                                 ,font-lock-list ,function-list)))))
 
 ;;;###autoload
-(defun generic-mode-internal (mode comments keywords font-lock-list funs)
+(defun generic-mode-internal (mode comment-list keyword-list
+                                   font-lock-list function-list)
   "Go into the generic mode MODE."
-  (let* ((modename (symbol-name mode))
-         (generic-mode-hooks (intern (concat modename "-hook")))
+  (let* ((mode-name (symbol-name mode))
          (pretty-name (capitalize (replace-regexp-in-string
-                                   "-mode\\'" "" modename))))
+                                   "-mode\\'" "" mode-name)))
+         (mode-hook (intern (concat mode-name "-hook"))))
 
     (kill-all-local-variables)
 
     (setq major-mode mode
           mode-name pretty-name)
 
-    (generic-mode-set-comments comments)
+    (generic-mode-set-comments comment-list)
 
     ;; Font-lock functionality.
     ;; Font-lock-defaults is always set even if there are no keywords
     ;; or font-lock expressions, so comments can be highlighted.
     (setq generic-font-lock-keywords
           (append
-           (when keywords
-             (list (generic-make-keywords-list keywords font-lock-keyword-face)))
+           (when keyword-list
+             (list (generic-make-keywords-list keyword-list
+                                               font-lock-keyword-face)))
            font-lock-list))
     (setq font-lock-defaults '(generic-font-lock-keywords nil))
 
     ;; Call a list of functions
-    (mapcar 'funcall funs)
+    (mapcar 'funcall function-list)
 
-    (run-hooks generic-mode-hooks)))
+    (run-mode-hooks mode-hook)))
 
 ;;;###autoload
 (defun generic-mode (mode)
@@ -392,7 +404,7 @@ This hook will be installed if the variable
 
 (defun generic-mode-ini-file-find-file-hook ()
   "Hook function to enter Default-Generic mode automatically for INI files.
-Done if the first few lines of a file in Fundamental mode look like an 
+Done if the first few lines of a file in Fundamental mode look like an
 INI file.  This hook is NOT installed by default."
   (and (eq major-mode 'fundamental-mode)
        (save-excursion