*** empty log message ***
[gnus] / lisp / gnus-cite.el
index 01136a1..0982f04 100644 (file)
 (require 'gnus)
 (require 'gnus-msg)
 (require 'gnus-ems)
+(eval-when-compile (require 'cl))
 
 (eval-and-compile
   (autoload 'gnus-article-add-button "gnus-vis"))
 
 ;;; Customization:
 
-(defvar gnus-cited-text-button-line-format "%(%{[...]%}%)"
+(defvar gnus-cited-text-button-line-format "%(%{[...]%}%)\n"
   "Format of cited text buttons.")
 
 (defvar gnus-cited-lines-visible nil
@@ -54,12 +55,12 @@ Set it to nil to parse all articles.")
 (defvar gnus-supercite-regexp 
   (concat "^\\(" gnus-cite-prefix-regexp "\\)? *"
          ">>>>> +\"\\([^\"\n]+\\)\" +==")
-  "Regexp matching normal SuperCite attribution lines.
+  "Regexp matching normal Supercite attribution lines.
 The first grouping must match prefixes added by other packages.")
 
 (defvar gnus-supercite-secondary-regexp "^.*\"\\([^\"\n]+\\)\" +=="
-  "Regexp matching mangled SuperCite attribution lines.
-The first regexp group should match the SuperCite attribution.")
+  "Regexp matching mangled Supercite attribution lines.
+The first regexp group should match the Supercite attribution.")
 
 (defvar gnus-cite-minimum-match-count 2
   "Minimum number of identical prefixes before we believe it's a citation.")
@@ -116,9 +117,7 @@ The text matching the first grouping will be used as a button.")
 
 ;;; Internal Variables:
 
-(defvar gnus-article-length nil)
-;; Length of article last time we parsed it.
-;; BUG! KLUDGE! UGLY! FIX ME!
+(defvar gnus-cite-article nil)
 
 (defvar gnus-cite-prefix-alist nil)
 ;; Alist of citation prefixes.  
@@ -139,7 +138,7 @@ The text matching the first grouping will be used as a button.")
 ;; WROTE: is the attribution line number
 ;; IN: is the line number of the previous line if part of the same attribution,
 ;; PREFIX: Is the citation prefix of the attribution line(s), and
-;; TAG: Is a SuperCite tag, if any.
+;; TAG: Is a Supercite tag, if any.
 
 (defvar gnus-cited-text-button-line-format-alist 
   `((?b beg ?d)
@@ -226,7 +225,7 @@ Lines matching `gnus-cite-attribution-suffix' and perhaps
     (set-buffer gnus-article-buffer)
     (gnus-cite-parse-maybe)
     (let ((alist gnus-cite-prefix-alist)
-         prefix numbers number marks)
+         prefix numbers number marks m)
       ;; Loop through citation prefixes.
       (while alist
        (setq numbers (pop alist)
@@ -263,7 +262,17 @@ Lines matching `gnus-cite-attribution-suffix' and perhaps
          (setq omarks (cdr omarks)))
        (when (car omarks)
          (push (car omarks) marks))
-       (nreverse marks)))))
+       (setq marks (setq m (nreverse marks)))
+       (while (cddr m)
+         (if (and (equal (cdadr m) "")
+                  (equal (cdar m) (cdaddr m))
+                  (goto-char (caadr m))
+                  (forward-line 1)
+                  (= (point) (caaddr m)))
+             (setcdr m (cdddr m))
+           (setq m (cdr m))))
+       marks))))
+           
 
 (defun gnus-article-fill-cited-article (&optional force)
   "Do word wrapping in the current article."
@@ -277,22 +286,22 @@ Lines matching `gnus-cite-attribution-suffix' and perhaps
       (save-restriction
        (while (cdr marks)
          (widen)
-         (narrow-to-region (car (car marks)) (car (cadr marks)))
-         (let ((adaptive-fill-regexp (concat "^" (regexp-quote
-                                                  (cdr (car marks)))
-                                             " *"))
-               (fill-prefix (cdr (car marks))))
+         (narrow-to-region (caar marks) (caadr marks))
+         (let ((adaptive-fill-regexp
+                (concat "^" (regexp-quote (cdar marks)) " *"))
+               (fill-prefix (cdar marks)))
            (fill-region (point-min) (point-max)))
          (set-marker (caar marks) nil)
          (setq marks (cdr marks)))
-       (set-marker (caar marks) nil)))))
+       (when marks
+         (set-marker (caar marks) nil))))))
 
 (defun gnus-article-hide-citation (&optional arg force)
   "Toggle hiding of all cited text except attribution lines.
 See the documentation for `gnus-article-highlight-citation'.
 If given a negative prefix, always show; if given a positive prefix,
 always hide."
-  (interactive (list current-prefix-arg 'force))
+  (interactive (append (gnus-hidden-arg) (list 'force)))
   (setq gnus-cited-text-button-line-format-spec 
        (gnus-parse-format gnus-cited-text-button-line-format 
                           gnus-cited-text-button-line-format-alist t))
@@ -317,29 +326,32 @@ always hide."
          (when marks
            (setq end (caar marks)))
          ;; Skip past lines we want to leave visible.
-         (when (and beg gnus-cited-lines-visible)
+         (when (and beg end gnus-cited-lines-visible)
            (goto-char beg)
            (forward-line gnus-cited-lines-visible)
-           (if (> (point) end)
+           (if (>= (point) end)
                (setq beg nil)
-             (setq beg (point))))
+             (setq beg (point-marker))))
          (when (and beg end)
-           (add-text-properties beg end props)
+           (gnus-add-text-properties beg end props)
            (goto-char beg)
-           (put-text-property beg end 'gnus-type 'cite)
+           (unless (save-excursion (search-backward "\n\n" nil t))
+             (insert "\n"))
            (gnus-article-add-button
             (point)
             (progn (eval gnus-cited-text-button-line-format-spec) (point))
-            `gnus-article-toggle-cited-text (cons beg end))))))))
+            `gnus-article-toggle-cited-text (cons beg end))
+           (set-marker beg (point))))))))
 
 (defun gnus-article-toggle-cited-text (region)
   "Toggle hiding the text in REGION."
-  (funcall
-   (if (text-property-any
-       (car region) (cdr region) 
-       (car gnus-hidden-properties) (cadr gnus-hidden-properties))
-       'remove-text-properties 'add-text-properties)
-   (car region) (cdr region) gnus-hidden-properties))
+  (let (buffer-read-only)
+    (funcall
+     (if (text-property-any
+         (car region) (1- (cdr region))
+         (car gnus-hidden-properties) (cadr gnus-hidden-properties))
+        'remove-text-properties 'gnus-add-text-properties)
+     (car region) (cdr region) gnus-hidden-properties)))
 
 (defun gnus-article-hide-citation-maybe (&optional arg force)
   "Toggle hiding of cited text that has an attribution line.
@@ -350,7 +362,7 @@ percent and at least `gnus-cite-hide-absolute' lines of the body is
 cited text with attributions.  When called interactively, these two
 variables are ignored.
 See also the documentation for `gnus-article-highlight-citation'."
-  (interactive (list current-prefix-arg 'force))
+  (interactive (append (gnus-hidden-arg) (list 'force)))
   (unless (gnus-article-check-hidden-text 'cite arg)
     (save-excursion
       (set-buffer gnus-article-buffer)
@@ -367,7 +379,7 @@ See also the documentation for `gnus-article-highlight-citation'."
        (re-search-backward gnus-signature-separator nil t)
        (setq total (count-lines start (point)))
        (while atts
-         (setq hiden (+ hiden (length (cdr (assoc (cdr (car atts))
+         (setq hiden (+ hiden (length (cdr (assoc (cdar atts)
                                                   gnus-cite-prefix-alist))))
                atts (cdr atts)))
        (if (or force
@@ -376,15 +388,14 @@ See also the documentation for `gnus-article-highlight-citation'."
            (progn
              (setq atts gnus-cite-attribution-alist)
              (while atts
-               (setq total (cdr (assoc (cdr (car atts)) 
-                                       gnus-cite-prefix-alist))
+               (setq total (cdr (assoc (cdar atts) gnus-cite-prefix-alist))
                      atts (cdr atts))
                (while total
                  (setq hiden (car total)
                        total (cdr total))
                  (goto-line hiden)
                  (or (assq hiden gnus-cite-attribution-alist)
-                     (add-text-properties 
+                     (gnus-add-text-properties 
                       (point) (progn (forward-line 1) (point))
                       (nconc (list 'gnus-type 'cite)
                              gnus-hidden-properties)))))))))))
@@ -397,15 +408,14 @@ See also the documentation for `gnus-article-highlight-citation'."
     (let ((article (cdr gnus-article-current)))
       (unless (save-excursion
                (set-buffer gnus-summary-buffer)
-               (gnus-root-id (mail-header-id 
-                              (gnus-summary-article-header article))))
+               (gnus-article-displayed-root-p article))
        (gnus-article-hide-citation)))))
 
 ;;; Internal functions:
 
 (defun gnus-cite-parse-maybe (&optional force)
   ;; Parse if the buffer has changes since last time.
-  (if (eq gnus-article-length (- (point-max) (point-min)))
+  (if (equal gnus-cite-article gnus-article-current)
       ()
     ;;Reset parser information.
     (setq gnus-cite-prefix-alist nil
@@ -417,7 +427,8 @@ See also the documentation for `gnus-article-highlight-citation'."
             gnus-cite-parse-max-size
             (> (buffer-size) gnus-cite-parse-max-size))
        ()
-      (setq gnus-article-length (- (point-max) (point-min)))
+      (setq gnus-cite-article (cons (car gnus-article-current)
+                                   (cdr gnus-article-current)))
       (gnus-cite-parse))))
 
 (defun gnus-cite-parse ()
@@ -441,7 +452,7 @@ See also the documentation for `gnus-article-highlight-citation'."
            end (progn (beginning-of-line 2) (point))
            start end)
       (goto-char begin)
-      ;; Ignore standard SuperCite attribution prefix.
+      ;; Ignore standard Supercite attribution prefix.
       (if (looking-at gnus-supercite-regexp)
          (if (match-end 1)
              (setq end (1+ (match-end 1)))
@@ -453,7 +464,7 @@ See also the documentation for `gnus-article-highlight-citation'."
        ;; Each prefix.
        (setq end (match-end 0)
              prefix (buffer-substring begin end))
-       (set-text-properties 0 (length prefix) nil prefix)
+       (gnus-set-text-properties 0 (length prefix) nil prefix)
        (setq entry (assoc prefix alist))
        (if entry 
            (setcdr entry (cons line (cdr entry)))
@@ -589,7 +600,7 @@ See also the documentation for `gnus-article-highlight-citation'."
   ;; 
   ;; WROTE is the attribution line number.
   ;; PREFIX is the attribution line prefix.
-  ;; TAG is the SuperCite tag on the attribution line.
+  ;; TAG is the Supercite tag on the attribution line.
   (let ((atts gnus-cite-loose-attribution-alist)
        (case-fold-search t)
        att wrote in prefix tag regexp limit smallest best size)
@@ -662,18 +673,19 @@ See also the documentation for `gnus-article-highlight-citation'."
 
 (defun gnus-cite-add-face (number prefix face)
   ;; At line NUMBER, ignore PREFIX and add FACE to the rest of the line.
-  (if face
-      (let ((inhibit-point-motion-hooks t)
-           from to)
-       (goto-line number)
+  (when face
+    (let ((inhibit-point-motion-hooks t)
+         from to)
+      (goto-line number)
+      (unless (eobp) ;; Sometimes things become confused.
        (forward-char (length prefix))
        (skip-chars-forward " \t")
        (setq from (point))
        (end-of-line 1)
        (skip-chars-backward " \t")
        (setq to (point))
-       (if (< from to)
-           (gnus-overlay-put (gnus-make-overlay from to) 'face face)))))
+       (when (< from to)
+         (gnus-overlay-put (gnus-make-overlay from to) 'face face))))))
 
 (defun gnus-cite-toggle (prefix)
   (save-excursion
@@ -691,7 +703,7 @@ See also the documentation for `gnus-article-highlight-citation'."
                                       gnus-hidden-properties))
              ((assq number gnus-cite-attribution-alist))
              (t
-              (add-text-properties 
+              (gnus-add-text-properties 
                (point) (progn (forward-line 1) (point))
                 (nconc (list 'gnus-type 'cite)
                        gnus-hidden-properties))))))))