Re: grep.el regexp

[Juri Linkov]

>     Perhaps what is needed to start with is to ask people to send output
>     examples of different kinds of grep.  We could put these example in a
>     new file etc/grep.txt similarly to etc/compilation.txt.
>
> That is a good idea.  Please start that file.

After visiting this file in `emacs -Q -D' it fails with the error
"Ignoring unknown mode `grep-mode'".  Is it ok to add an autoload cookie
to `grep-mode'?

I also noticed that `C-h f' can't find the definition of `grep-mode'.
The second patch fixes this.

Index: lisp/progmodes/grep.el
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/progmodes/grep.el,v
retrieving revision 1.38
diff -c -r1.38 grep.el
*** lisp/progmodes/grep.el      4 Jul 2005 16:59:19 -0000       1.38
--- lisp/progmodes/grep.el      12 Jul 2005 06:55:11 -0000
***************
*** 514,519 ****
--- 518,524 ----
                         command-args)
                       'grep-mode nil highlight-regexp)))
  
+ ;;;###autoload
  (define-compilation-mode grep-mode "Grep"
    "Sets `grep-last-buffer' and `compilation-window-height'."
    (setq grep-last-buffer (current-buffer))

Index: lisp/emacs-lisp/find-func.el
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/emacs-lisp/find-func.el,v
retrieving revision 1.58
diff -c -r1.58 find-func.el
*** lisp/emacs-lisp/find-func.el        4 Jul 2005 17:33:35 -0000       1.58
--- lisp/emacs-lisp/find-func.el        12 Jul 2005 06:55:32 -0000
***************
*** 62,68 ****
    ;;  (define-derived-mode foo ...), (define-minor-mode foo)
    (concat
     "^\\s-*(\\(def\\(ine-skeleton\\|ine-generic-mode\\|ine-derived-mode\\|\
! ine-minor-mode\\|un-cvs-mode\\|foo\\|[^cfgv]\\w+\\*?\\)\
  \\|easy-mmode-define-global-mode\\|menu-bar-make-toggle\\)"
     find-function-space-re
     "\\('\\|\(quote \\)?%s\\(\\s-\\|$\\|\(\\|\)\\)")
--- 62,68 ----
    ;;  (define-derived-mode foo ...), (define-minor-mode foo)
    (concat
     "^\\s-*(\\(def\\(ine-skeleton\\|ine-generic-mode\\|ine-derived-mode\\|\
! 
ine-minor-mode\\|ine-compilation-mode\\|un-cvs-mode\\|foo\\|[^cfgv]\\w+\\*?\\)\
  \\|easy-mmode-define-global-mode\\|menu-bar-make-toggle\\)"
     find-function-space-re
     "\\('\\|\(quote \\)?%s\\(\\s-\\|$\\|\(\\|\)\\)")

-- 
Juri Linkov
http://www.jurta.org/emacs/