(f90-get-beg-of-line): Removed and replaced with line-beginning-position.
[bpt/emacs.git] / lisp / progmodes / f90.el
index a43143d..a70e4e7 100644 (file)
@@ -1,9 +1,9 @@
 ;;; f90.el --- Fortran-90 mode (free format)
 
-;; Copyright (C) 1995, 1996 Free Software Foundation, Inc.
+;; Copyright (C) 1995, 1996, 1997, 2000 Free Software Foundation, Inc.
 
-;; Author: Torbj\"orn Einarsson <T.Einarsson@clab.ericsson.se>
-;; Created: Sep. 21, 1995
+;; Author: Torbj\"orn Einarsson <Torbjorn.Einarsson@era.ericsson.se>
+;; Maintainer: Glenn Morris <gmorris@ast.cam.ac.uk>
 ;; Keywords: fortran, f90, languages
 
 ;; This file is part of GNU Emacs.
@@ -74,7 +74,7 @@
 
 ;; To customize the f90-mode for your taste, use, for example:
 ;;    (you don't have to specify values for all the parameters below)
-;;(setq f90-mode-hook
+;;(add-hook 'f90-mode-hook
 ;;      '(lambda () (setq f90-do-indent 3
 ;;                        f90-if-indent 3
 ;;                        f90-type-indent 3
 ;;                        f90-leave-line-no  nil
 ;;                        f90-startup-message t
 ;;                        indent-tabs-mode nil
+;;                        f90-font-lock-keywords f90-font-lock-keywords-2
 ;;                  )
 ;;       ;;The rest is not default.
 ;;       (abbrev-mode 1)             ; turn on abbreviation mode
-;;       (f90-auto-fill-mode 1)      ; turn on auto-filling
-;;       (turn-on-font-lock)         ; for highlighting
+;;       (f90-add-imenu-menu)        ; extra menu with functions etc.
 ;;       (if f90-auto-keyword-case   ; change case of all keywords on startup
 ;;           (f90-change-keywords f90-auto-keyword-case))
 ;;      ))
 ;; in your .emacs file (the shown values are the defaults). You can also
 ;; change the values of the lists f90-keywords etc.
 ;; The auto-fill and abbreviation minor modes are accessible from the menu,
-;; or by using M-x f90-auto-fill-mode and M-x abbrev-mode, respectively.
+;; or by using M-x auto-fill-mode and M-x abbrev-mode, respectively.
 
 ;; Remarks
 ;; 1) Line numbers are by default left-justified. If f90-leave-line-no is
 ;;    mechanism for treating multi-line directives (continued by \ ).
 ;; 7) f77 do-loops do 10 i=.. ; ; 10 continue are not correctly indented.
 ;;    You are urged to use f90-do loops (with labels if you wish).
+;; 8) The highlighting mode under XEmacs is not as complete as under Emacs.
 
 ;; List of user commands
 ;;   f90-previous-statement         f90-next-statement
 ;;   f90-indent-region    (can be called by calling indent-region)
 ;;   f90-indent-subprogram
 ;;   f90-break-line                 f90-join-lines
-;;   f90-auto-fill-mode
 ;;   f90-fill-region
 ;;   f90-insert-end
 ;;   f90-upcase-keywords            f90-upcase-region-keywords
 ;;   f90-downcase-keywords          f90-downcase-region-keywords
 ;;   f90-capitalize-keywords        f90-capitalize-region-keywords
+;;   f90-add-imenu-menu
+;;   f90-font-lock-1, f90-font-lock-2, f90-font-lock-3, f90-font-lock-4
 
 ;; Thanks to all the people who have tested the mode. Special thanks to Jens
 ;; Bloch Helmers for encouraging me to write this code, for creative
 
 ;;; Code:
 
-(defconst bug-f90-mode "T.Einarsson@clab.ericsson.se"
-  "Address of mailing list for F90 mode bugs.")
-
 ;; User options
-(defvar f90-do-indent 3
-  "*Extra indentation applied to DO blocks.")
 
