Fix my last change.
[gnus] / lisp / gnus-picon.el
index a6f4ed4..c0163fc 100644 (file)
@@ -1,5 +1,5 @@
 ;;; gnus-picon.el --- displaying pretty icons in Gnus
-;; Copyright (C) 1996,97 Free Software Foundation, Inc.
+;; Copyright (C) 1996,97,98,99 Free Software Foundation, Inc.
 
 ;; Author: Wes Hardaker <hardaker@ece.ucdavis.edu>
 ;; Keywords: news xpm annotation glyph faces
 
 ;;; Commentary:
 
-;;; TODO:
-;; See the comment in gnus-picons-remove
-
 ;;; Code:
 
 (require 'gnus)
-(require 'xpm)
+;; (require 'xpm)
 (require 'annotations)
 (require 'custom)
 (require 'gnus-art)
 
 (defgroup picons nil
   "Show pictures of people, domains, and newsgroups (XEmacs).
-For this to work, you must add gnus-group-display-picons to the
-gnus-summary-display-hook or to the gnus-article-display-hook
-depending on what gnus-picons-display-where is set to.  You must
-also add gnus-article-display-picons to gnus-article-display-hook."
+For this to work, you must switch on the `gnus-treat-display-picons'
+variable."
   :group 'gnus-visual)
 
-(defcustom gnus-picons-buffer "*Icon Buffer*"
-  "Buffer name to display the icons in if gnus-picons-display-where is 'picons."
-  :type 'string
-  :group 'picons)
-
 (defcustom gnus-picons-display-where 'picons
   "Where to display the group and article icons.
-Legal values are `article' and `picons'."
+Valid values are `article' and `picons'."
   :type '(choice symbol string)
   :group 'picons)
 
+(defcustom gnus-picons-has-modeline-p t
+  "*Whether the picons window should have a modeline.
+This is only useful if `gnus-picons-display-where' is `picons'."
+  :type 'boolean
+  :group 'picons)
+
 (defcustom gnus-picons-database "/usr/local/faces"
-  "Defines the location of the faces database.
+  "*Defines the location of the faces database.
 For information on obtaining this database of pretty pictures, please
 see http://www.cs.indiana.edu/picons/ftp/index.html"
   :type 'directory
   :group 'picons)
 
-(defcustom gnus-picons-news-directory "news"
-  "Sub-directory of the faces database containing the icons for newsgroups."
-  :type 'string
+(defcustom gnus-picons-news-directories '("news")
+  "*List of directories to search for newsgroups faces."
+  :type '(repeat string)
   :group 'picons)
+(define-obsolete-variable-alias 'gnus-picons-news-directory
+  'gnus-picons-news-directories)
 
 (defcustom gnus-picons-user-directories '("local" "users" "usenix" "misc")
-  "List of directories to search for user faces."
+  "*List of directories to search for user faces."
   :type '(repeat string)
   :group 'picons)
 
 (defcustom gnus-picons-domain-directories '("domains")
-  "List of directories to search for domain faces.
+  "*List of directories to search for domain faces.
 Some people may want to add \"unknown\" to this list."
   :type '(repeat string)
   :group 'picons)
 
 (defcustom gnus-picons-refresh-before-display nil
-  "If non-nil, display the article buffer before computing the picons."
+  "*If non-nil, display the article buffer before computing the picons."
   :type 'boolean
   :group 'picons)
 
-(defcustom gnus-picons-x-face-file-name
-  (format "/tmp/picon-xface.%s.xbm" (user-login-name))
-  "The name of the file in which to store the converted X-face header."
-  :type 'string
-  :group 'picons)
-
-(defcustom gnus-picons-convert-x-face (format "{ echo '/* Width=48, Height=48 */'; uncompface; } | icontopbm | pbmtoxbm > %s" gnus-picons-x-face-file-name)
-  "Command to convert the x-face header into a xbm file."
-  :type 'string
+(defcustom gnus-picons-group-excluded-groups nil
+  "*If this regexp matches the group name, group picons will be disabled."
+  :type 'regexp
   :group 'picons)
 
 (defcustom gnus-picons-display-as-address t
@@ -108,11 +100,11 @@ Some people may want to add \"unknown\" to this list."
       (when (featurep 'xpm)
        (push "xpm" types))
       types))
-  "List of suffixes on picon file names to try."
+  "*List of suffixes on picon file names to try."
   :type '(repeat string)
   :group 'picons)
 
-(defcustom gnus-picons-display-article-move-p t
+(defcustom gnus-picons-display-article-move-p nil
   "*Whether to move point to first empty line when displaying picons.
 This has only an effect if `gnus-picons-display-where' has value `article'."
   :type 'boolean
@@ -130,7 +122,7 @@ Otherwise the cache will be cleared every time you exit Gnus."
 
 (defcustom gnus-picons-piconsearch-url nil
   "*The url to query for picons.  Setting this to nil will disable it.
-The only plublicly available address currently known is
+The only publicly available address currently known is
 http://www.cs.indiana.edu:800/piconsearch.  If you know of any other,
 please tell me so that we can list it."
   :type '(choice (const :tag "Disable" :value nil)
@@ -139,8 +131,22 @@ please tell me so that we can list it."
                 (string))
   :group 'picons)
 
+(defface gnus-picons-xbm-face '((t (:foreground "black" :background "white")))
+  "Face to show xbm picons in."
+  :group 'picons)
+
+(defface gnus-picons-face '((t (:foreground "black" :background "white")))
+  "Face to show picons in."
+  :group 'picons)
+
+(defcustom gnus-picons-setup-hook nil
+  "Hook run in Picons buffers."
+  :group 'picons
+  :type 'hook)
+
 ;;; Internal variables:
 
+(defvar gnus-picons-setup-p nil)
 (defvar gnus-picons-processes-alist nil
   "Picons processes currently running and their environment.")
 (defvar gnus-picons-glyph-alist nil
@@ -150,17 +156,6 @@ List of pairs (KEY . GLYPH) where KEY is either a filename or an URL.")
   "Picons file names cache.
 List of pairs (KEY . NAME) where KEY is (USER HOST DBS) and NAME is an URL.")
 
-(defvar gnus-group-annotations nil
-  "List of annotations added/removed when selecting/exiting a group")
-(defvar gnus-group-annotations-lock nil)
-(defvar gnus-article-annotations nil
-  "List of annotations added/removed when selecting an article")
-(defvar gnus-article-annotations-lock nil)
-(defvar gnus-x-face-annotations nil
-  "List of annotations added/removed when selecting an article with an
-X-Face.")
-(defvar gnus-x-face-annotations-lock nil)
-
 (defvar gnus-picons-jobs-alist nil
   "List of jobs that still need be done.
 This is a list of (SYM-ANN TAG ARGS...) where SYM-ANN three annotations list,
@@ -173,121 +168,87 @@ arguments necessary for the job.")
 
 ;;; Functions:
 
-(defsubst gnus-picons-lock (symbol)
-  (intern (concat (symbol-name symbol) "-lock")))
-
-(defun gnus-picons-remove (symbol)
-  "Remove all annotations in variable named SYMBOL.
-This function is careful to set it to nil before removing anything so that
-asynchronous process don't get crazy."
-  ;; clear the lock
-  (set (gnus-picons-lock symbol) nil)
-  ;; clear all annotations
-  (mapc (function (lambda (item)
-                   (if (annotationp item)
-                       (delete-annotation item))))
-       (prog1 (symbol-value symbol)
-         (set symbol nil)))
-  ;; FIXME: there's a race condition here.  If a job is already
-  ;; running, it has already removed itself from this queue...  But
-  ;; will still display its picon.
-  ;; TODO: push a request to clear an annotation.  Then
-  ;; gnus-picons-next-job will be able to clean up when it gets the
-  ;; hand
-  (setq gnus-picons-jobs-alist (remassq symbol gnus-picons-jobs-alist)))
-
 (defun gnus-picons-remove-all ()
   "Removes all picons from the Gnus display(s)."
   (interactive)
-  (gnus-picons-remove 'gnus-article-annotations)
-  (gnus-picons-remove 'gnus-group-annotations)
-  (gnus-picons-remove 'gnus-x-face-annotations)
-  (when (bufferp gnus-picons-buffer)
-    (kill-buffer gnus-picons-buffer)))
+  (map-extents (function (lambda (ext unused) (delete-annotation ext) nil))
+              nil nil nil nil nil 'gnus-picon)
+  (setq gnus-picons-jobs-alist '())
+  ;; notify running job that it may have been preempted
+  (if (and (listp gnus-picons-job-already-running)
+          gnus-picons-job-already-running)
+      (setq gnus-picons-job-already-running t)))
 
 (defun gnus-get-buffer-name (variable)
   "Returns the buffer name associated with the contents of a variable."
-  (cond ((symbolp variable) (let ((newvar (cdr (assq variable
-                                                    gnus-window-to-buffer))))
-                             (cond ((symbolp newvar)
-                                    (symbol-value newvar))
-                                   ((stringp newvar) newvar))))
-        ((stringp variable) variable)))
-
-(defun gnus-picons-prepare-for-annotations (annotations)
-  "Prepare picons buffer for puting annotations memorized in ANNOTATIONS.
-ANNOTATIONS should be a symbol naming a variable wich contains a list of
-annotations.  Sets buffer to `gnus-picons-display-where'."
-  ;; let drawing catch up
-  (when gnus-picons-refresh-before-display
-    (sit-for 0))
-  (set-buffer (get-buffer-create
-              (gnus-get-buffer-name gnus-picons-display-where)))
-  (gnus-add-current-to-buffer-list)
+  (let ((buf (gnus-get-buffer-create
+             (gnus-window-to-buffer-helper
+              (cdr (assq variable gnus-window-to-buffer))))))
+    (and buf
+        (buffer-name buf))))
+
+(defun gnus-picons-buffer-name ()
+  (cond ((or (stringp gnus-picons-display-where)
+            (bufferp gnus-picons-display-where))
+        gnus-picons-display-where)
+       ((eq gnus-picons-display-where 'picons)
+        (if gnus-single-article-buffer
+            "*Picons*"
+          (concat "*Picons " gnus-newsgroup-name "*")))
+       (t
+        (gnus-get-buffer-name gnus-picons-display-where))))
+
+(defun gnus-picons-kill-buffer ()
+  (let ((buf (get-buffer (gnus-picons-buffer-name))))
+    (when (and (buffer-live-p buf)
+              (string-match "Picons" (buffer-name buf)))
+      (kill-buffer buf))))
+
+(defun gnus-picons-setup-buffer ()
+  (let ((name (gnus-picons-buffer-name)))
+    (save-excursion
+      (if (and (get-buffer name)
+              (with-current-buffer name
+                gnus-picons-setup-p))
+         (set-buffer name)
+       (set-buffer (gnus-get-buffer-create name))
+       (buffer-disable-undo)
+       (setq buffer-read-only t)
+       (run-hooks 'gnus-picons-setup-hook)
+       (set (make-local-variable 'gnus-picons-setup-p) t)
+       (add-hook 'gnus-summary-prepare-exit-hook 'gnus-picons-kill-buffer))
+      (current-buffer))))
+
+(defun gnus-picons-set-buffer ()
+  (set-buffer (gnus-picons-setup-buffer))
   (goto-char (point-min))
   (if (and (eq gnus-picons-display-where 'article)
           gnus-picons-display-article-move-p)
-      (when (search-forward "\n\n" nil t)
-       (forward-line -1))
-    (make-local-variable 'inhibit-read-only)
-    (setq buffer-read-only t
-         inhibit-read-only nil))
-  (gnus-picons-remove annotations))
-
-(defun gnus-picons-article-display-x-face ()
-  "Display the x-face header bitmap in the 'gnus-picons-display-where buffer."
-  ;; delete any old ones.
-  ;; This is needed here because gnus-picons-display-x-face will not
-  ;; be called if there is no X-Face header
-  (gnus-picons-remove 'gnus-x-face-annotations)
-  ;; display the new one.
-  (let ((gnus-article-x-face-command 'gnus-picons-display-x-face))
-    (gnus-article-display-x-face)))
-
-(defun gnus-picons-x-face-sentinel (process event)
-  (let* ((env (assq process gnus-picons-processes-alist))
-        (annot (cdr env)))
-    (setq gnus-picons-processes-alist (remassq process
-                                              gnus-picons-processes-alist))
-    (when annot
-      (set-annotation-glyph annot
-                           (make-glyph gnus-picons-x-face-file-name))
-      (if (memq annot gnus-x-face-annotations)
-         (delete-file gnus-picons-x-face-file-name)))))
-
-(defun gnus-picons-display-x-face (beg end)
-  "Function to display the x-face header in the picons window.
-To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
-  (interactive)
-  (if (featurep 'xface)
-      ;; Use builtin support
-      (let ((buf (current-buffer)))
-       (save-excursion
-         (gnus-picons-prepare-for-annotations 'gnus-x-face-annotations)
-         (setq gnus-x-face-annotations
-               (cons (make-annotation
-                      (vector 'xface
-                              :data (concat "X-Face: "
-                                            (buffer-substring beg end buf)))
-                                      nil 'text)
-                     gnus-x-face-annotations))))
-    ;; convert the x-face header to a .xbm file
-    (let* ((process-connection-type nil)
-          (annot (save-excursion
-                   (gnus-picons-prepare-for-annotations
-                    'gnus-x-face-annotations)
-                   (make-annotation nil nil 'text)))
-          (process (start-process-shell-command "gnus-x-face" nil 
-                                                gnus-picons-convert-x-face)))
-      (push annot gnus-x-face-annotations)
-      (push (cons process annot) gnus-picons-processes-alist)
-      (process-kill-without-query process)
-      (set-process-sentinel process 'gnus-picons-x-face-sentinel)
-      (process-send-region process beg end)
-      (process-send-eof process))))
+      (if (search-forward "\n\n" nil t)
+         (forward-line -1)
+       (goto-char (point-max)))
+    (setq buffer-read-only t)
+    (unless gnus-picons-has-modeline-p
+      (set-specifier has-modeline-p
+                    (list (list (current-buffer)
+                                (cons nil gnus-picons-has-modeline-p)))))))
+
+(defun gnus-picons-prepare-for-annotations ()
+  "Prepare picons buffer for putting annotations."
+  ;; let drawing catch up
+  (when gnus-picons-refresh-before-display
+    (sit-for 0))
+  (gnus-picons-set-buffer)
+  (gnus-picons-remove-all))
+
+(defun gnus-picons-make-annotation (&rest args)
+  (let ((annot (apply 'make-annotation args)))
+    (set-extent-property annot 'gnus-picon t)
+    (set-extent-property annot 'duplicable t)
+    annot))
 
 (defun gnus-article-display-picons ()
-  "Display faces for an author and his/her domain in gnus-picons-display-where."
+  "Display faces for an author and her domain in gnus-picons-display-where."
   (interactive)
   (let (from at-idx)
     (when (and (featurep 'xpm)
@@ -305,55 +266,83 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
                             (message-tokenize-header gnus-local-domain "."))
                       (message-tokenize-header (substring from (1+ at-idx))
                                                "."))))
-         (gnus-picons-prepare-for-annotations 'gnus-article-annotations)
+         (gnus-picons-prepare-for-annotations)
+         (gnus-group-display-picons)
+         (unless gnus-picons-display-article-move-p
+           (let ((buffer-read-only nil)
+                 (case-fold-search t))
+             (when (re-search-forward "^From *: *" nil t)
+               (when (search-forward from (gnus-point-at-eol) t)
+                 (gnus-put-text-property
+                  (match-beginning 0) (match-end 0)
+                  'invisible t)))))
          (if (null gnus-picons-piconsearch-url)
-             (setq gnus-article-annotations
-                   (nconc gnus-article-annotations
-                          (gnus-picons-display-pairs
-                           (gnus-picons-lookup-pairs
-                            addrs gnus-picons-domain-directories)
-                           (not (or gnus-picons-display-as-address
-                                    gnus-article-annotations))
-                           "." t)
-                          (if (and gnus-picons-display-as-address addrs)
-                              (list (make-annotation [string :data "@"] nil
-                                                     'text nil nil nil t)))
-                          (gnus-picons-display-picon-or-name
-                           (gnus-picons-lookup-user username addrs)
-                           username t)))
+             (progn
+               (gnus-picons-display-pairs (gnus-picons-lookup-pairs
+                                           addrs
+                                           gnus-picons-domain-directories)
+                                          gnus-picons-display-as-address
+                                          "." t)
+               (if (and gnus-picons-display-as-address addrs)
+                   (gnus-picons-make-annotation
+                    [string :data "@"] nil 'text nil nil nil t))
+               (gnus-picons-display-picon-or-name
+                (gnus-picons-lookup-user username addrs)
+                username t))
            (push (list 'gnus-article-annotations 'search username addrs
-                       gnus-picons-domain-directories t)
+                       gnus-picons-domain-directories t (point-marker))
                  gnus-picons-jobs-alist)
-           (gnus-picons-next-job))
-
-         (add-hook 'gnus-summary-exit-hook 'gnus-picons-remove-all))))))
+           (gnus-picons-next-job)))))))
 
 (defun gnus-group-display-picons ()
-  "Display icons for the group in the gnus-picons-display-where buffer."
+  "Display icons for the group in the `gnus-picons-display-where' buffer."
   (interactive)
   (when (and (featurep 'xpm)
-            (or (not (fboundp 'device-type)) (equal (device-type) 'x)))
-    (save-excursion
-      (gnus-picons-prepare-for-annotations 'gnus-group-annotations)
-      (if (null gnus-picons-piconsearch-url)
-         (setq gnus-group-annotations
-               (gnus-picons-display-pairs
-                (gnus-picons-lookup-pairs (reverse (message-tokenize-header
-                                                    gnus-newsgroup-name "."))
-                                          gnus-picons-news-directory)
-                t "."))
-       (push (list 'gnus-group-annotations 'search nil
-                   (message-tokenize-header gnus-newsgroup-name ".")
-                   (if (listp gnus-picons-news-directory)
-                       gnus-picons-news-directory
-                     (list gnus-picons-news-directory))
-                   nil)
-             gnus-picons-jobs-alist)
-       (gnus-picons-next-job))
-
-      (add-hook 'gnus-summary-exit-hook 'gnus-picons-remove-all))))
-
-(defsubst gnus-picons-lookup-internal (addrs dir)
+            (or (not (fboundp 'device-type)) (equal (device-type) 'x))
+            (or (null gnus-picons-group-excluded-groups)
+                (not (string-match gnus-picons-group-excluded-groups
+                                   gnus-newsgroup-name))))
+    (let* ((newsgroups (mail-fetch-field "newsgroups"))
+          (groups
+           (if (or gnus-picons-display-article-move-p
+                   (not newsgroups))
+               (list (gnus-group-real-name gnus-newsgroup-name))
+             (split-string newsgroups ",")))
+          group)
+      (save-excursion
+       (gnus-picons-prepare-for-annotations)
+       (while (setq group (pop groups))
+         (unless gnus-picons-display-article-move-p
+           (let ((buffer-read-only nil)
+                 (case-fold-search t))
+             (goto-char (point-min))
+             (if (and (re-search-forward "^Newsgroups *: *" nil t)
+                      (search-forward group (gnus-point-at-eol) t))
+                 (gnus-put-text-property
+                  (match-beginning 0) (match-end 0)
+                  'invisible t)
+               (let ((article-goto-body-goes-to-point-min-p nil))
+                 (article-goto-body))
+               (unless (bobp)
+                 (backward-char 1)))))
+         (if (null gnus-picons-piconsearch-url)
+             (gnus-picons-display-pairs
+              (gnus-picons-lookup-pairs
+               (reverse (split-string group "\\."))
+               gnus-picons-news-directories)
+              t ".")
+           (push (list 'gnus-group-annotations 'search nil
+                       (split-string group "\\.")
+                       (if (listp gnus-picons-news-directories)
+                           gnus-picons-news-directories
+                         (list gnus-picons-news-directories))
+                       nil (point-marker))
+                 gnus-picons-jobs-alist)
+           (gnus-picons-next-job))
+
+         (add-hook 'gnus-summary-exit-hook 'gnus-picons-remove-all))))))
+
+(defun gnus-picons-lookup-internal (addrs dir)
   (setq dir (expand-file-name dir gnus-picons-database))
   (gnus-picons-try-face (dolist (part (reverse addrs) dir)
                          (setq dir (expand-file-name part dir)))))
@@ -404,40 +393,30 @@ none, and whose CDR is the corresponding element of DOMAINS."
 
 (defun gnus-picons-display-picon-or-name (picon name &optional right-p)
   (cond (picon (gnus-picons-display-glyph picon name right-p))
-       (gnus-picons-display-as-address (list (make-annotation
+       (gnus-picons-display-as-address (list (gnus-picons-make-annotation
                                               (vector 'string :data name)
                                               nil 'text
                                               nil nil nil right-p)))))
 
 (defun gnus-picons-display-pairs (pairs &optional bar-p dot-p right-p)
   "Display picons in list PAIRS."
-  (let ((bar (and bar-p (or gnus-picons-display-as-address
-                           (annotations-in-region (point)
-                                                  (min (point-max)
-                                                       (1+ (point)))
-                                                  (current-buffer)))))
-       (domain-p (and gnus-picons-display-as-address dot-p))
+  (let ((domain-p (and gnus-picons-display-as-address dot-p))
        pair picons)
-    (while pairs
-      (setq pair (pop pairs)
-           picons (nconc (if (and domain-p picons (not right-p))
-                             (list (make-annotation
-                                    (vector 'string :data dot-p)
-                                    nil 'text nil nil nil right-p)))
-                         (gnus-picons-display-picon-or-name (car pair)
-                                                            (cadr pair)
-                                                            right-p)
-                         (if (and domain-p pairs right-p)
-                             (list (make-annotation
+    (when (and bar-p domain-p right-p
+              gnus-picons-display-article-move-p)
+      (setq picons (gnus-picons-display-glyph
+                   (let ((gnus-picons-file-suffixes '("xbm")))
+                     (gnus-picons-try-face
+                      gnus-xmas-glyph-directory "bar."))
+                   nil right-p)))
+    (while (setq pair (pop pairs))
+      (setq picons (nconc picons
+                         (gnus-picons-display-picon-or-name
+                          (car pair) (cadr pair) right-p)
+                         (if (and domain-p pairs)
+                             (list (gnus-picons-make-annotation
                                     (vector 'string :data dot-p)
-                                    nil 'text nil nil nil right-p)))
-                         (when (and bar domain-p)
-                           (setq bar nil)
-                           (gnus-picons-display-glyph
-                            (gnus-picons-try-face gnus-xmas-glyph-directory
-                                                  "bar.")
-                            nil t))
-                         picons)))
+                                    nil 'text nil nil nil right-p))))))
     picons))
 
 (defun gnus-picons-try-face (dir &optional filebase)
@@ -446,31 +425,37 @@ none, and whose CDR is the corresponding element of DOMAINS."
         (key (concat dir filebase))
         (glyph (cdr (assoc key gnus-picons-glyph-alist)))
         (suffixes gnus-picons-file-suffixes)
-        f)
-    (while (and suffixes (null glyph))
-      (when (file-exists-p (setq f (expand-file-name (concat filebase
-                                                                (pop suffixes))
-                                                        dir)))
-       (setq glyph (make-glyph f))
+        f suf)
+    (while (setq suf (pop suffixes))
+      (when (file-exists-p (setq f (expand-file-name
+                                   (concat filebase suf)
+                                   dir)))
+       (setq suffixes nil
+             glyph (make-glyph f))
+       (if (equal suf "xbm")
+           (set-glyph-face glyph 'gnus-picons-xbm-face)
+         (set-glyph-face glyph 'gnus-picons-face))
        (push (cons key glyph) gnus-picons-glyph-alist)))
     glyph))
 
 (defun gnus-picons-display-glyph (glyph &optional part rightp)
-  (let ((new (make-annotation glyph (point) 'text nil nil nil rightp)))
+  (set-glyph-baseline glyph 70)
+  (let ((new (gnus-picons-make-annotation
+             glyph (point) 'text nil nil nil rightp)))
     (when (and part gnus-picons-display-as-address)
-      (set-annotation-data new (cons new
-                                    (make-glyph (vector 'string :data part))))
+      (set-annotation-data
+       new (cons new (make-glyph (vector 'string :data part))))
       (set-annotation-action new 'gnus-picons-action-toggle))
     (nconc
      (list new)
      (if (and (eq major-mode 'gnus-article-mode)
              (not gnus-picons-display-as-address)
              (not part))
-        (list (make-annotation [string :data " "]
-                               (point) 'text nil nil nil rightp))))))
+        (list (gnus-picons-make-annotation [string :data " "] (point)
+                                           'text nil nil nil rightp))))))
 
 (defun gnus-picons-action-toggle (data)
-  "Toggle annotation"
+  "Toggle annotation."
   (interactive "e")
   (let* ((annot (car data))
         (glyph (annotation-glyph annot)))
@@ -478,9 +463,10 @@ none, and whose CDR is the corresponding element of DOMAINS."
     (set-annotation-data annot (cons annot glyph))))
 
 (defun gnus-picons-clear-cache ()
-  "Clear the picons cache"
+  "Clear the picons cache."
   (interactive)
-  (setq gnus-picons-glyph-alist nil))
+  (setq gnus-picons-glyph-alist nil
+       gnus-picons-url-alist nil))
 
 (gnus-add-shutdown 'gnus-picons-close 'gnus)
 
@@ -491,20 +477,21 @@ none, and whose CDR is the corresponding element of DOMAINS."
 
 ;;; Query a remote DB.  This requires some stuff from w3 !
 
-(require 'url)
-(require 'w3-forms)
+(eval-and-compile
+  (ignore-errors
+    (require 'url)
+    (require 'w3-forms)))
 
 (defun gnus-picons-url-retrieve (url fn arg)
   (let ((old-asynch (default-value 'url-be-asynchronous))
        (url-working-buffer (generate-new-buffer " *picons*"))
-       (url-request-method nil)
        (url-package-name "Gnus")
-       (url-package-version gnus-version-number))
+       (url-package-version gnus-version-number)
+       url-request-method)
     (setq-default url-be-asynchronous t)
     (save-excursion
       (set-buffer url-working-buffer)
       (setq url-be-asynchronous t
-           url-show-status nil
            url-current-callback-data arg
            url-current-callback-func fn)
       (url-retrieve url t))
@@ -534,84 +521,102 @@ none, and whose CDR is the corresponding element of DOMAINS."
 
 (defun gnus-picons-parse-value (name)
   (goto-char (point-min))
-  (re-search-forward (concat "<strong>"
-                            (regexp-quote name)
-                            "</strong> *= *<kbd> *\\([^ <][^<]*\\) *</kbd>"))
-  (buffer-substring (match-beginning 1) (match-end 1)))
+  (if (re-search-forward (concat "<strong>"
+                                (regexp-quote name)
+                                "</strong> *= *<kbd> *\\([^ <][^<]*\\) *</kbd>")
+                        nil t)
+      (buffer-substring (match-beginning 1) (match-end 1))))
 
 (defun gnus-picons-parse-filenames ()
   ;; returns an alist of ((USER ADDRS DB) . URL)
-  (let* ((case-fold-search t)
-        (user (gnus-picons-parse-value "user"))
-        (host (gnus-picons-parse-value "host"))
-        (dbs (message-tokenize-header (gnus-picons-parse-value "db") " "))
-        (start-re
-         (concat
-          ;; dbs
-          "^\\(" (mapconcat 'identity dbs "\\|") "\\)/"
-          ;; host
-          "\\(\\(" (replace-in-string host "\\." "/\\|" t) "/\\|MISC/\\)*\\)"
-          ;; user
-          "\\(" (regexp-quote user) "\\|unknown\\)/"
-          "face\\."))
-        cur-db cur-host cur-user types res)
+  (let ((case-fold-search t)
+       (user (gnus-picons-parse-value "user"))
+       (host (gnus-picons-parse-value "host"))
+       (dbs (message-tokenize-header (gnus-picons-parse-value "db") " "))
+       start-re cur-db cur-host cur-user types res)
     ;; now point will be somewhere in the header.  Find beginning of
     ;; entries
-    (re-search-forward "<p>[ \t\n]*")
-    (while (re-search-forward start-re nil t)
-      (setq cur-db (buffer-substring (match-beginning 1) (match-end 1))
-           cur-host (buffer-substring (match-beginning 2) (match-end 2))
-           cur-user (buffer-substring (match-beginning 4) (match-end 4))
-           cur-host (nreverse (message-tokenize-header cur-host "/")))
-      ;; XXX - KLUDGE: there is a blank picon in news/MISC/unknown
-      (unless (and (string-equal cur-db "news")
-                  (string-equal cur-user "unknown")
-                  (equal cur-host '("MISC")))
-       ;; ok now we have found an entry (USER HOST DB), find the
-       ;; corresponding picon URL
-       (save-restriction
-         ;; restrict region to this entry
-         (narrow-to-region (point) (search-forward "<br>"))
-         (goto-char (point-min))
-         (setq types gnus-picons-file-suffixes)
-         (while (and types
-                     (not (re-search-forward
-                           (concat "<a[ \t\n]+href=\"\\([^\"]*\\."
-                                   (regexp-quote (car types)) "\\)\"")
-                           nil t)))
-           (pop types))
-         (push (cons (list cur-user cur-host cur-db)
-                     (buffer-substring (match-beginning 1) (match-end 1)))
-               res))))
-    (nreverse res)))
+    (when (and user host dbs)
+      (setq start-re
+           (concat
+            ;; dbs
+            "^\\(" (mapconcat 'regexp-quote dbs "\\|") "\\)/"
+            ;; host
+            "\\(\\(" (mapconcat 'regexp-quote
+                                (message-tokenize-header host ".") "/\\|")
+            "/\\|MISC/\\)*\\)"
+            ;; user
+            "\\(" (regexp-quote user) "\\|unknown\\)/"
+            "face\\."))
+      (re-search-forward "<p>[ \t\n]*")
+      (while (re-search-forward start-re nil t)
+       (setq cur-db (buffer-substring (match-beginning 1) (match-end 1))
+             cur-host (buffer-substring (match-beginning 2) (match-end 2))
+             cur-user (buffer-substring (match-beginning 4) (match-end 4))
+             cur-host (nreverse (message-tokenize-header cur-host "/")))
+       ;; XXX - KLUDGE: there is a blank picon in news/MISC/unknown
+       (unless (and (string-equal cur-db "news")
+                    (string-equal cur-user "unknown")
+                    (equal cur-host '("MISC")))
+         ;; ok now we have found an entry (USER HOST DB), find the
+         ;; corresponding picon URL
+         (save-restriction
+           ;; restrict region to this entry
+           (narrow-to-region (point) (search-forward "<br>"))
+           (goto-char (point-min))
+           (setq types gnus-picons-file-suffixes)
+           (while (and types
+                       (not (re-search-forward
+                             (concat "<a[ \t\n]+href=\"\\([^\"]*\\."
+                                     (regexp-quote (car types)) "\\)\"")
+                             nil t)))
+             (pop types))
+           (push (cons (list cur-user cur-host cur-db)
+                       (buffer-substring (match-beginning 1) (match-end 1)))
+                 res))))
+      (nreverse res))))
 
 ;;; picon network display functions :
 
-(defun gnus-picons-network-display-internal (sym-ann glyph part right-p)
-  (set-buffer
-   (get-buffer-create (gnus-get-buffer-name gnus-picons-display-where)))
-  (set sym-ann (nconc (symbol-value sym-ann)
-                     (gnus-picons-display-picon-or-name glyph part right-p)))
+(defun gnus-picons-network-display-internal (sym-ann glyph part right-p marker)
+  (let ((buf (marker-buffer marker))
+       (pos (marker-position marker)))
+    (if (and buf pos)
+       (save-excursion
+         (set-buffer buf)
+         (goto-char pos)
+         (gnus-picons-display-picon-or-name glyph part right-p))))
   (gnus-picons-next-job-internal))
 
-(defun gnus-picons-network-display-callback (url part sym-ann right-p)
+(defun gnus-picons-network-display-callback (url part sym-ann right-p marker)
   (let ((glyph (gnus-picons-make-glyph (cdr (assoc url-current-mime-type
                                                   w3-image-mappings)))))
     (kill-buffer (current-buffer))
     (push (cons url glyph) gnus-picons-glyph-alist)
-    (gnus-picons-network-display-internal sym-ann glyph part right-p)))
-
-(defun gnus-picons-network-display (url part sym-ann right-p)
+    ;; only do the job if it has not been preempted.
+    (if (equal gnus-picons-job-already-running
+              (list sym-ann 'picon url part right-p marker))
+       (gnus-picons-network-display-internal
+        sym-ann glyph part right-p marker)
+      (gnus-picons-next-job-internal))))
+
+(defun gnus-picons-network-display (url part sym-ann right-p marker)
   (let ((cache (assoc url gnus-picons-glyph-alist)))
     (if (or cache (null url))
-       (gnus-picons-network-display-internal sym-ann (cdr cache) part right-p)
+       (gnus-picons-network-display-internal
+        sym-ann (cdr cache) part right-p marker)
       (gnus-picons-url-retrieve url 'gnus-picons-network-display-callback
-                               (list url part sym-ann right-p)))))
+                               (list url part sym-ann right-p marker)))))
 
 ;;; search job functions
 
+(defun gnus-picons-display-bar-p ()
+  (if (eq gnus-picons-display-where 'article)
+      gnus-picons-display-article-move-p
+    gnus-picons-display-as-address))
+
 (defun gnus-picons-network-search-internal (user addrs dbs sym-ann right-p
-                                                &optional fnames)
+                                                marker &optional fnames)
   (let (curkey dom pfx url dbs-tmp cache new-jobs)
     ;; First do the domain search
     (dolist (part (if right-p
@@ -640,8 +645,8 @@ none, and whose CDR is the corresponding element of DOMAINS."
        (push (setq cache (cons curkey url)) gnus-picons-url-alist))
       ;; Put this glyph in the job list
       (if (and (not (eq dom part)) gnus-picons-display-as-address)
-         (push (list sym-ann "." right-p) new-jobs))
-      (push (list sym-ann 'picon (cdr cache) part right-p) new-jobs))
+         (push (list sym-ann "." right-p marker) new-jobs))
+      (push (list sym-ann 'picon (cdr cache) part right-p marker) new-jobs))
     ;; next, the user search
     (when user
       (setq curkey (list user dom gnus-picons-user-directories))
@@ -663,22 +668,34 @@ none, and whose CDR is the corresponding element of DOMAINS."
            (push (setq cache (cons curkey (cdr picon)))
                  gnus-picons-url-alist)))
       (if (and gnus-picons-display-as-address new-jobs)
-         (push (list sym-ann "@" right-p) new-jobs))
-      (push (list sym-ann 'picon (cdr cache) user right-p) new-jobs))
-    (setq gnus-picons-jobs-alist (nconc (nreverse new-jobs)
-                                       gnus-picons-jobs-alist))
+         (push (list sym-ann "@" right-p marker) new-jobs))
+      (push (list sym-ann 'picon (cdr cache) user right-p marker) new-jobs))
+    (if (and (gnus-picons-display-bar-p) (not right-p))
+       (push (list sym-ann 'bar right-p marker) new-jobs))
+    ;; only put the jobs in the queue if this job has not been preempted.
+    (if (equal gnus-picons-job-already-running
+              (list sym-ann 'search user addrs dbs right-p marker))
+       (setq gnus-picons-jobs-alist
+             (nconc (if (and (gnus-picons-display-bar-p) right-p)
+                        (list (list sym-ann 'bar right-p marker)))
+                    (nreverse new-jobs)
+                    gnus-picons-jobs-alist)))
     (gnus-picons-next-job-internal)))
 
-(defun gnus-picons-network-search-callback (user addrs dbs sym-ann right-p)
-  (gnus-picons-network-search-internal user addrs dbs sym-ann right-p
-                                      (prog1 (gnus-picons-parse-filenames)
-                                        (kill-buffer (current-buffer)))))
+(defun gnus-picons-network-search-callback (user addrs dbs sym-ann right-p
+                                                marker)
+  (gnus-picons-network-search-internal
+   user addrs dbs sym-ann right-p marker
+   (prog1
+       (gnus-picons-parse-filenames)
+     (kill-buffer (current-buffer)))))
 
-(defun gnus-picons-network-search (user addrs dbs sym-ann right-p)
+;; Initiate a query on the picon database
+(defun gnus-picons-network-search (user addrs dbs sym-ann right-p marker)
   (let* ((host (mapconcat 'identity addrs "."))
         (key (list (or user "unknown") host (if user
-                                                 gnus-picons-user-directories
-                                               dbs)))
+                                                gnus-picons-user-directories
+                                              dbs)))
         (cache (assoc key gnus-picons-url-alist)))
     (if (null cache)
        (gnus-picons-url-retrieve
@@ -692,35 +709,42 @@ none, and whose CDR is the corresponding element of DOMAINS."
                                     dbs)
                                   "+"))
         'gnus-picons-network-search-callback
-        (list user addrs dbs sym-ann right-p))
-      (gnus-picons-network-search-internal user addrs dbs sym-ann right-p))))
+        (list user addrs dbs sym-ann right-p marker))
+      (gnus-picons-network-search-internal
+       user addrs dbs sym-ann right-p marker))))
 
 ;;; Main jobs dispatcher function
-;; Given that XEmacs is not really multi threaded, this locking should
-;; be sufficient
 
 (defun gnus-picons-next-job-internal ()
-  (if gnus-picons-jobs-alist
-      (let* ((job (pop gnus-picons-jobs-alist))
-            (sym-ann (pop job))
-            (tag (pop job)))
-       (if tag
-           (cond ((stringp tag);; (SYM-ANN "..." RIGHT-P)
-                  (gnus-picons-network-display-internal sym-ann nil tag
-                                                        (pop job)))
-                 ((eq 'search tag);; (SYM-ANN 'search USER ADDRS DBS RIGHT-P)
-                  (gnus-picons-network-search
-                   (pop job) (pop job) (pop job) sym-ann (pop job)))
-                 ((eq 'picon tag);; (SYM-ANN 'picon URL PART RIGHT-P)
-                  (gnus-picons-network-display
-                   (pop job) (pop job) sym-ann (pop job)))
-                 (t (error "Unknown picon job tag %s" tag)))))
-    (setq gnus-picons-job-already-running nil)))
+  (when (setq gnus-picons-job-already-running (pop gnus-picons-jobs-alist))
+    (let* ((job gnus-picons-job-already-running)
+          (sym-ann (pop job))
+          (tag (pop job)))
+      (when tag
+       (cond
+        ((stringp tag);; (SYM-ANN "..." RIGHT-P MARKER)
+         (gnus-picons-network-display-internal
+          sym-ann nil tag (pop job) (pop job)))
+        ((eq 'bar tag);; (SYM-ANN 'bar RIGHT-P MARKER)
+         (gnus-picons-network-display-internal
+          sym-ann
+          (let ((gnus-picons-file-suffixes '("xbm")))
+            (gnus-picons-try-face
+             gnus-xmas-glyph-directory "bar."))
+          nil (pop job) (pop job)))
+        ((eq 'search tag);; (SYM-ANN 'search USER ADDRS DBS RIGHT-P MARKER)
+         (gnus-picons-network-search
+          (pop job) (pop job) (pop job) sym-ann (pop job) (pop job)))
+        ((eq 'picon tag);; (SYM-ANN 'picon URL PART RIGHT-P MARKER)
+         (gnus-picons-network-display
+          (pop job) (pop job) sym-ann (pop job) (pop job)))
+        (t
+         (setq gnus-picons-job-already-running nil)
+         (error "Unknown picon job tag %s" tag)))))))
 
 (defun gnus-picons-next-job ()
-  "Start processing the job queue."
+  "Start processing the job queue if it is not in progress."
   (unless gnus-picons-job-already-running
-    (setq gnus-picons-job-already-running t)
     (gnus-picons-next-job-internal)))
 
 (provide 'gnus-picon)