]> code.delx.au - gnu-emacs/blobdiff - lisp/progmodes/idlwave.el
Update AUCTeX
[gnu-emacs] / lisp / progmodes / idlwave.el
index 7cefcf195085a68da97fea72c2821e0ac82038f6..a49f70aa0b05d836344a8ffe4fdd2103a02e242d 100644 (file)
@@ -1,11 +1,10 @@
-;;; idlwave.el --- IDL and WAVE CL editing mode for GNU Emacs
+;;; idlwave.el --- IDL editing mode for GNU Emacs
 ;; Copyright (c) 1999, 2000, 2001,2002 Free Software Foundation
 
 ;; Author: Carsten Dominik <dominik@astro.uva.nl>
 ;;         Chris Chase <chase@att.com>
 ;; Maintainer: J.D. Smith <jdsmith@as.arizona.edu>
-;; Version: 4.14
-;; Date: $Date: 2002/08/30 11:02:31 $
+;; Version: 4.15
 ;; Keywords: languages
 
 ;; This file is part of GNU Emacs.
@@ -57,7 +56,7 @@
 ;; SOURCE
 ;; ======
 ;;
-;; The newest version of this file is available from the maintainers
+;; The newest version of this file is available from the maintainer's
 ;; Webpage.
 ;;
 ;;   http://idlwave.org
@@ -69,7 +68,7 @@
 ;; A printable version of the documentation is available from the
 ;; maintainers webpage (see under SOURCE)
 ;;
-;; 
+;;
 ;; ACKNOWLEDGMENTS
 ;; ===============
 ;;
 ;;    Simon Marshall <Simon.Marshall@esrin.esa.it>
 ;;    Laurent Mugnier <mugnier@onera.fr>
 ;;    Lubos Pochman <lubos@rsinc.com>
+;;    Bob Portmann <portmann@al.noaa.gov>
 ;;    Patrick M. Ryan <pat@jaameri.gsfc.nasa.gov>
 ;;    Marty Ryba <ryba@ll.mit.edu>
 ;;    Phil Williams <williams@irc.chmcc.org>
-;;    J.D. Smith <jdsmith@astrosun.tn.cornell.edu>
 ;;    Phil Sterne <sterne@dublin.llnl.gov>
 ;;
 ;; CUSTOMIZATION:
 ;; =============
 ;;
-;; IDLWAVE has customize support - so if you want to learn about the
-;;  variables which control the behavior of the mode, use
+;; IDLWAVE has extensive customize support - so if you want to learn
+;; about the variables which control the behavior of the mode, use
 ;; `M-x idlwave-customize'.
 ;;
 ;; You can set your own preferred values with Customize, or with Lisp
 ;; code in .emacs.  For an example of what to put into .emacs, check
-;; the TexInfo documentation.
+;; the TexInfo documentation or see a complete .emacs at
+;; http://idlwave.org.
 ;;
 ;; KNOWN PROBLEMS:
 ;; ==============
 ;;
+;;   IDLWAVE support for the IDL-derived PV-WAVE CL language of Visual
+;;   Numerics, Inc. is growing less and less complete as the two
+;;   languages grow increasingly apart.  The mode probably shouldn't
+;;   even have "WAVE" in it's title, but it's catchy, and required to
+;;   avoid conflict with the CORBA idl.el mode.  Caveat WAVEor.
+;;
 ;;   Moving the point backwards in conjunction with abbrev expansion
 ;;   does not work as I would like it, but this is a problem with
 ;;   emacs abbrev expansion done by the self-insert-command.  It ends
 ;;   up inserting the character that expanded the abbrev after moving
 ;;   point backward, e.g., "\cl" expanded with a space becomes
 ;;   "LONG( )" with point before the close paren.  This is solved by
-;;   using a temporary function in `post-command-hook' - not pretty, 
+;;   using a temporary function in `post-command-hook' - not pretty,
 ;;   but it works.
 ;;
 ;;   Tabs and spaces are treated equally as whitespace when filling a
 ;;   limited to comments only and occurs only when a comment
 ;;   paragraph is filled via `idlwave-fill-paragraph'.
 ;;
-;;   "&" is ignored when parsing statements.
 ;;   Avoid muti-statement lines (using "&") on block begin and end
 ;;   lines.  Multi-statement lines can mess up the formatting, for
 ;;   example, multiple end statements on a line: endif & endif.
 ;;   Using "&" outside of block begin/end lines should be okay.
 ;;
-;;   It is possible that the parser which decides what to complete has
-;;   problems with pointer dereferencing statements.  I don't use
-;;   pointers often enough to find out - please report any problems.
+;;   Determining the expression at point for printing and other
+;;   examination commands is somewhat rough: currently only fairly
+;;   simple entities are found.  You can always drag-select or examine
+;;   a region.
 ;;
 ;;   When forcing completion of method keywords, the initial
 ;;   query for a method has multiple entries for some methods.  Would
       nil ;; We've got what we needed
     ;; We have the old or no custom-library, hack around it!
     (defmacro defgroup (&rest args) nil)