-(defvar f90-if-indent 3
-  "*Extra indentation applied to IF, SELECT CASE, WHERE and FORALL blocks.")
+(defgroup f90 nil
+  "Fortran-90 mode"
+  :group 'languages)
 
-(defvar f90-type-indent 3
-  "*Extra indentation applied to TYPE, INTERFACE and BLOCK DATA blocks.")
+(defgroup f90-indent nil
+  "Fortran-90 indentation"
+  :prefix "f90-"
+  :group 'f90)
 
-(defvar f90-program-indent 2
-  "*Extra indentation applied to PROGRAM/MODULE/SUBROUTINE/FUNCTION blocks.")
 
-(defvar f90-continuation-indent 5
-  "*Extra indentation applied to F90 continuation lines.")
+(defcustom f90-do-indent 3
+  "*Extra indentation applied to DO blocks."
+  :type 'integer
+  :group 'f90-indent)
 
-(defvar f90-comment-region "!!$"
-  "*String inserted by \\[f90-comment-region]\
- at start of each line in region.")
+(defcustom f90-if-indent 3
+  "*Extra indentation applied to IF, SELECT CASE, WHERE and FORALL blocks."
+  :type 'integer
+  :group 'f90-indent)
 
-(defvar f90-indented-comment-re "!"
-  "*Regexp saying which comments to be indented like code.")
+(defcustom f90-type-indent 3
+  "*Extra indentation applied to TYPE, INTERFACE and BLOCK DATA blocks."
+  :type 'integer
+  :group 'f90-indent)
 
-(defvar f90-directive-comment-re "!hpf\\$"
-  "*Regexp of comment-like directive like \"!HPF\\\\$\", not to be indented.")
+(defcustom f90-program-indent 2
+  "*Extra indentation applied to PROGRAM/MODULE/SUBROUTINE/FUNCTION blocks."
+  :type 'integer
+  :group 'f90-indent)
 
-(defvar f90-beginning-ampersand t
-  "*t makes automatic insertion of \& at beginning of continuation line.")
+(defcustom f90-continuation-indent 5
+  "*Extra indentation applied to F90 continuation lines."
+  :type 'integer
+  :group 'f90-indent)
 
-(defvar f90-smart-end 'blink
+(defcustom f90-comment-region "!!$"
+  "*String inserted by \\[f90-comment-region]\
+ at start of each line in region."
+  :type 'string
+  :group 'f90-indent)
+
+(defcustom f90-indented-comment-re "!"
+  "*Regexp saying which comments to be indented like code."
+  :type 'regexp
+  :group 'f90-indent)
+
+(defcustom f90-directive-comment-re "!hpf\\$"
+  "*Regexp of comment-like directive like \"!HPF\\\\$\", not to be indented."
+  :type 'regexp
+  :group 'f90-indent)
+
+(defcustom f90-beginning-ampersand t
+  "*t makes automatic insertion of \& at beginning of continuation line."
+  :type 'boolean
+  :group 'f90)
+
+(defcustom f90-smart-end 'blink
   "*From an END statement, check and fill the end using matching block start.
 Allowed values are 'blink, 'no-blink, and nil, which determine
-whether to blink the matching beginning.")
+whether to blink the matching beginning."
+  :type '(choice (const blink) (const no-blink) (const nil))
+  :group 'f90)
 
-(defvar f90-break-delimiters "[-+\\*/><=,% \t]"
-  "*Regexp holding list of delimiters at which lines may be broken.")
+(defcustom f90-break-delimiters "[-+\\*/><=,% \t]"
+  "*Regexp holding list of delimiters at which lines may be broken."
+  :type 'regexp
+  :group 'f90)
 
-(defvar f90-break-before-delimiters t
-  "*Non-nil causes `f90-do-auto-fill' to break lines before delimiters.")
+(defcustom f90-break-before-delimiters t
+  "*Non-nil causes `f90-do-auto-fill' to break lines before delimiters."
+  :type 'boolean
+  :group 'f90)
 
-(defvar f90-auto-keyword-case nil
+(defcustom f90-auto-keyword-case nil
   "*Automatic case conversion of keywords.
-  The options are 'downcase-word, 'upcase-word, 'capitalize-word and nil")
+  The options are 'downcase-word, 'upcase-word, 'capitalize-word and nil"
+  :type '(choice (const downcase-word) (const upcase-word)
+                (const capitalize-word) (const nil))
+  :group 'f90)
 
-(defvar f90-leave-line-no nil
-  "*If nil, left-justify linenumbers.")
+(defcustom f90-leave-line-no nil
+  "*If nil, left-justify linenumbers."
+  :type 'boolean
+  :group 'f90)
 
-(defvar f90-startup-message t
-  "*Non-nil displays a startup message when F90 mode is first called.")
+(defcustom f90-startup-message t
+  "*Non-nil displays a startup message when F90 mode is first called."
+  :type 'boolean
+  :group 'f90)
 
 (defconst f90-keywords-re
-  ;;("allocate" "allocatable" "assign" "assignment" "backspace" "block"
-  ;;"call" "case" "character" "close" "common" "complex" "contains"
-  ;;"continue" "cycle" "data" "deallocate" "dimension" "do" "double" "else"
-  ;;"elseif" "elsewhere" "end" "enddo" "endfile" "endif" "entry" "equivalence"
-  ;;"exit" "external" "forall" "format" "function" "goto" "if" "implicit"
-  ;;"include" "inquire" "integer" "intent" "interface" "intrinsic" "logical"
-  ;;"module" "namelist" "none" "nullify" "only" "open" "optional" "parameter"
-  ;;"pause" "pointer" "precision" "print" "private" "procedure" "program"
-  ;;"public" "read" "real" "recursive" "return" "rewind" "save" "select"
-  ;;"sequence" "stop" "subroutine" "target" "then" "type" "use" "where"
-  ;;"while" "write")
-  (concat
-   "\\<\\(a\\(llocat\\(able\\|e\\)\\|ssign\\(\\|ment\\)\\)\\|b\\(ackspace\\|"
-   "lock\\)\\|c\\(a\\(ll\\|se\\)\\|haracter\\|lose\\|o\\(m\\(mon\\|plex\\)\\|"
-   "nt\\(ains\\|inue\\)\\)\\|ycle\\)\\|d\\(ata\\|eallocate\\|imension\\|"
-   "o\\(\\|uble\\)\\)\\|e\\(lse\\(\\|if\\|where\\)\\|n\\(d\\(\\|do\\|file\\|"
-   "if\\)\\|try\\)\\|quivalence\\|x\\(it\\|ternal\\)\\)\\|f\\(or\\(all\\|"
-   "mat\\)\\|unction\\)\\|goto\\|i\\(f\\|mplicit\\|n\\(clude\\|quire\\|t\\("
-   "e\\(ger\\|nt\\|rface\\)\\|rinsic\\)\\)\\)\\|logical\\|module\\|n\\("
-   "amelist\\|one\\|ullify\\)\\|o\\(nly\\|p\\(en\\|tional\\)\\)\\|p\\(a\\("
-   "rameter\\|use\\)\\|ointer\\|r\\(ecision\\|i\\(nt\\|vate\\)\\|o\\("
-   "cedure\\|gram\\)\\)\\|ublic\\)\\|re\\(a[dl]\\|cursive\\|turn\\|wind\\)\\|"
-   "s\\(ave\\|e\\(lect\\|quence\\)\\|top\\|ubroutine\\)\\|t\\(arget\\|hen\\|"
-   "ype\\)\\|use\\|w\\(h\\(ere\\|ile\\)\\|rite\\)\\)\\>")
+  (regexp-opt '("allocatable" "allocate" "assign" "assignment" "backspace"
+               "block" "call" "case" "character" "close" "common" "complex"
+               "contains" "continue" "cycle" "data" "deallocate"
+               "dimension" "do" "double" "else" "elseif" "elsewhere" "end"
+               "enddo" "endfile" "endif" "entry" "equivalence" "exit"
+               "external" "forall" "format" "function" "goto" "if"
+               "implicit" "include" "inquire" "integer" "intent"
+               "interface" "intrinsic" "logical" "module" "namelist" "none"
+               "nullify" "only" "open" "operator" "optional" "parameter"
+               "pause" "pointer" "precision" "print" "private" "procedure"
+               "program" "public" "read" "real" "recursive" "result" "return"
+               "rewind" "save" "select" "sequence" "stop" "subroutine"
+               "target" "then" "type" "use" "where" "while" "write"
+               ;; F95 keywords.
+               "elemental" "pure") 'words)
   "Regexp for F90 keywords.")
 
 (defconst f90-keywords-level-3-re
- ;; ("allocate" "allocatable" "assign" "assignment" "backspace" "close"
- ;; "deallocate" "endfile" "entry" "equivalence" "external" "inquire" "intent"
- ;; "intrinsic" "only" "open" "optional" "parameter" "pause" "pointer" "print"
- ;; "private" "public" "read" "recursive" "rewind" "save" "select" "sequence"
- ;; "target"  "write")
-  (concat
-   "\\<\\(a\\(llocat\\(able\\|e\\)\\|ssign\\(\\|ment\\)\\)\\|backspace\\|"
-   "close\\|deallocate\\|e\\(n\\(dfile\\|try\\)\\|quivalence\\|xternal\\)\\|"
-   "in\\(quire\\|t\\(ent\\|rinsic\\)\\)\\|o\\(nly\\|p\\(en\\|tional\\)\\)\\|"
-   "p\\(a\\(rameter\\|use\\)\\|ointer\\|ri\\(nt\\|vate\\)\\|ublic\\)\\|re\\("
-   "ad\\|cursive\\|wind\\)\\|s\\(ave\\|e\\(lect\\|quence\\)\\)\\|target\\|"
-   "write\\)\\>")
-"Keyword-regexp for font-lock level >= 3.")
+  (regexp-opt
+   '("allocatable" "allocate" "assign" "assignment" "backspace"
+     "close" "deallocate" "dimension" "endfile" "entry" "equivalence"
+     "external" "inquire" "intent" "intrinsic" "nullify" "only" "open"
+     "operator" "optional" "parameter" "pause" "pointer" "print" "private"
+     "public" "read" "recursive" "result" "rewind" "save" "select"
+     "sequence" "target" "write"
+     ;; F95 keywords.
+     "elemental" "pure") 'words)
+  "Keyword-regexp for font-lock level >= 3.")
 
 
 (defconst f90-procedures-re
-  ;; ("abs" "achar" "acos" "adjustl" "adjustr" "aimag" "aint" "all" "allocated"
-  ;; "anint" "any" "asin" "associated" "atan" "atan2" "bit_size" "btest"
-  ;; "ceiling" "char" "cmplx" "conjg" "cos" "cosh" "count" "cshift"
-  ;; "date_and_time" "dble" "digits" "dim" "dot_product" "dprod" "eoshift"
-  ;; "epsilon" "exp" "exponent" "floor" "fraction" "huge" "iachar" "iand"
-  ;; "ibclr" "ibits" "ibset" "ichar" "ieor" "index" "int" "ior" "ishft"
-  ;; "ishftc" "kind" "lbound" "len" "len_trim" "lge" "lgt" "lle" "llt" "log"
-  ;; "logical" "log10" "matmul" "max" "maxexponent" "maxloc" "maxval" "merge"
-  ;; "min" "minexponent" "minloc" "minval" "mod" "modulo" "mvbits" "nearest"
-  ;; "nint" "not" "pack" "precision" "present" "product" "radix"
-  ;; "random_number" "random_seed" "range" "real" "repeat" "reshape"
-  ;; "rrspacing" "scale" "scan" "selected_int_kind" "selected_real_kind"
-  ;; "set_exponent" "shape" "sign" "sin" "sinh" "size" "spacing" "spread"
-  ;; "sqrt" "sum" "system_clock" "tan" "tanh" "tiny" "transfer" "transpose"
-  ;; "trim" "ubound" "unpack" "verify")
-  ;; A left parenthesis to avoid highlighting non-procedures.
-  ;; Real is taken out here to avoid highlighting declarations.
-  (concat
-   "\\<\\(a\\(bs\\|c\\(har\\|os\\)\\|djust[lr]\\|i\\(mag\\|nt\\)\\|ll\\(\\|"
-   "ocated\\)\\|n\\(int\\|y\\)\\|s\\(in\\|sociated\\)\\|tan2?\\)\\|b\\("
-   "it_size\\|test\\)\\|c\\(eiling\\|har\\|mplx\\|o\\(njg\\|sh?\\|unt\\)\\|"
-   "shift\\)\\|d\\(ate_and_time\\|ble\\|i\\(gits\\|m\\)\\|ot_product\\|prod"
-   "\\)\\|e\\(oshift\\|psilon\\|xp\\(\\|onent\\)\\)\\|f\\(loor\\|"
-   "raction\\)\\|huge\\|i\\(a\\(char\\|nd\\)\\|b\\(clr\\|its\\|set\\)\\|"
-   "char\\|eor\\|n\\(dex\\|t\\)\\|or\\|shftc?\\)\\|kind\\|l\\(bound\\|"
-   "en\\(\\|_trim\\)\\|g[et]\\|l[et]\\|og\\(\\|10\\|ical\\)\\)\\|m\\(a\\("
-   "tmul\\|x\\(\\|exponent\\|loc\\|val\\)\\)\\|erge\\|in\\(\\|exponent\\|"
-   "loc\\|val\\)\\|od\\(\\|ulo\\)\\|vbits\\)\\|n\\(earest\\|int\\|ot\\)\\|"
-   "p\\(ack\\|r\\(e\\(cision\\|sent\\)\\|oduct\\)\\)\\|r\\(a\\(dix\\|n\\("
-   "dom_\\(number\\|seed\\)\\|ge\\)\\)\\|e\\(peat\\|shape\\)\\|rspacing\\)\\|"
-   "s\\(ca\\(le\\|n\\)\\|e\\(lected_\\(int_kind\\|real_kind\\)\\|"
-   "t_exponent\\)\\|hape\\|i\\(gn\\|nh?\\|ze\\)\\|p\\(acing\\|read\\)\\|"
-   "qrt\\|um\\|ystem_clock\\)\\|t\\(anh?\\|iny\\|r\\(ans\\(fer\\|pose\\)\\|"
-   "im\\)\\)\\|u\\(bound\\|npack\\)\\|verify\\)[ \t]*(")
+  (concat "\\<"
+   (regexp-opt
+    '("abs" "achar" "acos" "adjustl" "adjustr" "aimag" "aint"
+      "all" "allocated" "anint" "any" "asin" "associated"
+      "atan" "atan2" "bit_size" "btest" "ceiling" "char" "cmplx"
+      "conjg" "cos" "cosh" "count" "cshift" "date_and_time" "dble"
+      "digits" "dim" "dot_product" "dprod" "eoshift" "epsilon"
+      "exp" "exponent" "floor" "fraction" "huge" "iachar" "iand"
+      "ibclr" "ibits" "ibset" "ichar" "ieor" "index" "int" "ior"
+      "ishft" "ishftc" "kind" "lbound" "len" "len_trim" "lge" "lgt"
+      "lle" "llt" "log" "log10" "logical" "matmul" "max"
+      "maxexponent" "maxloc" "maxval" "merge" "min" "minexponent"
+      "minloc" "minval" "mod" "modulo" "mvbits" "nearest" "nint"
+      "not" "pack" "precision" "present" "product" "radix"
+      ;; Real is taken out here to avoid highlighting declarations.
+      "random_number" "random_seed" "range" ;; "real"
+      "repeat" "reshape" "rrspacing" "scale" "scan"
+      "selected_int_kind" "selected_real_kind" "set_exponent"
+      "shape" "sign" "sin" "sinh" "size" "spacing" "spread" "sqrt"
+      "sum" "system_clock" "tan" "tanh" "tiny" "transfer"
+      "transpose" "trim" "ubound" "unpack" "verify"
+      ;; F95 intrinsic functions.
+      "null" "cpu_time") t)
+   ;; A left parenthesis to avoid highlighting non-procedures.
+   "[ \t]*(")
   "Regexp whose first part matches F90 intrinsic procedures.")
 
 (defconst f90-operators-re
-;; "and" "or" "not" "eqv" "neqv" "eq" "ne" "lt" "le" "gt" "ge" "true" "false"
- (concat
-   "\\.\\(and\\|eqv?\\|false\\|g[et]\\|l[et]\\|n\\(e\\(\\|qv\\)\\|"
-   "ot\\)\\|or\\|true\\)\\.")
+ (concat "\\."
+        (regexp-opt '("and" "eq" "eqv" "false" "ge" "gt" "le" "lt" "ne"
+                      "neqv" "not" "or" "true") t)
+        "\\.")
   "Regexp matching intrinsic operators.")
 
 (defconst f90-hpf-keywords-re
-  ;; Intrinsic procedures
-  ;; ("all_prefix" "all_scatter" "all_suffix" "any_prefix" "any_scatter"
-  ;; "any_suffix" "copy_prefix" "copy_scatter" "copy_suffix" "count_prefix"
-  ;; "count_scatter" "count_suffix" "grade_down" "grade_up" "hpf_alignment"
-  ;; "hpf_template" "hpf_distribution" "iall" "iall_prefix" "iall_scatter"
-  ;; "iall_suffix" "iany" "iany_prefix" "iany_scatter" "iany_suffix" "iparity"
-  ;; "iparity_prefix" "iparity_scatter" "iparity_suffix" "leadz"
-  ;; "maxval_prefix" "maxval_scatter" "maxval_suffix" "minval_prefix"
-  ;; "minval_scatter" "minval_suffix" "parity" "parity_prefix"
-  ;; "parity_scatter" "parity_suffix" "popcnt" "poppar" "product_prefix"
-  ;; "product_scatter" "product_suffix" "sum_prefix" "sum_scatter"
-  ;; "sum_suffix" "ilen" "number_of_processors" "processors_shape")
-  ;; Directives
-  ;; ("align" "distribute" "dynamic" "inherit" "template" "processors"
-  ;; "realign" "redistribute" "independent")
-  ;; Keywords
-  ;; ("pure" "extrinsic" "new" "with" "onto" "block" "cyclic")
-  (concat
-   "\\<\\(a\\(l\\(ign\\|l_\\(prefix\\|s\\(catter\\|uffix\\)\\)\\)\\|ny_\\("
-   "prefix\\|s\\(catter\\|uffix\\)\\)\\)\\|block\\|c\\(o\\(py_\\(prefix\\|"
-   "s\\(catter\\|uffix\\)\\)\\|unt_\\(prefix\\|s\\(catter\\|uffix\\)\\)\\)\\|"
-   "yclic\\)\\|d\\(istribute\\|ynamic\\)\\|extrinsic\\|grade_\\(down\\|"
-   "up\\)\\|hpf_\\(alignment\\|distribution\\|template\\)\\|i\\(a\\(ll\\(\\|"
-   "_\\(prefix\\|s\\(catter\\|uffix\\)\\)\\)\\|ny\\(\\|_\\(prefix\\|s\\("
-   "catter\\|uffix\\)\\)\\)\\)\\|len\\|n\\(dependent\\|herit\\)\\|parity\\(\\|"
-   "_\\(prefix\\|s\\(catter\\|uffix\\)\\)\\)\\)\\|leadz\\|m\\(axval_\\("
-   "prefix\\|s\\(catter\\|uffix\\)\\)\\|inval_\\(prefix\\|s\\(catter\\|"
-   "uffix\\)\\)\\)\\|n\\(ew\\|umber_of_processors\\)\\|onto\\|p\\(arity\\(\\|"
-   "_\\(prefix\\|s\\(catter\\|uffix\\)\\)\\)\\|op\\(cnt\\|par\\)\\|ro\\("
-   "cessors\\(\\|_shape\\)\\|duct_\\(prefix\\|s\\(catter\\|uffix\\)\\)\\)\\|"
-   "ure\\)\\|re\\(align\\|distribute\\)\\|sum_\\(prefix\\|s\\(catter\\|"
-   "uffix\\)\\)\\|template\\|with\\)\\>")
+  (regexp-opt
+   ;; Intrinsic procedures
+   '("all_prefix" "all_scatter" "all_suffix" "any_prefix"
+     "any_scatter" "any_suffix" "copy_prefix" "copy_scatter"
+     "copy_suffix" "count_prefix" "count_scatter" "count_suffix"
+     "grade_down" "grade_up"
+     "hpf_alignment" "hpf_distribution" "hpf_template" "iall" "iall_prefix"
+     "iall_scatter" "iall_suffix" "iany" "iany_prefix" "iany_scatter"
+     "iany_suffix" "ilen" "iparity" "iparity_prefix"
+     "iparity_scatter" "iparity_suffix" "leadz" "maxval_prefix"
+     "maxval_scatter" "maxval_suffix" "minval_prefix" "minval_scatter"
+     "minval_suffix" "number_of_processors" "parity"
+     "parity_prefix" "parity_scatter" "parity_suffix" "popcnt" "poppar"
+     "processors_shape" "product_prefix" "product_scatter"
+     "product_suffix" "sum_prefix" "sum_scatter" "sum_suffix"
+     ;; Directives
+     "align" "distribute" "dynamic" "independent" "inherit" "processors"
+     "realign" "redistribute" "template" 
+     ;; Keywords
+     "block" "cyclic" "extrinsic" "new" "onto" "pure" "with") 'words)
   "Regexp for all HPF keywords, procedures and directives.")
 
 ;; Highlighting patterns
 
 (defvar f90-font-lock-keywords-1
-  (if (string-match "XEmacs" emacs-version)
-      (list                            ; XEmacs
-       '("\\<\\(end[ \t]*\\(program\\|module\\|function\\|subroutine\\|type\\)\\)\\>"
-        1 font-lock-keyword-face)
-       '("\\<\\(end[ \t]*\\(program\\|module\\|function\\|subroutine\\|type\\)\\)\\>[ \t]*\\(\\sw+\\)"
-        3 font-lock-function-name-face)
-       '("\\<\\(program\\|call\\|module\\|subroutine\\|function\\|use\\)\\>"
-        1 font-lock-keyword-face)
-       '("\\<\\(program\\|call\\|module\\|subroutine\\|function\\|use\\)\\>[ \t]*\\(\\sw+\\)"
-        2 font-lock-function-name-face nil t)
-       ;; Special highlighting of "module procedure foo-list"
-       '("\\<\\(module[ \t]*procedure\\)\\>" 1 font-lock-keyword-face t)
-       ;; Highlight definition of new type
-       '("\\<\\(type\\)[ \t]*\\(,.*::[ \t]*\\|[ \t]+\\)\\(\\sw+\\)"
-        1 font-lock-keyword-face)
-       '("\\<\\(type\\)[ \t]*\\(,.*::[ \t]*\\|[ \t]+\\)\\(\\sw+\\)"
-        3 font-lock-function-name-face)
-       "\\<\\(\\(end[ \t]*\\)?\\(interface\\|block[ \t]*data\\)\\|contains\\)\\>")
-    (list                              ; Emacs
-     '("\\<\\(end[ \t]*\\(program\\|module\\|function\\|subroutine\\|type\\)\\)\\>[ \t]*\\(\\sw+\\)?"
-       (1 font-lock-keyword-face) (3 font-lock-function-name-face nil t))
-     '("\\<\\(program\\|call\\|module\\|subroutine\\|function\\|use\\)\\>[ \t]*\\(\\sw+\\)?"
-       (1 font-lock-keyword-face) (2 font-lock-function-name-face nil t))
-     ;; Special highlighting of "module procedure foo-list"
-     '("\\<\\(module[ \t]*procedure\\)\\>" (1 font-lock-keyword-face t))
-     ;; Highlight definition of new type
-     '("\\<\\(type\\)[ \t]*\\(,.*::[ \t]*\\|[ \t]+\\)\\(\\sw+\\)"
-       (1 font-lock-keyword-face) (3 font-lock-function-name-face))
-     "\\<\\(\\(end[ \t]*\\)?\\(interface\\|block[ \t]*data\\)\\|contains\\)\\>"))
+  (list
+   ;; Special highlighting of "module procedure foo-list"
+   '("\\<\\(module[ \t]*procedure\\)\\>" (1 font-lock-keyword-face))
+   ;; Highlight definition of new type
+   '("\\<\\(type\\)[ \t]*\\(.*::[ \t]*\\|[ \t]+\\)\\(\\sw+\\)"
+     (1 font-lock-keyword-face) (3 font-lock-function-name-face))
+   ;; Other functions and declarations.
+   '("\\<\\(\\(?:end[ \t]*\\)?\\(program\\|module\\|function\\|subroutine\\|type\\)\\|use\\|call\\)\\>[ \t]*\\(\\sw+\\)?"
+     (1 font-lock-keyword-face) (3 font-lock-function-name-face nil t))
+   "\\<\\(\\(end[ \t]*\\)?\\(interface\\|block[ \t]*data\\)\\|contains\\)\\>")
   "This does fairly subdued highlighting of comments and function calls.")
 
 (defvar f90-font-lock-keywords-2
   (append f90-font-lock-keywords-1
-     (if (string-match "XEmacs" emacs-version)
-      (list                            ; XEmacs
-       ;; Variable declarations (avoid the real function call)
-       '("^[ \t0-9]*\\(real\\|integer\\|c\\(haracter\\|omplex\\)\\|logical\\|type[ \t]*(\\sw+)\\)"
-        1 font-lock-type-face)
-       '("^[ \t0-9]*\\(real\\|integer\\|c\\(haracter\\|omplex\\)\\|logical\\|type[ \t]*(\\sw+)\\)\\(.*::\\|[ \t]*(.*)\\)?\\(.*\\)"
-        4 font-lock-doc-string-face)
-       ;; do, if and select constructs
-       '("\\<\\(end[ \t]*\\(do\\|if\\|select\\)\\)\\>"
-        1 font-lock-keyword-face)
-       '("\\<\\(end[ \t]*\\(do\\|if\\|select\\)\\)\\>\\([ \t]+\\(\\sw+\\)\\)"
-        3 font-lock-doc-string-face)
-       '("^[ \t0-9]*\\(\\(\\sw+\\)[ \t]*:[ \t]*\\)\\(\\(if\\|do\\([ \t]*while\\)?\\|select[ \t]*case\\)\\)\\>"
-        2 font-lock-doc-string-face)
-       '("^[ \t0-9]*\\(\\(\\sw+\\)[ \t]*:[ \t]*\\)?\\(\\(if\\|do\\([ \t]*while\\)?\\|select[ \t]*case\\)\\)\\>"
-        3 font-lock-keyword-face)
-      ;; implicit declaration
-       '("\\<\\(implicit\\)[ \t]*\\(real\\|integer\\|c\\(haracter\\|omplex\\)\\|logical\\|type[ \t]*(\\sw+)\\|none\\)\\>"
-        1 font-lock-keyword-face)
-       '("\\<\\(implicit\\)[ \t]*\\(real\\|integer\\|c\\(haracter\\|omplex\\)\\|logical\\|type[ \t]*(\\sw+)\\|none\\)\\>"
-        2 font-lock-type-face)
-       '("\\<\\(namelist\\|common\\)[ \t]*\/\\(\\sw+\\)?\/"
-        1 font-lock-keyword-face)
-       '("\\<\\(namelist\\|common\\)[ \t]*\/\\(\\sw+\\)\/"
-        2 font-lock-doc-string-face nil t)
-       '("\\<\\(where\\|forall\\)[ \t]*(" . 1)
-       "\\<e\\(lse\\([ \t]*if\\|where\\)?\\|nd[ \t]*\\(where\\|forall\\)\\)\\>"
-       "\\<\\(then\\|continue\\|format\\|include\\|stop\\|return\\)\\>"
-       '("\\<\\(exit\\|cycle\\)\\>" 
-        1 font-lock-keyword-face)
-       '("\\<\\(exit\\|cycle\\)[ \t]*\\(\\sw+\\)2\\>" 
-        2 font-lock-doc-string-face)
-       '("\\<\\(case\\)[ \t]+\\(default\\|(\\)" . 1)
-       '("\\<\\(do\\|go *to\\)\\>[ \t]*\\([0-9]+\\)"
-        1 font-lock-keyword-face)
-       '("\\<\\(do\\|go *to\\)\\>[ \t]*\\([0-9]+\\)"
-        2 font-lock-doc-string-face)
-       '("^[ \t]*\\([0-9]+\\)" 1 font-lock-doc-string-face t))
-      (list                            ; Emacs
+      (list
        ;; Variable declarations (avoid the real function call)
-       '("^[ \t0-9]*\\(real\\|integer\\|c\\(haracter\\|omplex\\)\\|logical\\|type[ \t]*(\\sw+)\\)\\(.*::\\|[ \t]*(.*)\\)?\\(.*\\)"
+       '("^[ \t0-9]*\\(real\\|integer\\|c\\(haracter\\|omplex\\)\\|logical\\|type[ \t]*(\\sw+)\\)\\(.*::\\|[ \t]*(.*)\\)?\\([^!\n]*\\)"
         (1 font-lock-type-face) (4 font-lock-variable-name-face))
-       ;; do, if and select constructs
-       '("\\<\\(end[ \t]*\\(do\\|if\\|select\\)\\)\\>\\([ \t]+\\(\\sw+\\)\\)?"
-        (1 font-lock-keyword-face) (3 font-lock-reference-face nil t))
-       '("^[ \t0-9]*\\(\\(\\sw+\\)[ \t]*:[ \t]*\\)?\\(\\(if\\|do\\([ \t]*while\\)?\\|select[ \t]*case\\)\\)\\>"
-        (2 font-lock-reference-face nil t) (3 font-lock-keyword-face))
+       ;; do, if, select, where, and forall constructs
+       '("\\<\\(end[ \t]*\\(do\\|if\\|select\\|forall\\|where\\)\\)\\>\\([ \t]+\\(\\sw+\\)\\)?"
+        (1 font-lock-keyword-face) (3 font-lock-constant-face nil t))
+       '("^[ \t0-9]*\\(\\(\\sw+\\)[ \t]*:[ \t]*\\)?\\(\\(if\\|do\\([ \t]*while\\)?\\|select[ \t]*case\\|where\\|forall\\)\\)\\>"
+        (2 font-lock-constant-face nil t) (3 font-lock-keyword-face))
        ;; implicit declaration
        '("\\<\\(implicit\\)[ \t]*\\(real\\|integer\\|c\\(haracter\\|omplex\\)\\|logical\\|type[ \t]*(\\sw+)\\|none\\)\\>" (1 font-lock-keyword-face) (2 font-lock-type-face))
