(completion-fixup-function): Variable deleted.
[bpt/emacs.git] / lisp / imenu.el
index 64d5007..2f353e1 100644 (file)
@@ -1,11 +1,12 @@
 ;;; imenu.el --- Framework for mode-specific buffer indexes.
 
-;; Copyright (C) 1994, 1995, 1996 Free Software Foundation, Inc.
+;; Copyright (C) 1994, 1995, 1996, 1997, 1998 Free Software Foundation, Inc.
 
 ;; Author: Ake Stenhoff <etxaksf@aom.ericsson.se>
 ;;         Lars Lindberg <lli@sypro.cap.se>
+;; Maintainer: FSF
 ;; Created: 8 Feb 1994
-;; Keywords: tools
+;; Keywords: tools convenience
 
 ;; This file is part of GNU Emacs.
 
@@ -31,6 +32,8 @@
 ;;   A buffer index is an alist of names and buffer positions.
 ;;   For instance all functions in a C-file and their positions.
 ;;
+;;   It is documented in the Emacs Lisp manual.
+;;
 ;; How it works:
 
 ;;   A mode-specific function is called to generate the index.  It is
 ;;   customize for other modes.  A function for jumping to the chosen
 ;;   index position is also supplied.
 
-;;; Thanks goes to
+;;; History:
+;;  Thanks go to
 ;;  [simon] - Simon Leinen simon@lia.di.epfl.ch
 ;;  [dean] - Dean Andrews ada@unison.com