-    (defmacro defcustom (var value doc &rest args) 
+    (defmacro defcustom (var value doc &rest args)
       `(defvar ,var ,value ,doc))))
 
 (defgroup idlwave nil
   "Major mode for editing IDL .pro files"
   :tag "IDLWAVE"
-  :link '(url-link :tag "Home Page" 
+  :link '(url-link :tag "Home Page"
                   "http://idlwave.org")
   :link '(emacs-commentary-link :tag "Commentary in idlw-shell.el"
                                "idlw-shell.el")
@@ -258,8 +264,8 @@ extends to the end of the match for the regular expression."
 
 (defcustom idlwave-auto-fill-split-string t
   "*If non-nil then auto fill will split strings with the IDL `+' operator.
-When the line end falls within a string, string concatenation with the 
-'+' operator will be used to distribute a long string over lines.  
+When the line end falls within a string, string concatenation with the
+'+' operator will be used to distribute a long string over lines.
 If nil and a string is split then a terminal beep and warning are issued.
 
 This variable is ignored when `idlwave-fill-comment-line-only' is
@@ -315,7 +321,7 @@ the comment is not preceded by whitespace it is unchanged."
 Initializing the routine info can take long, in particular if a large
 library catalog is involved.  When Emacs is idle for more than the number
 of seconds specified by this variable, it starts the initialization.
-The process is split into five steps, in order to keep possible work 
+The process is split into five steps, in order to keep possible work
 interruption as short as possible.  If one of the steps finishes, and no
 user input has arrived in the mean time, initialization proceeds immediately
 to the next step.
@@ -353,7 +359,7 @@ scan, this is not necessary."
 Possible values:
 nil       Never
 t         All available
-(...)     A list of circumstances. Allowed members are:
+\(...)     A list of circumstances. Allowed members are:
            find-file       Add info for new IDLWAVE buffers.
            save-buffer     Update buffer info when buffer is saved
            kill-buffer     Remove buffer info when buffer gets killed
@@ -367,7 +373,7 @@ t         All available
               (const :tag "When saving a buffer" save-buffer)
               (const :tag "After a buffer was killed" kill-buffer)
               (const :tag "After a buffer was compiled successfully, update shell info" compile-buffer))))
-              
+
 (defcustom idlwave-rinfo-max-source-lines 5
   "*Maximum number of source files displayed in the Routine Info window.
 When an integer, it is the maximum number of source files displayed.
@@ -412,7 +418,7 @@ will be used to store the result."
 (defcustom idlwave-special-lib-alist nil
   "Alist of regular expressions matching special library directories.
 When listing routine source locations, IDLWAVE gives a short hint where
-the file defining the routine is located.  By default it lists `SystemLib' 
+the file defining the routine is located.  By default it lists `SystemLib'
 for routines in the system library `!DIR/lib' and `Library' for anything
 else.  This variable can define additional types.  The car of each entry
 is a regular expression matching the file name (they normally will match
@@ -506,7 +512,7 @@ This just activates all words which are also a help topic - some links may
 be misleading."
   :group 'idlwave-online-help
   :type 'boolean)
-  
+
 
 (defgroup idlwave-completion nil
   "Completion options for IDLWAVE mode."
@@ -531,7 +537,7 @@ be misleading."
 This variable determines the case (UPPER/lower/Capitalized...) of
 words inserted into the buffer by completion.  The preferred case can
 be specified separately for routine names, keywords, classes and
-methods. 
+methods.
 This alist should therefore have entries for `routine' (normal
 functions and procedures, i.e. non-methods), `keyword', `class', and
 `method'.  Plausible values are
@@ -618,7 +624,7 @@ certain methods this assumption is almost always true.  The methods
 for which to assume this can be set here."
   :group 'idlwave-routine-info
   :type '(repeat (regexp :tag "Match method:")))
-  
+
 
 (defcustom idlwave-completion-show-classes 1
   "*Number of classes to show when completing object methods and keywords.
@@ -653,17 +659,18 @@ class info listed.  See `idlwave-completion-show-classes'."
 When IDLWAVE tries to complete object-oriented methods, it usually
 cannot determine the class of a given object from context.  In order
 to provide the user with a correct list of methods or keywords, it
-needs have to determine the appropriate class.  IDLWAVE has two ways
-to deal with this problem.
-
-1. One possibility is to combine the items of all available classes
-   which contain this method for the purpose of completion.  So when
-   completing a method, all methods of all known classes are
-   available, and when completing a keyword, all keywords allowed for
-   this method in any class are shown.  This behavior is very much
-   like normal completion and is therefore the default.  It works much
-   better than one might think - only for the INIT, GETPROPERTY and
-   SETPROPERTY the keyword lists become uncomfortably long.  See also
+needs to determine the appropriate class.  IDLWAVE has two ways of
+doing this (well, three ways if you count the shell... see
+`idlwave-shell-query-for-class'):
+
+1. Combine the items of all available classes which contain this
+   method for the purpose of completion.  So when completing a method,
+   all methods of all known classes are available, and when completing
+   a keyword, all keywords allowed for this method in any class are
+   shown.  This behavior is very much like normal completion and is
+   therefore the default.  It works much better than one might think -
+   only for the INIT, GETPROPERTY and SETPROPERTY the keyword lists
+   become uncomfortably long.  See also
    `idlwave-completion-show-classes'.
 
 2. The second possibility is to ask the user on each occasion.  To
@@ -678,11 +685,11 @@ to deal with this problem.
 This variable allows you to configure IDLWAVE's method and
 method-keyword completion behavior.  Its value is an alist, which
 should contain at least two elements: (method-default . VALUE) and
-(keyword-default . VALUE), where VALUE is either t or nil.  These
+\(keyword-default . VALUE), where VALUE is either t or nil.  These
 specify if the class should be found during method and keyword
 completion, respectively.
 
-The alist may have additional entries specifying exceptions from the 
+The alist may have additional entries specifying exceptions from the
 keyword completion rule for specific methods, like INIT or
 GETPROPERTY.  In order to turn on class specification for the INIT
 method, add an entry (\"INIT\" . t).  The method name must be ALL-CAPS."
@@ -706,7 +713,7 @@ particular object method call.  This happens during the commands
 value of the variable `idlwave-query-class'.
 
 When you specify a class, this information can be stored as a text
-property on the `->' arrow in the source code, so that during the same 
+property on the `->' arrow in the source code, so that during the same
 editing session, IDLWAVE will not have to ask again.  When this
 variable is non-nil, IDLWAVE will store and reuse the class information.
 The class stored can be checked and removed with `\\[idlwave-routine-info]'
@@ -726,7 +733,7 @@ at point."
 (defcustom idlwave-class-arrow-face 'bold
   "*Face to highlight object operator arrows `->' which carry a class property.
 When IDLWAVE stores a class name as text property on an object arrow
-(see variable `idlwave-store-inquired-class', it highlights the arrow
+\(see variable `idlwave-store-inquired-class', it highlights the arrow
 with this font in order to remind the user that this arrow is special."
   :group 'idlwave-completion
   :type 'symbol)
@@ -1086,8 +1093,8 @@ IDL process is made."
   :group 'idlwave-misc
   :type 'boolean)
 
-(defcustom idlwave-default-font-lock-items 
-  '(pros-and-functions batch-files idl-keywords label goto
+(defcustom idlwave-default-font-lock-items
+  '(pros-and-functions batch-files idlwave-idl-keywords label goto
                       common-blocks class-arrows)
   "Items which should be fontified on the default fontification level 2.
 IDLWAVE defines 3 levels of fontification.  Level 1 is very little, level 3
@@ -1097,7 +1104,7 @@ a list of symbols, the following symbols are allowed.
 
 pros-and-functions   Procedure and Function definitions
 batch-files          Batch Files
-idl-keywords         IDL Keywords
+idlwave-idl-keywords IDL Keywords
 label                Statement Labels
 goto                 Goto Statements
 common-blocks        Common Blocks
@@ -1109,16 +1116,16 @@ class-arrows         Object Arrows with class property"
   :type '(set
          :inline t :greedy t
          (const :tag "Procedure and Function definitions" pros-and-functions)
-         (const :tag "Batch Files"                        batch-files)
-         (const :tag "IDL Keywords (reserved words)"      idl-keywords)
-         (const :tag "Statement Labels"                   label)
-         (const :tag "Goto Statements"                    goto)
-         (const :tag "Tags in Structure Definition"       structtag)
-         (const :tag "Structure Name"                     structname)
-         (const :tag "Common Blocks"                      common-blocks)
-         (const :tag "Keyword Parameters"                 keyword-parameters)
-         (const :tag "System Variables"                   system-variables)
-         (const :tag "FIXME: Warning"                     fixme)
+         (const :tag "Batch Files"                       batch-files)
+         (const :tag "IDL Keywords (reserved words)"     idlwave-idl-keywords)
+         (const :tag "Statement Labels"                  label)
+         (const :tag "Goto Statements"                   goto)
+         (const :tag "Tags in Structure Definition"      structtag)
+         (const :tag "Structure Name"                    structname)
+         (const :tag "Common Blocks"                     common-blocks)
+         (const :tag "Keyword Parameters"                keyword-parameters)
+         (const :tag "System Variables"                  system-variables)
+         (const :tag "FIXME: Warning"                    fixme)
          (const :tag "Object Arrows with class property " class-arrows)))
 
 (defcustom idlwave-mode-hook nil
@@ -1147,48 +1154,34 @@ As a user, you should not set this to t.")
 ;;; Simon Marshall <simon@gnu.ai.mit.edu>
 ;;; and Carsten Dominik...
 
-(defconst idlwave-font-lock-keywords-1 nil
-  "Subdued level highlighting for IDLWAVE mode.")
-
-(defconst idlwave-font-lock-keywords-2 nil
-  "Medium level highlighting for IDLWAVE mode.")
-
-(defconst idlwave-font-lock-keywords-3 nil
-  "Gaudy level highlighting for IDLWAVE mode.")
-
-(let* ((oldp (or (string-match "Lucid" emacs-version)
-                (not (boundp 'emacs-minor-version))
-                (and (<= emacs-major-version 19) 
-                     (<= emacs-minor-version 29))))
-
-       ;; The following are the reserved words in IDL.  Maybe we should
-       ;; highlight some more stuff as well?       
-       (idl-keywords
-       ;; To update this regexp, update the list of keywords and 
-       ;; evaluate the form.
-;      (insert 
-;       (prin1-to-string
-;        (concat 
-;         "\\<\\("
-;         (regexp-opt 
-;          '("and" "or" "xor" "not"
-;            "eq" "ge" "gt" "le" "lt" "ne" 
-;            "for" "do" "endfor"
-;            "if" "then" "endif" "else" "endelse" 
-;            "case" "of" "endcase"
-;            "switch" "break" "continue" "endswitch"
-;            "begin" "end"
-;            "repeat" "until" "endrep"
-;            "while" "endwhile" 
-;            "goto" "return"
-;            "inherits" "mod"
-;            "compile_opt" "forward_function"
-;            "on_error" "on_ioerror"))  ; on_error is not officially reserved
-;         "\\)\\>")))
-
-       "\\<\\(and\\|b\\(egin\\|reak\\)\\|c\\(ase\\|o\\(mpile_opt\\|ntinue\\)\\)\\|do\\|e\\(lse\\|nd\\(case\\|else\\|for\\|if\\|rep\\|switch\\|while\\)?\\|q\\)\\|for\\(ward_function\\)?\\|g\\(oto\\|[et]\\)\\|i\\(f\\|nherits\\)\\|l[et]\\|mod\\|n\\(e\\|ot\\)\\|o\\(n_\\(error\\|ioerror\\)\\|[fr]\\)\\|re\\(peat\\|turn\\)\\|switch\\|then\\|until\\|while\\|xor\\)\\>")
-
-       ;; Procedure declarations.  Fontify keyword plus procedure name.
+;; The following are the reserved words in IDL.  Maybe we should
+;; highlight some more stuff as well?
+;; Procedure declarations.  Fontify keyword plus procedure name.
+(defvar idlwave-idl-keywords
+  ;; To update this regexp, update the list of keywords and
+  ;; evaluate the form.
+  ;;   (insert
+  ;;    (prin1-to-string
+  ;;     (concat
+  ;;      "\\<\\("
+  ;;      (regexp-opt
+  ;;       '("and" "or" "xor" "not"
+  ;;         "eq" "ge" "gt" "le" "lt" "ne"
+  ;;         "for" "do" "endfor"
+  ;;         "if" "then" "endif" "else" "endelse"
+  ;;         "case" "of" "endcase"
+  ;;         "switch" "break" "continue" "endswitch"
+  ;;         "begin" "end"
+  ;;         "repeat" "until" "endrep"
+  ;;         "while" "endwhile"
+  ;;         "goto" "return"
+  ;;         "inherits" "mod"
+  ;;         "compile_opt" "forward_function"
+  ;;         "on_error" "on_ioerror"))  ; on_error is not officially reserved
+  ;;      "\\)\\>")))
+  "\\<\\(and\\|b\\(egin\\|reak\\)\\|c\\(ase\\|o\\(mpile_opt\\|ntinue\\)\\)\\|do\\|e\\(lse\\|nd\\(case\\|else\\|for\\|if\\|rep\\|switch\\|while\\)?\\|q\\)\\|for\\(ward_function\\)?\\|g\\(oto\\|[et]\\)\\|i\\(f\\|nherits\\)\\|l[et]\\|mod\\|n\\(e\\|ot\\)\\|o\\(n_\\(error\\|ioerror\\)\\|[fr]\\)\\|re\\(peat\\|turn\\)\\|switch\\|then\\|until\\|while\\|xor\\)\\>")
+
+(let* (;; Procedure declarations.  Fontify keyword plus procedure name.
        ;; Function  declarations.  Fontify keyword plus function  name.
        (pros-and-functions
        '("\\<\\(function\\|pro\\)\\>[ \t]+\\(\\sw+\\(::\\sw+\\)?\\)"
@@ -1202,7 +1195,7 @@ As a user, you should not set this to t.")
          (2 font-lock-reference-face nil t)      ; block name
          (font-lock-match-c++-style-declaration-item-and-skip-to-next
           ;; Start with point after block name and comma
-          (goto-char (match-end 0))  ; needed for XEmacs, could be nil 
+          (goto-char (match-end 0))  ; needed for XEmacs, could be nil
           nil
           (1 font-lock-variable-name-face)       ; variable names
           )))
@@ -1255,55 +1248,31 @@ As a user, you should not set this to t.")
        ;; All operators (not used because too noisy)
        (all-operators
        '("[-*^#+<>/]" (0 font-lock-keyword-face)))
-       
+
        ;; Arrows with text property `idlwave-class'
        (class-arrows
-       (list 'idlwave-match-class-arrows 
-             (list 0 (if (featurep 'xemacs) 
-                         idlwave-class-arrow-face
-                       'idlwave-class-arrow-face))))
-
-       )
-
-  ;; The following lines are just a dummy to make the compiler shut up
-  ;; about variables bound but not used.
-  (setq oldp oldp
-       idl-keywords idl-keywords
-       pros-and-functions pros-and-functions
-       common-blocks common-blocks
-       batch-files batch-files
-       fixme fixme
-       label label
-       goto goto
-       structtag structtag
-       structname structname
-       keyword-parameters keyword-parameters
-       system-variables system-variables
-       special-operators special-operators
-       all-operators all-operators
-       class-arrows class-arrows)
-
-  (setq idlwave-font-lock-keywords-1
-       (list pros-and-functions
-             batch-files
-             ))
+       '(idlwave-match-class-arrows (0 idlwave-class-arrow-face))))
 
-  (setq idlwave-font-lock-keywords-2
-       (mapcar 'symbol-value idlwave-default-font-lock-items))
+  (defconst idlwave-font-lock-keywords-1
+    (list pros-and-functions batch-files)
+    "Subdued level highlighting for IDLWAVE mode.")
 
-  (setq idlwave-font-lock-keywords-3 
+  (defconst idlwave-font-lock-keywords-2
+    (mapcar 'symbol-value idlwave-default-font-lock-items)
+    "Medium level highlighting for IDLWAVE mode.")
+
+  (defconst idlwave-font-lock-keywords-3
        (list pros-and-functions
              batch-files
-             idl-keywords
+             idlwave-idl-keywords
              label goto
              structtag
              structname
              common-blocks
              keyword-parameters
              system-variables
-             class-arrows
-             ))
-  )
+         class-arrows)
+    "Gaudy level highlighting for IDLWAVE mode."))
 
 (defun idlwave-match-class-arrows (limit)
   ;; Match an object arrow with class property
@@ -1316,14 +1285,14 @@ As a user, you should not set this to t.")
 
 (defvar idlwave-font-lock-defaults
   '((idlwave-font-lock-keywords
-     idlwave-font-lock-keywords-1 
+     idlwave-font-lock-keywords-1
      idlwave-font-lock-keywords-2
      idlwave-font-lock-keywords-3)
-    nil t 
-    ((?$ . "w") (?_ . "w") (?. . "w")) 
+    nil t
+    ((?$ . "w") (?_ . "w") (?. . "w"))
     beginning-of-line))
 
-(put 'idlwave-mode 'font-lock-defaults 
+(put 'idlwave-mode 'font-lock-defaults
      idlwave-font-lock-defaults) ; XEmacs
 
 (defconst idlwave-comment-line-start-skip "^[ \t]*;"
@@ -1331,7 +1300,7 @@ As a user, you should not set this to t.")
 That is the _beginning_ of a line containing a comment delimiter `;' preceded
 only by whitespace.")
 
-(defconst idlwave-begin-block-reg 
+(defconst idlwave-begin-block-reg
   "\\<\\(pro\\|function\\|begin\\|case\\|switch\\)\\>"
   "Regular expression to find the beginning of a block. The case does
 not matter. The search skips matches in comments.")
@@ -1406,7 +1375,7 @@ blocks starting with a BEGIN statement.  The matches must have associations
    '(switch . ("switch\\>" nil))
    (cons 'call (list (concat idlwave-identifier "\\(\\s *$\\|\\s *,\\)") nil))
    '(assign . ("[^=>\n]*=" nil)))
-  
+
   "Associated list of statement matching regular expressions.
 Each regular expression matches the start of an IDL statement.  The
 first element of each association is a symbol giving the statement
@@ -1429,7 +1398,7 @@ the leftover unidentified statements containing an equal sign."  )
 ;; Note that this is documented in the v18 manuals as being a string
 ;; of length one rather than a single character.
 ;; The code in this file accepts either format for compatibility.
-(defvar idlwave-comment-indent-char ?\ 
+(defvar idlwave-comment-indent-char ?\s
   "Character to be inserted for IDL comment indentation.
 Normally a space.")
 
@@ -1437,7 +1406,7 @@ Normally a space.")
   "Character which is inserted as a last character on previous line by
    \\[idlwave-split-line] to begin a continuation line.  Normally $.")
 
-(defconst idlwave-mode-version " 4.14")
+(defconst idlwave-mode-version " 4.15")
 
 (defmacro idlwave-keyword-abbrev (&rest args)
   "Creates a function for abbrev hooks to call `idlwave-check-abbrev' with args."
@@ -1492,29 +1461,13 @@ Otherwise ARGS forms a list that is evaluated."
   (copy-syntax-table idlwave-mode-syntax-table)
   "Syntax table that treats symbol characters as word characters.")
 
-(modify-syntax-entry ?$   "w"  idlwave-find-symbol-syntax-table)
-(modify-syntax-entry ?_   "w"  idlwave-find-symbol-syntax-table)
-
-(defmacro idlwave-with-special-syntax (&rest body)
-  "Execute BODY with a different systax table."
-  `(let ((saved-syntax (syntax-table)))
-     (unwind-protect
-        (progn
-          (set-syntax-table idlwave-find-symbol-syntax-table)
-          ,@body)
-       (set-syntax-table saved-syntax))))
-
-(defvar idlwave-print-symbol-syntax-table
-  (copy-syntax-table idlwave-mode-syntax-table)
-  "Syntax table that treats symbol characters as word characters.")
-
 (modify-syntax-entry ?$   "w"  idlwave-find-symbol-syntax-table)
 (modify-syntax-entry ?_   "w"  idlwave-find-symbol-syntax-table)
 (modify-syntax-entry ?!   "w"  idlwave-find-symbol-syntax-table)
 (modify-syntax-entry ?.   "w"  idlwave-find-symbol-syntax-table)
 
-(defmacro idlwave-with-special-syntax1 (&rest body)
-  "Execute BODY with a different systax table."
+(defmacro idlwave-with-special-syntax (&rest body)
+  "Execute BODY with a different syntax table."
   `(let ((saved-syntax (syntax-table)))
      (unwind-protect
         (progn
@@ -1522,6 +1475,15 @@ Otherwise ARGS forms a list that is evaluated."
           ,@body)
        (set-syntax-table saved-syntax))))
 
+;(defmacro idlwave-with-special-syntax1 (&rest body)
+;  "Execute BODY with a different syntax table."
+;  `(let ((saved-syntax (syntax-table)))
+;     (unwind-protect
+;        (progn
+;          (set-syntax-table idlwave-find-symbol-syntax-table)
+;          ,@body)
+;      (set-syntax-table saved-syntax))))
+
 (defun idlwave-action-and-binding (key cmd &optional select)
   "KEY and CMD are made into a key binding and an indent action.
 KEY is a string - same as for the `define-key' function.  CMD is a
@@ -1576,6 +1538,7 @@ Capitalize system variables - action only
 ;(define-key idlwave-mode-map "\C-c\C- " 'idlwave-hard-tab)
 (define-key idlwave-mode-map "'"        'idlwave-show-matching-quote)
 (define-key idlwave-mode-map "\""       'idlwave-show-matching-quote)
+(define-key idlwave-mode-map "\C-g"     'idlwave-keyboard-quit)
 (define-key idlwave-mode-map "\C-c;"    'idlwave-toggle-comment-region)
 (define-key idlwave-mode-map "\C-\M-a"  'idlwave-beginning-of-subprogram)
 (define-key idlwave-mode-map "\C-\M-e"  'idlwave-end-of-subprogram)
@@ -1593,7 +1556,7 @@ Capitalize system variables - action only
 (define-key idlwave-mode-map "\C-c\C-n" 'idlwave-next-statement)
 ;; (define-key idlwave-mode-map "\r"       'idlwave-newline)
 ;; (define-key idlwave-mode-map "\t"       'idlwave-indent-line)
-(define-key idlwave-mode-map (kbd "S-<iso-lefttab>") 'idlwave-indent-statement)
+(define-key idlwave-mode-map [(shift tab)] 'idlwave-indent-statement)
 (define-key idlwave-mode-map "\C-c\C-a" 'idlwave-auto-fill-mode)
 (define-key idlwave-mode-map "\M-q"     'idlwave-fill-paragraph)
 (define-key idlwave-mode-map "\M-s"     'idlwave-edit-in-idlde)
@@ -1608,12 +1571,12 @@ Capitalize system variables - action only
         (not (equal idlwave-shell-debug-modifiers '())))
   ;; Bind the debug commands also with the special modifiers.
   (let ((shift (memq 'shift idlwave-shell-debug-modifiers))
-       (mods-noshift (delq 'shift 
+       (mods-noshift (delq 'shift
                            (copy-sequence idlwave-shell-debug-modifiers))))
-    (define-key idlwave-mode-map 
+    (define-key idlwave-mode-map
       (vector (append mods-noshift (list (if shift ?C ?c))))
       'idlwave-shell-save-and-run)
-    (define-key idlwave-mode-map 
+    (define-key idlwave-mode-map
       (vector (append mods-noshift (list (if shift ?B ?b))))
       'idlwave-shell-break-here)))
 (define-key idlwave-mode-map "\C-c\C-f" 'idlwave-for)
@@ -1637,10 +1600,11 @@ Capitalize system variables - action only
 (define-key idlwave-mode-map "\C-c\C-v"   'idlwave-find-module)
 (define-key idlwave-mode-map "\C-c?"      'idlwave-routine-info)
 (define-key idlwave-mode-map "\M-?"       'idlwave-context-help)
+(define-key idlwave-mode-map [(control meta ?\?)] 'idlwave-online-help)
 (define-key idlwave-mode-map [(meta tab)] 'idlwave-complete)
 (define-key idlwave-mode-map "\C-c\C-i"   'idlwave-update-routine-info)
 (define-key idlwave-mode-map "\C-c="      'idlwave-resolve)
-(define-key idlwave-mode-map 
+(define-key idlwave-mode-map
   (if (featurep 'xemacs) [(shift button3)] [(shift mouse-3)])
   'idlwave-mouse-context-help)
 
@@ -1650,7 +1614,7 @@ Capitalize system variables - action only
 (idlwave-action-and-binding "&"  '(idlwave-surround -1 -1))
 (idlwave-action-and-binding "<"  '(idlwave-surround -1 -1))
 ;; Binding works for both > and ->, by changing the length of the token.
-(idlwave-action-and-binding ">"  '(idlwave-surround -1 -1 '(?-) 1 
+(idlwave-action-and-binding ">"  '(idlwave-surround -1 -1 '(?-) 1
                                                    'idlwave-gtr-pad-hook))
 (idlwave-action-and-binding "->" '(idlwave-surround -1 -1 nil 2) t)
 (idlwave-action-and-binding ","  '(idlwave-surround 0 -1))
@@ -1683,7 +1647,7 @@ idlwave-mode-abbrev-table unless TABLE is non-nil."
       (error (apply 'define-abbrev args)))))
 
 (condition-case nil
-    (modify-syntax-entry (string-to-char idlwave-abbrev-start-char) 
+    (modify-syntax-entry (string-to-char idlwave-abbrev-start-char)
                         "w" idlwave-mode-syntax-table)
   (error nil))
 
@@ -1756,7 +1720,7 @@ idlwave-mode-abbrev-table unless TABLE is non-nil."
 (idlwave-define-abbrev "s"  "size()"       (idlwave-keyword-abbrev 1))
 (idlwave-define-abbrev "wi" "widget_info()" (idlwave-keyword-abbrev 1))
 (idlwave-define-abbrev "wc" "widget_control," (idlwave-keyword-abbrev 0))
-  
+
 ;; This section is reserved words only. (From IDL user manual)
 ;;
 (idlwave-define-abbrev "and"        "and"       (idlwave-keyword-abbrev 0 t) t)
@@ -1805,7 +1769,7 @@ idlwave-mode-abbrev-table unless TABLE is non-nil."
 (defvar imenu-extract-index-name-function)
 (defvar imenu-prev-index-position-function)
 ;; defined later - so just make the compiler hush
-(defvar idlwave-mode-menu)  
+(defvar idlwave-mode-menu)
 (defvar idlwave-mode-debug-menu)
 
 ;;;###autoload
@@ -1887,7 +1851,7 @@ The main features of this mode are
    \\i         IF statement template
    \\elif      IF-ELSE statement template
    \\b         BEGIN
-   
+
    For a full list, use \\[idlwave-list-abbrevs].  Some templates also have
    direct keybindings - see the list of keybindings below.
 
@@ -1928,26 +1892,26 @@ The main features of this mode are
 
   (interactive)
   (kill-all-local-variables)
-  
+
   (if idlwave-startup-message
       (message "Emacs IDLWAVE mode version %s." idlwave-mode-version))
   (setq idlwave-startup-message nil)
-  
+
   (setq local-abbrev-table idlwave-mode-abbrev-table)
   (set-syntax-table idlwave-mode-syntax-table)
-  
+
   (set (make-local-variable 'indent-line-function) 'idlwave-indent-and-action)
-  
+
   (make-local-variable idlwave-comment-indent-function)
   (set idlwave-comment-indent-function 'idlwave-comment-hook)
-  
+
   (set (make-local-variable 'comment-start-skip) ";+[ \t]*")
   (set (make-local-variable 'comment-start) ";")
   (set (make-local-variable 'require-final-newline) t)
   (set (make-local-variable 'abbrev-all-caps) t)
   (set (make-local-variable 'indent-tabs-mode) nil)
   (set (make-local-variable 'completion-ignore-case) t)
-  
+
   (use-local-map idlwave-mode-map)
 
   (when (featurep 'easymenu)
@@ -1957,20 +1921,20 @@ The main features of this mode are
   (setq mode-name "IDLWAVE")
   (setq major-mode 'idlwave-mode)
   (setq abbrev-mode t)
-  
+
   (set (make-local-variable idlwave-fill-function) 'idlwave-auto-fill)
   (setq comment-end "")
   (set (make-local-variable 'comment-multi-line) nil)
-  (set (make-local-variable 'paragraph-separate) 
+  (set (make-local-variable 'paragraph-separate)
        "[ \t\f]*$\\|[ \t]*;+[ \t]*$\\|;+[+=-_*]+$")
   (set (make-local-variable 'paragraph-start) "[ \t\f]\\|[ \t]*;+[ \t]")
   (set (make-local-variable 'paragraph-ignore-fill-prefix) nil)
-  (set (make-local-variable 'parse-sexp-ignore-comments) nil)
-  
+  (set (make-local-variable 'parse-sexp-ignore-comments) t)
+
   ;; Set tag table list to use IDLTAGS as file name.
   (if (boundp 'tag-table-alist)
       (add-to-list 'tag-table-alist '("\\.pro$" . "IDLTAGS")))
-  
+
   ;; Font-lock additions - originally Phil Williams, then Ulrik Dickow
   ;; Following line is for Emacs - XEmacs uses the corresponding porperty
   ;; on the `idlwave-mode' symbol.
@@ -2098,7 +2062,7 @@ Returns point if comment found and nil otherwise."
 (defvar zmacs-regions)
 (defvar mark-active)
 (defun idlwave-region-active-p ()
-  "Is transien-mark-mode on an the region active?
+  "Is transien-mark-mode on and the region active?
 Works on both Emacs and XEmacs."
   (if (featurep 'xemacs)
       (and zmacs-regions (region-active-p))
@@ -2134,18 +2098,33 @@ An END token must be preceded by whitespace."
   "Finds the start of current block and blinks to it for a second.
 Also checks if the correct end statement has been used."
   ;; All end statements are reserved words
-  (let* ((pos (point))
-        end end1)
+  ;; Re-indent end line
+  (insert-char ?\  1) ;; So indent, etc. work well
+  (backward-char 1)
+  (let* ((pos (point-marker))
+        (last-abbrev-marker (copy-marker last-abbrev-location))
+        (eol-pos (save-excursion (end-of-line) (point)))
+        begin-pos end-pos end end1 )
+    (if idlwave-reindent-end  (idlwave-indent-line))
+
     (when (and (idlwave-check-abbrev 0 t)
               idlwave-show-block)
       (save-excursion
        ;; Move inside current block
-       (setq end (buffer-substring 
-                  (save-excursion (skip-chars-backward "a-zA-Z")
-                                  (point))
-                  (point)))
-       (idlwave-beginning-of-statement)
+       (goto-char last-abbrev-marker)
        (idlwave-block-jump-out -1 'nomark)
+       (setq begin-pos (point))
+       (idlwave-block-jump-out 1 'nomark)
+       (setq end-pos (point))
+       (if (> end-pos eol-pos)
+           (setq end-pos pos))
+       (goto-char end-pos)
+       (setq end (buffer-substring
+                  (progn
+                    (skip-chars-backward "a-zA-Z")
+                    (point))
+                  end-pos))
+       (goto-char begin-pos)
        (when (setq end1 (cdr (idlwave-block-master)))
          (cond
           ((null end1)) ; no-operation
@@ -2162,12 +2141,10 @@ Also checks if the correct end statement has been used."
            (sit-for 1))
           (t
            (beep)
-           (message "Warning: Shouldn't this be \"%s\" instead of \"%s\"?" 
+           (message "Warning: Shouldn't this be \"%s\" instead of \"%s\"?"
                     end1 end)
            (sit-for 1)))))))
-  ;; Re-indent end line
-  (if idlwave-reindent-end
-      (idlwave-indent-line)))
+  (delete-char 1))
 
 (defun idlwave-block-master ()
   (let ((case-fold-search t))
@@ -2176,8 +2153,8 @@ Also checks if the correct end statement has been used."
        ((looking-at "pro\\|case\\|switch\\|function\\>")
        (assoc (downcase (match-string 0)) idlwave-block-matches))
        ((looking-at "begin\\>")
-       (let ((limit (save-excursion 
-                      (idlwave-beginning-of-statement) 
+       (let ((limit (save-excursion
+                      (idlwave-beginning-of-statement)
                       (point))))
          (cond
           ((re-search-backward idlwave-block-match-regexp limit t)
@@ -2205,9 +2182,9 @@ Also checks if the correct end statement has been used."
     (insert "end")
     (idlwave-show-begin)))
 
-(defun idlwave-gtr-pad-hook (char) 
+(defun idlwave-gtr-pad-hook (char)
   "Let the > symbol expand around -> if present.  The new token length
-is returned."  
+is returned."
   2)
 
 (defun idlwave-surround (&optional before after escape-chars length ec-hook)
@@ -2237,8 +2214,8 @@ return value."
     (let* ((length (or length 1)) ; establish a default for LENGTH
           (prev-char (char-after (- (point) (1+ length)))))
       (when (or (not (memq prev-char escape-chars))
-               (and (fboundp ec-hook) 
-                    (setq length 
+               (and (fboundp ec-hook)
+                    (setq length
                           (save-excursion (funcall ec-hook prev-char)))))
        (backward-char length)
        (save-restriction
@@ -2460,7 +2437,7 @@ Returns non-nil if successfull."
         (let ((eos (save-excursion
                      (idlwave-block-jump-out -1 'nomark)
                      (point))))
-          (if (setq status (idlwave-find-key 
+          (if (setq status (idlwave-find-key
                            idlwave-end-block-reg -1 'nomark eos))
               (idlwave-beginning-of-statement)
             (message "No nested block before beginning of containing block.")))
@@ -2468,7 +2445,7 @@ Returns non-nil if successfull."
       (let ((eos (save-excursion
                    (idlwave-block-jump-out 1 'nomark)
                    (point))))
-        (if (setq status (idlwave-find-key 
+        (if (setq status (idlwave-find-key
                          idlwave-begin-block-reg 1 'nomark eos))
             (idlwave-end-of-statement)
           (message "No nested block before end of containing block."))))
@@ -2482,7 +2459,7 @@ The marks are pushed."
         (here (point)))
     (goto-char (point-max))
     (if (re-search-backward idlwave-doclib-start nil t)
-        (progn 
+        (progn
          (setq beg (progn (beginning-of-line) (point)))
          (if (re-search-forward idlwave-doclib-end nil t)
              (progn
@@ -2516,7 +2493,7 @@ actual statement."
    ((eq major-mode 'idlwave-shell-mode)
     (if (re-search-backward idlwave-shell-prompt-pattern nil t)
        (goto-char (match-end 0))))
-   (t  
+   (t
     (if (save-excursion (forward-line -1) (idlwave-is-continuation-line))
        (idlwave-previous-statement)
       (beginning-of-line)))))
@@ -2588,6 +2565,14 @@ If not in a statement just moves to end of line. Returns position."
                        (looking-at "^@")))))
     last-statement))
 
+(defun idlwave-skip-multi-commands (&optional lim)
+  "Skip past multiple commands on a line (with `&')."
+  (let ((save-point (point)))
+    (when (re-search-forward ".*&" lim t)
+      (goto-char (match-end 0))
+      (if (idlwave-in-quote) (goto-char save-point)))
+    (point)))
+
 (defun idlwave-skip-label-or-case ()
   "Skip label or case statement element.
 Returns position after label.
@@ -2600,7 +2585,7 @@ If there is no label point is not moved and nil is returned."
   ;; - not in parenthesis (like a[0:3])
   ;; - not followed by another ":" in explicit class, ala a->b::c
   ;; As many in this mode, this function is heuristic and not an exact
-  ;; parser. 
+  ;; parser.
   (let* ((start (point))
         (eos (save-excursion (idlwave-end-of-statement) (point)))
         (end (idlwave-find-key ":" 1 'nomark eos)))
@@ -2627,6 +2612,7 @@ substatement."
         st nst last)
     (idlwave-beginning-of-statement)
     (idlwave-skip-label-or-case)
+    (idlwave-skip-multi-commands orig)
     (setq last (point))
     ;; Continue looking for substatements until we are past orig
     (while (and (<= (point) orig) (not (eobp)))
@@ -2656,7 +2642,8 @@ list not just the type symbol. Returns nil if not an identifiable
 statement."
   (save-excursion
     ;; Skip whitespace within a statement which is spaces, tabs, continuations
-    (while (looking-at "[ \t]*\\<\\$")
+    ;; and possibly comments
+    (while (looking-at "[ \t]*\\$")
       (forward-line 1))
     (skip-chars-forward " \t")
     (let ((st idlwave-statement-match)
@@ -2692,22 +2679,22 @@ See `idlwave-surround'. "
              ((null idlwave-pad-keyword)
               ;; Spaces should be removed at a keyword
               (idlwave-surround 0 0))
-             (t)))))          
+             (t)))))
 
 (defun idlwave-indent-and-action (&optional arg)
   "Call `idlwave-indent-line' and do expand actions.
 With prefix ARG non-nil, indent the entire sub-statement."
   (interactive "p")
   (save-excursion
-    (if        (and idlwave-expand-generic-end 
-            (re-search-backward "\\<\\(end\\)\\s-*\\=" 
+    (if        (and idlwave-expand-generic-end
+            (re-search-backward "\\<\\(end\\)\\s-*\\="
                                 (max 0 (- (point) 10)) t)
             (looking-at "\\(end\\)\\([ \n\t]\\|\\'\\)"))
        (progn (goto-char (match-end 1))
               ;;Expand the END abbreviation, just as RET or Space would have.
               (if abbrev-mode (expand-abbrev)
                 (idlwave-show-begin)))))
-  (if arg 
+  (if arg
       (idlwave-indent-statement)
     (idlwave-indent-line t)))
 
@@ -2757,13 +2744,16 @@ If the optional argument EXPAND is non-nil then the actions in
           ;; indent the line
           (idlwave-indent-left-margin (idlwave-calculate-indent)))
         ;; Adjust parallel comment
-        (end-of-line)
-        (if (idlwave-in-comment)
-            (indent-for-comment))))
+       (end-of-line)
+       (if (idlwave-in-comment)
+           ;; Emacs 21 is too smart with fill-column on comment indent
+           (let ((fill-column (if (fboundp 'comment-indent-new-line)
+                                  (1- (frame-width))
+                                fill-column)))
+             (indent-for-comment)))))
     (goto-char mloc)
     ;; Get rid of marker
-    (set-marker mloc nil)
-    ))
+    (set-marker mloc nil)))
 
 (defun idlwave-do-action (action)
   "Perform an action repeatedly on a line.
@@ -2863,7 +2853,7 @@ Inserts spaces before markers at point."
 
 (defun idlwave-calculate-paren-indent (beg-reg end-reg close-exp)
   "Calculate the continuation indent inside a paren group.
-Returns a cons-cell with (open . indent), where open is the 
+Returns a cons-cell with (open . indent), where open is the
 location of the open paren"
   (let ((open (nth 1 (parse-partial-sexp beg-reg end-reg))))
     ;; Found an innermost open paren.
@@ -2895,7 +2885,12 @@ statement if this statement is a continuation of the previous line."
            (case-fold-search t)
            (end-reg (progn (beginning-of-line) (point)))
            (close-exp (progn (skip-chars-forward " \t") (looking-at "\\s)")))
-           (beg-reg (progn (idlwave-previous-statement) (point)))
+;           (beg-reg (progn (idlwave-previous-statement) (point)))
+           (beg-reg (progn ;; Use substatement indent unless it's this line
+                     (idlwave-start-of-substatement 'pre)
+                     (if (eq (line-beginning-position) end-reg)
+                         (idlwave-previous-statement))
+                     (point)))
           (cur-indent (idlwave-current-indent))
           (else-cont (and (goto-char end-reg) (looking-at "[ \t]*else")))
           (basic-indent           ;; The basic, non-fancy indent
@@ -2908,7 +2903,7 @@ statement if this statement is a continuation of the previous line."
              (cond
               ;; A continued Procedure call or definition
               ((progn
-                 (idlwave-look-at "\\(pro\\|function\\)")
+                 (idlwave-look-at "^[ \t]*\\(pro\\|function\\)") ;skip over
                  (looking-at "[ \t]*\\([a-zA-Z0-9$_]+[ \t]*->[ \t]*\\)?[a-zA-Z][:a-zA-Z0-9$_]*[ \t]*\\(,\\)[ \t]*"))
                (goto-char (match-end 0))
                ;; Comment only, or blank line with "$"?  Align with ,
@@ -2916,14 +2911,14 @@ statement if this statement is a continuation of the previous line."
                    (goto-char (match-end 2)))
                (current-column))
 
-              ;; Continued assignment (with =), 
-              ((looking-at "[ \t]*[a-zA-Z0-9$_]+[ \t]*\\(=\\)[ \t]*")
+              ;; Continued assignment (with =),
+              ((looking-at "[ \t]*[][().a-zA-Z0-9$_]+[ \t]*\\(=\\)[ \t]*")
                (goto-char (match-end 0))
                ;; Comment only?  Align with =
                (if (save-match-data (looking-at "[ \t$]*\\(;.*\\)?$"))
-                   (progn 
+                   (progn
                      (goto-char (match-end 1))
-                     (if idlwave-surround-by-blank 
+                     (if idlwave-surround-by-blank
                          (1+ (current-column))
                        (current-column)))
                  (current-column))))))
@@ -2941,14 +2936,14 @@ statement if this statement is a continuation of the previous line."
                 (or idlwave-indent-to-open-paren ;; override
                     (< (- fancy-paren-indent basic-indent)
                        idlwave-max-extra-continuation-indent))))
-           fancy-enclosing-parent-indent)
-      (cond 
+           fancy-enclosing-paren-indent)
+      (cond
        ;; else continuations are always standard
-       (else-cont 
+       (else-cont
        cur-indent)
 
        ;; an allowed parenthesis-indent
-       (fancy-paren-indent-allowed 
+       (fancy-paren-indent-allowed
        fancy-paren-indent)
 
        ;; a disallowed paren indent nested inside one or more other
@@ -2963,7 +2958,7 @@ statement if this statement is a continuation of the previous line."
                         enclose-indent)
                     (catch 'loop
                       (while (setq enclose-indent-cons
-                                   (idlwave-calculate-paren-indent 
+                                   (idlwave-calculate-paren-indent
                                     beg-reg (max (1- enclose-open) beg-reg)
                                     nil)
                                    enclose-open   (car enclose-indent-cons)
@@ -2973,9 +2968,9 @@ statement if this statement is a continuation of the previous line."
                             (throw 'loop enclose-indent)))))))
        (min fancy-paren-indent
             (+ fancy-enclosing-paren-indent idlwave-continuation-indent)))
-               
+
        ;; a disallowed paren indent inside another type: indent relative
-       ((and fancy-paren-indent        
+       ((and fancy-paren-indent
             (not fancy-paren-indent-allowed)
             fancy-nonparen-indent-allowed )
        (+ fancy-nonparen-indent idlwave-continuation-indent))
@@ -2985,7 +2980,7 @@ statement if this statement is a continuation of the previous line."
        fancy-nonparen-indent)
 
        ;; everything else
-       (t 
+       (t
        basic-indent)))))
 
 (defun idlwave-find-key (key-re &optional dir nomark limit)
@@ -3027,7 +3022,7 @@ possibility of unbalanced blocks."
   (let* ((here (point))
          (case-fold-search t)
          (limit (if (>= dir 0) (point-max) (point-min)))
-         (block-limit (if (>= dir 0) 
+         (block-limit (if (>= dir 0)
                          idlwave-begin-block-reg
                        idlwave-end-block-reg))
          found
@@ -3038,7 +3033,7 @@ possibility of unbalanced blocks."
                               (idlwave-find-key
                                idlwave-begin-unit-reg dir t limit)
                             (end-of-line)
-                            (idlwave-find-key 
+                            (idlwave-find-key
                              idlwave-end-unit-reg dir t limit)))
                         limit)))
     (if (>= dir 0) (end-of-line)) ;Make sure we are in current block
@@ -3072,10 +3067,10 @@ Skips any whitespace. Returns 0 if the end-of-line follows the whitespace."
 Blank or comment-only lines following regular continuation lines (with
 `$') count as continuations too."
   (save-excursion
-    (or 
+    (or
      (idlwave-look-at "\\<\\$")
      (catch 'loop
-       (while (and (looking-at "^[ \t]*\\(;.*\\)?$") 
+       (while (and (looking-at "^[ \t]*\\(;.*\\)?$")
                   (eq (forward-line -1) 0))
         (if (idlwave-look-at "\\<\\$") (throw 'loop t)))))))
 
@@ -3173,7 +3168,7 @@ ignored."
                                           (beginning-of-line) (point))
                                         (point))))
                "[^;]"))
-       
+
         ;; Mark the beginning and end of the paragraph
         (goto-char bcl)
         (while (and (looking-at fill-prefix-reg)
@@ -3237,7 +3232,7 @@ ignored."
                       (insert (make-string diff ?\ ))))
                 (forward-line -1))
               )
-         
+
           ;; No hang. Instead find minimum indentation of paragraph
           ;; after first line.
           ;; For the following while statement, since START is at the
@@ -3269,7 +3264,7 @@ ignored."
                   t)
                  (current-column))
                indent))
-       
+
         ;; try to keep point at its original place
         (goto-char here)
 
@@ -3458,7 +3453,7 @@ is non-nil."
   (insert (current-time-string))
   (insert ", " (user-full-name))
   (if (boundp 'user-mail-address)
-      (insert " <" user-mail-address ">") 
+      (insert " <" user-mail-address ">")
     (insert " <" (user-login-name) "@" (system-name) ">"))
   ;; Remove extra spaces from line
   (idlwave-fill-paragraph)
@@ -3484,7 +3479,7 @@ location on mark ring so that the user can return to previous point."
             (setq end (match-end 0)))
        (progn
          (goto-char beg)
-         (if (re-search-forward 
+         (if (re-search-forward
               (concat idlwave-doc-modifications-keyword ":")
               end t)
              (end-of-line)
@@ -3574,6 +3569,18 @@ constants - a double quote followed by an octal digit."
       ;; return string beginning position or nil
       (if (> start bq) bq))))
 
+(defun idlwave-is-pointer-dereference (&optional limit)
+  "Determines if the character after point is a pointer dereference *."
+  (let ((pos (point)))
+    (and
+     (eq (char-after) ?\*)
+     (not (idlwave-in-quote))
+     (save-excursion
+       (forward-char)
+       (re-search-backward (concat "\\(" idlwave-idl-keywords
+                                  "\\|[[(*+-/=,^><]\\)\\s-*\\*") limit t)))))
+
+
 ;; Statement templates
 
 ;; Replace these with a general template function, something like
@@ -3616,7 +3623,7 @@ unless the optional second argument NOINDENT is non-nil."
          (indent-region beg end nil))
       (if (stringp prompt)
          (message prompt)))))
-  
+
 (defun idlwave-rw-case (string)
   "Make STRING have the case required by `idlwave-reserved-word-upcase'."
   (if idlwave-reserved-word-upcase
@@ -3634,7 +3641,7 @@ unless the optional second argument NOINDENT is non-nil."
 (defun idlwave-case ()
   "Build skeleton IDL case statement."
   (interactive)
-  (idlwave-template 
+  (idlwave-template
    (idlwave-rw-case "case")
    (idlwave-rw-case " of\n\nendcase")
    "Selector expression"))
@@ -3642,7 +3649,7 @@ unless the optional second argument NOINDENT is non-nil."
 (defun idlwave-switch ()
   "Build skeleton IDL switch statement."
   (interactive)
-  (idlwave-template 
+  (idlwave-template
    (idlwave-rw-case "switch")
    (idlwave-rw-case " of\n\nendswitch")
    "Selector expression"))
@@ -3650,7 +3657,7 @@ unless the optional second argument NOINDENT is non-nil."
 (defun idlwave-for ()
   "Build skeleton for loop statment."
   (interactive)
-  (idlwave-template 
+  (idlwave-template
    (idlwave-rw-case "for")
    (idlwave-rw-case " do begin\n\nendfor")
    "Loop expression"))
@@ -3665,14 +3672,14 @@ unless the optional second argument NOINDENT is non-nil."
 
 (defun idlwave-procedure ()
   (interactive)
-  (idlwave-template 
+  (idlwave-template
    (idlwave-rw-case "pro")
    (idlwave-rw-case "\n\nreturn\nend")
    "Procedure name"))
 
 (defun idlwave-function ()
   (interactive)
-  (idlwave-template 
+  (idlwave-template
    (idlwave-rw-case "function")
    (idlwave-rw-case "\n\nreturn\nend")
    "Function name"))
@@ -3686,7 +3693,7 @@ unless the optional second argument NOINDENT is non-nil."
 
 (defun idlwave-while ()
   (interactive)
-  (idlwave-template 
+  (idlwave-template
    (idlwave-rw-case "while")
    (idlwave-rw-case " do begin\n\nendwhile")
    "Entry condition"))
@@ -3765,8 +3772,8 @@ Buffer containing unsaved changes require confirmation before they are killed."
 (defun idlwave-count-outlawed-buffers (tag)
   "How many outlawed buffers have tag TAG?"
   (length (delq nil
-               (mapcar 
-                (lambda (x) (eq (cdr x) tag)) 
+               (mapcar
+                (lambda (x) (eq (cdr x) tag))
                 idlwave-outlawed-buffers))))
 
 (defun idlwave-do-kill-autoloaded-buffers (&rest reasons)
@@ -3780,9 +3787,9 @@ Buffer containing unsaved changes require confirmation before they are killed."
                   (memq (cdr entry) reasons))
               (kill-buffer (car entry))
               (incf cnt)
-              (setq idlwave-outlawed-buffers 
+              (setq idlwave-outlawed-buffers
                     (delq entry idlwave-outlawed-buffers)))
-       (setq idlwave-outlawed-buffers 
+       (setq idlwave-outlawed-buffers
              (delq entry idlwave-outlawed-buffers))))
     (message "%d buffer%s killed" cnt (if (= cnt 1) "" "s"))))
 
@@ -3794,7 +3801,7 @@ Intended for `after-save-hook'."
         (entry (assq buf idlwave-outlawed-buffers)))
     ;; Revoke license
     (if entry
-       (setq idlwave-outlawed-buffers 
+       (setq idlwave-outlawed-buffers
              (delq entry idlwave-outlawed-buffers)))
     ;; Remove this function from the hook.
     (remove-hook 'after-save-hook 'idlwave-revoke-license-to-kill 'local)))
@@ -3811,7 +3818,7 @@ Intended for `after-save-hook'."
            (throw 'exit efile))))))
 (defun idlwave-expand-lib-file-name (file)
   ;; Find FILE on the scanned lib path and return a buffer visiting it
-  (cond 
+  (cond
    ((null file) nil)
    ((string-match "\\`\\({\\([0-9]+\\)}/\\)\\(.*\\)" file)
     (expand-file-name (match-string 3 file)
@@ -3830,7 +3837,7 @@ you specify /."
   (interactive)
   (let (directory directories cmd append status numdirs dir getsubdirs
                  buffer save_buffer files numfiles item errbuf)
-    
+
     ;;
     ;; Read list of directories
     (setq directory (read-string "Tag Directories: " "."))
@@ -3881,9 +3888,12 @@ you specify /."
                  (progn
                    (message (concat "Tagging " item "..."))
                    (setq errbuf (get-buffer-create "*idltags-error*"))
-                   (setq status (+ status
-                                   (call-process "sh" nil errbuf nil "-c"
-                                                 (concat cmd append item))))
+                   (setq status
+                         (+ status
+                            (if (eq 0 (call-process "sh" nil errbuf nil "-c"
+                                                    (concat cmd append item)))
+                                0
+                              1)))
                    ;;
                    ;; Append additional tags
                    (setq append " --append ")
@@ -3893,13 +3903,13 @@ you specify /."
                  (setq numfiles (1+ numfiles))
                  (setq item (nth numfiles files))
                  )))
-           
+
            (setq numdirs (1+ numdirs))
            (setq dir (nth numdirs directories)))
        (progn
          (setq numdirs (1+ numdirs))
          (setq dir (nth numdirs directories)))))
-    
+
     (setq errbuf (get-buffer-create "*idltags-error*"))
     (if (= status 0)
        (kill-buffer errbuf))
@@ -3950,8 +3960,8 @@ blank lines."
 ;; substrings.  So most of the code can simply assume to deal with
 ;; "sinterned" strings.  The only exception is that the functions
 ;; which scan whole buffers for routine information do not intern the
-;; grabbed strings.  This is only done afterwards.  Therefore in these 
-;; functions it is *not* save to assume the strings can be compared
+;; grabbed strings.  This is only done afterwards.  Therefore in these
+;; functions it is *not* safe to assume the strings can be compared
 ;; with `eq' and be fed into the routine assq functions.
 
 ;; Here we define the hashing functions.
@@ -3968,7 +3978,7 @@ blank lines."
   ;; Make sure the hash functions are accessible.
   (if (or (not (fboundp 'gethash))
          (not (fboundp 'puthash)))
-      (progn 
+      (progn
        (require 'cl)
        (or (fboundp 'puthash)
            (defalias 'puthash 'cl-puthash))))
@@ -3987,7 +3997,7 @@ blank lines."
       ;; Reset the system & library hash
       (loop for entry in entries
        for var = (car entry) for size = (nth 1 entry)
-       do (setcdr (symbol-value var) 
+       do (setcdr (symbol-value var)
                   (make-hash-table ':size size ':test 'equal)))
       (setq idlwave-sint-files nil))
 
@@ -3996,7 +4006,7 @@ blank lines."
       ;; Reset the buffer & shell hash
       (loop for entry in entries
        for var = (car entry) for size = (nth 1 entry)
-       do (setcar (symbol-value var) 
+       do (setcar (symbol-value var)
                   (make-hash-table ':size size ':test 'equal))))))
 
 (defun idlwave-sintern-routine-or-method (name &optional class set)
@@ -4205,10 +4215,10 @@ will re-read the catalog."
                     "-l" (expand-file-name "~/.emacs")
                     "-l" "idlwave"
                     "-f" "idlwave-rescan-catalog-directories"))
-        (process (apply 'start-process "idlcat" 
+        (process (apply 'start-process "idlcat"
                         nil emacs args)))
     (setq idlwave-catalog-process process)
-    (set-process-sentinel 
+    (set-process-sentinel
      process
      (lambda (pro why)
        (when (string-match "finished" why)
@@ -4262,7 +4272,7 @@ With three prefix args, dispatch asynchronous process to do the update."
           ;; The override-idle means, even if the idle timer has done some
           ;; preparing work, load and renormalize everything anyway.
           (override-idle (or arg idlwave-buffer-case-takes-precedence)))
-      
+
       (setq idlwave-buffer-routines nil
            idlwave-compiled-routines nil
            idlwave-unresolved-routines nil)
@@ -4273,7 +4283,7 @@ With three prefix args, dispatch asynchronous process to do the update."
        (idlwave-reset-sintern (cond (load t)
                                     ((null idlwave-system-routines) t)
                                     (t 'bufsh))))
-      
+
       (if idlwave-buffer-case-takes-precedence
          ;; We can safely scan the buffer stuff first
          (progn
@@ -4288,7 +4298,7 @@ With three prefix args, dispatch asynchronous process to do the update."
                                    (idlwave-shell-is-running)))
             (ask-shell (and shell-is-running
                             idlwave-query-shell-for-routine-info)))
-      
+
        (if (or (not ask-shell)
                (not (interactive-p)))
            ;; 1. If we are not going to ask the shell, we need to do the
@@ -4299,7 +4309,7 @@ With three prefix args, dispatch asynchronous process to do the update."
            ;;    the current command.  Therefore, we do a concatenation
            ;;    now, even though the shell might do it again.
            (idlwave-concatenate-rinfo-lists nil t))
-      
+
        (when ask-shell
          ;; Ask the shell about the routines it knows.
          (message "Querying the shell")
@@ -4352,7 +4362,7 @@ With three prefix args, dispatch asynchronous process to do the update."
            (when (and (stringp idlwave-libinfo-file)
                       (file-regular-p idlwave-libinfo-file))
              (condition-case nil
-                 (progn 
+                 (progn
                    (message "Loading library catalog in idle time...")
                    (load-file idlwave-libinfo-file)
                    (message "Loading library catalog in idle time...done")
@@ -4413,11 +4423,11 @@ With three prefix args, dispatch asynchronous process to do the update."
 
 (defun idlwave-update-buffer-routine-info ()
   (let (res)
-    (cond 
+    (cond
      ((eq idlwave-scan-all-buffers-for-routine-info t)
       ;; Scan all buffers, current buffer last
       (message "Scanning all buffers...")
-      (setq res (idlwave-get-routine-info-from-buffers 
+      (setq res (idlwave-get-routine-info-from-buffers
                 (reverse (buffer-list)))))
      ((null idlwave-scan-all-buffers-for-routine-info)
       ;; Don't scan any buffers
@@ -4430,12 +4440,12 @@ With three prefix args, dispatch asynchronous process to do the update."
            (setq res (idlwave-get-routine-info-from-buffers
                       (list (current-buffer))))))))
     ;; Put the result into the correct variable
-    (setq idlwave-buffer-routines 
+    (setq idlwave-buffer-routines
          (idlwave-sintern-rinfo-list res t))))
 
 (defun idlwave-concatenate-rinfo-lists (&optional quiet run-hook)
   "Put the different sources for routine information together."
-  ;; The sequence here is important because earlier definitions shadow 
+  ;; The sequence here is important because earlier definitions shadow
   ;; later ones.  We assume that if things in the buffers are newer
   ;; then in the shell of the system, it is meant to be different.
 
@@ -4447,7 +4457,7 @@ With three prefix args, dispatch asynchronous process to do the update."
 
   ;; Give a message with information about the number of routines we have.
   (unless quiet
-    (message 
+    (message
      "Routine info updated:  buffer(%d)  compiled(%d)  catalog(%d)  system(%d)"
      (length idlwave-buffer-routines)
      (length idlwave-compiled-routines)
@@ -4464,7 +4474,7 @@ With three prefix args, dispatch asynchronous process to do the update."
          (when (and (setq class (nth 2 x))
                     (not (assq class idlwave-class-alist)))
            (push (list class) idlwave-class-alist)))
-       idlwave-class-alist)))      
+       idlwave-class-alist)))
 
 ;; Three functions for the hooks
 (defun idlwave-save-buffer-update ()
@@ -4497,7 +4507,7 @@ With three prefix args, dispatch asynchronous process to do the update."
 
 (defun idlwave-replace-buffer-routine-info (file new)
   "Cut the part from FILE out of `idlwave-buffer-routines' and add NEW."
-  (let ((list idlwave-buffer-routines) 
+  (let ((list idlwave-buffer-routines)
        found)
     (while list
       ;; The following test uses eq to make sure it works correctly
@@ -4508,7 +4518,7 @@ With three prefix args, dispatch asynchronous process to do the update."
            (setcar list nil)
            (setq found t))
        (if found
-           ;; End of that section reached. Jump. 
+           ;; End of that section reached. Jump.
            (setq list nil)))
       (setq list (cdr list)))
     (setq idlwave-buffer-routines
@@ -4540,11 +4550,11 @@ With three prefix args, dispatch asynchronous process to do the update."
       (save-restriction
        (widen)
        (goto-char (point-min))
-       (while (re-search-forward 
+       (while (re-search-forward
                "^[ \t]*\\(pro\\|function\\)[ \t]" nil t)
-         (setq string (buffer-substring 
+         (setq string (buffer-substring-no-properties
                        (match-beginning 0)
-                       (progn 
+                       (progn
                          (idlwave-end-of-statement)
                          (point))))
          (setq entry (idlwave-parse-definition string))
@@ -4582,7 +4592,7 @@ With three prefix args, dispatch asynchronous process to do the update."
        (push (match-string 1 string) args)))
     ;; Normalize and sort.
     (setq args (nreverse args))
-    (setq keywords (sort keywords (lambda (a b) 
+    (setq keywords (sort keywords (lambda (a b)
                                    (string< (downcase a) (downcase b)))))
     ;; Make and return the entry
     ;; We don't know which argument are optional, so this information
@@ -4592,7 +4602,7 @@ With three prefix args, dispatch asynchronous process to do the update."
          class
          (cond ((not (boundp 'idlwave-scanning-lib))
                 (cons 'buffer (buffer-file-name)))
-;              ((string= (downcase 
+;              ((string= (downcase
 ;                         (file-name-sans-extension
 ;                          (file-name-nondirectory (buffer-file-name))))
 ;                        (downcase name))
@@ -4600,7 +4610,7 @@ With three prefix args, dispatch asynchronous process to do the update."
 ;              (t (cons 'lib (file-name-nondirectory (buffer-file-name))))
                (t (cons 'lib (concat idlwave-scanning-lib-dir
                                      (file-name-nondirectory (buffer-file-name))))))
-         (concat 
+         (concat
           (if (string= type "function") "Result = " "")
           (if class "Obj ->[%s::]" "")
           "%s"
@@ -4644,10 +4654,10 @@ will pop up."
               (> (length idlwave-libinfo-file) 0)
               (file-accessible-directory-p
                (file-name-directory idlwave-libinfo-file))
-              (not (string= "" (file-name-nondirectory 
+              (not (string= "" (file-name-nondirectory
                                 idlwave-libinfo-file))))
     (error "`idlwave-libinfo-file' does not point to file in accessible directory"))
-  
+
   (cond
    ((and arg idlwave-path-alist
         (consp (car idlwave-path-alist))
@@ -4682,12 +4692,12 @@ will pop up."
     (let* ((rpl (idlwave-shell-path-filter))
           (sysdir (car rpl))
           (dirs (cdr rpl)))
-      (idlwave-display-libinfo-widget 
+      (idlwave-display-libinfo-widget
        sysdir dirs
        (delq nil (mapcar (lambda (x) (if (cdr x) (car x) nil))
                         idlwave-path-alist))))))
 
-(defconst idlwave-libinfo-widget-help-string 
+(defconst idlwave-libinfo-widget-help-string
   "This is the front-end to the creation of IDLWAVE library catalog.
 Please select below the directories on IDL's search path from which you
 would like to extract routine information, which will be stored in the file
@@ -4725,7 +4735,7 @@ directories and save the routine info.
   (make-local-variable 'idlwave-widget)
   (widget-insert (format idlwave-libinfo-widget-help-string
                         idlwave-libinfo-file))
-  
+
   (widget-create 'push-button
                 :notify 'idlwave-widget-scan-lib-files
                 "Scan & Save")
@@ -4757,7 +4767,7 @@ directories and save the routine info.
   (widget-insert "\n\n")
 
   (widget-insert "Select Directories\n")
-  
+
   (setq idlwave-widget
        (apply 'widget-create
               'checklist
@@ -4772,7 +4782,7 @@ directories and save the routine info.
   (widget-setup)
   (goto-char (point-min))
   (delete-other-windows))
-  
+
 (defun idlwave-delete-libinfo-file (&rest ignore)
   (if (yes-or-no-p
        (format "Delete file %s " idlwave-libinfo-file))
@@ -4831,7 +4841,7 @@ directories and save the routine info.
     ;; is used.  So we don't do it here - the catalog file looks nicer
     ;; when it is unsorted.
     ;;(message "Sorting...")
-    ;;(setq idlwave-library-routines 
+    ;;(setq idlwave-library-routines
     ;;(sort idlwave-library-routines 'idlwave-routine-entry-compare))
     ;;(message "Sorting...done")
     (message "Creating libinfo file...")
@@ -4906,7 +4916,7 @@ directories and save the routine info.
       (when (file-directory-p dir)
        (setq files (nreverse (directory-files dir t "[^.]")))
        (while (setq file (pop files))
-         (if (file-directory-p file) 
+         (if (file-directory-p file)
              (push (file-name-as-directory file) path)))
        (push dir path1)))
     path1))
@@ -4914,17 +4924,17 @@ directories and save the routine info.
 ;;----- Asking the shell -------------------
 
 ;; First, here is the idl program which can be used to query IDL for
-;; defined routines. 
+;; defined routines.
 (defconst idlwave-routine-info.pro
   "
 ;; START OF IDLWAVE SUPPORT ROUTINES
 pro idlwave_print_info_entry,name,func=func,separator=sep
   ;; See if it's an object method
   if name eq '' then return
-  func    = keyword_set(func) 
+  func    = keyword_set(func)
   methsep = strpos(name,'::')
   meth    = methsep ne -1
-  
+
   ;; Get routine info
   pars   = routine_info(name,/parameters,functions=func)
   source = routine_info(name,/source,functions=func)
@@ -4932,12 +4942,12 @@ pro idlwave_print_info_entry,name,func=func,separator=sep
   nkw    = pars.num_kw_args
   if nargs gt 0 then args = pars.args
   if nkw   gt 0 then kwargs = pars.kw_args
-  
+
   ;; Trim the class, and make the name
-  if meth then begin 
+  if meth then begin
       class = strmid(name,0,methsep)
       name  = strmid(name,methsep+2,strlen(name)-1)
-      if nargs gt 0 then begin 
+      if nargs gt 0 then begin
           ;; remove the self argument
           wh = where(args ne 'SELF',nargs)
           if nargs gt 0 then args = args(wh)
@@ -4946,7 +4956,7 @@ pro idlwave_print_info_entry,name,func=func,separator=sep
       ;; No class, just a normal routine.
       class = \"\"
   endelse
-   
+
   ;; Calling sequence
   cs = \"\"
   if func then cs = 'Result = '
@@ -4967,9 +4977,9 @@ pro idlwave_print_info_entry,name,func=func,separator=sep
           kwstring = kwstring + ' ' + kwargs(j)
       endfor
   endif
-  
+
   ret=(['IDLWAVE-PRO','IDLWAVE-FUN'])(func)
-  
+
   print,ret + ': ' + name + sep + class + sep + source(0).path  $
     + sep + cs + sep + kwstring
 end
@@ -5008,7 +5018,7 @@ pro idlwave_get_class_tags, class
   if res then print,'IDLWAVE-CLASS-TAGS: '+class+' '+strjoin(tags,' ',/single)
 end
 ;; END OF IDLWAVE SUPPORT ROUTINES
-" 
+"
   "The idl programs to get info from the shell.")
 
 (defvar idlwave-idlwave_routine_info-compiled nil
@@ -5016,7 +5026,7 @@ end
 
 (defvar idlwave-shell-temp-pro-file)
 (defvar idlwave-shell-temp-rinfo-save-file)
-(defun idlwave-shell-update-routine-info (&optional quiet run-hooks)
+(defun idlwave-shell-update-routine-info (&optional quiet run-hooks preempt)
   "Query the shell for routine_info of compiled modules and update the lists."
   ;; Save and compile the procedure.  The compiled procedure is then
   ;; saved into an IDL SAVE file, to allow for fast RESTORE.
@@ -5031,12 +5041,12 @@ end
       (erase-buffer)
       (insert idlwave-routine-info.pro)
       (save-buffer 0))
-    (idlwave-shell-send-command 
+    (idlwave-shell-send-command
      (concat ".run " idlwave-shell-temp-pro-file)
      nil 'hide)
 ;    (message "SENDING SAVE") ; ????????????????????????
     (idlwave-shell-send-command
-     (format "save,'idlwave_routine_info','idlwave_print_info_entry',FILE='%s',/ROUTINES" 
+     (format "save,'idlwave_routine_info','idlwave_print_info_entry',FILE='%s',/ROUTINES"
             (idlwave-shell-temp-file 'rinfo))
      nil 'hide))
 
@@ -5048,7 +5058,7 @@ end
    `(progn
       (idlwave-shell-routine-info-filter)
       (idlwave-concatenate-rinfo-lists ,quiet ,run-hooks))
-   'hide))
+   'hide preempt))
 
 ;; ---------------------------------------------------------------------------
 ;;
@@ -5127,7 +5137,7 @@ When we force a method or a method keyword, CLASS can specify the class."
 
      ;; Check for any special completion functions
      ((and idlwave-complete-special
-          (idlwave-complete-special)))
+          (idlwave-call-special idlwave-complete-special)))
 
      ((and (idlwave-in-quote)
           (not (eq what 'class)))
@@ -5148,7 +5158,7 @@ When we force a method or a method keyword, CLASS can specify the class."
                              (idlwave-all-class-inherits class-selector)))
             (isa (concat "procedure" (if class-selector "-method" "")))
             (type-selector 'pro))
-       (setq idlwave-completion-help-info 
+       (setq idlwave-completion-help-info
              (list 'routine nil type-selector class-selector nil super-classes))
        (idlwave-complete-in-buffer
         'procedure (if class-selector 'method 'routine)
@@ -5156,7 +5166,9 @@ When we force a method or a method keyword, CLASS can specify the class."
         (format "Select a %s name%s"
                 isa
                 (if class-selector
-                    (format " (class is %s)" class-selector)
+                    (format " (class is %s)"
+                            (if (eq class-selector t)
+                                "unknown" class-selector))
                   ""))
         isa
         'idlwave-attach-method-classes)))
@@ -5169,7 +5181,7 @@ When we force a method or a method keyword, CLASS can specify the class."
                              (idlwave-all-class-inherits class-selector)))
             (isa (concat "function" (if class-selector "-method" "")))
             (type-selector 'fun))
-       (setq idlwave-completion-help-info 
+       (setq idlwave-completion-help-info
              (list 'routine nil type-selector class-selector nil super-classes))
        (idlwave-complete-in-buffer
         'function (if class-selector 'method 'routine)
@@ -5177,7 +5189,9 @@ When we force a method or a method keyword, CLASS can specify the class."
         (format "Select a %s name%s"
                 isa
                 (if class-selector
-                    (format " (class is %s)" class-selector)
+                    (format " (class is %s)"
+                            (if (eq class-selector t)
+                                "unknown" class-selector))
                   ""))
         isa
         'idlwave-attach-method-classes)))
@@ -5205,14 +5219,14 @@ When we force a method or a method keyword, CLASS can specify the class."
        (setq list (idlwave-fix-keywords name 'pro class list))
        (unless list (error (format "No keywords available for procedure %s"
                                    (idlwave-make-full-name class name))))
-       (setq idlwave-completion-help-info 
+       (setq idlwave-completion-help-info
              (list 'keyword name type-selector class-selector nil super-classes))
        (idlwave-complete-in-buffer
         'keyword 'keyword list nil
         (format "Select keyword for procedure %s%s"
                 (idlwave-make-full-name class name)
                 (if (or (member '("_EXTRA") list)
-                        (member '("_REF_EXTRA") list))                  
+                        (member '("_REF_EXTRA") list))
                     " (note _EXTRA)" ""))
         isa
         'idlwave-attach-keyword-classes)))
@@ -5243,13 +5257,13 @@ When we force a method or a method keyword, CLASS can specify the class."
                         (idlwave-make-full-name class name)))
        (unless list (error (format "No keywords available for function %s"
                                    msg-name)))
-       (setq idlwave-completion-help-info 
+       (setq idlwave-completion-help-info
              (list 'keyword name type-selector class-selector nil super-classes))
        (idlwave-complete-in-buffer
         'keyword 'keyword list nil
         (format "Select keyword for function %s%s" msg-name
                 (if (or (member '("_EXTRA") list)
-                        (member '("_REF_EXTRA") list))                  
+                        (member '("_REF_EXTRA") list))
                     " (note _EXTRA)" ""))
         isa
         'idlwave-attach-keyword-classes)))
@@ -5260,17 +5274,18 @@ When we force a method or a method keyword, CLASS can specify the class."
   "List of special completion functions.
 These functions are called for each completion.  Each function must check
 if its own special completion context is present.  If yes, it should
-use `idlwave-complete-in-buffer' to do some completion and return `t'.
-If such a function returns `t', *no further* attempts to complete
-other contexts will be done.  If the function returns `nil', other completions
+use `idlwave-complete-in-buffer' to do some completion and return t.
+If such a function returns t, *no further* attempts to complete
+other contexts will be done.  If the function returns nil, other completions
 will be tried.")
-(defun idlwave-complete-special ()
-  (let ((functions idlwave-complete-special)
-       fun)
+
+(defun idlwave-call-special (functions &rest args)
+  (let ((funcs functions)
+       fun ret)
     (catch 'exit
-      (while (setq fun (pop functions))
-       (if (funcall fun)
-           (throw 'exit t)))
+      (while (setq fun (pop funcs))
+       (if (setq ret (apply fun args))
+           (throw 'exit ret)))
       nil)))
 
 (defun idlwave-make-force-complete-where-list (what &optional module class)
@@ -5286,10 +5301,10 @@ will be tried.")
                      ("class")))
         (module (idlwave-sintern-routine-or-method module class))
         (class (idlwave-sintern-class class))
-        (what (cond 
+        (what (cond
                ((equal what 0)
                 (setq what
-                      (intern (completing-read 
+                      (intern (completing-read
                                "Complete what? " what-list nil t))))
                ((integerp what)
                 (setq what (intern (car (nth (1- what) what-list)))))
@@ -5311,7 +5326,7 @@ will be tried.")
             (super-classes nil)
             (type-selector 'pro)
             (pro (or module
-                     (idlwave-completing-read 
+                     (idlwave-completing-read
                       "Procedure: " (idlwave-routines) 'idlwave-selector))))
        (setq pro (idlwave-sintern-routine pro))
        (list nil-list nil-list 'procedure-keyword
@@ -5325,7 +5340,7 @@ will be tried.")
             (super-classes nil)
             (type-selector 'fun)
             (func (or module
-                      (idlwave-completing-read 
+                      (idlwave-completing-read
                        "Function: " (idlwave-routines) 'idlwave-selector))))
        (setq func (idlwave-sintern-routine func))
        (list nil-list nil-list 'function-keyword
@@ -5365,7 +5380,7 @@ will be tried.")
 
      ((eq what 'class)
       (list nil-list nil-list 'class nil-list nil))
-     
+
      (t (error "Illegal value for WHAT")))))
 
 (defun idlwave-completing-read (&rest args)
@@ -5388,7 +5403,7 @@ will be tried.")
                    (stringp idlwave-shell-default-directory)
                    (file-directory-p idlwave-shell-default-directory))
               idlwave-shell-default-directory
-            default-directory)))          
+            default-directory)))
     (comint-dynamic-complete-filename)))
 
 (defun idlwave-make-full-name (class name)
@@ -5397,7 +5412,7 @@ will be tried.")
 
 (defun idlwave-rinfo-assoc (name type class list)
   "Like `idlwave-rinfo-assq', but sintern strings first."
-  (idlwave-rinfo-assq 
+  (idlwave-rinfo-assq
    (idlwave-sintern-routine-or-method name class)
    type (idlwave-sintern-class class) list))
 
@@ -5522,14 +5537,18 @@ INFO is as returned by idlwave-what-function or -procedure."
        (save-excursion (goto-char apos)
                        (looking-at "->[a-zA-Z][a-zA-Z0-9$_]*::")))))
 
+(defvar idlwave-determine-class-special nil
+  "List of special functions for determining class.
+Must accept two arguments: `apos' and `info'")
+
 (defun idlwave-determine-class (info type)
-  ;; Determine the class of a routine call.  INFO is the structure returned
-  ;; `idlwave-what-function' or `idlwave-what-procedure'.
-  ;; The third element in this structure is the class.  When nil, we return nil.
-  ;; When t, try to get the class from text properties at the arrow.  When
-  ;; the object is "self", we use the class of the current routine.
-  ;; otherwise prompt the user for a class name.  Also stores the selected
-  ;; class as a text property at the arrow.
+  ;; Determine the class of a routine call.
+  ;; INFO is the `cw-list' structure as returned by idlwave-where.
+  ;; The second element in this structure is the class.  When nil, we
+  ;; return nil.  When t, try to get the class from text properties at
+  ;; the arrow.  When the object is "self", we use the class of the
+  ;; current routine.  otherwise prompt the user for a class name.
+  ;; Also stores the selected class as a text property at the arrow.
   ;; TYPE is 'fun or 'pro.
   (let* ((class (nth 2 info))
         (apos (nth 3 info))
@@ -5543,14 +5562,14 @@ INFO is as returned by idlwave-what-function or -procedure."
                      (dassoc (cdr dassoc))
                      (t t)))
         (arrow (and apos (string= (buffer-substring apos (+ 2 apos)) "->")))
-        (is-self 
+        (is-self
          (and arrow
               (save-excursion (goto-char apos)
                               (forward-word -1)
                               (let ((case-fold-search t))
                                 (looking-at "self\\>")))))
         (force-query idlwave-force-class-query)
-        store class-alist)
+        store special-class class-alist)
     (cond
      ((null class) nil)
      ((eq t class)
@@ -5560,12 +5579,23 @@ INFO is as returned by idlwave-what-function or -procedure."
               (not force-query))
          (setq class (get-text-property apos 'idlwave-class)
                class (idlwave-sintern-class class)))
+      (if (and (eq t class) is-self)
+         (setq class (or (nth 2 (idlwave-current-routine)) class)))
+
+      ;; Before prompting, try any special class determination routines
       (when (and (eq t class)
-                is-self)
-       (setq class (or (nth 2 (idlwave-current-routine)) class)))
+                idlwave-determine-class-special
+                (not force-query))
+       (setq special-class
+             (idlwave-call-special idlwave-determine-class-special apos))
+       (if special-class
+           (setq class (idlwave-sintern-class special-class)
+                 store idlwave-store-inquired-class)))
+
+      ;; Prompt for a class, if we need to
       (when (and (eq class t)
                 (or force-query query))
-       (setq class-alist 
+       (setq class-alist
              (mapcar 'list (idlwave-all-method-classes (car info) type)))
        (setq class
              (idlwave-sintern-class
@@ -5574,19 +5604,25 @@ INFO is as returned by idlwave-what-function or -procedure."
                 (error "No classes available with method %s" (car info)))
                ((and (= (length class-alist) 1) (not force-query))
                 (car (car class-alist)))
-               (t 
+               (t
                 (setq store idlwave-store-inquired-class)
-                (idlwave-completing-read 
+                (idlwave-completing-read
                  (format "Class%s: " (if (stringp (car info))
                                          (format " for %s method %s"
                                                  type (car info))
                                        ""))
                  class-alist nil nil nil 'idlwave-class-history))))))
+
+      ;; Store it, if requested
       (when (and class (not (eq t class)))
        ;; We have a real class here
        (when (and store arrow)
-         (put-text-property apos (+ apos 2) 'idlwave-class class)
-         (put-text-property apos (+ apos 2) 'face idlwave-class-arrow-face))
+         (condition-case ()
+             (add-text-properties
+              apos (+ apos 2)
+              `(idlwave-class ,class face ,idlwave-class-arrow-face
+                              rear-nonsticky t))
+           (error nil)))
        (setf (nth 2 info) class))
       ;; Return the class
       class)
@@ -5605,14 +5641,14 @@ INFO is as returned by idlwave-what-function or -procedure."
           )))
 
 (defun idlwave-where ()
-  "Find out where we are. 
+  "Find out where we are.
 The return value is a list with the following stuff:
 \(PRO-LIST FUNC-LIST COMPLETE-WHAT CW-LIST LAST-CHAR)
 
 PRO-LIST       (PRO POINT CLASS ARROW)
 FUNC-LIST      (FUNC POINT CLASS ARROW)
 COMPLETE-WHAT  a symbol indicating what kind of completion makes sense here
-CW-LIST        (PRO-OR-FUNC POINT CLASS ARROW)  Like PRO-LIST, for what can 
+CW-LIST        (PRO-OR-FUNC POINT CLASS ARROW)  Like PRO-LIST, for what can
                be completed here.
 LAST-CHAR      last relevant character before point (non-white non-comment,
                not part of current identifier or leading slash).
@@ -5624,7 +5660,7 @@ POINT:  Where is this
 CLASS:  What class has the routine (nil=no, t=is method, but class unknown)
 ARROW:  Location of the arrow"
   (idlwave-routines)
-  (let* (;(bos (save-excursion (idlwave-beginning-of-statement) (point))) 
+  (let* (;(bos (save-excursion (idlwave-beginning-of-statement) (point)))
          (bos (save-excursion (idlwave-start-of-substatement 'pre) (point)))
         (func-entry (idlwave-what-function bos))
          (func (car func-entry))
@@ -5645,8 +5681,8 @@ ARROW:  Location of the arrow"
      ((string-match "\\`[ \t]*\\(pro\\|function\\)[ \t]+[a-zA-Z0-9_]*\\'"
                     (buffer-substring bos (point)))
       (setq cw 'class))
-     ((string-match 
-       "\\`[ \t]*\\([a-zA-Z][a-zA-Z0-9$_]*\\)?\\'" 
+     ((string-match
+       "\\`[ \t]*\\([a-zA-Z][a-zA-Z0-9$_]*\\)?\\'"
        (buffer-substring (if (> pro-point 0) pro-point bos) (point)))
       (setq cw 'procedure cw-class pro-class cw-point pro-point
            cw-arrow pro-arrow))
@@ -5655,11 +5691,11 @@ ARROW:  Location of the arrow"
       nil)
      ((string-match "OBJ_NEW([ \t]*['\"]\\([a-zA-Z0-9$_]*\\)?\\'"
                    (buffer-substring bos (point)))
-      (setq cw 'class))                    
+      (setq cw 'class))
      ((string-match "\\<inherits\\s-+\\([a-zA-Z0-9$_]*\\)?\\'"
                    (buffer-substring bos (point)))
-      (setq cw 'class))                    
-     ((and func 
+      (setq cw 'class))
+     ((and func
           (> func-point pro-point)
           (= func-level 1)
           (memq last-char '(?\( ?,)))
@@ -5678,7 +5714,7 @@ ARROW:  Location of the arrow"
       (setq cw 'function)
       (save-excursion
        (if (re-search-backward "->[ \t]*\\(\\([$a-zA-Z0-9_]+\\)::\\)?[$a-zA-Z0-9_]*\\=" bos t)
-           (setq cw-arrow (match-beginning 0)
+           (setq cw-arrow (copy-marker (match-beginning 0))
                  cw-class (if (match-end 2)
                               (idlwave-sintern-class (match-string 2))
                             t))))))
@@ -5698,14 +5734,14 @@ ARROW:  Location of the arrow"
 
 (defun idlwave-what-function (&optional bound)
   ;; Find out if point is within the argument list of a function.
-  ;; The return value is ("function-name" (point) level).
-  ;; Level is 1 on the to level parenthesis, higher further down.
+  ;; The return value is ("function-name" class arrow-start (point) level).
+  ;; Level is 1 on the top level parentheses, higher further down.
 
   ;; If the optional BOUND is an integer, bound backwards directed
   ;;    searches to this point.
 
   (catch 'exit
-    (let (pos 
+    (let (pos
          func-point
          (cnt 0)
          func arrow-start class)
@@ -5720,18 +5756,18 @@ ARROW:  Location of the arrow"
             (setq pos (point))
             (incf cnt)
             (when (and (= (following-char) ?\()
-                       (re-search-backward 
+                       (re-search-backward
                         "\\(::\\|\\<\\)\\([a-zA-Z][a-zA-Z0-9$_]*\\)[ \t]*\\="
                         bound t))
               (setq func (match-string 2)
                     func-point (goto-char (match-beginning 2))
                     pos func-point)
-              (if (re-search-backward 
+              (if (re-search-backward
                    "->[ \t]*\\(\\([a-zA-Z][a-zA-Z0-9$_]*\\)::\\)?\\=" bound t)
-                  (setq arrow-start (match-beginning 0)
+                  (setq arrow-start (copy-marker (match-beginning 0))
                         class (or (match-string 2) t)))
-              (throw 
-               'exit 
+              (throw
+               'exit
                (list
                 (idlwave-sintern-routine-or-method func class)
                 (idlwave-sintern-class class)
@@ -5747,24 +5783,24 @@ ARROW:  Location of the arrow"
   ;;    searches to this point.
   (let ((pos (point)) pro-point
        pro class arrow-start string)
-    (save-excursion 
+    (save-excursion
       ;;(idlwave-beginning-of-statement)
       (idlwave-start-of-substatement 'pre)
       (setq string (buffer-substring (point) pos))
-      (if (string-match 
-          "\\`\\(.*&\\)?[ \t]*\\([a-zA-Z][a-zA-Z0-9$_]*\\)[ \t]*\\(,\\|\\'\\)" string)
-         (setq pro (match-string 2 string)
-               pro-point (+ (point) (match-beginning 2)))
+      (if (string-match
+          "\\`[ \t]*\\([a-zA-Z][a-zA-Z0-9$_]*\\)[ \t]*\\(,\\|\\'\\)" string)
+         (setq pro (match-string 1 string)
+               pro-point (+ (point) (match-beginning 1)))
        (if (and (idlwave-skip-object)
                 (setq string (buffer-substring (point) pos))
-                (string-match 
+                (string-match
                  "\\`[ \t]*\\(->\\)[ \t]*\\(\\([a-zA-Z][a-zA-Z0-9$_]*\\)::\\)?\\([a-zA-Z][a-zA-Z0-9$_]*\\)?[ \t]*\\(,\\|\\'\\)" string))
            (setq pro (if (match-beginning 4)
                          (match-string 4 string))
                  pro-point (if (match-beginning 4)
                                (+ (point) (match-beginning 4))
                        pos)
-                 arrow-start (+ (point) (match-beginning 1))
+                 arrow-start (copy-marker (+ (point) (match-beginning 1)))
                  class (or (match-string 3 string) t)))))
     (list (idlwave-sintern-routine-or-method pro class)
          (idlwave-sintern-class class)
@@ -5801,7 +5837,6 @@ ARROW:  Location of the arrow"
              (throw 'exit nil))))
        (goto-char pos)
       nil)))
-  
 
 (defun idlwave-last-valid-char ()
   "Return the last character before point which is not white or a comment
@@ -5839,8 +5874,9 @@ This function is not general, can only be used for completion stuff."
 (defun idlwave-complete-in-buffer (type stype list selector prompt isa
                                        &optional prepare-display-function)
   "Perform TYPE completion of word before point against LIST.
-SELECTOR is the PREDICATE argument for the completion function.
-Show PROMPT in echo area.  TYPE is one of 'function, 'procedure or 'keyword."
+SELECTOR is the PREDICATE argument for the completion function.  Show
+PROMPT in echo area.  TYPE is one of 'function, 'procedure,
+'class-tag, or 'keyword."
   (let* ((completion-ignore-case t)
         beg (end (point)) slash part spart completion all-completions
         dpart dcompletion)
@@ -5869,7 +5905,7 @@ Show PROMPT in echo area.  TYPE is one of 'function, 'procedure or 'keyword."
     (cond
      ((null completion)
       ;; nothing available.
-      (error "Can't find %s completion for \"%s\"" isa part))
+      (error (concat prompt ": no completion for \"%s\"") part))
      ((and (not (equal dpart dcompletion))
           (not (eq t completion)))
       ;; We can add something
@@ -5912,7 +5948,7 @@ Show PROMPT in echo area.  TYPE is one of 'function, 'procedure or 'keyword."
                     idlwave-complete-empty-string-as-lower-case)
                 (not idlwave-completion-force-default-case))
            (setq list (mapcar (lambda (x)
-                                (if (listp x) 
+                                (if (listp x)
                                     (setcar x (downcase (car x)))
                                   (setq x (downcase x)))
                                 x)
@@ -5932,49 +5968,55 @@ Show PROMPT in echo area.  TYPE is one of 'function, 'procedure or 'keyword."
           (re-search-backward "\\<\\(pro\\|function\\)[ \t]+\\="
                               (- (point) 15) t)
           (goto-char (point-min))
-          (re-search-forward 
+          (re-search-forward
            "^[ \t]*\\(pro\\|function\\)[ \t]+\\([a-zA-Z0-9_]+::\\)" nil t))))
       ;; Yank the full class specification
       (insert (match-string 2))
     ;; Do the completion
-    (idlwave-complete-in-buffer 'class 'class (idlwave-class-alist) nil 
+    (idlwave-complete-in-buffer 'class 'class (idlwave-class-alist) nil
                                "Select a class" "class")))
 
-(defun idlwave-attach-classes (list is-kwd show-classes)
+(defun idlwave-attach-classes (list type show-classes)
   ;; Attach the proper class list to a LIST of completion items.
-  ;; IS-KWD, when non-nil, shows its keywords - otherwise its methods
+  ;; TYPE, when 'kwd, shows classes for method keywords, when
+  ;; 'class-tag, for class tags, and otherwise for methods.
   ;; SHOW-CLASSES is the value of `idlwave-completion-show-classes'.
-  (catch 'exit
-    (if (or (null show-classes)           ; don't want to see classes
-           (null class-selector)         ; not a method call
-           (and (stringp class-selector) ; the class is already known
-                (not super-classes)))    ; no possibilities for inheritance
-       ;; In these cases, we do not have to do anything
-       (throw 'exit list))
-    
+  (if (or (null show-classes)           ; don't want to see classes
+         (null class-selector)         ; not a method call
+         (and
+          (stringp class-selector) ; the class is already known
+          (not super-classes)))    ; no possibilities for inheritance
+      ;; In these cases, we do not have to do anything
+      list
     (let* ((do-prop (and (>= show-classes 0)
                         (>= emacs-major-version 21)))
           (do-buf (not (= show-classes 0)))
-          ; (do-dots (featurep 'xemacs))
+          ;; (do-dots (featurep 'xemacs))
           (do-dots t)
-          (inherit (if super-classes
+          (inherit (if (and (not (eq type 'class-tag)) super-classes)
                        (cons class-selector super-classes)))
           (max (abs show-classes))
           (lmax (if do-dots (apply 'max (mapcar 'length list))))
          classes nclasses class-info space)
-      (mapcar 
+      (mapcar
        (lambda (x)
         ;; get the classes
-        (setq classes
-              (if is-kwd 
-                  (idlwave-all-method-keyword-classes
-                   method-selector x type-selector)
-                (idlwave-all-method-classes x type-selector)))
-        (if inherit
-            (setq classes 
-                  (delq nil
-                        (mapcar (lambda (x) (if (memq x inherit) x nil))
-                                classes))))
+        (if (eq type 'class-tag)
+            ;; Just one class for tags
+            (setq classes
+                  (list
+                   (idlwave-class-or-superclass-with-tag class-selector x)))
+          ;; Multiple classes for method of method-keyword
+          (setq classes
+                (if (eq type 'kwd)
+                    (idlwave-all-method-keyword-classes
+                     method-selector x type-selector)
+                  (idlwave-all-method-classes x type-selector)))
+          (if inherit
+              (setq classes
+                    (delq nil
+                          (mapcar (lambda (x) (if (memq x inherit) x nil))
+                                  classes)))))
         (setq nclasses (length classes))
         ;; Make the separator between item and class-info
         (if do-dots
@@ -6001,10 +6043,14 @@ Show PROMPT in echo area.  TYPE is one of 'function, 'procedure or 'keyword."
 
 (defun idlwave-attach-method-classes (list)
   ;; Call idlwave-attach-classes with method parameters
-  (idlwave-attach-classes list nil idlwave-completion-show-classes))
+  (idlwave-attach-classes list 'method idlwave-completion-show-classes))
 (defun idlwave-attach-keyword-classes (list)
   ;; Call idlwave-attach-classes with keyword parameters
-  (idlwave-attach-classes list t idlwave-completion-show-classes))
+  (idlwave-attach-classes list 'kwd idlwave-completion-show-classes))
+(defun idlwave-attach-class-tag-classes (list)
+  ;; Call idlwave-attach-classes with class structure tags
+  (idlwave-attach-classes list 'class-tag idlwave-completion-show-classes))
+
 
 ;;----------------------------------------------------------------------
 ;;----------------------------------------------------------------------
@@ -6025,7 +6071,7 @@ sort the list before displaying"
          ((= 1 (length list))
           (setq rtn (car list)))
          ((featurep 'xemacs)
-          (if sort (setq list (sort list (lambda (a b) 
+          (if sort (setq list (sort list (lambda (a b)
                                            (string< (upcase a) (upcase b))))))
           (setq menu
                 (append (list title)
@@ -6036,7 +6082,7 @@ sort the list before displaying"
           (setq resp (get-popup-menu-response menu))
           (funcall (event-function resp) (event-object resp)))
          (t
-          (if sort (setq list (sort list (lambda (a b) 
+          (if sort (setq list (sort list (lambda (a b)
                                            (string< (upcase a) (upcase b))))))
           (setq menu (cons title
                            (list
@@ -6127,7 +6173,7 @@ sort the list before displaying"
     (setq idlwave-before-completion-wconf (current-window-configuration)))
 
   (if (featurep 'xemacs)
-      (idlwave-display-completion-list-xemacs 
+      (idlwave-display-completion-list-xemacs
        list)
     (idlwave-display-completion-list-emacs list))
 
@@ -6163,6 +6209,13 @@ sort the list before displaying"
        (remove-text-properties beg (point) '(face nil))))
   (eval idlwave-complete-after-success-form-force))
 
+(defun idlwave-keyboard-quit ()
+  (interactive)
+  (unwind-protect
+      (if (eq (car-safe last-command) 'idlwave-display-completion-list)
+         (idlwave-restore-wconf-after-completion))
+    (keyboard-quit)))
+
 (defun idlwave-restore-wconf-after-completion ()
   "Restore the old (before completion) window configuration."
   (and idlwave-completion-restore-window-configuration
@@ -6215,7 +6268,7 @@ sort the list before displaying"
                                    (setq this-command last-command)))
     new-map))
 
-;; In Emacs we also to replace choose keybindings in the completion
+;; In Emacs we also replace keybindings in the completion
 ;; map in order to install our wrappers.
 
 (defun idlwave-display-completion-list-emacs (list)
@@ -6233,7 +6286,7 @@ sort the list before displaying"
 (defun idlwave-make-modified-completion-map-emacs (old-map)
   "Replace `choose-completion' and `mouse-choose-completion' in OLD-MAP."
   (let ((new-map (copy-keymap old-map)))
-    (substitute-key-definition 
+    (substitute-key-definition
      'choose-completion 'idlwave-choose-completion new-map)
     (substitute-key-definition
      'mouse-choose-completion 'idlwave-mouse-choose-completion new-map)
@@ -6259,8 +6312,8 @@ sort the list before displaying"
 ;;
 ;; - Go again over the documentation how to write a completion
 ;;   plugin.  It is in self.el, but currently still very bad.
-;;   This could be in a separate file in the distribution, or 
-;;   in an appendix for the manual.  
+;;   This could be in a separate file in the distribution, or
+;;   in an appendix for the manual.
 
 (defun idlwave-struct-tags ()
   "Return a list of all tags in the structure defined at point.
@@ -6271,14 +6324,23 @@ Point is expected just before the opening `{' of the struct definition."
           (end (cdr borders))
           tags)
       (goto-char beg)
-      (while (re-search-forward "[{,][ \t]*\\(\\$.*\n[ \t]*\\)?\\([a-zA-Z][a-zA-Z0-9_]*\\)[ \t]*:" end t)
+      (while (re-search-forward "[{,][ \t]*\\(\\$.*\n\\(^[ \t]*\\(\\$[ \t]*\\)?\\(;.*\\)?\n\\)*[ \t]*\\)?\\([a-zA-Z][a-zA-Z0-9_]*\\)[ \t]*:" end t)
        ;; Check if we are still on the top level of the structure.
        (if (and (condition-case nil (progn (up-list -1) t) (error nil))
                 (= (point) beg))
-           (push (match-string 2) tags))
+           (push (match-string 5) tags))
        (goto-char (match-end 0)))
       (nreverse tags))))
 
+(defun idlwave-find-struct-tag (tag)
+  "Find a given TAG in the structure defined at point."
+  (let* ((borders (idlwave-struct-borders))
+        (beg (car borders))
+        (end (cdr borders))
+        (case-fold-search t))
+    (re-search-forward (concat "\\(^[ \t]*\\|[,{][ \t]*\\)" tag "[ \t]*:")
+                      end t)))
+
 (defun idlwave-struct-inherits ()
   "Return a list of all `inherits' names in the struct at point.
 Point is expected just before the opening `{' of the struct definition."
@@ -6318,13 +6380,13 @@ Point is expected just before the opening `{' of the struct definition."
       (cons beg (point)))))
 
 (defun idlwave-find-structure-definition (&optional var name bound)
-  "Search forward for a structure definition.  
+  "Search forward for a structure definition.
 If VAR is non-nil, search for a structure assigned to variable VAR.
 If NAME is non-nil, search for a named structure NAME.  If BOUND is an
 integer, limit the search.  If BOUND is the symbol `all', we search
 first back and then forward through the entire file.  If BOUND is the
 symbol `back' we search only backward."
-  (let* ((ws "[ \t]*\\(\\$.*\n[ \t]*\\)?")
+  (let* ((ws "[ \t]*\\(\\$.*\n[ \t]*\\)*")
         (case-fold-search t)
         (lim (if (integerp bound) bound nil))
         (re (concat
@@ -6372,16 +6434,23 @@ symbol `back' we search only backward."
        (setcdr inherits (mapcar (lambda (x) (idlwave-sintern-class x 'set))
                                 (cdr inherits))))))
 
+(defun idlwave-find-class-definition (class)
+  (let ((case-fold-search t))
+    (if (re-search-forward
+        (concat "^[ \t]*pro[ \t]+" (downcase class) "__define" "\\>") nil t)
+       ;; FIXME: should we limit to end of pro here?
+       (idlwave-find-structure-definition nil class))))
+
 (defun idlwave-find-class-info (class)
   "Find the __define procedure for a class structure and return info entry."
   (let* ((pro (concat (downcase class) "__define"))
         (class (idlwave-sintern-class class))
         (idlwave-auto-routine-info-updates nil)
-        (file (cdr (nth 3 (idlwave-rinfo-assoc pro 'pro nil 
+        (file (cdr (nth 3 (idlwave-rinfo-assoc pro 'pro nil
                                                (idlwave-routines)))))
         buf)
     (if (or (not file)
-           (not (file-regular-p 
+           (not (file-regular-p
                  (setq file (idlwave-expand-lib-file-name file)))))
        nil ; Cannot get info
       (save-excursion
@@ -6394,14 +6463,10 @@ symbol `back' we search only backward."
          (insert-file-contents file))
        (save-excursion
          (goto-char 1)
-         (setq case-fold-search t)
-         (when (and (re-search-forward
-                     (concat "^[ \t]*pro[ \t]+" pro "\\>") nil t)
-                    ;; FIXME: should we limit to end of pro here?
-                    (idlwave-find-structure-definition nil class))
-           (list class
-                 (cons 'tags (idlwave-struct-tags))
-                 (cons 'inherits (idlwave-struct-inherits)))))))))
+         (if (idlwave-find-class-definition class)
+             (list class
+                   (cons 'tags (idlwave-struct-tags))
+                   (cons 'inherits (idlwave-struct-inherits)))))))))
 
 (defun idlwave-class-tags (class)
   "Return the native tags in CLASS."
@@ -6412,8 +6477,13 @@ symbol `back' we search only backward."
 
 (defun idlwave-all-class-tags (class)
   "Return a list of native and inherited tags in CLASS."
-  (apply 'append (mapcar 'idlwave-class-tags
-                        (cons class (idlwave-all-class-inherits class)))))
+  (condition-case err
+      (apply 'append (mapcar 'idlwave-class-tags
+                            (cons class (idlwave-all-class-inherits class))))
+    (error
+     (idlwave-class-tag-reset)
+     (error "%s" (error-message-string err)))))
+
 
 (defun idlwave-all-class-inherits (class)
   "Return a list of all superclasses of CLASS (recursively expanded).
@@ -6427,12 +6497,21 @@ The list is cached in `idlwave-class-info' for faster access."
          entry)
       (if (setq entry (assq 'all-inherits info))
          (cdr entry)
-       (let ((inherits (idlwave-class-inherits class))
+       ;; Save the depth of inheritance scan to check for circular references
+       (let ((inherits (mapcar (lambda (x) (cons x 0))
+                               (idlwave-class-inherits class)))
              rtn all-inherits cl)
          (while inherits
            (setq cl (pop inherits)
-                 rtn (cons cl rtn)
-                 inherits (append inherits (idlwave-class-inherits cl))))
+                 rtn (cons (car cl) rtn)
+                 inherits (append (mapcar (lambda (x)
+                                            (cons x (1+ (cdr cl))))
+                                          (idlwave-class-inherits (car cl)))
+                                  inherits))
+           (if (> (cdr cl) 999)
+             (error
+              "Class scan: inheritance depth exceeded. Circular inheritance?")
+             ))
          (setq all-inherits (nreverse rtn))
          (nconc info (list (cons 'all-inherits all-inherits)))
          all-inherits))))))
@@ -6446,10 +6525,10 @@ The list is cached in `idlwave-class-info' for faster access."
 (defvar idlwave-current-tags-class nil)
 (defvar idlwave-current-class-tags nil)
 (defvar idlwave-current-native-class-tags nil)
-(defvar idlwave-sint-classtags nil)
-(idlwave-new-sintern-type 'classtag)
+(defvar idlwave-sint-class-tags nil)
+(idlwave-new-sintern-type 'class-tag)
 (add-to-list 'idlwave-complete-special 'idlwave-complete-class-structure-tag)
-(add-hook 'idlwave-update-rinfo-hook 'idlwave-classtag-reset)
+(add-hook 'idlwave-update-rinfo-hook 'idlwave-class-tag-reset)
 
 (defun idlwave-complete-class-structure-tag ()
   "Complete a structure tag on a `self' argument in an object method."
@@ -6461,33 +6540,39 @@ The list is cached in `idlwave-class-info' for faster access."
          (skip-chars-backward "[a-zA-Z0-9._$]")
          (and (< (point) (- pos 4))
               (looking-at "self\\.")))
-       (let* ((class (nth 2 (idlwave-current-routine))))
+       (let* ((class-selector (nth 2 (idlwave-current-routine)))
+              (super-classes (idlwave-all-class-inherits class-selector)))
          ;; Check if we are in a class routine
-         (unless class
+         (unless class-selector
            (error "Not in a method procedure or function"))
          ;; Check if we need to update the "current" class
-         (if (not (equal class idlwave-current-tags-class))
-             (idlwave-prepare-class-tag-completion class))
-         (setq idlwave-completion-help-info nil)
+         (if (not (equal class-selector idlwave-current-tags-class))
+             (idlwave-prepare-class-tag-completion class-selector))
+         (setq idlwave-completion-help-info
+               (list 'idlwave-complete-class-structure-tag-help
+                     (idlwave-sintern-routine
+                      (concat class-selector "__define"))
+                     nil))
          (let  ((idlwave-cpl-bold idlwave-current-native-class-tags))
            (idlwave-complete-in-buffer
-            'classtag 'classtag 
+            'class-tag 'class-tag
             idlwave-current-class-tags nil
-            (format "Select a tag of class %s" class)
-            "class tag"))
+            (format "Select a tag of class %s" class-selector)
+            "class tag"
+            'idlwave-attach-class-tag-classes))
          t) ; return t to skip other completions
       nil)))
 
-(defun idlwave-classtag-reset ()
+(defun idlwave-class-tag-reset ()
   (setq idlwave-current-tags-class nil))
 
 (defun idlwave-prepare-class-tag-completion (class)
   "Find and parse the necessary class definitions for class structure tags."
-  (setq idlwave-sint-classtags nil)
+  (setq idlwave-sint-class-tags nil)
   (setq idlwave-current-tags-class class)
   (setq idlwave-current-class-tags
        (mapcar (lambda (x)
-                 (list (idlwave-sintern-classtag x 'set)))
+                 (list (idlwave-sintern-class-tag x 'set)))
                (idlwave-all-class-tags class)))
   (setq idlwave-current-native-class-tags
        (mapcar 'downcase (idlwave-class-tags class))))
@@ -6497,7 +6582,7 @@ The list is cached in `idlwave-class-info' for faster access."
 ;; Completing system variables and their structure fields
 ;; This is also a plugin.  It is a bit bigger since we support loading
 ;; current system variables from the shell and highlighting in the
-;; completions buffer. 
+;; completions buffer.
 
 (defvar idlwave-sint-sysvars nil)
 (defvar idlwave-sint-sysvartags nil)
@@ -6523,7 +6608,7 @@ Gets set in `idlw-rinfo.el'.")
             (skip-chars-backward "[a-zA-Z0-9_$]")
             (equal (char-before) ?!))
           (setq idlwave-completion-help-info '(idlwave-complete-sysvar-help))
-          (idlwave-complete-in-buffer 'sysvar 'sysvar 
+          (idlwave-complete-in-buffer 'sysvar 'sysvar
                                       idlwave-system-variables-alist nil
                                       "Select a system variable"
                                       "system variable")
@@ -6542,14 +6627,16 @@ Gets set in `idlw-rinfo.el'.")
             (or tags (error "System variable !%s is not a structure" var))
             (setq idlwave-completion-help-info
                   (list 'idlwave-complete-sysvar-help var))
-            (idlwave-complete-in-buffer 'sysvartag 'sysvartag 
+            (idlwave-complete-in-buffer 'sysvartag 'sysvartag
                                         tags nil
                                         "Select a system variable tag"
                                         "system variable tag")
             t)) ; return t to skip other completions
          (t nil))))
 
-(defvar name) 
+;; Here we fake help using the routine "system variables" with keyword
+;; set to the sysvar.  Name and kwd are global variables here.
+(defvar name)
 (defvar kwd)
 (defun idlwave-complete-sysvar-help (mode word)
   (cond
@@ -6566,7 +6653,43 @@ Gets set in `idlw-rinfo.el'.")
                          (nth 1 idlwave-completion-help-info)
                        word))))
    (t (error "This should not happen"))))
-                         
+
+;; Fake help in the source buffer for class structure tags.
+;; kwd and name are global-variables here.
+(defvar idlwave-help-do-class-struct-tag nil)
+(defun idlwave-complete-class-structure-tag-help (mode word)
+  (cond
+   ((eq mode 'test) ; nothing gets fontified for class tags
+    nil)
+   ((eq mode 'set)
+    (let (class-with)
+      (when (setq class-with
+               (idlwave-class-or-superclass-with-tag
+                idlwave-current-tags-class
+                word))
+       (if (assq (idlwave-sintern-class class-with)
+                 idlwave-system-class-info)
+           (error "No help available for system class tags."))
+       (setq name (concat class-with "__define"))))
+    (setq kwd word
+         idlwave-help-do-class-struct-tag t))
+   (t (error "This should not happen"))))
+
+(defun idlwave-class-or-superclass-with-tag (class tag)
+  "Find and return the CLASS or one of its superclass with the
+associated TAG, if any."
+  (let ((sclasses (cons class (cdr (assq 'all-inherits
+                                        (idlwave-class-info class)))))
+       cl)
+   (catch 'exit
+     (while sclasses
+       (setq cl (pop sclasses))
+       (let ((tags (idlwave-class-tags cl)))
+        (while tags
+          (if (eq t (compare-strings tag 0 nil (car tags) 0 nil t))
+            (throw 'exit cl))
+          (setq tags (cdr tags))))))))
+
 
 (defun idlwave-sysvars-reset ()
   (if (and (fboundp 'idlwave-shell-is-running)
@@ -6582,14 +6705,14 @@ Gets set in `idlw-rinfo.el'.")
 
 (defun idlwave-remember-builtin-sysvars ()
   (setq idlwave-builtin-system-variables
-       (mapcar 'downcase 
+       (mapcar 'downcase
                (mapcar 'car idlwave-system-variables-alist))))
 
 (defun idlwave-sintern-sysvar-alist ()
   (let ((list idlwave-system-variables-alist) entry)
     (while (setq entry (pop list))
       (setcar entry (idlwave-sintern-sysvar (car entry) 'set))
-      (setcdr entry (mapcar (lambda (x) 
+      (setcdr entry (mapcar (lambda (x)
                              (list (idlwave-sintern-sysvartag (car x) 'set)))
                            (cdr entry))))))
 
@@ -6620,9 +6743,10 @@ Gets set in `idlw-rinfo.el'.")
       (set-buffer "*Completions*")
       (save-excursion
        (goto-char (point-min))
-       (while (re-search-forward "\\.*<[^>]+>" nil t)
-         (put-text-property (match-beginning 0) (match-end 0)
-                            'face 'font-lock-string-face))))))
+       (let ((buffer-read-only nil))
+         (while (re-search-forward "\\.*<[^>]+>" nil t)
+           (put-text-property (match-beginning 0) (match-end 0)
+                              'face 'font-lock-string-face)))))))
 
 (defun idlwave-uniquify (list)
   (let (nlist)
@@ -6653,11 +6777,11 @@ Restore the pre-completion window configuration if possible."
       nil)))
 
   ;; Restore the pre-completion window configuration if this is safe.
-  
-  (if (or (eq verify 'force)                                    ; force 
-         (and 
+
+  (if (or (eq verify 'force)                                    ; force
+         (and
           (get-buffer-window "*Completions*")                  ; visible
-          (idlwave-local-value 'idlwave-completion-p 
+          (idlwave-local-value 'idlwave-completion-p
                                "*Completions*")                ; cib-buffer
           (eq (marker-buffer idlwave-completion-mark)
               (current-buffer))                                ; buffer OK
@@ -6676,10 +6800,13 @@ Restore the pre-completion window configuration if possible."
 (defvar idlwave-last-context-help-pos nil)
 (defun idlwave-context-help (&optional arg)
   "Display IDL Online Help on context.
-If point is on a keyword, help for that keyword will be shown.
-If point is on a routine name or in the argument list of a routine,
-help for that routine will be displayed.
-Works for system routines and keywords only."
+If point is on a keyword, help for that keyword will be shown.  If
+point is on a routine name or in the argument list of a routine, help
+for that routine will be displayed.  Works for system routines and
+keywords, it pulls up text help.  For other routies and keywords,
+visits the source file, finding help in the header (if
+`idlwave-help-source-try-header' is non-nil) or the routine definition
+itself."
   (interactive "P")
   (idlwave-require-online-help)
   (idlwave-do-context-help arg))
@@ -6711,19 +6838,19 @@ Works for system routines and keywords only."
   (if idlwave-help-is-loaded
       t  ;; everything is OK.
     (let* ((dir (or (idlwave-help-directory)
-                   (error "Online Help is not installed (idlwave-help-directory is unknown)")))
+                   (error "Online Help not installed (help directory unknown) - download at idlwave.org")))
           (lfile1 (expand-file-name "idlw-help.elc" dir))
           (lfile2 (expand-file-name "idlw-help.el" dir))
           (hfile (expand-file-name "idlw-help.txt" dir)))
       (if (or (and (file-regular-p lfile1) (load-file lfile1))
              (and (file-regular-p lfile2) (load-file lfile2)))
-         (progn 
+         (progn
            (if (and idlwave-help-frame-parameters
                     (not (assoc 'width idlwave-help-frame-parameters)))
                (push (cons 'width idlwave-help-frame-width)
                      idlwave-help-frame-parameters))
            (or idlwave-help-topics
-               (error "File `%s' in help dir `%s' does not define `idlwave-help-topics'" 
+               (error "File `%s' in help dir `%s' does not define `idlwave-help-topics'"
                         "idlw-help.el" dir)))
        (error "No such file `%s' in help dir `%s'" "idlw-help.el" dir))
       (if (file-regular-p hfile)
@@ -6793,7 +6920,7 @@ With ARG, enforce query for the class of object methods."
     (if (string-match "\\(pro\\|function\\)[ \t]+\\(\\(.*\\)::\\)?\\(.*\\)"
                      resolve)
        (setq type (match-string 1 resolve)
-             class (if (match-beginning 2) 
+             class (if (match-beginning 2)
                        (match-string 3 resolve)
                      nil)
              name (match-string 4 resolve)))
@@ -6802,15 +6929,15 @@ With ARG, enforce query for the class of object methods."
 
     (cond
      ((null class)
-      (idlwave-shell-send-command 
+      (idlwave-shell-send-command
        (format "resolve_routine,'%s'%s" (downcase name) kwd)
        'idlwave-update-routine-info
        nil t))
      (t
-      (idlwave-shell-send-command 
+      (idlwave-shell-send-command
        (format "resolve_routine,'%s__define'%s" (downcase class) kwd)
-       (list 'idlwave-shell-send-command 
-            (format "resolve_routine,'%s__%s'%s" 
+       (list 'idlwave-shell-send-command
+            (format "resolve_routine,'%s__%s'%s"
                     (downcase class) (downcase name) kwd)
             '(idlwave-update-routine-info)
             nil t))))))
@@ -6826,9 +6953,9 @@ use.  With ARG force class query for object methods."
         (module (idlwave-fix-module-if-obj_new (idlwave-what-module)))
         (default (concat (idlwave-make-full-name (nth 2 module) (car module))
                          (if (eq (nth 1 module) 'pro) "<p>" "<f>")))
-        (list 
+        (list
          (delq nil
-               (mapcar (lambda (x) 
+               (mapcar (lambda (x)
                          (if (eq 'system (car-safe (nth 3 x)))
                              ;; Take out system routines with no source.
                              nil
@@ -6838,7 +6965,7 @@ use.  With ARG force class query for object methods."
                             (cdr x))))
                        (idlwave-routines))))
         (name (idlwave-completing-read
-               (format "Module (Default %s): " 
+               (format "Module (Default %s): "
                        (if default default "none"))
                list))
         type class)
@@ -6868,20 +6995,20 @@ use.  With ARG force class query for object methods."
          source (or force-source (nth 3 entry))
          name2 (if (nth 2 entry)
                    (idlwave-make-full-name (nth 2 entry) name)
-                 name1))                 
+                 name1))
     (cond
      ((or (null name) (equal name ""))
       (error "Abort"))
      ((null entry)
       (error "Nothing known about a module %s" name2))
      ((eq (car source) 'system)
-      (error "Source code for system routine %s is not available" 
+      (error "Source code for system routine %s is not available"
             name2))
      ((equal (cdr source) "")
       (error "Source code for routine %s is not available"
             name2))
      ((memq (car source) '(buffer lib compiled))
-      (setq buf1 
+      (setq buf1
            (if (eq (car source) 'lib)
                (idlwave-find-file-noselect
                 (idlwave-expand-lib-file-name
@@ -6896,7 +7023,7 @@ use.  With ARG force class query for object methods."
                     (cond ((equal type "f") "function")
                           ((equal type "p") "pro")
                           (t "\\(pro\\|function\\)"))
-                    "\\>[ \t]+" 
+                    "\\>[ \t]+"
                     (regexp-quote (downcase name2))
                     "[^a-zA-Z0-9_$]")
             nil t)
@@ -6933,17 +7060,17 @@ Used by `idlwave-routine-info' and `idlwave-find-module'."
       (cond
        ((and (eq cw 'procedure)
             (not (equal this-word "")))
-       (setq this-word (idlwave-sintern-routine-or-method 
+       (setq this-word (idlwave-sintern-routine-or-method
                         this-word (nth 2 (nth 3 where))))
        (list this-word 'pro
-             (idlwave-determine-class 
+             (idlwave-determine-class
               (cons this-word (cdr (nth 3 where)))
               'pro)))
-       ((and (eq cw 'function) 
+       ((and (eq cw 'function)
             (not (equal this-word ""))
             (or (eq next-char ?\()     ; exclude arrays, vars.
                 (looking-at "[a-zA-Z0-9_]*[ \t]*(")))
-       (setq this-word (idlwave-sintern-routine-or-method 
+       (setq this-word (idlwave-sintern-routine-or-method
                         this-word (nth 2 (nth 3 where))))
        (list this-word 'fun
              (idlwave-determine-class
@@ -7018,8 +7145,8 @@ keyword region, change to the appropriate Init method."
                             string)
               (setq class (idlwave-sintern-class (match-string 1 string)))
               (setq idlwave-current-obj_new-class class)
-              (setq keywords 
-                    (append keywords 
+              (setq keywords
+                    (append keywords
                             (nth 5 (idlwave-rinfo-assq
                                     (idlwave-sintern-method "INIT")
                                     'fun
@@ -7035,11 +7162,11 @@ keyword region, change to the appropriate Init method."
                 (mapcar (lambda (k) (add-to-list 'keywords k))
                         (nth 5 x))))
       (setq keywords (idlwave-uniquify keywords)))
-    
+
     ;; If we have inheritance, add all keywords from superclasses, if
     ;; the user indicated that method in
     ;; `idlwave-keyword-class-inheritance'
-    (when (and 
+    (when (and
           idlwave-keyword-class-inheritance
           (stringp class)
           (or (assq (idlwave-sintern-keyword "_extra") keywords)
@@ -7059,7 +7186,7 @@ keyword region, change to the appropriate Init method."
                 (mapcar (lambda (k) (add-to-list 'keywords k))
                         (nth 5 x))))
       (setq keywords (idlwave-uniquify keywords)))
-    
+
     ;; Return the final list
     keywords))
 
@@ -7084,14 +7211,14 @@ If we do not know about MODULE, just return KEYWORD literally."
                    (assq (idlwave-sintern-keyword "_REF_EXTRA") kwd-alist)))
         (completion-ignore-case t)
         candidates)
-    (cond ((assq kwd kwd-alist) 
+    (cond ((assq kwd kwd-alist)
           kwd)
          ((setq candidates (all-completions kwd kwd-alist))
           (if (= (length candidates) 1)
               (car candidates)
             candidates))
          ((and entry extra)
-          ;; Inheritance may cause this keyword to be correct 
+          ;; Inheritance may cause this keyword to be correct
           keyword)
          (entry
           ;; We do know the function, which does not have the keyword.
@@ -7103,13 +7230,13 @@ If we do not know about MODULE, just return KEYWORD literally."
 
 (defvar idlwave-rinfo-mouse-map (make-sparse-keymap))
 (defvar idlwave-rinfo-map (make-sparse-keymap))
-(define-key idlwave-rinfo-mouse-map 
+(define-key idlwave-rinfo-mouse-map
   (if (featurep 'xemacs) [button2] [mouse-2])
   'idlwave-mouse-active-rinfo)
-(define-key idlwave-rinfo-mouse-map 
+(define-key idlwave-rinfo-mouse-map
   (if (featurep 'xemacs) [(shift button2)] [(shift mouse-2)])
   'idlwave-mouse-active-rinfo-shift)
-(define-key idlwave-rinfo-mouse-map 
+(define-key idlwave-rinfo-mouse-map
   (if (featurep 'xemacs) [button3] [mouse-3])
   'idlwave-mouse-active-rinfo-right)
 (define-key idlwave-rinfo-mouse-map " " 'idlwave-active-rinfo-space)
@@ -7135,7 +7262,7 @@ If we do not know about MODULE, just return KEYWORD literally."
   (let* ((initial-class (or initial-class class))
         (entry (or (idlwave-best-rinfo-assq name type class
                                             (idlwave-routines))
-                   (idlwave-rinfo-assq name type class 
+                   (idlwave-rinfo-assq name type class
                                        idlwave-unresolved-routines)))
         (name (or (car entry) name))
         (class (or (nth 2 entry) class))
@@ -7219,14 +7346,14 @@ If we do not know about MODULE, just return KEYWORD literally."
                  (format calling-seq name))
                "\n")
        (add-text-properties beg (point) props)
-       
+
        (insert "Keywords:")
        (if (null keywords)
            (insert " No keywords accepted.")
          (setq col 9)
          (mapcar
           (lambda (x)
-            (if (>= (+ col 1 (length (car x))) 
+            (if (>= (+ col 1 (length (car x)))
                     (window-width))
                 (progn
                   (insert "\n         ")
@@ -7243,7 +7370,7 @@ If we do not know about MODULE, just return KEYWORD literally."
             (add-text-properties beg (point) props)
             (setq col (+ col 1 (length (car x)))))
           keywords))
-       
+
        (setq cnt 1 total (length all))
        (while (setq entry (pop all))
          (setq props (list 'mouse-face 'highlight
@@ -7252,7 +7379,7 @@ If we do not know about MODULE, just return KEYWORD literally."
                            'source (cons (car (nth 2 entry)) (nth 1 entry))
                            'data (cons 'source data)))
          (idlwave-insert-source-location
-          (format "\n%-8s  %s" 
+          (format "\n%-8s  %s"
                   (if (equal cnt 1)
                       (if (> total 1) "Sources:" "Source:")
                     "")
@@ -7261,7 +7388,7 @@ If we do not know about MODULE, just return KEYWORD literally."
          (incf cnt)
          (when (and all (> cnt idlwave-rinfo-max-source-lines))
            ;; No more source lines, please
-           (insert (format 
+           (insert (format
                     "\n          Source information truncated to %d entries."
                     idlwave-rinfo-max-source-lines))
            (setq all nil)))
@@ -7275,7 +7402,7 @@ If we do not know about MODULE, just return KEYWORD literally."
              (unwind-protect
                  (progn
                    (select-window win)
-                   (enlarge-window (- (/ (frame-height) 2) 
+                   (enlarge-window (- (/ (frame-height) 2)
                                       (window-height)))
                    (shrink-window-if-larger-than-buffer))
                (select-window ww)))))))))
@@ -7325,7 +7452,7 @@ If a file name is inserted, add FILE-PROPS to it."
               (if shell-flag "S" "-")
               (if buffer-flag "B" "-")
               "] ")))
-    (when (> ndupl 1) 
+    (when (> ndupl 1)
       (setq beg (point))
       (insert (format "(%dx) " ndupl))
       (add-text-properties beg (point) (list 'face 'bold)))
@@ -7349,7 +7476,7 @@ Return the name of the special lib if there is a match."
                  alist nil)))
       rtn)
      (t nil))))
-  
+
 (defun idlwave-mouse-active-rinfo-right (ev)
   (interactive "e")
   (idlwave-mouse-active-rinfo ev 'right))
@@ -7380,9 +7507,9 @@ was pressed."
          bufwin (get-buffer-window buf t))
     (cond ((eq id 'class)
           (if (window-live-p bufwin) (select-window bufwin))
-          (idlwave-display-calling-sequence 
+          (idlwave-display-calling-sequence
            (idlwave-sintern-method name)
-           type (idlwave-sintern-class word) 
+           type (idlwave-sintern-class word)
            initial-class))
          ((eq id 'usage)
           (idlwave-require-online-help)
@@ -7426,9 +7553,9 @@ was pressed."
       (setq bwin (get-buffer-window buffer)))
     (if (eq (preceding-char) ?/)
        (insert keyword)
-      (unless (save-excursion 
+      (unless (save-excursion
                (re-search-backward
-                "[(,][ \t]*\\(\\$[ \t]*\\(;.*\\)?\n\\)?[ \t]*\\=" 
+                "[(,][ \t]*\\(\\$[ \t]*\\(;.*\\)?\n\\)?[ \t]*\\="
                 (min (- (point) 100) (point-min)) t))
        (insert ", "))
       (if shift (insert "/"))
@@ -7480,7 +7607,7 @@ the load path in order to find a definition.  The output of this
 command can be used to detect possible name clashes during this process."
   (idlwave-routines)  ; Make sure everything is loaded.
   (unless idlwave-library-routines
-    (or (y-or-n-p 
+    (or (y-or-n-p
         "You don't have a library catalog.  Continue anyway? ")
        (error "Abort")))
   (let* ((routines (append idlwave-system-routines
@@ -7492,7 +7619,7 @@ command can be used to detect possible name clashes during this process."
         (keymap (make-sparse-keymap))
         (props (list 'mouse-face 'highlight
                      km-prop keymap
-                     'help-echo "Mouse2: Find source"))      
+                     'help-echo "Mouse2: Find source"))
         (nroutines (length (or special-routines routines)))
         (step (/ nroutines 99))
         (n 0)
@@ -7516,13 +7643,13 @@ command can be used to detect possible name clashes during this process."
     (message "Sorting routines...done")
 
     (define-key keymap (if (featurep 'xemacs) [(button2)] [(mouse-2)])
-      (lambda (ev) 
+      (lambda (ev)
        (interactive "e")
        (mouse-set-point ev)
        (apply 'idlwave-do-find-module
               (get-text-property (point) 'find-args))))
     (define-key keymap [(return)]
-      (lambda () 
+      (lambda ()
        (interactive)
        (apply 'idlwave-do-find-module
               (get-text-property (point) 'find-args))))
@@ -7553,8 +7680,8 @@ command can be used to detect possible name clashes during this process."
                          (if (eq (nth 1 routine) 'fun) "()" "")))
          (while (setq twin (pop dtwins))
            (setq props1 (append (list 'find-args
-                                      (list (nth 0 routine) 
-                                            (nth 1 routine) 
+                                      (list (nth 0 routine)
+                                            (nth 1 routine)
                                             (nth 2 routine)
                                             (cons 'lib (nth 1 twin))))
                                 props))
@@ -7581,7 +7708,7 @@ command can be used to detect possible name clashes during this process."
             (or (not (stringp sfile))
                 (not (string-match "\\S-" sfile))))
        (setq stype 'unresolved))
-    (princ (format "      %-10s %s\n" 
+    (princ (format "      %-10s %s\n"
                   stype
                   (if sfile sfile "No source code available")))))
 
@@ -7600,7 +7727,7 @@ ENTRY will also be returned, as the first item of this list."
               (eq type (nth 1 candidate))
               (eq class (nth 2 candidate)))
          (push candidate twins)))
-    (if (setq candidate (idlwave-rinfo-assq name type class 
+    (if (setq candidate (idlwave-rinfo-assq name type class
                                            idlwave-unresolved-routines))
        (push candidate twins))
     (cons entry (nreverse twins))))
@@ -7610,10 +7737,10 @@ ENTRY will also be returned, as the first item of this list."
 Dangerous twins are routines with same name, but in different files
 on the load path.
 If a file is in the system library and has an entry in the
-`idlwave-system-routines' list, we omit the latter because many IDL 
+`idlwave-system-routines' list, we omit the latter because many IDL
 routines are implemented as library routines."
   (let* ((entry (car entries))
-        (name (car entry))      ; 
+        (name (car entry))      ;
         (type (nth 1 entry))    ; Must be bound for
         (class (nth 2 entry))   ;  idlwave-routine-twin-compare
         (cnt 0)
@@ -7629,23 +7756,23 @@ routines are implemented as library routines."
       (setq key (cond ((eq type 'system) type)
                      (file (file-truename file))
                      (t 'unresolved)))
-      (if (and file 
+      (if (and file
               (not syslibp)
               (idlwave-syslib-p file))
          ;; We do have an entry in the system library
          (setq syslibp t))
-      
+
       (setq thefile (or thefile file))
       (if (setq entry (assoc key alist))
          (push type (nth 2 entry))
        (push (list key file (list type)) alist)))
-    
+
     (setq alist (nreverse alist))
-    
+
     (when syslibp
       ;; File is system *library* - remove any system entry
       (setq alist (delq (assoc 'system alist) alist)))
-    
+
     (when (and (idlwave-syslib-scanned-p)
               (setq entry (assoc 'system alist)))
       (setcar entry 'builtin))
@@ -7680,7 +7807,7 @@ compares twins on the basis of their file names and path locations."
      ((not (eq type (nth 1 b)))
       ;; Type decides
       (< (if (eq type 'fun) 1 0) (if (eq (nth 1 b) 'fun) 1 0)))
-     (t        
+     (t
       ;; A and B are twins - so the decision is more complicated.
       ;; Call twin-compare with the proper arguments.
       (idlwave-routine-entry-compare-twins a b)))))
@@ -7842,15 +7969,16 @@ Assumes that point is at the beginning of the unit as found by
         (buffer-substring-no-properties begin (point))
       (buffer-substring begin (point)))))
 
-(defun idlwave-function-menu ()
-  "Use `imenu' or `function-menu' to jump to a procedure or function."
-  (interactive)
-  (if (string-match "XEmacs" emacs-version)
+(defalias 'idlwave-function-menu
+  (condition-case nil
       (progn
        (require 'func-menu)
-       (function-menu))
-    (require 'imenu)
-    (imenu (imenu-choose-buffer-index))))
+       'function-menu)
+    (error (condition-case nil
+              (progn
+                (require 'imenu)
+                'imenu)
+            (error nil)))))
 
 ;; Here we kack func-menu.el in order to support this new mode.
 ;; The latest versions of func-menu.el already have this stuff in, so
@@ -7875,12 +8003,12 @@ Assumes that point is at the beginning of the unit as found by
   (start-process "idldeclient" nil
                 idlwave-shell-explicit-file-name "-c" "-e"
                  (buffer-file-name) "&"))
-                
+
 (defun idlwave-launch-idlhelp ()
   "Start the IDLhelp application."
   (interactive)
   (start-process "idlhelp" nil idlwave-help-application))
+
 ;; Menus - using easymenu.el
 (defvar idlwave-mode-menu-def
   `("IDLWAVE"
@@ -7967,7 +8095,7 @@ Assumes that point is at the beginning of the unit as found by
     ("Customize"
      ["Browse IDLWAVE Group" idlwave-customize t]
      "--"
-     ["Build Full Customize Menu" idlwave-create-customize-menu 
+     ["Build Full Customize Menu" idlwave-create-customize-menu
       (fboundp 'customize-menu-create)])
     ("Documentation"
      ["Describe Mode" describe-mode t]
@@ -7984,22 +8112,22 @@ Assumes that point is at the beginning of the unit as found by
   '("Debug"
     ["Start IDL shell" idlwave-shell t]
     ["Save and .RUN buffer" idlwave-shell-save-and-run
-     (and (boundp 'idlwave-shell-automatic-start) 
+     (and (boundp 'idlwave-shell-automatic-start)
          idlwave-shell-automatic-start)]))
 
 (if (or (featurep 'easymenu) (load "easymenu" t))
     (progn
-      (easy-menu-define idlwave-mode-menu idlwave-mode-map 
-                       "IDL and WAVE CL editing menu" 
+      (easy-menu-define idlwave-mode-menu idlwave-mode-map
+                       "IDL and WAVE CL editing menu"
                        idlwave-mode-menu-def)
-      (easy-menu-define idlwave-mode-debug-menu idlwave-mode-map 
-                       "IDL and WAVE CL editing menu" 
+      (easy-menu-define idlwave-mode-debug-menu idlwave-mode-map
+                       "IDL and WAVE CL editing menu"
                        idlwave-mode-debug-menu-def)))
 
 (defun idlwave-customize ()
   "Call the customize function with idlwave as argument."
   (interactive)
-  ;; Try to load the code for the shell, so that we can customize it 
+  ;; Try to load the code for the shell, so that we can customize it
   ;; as well.
   (or (featurep 'idlw-shell)
       (load "idlw-shell" t))
@@ -8010,11 +8138,11 @@ Assumes that point is at the beginning of the unit as found by
   (interactive)
   (if (fboundp 'customize-menu-create)
       (progn
-       ;; Try to load the code for the shell, so that we can customize it 
+       ;; Try to load the code for the shell, so that we can customize it
        ;; as well.
        (or (featurep 'idlw-shell)
            (load "idlw-shell" t))
-       (easy-menu-change 
+       (easy-menu-change
         '("IDLWAVE") "Customize"
         `(["Browse IDLWAVE group" idlwave-customize t]
           "--"
@@ -8062,7 +8190,7 @@ This function was written since `list-abbrevs' looks terrible for IDLWAVE mode."
   (let ((table (symbol-value 'idlwave-mode-abbrev-table))
        abbrevs
        str rpl func fmt (len-str 0) (len-rpl 0))
-    (mapatoms 
+    (mapatoms
      (lambda (sym)
        (if (symbol-value sym)
           (progn
@@ -8088,7 +8216,7 @@ This function was written since `list-abbrevs' looks terrible for IDLWAVE mode."
     (with-output-to-temp-buffer "*Help*"
       (if arg
          (progn
-           (princ "Abbreviations and Actions in IDLWAVE-Mode\n") 
+           (princ "Abbreviations and Actions in IDLWAVE-Mode\n")
            (princ "=========================================\n\n")
            (princ (format fmt "KEY" "REPLACE" "HOOK"))
            (princ (format fmt "---" "-------" "----")))
@@ -8125,4 +8253,5 @@ This function was written since `list-abbrevs' looks terrible for IDLWAVE mode."
 
 (provide 'idlwave)
 
+;;; arch-tag: f77f3b0c-c37c-424f-a328-0886fd42b6fb
 ;;; idlwave.el ends here