Synch with Oort Gnus.
[elisp/gnus.git-] / lisp / message.el
index f7b7f43..4e67115 100644 (file)
@@ -668,7 +668,12 @@ The function `message-supersede' runs this hook."
 
 ;;;###autoload
 (defcustom message-citation-line-function 'message-insert-citation-line
-  "*Function called to insert the \"Whomever writes:\" line."
+  "*Function called to insert the \"Whomever writes:\" line.
+
+Note that Gnus provides a feature where the reader can click on
+`writes:' to hide the cited text.  If you change this line too much,
+people who read your message will have to change their Gnus
+configuration.  See the variable `gnus-cite-attribution-suffix'."
   :type 'function
   :group 'message-insertion)
 
@@ -949,10 +954,6 @@ candidates:
     table)
   "Syntax table used while in Message mode.")
 
-(defvar message-mode-abbrev-table text-mode-abbrev-table
-  "Abbrev table used in Message mode buffers.
-Defaults to `text-mode-abbrev-table'.")
-
 (defface message-header-to-face
   '((((class color)
       (background dark))
@@ -1073,151 +1074,7 @@ Defaults to `text-mode-abbrev-table'.")
   "Face used for displaying MML."
   :group 'message-faces)
 
-(defvar message-font-lock-fence-open-regexp "[+|]"
-  "*Regexp that matches fence open string.")
-
-(defvar message-font-lock-fence-close-regexp "|"
-  "*Regexp that matches fence close string.")
-
-(defvar message-font-lock-fence-open-position nil
-  "*Cons of SYMBOL of a function or a variable and a number of OFFSET that
-indicate the fence open position.  If it is non-nil,
-`message-font-lock-fence-open-regexp' is not used for searching for the
-fence open position.  If SYMBOL is a function, it is called with one argument
-last cursor position and should return the fence open position as a number
-or a marker.  If SYMBOL is a variable symbol, the value is examined with
-`symbol-value'.  OFFSET is added to the position to compensate the value.
-For example, the following combinations of variable symbol and offset value
-can be used:
-
-Egg v3: '(egg:*region-start* . -1)
-Canna:  '(canna:*region-start* . 0)
-")
-
-(defvar message-font-lock-fence-close-position nil
-  "*Cons of SYMBOL of a function or a variable and a number of OFFSET that
-indicate the fence close position.  If it is non-nil,
-`message-font-lock-fence-close-regexp' is not used for searching for the
-fence close position.  If SYMBOL is a function, it is called with one argument
-last cursor position and should return the fence close position as a number
-or a marker.  If SYMBOL is a variable symbol, the value is examined with
-`symbol-value'.  OFFSET is added to the position to compensate the value.
-For example, the following combinations of variable symbol and offset value
-can be used:
-
-Egg v3: '(egg:*region-end* . 0)
-Canna:  '(canna:*region-end* . 0)
-")
-
-(defvar message-font-lock-cited-text-regexp
-  "^[\t ]*\\([^\000- :>|}\177]*\\)[:>|}].*"
-  "*Regexp that matches cited text.  It should have a grouping for the
-citation prefix which is ended at the beginning of citation mark string.")
-
-(defvar message-font-lock-citation-name-max-column 10
-  "*Maximun number of column for citation name for fontifying.")
-
-(defvar message-font-lock-last-position nil
-  "Internal buffer local variable to save the last cursor position
-before fontifying.")
-
-(eval-after-load "font-lock"
-  '(defadvice font-lock-after-change-function
-     (before message-font-lock-save-last-position activate)
-     "Save last cursor position before fontifying."
-     (if (eq 'message-mode major-mode)
-        (setq message-font-lock-last-position (point)))))
-
-(defun message-font-lock-cited-text-matcher (limit)
-  "Search for a cited text containing `message-font-lock-cited-text-regexp'
-forward.  Argument LIMIT bounds the search.  If a cited text is found, it
-returns t and sets match data 1 and 2, otherwise it returns nil.  Normally,
-match data 2 has zero length, but if the FENCE (for input method) is detected
-in matched text, result is divided into match data 1 and 2 across the FENCE.
-See also the documentations for the following variables:
- `message-font-lock-fence-open-regexp'
- `message-font-lock-fence-close-regexp'
- `message-font-lock-fence-open-position'
- `message-font-lock-fence-close-position'
-"
-  (prog1
-      (when (re-search-forward message-font-lock-cited-text-regexp limit t)
-       (let* ((start0 (match-beginning 0))
-              (end0 (match-end 0))
-              (cite-mark (match-end 1))
-              (should-fontify
-               (progn
-                 (goto-char cite-mark)
-                 (<= (current-column)
-                     message-font-lock-citation-name-max-column)))
-              end1 start2)
-         (and
-          should-fontify
-          message-font-lock-last-position
-          (>= message-font-lock-last-position start0)
-          (<= message-font-lock-last-position end0)
-          (cond
-           (message-font-lock-fence-open-position
-            (let* ((symbol (car message-font-lock-fence-open-position))
-                   (open
-                    (cond ((functionp symbol)
-                           (funcall symbol message-font-lock-last-position))
-                          ((and (symbolp symbol)
-                                (boundp symbol))
-                           (symbol-value symbol)))))
-              (when (markerp open)
-                (setq open (marker-position open)))
-              (and (numberp open)
-                   (setq open
-                         (+ open
-                            (cdr message-font-lock-fence-open-position)))
-                   (>= message-font-lock-last-position open)
-                   (goto-char open)
-                   (or (not message-font-lock-fence-open-regexp)
-                       (looking-at message-font-lock-fence-open-regexp))
-                   (setq end1 open))))
-           (message-font-lock-fence-open-regexp
-            (goto-char message-font-lock-last-position)
-            (when (re-search-backward
-                   message-font-lock-fence-open-regexp start0 t)
-              (setq end1 (match-beginning 0)))))
-          (setq should-fontify
-                (and message-font-lock-fence-open-position
-                     (not (eq cite-mark end1))))
-          (cond
-           (message-font-lock-fence-close-position
-            (let* ((symbol (car message-font-lock-fence-close-position))
-                   (close
-                    (cond ((functionp symbol)
-                           (funcall symbol message-font-lock-last-position))
-                          ((and (symbolp symbol)
-                                (boundp symbol))
-                           (symbol-value symbol)))))
-              (when (markerp close)
-                (setq close (marker-position close)))
-              (and (numberp close)
-                   (setq close
-                         (+ close
-                            (cdr message-font-lock-fence-close-position)))
-                   (<= message-font-lock-last-position close)
-                   (setq start2 close))))
-           (message-font-lock-fence-close-regexp
-            (goto-char message-font-lock-last-position)
-            (when (looking-at message-font-lock-fence-close-regexp)
-              (setq start2 (match-end 0)))))
-          (setq should-fontify
-                (and (not (and (not message-font-lock-fence-open-position)
-                               (eq cite-mark end1)))
-                     (not (eq cite-mark start2)))))
-         (goto-char end0)
-         (when should-fontify
-           (if start2
-               (store-match-data (list start0 end0 start0 end1 start2 end0))
-             (store-match-data (list start0 end0 start0 end0 end0 end0)))
-           t)))
-    (setq message-font-lock-last-position nil)))
-
-(defvar message-font-lock-keywords-1
+(defvar message-font-lock-keywords
   (let ((content "[ \t]*\\(.+\\(\n[ \t].*\\)*\\)\n?"))
     `((,(concat "^\\([Tt]o:\\)" content)
        (1 'message-header-name-face)
@@ -1244,29 +1101,16 @@ See also the documentations for the following variables:
                 (not (equal mail-header-separator "")))
            `((,(concat "^\\(" (regexp-quote mail-header-separator) "\\)$")
               1 'message-separator-face))
-         nil))))
-
-(defvar message-font-lock-keywords-2
-  (append message-font-lock-keywords-1
-         `((message-font-lock-cited-text-matcher
-            (1 'message-cited-text-face)
-            (2 'message-cited-text-face))
-           (,(concat "^\\(" message-cite-prefix-regexp "\\).*")
-            (0 'message-cited-text-face))
-           ("<#/?\\(multipart\\|part\\|external\\|mml\\).*>"
-            (0 'message-mml-face)))))
-
-(defvar message-font-lock-keywords message-font-lock-keywords-2
+         nil)
+      (,(concat "^\\(" message-cite-prefix-regexp "\\).*")
+       (0 'message-cited-text-face))
+      (,mime-edit-tag-regexp
+       (0 'message-mml-face))))
   "Additional expressions to highlight in Message mode.")
 
 ;; XEmacs does it like this.  For Emacs, we have to set the
 ;; `font-lock-defaults' buffer-local variable.
-(put 'message-mode 'font-lock-defaults
-     '((message-font-lock-keywords
-       message-font-lock-keywords-1
-       message-font-lock-keywords-2)
-       nil nil nil nil
-       (font-lock-mark-block-function . mark-paragraph)))
+(put 'message-mode 'font-lock-defaults '(message-font-lock-keywords t))
 
 (defvar message-face-alist
   '((bold . bold-region)
@@ -1935,7 +1779,7 @@ Point is left at the beginning of the narrowed-to region."
   (defvar facemenu-remove-face-function))
 
 ;;;###autoload
-(defun message-mode ()
+(define-derived-mode message-mode text-mode "Message"
   "Major mode for editing mail and news to be sent.
 Like Text Mode but with these additional commands:\\<message-mode-map>
 C-c C-s  `message-send' (send the message)  C-c C-c  `message-send-and-exit'
@@ -1948,6 +1792,7 @@ C-c C-f  move to a header field (and create it if there isn't):
         C-c C-f C-k  move to Keywords  C-c C-f C-d  move to Distribution
         C-c C-f C-m  move to Mail-Followup-To
         C-c C-f C-f  move to Followup-To
+        C-c C-f c    move to Mail-Copies-To
 C-c C-t  `message-insert-to' (add a To header to a news followup)
 C-c C-n  `message-insert-newsgroups' (add a Newsgroup header to a news reply)
 C-c C-b  `message-goto-body' (move to beginning of message text).
@@ -1960,33 +1805,22 @@ C-c C-v  `message-delete-not-region' (remove the text outside the region).
 C-c C-z  `message-kill-to-signature' (kill the text up to the signature).
 C-c C-r  `message-caesar-buffer-body' (rot13 the message body).
 M-RET    `message-newline-and-reformat' (break the line and reformat)."
-  (interactive)
-  (kill-all-local-variables)
   (set (make-local-variable 'message-reply-buffer) nil)
   (make-local-variable 'message-send-actions)
   (make-local-variable 'message-exit-actions)
   (make-local-variable 'message-kill-actions)
   (make-local-variable 'message-postpone-actions)
   (make-local-variable 'message-draft-article)
-  (make-local-hook 'kill-buffer-hook)
-  (set-syntax-table message-mode-syntax-table)
-  (use-local-map message-mode-map)
-  (setq local-abbrev-table message-mode-abbrev-table)
-  (setq major-mode 'message-mode)
-  (setq mode-name "Message")
   (setq buffer-offer-save t)
-  (make-local-variable 'facemenu-add-face-function)
-  (make-local-variable 'facemenu-remove-face-function)
-  (setq facemenu-add-face-function
-       (lambda (face end)
-         (let ((face-fun (cdr (assq face message-face-alist))))
-           (if face-fun
-               (funcall face-fun (point) end)
-             (error "Face %s not configured for %s mode" face mode-name)))
-         "")
-       facemenu-remove-face-function t)
-  (make-local-variable 'message-reply-headers)
-  (setq message-reply-headers nil)
+  (set (make-local-variable 'facemenu-add-face-function)
+       (lambda (face end)
+        (let ((face-fun (cdr (assq face message-face-alist))))
+          (if face-fun
+              (funcall face-fun (point) end)
+            (error "Face %s not configured for %s mode" face mode-name)))
+        ""))
+  (set (make-local-variable 'facemenu-remove-face-function) t)
+  (set (make-local-variable 'message-reply-headers) nil)
   (make-local-variable 'message-user-agent)
   (make-local-variable 'message-post-method)
   (set (make-local-variable 'message-sent-message-via) nil)
@@ -2000,14 +1834,9 @@ M-RET    `message-newline-and-reformat' (break the line and reformat)."
   (if (featurep 'xemacs)
       (message-setup-toolbar)
     (set (make-local-variable 'font-lock-defaults)
-        '((message-font-lock-keywords
-           message-font-lock-keywords-1
-           message-font-lock-keywords-2)
-          nil nil nil nil
-          (font-lock-mark-block-function . mark-paragraph)))
+        '(message-font-lock-keywords t))
     (if (boundp 'tool-bar-map)
        (set (make-local-variable 'tool-bar-map) (message-tool-bar-map))))
-  (set (make-local-variable 'message-font-lock-last-position) nil)
   (easy-menu-add message-mode-menu message-mode-map)
   (easy-menu-add message-mode-field-menu message-mode-map)
   ;; Allow mail alias things.
@@ -2016,9 +1845,7 @@ M-RET    `message-newline-and-reformat' (break the line and reformat)."
        (mail-abbrevs-setup)
       (mail-aliases-setup)))
   (message-set-auto-save-file-name)
-  (make-local-variable 'indent-tabs-mode) ;Turn off tabs for indentation.
-  (setq indent-tabs-mode nil)
-  (run-hooks 'text-mode-hook 'message-mode-hook))
+  (set (make-local-variable 'indent-tabs-mode) nil)) ;No tabs for indentation.
 
 (defun message-setup-fill-variables ()
   "Setup message fill variables."
@@ -2623,10 +2450,11 @@ be added to \"References\" field.
        (if (and message-suspend-font-lock-when-citing
                 (boundp 'font-lock-mode)
                 (symbol-value 'font-lock-mode))
-           (progn
-             (sit-for 0)
-             (font-lock-mode 0)
-             (funcall message-cite-function)
+           (unwind-protect
+               (progn
+                 (sit-for 0)
+                 (font-lock-mode 0)
+                 (funcall message-cite-function))
              (font-lock-mode 1))
          (funcall message-cite-function)))
       (message-exchange-point-and-mark)
@@ -2934,7 +2762,7 @@ It should typically alter the sending method in some way or other."
                                (format
                                 "Already sent message via %s; resend? "
                                 (car elem)))
-                            (error "Denied posting -- multiple copies.")))
+                            (error "Denied posting -- multiple copies")))
                       (setq success (funcall (caddr elem) arg)))
              (setq sent t)))))
       (unless (or sent (not success))
@@ -3620,17 +3448,23 @@ This sub function is for exclusive use of `message-send-news'."
                     (if followup-to
                         (concat newsgroups "," followup-to)
                       newsgroups)))
-           (hashtb (and (boundp 'gnus-active-hashtb)
-                        gnus-active-hashtb))
+            (known-groups
+             (mapcar '(lambda (n) (gnus-group-real-name n))
+                     (gnus-groups-from-server
+                      (cond ((equal gnus-post-method 'current)
+                             gnus-current-select-method)
+                            (gnus-post-method gnus-post-method)
+                            (t gnus-select-method)))))
            errors)
        (while groups
-        (when (and (not (boundp (intern (car groups) hashtb)))
-                   (not (equal (car groups) "poster")))
-          (push (car groups) errors))
-        (pop groups))
+         (unless (or (equal (car groups) "poster")
+                     (member (car groups) known-groups))
+           (push (car groups) errors))
+         (pop groups))
        (cond
        ;; Gnus is not running.
-       ((or (not hashtb)
+       ((or (not (and (boundp 'gnus-active-hashtb)
+                       gnus-active-hashtb))
             (not (boundp 'gnus-read-active-file)))
         t)
        ;; We don't have all the group names.
@@ -4496,12 +4330,12 @@ Headers already prepared in the buffer are not modified."
          (nthcdr (+ (- cut 2) surplus 1) list)))
 
 (defun message-shorten-references (header references)
-  "Trim REFERENCES to be less than 31 Message-ID long, and fold them.
+  "Trim REFERENCES to be 21 Message-ID long or less, and fold them.
 If folding is disallowed, also check that the REFERENCES are less
 than 988 characters long, and if they are not, trim them until they are."
-  (let ((maxcount 31)
+  (let ((maxcount 21)
        (count 0)
-       (cut 6)
+       (cut 2)
        refs)
     (with-temp-buffer
       (insert references)
@@ -5704,14 +5538,21 @@ which specify the range to operate on."
   "^\\(Newsgroups\\|Followup-To\\|Posted-To\\|Gcc\\):"
   "Regexp that match headers that lists groups.")
 
+(defvar message-completion-alist
+  (list (cons message-newgroups-header-regexp 'message-expand-group)
+       '("^\\(Resent-\\)?\\(To\\|B?Cc\\):" . message-expand-name))
+  "Alist of (RE . FUN).  Use FUN for completion on header lines matching RE.")
+
 (defun message-tab ()
   "Expand group names in Newsgroups and Followup-To headers.
 Do a `tab-to-tab-stop' if not in those headers."
   (interactive)
-  (if (let ((mail-abbrev-mode-regexp message-newgroups-header-regexp))
-       (mail-abbrev-in-expansion-header-p))
-      (message-expand-group)
-    (tab-to-tab-stop)))
+  (let ((alist message-completion-alist))
+    (while (and alist
+               (let ((mail-abbrev-mode-regexp (caar alist)))
+                 (not (mail-abbrev-in-expansion-header-p))))
+      (setq alist (cdr alist)))
+    (funcall (or (cdar alist) (default-value 'indent-line-function)))))
 
 (defun message-expand-group ()
   "Expand the group name under point."
@@ -5755,6 +5596,11 @@ Do a `tab-to-tab-stop' if not in those headers."
            (goto-char (point-min))
            (delete-region (point) (progn (forward-line 3) (point))))))))))
 
+(defun message-expand-name ()
+  (if (fboundp 'bbdb-complete-name)
+      (bbdb-complete-name)
+    (expand-abbrev)))
+
 ;;; Help stuff.
 
 (defun message-talkative-question (ask question show &rest text)