-       '("\\<\\(namelist\\|common\\)[ \t]*\/\\(\\sw+\\)?\/" (1 font-lock-keyword-face) (2 font-lock-reference-face nil t))
-       '("\\<\\(where\\|forall\\)[ \t]*(" . 1)
-       "\\<e\\(lse\\([ \t]*if\\|where\\)?\\|nd[ \t]*\\(where\\|forall\\)\\)\\>"
+       '("\\<\\(namelist\\|common\\)[ \t]*\/\\(\\sw+\\)?\/" (1 font-lock-keyword-face) (2 font-lock-constant-face nil t))
+       "\\<else\\([ \t]*if\\|where\\)?\\>"
        "\\<\\(then\\|continue\\|format\\|include\\|stop\\|return\\)\\>"
        '("\\<\\(exit\\|cycle\\)[ \t]*\\(\\sw+\\)?\\>" 
-        (1 font-lock-keyword-face) (2 font-lock-reference-face nil t))
-       '("\\<\\(case\\)[ \t]+\\(default\\|(\\)" . 1)
+        (1 font-lock-keyword-face) (2 font-lock-constant-face nil t))
+       '("\\<\\(case\\)[ \t]*\\(default\\|(\\)" . 1)
        '("\\<\\(do\\|go *to\\)\\>[ \t]*\\([0-9]+\\)"
