]> code.delx.au - gnu-emacs/blobdiff - lisp/tempo.el
Merge from emacs--rel--22
[gnu-emacs] / lisp / tempo.el
index 9f809097dc32f12cf6d9a15888a79d848646cd8f..383dcd40d24c7fba75452f967bac02efe9b841cb 100644 (file)
@@ -1,16 +1,18 @@
-;;; tempo.el --- templates with hotspots
-;; Copyright (C) 1994 Free Software Foundation, Inc.
+;;; tempo.el --- Flexible template insertion
 
-;; Author: David K}gedal <davidk@lysator.liu.se >
+;; Copyright (C) 1994, 1995, 2001, 2002, 2003, 2004,
+;;   2005, 2006, 2007 Free Software Foundation, Inc.
+
+;; Author: David K}gedal <davidk@lysator.liu.se>
 ;; Created: 16 Feb 1994
-;; Version: 1.1.1
+;; K}gedal's last version number: 1.2.4
 ;; Keywords: extensions, languages, tools
 
 ;; This file is part of GNU Emacs.
 
 ;; GNU Emacs is free software; you can redistribute it and/or modify
 ;; it under the terms of the GNU General Public License as published by
-;; the Free Software Foundation; either version 2, or (at your option)
+;; the Free Software Foundation; either version 3, or (at your option)
 ;; any later version.
 
 ;; GNU Emacs is distributed in the hope that it will be useful,
@@ -19,8 +21,9 @@
 ;; GNU General Public License for more details.
 
 ;; You should have received a copy of the GNU General Public License
-;; along with GNU Emacs; see the file COPYING.  If not, write to
-;; the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.
+;; along with GNU Emacs; see the file COPYING.  If not, write to the
+;; Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+;; Boston, MA 02110-1301, USA.
 
 ;;; Commentary:
 
@@ -28,9 +31,9 @@
 ;; macros, if you wish. It is mainly intended for, but not limited to,
 ;; other programmers to be used for creating shortcuts for editing
 ;; certain kind of documents. It was originally written to be used by
-;; a HTML editing mode written by Nelson Minar <nelson@reed.edu>, and
-;; his html-helper-mode.el is probably the best example of how to use
-;; this program.
+;; a HTML editing mode written by Nelson Minar <nelson@santafe.edu>,
+;; and his html-helper-mode.el is probably the best example of how to
+;; use this program.
 
 ;; A template is defined as a list of items to be inserted in the
 ;; current buffer at point. Some of the items can be simple strings,
@@ -50,7 +53,7 @@
 ;; a non-nil argument).
 
 ;; More flexible templates can be created by including lisp symbols,
-;; which will be evaluated as variables, or lists, which will will be
+;; which will be evaluated as variables, or lists, which will be
 ;; evaluated as lisp expressions.
 
 ;; See the documentation for tempo-define-template for the different
 ;; be the template definition, and its function value will be an
 ;; interactive function that inserts the template at the point.
 
-;; Full documentation for tempo.el can be found on the World Wide Web
-;; at http://www.lysator.liu.se:7500/~davidk/tempo.html (not yet
-;; completed)
-
 ;; The latest tempo.el distribution can be fetched from
 ;; ftp.lysator.liu.se in the directory /pub/emacs
 
+;; There is also a WWW page at
+;; http://www.lysator.liu.se/~davidk/elisp/ which has some information
+
+;;; Known bugs:
+
+;; If the 'o is the first element in a template, strange things can
+;; happen when the template is inserted at the beginning of a
+;; line. This is due to strange behaviour in open-line. But it should
+;; be easily avoided.
+
+;; The 'o tag is also a problem when including the region. This will
+;; be looked into.
+
+;; Clicking mouse-2 in the completion buffer gives strange results.
+
+;; There is a bug in some emacs versions that prevents completion from
+;; working. If it doesn't work for you, send me a note indicating your
+;; emacs version and your problems.
+
+;;; Contributors:
+
+;; These people have given me important feedback and new ideas for
+;; tempo.el. Thanks.
+
+;; Nelson Minar <nelson@santafe.edu>
+;; Richard Stallman <rms@gnu.org>
+;; Lars Lindberg <Lars.Lindberg@sypro.cap.se>
+;; Glen Whitney <Glen.Whitney@math.lsa.umich.edu>
+
 ;;; Code:
 
