# HG changeset patch # User Roland Winkler # Date 1176652987 0 # Node ID f00e4356c78dd55a88deeda1abd4babb288ab4ca # Parent 5a8fd913621caea4b759a5629c49047ec6956f86 (bibtex-field-list): Use functionp. (bibtex-make-field): Check that INIT is a string. Use functionp. diff -r 5a8fd913621c -r f00e4356c78d lisp/textmodes/bibtex.el --- a/lisp/textmodes/bibtex.el Sun Apr 15 14:52:51 2007 +0000 +++ b/lisp/textmodes/bibtex.el Sun Apr 15 16:03:07 2007 +0000 @@ -2831,7 +2831,7 @@ (push (list "key" "Used for reference key creation if author and editor fields are missing" (if (or (stringp bibtex-include-OPTkey) - (fboundp bibtex-include-OPTkey)) + (functionp bibtex-include-OPTkey)) bibtex-include-OPTkey)) optional)) (if (member-ignore-case entry-type bibtex-include-OPTcrossref) @@ -3029,7 +3029,7 @@ the new field. If INTERACTIVE is non-nil, move point to the end of the new field. Otherwise move point past the new field. MOVE and INTERACTIVE are t when called interactively. -INIT is surrounded by delimiters, unless NODELIM is non-nil." +INIT is surrounded by field delimiters, unless NODELIM is non-nil." (interactive (list (let ((completion-ignore-case t) (field-list (bibtex-field-list @@ -3059,14 +3059,12 @@ (indent-to-column (+ bibtex-entry-offset bibtex-text-indentation))) (let ((init (nth 2 field))) - (insert (if nodelim - "" - (bibtex-field-left-delimiter)) - (cond ((stringp init) init) - ((fboundp init) (funcall init)) - (t "")) - (if nodelim - "" + (if (not init) (setq init "") + (if (functionp init) (setq init (funcall init))) + (unless (stringp init) (error "`%s' is not a string" init))) + ;; NODELIM is required by `bibtex-insert-kill' + (if nodelim (insert init) + (insert (bibtex-field-left-delimiter) init (bibtex-field-right-delimiter)))) (when interactive ;; (bibtex-find-text nil nil bibtex-help-message)