-        (1 font-lock-keyword-face) (2 font-lock-reference-face))
-       '("^[ \t]*\\([0-9]+\\)" (1 font-lock-reference-face t)))))
+        (1 font-lock-keyword-face) (2 font-lock-constant-face))
+       ;; line numbers (lines whose first character after number is letter)
+       '("^[ \t]*\\([0-9]+\\)[ \t]*[a-z]+" (1 font-lock-constant-face t))))
   "Highlights declarations, do-loops and other constructions")
 
 (defvar f90-font-lock-keywords-3
@@ -418,11 +369,9 @@ whether to blink the matching beginning.")
    (list
     f90-keywords-level-3-re
     f90-operators-re
-    (if (string-match "XEmacs" emacs-version)
-       (append (list f90-procedures-re) '(1 font-lock-keyword-face t))
-      (list f90-procedures-re '(1 font-lock-keyword-face t)))
+    (list f90-procedures-re '(1 font-lock-keyword-face keep))
     "\\<real\\>"                       ; Avoid overwriting real defs.
-    ))
+   ))
   "Highlights all F90 keywords and intrinsic procedures.")
 
 (defvar f90-font-lock-keywords-4
@@ -453,7 +402,7 @@ whether to blink the matching beginning.")
   (modify-syntax-entry ?= "." f90-mode-syntax-table)
   (modify-syntax-entry ?* "." f90-mode-syntax-table)
   (modify-syntax-entry ?/ "." f90-mode-syntax-table)
-  (modify-syntax-entry ?\\ "/" f90-mode-syntax-table)) ; escape chars
+  (modify-syntax-entry ?\\ "\\" f90-mode-syntax-table)) ; escape chars
 
 ;; keys
 (defvar f90-mode-map ()
@@ -477,8 +426,14 @@ whether to blink the matching beginning.")
   (define-key f90-mode-map "\C-c\C-p" 'f90-previous-statement)
   (define-key f90-mode-map "\C-c\C-n" 'f90-next-statement)
   (define-key f90-mode-map "\C-c\C-w" 'f90-insert-end)
-  (define-key f90-mode-map "\t"       'f90-indent-line))
-
+  (define-key f90-mode-map "\t"       'f90-indent-line)
+  (define-key f90-mode-map ","        'f90-electric-insert)
+  (define-key f90-mode-map "+"        'f90-electric-insert)
+  (define-key f90-mode-map "-"        'f90-electric-insert)
+  (define-key f90-mode-map "*"        'f90-electric-insert)
+  (define-key f90-mode-map "/"        'f90-electric-insert))
+
 ;; menus
 (if (string-match "XEmacs" emacs-version)
     (defvar f90-xemacs-menu
@@ -495,7 +450,7 @@ whether to blink the matching beginning.")
        ["Break Line at Point"     f90-break-line t]
        ["Join with Next Line"     f90-join-lines t]
        ["Insert Newline"          newline t]
-       ["Insert End"              f90-insert-end t]
+       ["Insert Block End"        f90-insert-end t]
        "-----"
        ["Upcase Keywords (buffer)"      f90-upcase-keywords t]
        ["Upcase Keywords (region)"      f90-upcase-region-keywords
@@ -508,40 +463,135 @@ whether to blink the matching beginning.")
         f90-downcase-region-keywords t]
        "-----"
        ["Toggle abbrev-mode"   abbrev-mode             t]
-       ["Toggle auto-fill"     f90-auto-fill-mode      t])
+       ["Toggle auto-fill"     auto-fill-mode          t])
       "XEmacs menu for F90 mode.")
   ;; Emacs
+
+  (defvar f90-change-case-menu
+    (let ((map (make-sparse-keymap "Change Keyword Case")))
+      
+      (define-key map [dkr] (cons "Downcase Keywords (region)"
+                                 'f90-downcase-region-keywords))
+      (put 'f90-downcase-region-keywords 'menu-enable 'mark-active)
+      
+      (define-key map [ckr] (cons "Capitalize Keywords (region)"
+                                 'f90-capitalize-region-keywords))
+      (put 'f90-capitalize-region-keywords 'menu-enable 'mark-active)
+      
+      (define-key map [ukr] (cons "Upcase Keywords (region)"
+                                 'f90-upcase-region-keywords))
+      (put 'f90-upcase-region-keywords 'menu-enable 'mark-active)
+      
+      (define-key map [line] (list "-----------------"))
+      
+      (define-key map [dkb] (cons "Downcase Keywords (buffer)"
+                                 'f90-downcase-keywords))
+      
+      (define-key map [ckb] (cons "Capitalize Keywords (buffer)"
+                                 'f90-capitalize-keywords))
+      
+      (define-key map [ukb] (cons "Upcase Keywords (buffer)"
+                                 'f90-upcase-keywords))
+      map)
+    "Submenu for change of case.")
+  (defalias 'f90-change-case-menu f90-change-case-menu)
+
+  ;; font-lock-menu and function calls
+  (defalias 'f90-font-lock-on  'font-lock-mode)
+  (defalias 'f90-font-lock-off 'font-lock-mode)
+  (put 'f90-font-lock-on  'menu-enable 'font-lock-mode)
+  (put 'f90-font-lock-off 'menu-enable '(not font-lock-mode))
+  
+  (defun f90-font-lock-1 ()
+    (interactive)
+    "Set font-lock-keywords to f90-font-lock-keywords-1."
+    (font-lock-mode 1)
+    (setq font-lock-keywords f90-font-lock-keywords-1)
+    (font-lock-fontify-buffer))
+    
+  (defun f90-font-lock-2 ()
+    (interactive)
+    "Set font-lock-keywords to f90-font-lock-keywords-2."
+    (font-lock-mode 1)
+    (setq font-lock-keywords f90-font-lock-keywords-2)
+    (font-lock-fontify-buffer))
+    
+  (defun f90-font-lock-3 ()
+    (interactive)
+    "Set font-lock-keywords to f90-font-lock-keywords-3."
+    (font-lock-mode 1)
+    (setq font-lock-keywords f90-font-lock-keywords-3)
+    (font-lock-fontify-buffer))
+    
+  (defun f90-font-lock-4 ()
+    (interactive)
+    "Set font-lock-keywords to f90-font-lock-keywords-4."
+    (font-lock-mode 1)
+    (setq font-lock-keywords f90-font-lock-keywords-4)
+    (font-lock-fontify-buffer))
+    
+  (defvar f90-font-lock-menu
+    (let ((map (make-sparse-keymap "f90-font-lock-menu")))
+      (define-key map [h4] (cons "Maximum highlighting (level 4)"
+                                'f90-font-lock-4))
+      (define-key map [h3] (cons "Heavy highlighting (level 3)"
+                                'f90-font-lock-3))
+      (define-key map [h2] (cons "Default highlighting (level 2)"
+                                'f90-font-lock-2))
+      (define-key map [h1] (cons "Light highlighting (level 1)"
+                                'f90-font-lock-1))
+      (define-key map [line] (list "-----------------"))
+      (define-key map [floff] (cons "Turn off font-lock-mode"
+                                   'f90-font-lock-on))
+      (define-key map [flon] (cons "Turn on font-lock-mode"
+                                  'f90-font-lock-off))
+      map)
+    "Submenu for highlighting using font-lock-mode.")
+  (defalias 'f90-font-lock-menu f90-font-lock-menu)
+
   (define-key f90-mode-map [menu-bar] (make-sparse-keymap))
   (define-key f90-mode-map [menu-bar f90] 
     (cons "F90" (make-sparse-keymap "f90"))) 
+
+  (define-key f90-mode-map [menu-bar f90 f90-imenu-menu]
+    '("Add imenu Menu" . f90-add-imenu-menu))
   (define-key f90-mode-map [menu-bar f90 abbrev-mode]
     '("Toggle abbrev-mode" . abbrev-mode))
-  (define-key f90-mode-map [menu-bar f90 f90-auto-fill-mode]
-    '("Toggle auto-fill" . f90-auto-fill-mode))
-  (define-key f90-mode-map [menu-bar f90 f90-downcase-region-keywords]
-    '("Downcase Keywords (region)" . f90-downcase-region-keywords))
-  (define-key f90-mode-map [menu-bar f90 f90-downcase-keywords]
-    '("Downcase Keywords (buffer)" . f90-downcase-keywords))
-  (define-key f90-mode-map [menu-bar f90 f90-capitalize-keywords]
-    '("Capitalize Keywords (region)" . f90-capitalize-region-keywords))
-  (define-key f90-mode-map [menu-bar f90 f90-capitalize-region-keywords]
-    '("Capitalize Keywords (buffer)" . f90-capitalize-keywords))
-  (define-key f90-mode-map [menu-bar f90 f90-upcase-region-keywords]
-    '("Upcase Keywords (region)" . f90-upcase-region-keywords))
-  (define-key f90-mode-map [menu-bar f90 f90-upcase-keywords]
-    '("Upcase Keywords (buffer)" . f90-upcase-keywords))
+  (define-key f90-mode-map [menu-bar f90 auto-fill-mode]
+    '("Toggle auto-fill" . auto-fill-mode))
+  (define-key f90-mode-map [menu-bar f90 line1]
+    '("----"))
+  (define-key f90-mode-map [menu-bar f90 f90-change-case-menu]
+    (cons "Change Keyword Case" 'f90-change-case-menu))
+  (define-key f90-mode-map [menu-bar f90 f90-font-lock-menu]
+    (cons "Highlighting" 'f90-font-lock-menu))
+  (define-key f90-mode-map [menu-bar f90 line2]
+    '("----"))
+
   (define-key f90-mode-map [menu-bar f90 f90-insert-end]
-    '("Insert end" . f90-insert-end))
+    '("Insert Block End" . f90-insert-end))
   (define-key f90-mode-map [menu-bar f90 f90-join-lines]
     '("Join with Next Line" . f90-join-lines))
   (define-key f90-mode-map [menu-bar f90 f90-break-line]
     '("Break Line at Point" . f90-break-line))
+  
+  (define-key f90-mode-map [menu-bar f90 line3]
+    '("----"))
+  
   (define-key f90-mode-map [menu-bar f90 f90-fill-region]
     '("Fill Region" . f90-fill-region))
+  (put 'f90-fill-region 'menu-enable 'mark-active)
+  
   (define-key f90-mode-map [menu-bar f90 indent-region]
     '("Indent Region" . indent-region))
+  
   (define-key f90-mode-map [menu-bar f90 f90-comment-region]
     '("(Un)Comment Region" . f90-comment-region))
+  (put 'f90-comment-region 'menu-enable 'mark-active)
+  
+  (define-key f90-mode-map [menu-bar f90 line4]
+    '("----"))
+  
   (define-key f90-mode-map [menu-bar f90 f90-end-of-subprogram]
     '("End of Subprogram" . f90-end-of-subprogram))
   (define-key f90-mode-map [menu-bar f90 f90-beginning-of-subprogram]
@@ -549,8 +599,9 @@ whether to blink the matching beginning.")
   (define-key f90-mode-map [menu-bar f90 f90-mark-subprogram]
     '("Mark Subprogram" . f90-mark-subprogram))
   (define-key f90-mode-map [menu-bar f90 f90-indent-subprogram]
-    '("Indent Subprogram" . f90-indent-subprogram)))
-  
+    '("Indent Subprogram" . f90-indent-subprogram))
+  )
+
 ;; Regexps for finding program structures.
 (defconst f90-blocks-re 
   "\\(block[ \t]*data\\|do\\|if\\|interface\\|function\\|module\\|\
@@ -558,33 +609,61 @@ program\\|select\\|subroutine\\|type\\|where\\|forall\\)\\>")
 (defconst f90-program-block-re 
   "\\(program\\|module\\|subroutine\\|function\\)")
 (defconst f90-else-like-re 
-  "\\(else\\([ \t]*if\\|where\\)?\\|case[ \t]*\\(default\\|(\\)\\)\\>")
+  "\\(else\\([ \t]*if\\|where\\)?\\|case[ \t]*\\(default\\|(\\)\\)")
 (defconst f90-end-if-re 
   "end[ \t]*\\(if\\|select\\|where\\|forall\\)\\>")
 (defconst f90-end-type-re 
   "end[ \t]*\\(type\\|interface\\|block[ \t]*data\\)")
 (defconst f90-type-def-re
-  "\\<\\(type\\)[ \t]*\\(,.*::[ \t]*\\|[ \t]+\\)\\(\\sw+\\)")
+  "\\<\\(type\\)\\([^(\n]*\\)\\(::\\)?[ \t]*\\b\\(\\sw+\\)")
 (defconst f90-no-break-re  "\\(\\*\\*\\|//\\|=>\\)")
 ;; A temporary position to make region operators faster
 (defvar f90-cache-position nil)
 (make-variable-buffer-local 'f90-cache-position)
+;; A flag to tell whether f90-imenu is turned on.
+(defvar f90-imenu nil)
+(make-variable-buffer-local 'f90-imenu)
 
+\f
 ;; Imenu support
 (defvar f90-imenu-generic-expression
-  (cons
-   (concat
-    "^[ \t0-9]*\\("
-    "program[ \t]+\\(\\sw+\\)\\|"
-    "module[ \t]+\\(\\sw+\\)[ \t]*\\(!\\|$\\)\\|"
-    "\\(recursive[ \t]*\\)?subroutine[ \t]+\\(\\sw+\\)\\|"
-    ; avoid end function, but allow for most other things
-    "\\([^!]*\\([^e!].[^ \t!]\\|.[^n!][^ \t!]\\|..[^d \t!]\\)"
-    "\\|[^!]?[^!]?\\)[ \t]*function[ \t]+\\(\\sw+\\)"
-    "\\)")
-   (list 2 3 6 9))
+  (let ((good-char "[^!\"\&\n \t]") (not-e "[^e!\n\"\& \t]")
+       (not-n "[^n!\n\"\& \t]") (not-d "[^d!\n\"\& \t]"))
+    (list
+     '(nil "^[ \t0-9]*program[ \t]+\\(\\sw+\\)" 1)
+     '("Modules" "^[ \t0-9]*module[ \t]+\\(\\sw+\\)[ \t]*\\(!\\|$\\)" 1)
+     '("Types" "^[ \t0-9]*type[ \t]+\\(\\sw+\\)" 1)
+     (list
+      "Procedures" 
+      (concat
+       "^[ \t0-9]*"
+       "\\("
+       ;; At least three non-space characters before function/subroutine
+       ;; Check that the last three non-space characters don't spell E N D
+       "[^!\"\&\n]*\\("
+       not-e good-char good-char "\\|"
+       good-char not-n good-char "\\|"
+       good-char good-char not-d "\\)"
+       "\\|"
+       ;; Less than three non-space characters before function/subroutine
+       good-char "?" good-char "?"
+       "\\)"
+       "[ \t]*\\(function\\|subroutine\\)[ \t]+\\(\\sw+\\)")
+    4)))
   "imenu generic expression for F90 mode.")
 
+(defun f90-add-imenu-menu ()
+  (interactive)
+  "Add an imenu menu to the menubar."
+  (if (not f90-imenu)
+      (progn
+       (imenu-add-to-menubar "F90-imenu")
+       (redraw-frame (selected-frame))
+       (setq f90-imenu t))
+    (message "%s" "F90-imenu already exists.")))
+(put 'f90-add-imenu-menu 'menu-enable '(not f90-imenu))
+
+
 ;; When compiling under GNU Emacs, load imenu during compilation.  If
 ;; you have 19.22 or earlier, comment this out, or get imenu.
 (and (fboundp 'eval-when-compile)
@@ -592,8 +671,6 @@ program\\|select\\|subroutine\\|type\\|where\\|forall\\)\\>")
        (if (not (string-match "XEmacs" emacs-version))
           (require 'imenu))
        ()))
-
-
 \f
 ;; abbrevs have generally two letters, except standard types `c, `i, `r, `t
 (defvar f90-mode-abbrev-table nil)
@@ -601,61 +678,67 @@ program\\|select\\|subroutine\\|type\\|where\\|forall\\)\\>")
     ()
   (let ((ac abbrevs-changed))
     (define-abbrev-table 'f90-mode-abbrev-table ())
-    (define-abbrev f90-mode-abbrev-table  "`al"  "allocate" nil)
-    (define-abbrev f90-mode-abbrev-table  "`ab"  "allocatable" nil)
-    (define-abbrev f90-mode-abbrev-table  "`as"  "assignment" nil)
-    (define-abbrev f90-mode-abbrev-table  "`ba"  "backspace" nil)
-    (define-abbrev f90-mode-abbrev-table  "`bd"  "block data" nil)
-    (define-abbrev f90-mode-abbrev-table  "`c"   "character" nil)
-    (define-abbrev f90-mode-abbrev-table  "`cl"  "close" nil)
-    (define-abbrev f90-mode-abbrev-table  "`cm"  "common" nil)
-    (define-abbrev f90-mode-abbrev-table  "`cx"  "complex" nil)
-    (define-abbrev f90-mode-abbrev-table  "`cn"  "contains" nil)
-    (define-abbrev f90-mode-abbrev-table  "`cy"  "cycle" nil)
-    (define-abbrev f90-mode-abbrev-table  "`de"  "deallocate" nil)
-    (define-abbrev f90-mode-abbrev-table  "`df"  "define" nil)
-    (define-abbrev f90-mode-abbrev-table  "`di"  "dimension" nil)
-    (define-abbrev f90-mode-abbrev-table  "`dw"  "do while" nil)
-    (define-abbrev f90-mode-abbrev-table  "`el"  "else" nil)
-    (define-abbrev f90-mode-abbrev-table  "`eli" "else if" nil)
-    (define-abbrev f90-mode-abbrev-table  "`elw" "elsewhere" nil)
-    (define-abbrev f90-mode-abbrev-table  "`eq"  "equivalence" nil)
-    (define-abbrev f90-mode-abbrev-table  "`ex"  "external" nil)
-    (define-abbrev f90-mode-abbrev-table  "`ey"  "entry" nil)
-    (define-abbrev f90-mode-abbrev-table  "`fl"  "forall" nil)
-    (define-abbrev f90-mode-abbrev-table  "`fo"  "format" nil)
-    (define-abbrev f90-mode-abbrev-table  "`fu"  "function" nil)
-    (define-abbrev f90-mode-abbrev-table  "`fa"  ".false." nil)
-    (define-abbrev f90-mode-abbrev-table  "`im"  "implicit none" nil)
-    (define-abbrev f90-mode-abbrev-table  "`in " "include" nil)
-    (define-abbrev f90-mode-abbrev-table  "`i"   "integer" nil)
-    (define-abbrev f90-mode-abbrev-table  "`it"  "intent" nil)
-    (define-abbrev f90-mode-abbrev-table  "`if"  "interface" nil)
-    (define-abbrev f90-mode-abbrev-table  "`lo"  "logical" nil)
-    (define-abbrev f90-mode-abbrev-table  "`mo"  "module" nil)
-    (define-abbrev f90-mode-abbrev-table  "`na"  "namelist" nil)
-    (define-abbrev f90-mode-abbrev-table  "`nu"  "nullify" nil)
-    (define-abbrev f90-mode-abbrev-table  "`op"  "optional" nil)
-    (define-abbrev f90-mode-abbrev-table  "`pa"  "parameter" nil)
-    (define-abbrev f90-mode-abbrev-table  "`po"  "pointer" nil)
-    (define-abbrev f90-mode-abbrev-table  "`pr"  "print" nil)
-    (define-abbrev f90-mode-abbrev-table  "`pi"  "private" nil)
-    (define-abbrev f90-mode-abbrev-table  "`pm"  "program" nil)
-    (define-abbrev f90-mode-abbrev-table  "`pu"  "public" nil)
-    (define-abbrev f90-mode-abbrev-table  "`r"   "real" nil)
-    (define-abbrev f90-mode-abbrev-table  "`rc"  "recursive" nil)
-    (define-abbrev f90-mode-abbrev-table  "`rt"  "return" nil)
-    (define-abbrev f90-mode-abbrev-table  "`rw"  "rewind" nil)
-    (define-abbrev f90-mode-abbrev-table  "`se"  "select" nil)
-    (define-abbrev f90-mode-abbrev-table  "`sq"  "sequence" nil)
-    (define-abbrev f90-mode-abbrev-table  "`su"  "subroutine" nil)
-    (define-abbrev f90-mode-abbrev-table  "`ta"  "target" nil)
-    (define-abbrev f90-mode-abbrev-table  "`tr"  ".true." nil)
-    (define-abbrev f90-mode-abbrev-table  "`t"   "type" nil)
-    (define-abbrev f90-mode-abbrev-table  "`wh"  "where" nil)
-    (define-abbrev f90-mode-abbrev-table  "`wr"  "write" nil)
+    (define-abbrev f90-mode-abbrev-table  "`al"  "allocate" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`ab"  "allocatable" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`as"  "assignment" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`ba"  "backspace" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`bd"  "block data" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`c"   "character" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`cl"  "close" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`cm"  "common" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`cx"  "complex" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`cn"  "contains" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`cy"  "cycle" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`de"  "deallocate" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`df"  "define" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`di"  "dimension" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`dw"  "do while" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`el"  "else" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`eli" "else if" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`elw" "elsewhere" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`eq"  "equivalence" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`ex"  "external" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`ey"  "entry" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`fl"  "forall" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`fo"  "format" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`fu"  "function" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`fa"  ".false." nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`im"  "implicit none" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`in " "include" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`i"   "integer" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`it"  "intent" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`if"  "interface" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`lo"  "logical" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`mo"  "module" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`na"  "namelist" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`nu"  "nullify" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`op"  "optional" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`pa"  "parameter" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`po"  "pointer" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`pr"  "print" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`pi"  "private" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`pm"  "program" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`pu"  "public" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`r"   "real" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`rc"  "recursive" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`rt"  "return" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`rw"  "rewind" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`se"  "select" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`sq"  "sequence" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`su"  "subroutine" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`ta"  "target" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`tr"  ".true." nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`t"   "type" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`wh"  "where" nil 0 t)
+    (define-abbrev f90-mode-abbrev-table  "`wr"  "write" nil 0 t)
     (setq abbrevs-changed ac)))
 \f
+(defcustom f90-mode-hook nil
+  "Hook run by F90 mode."
+  :type 'hook
+  :options '(f90-add-imenu-menu)
+  :group 'f90)
+
 ;;;###autoload
 (defun f90-mode ()
   "Major mode for editing Fortran 90 code in free format.
@@ -712,7 +795,7 @@ Variables controlling indentation style and extra features:
     Do not left-justify line numbers. (default nil)
  f90-startup-message
     Set to nil to inhibit message first time F90 mode is used. (default t)
- f90-keywords
+ f90-keywords-re
     List of keywords used for highlighting/upcase-keywords etc.
 
 Turning on F90 mode calls the value of the variable `f90-mode-hook'
@@ -738,41 +821,37 @@ with no args, if that value is non-nil."
   (setq comment-indent-function 'f90-comment-indent)
   (make-local-variable 'abbrev-all-caps)
   (setq abbrev-all-caps t)
+  (make-local-variable 'normal-auto-fill-function)
+  (setq normal-auto-fill-function 'f90-do-auto-fill)
   (setq indent-tabs-mode nil)
   ;; Setting up things for font-lock
-  (if (string-match "Xemacs" emacs-version)
+  (if (string-match "XEmacs" emacs-version)
       (progn
        (put 'f90-mode 'font-lock-keywords-case-fold-search t)
-       (if (and current-menubar
+       (if (and (featurep 'menubar)
+                current-menubar
                 (not (assoc "F90" current-menubar)))
            (progn
              (set-buffer-menubar (copy-sequence current-menubar))
-             (add-submenu nil f90-xemacs-menu)))
-       (make-local-variable 'font-lock-keywords)
-       (setq font-lock-keywords f90-font-lock-keywords))
-    ;; Emacs
-    (make-local-variable 'font-lock-defaults)
-    (setq font-lock-defaults 
-         '((f90-font-lock-keywords f90-font-lock-keywords-1
-                                   f90-font-lock-keywords-2
-                                   f90-font-lock-keywords-3
-                                   f90-font-lock-keywords-4)
-           nil t))
-    ;; Tell imenu how to handle f90.
-    (make-local-variable 'imenu-generic-expression)
-    (setq imenu-generic-expression f90-imenu-generic-expression))
+             (add-submenu nil f90-xemacs-menu)))))
+  ;; XEmacs: (Don't need a special case, since both emacsen work alike -sb)
+  (make-local-variable 'font-lock-defaults)
+  (setq font-lock-defaults 
+       '((f90-font-lock-keywords f90-font-lock-keywords-1
+                                 f90-font-lock-keywords-2
+                                 f90-font-lock-keywords-3
+                                 f90-font-lock-keywords-4)
+         nil t))
+  ;; Tell imenu how to handle f90.
+  (set (make-local-variable 'imenu-case-fold-search) t)
+  (make-local-variable 'imenu-generic-expression)
+  (setq imenu-generic-expression f90-imenu-generic-expression)
+  (set (make-local-variable 'add-log-current-defun-function)
+       #'f90-current-defun)
   (run-hooks 'f90-mode-hook)
-  (if f90-startup-message
-      (message "Emacs F90 mode; please report bugs to %s" bug-f90-mode))
   (setq f90-startup-message nil))
 \f
 ;; inline-functions
-(defsubst f90-get-beg-of-line ()
-  (save-excursion (beginning-of-line) (point)))
-
-(defsubst f90-get-end-of-line ()
-  (save-excursion (end-of-line) (point)))
-
 (defsubst f90-in-string ()
   (let ((beg-pnt
         (if (and f90-cache-position (> (point) f90-cache-position))
@@ -789,7 +868,9 @@ with no args, if that value is non-nil."
 
 (defsubst f90-line-continued ()
   (save-excursion
-    (let ((bol (f90-get-beg-of-line)))
+    (beginning-of-line)
+    (while (and (looking-at "[ \t]*\\(!\\|$\\)") (zerop (forward-line -1))))
+    (let ((bol (line-beginning-position)))
       (end-of-line)
       (while (f90-in-comment)
        (search-backward "!" bol)
@@ -801,8 +882,7 @@ with no args, if that value is non-nil."
   "Return indentation of current line.
 Line-numbers are considered whitespace characters."
   (save-excursion
-    (beginning-of-line) (skip-chars-forward " \t0-9")
-    (current-column)))
+    (beginning-of-line) (skip-chars-forward " \t0-9")))
 
 (defsubst f90-indent-to (col &optional no-line-number)
   "Indent current line to column COL.
@@ -815,20 +895,16 @@ If no-line-number nil, jump over a possible line-number."
       (indent-to col)
     (indent-to col 1)))
 
-(defsubst f90-match-piece (arg)
-  (if (match-beginning arg)
-      (buffer-substring (match-beginning arg) (match-end arg))))
-
 (defsubst f90-get-present-comment-type ()
   (save-excursion
-    (let ((type nil) (eol (f90-get-end-of-line)))
+    (let ((type nil) (eol (line-end-position)))
       (if (f90-in-comment)
          (progn
            (beginning-of-line)
            (re-search-forward "[!]+" eol)
            (while (f90-in-string)
              (re-search-forward "[!]+" eol))
-           (setq type (buffer-substring (match-beginning 0) (match-end 0)))))
+           (setq type (match-string 0))))
       type)))
 
 (defsubst f90-equal-symbols (a b)
@@ -845,9 +921,9 @@ If no-line-number nil, jump over a possible line-number."
 Name is nil if the statement has no label."
   (if (looking-at "\\(\\(\\sw+\\)[ \t]*\:\\)?[ \t]*\\(do\\)\\>")
       (let (label
-           (struct (f90-match-piece 3)))
+           (struct (match-string 3)))
        (if (looking-at "\\(\\sw+\\)[ \t]*\:")
-           (setq label (f90-match-piece 1)))
+           (setq label (match-string 1)))
        (list struct label))))
 
 (defsubst f90-looking-at-select-case ()
@@ -855,9 +931,9 @@ Name is nil if the statement has no label."
 Name is nil if the statement has no label."
   (if (looking-at "\\(\\(\\sw+\\)[ \t]*\:\\)?[ \t]*\\(select\\)[ \t]*case[ \t]*(")
       (let (label
-           (struct (f90-match-piece 3)))
+           (struct (match-string 3)))
        (if (looking-at "\\(\\sw+\\)[ \t]*\:")
-           (setq label (f90-match-piece 1)))
+           (setq label (match-string 1)))
        (list struct label))))
 
 (defsubst f90-looking-at-if-then ()
@@ -867,10 +943,11 @@ Name is nil if the statement has no label."
     (let (struct (label nil))
       (if (looking-at "\\(\\(\\sw+\\)[ \t]*\:\\)?[ \t]*\\(if\\)\\>")
          (progn
-           (setq struct (f90-match-piece 3))
+           (setq struct (match-string 3))
            (if (looking-at "\\(\\sw+\\)[ \t]*\:")
-               (setq label (f90-match-piece 1)))
-           (goto-char (scan-lists (point) 1 0))
+               (setq label (match-string 1)))
+           (let ((pos (scan-lists (point) 1 0)))
+             (and pos (goto-char pos)))
            (skip-chars-forward " \t")
            (if (or (looking-at "then\\>")
                    (if (f90-line-continued)
@@ -881,52 +958,52 @@ Name is nil if the statement has no label."
                (list struct label)))))))
 
 (defsubst f90-looking-at-where-or-forall ()
-  "Return (kind nil) if where/forall...end starts after point."
-  (save-excursion
-    (let (command)
-      (if (looking-at "\\(where\\|forall\\)[ \t]*(")
-         (progn
-           (setq command (list (f90-match-piece 1) nil))
-           (goto-char (scan-lists (point) 1 0))
-           (skip-chars-forward " \t")
-           (if (looking-at "\\(!\\|$\\)")
-               command))))))
+  "Return (kind name) if a where or forall block starts after point.
+Name is nil if the statement has no label."
+  (if (looking-at "\\(\\(\\sw+\\)[ \t]*\:\\)?[ \t]*\\(where\\|forall\\)[ \t]*(.*)[ \t]*\\(!\\|$\\)")
+      (let (label
+           (struct (match-string 3)))
+       (if (looking-at "\\(\\sw+\\)[ \t]*\:")
+           (setq label (match-string 1)))
+       (list struct label))))
 
 (defsubst f90-looking-at-type-like ()
   "Return (kind name) at the start of a type/interface/block-data block.
 Name is non-nil only for type."
   (cond 
    ((looking-at f90-type-def-re)
-    (list (f90-match-piece 1) (f90-match-piece 3)))
+    (list (match-string 1) (match-string 4)))
    ((looking-at "\\(interface\\|block[\t]*data\\)\\>")
-    (list (f90-match-piece 1) nil))))
+    (list (match-string 1) nil))))
 
 (defsubst f90-looking-at-program-block-start ()
   "Return (kind name) if a program block with name name starts after point."
   (cond
    ((looking-at "\\(program\\)[ \t]+\\(\\sw+\\)\\>")
-    (list (f90-match-piece 1) (f90-match-piece 2)))
+    (list (match-string 1) (match-string 2)))
    ((and (not (looking-at "module[ \t]*procedure\\>"))
         (looking-at "\\(module\\)[ \t]+\\(\\sw+\\)\\>"))
-    (list (f90-match-piece 1) (f90-match-piece 2)))
-   ((looking-at "\\(recursive[ \t]*\\)?\\(subroutine\\)[ \t]+\\(\\sw+\\)")
-    (list (f90-match-piece 2) (f90-match-piece 3)))
-   ((and (not (looking-at "end[ \t]*function"))
-        (looking-at ".*\\(function\\)[ \t]+\\(\\sw+\\)"))
-    (list (f90-match-piece 1) (f90-match-piece 2)))))
+    (list (match-string 1) (match-string 2)))
+   ((and (not (looking-at "end[ \t]*\\(function\\|subroutine\\)"))
+        (looking-at "[^!'\"\&\n]*\\(function\\|subroutine\\)[ \t]+\\(\\sw+\\)"))
+    (list (match-string 1) (match-string 2)))))
 
 (defsubst f90-looking-at-program-block-end ()
   "Return list of type and name of end of block."
   (if (looking-at (concat "end[ \t]*" f90-blocks-re 
                          "?\\([ \t]+\\(\\sw+\\)\\)?\\>"))
-      (list (f90-match-piece 1) (f90-match-piece 3))))
+      (list (match-string 1) (match-string 3))))
 
 (defsubst f90-comment-indent ()
   (cond ((looking-at "!!!") 0)
        ((and f90-directive-comment-re
              (looking-at f90-directive-comment-re)) 0)
        ((looking-at (regexp-quote f90-comment-region)) 0)
-       ((looking-at f90-indented-comment-re)
+       ((and (looking-at f90-indented-comment-re)
+             ;; Don't attempt to indent trailing comment as code.
+             (save-excursion
+               (skip-chars-backward " \t")
+               (bolp)))
         (f90-calculate-indent))
        (t (skip-chars-backward " \t")
           (max (if (bolp) 0 (1+ (current-column))) comment-column))))
@@ -952,11 +1029,11 @@ Name is non-nil only for type."
   (skip-chars-forward " \t0-9"))
 
 (defsubst f90-no-block-limit ()
-  (let ((eol (f90-get-end-of-line)))
+  (let ((eol (line-end-position)))
     (save-excursion
       (not (or (looking-at "end")
-              (looking-at "\\(do\\|if\\|else\\|select[ \t]*case\\|\
-case\\|where\\|forall\\)\\>")
+              (looking-at "\\(do\\|if\\|else\\(if\\|where\\)?\
+\\|select[ \t]*case\\|case\\|where\\|forall\\)\\>")
               (looking-at "\\(program\\|module\\|interface\\|\
 block[ \t]*data\\)\\>")
               (looking-at "\\(contains\\|\\sw+[ \t]*:\\)")
@@ -966,15 +1043,22 @@ block[ \t]*data\\)\\>")
 (defsubst f90-update-line ()
   (let (bol eol)
     (if f90-auto-keyword-case
-       (progn (setq bol (f90-get-beg-of-line)
-                    eol (f90-get-end-of-line))
+       (progn (setq bol (line-beginning-position)
+                    eol (line-end-position))
               (if f90-auto-keyword-case
                   (f90-change-keywords f90-auto-keyword-case bol eol))))))
 \f
+(defun f90-electric-insert ()
+  (interactive)
+  "Calls f90-do-auto-fill at each operator insertion."
+  (self-insert-command 1)
+  (f90-update-line)
+  (if auto-fill-function (f90-do-auto-fill)))
+
 (defun f90-get-correct-indent ()
   "Get correct indent for a line starting with line number.
 Does not check type and subprogram indentation."
-  (let ((epnt (f90-get-end-of-line)) icol cont)
+  (let ((epnt (line-end-position)) icol cont)
     (save-excursion
       (while (and (f90-previous-statement)
                  (or (progn
@@ -984,7 +1068,7 @@ Does not check type and subprogram indentation."
       (setq icol (current-indentation))
       (beginning-of-line)
       (if (re-search-forward "\\(if\\|do\\|select\\|where\\|forall\\)"
-                            (f90-get-end-of-line) t)
+                            (line-end-position) t)
          (progn
            (beginning-of-line) (skip-chars-forward " \t")
            (cond ((f90-looking-at-do)
@@ -1187,9 +1271,7 @@ or, if already present, remove it."
     (if (< (point) (marker-position pos))
        (goto-char (marker-position pos)))
     (if (not no-update) (f90-update-line))
-    (if (and auto-fill-function
-            (> (save-excursion (end-of-line) (current-column)) fill-column))
-       (save-excursion (f90-do-auto-fill)))
+    (if auto-fill-function (f90-do-auto-fill))
     (set-marker pos nil)))
 
 (defun f90-indent-new-line ()
@@ -1312,7 +1394,7 @@ If run in the middle of a line, the line is not broken."
     (goto-char save-point)
     (set-marker end-region-mark nil)
     (set-marker save-point nil)
-    (if (string-match "Xemacs" emacs-version)
+    (if (string-match "XEmacs" emacs-version)
        (zmacs-deactivate-region)
       (deactivate-mark))))
 
@@ -1324,14 +1406,14 @@ If run in the middle of a line, the line is not broken."
       (setq program (f90-mark-subprogram))
       (if program
          (progn
-           (message (concat "Indenting " (car program) " "
-                            (car (cdr program))"."))
-           (f90-indent-region (point) (mark))
-           (message (concat "Indenting " (car program) " "
-                            (car (cdr program)) "...done.")))
-       (message "Indenting the whole file.")
-       (f90-indent-region (point) (mark))
-       (message (concat "Indenting the whole file...done."))))))
+           (message "Indenting %s %s..."
+                    (car program) (car (cdr program)))
+           (indent-region (point) (mark) nil)
+           (message "Indenting %s %s...done"
+                    (car program) (car (cdr program))))
+       (message "Indenting the whole file...")
+       (indent-region (point) (mark) nil)
+       (message "Indenting the whole file...done")))))
 
 ;; autofill and break-line
 (defun f90-break-line (&optional no-update)
@@ -1339,21 +1421,20 @@ If run in the middle of a line, the line is not broken."
   (interactive)
   (let (ctype)
     (cond ((f90-in-string)
-          (insert "&") (newline) (insert "&"))
+          (insert "&") (newline 1) (insert "&"))
          ((f90-in-comment)
-          (delete-horizontal-space)
           (setq ctype (f90-get-present-comment-type))
-          (newline) (insert (concat ctype " ")))
-         (t (delete-horizontal-space)
-            (insert "&")
+          (newline 1)
+          (insert ctype))
+         (t (insert "&")
             (if (not no-update) (f90-update-line))
-            (newline)
-            (if f90-beginning-ampersand (insert "& ")))))
-  (if (not no-update) (f90-indent-line)))
+            (newline 1)
+            (if f90-beginning-ampersand (insert "&")))))
+  (indent-according-to-mode))
   
 (defun f90-find-breakpoint ()
   "From fill-column, search backward for break-delimiter."
-  (let ((bol (f90-get-beg-of-line)))
+  (let ((bol (line-beginning-position)))
     (re-search-backward f90-break-delimiters bol)
     (if f90-break-before-delimiters
        (progn (backward-char)
@@ -1363,33 +1444,22 @@ If run in the middle of a line, the line is not broken."
          (forward-char 2)
        (forward-char)))))
 
-(defun f90-auto-fill-mode (arg)
-  "Toggle f90-auto-fill mode.
-With ARG, turn `f90-auto-fill' mode on iff ARG is positive.
-In `f90-auto-fill' mode, inserting a space at a column beyond `fill-column'
-automatically breaks the line at a previous space."
-  (interactive "P")
-  (prog1 (setq auto-fill-function
-              (if (if (null arg)
-                      (not auto-fill-function)
-                    (> (prefix-numeric-value arg) 0))
-                  'f90-do-auto-fill))
-    (force-mode-line-update)))
-
 (defun f90-do-auto-fill ()
-  "Break line if non-white characters beyond fill-column."
+  "Break line if non-white characters beyond fill-column. Also, update line. "
   (interactive)
-  ;; Break the line before or after the last delimiter (non-word char).
+  ;; Break the line before or after the last delimiter (non-word char) if
+  ;; position is beyond fill-column.
   ;; Will not break **, //, or => (specified by f90-no-break-re).
-  ;; Start by checking that line is longer than fill-column.
-  (if (> (save-excursion (end-of-line) (current-column)) fill-column)
-      (progn
+  (f90-update-line)
+  (while (> (current-column) fill-column)
+      (let ((pos-mark (point-marker)))
        (move-to-column fill-column)
-       (if (and (looking-at "[ \t]*$") (not (f90-in-string)))
-           (delete-horizontal-space)
-         (f90-find-breakpoint)
-         (f90-break-line)
-         (end-of-line)))))
+       (if (not (f90-in-string))
+           (f90-find-breakpoint))
+       (f90-break-line)
+       (goto-char pos-mark)
+       (set-marker pos-mark nil))))
+
 
 (defun f90-join-lines ()
   "Join present line with next line, if this line ends with \&."
@@ -1418,27 +1488,22 @@ automatically breaks the line at a previous space."
   "Fill every line in region by forward parsing. Join lines if possible."
   (interactive "*r")
   (let ((end-region-mark (make-marker))
-       (f90-smart-end nil) (f90-auto-keyword-case nil) indent (go-on t)
-       (af-function auto-fill-function) (auto-fill-function nil))
+       (f90-smart-end nil) (f90-auto-keyword-case nil) (go-on t)
+       (auto-fill-function nil))
     (set-marker end-region-mark end-region)
     (goto-char beg-region)
     (while go-on
       ;; join as much as possible
-      (while (f90-join-lines));
-      (setq indent (+ (f90-current-indentation) f90-continuation-indent))
+      (while (f90-join-lines))
       ;; chop the line if necessary
       (while (> (save-excursion (end-of-line) (current-column))
                fill-column)
        (move-to-column fill-column)
-       (if (and (looking-at "[ \t]*$") (not (f90-in-string)))
-           (delete-horizontal-space)
-         (f90-find-breakpoint)
-         (f90-break-line 'no-update)
-         (f90-indent-to indent 'no-line-no)))
+       (f90-find-breakpoint)
+       (f90-break-line 'no-update))
       (setq go-on (and  (< (point) (marker-position end-region-mark))
                        (zerop (forward-line 1))))
       (setq f90-cache-position (point)))
-    (setq auto-fill-function af-function)
     (setq f90-cache-position nil)
     (if (string-match "XEmacs" emacs-version)
        (zmacs-deactivate-region)
@@ -1447,7 +1512,7 @@ automatically breaks the line at a previous space."
 (defun f90-block-match (beg-block beg-name end-block end-name)
   "Match end-struct with beg-struct and complete end-block if possible.
 Leave point at the end of line."
-  (search-forward "end" (f90-get-end-of-line))
+  (search-forward "end" (line-end-position))
   (catch 'no-match
     (if (not (f90-equal-symbols beg-block end-block))
        (if end-block
@@ -1486,9 +1551,19 @@ Leave point at the end of line."
          (setq end-name  (car (cdr end-struct)))
          (save-excursion
            (beginning-of-line)
-           (while (and (not (zerop count))
-                       (re-search-backward 
-                        (concat "\\(" f90-blocks-re "\\)") nil t))
+           (while 
+               (and (not (zerop count))
+                    (let ((stop nil) notexist)
+                      (while (not stop)
+                        (setq notexist
+                              (not (re-search-backward 
+                                    (concat "\\(" f90-blocks-re "\\)") nil t)))
+                        (if notexist
+                            (setq stop t)
+                          (setq stop
+                                (not (or (f90-in-string)
+                                         (f90-in-comment))))))
+                      (not notexist)))
              (beginning-of-line) (skip-chars-forward " \t0-9")
              (cond ((setq matching-beg
                           (cond
@@ -1506,11 +1581,11 @@ Leave point at the end of line."
              (f90-update-line)
              (if (eq f90-smart-end 'blink)
                  (if (< (point) top-of-window)
-                     (message (concat 
-                               "Matches " (what-line) ": "
-                               (buffer-substring
-                                (progn (beginning-of-line) (point))
-                                (progn (end-of-line) (point)))))
+                     (message "Matches %s: %s"
+                              (what-line)
+                              (buffer-substring
+                               (line-beginning-position)
+                               (line-end-position)))
                    (sit-for 1)))
              (setq beg-block (car matching-beg))
              (setq beg-name (car (cdr matching-beg)))
@@ -1533,12 +1608,13 @@ Any other key combination is executed normally."
   (interactive)
   (let (e c)
     (insert last-command-char)
-    (setq e (next-command-event)
-         c (if (string-match "XEmacs" emacs-version)
-               (event-to-character e)
-             (read-event)))
+    (if (string-match "XEmacs" emacs-version)
+       (progn
+         (setq e (next-command-event))
+         (setq c (event-to-character e)))
+      (setq c (read-event)))
     ;; insert char if not equal to `?'
-    (if (or (= c ??) (eq c help-char))
+    (if (or (eq c ??) (eq c help-char))
        (f90-abbrev-help)
       (if (string-match "XEmacs" emacs-version)
          (setq unread-command-event e)
@@ -1621,6 +1697,12 @@ Any other key combination is executed normally."
                  (setq modified t))))
        (or modified (set-buffer-modified-p nil))))))
 
+
+(defun f90-current-defun ()
+  "Function to use for `add-log-current-defun-function' in F90 mode."
+  (save-excursion
+    (nth 1 (f90-beginning-of-subprogram))))
+
 (provide 'f90)
 
 ;;; f90.el ends here