-(provide 'tempo)
+;;; User options
 
-;;; Variables
+(defgroup tempo nil
+  "Flexible template insertion."
+  :prefix "tempo-"
+  :group 'tools)
 
-(defvar tempo-interactive nil
+(defcustom tempo-interactive nil
   "*Prompt user for strings in templates.
 If this variable is non-nil, `tempo-insert' prompts the
-user for text to insert in the templates")
+user for text to insert in the templates."
+  :type 'boolean
+  :group 'tempo)
 
-(defvar tempo-insert-region nil
+(defcustom tempo-insert-region nil
   "*Automatically insert current region when there is a `r' in the template
-If this variable is NIL, `r' elements will be treated just like `p'
+If this variable is nil, `r' elements will be treated just like `p'
 elements, unless the template function is given a prefix (or a non-nil
-argument). If this variable is non-NIL, the behaviour is reversed.")
+argument).  If this variable is non-nil, the behavior is reversed.
+
+In Transient Mark mode, this option is unused."
+  :type 'boolean
+  :group 'tempo)
+
+(defcustom tempo-show-completion-buffer t
+  "*If non-nil, show a buffer with possible completions, when only
+a partial completion can be found."
+  :type 'boolean
+  :group 'tempo)
 
-(defvar tempo-show-completion-buffer t
-  "*If non-NIL, show a buffer with possible completions, when only
-a partial completion can be found")
+(defcustom tempo-leave-completion-buffer nil
+  "*If nil, a completion buffer generated by \\[tempo-complete-tag]
+disappears at the next keypress; otherwise, it remains forever."
+  :type 'boolean
+  :group 'tempo)
 
-(defvar tempo-leave-completion-buffer nil
-  "*If NIL, a completion buffer generated by \\[tempo-complete-tag]
-disappears at the next keypress; otherwise, it remains forever.")
+;;; Internal variables
 
 (defvar tempo-insert-string-functions nil
   "List of functions to run when inserting a string.
-Each function is called with a single arg, STRING."  )
+Each function is called with a single arg, STRING and should return
+another string.  This could be used for making all strings upcase by
+setting it to '(upcase), for example.")
 
 (defvar tempo-tags nil
-  "An association list with tags and corresponding templates")
+  "An association list with tags and corresponding templates.")
 
 (defvar tempo-local-tags '((tempo-tags . nil))
   "A list of locally installed tag completion lists.
-
 It is a association list where the car of every element is a symbol
-whose varable value is a template list. The cdr part, if non-nil, is a
-function or a regexp that defines the string to match. See the
+whose variable value is a template list.  The cdr part, if non-nil,
+is a function or a regexp that defines the string to match.  See the
 documentation for the function `tempo-complete-tag' for more info.
 
 `tempo-tags' is always in the last position in this list.")
 
+(defvar tempo-collection nil
+  "A collection of all the tags defined for the current buffer.")
+
+(defvar tempo-dirty-collection t
+  "Indicates if the tag collection needs to be rebuilt.")
+
 (defvar tempo-marks nil
   "A list of marks to jump to with `\\[tempo-forward-mark]' and `\\[tempo-backward-mark]'.")
 
-(defvar tempo-default-match-finder "\\b\\([^\\b]*\\)\\="
-  "The default regexp used to find the string to match against the tags.")
+(defvar tempo-match-finder "\\b\\([[:word:]]+\\)\\="
+  "The regexp or function used to find the string to match against tags.
+
+If `tempo-match-finder' is a string, it should contain a regular
+expression with at least one \\( \\) pair.  When searching for tags,
+`tempo-complete-tag' calls `re-search-backward' with this string, and
+the string between the first \\( and \\) is used for matching against
+each string in the tag list. If one is found, the whole text between
+the first \\( and the point is replaced with the inserted template.
+
+You will probably want to include \\=\\= at the end of the regexp to
+make sure that the string is matched only against text adjacent to the
+point.
+
+If `tempo-match-finder' is a symbol, it should be a function that
+returns a pair of the form (STRING . POS), where STRING is the string
+used for matching and POS is the buffer position after which text
+should be replaced with a template.")
+
+(defvar tempo-user-elements nil
+  "Element handlers for user-defined elements.
+A list of symbols which are bound to functions that take one argument.
+This function should return something to be sent to `tempo-insert' if
+it recognizes the argument, and nil otherwise.")
 
 (defvar tempo-named-insertions nil
-  "Temporary storage for named insertions")
+  "Temporary storage for named insertions.")
+
+(defvar tempo-region-start (make-marker)
+  "Region start when inserting around the region.")
+
+(defvar tempo-region-stop (make-marker)
+  "Region stop when inserting around the region.")
 
 ;; Make some variables local to every buffer
 
 (make-variable-buffer-local 'tempo-marks)
 (make-variable-buffer-local 'tempo-local-tags)
+(make-variable-buffer-local 'tempo-match-finder)
+(make-variable-buffer-local 'tempo-collection)
+(make-variable-buffer-local 'tempo-dirty-collection)
 
 ;;; Functions
 
@@ -149,45 +230,58 @@ NAME is a string that contains the name of the template, ELEMENTS is a
 list of elements in the template, TAG is the tag used for completion,
 DOCUMENTATION is the documentation string for the insertion command
 created, and TAGLIST (a symbol) is the tag list that TAG (if provided)
-should be added to).  If TAGLIST is nil and TAG is non-nil, TAG is
-added to `tempo-tags'
+should be added to.  If TAGLIST is nil and TAG is non-nil, TAG is
+added to `tempo-tags'.
 
 The elements in ELEMENTS can be of several types:
 
- - A string. It is sent to the hooks in `tempo-insert-string-functions',
+ - A string: It is sent to the hooks in `tempo-insert-string-functions',
    and the result is inserted.
- - The symbol 'p. This position is saved in `tempo-marks'.
- - The symbol 'r. If `tempo-insert' is called with ON-REGION non-nil
-   the current region is placed here. Otherwise it works like 'p.
- - (p PROMPT <NAME>) If `tempo-interactive' is non-nil, the user is
-   prompted in the minbuffer with PROMPT for a string to be inserted.
-   If the optional parameter NAME is non-nil, the text is saved for
-   later insertion with the `s' tag.
-   If `tempo-interactive is nil, it works like 'p.
- - (r PROMPT) like the previous, but if `tempo-interactive' is nil
-   and `tempo-insert' is called with ON-REGION non-nil, the current
-   region is placed here.
- - (s NAME) Inserts text previously read with the (p ..) construct.
-   Finds the insertion saved under NAME and inserts it. Acts like 'p
+ - The symbol `p': This position is saved in `tempo-marks'.
+ - The symbol `r': If `tempo-insert' is called with ON-REGION non-nil
+   the current region is placed here.  Otherwise it works like `p'.
+ - (p PROMPT <NAME> <NOINSERT>): If `tempo-interactive' is non-nil, the
+   user is prompted in the minibuffer with PROMPT for a string to be
+   inserted.  If the optional parameter NAME is non-nil, the text is
+   saved for later insertion with the `s' tag.  If there already is
+   something saved under NAME that value is used instead and no
+   prompting is made.  If NOINSERT is provided and non-nil, nothing is
+   inserted, but text is still saved when a NAME is provided.  For
+   clarity, the symbol `noinsert' should be used as argument.
+ - (P PROMPT <NAME> <NOINSERT>): Works just like the previous tag, but
+   forces `tempo-interactive' to be true.
+ - (r PROMPT <NAME> <NOINSERT>): Like the previous tag, but if
+   `tempo-interactive' is nil and `tempo-insert' is called with
+   ON-REGION non-nil, the current region is placed here.  This usually
+   happens when you call the template function with a prefix argument.
+ - (s NAME): Inserts text previously read with the (p ..) construct.
+   Finds the insertion saved under NAME and inserts it.  Acts like `p'
    if tempo-interactive is nil.
- - '& If there is only whitespace between the line start and point,
-   nothing happens. Otherwise a newline is inserted.
- - '% If there is only whitespace between point and end-of-line
-   nothing happens. Otherwise a newline is inserted.
- - 'n inserts a newline.
- - '> The line is indented using `indent-according-to-mode'. Note that
-   you often should place this item after the text you want on the
-   line.
- - 'n> inserts a newline and indents line.
- - nil. It is ignored.
- - Anything else. It is evaluated and the result is parsed again."
-
+ - `&': If there is only whitespace between the line start and point,
+   nothing happens.  Otherwise a newline is inserted.
+ - `%': If there is only whitespace between point and end of line,
+   nothing happens.  Otherwise a newline is inserted.
+ - `n': Inserts a newline.
+ - `>': The line is indented using `indent-according-to-mode'.  Note
+   that you often should place this item after the text you want on
+   the line.
+ - `r>': Like `r', but it also indents the region.
+ - (r> PROMPT <NAME> <NOINSERT>): Like (r ...), but is also indents
+   the region.
+ - `n>': Inserts a newline and indents line.
+ - `o': Like `%' but leaves the point before the newline.
+ - nil: It is ignored.
+ - Anything else: It is evaluated and the result is treated as an
+   element to be inserted.  One additional tag is useful for these
+   cases.  If an expression returns a list '(l foo bar), the elements
+   after `l' will be inserted according to the usual rules.  This makes
+   it possible to return several elements from one expression."
   (let* ((template-name (intern (concat "tempo-template-"
                                       name)))
         (command-name template-name))
     (set template-name elements)
     (fset command-name (list 'lambda (list '&optional 'arg)
-                            (or documentation 
+                            (or documentation
                                 (concat "Insert a " name "."))
                             (list 'interactive "*P")
                             (list 'tempo-insert-template (list 'quote
@@ -204,40 +298,85 @@ The elements in ELEMENTS can be of several types:
 (defun tempo-insert-template (template on-region)
   "Insert a template.
 TEMPLATE is the template to be inserted.  If ON-REGION is non-nil the
-`r' elements are replaced with the current region."
-  (and on-region
-       (< (mark) (point))
-       (exchange-point-and-mark))
-  (save-excursion
-    (tempo-insert-mark (point-marker))
-    (mapcar 'tempo-insert 
-           (symbol-value template))
-    (tempo-insert-mark (point-marker)))
-  (tempo-forward-mark)
-  (tempo-forget-insertions))
+`r' elements are replaced with the current region.  In Transient Mark
+mode, ON-REGION is ignored and assumed true if the region is active."
+  (unwind-protect
+      (progn
+       (if (or (and (boundp 'transient-mark-mode) ; For Emacs
+                    transient-mark-mode
+                    mark-active)
+               (if (featurep 'xemacs)
+                   (and zmacs-regions (mark))))
+           (setq on-region t))
+       (and on-region
+            (set-marker tempo-region-start (min (mark) (point)))
+            (set-marker tempo-region-stop (max (mark) (point))))
+       (if on-region
+           (goto-char tempo-region-start))
+       (save-excursion
+         (tempo-insert-mark (point-marker))
+         (mapc (function (lambda (elt)
+                           (tempo-insert elt on-region)))
+               (symbol-value template))
+         (tempo-insert-mark (point-marker)))
+       (tempo-forward-mark))
+    (tempo-forget-insertions)
+    ;; Should I check for zmacs here too???
+    (and (boundp 'transient-mark-mode)
+        transient-mark-mode
+        (deactivate-mark))))
 
 ;;;
 ;;; tempo-insert
 
-(defun tempo-insert (element
+(defun tempo-insert (element on-region)
   "Insert a template element.
-Insert one element from a template. See documentation for
-`tempo-define-template' for the kind of elements possible."
+Insert one element from a template. If ON-REGION is non-nil the `r'
+elements are replaced with the current region.
+
+See documentation for `tempo-define-template' for the kind of elements
+possible."
   (cond ((stringp element) (tempo-process-and-insert-string element))
-       ((and (consp element) (eq (car element) 'p))
-        (tempo-insert-prompt (cdr element)))
-       ((and (consp element) (eq (car element) 'r))
-        (if on-region
-            (exchange-point-and-mark)
-          (tempo-insert-prompt (cdr element))))
-       ((and (consp element) (eq (car element) 's))
-        (if tempo-interactive
-            (tempo-insert-named (cdr element))
-          (tempo-insert-mark (point-marker))))
+       ((and (consp element)
+             (eq (car element) 'p)) (tempo-insert-prompt-compat
+                                     (cdr element)))
+       ((and (consp element)
+             (eq (car element) 'P)) (let ((tempo-interactive t))
+                                      (tempo-insert-prompt-compat
+                                       (cdr element))))
+;;;    ((and (consp element)
+;;;          (eq (car element) 'v)) (tempo-save-named
+;;;                                  (nth 1 element)
+;;;                                  nil
+;;;                                  (nth 2 element)))
+       ((and (consp element)
+             (eq (car element) 'r)) (if on-region
+                                        (goto-char tempo-region-stop)
+                                      (tempo-insert-prompt-compat
+                                       (cdr element))))
+        ((and (consp element)
+              (eq (car element) 'r>)) (if on-region
+                                          (progn
+                                            (goto-char tempo-region-stop)
+                                            (indent-region (mark) (point) nil))
+                                        (tempo-insert-prompt-compat
+                                         (cdr element))))
+       ((and (consp element)
+             (eq (car element) 's)) (tempo-insert-named (car (cdr element))))
+       ((and (consp element)
+             (eq (car element) 'l)) (mapcar (function
+                                             (lambda (elt)
+                                               (tempo-insert elt on-region)))
+                                            (cdr element)))
        ((eq element 'p) (tempo-insert-mark (point-marker)))
        ((eq element 'r) (if on-region
-                            (exchange-point-and-mark)
+                            (goto-char tempo-region-stop)
                           (tempo-insert-mark (point-marker))))
+       ((eq element 'r>) (if on-region
+                             (progn
+                               (goto-char tempo-region-stop)
+                               (indent-region (mark) (point) nil))
+                           (tempo-insert-mark (point-marker))))
        ((eq element '>) (indent-according-to-mode))
        ((eq element '&) (if (not (or (= (current-column) 0)
                                      (save-excursion
@@ -251,42 +390,81 @@ Insert one element from a template. See documentation for
                             (insert "\n")))
        ((eq element 'n) (insert "\n"))
        ((eq element 'n>) (insert "\n") (indent-according-to-mode))
+       ;; Bug: If the 'o is the first element in a template, strange
+       ;; things can happen when the template is inserted at the
+       ;; beginning of a line.
+       ((eq element 'o) (if (not (or on-region
+                                     (eolp)
+                                     (save-excursion
+                                       (re-search-forward
+                                        "\\=\\s-*$" nil t))))
+                            (open-line 1)))
        ((null element))
-       (t (tempo-insert (eval element)))))
+       (t (tempo-insert (or (tempo-is-user-element element)
+                            (eval element))
+                        on-region))))
 
 ;;;
 ;;; tempo-insert-prompt
 
-(defun tempo-insert-prompt (prompt)
+(defun tempo-insert-prompt-compat (prompt)
+  "Compatibility hack for `tempo-insert-prompt'.
+PROMPT can be either a prompt string, or a list of arguments to
+`tempo-insert-prompt', or nil."
+  (if (consp prompt)                   ; not nil either
+      (apply 'tempo-insert-prompt prompt)
+    (tempo-insert-prompt prompt)))
+
+(defun tempo-insert-prompt (prompt &optional save-name no-insert)
   "Prompt for a text string and insert it in the current buffer.
 If the variable `tempo-interactive' is non-nil the user is prompted
 for a string in the minibuffer, which is then inserted in the current
-buffer. If `tempo-interactive' is nil, the current point is placed on
+buffer.  If `tempo-interactive' is nil, the current point is placed on
 `tempo-mark'.
 
-PROMPT is the prompt string or a list containing the prompt string and
-a name to save the inserted text under."
-  (if tempo-interactive
-      (let ((prompt-string (if (listp prompt)
-                              (car prompt)
-                            prompt))
-           (save-name (and (listp prompt) (nth 1 prompt)))
-           inserted-text)
-
-       (progn
-         (setq inserted-text (read-string prompt-string))
-         (insert inserted-text)
-         (if save-name
-             (tempo-remember-insertion save-name inserted-text))))
-    (tempo-insert-mark (point-marker))))
+PROMPT is the prompt string, SAVE-NAME is a name to save the inserted
+text under.  If the optional argument NO-INSERT is non-nil, no text is
+inserted.  This can be useful when there is a SAVE-NAME.
+
+If there already is a value for SAVE-NAME, it is used and the user is
+never prompted."
+  (let (insertion
+       (previous (and save-name
+                      (tempo-lookup-named save-name))))
+    (cond
+     ;; Insert  previous value, unless no-insert is non-nil
+     ((and previous
+          (not no-insert))
+      (tempo-insert-named save-name)) ; A double lookup here, but who
+                                     ; cares
+     ;; If no-insert is non-nil, don't insert the previous value. Just
+     ;; keep it
+     (previous
+      nil)
+     ;; No previous value. Prompt or insert mark
+     (tempo-interactive
+      (if (not (stringp prompt))
+         (error "tempo: The prompt (%s) is not a string" prompt))
+      (setq insertion (read-string prompt))
+      (or no-insert
+         (insert insertion))
+      (if save-name
+         (tempo-save-named save-name insertion)))
+     (t
+      (tempo-insert-mark (point-marker))))))
 
 ;;;
-;;; tempo-remember-insertion
-
-(defun tempo-remember-insertion (save-name string)
-  "Save the text in STRING under the name SAVE-NAME for later retrieval."
-  (setq tempo-named-insertions (cons (cons save-name string)
-                                    tempo-named-insertions)))
+;;; tempo-is-user-element
+
+(defun tempo-is-user-element (element)
+  "Tries all the user-defined element handlers in `tempo-user-elements'."
+  ;; Sigh... I need (some list)
+  (catch 'found
+    (mapc (function (lambda (handler)
+                     (let ((result (funcall handler element)))
+                       (if result (throw 'found result)))))
+         tempo-user-elements)
+    (throw 'found nil)))
 
 ;;;
 ;;; tempo-forget-insertions
@@ -295,17 +473,47 @@ a name to save the inserted text under."
   "Forget all the saved named insertions."
   (setq tempo-named-insertions nil))
 
+;;;
+;;; tempo-save-named
+
+(defun tempo-save-named (name data)    ; Had an optional prompt for 'v
+  "Save some data for later insertion
+The contents of DATA is saved under the name NAME.
+
+The data can later be retrieved with `tempo-lookup-named'.
+
+This function returns nil, so it can be used in a template without
+inserting anything."
+  (setq tempo-named-insertions
+       (cons (cons name data)
+             tempo-named-insertions))
+  nil)
+
+;;;
+;;; tempo-lookup-named
+
+(defun tempo-lookup-named (name)
+  "Lookup some saved data under the name NAME.
+Returns the data if NAME was found, and nil otherwise."
+  (cdr (assq name tempo-named-insertions)))
+
 ;;;
 ;;; tempo-insert-named
 
-(defun tempo-insert-named (elt)
-  "Insert the previous insertion saved under a named specified in ELT.
-The name is in the car of ELT."
-  (let* ((name (car elt))
-        (insertion (cdr (assq name tempo-named-insertions))))
-    (if insertion
-       (insert insertion)
-      (error "Named insertion not found"))))
+(defun tempo-insert-named (name)
+  "Insert the previous insertion saved under a named specified in NAME.
+If there is no such name saved, a tempo mark is inserted.
+
+Note that if the data is a string, it will not be run through the string
+processor."
+  (let* ((insertion (tempo-lookup-named name)))
+    (cond ((null insertion)
+          (tempo-insert-mark (point-marker)))
+         ((stringp insertion)
+          (insert insertion))
+         (t
+          (tempo-insert insertion nil)))))
+
 
 ;;;
 ;;; tempo-process-and-insert-string
@@ -318,11 +526,10 @@ and insert the results."
         nil)
        ((symbolp tempo-insert-string-functions)
         (setq string
-              (apply tempo-insert-string-functions (list string))))
+              (funcall tempo-insert-string-functions string)))
        ((listp tempo-insert-string-functions)
-        (mapcar (function (lambda (fn)
-                            (setq string (apply fn string))))
-                tempo-insert-string-functions))
+        (dolist (fn tempo-insert-string-functions)
+          (setq string (funcall fn string))))
        (t
         (error "Bogus value in tempo-insert-string-functions: %s"
                tempo-insert-string-functions)))
@@ -332,7 +539,7 @@ and insert the results."
 ;;; tempo-insert-mark
 
 (defun tempo-insert-mark (mark)
-  "Insert a mark `tempo-marks' while keeping it sorted"
+  "Insert a mark `tempo-marks' while keeping it sorted."
   (cond ((null tempo-marks) (setq tempo-marks (list mark)))
        ((< mark (car tempo-marks)) (setq tempo-marks (cons mark tempo-marks)))
        (t (let ((lp tempo-marks))
@@ -341,7 +548,7 @@ and insert the results."
               (setq lp (cdr lp)))
             (if (not (= mark (car lp)))
                 (setcdr lp (cons mark (cdr lp))))))))
-         
+
 ;;;
 ;;; tempo-forward-mark
 
@@ -349,7 +556,7 @@ and insert the results."
   "Jump to the next mark in `tempo-forward-mark-list'."
   (interactive)
   (let ((next-mark (catch 'found
-                    (mapcar
+                    (mapc
                      (function
                       (lambda (mark)
                         (if (< (point) mark)
@@ -368,7 +575,7 @@ and insert the results."
   (interactive)
   (let ((prev-mark (catch 'found
                     (let (last)
-                      (mapcar
+                      (mapc
                        (function
                         (lambda (mark)
                           (if (<= (point) mark)
@@ -378,13 +585,12 @@ and insert the results."
                       last))))
     (if prev-mark
        (goto-char prev-mark))))
-       
+
 ;;;
 ;;; tempo-add-tag
 
 (defun tempo-add-tag (tag template &optional tag-list)
   "Add a template tag.
-
 Add the TAG, that should complete to TEMPLATE to the list in TAG-LIST,
 or to `tempo-tags' if TAG-LIST is nil."
 
@@ -392,53 +598,76 @@ or to `tempo-tags' if TAG-LIST is nil."
   (if (null tag-list)
       (setq tag-list 'tempo-tags))
   (if (not (assoc tag (symbol-value tag-list)))
-      (set tag-list (cons (cons tag template) (symbol-value tag-list)))))
+      (set tag-list (cons (cons tag template) (symbol-value tag-list))))
+  (tempo-invalidate-collection))
 
 ;;;
 ;;; tempo-use-tag-list
 
 (defun tempo-use-tag-list (tag-list &optional completion-function)
   "Install TAG-LIST to be used for template completion in the current buffer.
-
 TAG-LIST is a symbol whose variable value is a tag list created with
-`tempo-add-tag' and COMPLETION-FUNCTION is an optional function or
-string that is used by `\\[tempo-complete-tag]' to find a string to
-match the tag against.
-
-If COMPLETION-FUNCTION is a string, it should contain a regular
-expression with at least one \\( \\) pair. When searching for tags,
-`tempo-complete-tag' calls `re-search-backward' with this string, and
-the string between the first \\( and \\) is used for matching against
-each string in the tag list. If one is found, the whole text between
-the first \\( and the point is replaced with the inserted template.
-
-You will probably want to include \\ \= at the end of the regexp to make
-sure that the string is matched only against text adjacent to the
-point.
-
-If COPMLETION-FUNCTION is a symbol, it should be a function that
-returns a cons cell of the form (STRING . POS), where STRING is the
-string used for matching and POS is the buffer position after which
-text should be replaced with a template."
+`tempo-add-tag'.
 
+COMPLETION-FUNCTION is an obsolete option for specifying an optional
+function or string that is used by `\\[tempo-complete-tag]' to find a
+string to match the tag against.  It has the same definition as the
+variable `tempo-match-finder'.  In this version, supplying a
+COMPLETION-FUNCTION just sets `tempo-match-finder' locally."
   (let ((old (assq tag-list tempo-local-tags)))
     (if old
        (setcdr old completion-function)
       (setq tempo-local-tags (cons (cons tag-list completion-function)
-                                  tempo-local-tags)))))
+                                  tempo-local-tags))))
+  (if completion-function
+      (setq tempo-match-finder completion-function))
+  (tempo-invalidate-collection))
+
+;;;
+;;; tempo-invalidate-collection
+
+(defun tempo-invalidate-collection ()
+  "Marks the tag collection as obsolete.
+Whenever it is needed again it will be rebuilt."
+  (setq tempo-dirty-collection t))
+
+;;;
+;;; tempo-build-collection
+
+(defun tempo-build-collection ()
+  "Build a collection of all the tags and return it.
+If `tempo-dirty-collection' is nil, the old collection is reused."
+  (prog1
+      (or (and (not tempo-dirty-collection)
+              tempo-collection)
+         (setq tempo-collection
+               (apply (function append)
+                      (mapcar (function (lambda (tag-list)
+                                       ; If the format for
+                                       ; tempo-local-tags changes,
+                                       ; change this
+                                          (eval (car tag-list))))
+                              tempo-local-tags))))
+    (setq tempo-dirty-collection nil)))
 
 ;;;
 ;;; tempo-find-match-string
 
 (defun tempo-find-match-string (finder)
   "Find a string to be matched against a tag list.
-
-FINDER is a function or a string. Returns (STRING . POS)."
+FINDER is a function or a string.  Returns (STRING . POS), or nil
+if no reasonable string is found."
   (cond ((stringp finder)
-        (save-excursion
-          (re-search-backward finder nil t))
-        (cons (buffer-substring (match-beginning 1) (1+ (match-end 1)))
-              (match-beginning 1)))
+        (let (successful)
+          (save-excursion
+            (or (setq successful (re-search-backward finder nil t))
+                0))
+          (if successful
+              (cons (buffer-substring (match-beginning 1)
+                                      (match-end 1)) ; This seems to be a
+                                       ; bug in emacs
+                    (match-beginning 1))
+            nil)))
        (t
         (funcall finder))))
 
@@ -447,76 +676,47 @@ FINDER is a function or a string. Returns (STRING . POS)."
 
 (defun tempo-complete-tag (&optional silent)
   "Look for a tag and expand it.
+All the tags in the tag lists in `tempo-local-tags'
+\(this includes `tempo-tags') are searched for a match for the text
+before the point.  The way the string to match for is determined can
+be altered with the variable `tempo-match-finder'.  If
+`tempo-match-finder' returns nil, then the results are the same as
+no match at all.
 
-It goes through the tag lists in `tempo-local-tags' (this includes
-`tempo-tags') and for each list it uses the corresponding match-finder
-function, or `tempo-default-match-finder' if none is given, and tries
-to match the match string against the tags in the list using
-`try-completion'. If none is found it proceeds to the next list until
-one is found. If a partial completion is found, it is replaced by the
-template if it can be completed uniquely, or completed as far as
-possible.
-
-When doing partial completion, only tags in the currently examined
-list are considered, so if you provide similar tags in different lists
-in `tempo-local-tags', the result may not be desirable.
+If a single match is found, the corresponding template is expanded in
+place of the matching string.
 
-If no match is found or a partial match is found, and SILENT is
+If a partial completion or no match at all is found, and SILENT is
 non-nil, the function will give a signal.
 
-If tempo-show-completion-buffer is non-NIL, a buffer containing
-possible completions is displayed when a partial completion is found."
+If a partial completion is found and `tempo-show-completion-buffer' is
+non-nil, a buffer containing possible completions is displayed."
+
+  ;; This function may look like a hack, but this is how I want it to
+  ;; work.
+  (interactive "*")
+  (let* ((collection (tempo-build-collection))
+        (match-info (tempo-find-match-string tempo-match-finder))
+        (match-string (car match-info))
+        (match-start (cdr match-info))
+        (exact (assoc match-string collection))
+        (compl (or (car exact)
+                   (and match-info (try-completion match-string collection)))))
+    (if compl (delete-region match-start (point)))
+    (cond ((null match-info) (or silent (ding)))
+         ((null compl) (or silent (ding)))
+         ((eq compl t) (tempo-insert-template
+                        (cdr (assoc match-string
+                                    collection))
+                        nil))
+         (t (if (setq exact (assoc compl collection))
+                (tempo-insert-template (cdr exact) nil)
+              (insert compl)
+              (or silent (ding))
+              (if tempo-show-completion-buffer
+                  (tempo-display-completions match-string
+                                             collection)))))))
 
-  ;; This function is really messy. Some cleaning up is necessary.
-  (interactive)
-  (if (catch 'completed
-       (mapcar
-        (function
-         (lambda (tag-list-a)
-           (let* ((tag-list (symbol-value(car tag-list-a)))
-                  (match-string-finder (or (cdr tag-list-a)
-                                           tempo-default-match-finder))
-                  (match-info (tempo-find-match-string match-string-finder))
-                  (match-string (car match-info))
-                  (match-start (cdr match-info))
-                  (compl (or (cdr (assoc match-string tag-list))
-                             (try-completion match-string
-                                             tag-list))))
-       
-             (if compl                 ;any match
-                 (delete-region match-start (point)))
-
-             (cond
-              ((null compl)            ; No match
-               nil)
-              ((symbolp compl)         ; ??
-               (tempo-insert-template compl nil)
-               (throw 'completed t))
-              ((eq compl t)            ; Exact, sole match
-               (tempo-insert-template (cdr (assoc match-string tag-list))
-                                      nil)
-               (throw 'completed t))
-              ((stringp compl)         ; (partial) completion found
-               (let ((compl2 (assoc compl tag-list)))
-                 (if compl2
-                     (tempo-insert-template (cdr compl2) nil)
-                   (insert compl)
-                   (if t ;(string= match-string compl)
-                       (if tempo-show-completion-buffer
-                           (tempo-display-completions match-string
-                                                      tag-list)
-                         (if (not silent)
-                             (ding))))))
-               (throw 'completed t))))))
-        tempo-local-tags)
-       ;; No completion found. Return nil
-       nil)
-      ;; Do nothing if a completion was found
-      t
-    ;; No completion was found
-    (if (not silent)
-       (ding))
-    nil))
 
 ;;;
 ;;; tempo-display-completions
@@ -526,11 +726,44 @@ possible completions is displayed when a partial completion is found."
   (if tempo-leave-completion-buffer
       (with-output-to-temp-buffer "*Completions*"
        (display-completion-list
-        (all-completions string tag-list)))
+        (all-completions string tag-list)
+        string))
     (save-window-excursion
       (with-output-to-temp-buffer "*Completions*"
        (display-completion-list
-        (all-completions string tag-list)))
+        (all-completions string tag-list)
+        string))
       (sit-for 32767))))
 
+;;;
+;;; tempo-expand-if-complete
+
+(defun tempo-expand-if-complete ()
+  "Expand the tag before point if it is complete.
+Returns non-nil if an expansion was made and nil otherwise.
+
+This could as an example be used in a command that is bound to the
+space bar, and looks something like this:
+
+\(defun tempo-space ()
+  (interactive \"*\")
+  (or (tempo-expand-if-complete)
+      (insert \" \")))"
+
+  (interactive "*")
+  (let* ((collection (tempo-build-collection))
+        (match-info (tempo-find-match-string tempo-match-finder))
+        (match-string (car match-info))
+        (match-start (cdr match-info))
+        (exact (assoc match-string collection)))
+    (if exact
+       (progn
+         (delete-region match-start (point))
+         (tempo-insert-template (cdr exact) nil)
+         t)
+      nil)))
+
+(provide 'tempo)
+
+;;; arch-tag: b3c0ee36-db3b-47bc-875f-091b4e27a063
 ;;; tempo.el ends here