Implement tool-bar separators for non-GTK tool-bars.

* lisp/tool-bar.el (tool-bar--image-expression): New function.
(tool-bar-local-item, tool-bar--image-exp): Use it.
(tool-bar-setup): Initialize tool-bar-separator-image-expression.
Use :enable instead of :visible to avoid changing the tool-bar
configuration unnecessarily.

* src/keyboard.c (Vtool_bar_separator_image_expression): New variable.
(parse_tool_bar_item): Use it to obtain image separators for
displays not using native tool-bar separators.

* src/xdisp.c (build_desired_tool_bar_string): Don't handle separators
specially, since this is now done in parse_tool_bar_item.

* lisp/info.el (info-tool-bar-map): Add separators.

3 files changed, 46 insertions, 41 deletions

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 4ed1a75cda2..0c363ab7a03 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,17 @@
+2010-12-20  Chong Yidong  <cyd@stupidchicken.com>
+
+        * vc/diff.el (diff-better-file-name): Function deleted;
+        abbreviating file name creates problems with shell-quote-argument.
+        (diff-no-select): Just use expand-file-name.
+
+        * tool-bar.el (tool-bar--image-expression): New function.
+        (tool-bar-local-item, tool-bar--image-exp): Use it.
+        (tool-bar-setup): Initialize tool-bar-separator-image-expression.
+        Use :enable instead of :visible to avoid changing the tool-bar
+        configuration unnecessarily.
+
+        * info.el (info-tool-bar-map): Add separators.
+
 2010-12-17  Ken Brown  <kbrown@cornell.edu>
 
         * loadup.el: Use version numbers in Cygwin build.
