]> code.delx.au - gnu-emacs/blobdiff - lisp/tempo.el
When compiling, require compare-w and skeleton
[gnu-emacs] / lisp / tempo.el
index f6c6676ce400a320ce1120e0f8d36b8569384f03..5434a5f331735349552bbcc9a9ba9c479cc8c37b 100644 (file)
@@ -1,9 +1,10 @@
-;;; tempo.el --- templates with hotspots
-;; Copyright (C) 1994 Free Software Foundation, Inc.
+;;; tempo.el --- Flexible template insertion
 
-;; Author: David Kegedal <davidk@lysator.liu.se >
+;; Copyright (C) 1994, 1995 Free Software Foundation, Inc.
+
+;; Author: David K}gedal <davidk@lysator.liu.se >
 ;; Created: 16 Feb 1994
-;; Version: 1.0
+;; K}gedal's last version number: 1.2.4
 ;; Keywords: extensions, languages, tools
 
 ;; This file is part of GNU Emacs.
@@ -19,8 +20,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., 59 Temple Place - Suite 330,
+;; Boston, MA 02111-1307, USA.
 
 ;;; Commentary:
 
@@ -28,9 +30,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,
 ;; 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.ai.mit.edu>
+;; Lars Lindberg <Lars.Lindberg@sypro.cap.se>
+;; Glen Whitney <Glen.Whitney@math.lsa.umich.edu>
+
 ;;; Code:
 
-(provide 'tempo)
+;; (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)
+
+(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'
+elements, unless the template function is given a prefix (or a non-nil
+argument). If this variable is non-NIL, the behaviour 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)
+
+(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)
+
+;;; 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")
 
 (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
+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\\([^\\b]+\\)\\="
+  "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")
+
+(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
 
+;;; First some useful functions and macros
+
+(defun tempo-mapc (fun lst)
+  (if (null lst) nil
+    (funcall fun (car lst))
+    (tempo-mapc fun (cdr lst))))
+
+(defmacro tempo-dolist (il &rest forms)
+  (let ((i (car il))
+       (l (car (cdr il))))
+    (list 'tempo-mapc
+         (list 'function (append (list 'lambda
+                                       (list i))
+                                 forms))
+         l)))
+(put 'tempo-dolist 'lisp-indent-function 1)
+
 ;;
 ;; tempo-define-template
 
@@ -142,12 +258,23 @@ The elements in ELEMENTS can be of several types:
  - 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) If `tempo-interactive' is non-nil, the user is
-   prompted in the minbuffer with PROMPT for a string to be inserted.
-   If `tempo-interactive is nil, it works like 'p.
- - (r . PROMPT) like the previou, but if `tempo-interactive' is nil
-   and `tempo-insert' is called with ON-REGION non-nil, the current
-   region is placed here.
+ - (p PROMPT <NAME> <NOINSERT>) 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 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, 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
@@ -156,9 +283,15 @@ The elements in ELEMENTS can be of several types:
  - '> 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.
+ - 'r> Like r, but it 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 parsed again."
+ - 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)))
@@ -169,8 +302,9 @@ The elements in ELEMENTS can be of several types:
                                 (concat "Insert a " name "."))
                             (list 'interactive "*P")
                             (list 'tempo-insert-template (list 'quote
-                                                      template-name)
-                                  'arg)))
+                                                               template-name)
+                                  (list 'if 'tempo-insert-region
+                                        (list 'not 'arg) 'arg))))
     (and tag
         (tempo-add-tag tag template-name taglist))
     command-name))
@@ -181,35 +315,78 @@ 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))
+`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)
+               (and (boundp 'zmacs-regions) ; For XEmacs
+                    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))
+         (mapcar (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) '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) '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
@@ -223,23 +400,131 @@ 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
-`tempo-forward-mark-list'.
+`tempo-mark'.
+
+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 i
+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-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
+    (mapcar (function (lambda (handler)
+                       (let ((result (funcall handler element)))
+                         (if result (throw 'found result)))))
+           tempo-user-elements)
+    (throw 'found nil)))
+
+;;;
+;;; tempo-forget-insertions
+
+(defun tempo-forget-insertions ()
+  "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 (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)))))
 
-PROMPT is the prompt string."
-  (if tempo-interactive
-      (insert (read-string prompt))
-    (tempo-insert-mark (point-marker))))
 
 ;;;
 ;;; tempo-process-and-insert-string
@@ -248,16 +533,14 @@ PROMPT is the prompt string."
   "Insert a string from a template.
 Run a string through the preprocessors in `tempo-insert-string-functions'
 and insert the results."
-
   (cond ((null tempo-insert-string-functions)
         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))
+        (tempo-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)))
@@ -319,7 +602,6 @@ and insert the results."
 
 (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."
 
@@ -327,53 +609,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))))
 
@@ -381,69 +686,92 @@ FINDER is a function or a string. Returns (STRING . POS)."
 ;;; tempo-complete-tag
 
 (defun tempo-complete-tag (&optional silent)
-  "Look for a tag and expand it..
+  "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.
+
+If a single match is found, the corresponding template is expanded in
+place of the matching string.
+
+If a partial completion or no match at all is found, and SILENT is
+non-NIL, the function will give a signal.
+
+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)))))))
 
-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.
+;;;
+;;; tempo-display-completions
+
+(defun tempo-display-completions (string tag-list)
+  "Show a buffer containing possible completions for STRING."
+  (if tempo-leave-completion-buffer
+      (with-output-to-temp-buffer "*Completions*"
+       (display-completion-list
+        (all-completions string tag-list)))
+    (save-window-excursion
+      (with-output-to-temp-buffer "*Completions*"
+       (display-completion-list
+        (all-completions string tag-list)))
+      (sit-for 32767))))
 
-If no match is found or a partial match is found, and SILENT is
-non-nil, the function will give a signal."
+;;;
+;;; 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)))
 
-  (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 (car match-info)
-                                             tag-list))))
-       
-             (if compl                 ;any match
-                 (delete-region match-start (point)))
-
-             (cond
-              ((null compl)
-               nil)
-              ((symbolp compl)
-               (tempo-insert-template compl nil)
-               (throw 'completed t))
-              ((eq compl t)
-               (tempo-insert-template (cdr (assoc match-string tag-list))
-                                      nil)
-               (throw 'completed t))
-              ((stringp compl)
-               (let ((compl2 (assoc compl tag-list)))
-                 (if compl2
-                     (tempo-insert-template (cdr compl2) nil)
-                   (insert compl)
-                   (if (string= match-string compl)
-                       (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))
+(provide 'tempo)
 
 ;;; tempo.el ends here