-;;  [alon] - Alon Albert al@mercury.co.il 
+;;  [alon] - Alon Albert al@mercury.co.il
 ;;  [greg] - Greg Thompson gregt@porsche.visix.COM
 ;;  [wolfgang] - Wolfgang Bangerth zcg51122@rpool1.rus.uni-stuttgart.de
 ;;  [kai] - Kai Grossjohann grossjoh@linus.informatik.uni-dortmund.de
 ;;;
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 
-(defvar imenu-auto-rescan nil
-  "*Non-nil means Imenu should always rescan the buffers.")
+(defgroup imenu nil
+  "Mode-specific buffer indexes."
+  :group 'matching
+  :group 'frames
+  :group 'convenience
+  :link '(custom-manual "(elisp)Imenu"))
+
+(defcustom imenu-use-markers t
+  "*Non-nil means use markers instead of integers for Imenu buffer positions.
+
+Setting this to nil makes Imenu work a little faster but editing the
+buffer will make the generated index positions wrong.
+
+This might not yet be honored by all index-building functions."
+  :type 'boolean
+  :group 'imenu)
+
 
-(defvar imenu-auto-rescan-maxout 60000 
-  "* auto-rescan is disabled in buffers larger than this.
-This variable is buffer-local.")
+(defcustom imenu-max-item-length 60
+  "*If a number, truncate Imenu entries to that length."
+  :type '(choice integer
+                (const :tag "Unlimited"))
+  :group 'imenu)
 
-(defvar imenu-always-use-completion-buffer-p nil
+(defcustom imenu-auto-rescan nil
+  "*Non-nil means Imenu should always rescan the buffers."
+  :type 'boolean
+  :group 'imenu)
+
+(defcustom imenu-auto-rescan-maxout 60000
+  "*Imenu auto-rescan is disabled in buffers larger than this size (in bytes).
+This variable is buffer-local."
+  :type 'integer
+  :group 'imenu)
+
+(defcustom imenu-always-use-completion-buffer-p nil
   "*Set this to non-nil for displaying the index in a completion buffer.
 
-Non-nil means always display the index in a completion buffer.
-Nil means display the index as a mouse menu when the mouse was
-used to invoke `imenu'.
-`never' means never automatically display a listing of any kind.")
+`never' means never automatically display a listing of any kind.
+A value of nil (the default) means display the index as a mouse menu
+if the mouse was used to invoke `imenu'.
+Another non-nil value means always display the index in a completion buffer."
+  :type '(choice (const :tag "On Mouse" nil)
+                (const :tag "Never" never)
+                (other :tag "Always" t))
+  :group 'imenu)
 
-(defvar imenu-sort-function nil
+;;;###autoload
+(defcustom imenu-sort-function nil
   "*The function to use for sorting the index mouse-menu.
 
 Affects only the mouse index menu.
@@ -91,35 +128,49 @@ in the buffer.
 
 Set it to `imenu--sort-by-name' if you want alphabetic sorting.
 
-The function should take two arguments and return T if the first
+The function should take two arguments and return t if the first
 element should come before the second.  The arguments are cons cells;
-\(NAME . POSITION).  Look at `imenu--sort-by-name' for an example.")
-
-(defvar imenu-max-items 25
-  "*Maximum number of elements in an mouse menu for Imenu.")
-
-(defvar imenu-scanning-message "Scanning buffer for index (%3d%%)"
+\(NAME . POSITION).  Look at `imenu--sort-by-name' for an example."
+  :type '(choice (const :tag "No sorting" nil)
+                (const :tag "Sort by name" 'imenu--sort-by-name)
+                (function :tag "Another function"))
+  :group 'imenu)
+
+(defcustom imenu-max-items 25
+  "*Maximum number of elements in a mouse menu for Imenu."
+  :type 'integer
+  :group 'imenu)
+
+(defcustom imenu-scanning-message "Scanning buffer for index (%3d%%)"
   "*Progress message during the index scanning of the buffer.
-If non-nil, user gets a message during the scanning of the buffer
+If non-nil, user gets a message during the scanning of the buffer.
 
 Relevant only if the mode-specific function that creates the buffer
-index use `imenu-progress-message'.")
+index use `imenu-progress-message', and not useful if that is fast, in
+which case you might as well set this to nil."
+  :type '(choice string
+                (const :tag "None" nil))
+  :group 'imenu)
 
-(defvar imenu-space-replacement "^"
+(defcustom imenu-space-replacement "."
   "*The replacement string for spaces in index names.
-Used when presenting the index in a completion-buffer to make the
-names work as tokens.")
+Used when presenting the index in a completion buffer to make the
+names work as tokens."
+  :type 'string
+  :group 'imenu)
 
-(defvar imenu-level-separator ":"
+(defcustom imenu-level-separator ":"
   "*The separator between index names of different levels.
 Used for making mouse-menu titles and for flattening nested indexes
-with name concatenation.")
+with name concatenation."
+  :type 'string
+  :group 'imenu)
 
 ;;;###autoload
 (defvar imenu-generic-expression nil
   "The regex pattern to use for creating a buffer index.
 
-If non-nil this pattern is passed to `imenu-create-index-with-pattern'
+If non-nil this pattern is passed to `imenu--generic-function'
 to create a buffer index.
 
 The value should be an alist with elements that look like this:
@@ -128,7 +179,7 @@ or like this:
  (MENU-TITLE REGEXP INDEX FUNCTION ARGUMENTS...)
 with zero or more ARGUMENTS.  The former format creates a simple element in
 the index alist when it matches; the latter creates a special element
-of the form  (NAME FUNCTION NAME POSITION-MARKER ARGUMENTS...)
+of the form  (NAME FUNCTION POSITION-MARKER ARGUMENTS...)
 with FUNCTION and ARGUMENTS beiong copied from `imenu-generic-expression'.
 
 MENU-TITLE is a string used as the title for the submenu or nil if the
@@ -142,19 +193,23 @@ menu.  See the info section on Regexps for more information.
 INDEX points to the substring in REGEXP that contains the name (of the
 function, variable or type) that is to appear in the menu.
 
-For emacs-lisp-mode for example PATTERN would look like:
+The variable is buffer-local.
 
-'((nil \"^\\\\s-*(def\\\\(un\\\\|subst\\\\|macro\\\\|advice\\\\)\\\\s-+\\\\([-A-Za-z0-9+]+\\\\)\" 2)
-  (\"*Vars*\" \"^\\\\s-*(def\\\\(var\\\\|const\\\\)\\\\s-+\\\\([-A-Za-z0-9+]+\\\\)\" 2)
-  (\"*Types*\" \"^\\\\s-*(def\\\\(type\\\\|struct\\\\|class\\\\|ine-condition\\\\)\\\\s-+\\\\([-A-Za-z0-9+]+\\\\)\" 2))
+The variable `imenu-case-fold-search' determines whether or not the
+regexp matches are case sensitive. and `imenu-syntax-alist' can be
+used to alter the syntax table for the search.
 
-The variable is buffer-local.")
+For example, see the value of `lisp-imenu-generic-expression' used by
+`lisp-mode' and `emacs-lisp-mode' with `imenu-syntax-alist' set
+locally to give the characters which normally have \"punctuation\"
+syntax \"word\" syntax during matching.")
 
 ;;;###autoload
 (make-variable-buffer-local 'imenu-generic-expression)
 
 ;;;; Hooks
 
+;;;###autoload
 (defvar imenu-create-index-function 'imenu-default-create-index-function
   "The function to use for creating a buffer index.
 
@@ -162,16 +217,18 @@ It should be a function that takes no arguments and returns an index
 of the current buffer as an alist.
 
 Simple elements in the alist look like (INDEX-NAME . INDEX-POSITION).
-Special elements look like (INDEX-NAME FUNCTION ARGUMENTS...).
+Special elements look like (INDEX-NAME INDEX-POSITION FUNCTION ARGUMENTS...).
 A nested sub-alist element looks like (INDEX-NAME SUB-ALIST).
 The function `imenu--subalist-p' tests an element and returns t
- if it is a sub-alist.
+if it is a sub-alist.
 
 This function is called within a `save-excursion'.
 
 The variable is buffer-local.")
+;;;###autoload
 (make-variable-buffer-local 'imenu-create-index-function)
 
+;;;###autoload
 (defvar imenu-prev-index-position-function 'beginning-of-defun
   "Function for finding the next index position.
 
@@ -181,37 +238,52 @@ to a function that will find the next index, looking backwards in the
 file.
 
 The function should leave point at the place to be connected to the
-index and it should return nil when it doesn't find another index.")
+index and it should return nil when it doesn't find another index.
+
+This variable is local in all buffers.")
+;;;###autoload
 (make-variable-buffer-local 'imenu-prev-index-position-function)
 
+;;;###autoload
 (defvar imenu-extract-index-name-function nil
-  "Function for extracting the index name.
+  "Function for extracting the index item name, given a position.
+
+This function is called after `imenu-prev-index-position-function'
+finds a position for an index item, with point at that position.
+It should return the name for that index item.
 
-This function is called after the function pointed out by
-`imenu-prev-index-position-function'.")
+This variable is local in all buffers.")
+;;;###autoload
 (make-variable-buffer-local 'imenu-extract-index-name-function)
 
+;;;###autoload
+(defvar imenu-default-goto-function 'imenu-default-goto-function
+  "The default function called when selecting an Imenu item.
+The function in this variable is called when selecting a normal index-item.")
+;;;###autoload
+(make-variable-buffer-local 'imenu-default-goto-function)
+
+
 (defun imenu--subalist-p (item)
   (and (consp (cdr item)) (listp (cadr item))
-       (not (eq (caadr item) 'lambda))))
+       (not (eq (car (cadr item)) 'lambda))))
 
-;;;
-;;; Macro to display a progress message.
-;;; RELPOS is the relative position to display.
-;;; If RELPOS is nil, then the relative position in the buffer
-;;; is calculated.
-;;; PREVPOS is the variable in which we store the last position displayed.
+;; Macro to display a progress message.
+;; RELPOS is the relative position to display.
+;; If RELPOS is nil, then the relative position in the buffer
+;; is calculated.
+;; PREVPOS is the variable in which we store the last position displayed.
 (defmacro imenu-progress-message (prevpos &optional relpos reverse)
-  (` (and
-      imenu-scanning-message
-      (let ((pos (, (if relpos
-                       relpos
-                     (` (imenu--relative-position (, reverse)))))))
-       (if (, (if relpos t
-                (` (> pos (+ 5 (, prevpos))))))
-           (progn
-             (message imenu-scanning-message pos)
-             (setq (, prevpos) pos)))))))
+  `(and
+    imenu-scanning-message
+    (let ((pos ,(if relpos
+                   relpos
+                 `(imenu--relative-position ,reverse))))
+      (if ,(if relpos t
+            `(> pos (+ 5 ,prevpos)))
+         (progn
+           (message imenu-scanning-message pos)
+           (setq ,prevpos pos))))))
 
 
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
@@ -226,16 +298,15 @@ This function is called after the function pointed out by
 (defun imenu-example--name-and-position ()
   (save-excursion
     (forward-sexp -1)
-    (let ((beg (point))
-         (end (progn (forward-sexp) (point)))
-         (marker (make-marker)))
-      (set-marker marker beg)
+    ;; [ydi] modified for imenu-use-markers
+    (let ((beg (if imenu-use-markers (point-marker) (point)))
+         (end (progn (forward-sexp) (point))))
       (cons (buffer-substring beg end)
-           marker))))
+           beg))))
 
 ;;;
 ;;; Lisp
-;;; 
+;;;
 
 (defun imenu-example--lisp-extract-index-name ()
   ;; Example of a candidate for `imenu-extract-index-name-function'.
@@ -264,31 +335,31 @@ This function is called after the function pointed out by
     ;; Search for the function
     (while (beginning-of-defun)
       (imenu-progress-message prev-pos nil t)
-      (save-match-data
-       (and (looking-at "(def")
-            (save-excursion
+         (save-match-data
+           (and (looking-at "(def")
+                (save-excursion
               (down-list 1)
-              (cond
+                  (cond
                ((looking-at "def\\(var\\|const\\)")
-                (forward-sexp 2)
-                (push (imenu-example--name-and-position)
-                      index-var-alist))
+                    (forward-sexp 2)
+                    (push (imenu-example--name-and-position)
+                          index-var-alist))
                ((looking-at "def\\(un\\|subst\\|macro\\|advice\\)")
-                (forward-sexp 2)
-                (push (imenu-example--name-and-position)
-                      index-alist))
+                    (forward-sexp 2)
+                    (push (imenu-example--name-and-position)
+                          index-alist))
                ((looking-at "def\\(type\\|struct\\|class\\|ine-condition\\)")
-                (forward-sexp 2)
+                    (forward-sexp 2)
                 (if (= (char-after (1- (point))) ?\))
-                    (progn
+                        (progn
                       (forward-sexp -1)
-                      (down-list 1)
+                          (down-list 1)
                       (forward-sexp 1)))
-                (push (imenu-example--name-and-position)
-                      index-type-alist))
-               (t
-                (forward-sexp 2)
-                (push (imenu-example--name-and-position)
+                    (push (imenu-example--name-and-position)
+                          index-type-alist))
+                   (t
+                    (forward-sexp 2)
+                    (push (imenu-example--name-and-position)
                       index-unknown-alist)))))))
     (imenu-progress-message prev-pos 100)
     (and index-var-alist
@@ -304,7 +375,7 @@ This function is called after the function pointed out by
 
 ;; Regular expression to find C functions
 (defvar imenu-example--function-name-regexp-c
-  (concat 
+  (concat
    "^[a-zA-Z0-9]+[ \t]?"               ; type specs; there can be no
    "\\([a-zA-Z0-9_*]+[ \t]+\\)?"       ; more than 3 tokens, right?
    "\\([a-zA-Z0-9_*]+[ \t]+\\)?"
@@ -345,7 +416,14 @@ This function is called after the function pointed out by
 
 ;; The latest buffer index.
 ;; Buffer local.
-(defvar imenu--index-alist nil)
+(defvar imenu--index-alist nil
+  "The buffer index computed for this buffer in Imenu.
+Simple elements in the alist look like (INDEX-NAME . INDEX-POSITION).
+Special elements look like (INDEX-NAME INDEX-POSITION FUNCTION ARGUMENTS...).
+A nested sub-alist element looks like (INDEX-NAME SUB-ALIST).
+
+This variable is local in all buffers, once set.")
+
 (make-variable-buffer-local 'imenu--index-alist)
 
 ;; The latest buffer index used to update the menu bar menu.
@@ -365,11 +443,14 @@ This function is called after the function pointed out by
 ;;;
 ;;; Sort function
 ;;; Sorts the items depending on their index name.
-;;; An item look like (NAME . POSITION).
+;;; An item looks like (NAME . POSITION).
 ;;;
 (defun imenu--sort-by-name (item1 item2)
   (string-lessp (car item1) (car item2)))
 
+(defun imenu--sort-by-position (item1 item2)
+  (< (cdr item1) (cdr item2)))
+
 (defun imenu--relative-position (&optional reverse)
   ;; Support function to calculate relative position in buffer
   ;; Beginning of buffer is 0 and end of buffer is 100
@@ -423,7 +504,8 @@ This function is called after the function pointed out by
                     (oldlist menulist))
                 ;; Copy list method from the cl package `copy-list'
                 (while (consp oldlist) (push (pop oldlist) res))
-                (prog1 (nreverse res) (setcdr res oldlist)))
+                (if res                ; in case, e.g. no functions defined
+                    (prog1 (nreverse res) (setcdr res oldlist))))
               imenu-sort-function)))
     (if (> (length menulist) imenu-max-items)
        (let ((count 0))
@@ -439,46 +521,77 @@ This function is called after the function pointed out by
 ;;; Split up each long alist that are nested within ALIST
 ;;; into nested alists.
 (defun imenu--split-submenus (alist)
-  (mapcar (function (lambda (elt)
-                     (if (and (consp elt)
-                              (stringp (car elt))
-                              (listp (cdr elt)))
-                         (imenu--split-menu (cdr elt) (car elt))
-                       elt)))
+  (mapcar (function
+          (lambda (elt)
+            (if (and (consp elt)
+                     (stringp (car elt))
+                     (listp (cdr elt)))
+                (imenu--split-menu (cdr elt) (car elt))
+              elt)))
          alist))
 
-;;;
-;;; Find all items in this buffer that should be in the index.
-;;; Returns an alist on the form
-;;; ((NAME . POSITION) (NAME . POSITION) ...)
-;;;
+;;; Truncate all strings in MENULIST to imenu-max-item-length
+(defun imenu--truncate-items (menulist)
+  (mapcar (function
+          (lambda (item)
+            (cond
+             ((consp (cdr item))
+              (imenu--truncate-items (cdr item)))
+             (t
+              ;; truncate if necessary
+              (if (and (numberp imenu-max-item-length)
+                       (> (length (car item)) imenu-max-item-length))
+                  (setcar item (substring (car item) 0 imenu-max-item-length)))))))
+         menulist))
+
 
 (defun imenu--make-index-alist (&optional noerror)
-  ;; Create a list for this buffer only when needed.
+  "Create an index-alist for the definitions in the current buffer.
+
+Report an error if the list is empty unless NOERROR is supplied and
+non-nil.
+
+Simple elements in the alist look like (INDEX-NAME . INDEX-POSITION).
+Special elements look like (INDEX-NAME FUNCTION ARGUMENTS...).
+A nested sub-alist element looks like (INDEX-NAME SUB-ALIST).
+The function `imenu--subalist-p' tests an element and returns t
+if it is a sub-alist.
+
+There is one simple element with negative POSITION; that's intended
+as a way for the user to ask to recalculate the buffer's index alist."
   (or (and imenu--index-alist
           (or (not imenu-auto-rescan)
               (and imenu-auto-rescan
                    (> (buffer-size)  imenu-auto-rescan-maxout))))
-      ;; Get the index
-      (setq imenu--index-alist
-           (save-excursion
-             (funcall imenu-create-index-function))))
+      ;; Get the index; truncate if necessary
+      (progn
+       (setq imenu--index-alist
+             (save-excursion
+               (save-restriction
+                 (widen)
+                 (funcall imenu-create-index-function))))
+       (imenu--truncate-items imenu--index-alist)))
   (or imenu--index-alist noerror
       (error "No items suitable for an index found in this buffer"))
   (or imenu--index-alist
       (setq imenu--index-alist (list nil)))
   ;; Add a rescan option to the index.
   (cons imenu--rescan-item imenu--index-alist))
-;;;
+
 ;;; Find all markers in alist and makes
 ;;; them point nowhere.
-;;;
+;;; The top-level call uses nil as the argument;
+;;; non-nil arguments are in recursivecalls.
+(defvar imenu--cleanup-seen)
+
 (defun imenu--cleanup (&optional alist)
-  ;; Sets the markers in imenu--index-alist 
-  ;; point nowhere.
-  ;; if alist is provided use that list.
-  (or alist
-      (setq alist imenu--index-alist))
+  ;; If alist is provided use that list.
+  ;; If not, empty the table of lists already seen
+  ;; and use imenu--index-alist.
+  (if alist
+      (setq imenu--cleanup-seen (cons alist imenu--cleanup-seen))
+    (setq alist imenu--index-alist imenu--cleanup-seen (list alist)))
+
   (and alist
        (mapcar
        (function
@@ -486,6 +599,8 @@ This function is called after the function pointed out by
           (cond
            ((markerp (cdr item))
             (set-marker (cdr item) nil))
+           ;; Don't process one alist twice.
+           ((memq (cdr item) imenu--cleanup-seen))
            ((imenu--subalist-p item)
             (imenu--cleanup (cdr item))))))
        alist)
@@ -506,8 +621,7 @@ This function is called after the function pointed out by
                                     (imenu--menubar-select ',item))
                       (cons '(nil) item))))
            (cons (car item)
-                 (cons (car item) end))))
-        )))
+                 (cons (car item) end)))))))
      alist)))
 
 ;; If COMMANDS is non-nil, make a real keymap
@@ -516,16 +630,15 @@ This function is called after the function pointed out by
 (defun imenu--create-keymap-1 (title alist &optional commands)
   (append (list 'keymap title) (imenu--create-keymap-2 alist 0 commands)))
 
-
 (defun imenu--in-alist (str alist)
   "Check whether the string STR is contained in multi-level ALIST."
   (let (elt head tail res)
     (setq res nil)
     (while alist
-      (setq elt (car alist) 
+      (setq elt (car alist)
            tail (cdr elt)
-           alist (cdr alist) 
-           head (car elt)) 
+           alist (cdr alist)
+           head (car elt))
       ;; A nested ALIST element looks like
       ;;   (INDEX-NAME (INDEX-NAME . INDEX-POSITION) ...)
       ;; while a bottom-level element looks like
@@ -539,6 +652,18 @@ This function is called after the function pointed out by
             (setq alist nil res elt))))
     res))
 
+(defvar imenu-syntax-alist nil
+  "Alist of syntax table modifiers to use while executing `imenu--generic-function'.
+
+The car of the assocs may be either a character or a string and the
+cdr is a syntax description appropriate fo `modify-syntax-entry'.  For
+a string, all the characters in the string get the specified syntax.
+
+This is typically used to give word syntax to characters which
+normally have symbol syntax to simplify `imenu-expression'
+and speed-up matching.")
+(make-variable-buffer-local 'imenu-syntax-alist)
+
 (defun imenu-default-create-index-function ()
   "*Wrapper for index searching functions.
 
@@ -547,27 +672,28 @@ Moves point to end of buffer and then repeatedly calls
 Their results are gathered into an index alist."
   ;; These should really be done by setting imenu-create-index-function
   ;; in these major modes.  But save that change for later.
-  (cond ((and (fboundp imenu-prev-index-position-function)
-             (fboundp imenu-extract-index-name-function))
+  (cond ((and imenu-prev-index-position-function
+             imenu-extract-index-name-function)
         (let ((index-alist '())
               prev-pos name)
           (goto-char (point-max))
           (imenu-progress-message prev-pos 0 t)
           ;; Search for the function
-          (while (funcall imenu-prev-index-position-function) 
+          (while (funcall imenu-prev-index-position-function)
             (imenu-progress-message prev-pos nil t)
             (save-excursion
               (setq name (funcall imenu-extract-index-name-function)))
             (and (stringp name)
-                 (push (cons name (point)) index-alist)))
+                 ;; [ydi] updated for imenu-use-markers
+                 (push (cons name (if imenu-use-markers (point-marker) (point)))
+                       index-alist)))
           (imenu-progress-message prev-pos 100 t)
           index-alist))
        ;; Use generic expression if possible.
        ((and imenu-generic-expression)
-        (imenu--generic-function imenu-generic-expression)) 
+        (imenu--generic-function imenu-generic-expression))
        (t
-        (error "The mode \"%s\" does not take full advantage of imenu.el yet."
-               mode-name))))      
+        (error "This buffer cannot use `imenu-default-create-index-function'"))))
 
 (defun imenu--replace-spaces (name replacement)
   ;; Replace all spaces in NAME with REPLACEMENT.
@@ -581,38 +707,48 @@ Their results are gathered into an index alist."
    name
    ""))
 
-(defun imenu--flatten-index-alist (index-alist &optional concat-names prefix)
-  ;; Takes a nested INDEX-ALIST and returns a flat index alist.
-  ;; If optional CONCAT-NAMES is non-nil, then a nested index has its
-  ;; name and a space concatenated to the names of the children.
-  ;; Third argument PREFIX is for internal use only.
-  (mapcan
-   (function
-    (lambda (item)
-      (let* ((name (car item))
-            (pos (cdr item))
-            (new-prefix (and concat-names
-                             (if prefix
-                                 (concat prefix imenu-level-separator name)
-                               name))))
-       (cond
-        ((or (markerp pos) (numberp pos))
-         (list (cons new-prefix pos)))
-        (t
-         (imenu--flatten-index-alist pos new-prefix))))))
-   index-alist))
+;; Not used and would require cl at run time
+;;; (defun imenu--flatten-index-alist (index-alist &optional concat-names prefix)
+;;;   ;; Takes a nested INDEX-ALIST and returns a flat index alist.
+;;;   ;; If optional CONCAT-NAMES is non-nil, then a nested index has its
+;;;   ;; name and a space concatenated to the names of the children.
+;;;   ;; Third argument PREFIX is for internal use only.
+;;;   (mapcan
+;;;    (lambda (item)
+;;;      (let* ((name (car item))
+;;;        (pos (cdr item))
+;;;        (new-prefix (and concat-names
+;;;                         (if prefix
+;;;                             (concat prefix imenu-level-separator name)
+;;;                           name))))
+;;;        (cond
+;;;    ((or (markerp pos) (numberp pos))
+;;;     (list (cons new-prefix pos)))
+;;;    (t
+;;;     (imenu--flatten-index-alist pos new-prefix)))))
+;;;    index-alist))
 
 ;;;
 ;;; Generic index gathering function.
 ;;;
 
+(defvar imenu-case-fold-search t
+  "Defines whether `imenu--generic-function' should fold case when matching.
+
+This buffer-local variable should be set (only) by initialization code
+for modes which use `imenu--generic-function'.  If it is not set, that
+function will use the current value of `case-fold-search' to match
+patterns.")
+;;;###autoload
+(make-variable-buffer-local 'imenu-case-fold-search)
+
+;; Originally "Built on some ideas that Erik Naggum <erik@naggum.no>
+;; once posted to comp.emacs" but since substantially re-written.
 (defun imenu--generic-function (patterns)
-;; Built on some ideas that Erik Naggum <erik@naggum.no> once posted
-;; to comp.emacs
   "Return an index of the current buffer as an alist.
 
-PATTERN is an alist with elements that look like this: (MENU-TITLE
-REGEXP INDEX).
+PATTERNS is an alist with elements that look like this:
+ (MENU-TITLE REGEXP INDEX).
 
 MENU-TITLE is a string used as the title for the submenu or nil if the
 entries are not nested.
@@ -625,65 +761,80 @@ menu.  See the info section on Regexps for more information.
 INDEX points to the substring in REGEXP that contains the name (of the
 function, variable or type) that is to appear in the menu.
 
-For emacs-lisp-mode for example PATTERN would look like:
-
-'((nil \"^\\\\s-*(def\\\\(un\\\\|subst\\\\|macro\\\\|advice\\\\)\\\\s-+\\\\([-A-Za-z0-9]+\\\\)\" 2)
-  (\"*Vars*\" \"^\\\\s-*(def\\\\(var\\\\|const\\\\)\\\\s-+\\\\([-A-Za-z0-9]+\\\\)\" 2)
-  (\"*Types*\" \"^\\\\s-*(def\\\\(type\\\\|struct\\\\|class\\\\|ine-condition\\\\)\\\\s-+\\\\([-A-Za-z0-9]+\\\\)\" 2))'
+See `lisp-imenu-generic-expression' for an example of PATTERNS.
 
 Returns an index of the current buffer as an alist.  The elements in
 the alist look like: (INDEX-NAME . INDEX-POSITION).  They may also be
 nested index lists like (INDEX-NAME . INDEX-ALIST) depending on
-pattern.
-
-\(imenu--generic-function PATTERN\)."
+PATTERNS."
 
   (let ((index-alist (list 'dummy))
-        (found nil)
-       (global-regexp 
-        (concat "\\(" 
-                (mapconcat
-                 (function (lambda (pattern) (identity (cadr pattern)))) 
-                 patterns "\\)\\|\\(") 
-                "\\)"))
-       prev-pos)
-
+       prev-pos beg
+        (case-fold-search imenu-case-fold-search)
+        (old-table (syntax-table))
+        (table (copy-syntax-table (syntax-table)))
+        (slist imenu-syntax-alist))
+    ;; Modify the syntax table used while matching regexps.
+    (while slist
+      ;; The character(s) to modify may be a single char or a string.
+      (if (numberp (caar slist))
+         (modify-syntax-entry (caar slist) (cdar slist) table)
+       (mapcar (function
+                (lambda (c)
+                  (modify-syntax-entry c (cdar slist) table)))
+               (caar slist)))
+      (setq slist (cdr slist)))
     (goto-char (point-max))
     (imenu-progress-message prev-pos 0 t)
-    (save-match-data
-      (while (re-search-backward global-regexp nil t)
-       (imenu-progress-message prev-pos nil t)
-        (setq found nil)
-       (save-excursion
-         (goto-char (match-beginning 0))
-         (mapcar 
-          (function 
-           (lambda (pat) 
+    (unwind-protect                    ; for syntax table
+       (save-match-data
+         (set-syntax-table table)
+         ;; map over the elements of imenu-generic-expression
+         ;; (typically functions, variables ...)
+         (mapcar
+          (function
+           (lambda (pat)
              (let ((menu-title (car pat))
-                   (regexp (cadr pat))
-                   (index (caddr pat))
-                   (function (cadddr pat))
-                   (rest (cddddr pat)))
-               (if (and (not found) ; Only allow one entry;
-                        (looking-at regexp))
-                   (let ((beg (make-marker))
-                         (end (match-end index)))
-                     (set-marker beg (match-beginning index))
-                     (setq found t)
-                     (push 
-                      (let ((name
-                             (buffer-substring-no-properties beg end)))
+                   (regexp (nth 1 pat))
+                   (index (nth 2 pat))
+                   (function (nth 3 pat))
+                   (rest (nthcdr 4 pat)))
+               ;; Go backwards for convenience of adding items in order.
+               (goto-char (point-max))
+               (while (re-search-backward regexp nil t)
+                 (imenu-progress-message prev-pos nil t)
+                 (setq beg (match-beginning index))
+                 ;; Add this sort of submenu only when we've found an
+                 ;; item for it, avoiding empty, duff menus.
+                 (unless (assoc menu-title index-alist)
+                   (push (list menu-title) index-alist))
+                 (if imenu-use-markers
+                     (setq beg (set-marker (make-marker) beg)))
+                 (let ((item
                         (if function
-                            (nconc (list name function name beg)
+                            (nconc (list (match-string-no-properties index)
+                                         beg function)
                                    rest)
-                          (cons name beg)))
-                      (cdr 
-                       (or (assoc menu-title index-alist)
-                           (car (push 
-                                 (cons menu-title '()) 
-                                 index-alist))))))))))
-          patterns))))
+                          (cons (match-string-no-properties index)
+                                beg)))
+                       ;; This is the desired submenu,
+                       ;; starting with its title (or nil).
+                       (menu (assoc menu-title index-alist)))
+                   ;; Insert the item unless it is already present.
+                   (unless (member item (cdr menu))
+                     (setcdr menu
+                             (cons item (cdr menu)))))))))
+          patterns)
+         (set-syntax-table old-table)))
     (imenu-progress-message prev-pos 100 t)
+    ;; Sort each submenu by position.
+    ;; This is in case one submenu gets items from two different regexps.
+    (let ((tail index-alist))
+      (while tail
+       (if (listp (car tail))
+           (setcdr (car tail)
+                   (sort (cdr (car tail)) 'imenu--sort-by-position)))
+       (setq tail (cdr tail))))
     (let ((main-element (assq nil index-alist)))
       (nconc (delq main-element (delq 'dummy index-alist))
             (cdr main-element)))))
@@ -699,33 +850,39 @@ pattern.
 
 Returns t for rescan and otherwise a position number."
   ;; Create a list for this buffer only when needed.
-  (let (name choice
-            (prepared-index-alist
-             (mapcar
-              (function
-               (lambda (item)
-                 (cons (imenu--replace-spaces (car item) imenu-space-replacement)
-                       (cdr item))))
-              index-alist)))
+  (let ((name (thing-at-point 'symbol))
+       choice
+       (prepared-index-alist
+        (mapcar
+         (function
+          (lambda (item)
+            (cons (imenu--replace-spaces (car item) imenu-space-replacement)
+                  (cdr item))))
+         index-alist)))
+    (cond (prompt)
+         ((and name (imenu--in-alist name prepared-index-alist))
+          (setq prompt (format "Index item (default %s): " name)))
+         (t (setq prompt "Index item: ")))
     (if (eq imenu-always-use-completion-buffer-p 'never)
-       (setq name (completing-read (or prompt "Index item: ")
+       (setq name (completing-read prompt
                                    prepared-index-alist
-                                   nil t nil 'imenu--history-list))
+                                   nil t nil 'imenu--history-list name))
       (save-window-excursion
        ;; Display the completion buffer
        (with-output-to-temp-buffer "*Completions*"
          (display-completion-list
           (all-completions "" prepared-index-alist )))
        (let ((minibuffer-setup-hook
-              (function (lambda ()
-                          (let ((buffer (current-buffer)))
-                            (save-excursion
-                              (set-buffer "*Completions*")
-                              (setq completion-reference-buffer buffer)))))))
+              (function
+               (lambda ()
+                 (let ((buffer (current-buffer)))
+                   (save-excursion
+                     (set-buffer "*Completions*")
+                     (setq completion-reference-buffer buffer)))))))
          ;; Make a completion question
-         (setq name (completing-read (or prompt "Index item: ")
+         (setq name (completing-read prompt
                                      prepared-index-alist
-                                     nil t nil 'imenu--history-list)))))
+                                     nil t nil 'imenu--history-list name)))))
     (cond ((not (stringp name))
           nil)
          ((string= name (car imenu--rescan-item))
@@ -744,9 +901,9 @@ INDEX-ALIST is the buffer index and EVENT is a mouse event.
 Returns t for rescan and otherwise an element or subelement of INDEX-ALIST."
   (setq index-alist (imenu--split-submenus index-alist))
   (let* ((menu         (imenu--split-menu index-alist
-                (or title (buffer-name))))
-        position)
-    (setq menu (imenu--create-keymap-1 (car menu) 
+                                  (or title (buffer-name))))
+       position)
+    (setq menu (imenu--create-keymap-1 (car menu)
                                       (if (< 1 (length (cdr menu)))
                                           (cdr menu)
                                         (cdr (car (cdr menu))))))
@@ -763,7 +920,7 @@ Returns t for rescan and otherwise an element or subelement of INDEX-ALIST."
               (setq final (assoc (car position) final))
               (setq position (cdr position)))
              (or (string= (car final) (car imenu--rescan-item))
-                 (cdr (cdr (cdr final))))))
+                 (nthcdr 3 final))))
          ;; If x-popup-menu went just one level and found a leaf item,
          ;; return the INDEX-ALIST element for that.
          ((and (consp position)
@@ -798,7 +955,7 @@ If `imenu-always-use-completion-buffer-p' is non-nil, then the
 completion buffer is always used, no matter if the mouse was used or
 not.
 
-The returned value is on the form (INDEX-NAME . INDEX-POSITION)."
+The returned value is of the form (INDEX-NAME . INDEX-POSITION)."
   (let (index-alist
        (mouse-triggered (listp last-nonmenu-event))
        (result t) )
@@ -823,19 +980,33 @@ The returned value is on the form (INDEX-NAME . INDEX-POSITION)."
 
 ;;;###autoload
 (defun imenu-add-to-menubar (name)
-  "Adds an `imenu' entry to the menu bar for the current buffer.
+  "Add an `imenu' entry to the menu bar for the current buffer.
 NAME is a string used to name the menu bar item.
 See the command `imenu' for more information."
   (interactive "sImenu menu item name: ")
-  (let ((newmap (make-sparse-keymap))
-       (menu-bar (lookup-key (current-local-map) [menu-bar])))
-    (define-key newmap [menu-bar]
-      (append (make-sparse-keymap) menu-bar))
-    (define-key newmap [menu-bar index]
-      (cons name (nconc (make-sparse-keymap "Imenu")
-                       (make-sparse-keymap))))
-    (use-local-map (append newmap (current-local-map))))
-  (add-hook 'menu-bar-update-hook 'imenu-update-menubar))
+  (if (or (and imenu-prev-index-position-function
+              imenu-extract-index-name-function)
+         imenu-generic-expression
+         (not (eq imenu-create-index-function
+                  'imenu-default-create-index-function)))
+      (let ((newmap (make-sparse-keymap))
+           (menu-bar (lookup-key (current-local-map) [menu-bar])))
+       (define-key newmap [menu-bar]
+         (append (make-sparse-keymap) menu-bar))
+       (define-key newmap [menu-bar index]
+         (cons name (nconc (make-sparse-keymap "Imenu")
+                           (make-sparse-keymap))))
+       (use-local-map (append newmap (current-local-map)))
+       (add-hook 'menu-bar-update-hook 'imenu-update-menubar))
+    (error "The mode `%s' does not support Imenu" mode-name)))
+
+;;;###autoload
+(defun imenu-add-menubar-index ()
+  "Add an Imenu \"Index\" entry on the menu bar for the current buffer.
+
+A trivial interface to `imenu-add-to-menubar' suitable for use in a hook."
+  (interactive)
+  (imenu-add-to-menubar "Index"))
 
 (defvar imenu-buffer-menubar nil)
 
@@ -850,56 +1021,56 @@ See the command `imenu' for more information."
               (setq imenu--last-menubar-index-alist index-alist)
               (setq index-alist (imenu--split-submenus index-alist))
               (setq menu (imenu--split-menu index-alist
-                          (buffer-name)))
+                                             (buffer-name)))
               (setq menu1 (imenu--create-keymap-1 (car menu) 
-                                                  (if (< 1 (length (cdr menu)))
-                                                      (cdr menu)
+                                                   (if (< 1 (length (cdr menu)))
+                                                       (cdr menu)
                                                     (cdr (car (cdr menu))))
-                                                  t))
+                                                   t))
               (setq old (lookup-key (current-local-map) [menu-bar index]))
               (setcdr old (cdr menu1)))))))
 
 (defun imenu--menubar-select (item)
   "Use Imenu to select the function or variable named in this menu item."
-  (if (equal item '("*Rescan*" . -99))
+  (if (equal item imenu--rescan-item)
       (progn
        (imenu--cleanup)
        (setq imenu--index-alist nil)
        (imenu-update-menubar))
     (imenu item)))
 
+(defun imenu-default-goto-function (name position &optional rest)
+  "Move the point to the given position.
+
+NAME is ignored.  POSITION is where to move.  REST is also ignored.
+The ignored args just make this function have the same interface as a
+function placed in a special index-item."
+  (if (or (< position (point-min))
+         (> position (point-max)))
+      ;; widen if outside narrowing
+      (widen))
+  (goto-char position))
+
 ;;;###autoload
 (defun imenu (index-item)
   "Jump to a place in the buffer chosen using a buffer menu or mouse menu.
-See `imenu-choose-buffer-index' for more information."
-  (interactive
-   (list (save-restriction 
-          (widen)
-          (imenu-choose-buffer-index))))
+INDEX-ITEM specifies the position.  See `imenu-choose-buffer-index'
+for more information."
+  (interactive (list (imenu-choose-buffer-index)))
   ;; Convert a string to an alist element.
   (if (stringp index-item)
       (setq index-item (assoc index-item (imenu--make-index-alist))))
   (and index-item
        (progn
         (push-mark)
-        (cond
-         ((markerp (cdr index-item))
-          (if (or (< (marker-position (cdr index-item)) (point-min))
-                  (> (marker-position (cdr index-item)) (point-max)))
-              ;; widen if outside narrowing
-              (widen))
-          (goto-char (marker-position (cdr index-item))))
-         ((imenu--subalist-p index-item)
-          (if (or (< (cdr index-item) (point-min))
-                  (> (cdr index-item) (point-max)))
-              ;; widen if outside narrowing
-              (widen))
-          (goto-char (cdr index-item)))
-         (t
-          ;; A special item with a function.
-          (let ((function (cadr index-item))
-                (rest (cddr index-item)))
-            (apply function (car index-item) rest)))))))
+        (let* ((is-special-item (listp (cdr index-item)))
+               (function
+                (if is-special-item
+                    (nth 2 index-item) imenu-default-goto-function))
+              (position (if is-special-item
+                            (cadr index-item) (cdr index-item)))
+              (rest (if is-special-item (cddr index-item))))
+          (apply function (car index-item) position rest)))))
 
 (provide 'imenu)