diff --git a/lisp/info.el b/lisp/info.el
index 7c0333f6b8e..ad92914a54d 100644
--- a/lisp/info.el
+++ b/lisp/info.el
@@ -3769,14 +3769,17 @@ If FORK is non-nil, it is passed to `Info-goto-node'."
                                    :rtl "left-arrow"
                                    :label "Forward"
                                    :vert-only t)
+    (define-key-after map [separator-1] menu-bar-separator)
     (tool-bar-local-item-from-menu 'Info-prev "prev-node" map Info-mode-map
                                    :rtl "next-node")
     (tool-bar-local-item-from-menu 'Info-next "next-node" map Info-mode-map
                                    :rtl "prev-node")
     (tool-bar-local-item-from-menu 'Info-up "up-node" map Info-mode-map
                                    :vert-only t)
+    (define-key-after map [separator-2] menu-bar-separator)
     (tool-bar-local-item-from-menu 'Info-top-node "home" map Info-mode-map)
     (tool-bar-local-item-from-menu 'Info-goto-node "jump-to" map Info-mode-map)
+    (define-key-after map [separator-3] menu-bar-separator)
     (tool-bar-local-item-from-menu 'Info-index "index" map Info-mode-map
                                    :label "Index Search")
     (tool-bar-local-item-from-menu 'Info-search "search" map Info-mode-map)
diff --git a/lisp/tool-bar.el b/lisp/tool-bar.el
index 7c7216ed7d3..e819cbe99a8 100644
--- a/lisp/tool-bar.el
+++ b/lisp/tool-bar.el
@@ -139,6 +139,26 @@ Use this function only to make bindings in the global value of `tool-bar-map'.
 To define items in any other map, use `tool-bar-local-item'."
   (apply 'tool-bar-local-item icon def key tool-bar-map props))
 
+(defun tool-bar--image-expression (icon)
+  "Return an expression that evaluates to an image spec for ICON."
+  (let* ((fg (face-attribute 'tool-bar :foreground))
+         (bg (face-attribute 'tool-bar :background))
+         (colors (nconc (if (eq fg 'unspecified) nil (list :foreground fg))
+                        (if (eq bg 'unspecified) nil (list :background bg))))
+         (xpm-spec (list :type 'xpm :file (concat icon ".xpm")))
+         (xpm-lo-spec (list :type 'xpm :file
+                            (concat "low-color/" icon ".xpm")))
+         (pbm-spec (append (list :type 'pbm :file
+                                 (concat icon ".pbm")) colors))
+         (xbm-spec (append (list :type 'xbm :file
+                                 (concat icon ".xbm")) colors)))
+    `(find-image (cond ((not (display-color-p))
+                        ',(list pbm-spec xbm-spec xpm-lo-spec xpm-spec))
+                       ((< (display-color-cells) 256)
+                        ',(list xpm-lo-spec xpm-spec pbm-spec xbm-spec))
+                       (t
+                        ',(list xpm-spec pbm-spec xbm-spec))))))
+
 ;;;###autoload
 (defun tool-bar-local-item (icon def key map &rest props)
   "Add an item to the tool bar in map MAP.
@@ -151,24 +171,7 @@ ICON is the base name of a file containing the image to use.  The
 function will first try to use low-color/ICON.xpm if `display-color-cells'
 is less or equal to 256, then ICON.xpm, then ICON.pbm, and finally
 ICON.xbm, using `find-image'."
-  (let* ((fg (face-attribute 'tool-bar :foreground))
-         (bg (face-attribute 'tool-bar :background))
-         (colors (nconc (if (eq fg 'unspecified) nil (list :foreground fg))
-                        (if (eq bg 'unspecified) nil (list :background bg))))
-         (xpm-spec (list :type 'xpm :file (concat icon ".xpm")))
-         (xpm-lo-spec (list :type 'xpm :file
-                            (concat "low-color/" icon ".xpm")))
-         (pbm-spec (append (list :type 'pbm :file
-                                 (concat icon ".pbm")) colors))
-         (xbm-spec (append (list :type 'xbm :file
-                                 (concat icon ".xbm")) colors))
-         (image-exp `(find-image
-                      (cond ((not (display-color-p))
-                             ',(list pbm-spec xbm-spec xpm-lo-spec xpm-spec))
-                            ((< (display-color-cells) 256)
-                             ',(list xpm-lo-spec xpm-spec pbm-spec xbm-spec))
-                            (t
-                             ',(list xpm-spec pbm-spec xbm-spec))))))
+  (let* ((image-exp (tool-bar--image-expression icon)))
     (define-key-after map (vector key)
       `(menu-item ,(symbol-name key) ,def :image ,image-exp ,@props))))
 
@@ -203,24 +206,7 @@ holds a keymap."
     (setq from-map global-map))
   (let* ((menu-bar-map (lookup-key from-map [menu-bar]))
          (keys (where-is-internal command menu-bar-map))
-         (fg (face-attribute 'tool-bar :foreground))
-         (bg (face-attribute 'tool-bar :background))
-         (colors (nconc (if (eq fg 'unspecified) nil (list :foreground fg))
-                        (if (eq bg 'unspecified) nil (list :background bg))))
-         (xpm-spec (list :type 'xpm :file (concat icon ".xpm")))
-         (xpm-lo-spec (list :type 'xpm :file
-                            (concat "low-color/" icon ".xpm")))
-         (pbm-spec (append (list :type 'pbm :file
-                                 (concat icon ".pbm")) colors))
-         (xbm-spec (append (list :type 'xbm :file
-                                 (concat icon ".xbm")) colors))
-         (image-exp `(find-image
-                      (cond ((not (display-color-p))
-                             ',(list pbm-spec xbm-spec xpm-lo-spec xpm-spec))
-                            ((< (display-color-cells) 256)
-                             ',(list xpm-lo-spec xpm-spec pbm-spec xbm-spec))
-                            (t
-                             ',(list xpm-spec pbm-spec xbm-spec)))))
+         (image-exp (tool-bar--image-expression icon))
          submap key)
     ;; We'll pick up the last valid entry in the list of keys if
     ;; there's more than one.
@@ -257,32 +243,34 @@ holds a keymap."
 ;;; Set up some global items.  Additions/deletions up for grabs.
 
 (defun tool-bar-setup ()
+  (setq tool-bar-separator-image-expression
+        (tool-bar--image-expression "separator"))
   (tool-bar-add-item-from-menu 'find-file "new" nil :label "New File"
                                :vert-only t)
   (tool-bar-add-item-from-menu 'menu-find-file-existing "open" nil
                                :label "Open" :vert-only t)
   (tool-bar-add-item-from-menu 'dired "diropen" nil :vert-only t)
   (tool-bar-add-item-from-menu 'kill-this-buffer "close" nil :vert-only t)
-  (tool-bar-add-item-from-menu 'save-buffer "save" nil :vert-only t
+  (tool-bar-add-item-from-menu 'save-buffer "save" nil
                                :label "Save"
-                               :visible '(or buffer-file-name
+                               :enable '(or buffer-file-name
                                              (not (eq 'special
                                                       (get major-mode
                                                            'mode-class)))))
   (define-key-after (default-value 'tool-bar-map) [separator-1] menu-bar-separator)
   (tool-bar-add-item-from-menu 'undo "undo" nil :vert-only t
-                               :visible '(not (eq 'special (get major-mode
+                               :enable '(not (eq 'special (get major-mode
                                                                 'mode-class))))
   (define-key-after (default-value 'tool-bar-map) [separator-2] menu-bar-separator)
   (tool-bar-add-item-from-menu (lookup-key menu-bar-edit-menu [cut])
                                "cut" nil :vert-only t
-                               :visible '(not (eq 'special (get major-mode
+                               :enable '(not (eq 'special (get major-mode
                                                                 'mode-class))))
   (tool-bar-add-item-from-menu (lookup-key menu-bar-edit-menu [copy])
                                "copy" nil :vert-only t)
   (tool-bar-add-item-from-menu (lookup-key menu-bar-edit-menu [paste])
                                "paste" nil :vert-only t
-                               :visible '(not (eq 'special (get major-mode
+                               :enable '(not (eq 'special (get major-mode
                                                                 'mode-class))))
   (define-key-after (default-value 'tool-bar-map) [separator-3] menu-bar-separator)
   (tool-bar-add-item-from-menu 'nonincremental-search-forward "search"