*** empty log message ***
[gnus] / lisp / gnus-picon.el
index 21c3f09..78e6a50 100644 (file)
@@ -1,5 +1,5 @@
 ;;; gnus-picon.el --- displaying pretty icons in Gnus
-;; Copyright (C) 1996 Free Software Foundation, Inc.
+;; Copyright (C) 1996,97 Free Software Foundation, Inc.
 
 ;; Author: Wes Hardaker <hardaker@ece.ucdavis.edu>
 ;; Keywords: news xpm annotation glyph faces
 
 ;;; Commentary:
 
-;; Usage:
-;;     - You must have XEmacs (19.12 or above I think) to use this.
-;;     - Read the variable descriptions below.
-;;
-;;     - chose a setup:
-;;
-;;       1) display the icons in its own buffer:
-;;
-;;          (add-hook 'gnus-article-display-hook 'gnus-article-display-picons t)
-;;          (add-hook 'gnus-summary-prepare-hook 'gnus-group-display-picons t)
-;;          (setq gnus-picons-display-where 'picons)
-;;
-;;          Then add the picons buffer to your display configuration:
-;;          The picons buffer needs to be at least 48 pixels high,
-;;          which for me is 5 lines:
-;;
-;;          (gnus-add-configuration
-;;           '(article (vertical 1.0 
-;;                             (group 6)
-;;                             (picons 5)
-;;                             (summary .25 point)
-;;                             (article 1.0))))
-;;
-;;          (gnus-add-configuration
-;;           '(summary (vertical 1.0 (group 6)
-;;                      (picons 5)
-;;                      (summary 1.0 point))))
-;;
-;;       2) display the icons in the summary buffer
-;;
-;;          (add-hook 'gnus-article-display-hook 'gnus-article-display-picons t)
-;;          (add-hook 'gnus-summary-prepare-hook 'gnus-group-display-picons t)
-;;          (setq gnus-picons-display-where 'summary)
-;;
-;;       3) display the icons in the article buffer
-;;
-;;          (add-hook 'gnus-article-display-hook 'gnus-article-display-picons t)
-;;          (add-hook 'gnus-article-prepare-hook 'gnus-group-display-picons t)
-;;          (setq gnus-picons-display-where 'article)
-;;
-;;
-;; Warnings:
-;;     - I'm not even close to being a lisp expert.
-;;     - The 't' (append) flag MUST be in the add-hook line
-;;
-;; TODO:
-;;     - Remove the TODO section in the headers.
-;;
-
 ;;; Code:
 
-(require 'gnus-load)
+(require 'gnus)
 (require 'xpm)
 (require 'annotations)
-
-(defvar gnus-picons-buffer "*Icon Buffer*"
-  "Buffer name to display the icons in if gnus-picons-display-where is 'picons.")
-
-(defvar gnus-picons-display-where 'picons
-  "Where to display the group and article icons.")
-
-(defvar gnus-picons-database "/usr/local/faces"
+(require 'custom)
+(require 'gnus-art)
+(require 'gnus-win)
+
+(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."
+  :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'."
+  :type '(choice symbol string)
+  :group 'picons)
+
+(defcustom gnus-picons-database "/usr/local/faces"
   "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" )
+see http://www.cs.indiana.edu/picons/ftp/index.html"
+  :type 'directory
+  :group 'picons)
 
-(defvar gnus-picons-news-directory "news"
+(defcustom gnus-picons-news-directory "news"
   "Sub-directory of the faces database containing the icons for newsgroups."
-)
+  :type 'string
+  :group 'picons)
 
-(defvar gnus-picons-user-directories '("local" "users" "usenix" "misc/MISC")
+(defcustom gnus-picons-user-directories '("local" "users" "usenix" "misc/MISC")
   "List of directories to search for user faces."
-)
+  :type '(repeat string)
+  :group 'picons)
 
-(defvar gnus-picons-domain-directories '("domains")
+(defcustom gnus-picons-domain-directories '("domains")
   "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."
+  :type 'boolean
+  :group 'picons)
 
-(defvar gnus-picons-x-face-file-name 
+(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.")
+  "The name of the file in which to store the converted X-face header."
+  :type 'string
+  :group 'picons)
 
-(defvar gnus-picons-convert-x-face (format "{ echo '/* Width=48, Height=48 */'; uncompface; } | icontopbm | pbmtoxbm > %s" gnus-picons-x-face-file-name)
+(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
+  :group 'picons)
 
-(defvar gnus-picons-display-as-address t
-  "*If t display textual email addresses along with pictures.")
+(defcustom gnus-picons-display-as-address t
+  "*If t display textual email addresses along with pictures."
+  :type 'boolean
+  :group 'picons)
 
-(defvar gnus-picons-file-suffixes
+(defcustom gnus-picons-file-suffixes
   (when (featurep 'x)
     (let ((types (list "xbm")))
       (when (featurep 'gif)
@@ -121,14 +103,18 @@ 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)
 
-(defvar gnus-picons-display-article-move-p t
+(defcustom gnus-picons-display-article-move-p t
   "*Whether to move point to first empty line when displaying picons.
-This has only an effect if `gnus-picons-display-where' hs value article.")
+This has only an effect if `gnus-picons-display-where' hs value article."
+  :type 'boolean
+  :group 'picons)
 
 (defvar gnus-picons-map (make-sparse-keymap "gnus-picons-keys")
- "keymap to hide/show picon glpyhs")
+ "keymap to hide/show picon glyphs")
 
 (define-key gnus-picons-map [(button2)] 'gnus-picons-toggle-extent)
 
@@ -141,10 +127,9 @@ This has only an effect if `gnus-picons-display-where' hs value article.")
 (defun gnus-picons-remove (plist)
   (let ((listitem (car plist)))
     (while (setq listitem (car plist))
-      (if (annotationp listitem)
-          (delete-annotation listitem))
-      (setq plist (cdr plist))))
-  )
+      (when (annotationp listitem)
+       (delete-annotation listitem))
+      (setq plist (cdr plist)))))
 
 (defun gnus-picons-remove-all ()
   "Removes all picons from the Gnus display(s)."
@@ -155,9 +140,8 @@ This has only an effect if `gnus-picons-display-where' hs value article.")
   (setq gnus-article-annotations nil
         gnus-group-annotations nil
        gnus-x-face-annotations nil)
-  (if (bufferp gnus-picons-buffer)
-      (kill-buffer gnus-picons-buffer))
-  )
+  (when (bufferp gnus-picons-buffer)
+    (kill-buffer gnus-picons-buffer)))
 
 (defun gnus-get-buffer-name (variable)
   "Returns the buffer name associated with the contents of a variable."
@@ -216,14 +200,16 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
   "Display faces for an author and his/her domain in gnus-picons-display-where."
   (interactive)
   ;; let drawing catch up
-  (sit-for 0)
-  (let (from at-idx databases)
-    (when (and (featurep 'xpm) 
+  (when gnus-picons-refresh-before-display
+    (sit-for 0))
+  (let ((first t)
+       from at-idx databases)
+    (when (and (featurep 'xpm)
               (or (not (fboundp 'device-type)) (equal (device-type) 'x))
               (setq from (mail-fetch-field "from"))
-              (setq from (downcase (or (cadr (mail-extract-address-components
-                                              from))
-                                       "")))
+              (setq from (downcase 
+                          (or (cadr (mail-extract-address-components from))
+                              "")))
               (or (setq at-idx (string-match "@" from))
                   (setq at-idx (length from))))
       (save-excursion
@@ -257,7 +243,8 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
                  (nconc (gnus-picons-insert-face-if-exists
                          (car databases)
                          addrs
-                         "unknown" t)
+                         "unknown" (or gnus-picons-display-as-address 
+                                       gnus-article-annotations) t t)
                         gnus-article-annotations))
            (setq databases (cdr databases)))
 
@@ -274,11 +261,11 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
            (setq username (downcase username))
            (while databases
              (setq found
-                   (nconc  (gnus-picons-insert-face-if-exists
-                            (car databases)
-                            addrs
-                            username)
-                           found))
+                   (nconc (gnus-picons-insert-face-if-exists
+                           (car databases) addrs username 
+                           (or gnus-picons-display-as-address 
+                               gnus-article-annotations) nil t)
+                          found))
              (setq databases (cdr databases)))
            ;; add their name if no face exists
            (when (and gnus-picons-display-as-address (not found))
@@ -294,8 +281,9 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
   "Display icons for the group in the gnus-picons-display-where buffer." 
   (interactive)
   ;; let display catch up so far
-  (sit-for 0)
-  (when (and (featurep 'xpm) 
+  (when gnus-picons-refresh-before-display
+    (sit-for 0))
+  (when (and (featurep 'xpm)
             (or (not (fboundp 'device-type)) (equal (device-type) 'x)))
     (save-excursion
       (set-buffer (get-buffer-create
@@ -304,14 +292,15 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
       (goto-char (point-min))
       (if (and (eq gnus-picons-display-where 'article)
               gnus-picons-display-article-move-p)
-         (if (search-forward "\n\n" nil t)
-             (forward-line -1))
+         (when (search-forward "\n\n" nil t)
+           (forward-line -1))
        (unless (eolp)
          (push (make-annotation "\n" (point) 'text)
                gnus-group-annotations)))
       (cond
        ((listp gnus-group-annotations)
-       (mapc #'(lambda (ext) (if (extent-live-p ext) (delete-annotation ext)))
+       (mapc #'(lambda (ext) (when (extent-live-p ext)
+                               (delete-annotation ext)))
              gnus-group-annotations)
        (setq gnus-group-annotations nil))
        ((annotationp gnus-group-annotations)
@@ -322,7 +311,7 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
            (gnus-picons-insert-face-if-exists
             gnus-picons-news-directory
             (message-tokenize-header gnus-newsgroup-name ".")
-            "unknown"))
+            "unknown" nil t))
       (add-hook 'gnus-summary-exit-hook 'gnus-picons-remove-all))))
 
 (defsubst gnus-picons-try-suffixes (file)
@@ -334,10 +323,10 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
     f))
 
 (defun gnus-picons-insert-face-if-exists (database addrs filename &optional
-                                                  nobar-p)
+                                                  nobar-p dots rightp)
   "Inserts a face at point if I can find one"
   ;; '(gnus-picons-insert-face-if-exists
-  ;     "Database" '("edu" "indiana" "cs") "Name")
+  ;;    "Database" '("edu" "indiana" "cs") "Name")
   ;; looks for:
   ;;  1. edu/indiana/cs/Name 
   ;;  2. edu/indiana/Name 
@@ -348,17 +337,17 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
   ;;  1. MISC/Name
   ;; The special treatment of MISC doesn't conform with the conventions for
   ;; picon databases, but otherwise we would always see the MISC/unknown face.
-  (let ((bar (and (not gnus-picons-display-as-address)
-                 (not nobar-p)
-                 (annotations-in-region 
-                  (point) (min (point-max) (1+ (point)))
-                  (current-buffer))))
+  (let ((bar (and (not nobar-p)
+                 (or gnus-picons-display-as-address
+                     (annotations-in-region 
+                      (point) (min (point-max) (1+ (point)))
+                      (current-buffer)))))
        (path (concat (file-name-as-directory gnus-picons-database)
                      database "/"))
-       (domainp (and gnus-picons-display-as-address nobar-p))
+       (domainp (and gnus-picons-display-as-address dots))
        picons found bar-ann cur first)
-    (if (string-match "/MISC" database)
-       (setq addrs '("")))
+    (when (string-match "/MISC" database)
+      (setq addrs '("")))
     (while (and addrs
                (file-accessible-directory-p path))
       (setq cur (pop addrs)
@@ -366,34 +355,45 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
       (if (setq found 
                (gnus-picons-try-suffixes (concat path filename "/face.")))
          (progn 
-           (when bar
-             (setq bar-ann (gnus-picons-try-to-find-face 
-                            (concat gnus-xmas-glyph-directory "bar.xbm")))
-             (when bar-ann
-               (setq picons (nconc picons bar-ann))
-               (setq bar nil)))
-           (setq picons (nconc (when (and domainp first)
-                                 (list (make-annotation "." (point) 'text 
-                                                        nil nil nil t) picons))
+           (setq picons (nconc (when (and domainp first rightp)
+                                 (list (make-annotation
+                                        "." (point) 'text 
+                                        nil nil nil rightp)
+                                       picons))
                                (gnus-picons-try-to-find-face 
-                                found nil (if domainp cur filename))
+                                found nil (if domainp cur filename) rightp)
+                               (when (and domainp first (not rightp))
+                                 (list (make-annotation
+                                        "." (point) 'text 
+                                        nil nil nil rightp)
+                                       picons))
                                picons)))
        (when domainp
          (setq picons 
-               (nconc (list (make-annotation (if first (concat cur ".") cur)
-                                             (point) 'text nil nil nil t)) 
+               (nconc (list (make-annotation 
+                             (if first (concat (if (not rightp) ".") cur 
+                                               (if rightp ".")) cur)
+                                             (point) 'text nil nil nil rightp))
                       picons))))
+      (when (and bar (or domainp found))
+       (setq bar-ann (gnus-picons-try-to-find-face 
+                      (concat gnus-xmas-glyph-directory "bar.xbm") 
+                      nil nil t))
+       (when bar-ann
+         (setq picons (nconc picons bar-ann))
+         (setq bar nil)))
       (setq first t))
     (when (and addrs domainp)
-      (let ((it (mapconcat 'downcase addrs ".")))
+      (let ((it (mapconcat 'downcase (nreverse addrs) ".")))
        (make-annotation 
-        (if first (concat it ".") it) (point) 'text nil nil nil t)))
+        (if first (concat (if (not rightp) ".") it (if rightp ".")) it) 
+        (point) 'text nil nil nil rightp)))
     picons))
 
 (defvar gnus-picons-glyph-alist nil)
       
-(defun gnus-picons-try-to-find-face (path &optional xface-p part)
-  "If PATH exists, display it as a bitmap.  Returns t if succedded."
+(defun gnus-picons-try-to-find-face (path &optional xface-p part rightp)
+  "If PATH exists, display it as a bitmap.  Returns t if succeeded."
   (let ((glyph (and (not xface-p)
                    (cdr (assoc path gnus-picons-glyph-alist)))))
     (when (or glyph (file-exists-p path))
@@ -402,15 +402,15 @@ To use:  (setq gnus-article-x-face-command 'gnus-picons-display-x-face)"
        (unless xface-p
          (push (cons path glyph) gnus-picons-glyph-alist))
        (set-glyph-face glyph 'default))
-      (let ((new (make-annotation glyph (point) 'text nil nil nil t)))
+      (let ((new (make-annotation glyph (point) 'text nil nil nil rightp)))
        (nconc
         (list new)
         (when (and (eq major-mode 'gnus-article-mode)
                    (not gnus-picons-display-as-address)
                    (not part))
-          (list (make-annotation " " (point) 'text nil nil nil t)))
+          (list (make-annotation " " (point) 'text nil nil nil rightp)))
         (when (and part gnus-picons-display-as-address)
-          (let ((txt (make-annotation part (point) 'text nil nil nil t)))
+          (let ((txt (make-annotation part (point) 'text nil nil nil rightp)))
             (hide-annotation txt)
             (set-extent-property txt 'its-partner new)
             (set-extent-property txt 'keymap gnus-picons-map)