*** empty log message ***
[gnus] / lisp / gnus-uu.el
index 8104d41..f93a869 100644 (file)
@@ -265,7 +265,6 @@ The headers will be included in the sequence they are matched.")
 (defconst gnus-uu-uudecode-process nil)
 (defvar gnus-uu-binhex-article-name nil)
 
-(defvar gnus-uu-generated-file-list nil)
 (defvar gnus-uu-work-dir nil)
 
 (defconst gnus-uu-output-buffer-name " *Gnus UU Output*")
@@ -321,7 +320,7 @@ The headers will be included in the sequence they are matched.")
 
 ;; Commands.
 
-(defun gnus-uu-decode-uu (n)
+(defun gnus-uu-decode-uu (&optional n)
   "Uudecodes the current article."
   (interactive "P") 
   (gnus-uu-decode-with-method 'gnus-uu-uustrip-article n))
@@ -336,7 +335,7 @@ The headers will be included in the sequence they are matched.")
                          gnus-uu-default-dir t))))
   (gnus-uu-decode-with-method 'gnus-uu-uustrip-article n dir nil nil t))
 
-(defun gnus-uu-decode-unshar (n)
+(defun gnus-uu-decode-unshar (&optional n)
   "Unshars the current article."
   (interactive "P")
   (gnus-uu-decode-with-method 'gnus-uu-unshar-article n nil nil 'scan t))
@@ -362,9 +361,7 @@ The headers will be included in the sequence they are matched.")
          gnus-uu-default-dir
          gnus-uu-default-dir)))
   (setq gnus-uu-saved-article-name file)
-  (gnus-uu-decode-with-method 'gnus-uu-save-article n nil t)
-  (setq gnus-uu-generated-file-list 
-       (delete file gnus-uu-generated-file-list)))
+  (gnus-uu-decode-with-method 'gnus-uu-save-article n nil t))
 
 (defun gnus-uu-decode-binhex (n dir)
   "Unbinhexes the current article."
@@ -378,7 +375,7 @@ The headers will be included in the sequence they are matched.")
        (make-temp-name (concat gnus-uu-work-dir "binhex")))
   (gnus-uu-decode-with-method 'gnus-uu-binhex-article n dir))
 
-(defun gnus-uu-decode-uu-view (n)
+(defun gnus-uu-decode-uu-view (&optional n)
   "Uudecodes and views the current article."    
   (interactive "P")
   (let ((gnus-view-pseudos (or gnus-view-pseudos 'automatic)))
@@ -394,7 +391,7 @@ The headers will be included in the sequence they are matched.")
   (let ((gnus-view-pseudos (or gnus-view-pseudos 'automatic)))
     (gnus-uu-decode-uu-and-save n dir)))
 
-(defun gnus-uu-decode-unshar-view (n)
+(defun gnus-uu-decode-unshar-view (&optional n)
   "Unshars and views the current article."
   (interactive "P")
   (let ((gnus-view-pseudos (or gnus-view-pseudos 'automatic)))
@@ -435,7 +432,7 @@ The headers will be included in the sequence they are matched.")
 
 ;; Digest and forward articles
 
-(defun gnus-uu-digest-mail-forward (n &optional post)
+(defun gnus-uu-digest-mail-forward (&optional n post)
   "Digests and forwards all articles in this series."
   (interactive "P")
   (let ((gnus-uu-save-in-digest t)
@@ -443,7 +440,6 @@ The headers will be included in the sequence they are matched.")
        buf subject from)
     (setq gnus-uu-digest-from-subject nil)
     (gnus-uu-decode-save n file)
-    (gnus-uu-add-file file)
     (setq buf (switch-to-buffer (get-buffer-create " *gnus-uu-forward*")))
     (gnus-add-current-to-buffer-list)
     (erase-buffer)
@@ -481,7 +477,7 @@ The headers will be included in the sequence they are matched.")
     (kill-buffer buf)
     (setq gnus-uu-digest-from-subject nil)))
 
-(defun gnus-uu-digest-post-forward (n)
+(defun gnus-uu-digest-post-forward (&optional n)
   "Digest and forward to a newsgroup."
   (interactive "P")
   (gnus-uu-digest-mail-forward n t))
@@ -565,7 +561,7 @@ The headers will be included in the sequence they are matched.")
                (> (gnus-summary-thread-level) level))))
   (gnus-summary-position-point))
 
-(defun gnus-uu-mark-over (score)
+(defun gnus-uu-mark-over (&optional score)
   "Mark all articles with a score over SCORE (the prefix.)"
   (interactive "P")
   (let ((score (gnus-score-default score))
@@ -590,7 +586,8 @@ The headers will be included in the sequence they are matched.")
     (setq gnus-newsgroup-processable nil)
     (save-excursion
       (while marked
-       (and (setq headers (gnus-summary-article-header (car marked)))
+       (and (vectorp (setq headers 
+                           (gnus-summary-article-header (car marked))))
             (setq subject (mail-header-subject headers)
                   articles (gnus-uu-find-articles-matching 
                             (gnus-uu-reginize-string subject))
@@ -612,8 +609,9 @@ The headers will be included in the sequence they are matched.")
     (let ((data gnus-newsgroup-data)
          number)
       (while data
-       (unless (memq (setq number (gnus-data-number (car data)))
-                     gnus-newsgroup-processable)
+       (when (and (not (memq (setq number (gnus-data-number (car data)))
+                             gnus-newsgroup-processable))
+                  (vectorp (gnus-data-header (car data))))
          (gnus-summary-goto-subject number)
          (gnus-uu-mark-series))
        (setq data (cdr data)))))
@@ -621,12 +619,12 @@ The headers will be included in the sequence they are matched.")
 
 ;; All PostScript functions written by Erik Selberg <speed@cs.washington.edu>. 
 
-(defun gnus-uu-decode-postscript (n)
+(defun gnus-uu-decode-postscript (&optional n)
   "Gets postscript of the current article."
   (interactive "P")
   (gnus-uu-decode-with-method 'gnus-uu-decode-postscript-article n))
 
-(defun gnus-uu-decode-postscript-view (n)
+(defun gnus-uu-decode-postscript-view (&optional n)
   "Gets and views the current article."
   (interactive "P")
   (let ((gnus-view-pseudos (or gnus-view-pseudos 'automatic)))
@@ -672,7 +670,6 @@ The headers will be included in the sequence they are matched.")
     (and save (gnus-uu-save-files files save))
     (if (eq gnus-uu-do-not-unpack-archives nil)
       (setq files (gnus-uu-unpack-files files)))
-    (gnus-uu-add-file (mapcar (lambda (file) (cdr (assq 'name file))) files))
     (setq files (nreverse (gnus-uu-get-actions files)))
     (or not-insert (not gnus-insert-pseudo-articles)
        (gnus-summary-insert-pseudos files save))))
@@ -708,7 +705,7 @@ The headers will be included in the sequence they are matched.")
          (when (or (not (file-exists-p to-file))
                    (gnus-y-or-n-p (format "%s exists; overwrite? " to-file)))
            (copy-file file to-file t t)))))
-    (message "Saved %d file%s" len (if (= len 1) "" "s"))))
+    (gnus-message 5 "Saved %d file%s" len (if (= len 1) "" "s"))))
 
 ;; Functions for saving and possibly digesting articles without
 ;; any decoding.
@@ -1127,10 +1124,12 @@ The headers will be included in the sequence they are matched.")
 ;; 
 ;; This function returns a list of files decoded if the grabbing and
 ;; the process-function has been successful and nil otherwise.
-(defun gnus-uu-grab-articles 
-  (articles process-function &optional sloppy limit no-errors)
+(defun gnus-uu-grab-articles (articles process-function 
+                                      &optional sloppy limit no-errors)
   (let ((state 'first) 
        has-been-begin article result-file result-files process-state
+       gnus-summary-display-article-function
+       gnus-article-display-hook gnus-article-prepare-hook
        article-series files)
  
     (while (and articles 
@@ -1147,8 +1146,8 @@ The headers will be included in the sequence they are matched.")
          (setq state 'last)))
 
       (let ((part (gnus-uu-part-number article)))
-       (message "Getting article %d%s..." 
-                article (if (string= part "") "" (concat ", " part))))
+       (gnus-message 6 "Getting article %d%s..." 
+                     article (if (string= part "") "" (concat ", " part))))
       (gnus-summary-display-article article)
       
       ;; Push the article to the processing function.
@@ -1224,7 +1223,7 @@ The headers will be included in the sequence they are matched.")
                   (memq 'middle process-state)))
          (progn
            (setq process-state (list 'error))
-           (message "No begin part at the beginning")
+           (gnus-message 2 "No begin part at the beginning")
            (sleep-for 2))
        (setq state 'middle)))
 
@@ -1233,12 +1232,12 @@ The headers will be included in the sequence they are matched.")
        (message "")
       (cond
        ((not has-been-begin)
-       (message "Wrong type file"))
+       (gnus-message 2 "Wrong type file"))
        ((memq 'error process-state)
-       (message "An error occurred during decoding"))
+       (gnus-message 2 "An error occurred during decoding"))
        ((not (or (memq 'ok process-state) 
                 (memq 'end process-state)))
-       (message "End of articles reached before end of file")))
+       (gnus-message 2 "End of articles reached before end of file")))
       ;; Make unsuccessfully decoded articles unread.
       (when gnus-uu-unmark-articles-not-decoded
        (while article-series
@@ -1265,10 +1264,11 @@ The headers will be included in the sequence they are matched.")
        (make-symbolic-link to-file file)))))
 
 (defun gnus-uu-part-number (article)
-  (let ((subject (mail-header-subject (gnus-summary-article-header article))))
-    (if (string-match "[0-9]+ */[0-9]+\\|[0-9]+ * of *[0-9]+"
-                     subject)
-       (substring subject (match-beginning 0) (match-end 0))
+  (let* ((header (gnus-summary-article-header article))
+        (subject (and header (mail-header-subject header))))
+    (if (and subject 
+            (string-match "[0-9]+ */[0-9]+\\|[0-9]+ * of *[0-9]+" subject))
+       (match-string 0 subject)
       "")))
 
 (defun gnus-uu-uudecode-sentinel (process event)
@@ -1328,8 +1328,7 @@ The headers will be included in the sequence they are matched.")
          (set-process-sentinel 
           gnus-uu-uudecode-process 'gnus-uu-uudecode-sentinel)
          (setq state (list 'begin))
-         (push (concat gnus-uu-work-dir gnus-uu-file-name) files)
-         (gnus-uu-add-file (car files)))
+         (push (concat gnus-uu-work-dir gnus-uu-file-name) files))
        
        ;; We look for the end of the thing to be decoded.
        (if (re-search-forward gnus-uu-end-string nil t)
@@ -1352,7 +1351,7 @@ The headers will be included in the sequence they are matched.")
              (error 
               (progn 
                 (delete-process gnus-uu-uudecode-process)
-                (message "gnus-uu: Couldn't uudecode")
+                (gnus-message 2 "gnus-uu: Couldn't uudecode")
                 (setq state (list 'wrong-type)))))
 
            (if (memq 'end state)
@@ -1450,13 +1449,13 @@ The headers will be included in the sequence they are matched.")
       (set-buffer (get-buffer-create gnus-uu-output-buffer-name))
       (erase-buffer))
 
-    (message "Unpacking: %s..." (gnus-uu-command action file-path))
+    (gnus-message 5 "Unpacking: %s..." (gnus-uu-command action file-path))
 
     (if (= 0 (call-process "sh" nil 
                           (get-buffer-create gnus-uu-output-buffer-name)
                           nil "-c" command))
        (message "")
-      (message "Error during unpacking of archive")
+      (gnus-message 2 "Error during unpacking of archive")
       (setq did-unpack nil))
 
     (if (member action gnus-uu-destructive-archivers)
@@ -1479,7 +1478,6 @@ The headers will be included in the sequence they are matched.")
   (let* ((totfiles (gnus-uu-ls-r gnus-uu-work-dir))
         (ofiles files)
         file did-unpack)
-    (gnus-uu-add-file totfiles) 
     (while files
       (setq file (cdr (assq 'name (car files))))
       (if (and (not (member file ignore))
@@ -1488,10 +1486,9 @@ The headers will be included in the sequence they are matched.")
          (progn
            (setq did-unpack (cons file did-unpack))
            (or (gnus-uu-treat-archive file)
-               (message "Error during unpacking of %s" file))
+               (gnus-message 2 "Error during unpacking of %s" file))
            (let* ((newfiles (gnus-uu-ls-r gnus-uu-work-dir))
                   (nfiles newfiles))
-             (gnus-uu-add-file newfiles)
              (while nfiles
                (or (member (car nfiles) totfiles)
                    (setq ofiles (cons (list (cons 'name (car nfiles))
@@ -1582,7 +1579,6 @@ The headers will be included in the sequence they are matched.")
 
       (setq gnus-uu-work-dir 
            (make-temp-name (concat gnus-uu-tmp-dir "gnus")))
-      (gnus-uu-add-file gnus-uu-work-dir)
       (if (not (file-directory-p gnus-uu-work-dir)) 
          (gnus-make-directory gnus-uu-work-dir))
       (set-file-modes gnus-uu-work-dir 448)
@@ -1595,38 +1591,12 @@ The headers will be included in the sequence they are matched.")
 (defun gnus-uu-clean-up ()
   (let (buf pst)
     (and gnus-uu-uudecode-process
-        (setq pst (process-status (or gnus-uu-uudecode-process "nevair")))
-        (if (or (eq pst 'stop) (eq pst 'run))
-            (delete-process gnus-uu-uudecode-process)))
+        (memq (process-status (or gnus-uu-uudecode-process "nevair"))
+              '(stop run))
+        (delete-process gnus-uu-uudecode-process))
     (and (setq buf (get-buffer gnus-uu-output-buffer-name))
         (kill-buffer buf))))
 
-;; `gnus-uu-check-for-generated-files' deletes any generated files that
-;; hasn't been deleted, if, for instance, the user terminated decoding
-;; with `C-g'.
-(defun gnus-uu-check-for-generated-files ()
-  (let (file dirs)
-    ;; First delete the generated files.
-    (while (setq file (pop gnus-uu-generated-file-list))
-      (unless (string-match "/\\.[\\.]?$" file)
-       (if (file-directory-p file)
-           (push file dirs)
-         (when (file-exists-p file)
-           (delete-file file)))))
-    ;; Then delete the directories.
-    (setq dirs (nreverse dirs))
-    (while (setq file (pop dirs))
-      (delete-directory (directory-file-name file)))))
-
-;; Add a file (or a list of files) to be checked (and deleted if it/they
-;; still exists upon exiting the newsgroup).
-(defun gnus-uu-add-file (file)
-  (if (stringp file)
-      (setq gnus-uu-generated-file-list 
-           (cons file gnus-uu-generated-file-list))
-    (setq gnus-uu-generated-file-list 
-         (append file gnus-uu-generated-file-list))))
-
 ;; Inputs an action and a file and returns a full command, putting
 ;; quotes round the file name and escaping any quotes in the file name.
 (defun gnus-uu-command (action file)
@@ -1642,11 +1612,28 @@ The headers will be included in the sequence they are matched.")
        (format action ofile)
       (concat action " " ofile))))
 
+(defun gnus-uu-delete-work-dir (&optional dir)
+  "Delete recursively all files and directories under `gnus-uu-work-dir'."
+  (if dir
+      (gnus-message 7 "Deleting directory %s..." dir)
+    (setq dir gnus-uu-work-dir))
+  (when (and dir
+            (file-exists-p dir))
+    (let ((files (directory-files dir t nil t))
+         file)
+      (while (setq file (pop files))
+       (unless (string-match "/\\.\\.?$" file)
+         (if (file-directory-p file)
+             (gnus-uu-delete-work-dir file)
+           (gnus-message 9 "Deleting file %s..." file)
+           (delete-file file))))
+      (delete-directory dir)))
+  (gnus-message 7 ""))
 
 ;; Initializing
 
 (add-hook 'gnus-exit-group-hook 'gnus-uu-clean-up)
-(add-hook 'gnus-exit-group-hook        'gnus-uu-check-for-generated-files)
+(add-hook 'gnus-exit-group-hook        'gnus-uu-delete-work-dir)
 
 \f
 
@@ -1654,9 +1641,6 @@ The headers will be included in the sequence they are matched.")
 ;;; uuencoded posting
 ;;;
 
-(require 'sendmail)
-(require 'rnews)
-
 ;; Any function that is to be used as and encoding method will take two
 ;; parameters: PATH-NAME and FILE-NAME. (E.g. "/home/gaga/spiral.jpg"
 ;; and "spiral.jpg", respectively.) The function should return nil if
@@ -1720,8 +1704,6 @@ is t.")
   "Inserts an encoded file in the buffer.
 The user will be asked for a file name."
   (interactive)
-  (if (not (eq (current-buffer) (get-buffer gnus-post-news-buffer)))
-      (error "Not in post-news buffer"))
   (save-excursion 
     (setq gnus-uu-post-inserted-file-name (gnus-uu-post-insert-binary))))
 
@@ -1757,7 +1739,7 @@ The user will be asked for a file name."
                  file-name))
   (insert (format "Content-Transfer-Encoding: %s\n\n" encoding))
   (save-restriction
-    (set-buffer gnus-post-news-buffer)
+    (set-buffer gnus-message-buffer)
     (goto-char (point-min))
     (re-search-forward (concat "^" (regexp-quote mail-header-separator) "$"))
     (forward-line -1)
@@ -1778,8 +1760,6 @@ The user will be asked for a file name."
   "Posts the composed news article and encoded file.
 If no file has been included, the user will be asked for a file."
   (interactive)
-  (if (not (eq (current-buffer) (get-buffer gnus-post-news-buffer)))
-      (error "Not in post news buffer"))
 
   (let (file-name)
 
@@ -1788,10 +1768,10 @@ If no file has been included, the user will be asked for a file."
       (setq file-name (gnus-uu-post-insert-binary)))
   
     (if gnus-uu-post-threaded
-       (let ((gnus-required-headers 
-              (if (memq 'Message-ID gnus-required-headers)
-                  gnus-required-headers
-                (cons 'Message-ID gnus-required-headers)))
+       (let ((message-required-news-headers 
+              (if (memq 'Message-ID message-required-news-headers)
+                  message-required-news-headers
+                (cons 'Message-ID message-required-news-headers)))
              gnus-inews-article-hook)
 
          (setq gnus-inews-article-hook (if (listp gnus-inews-article-hook)
@@ -1895,7 +1875,7 @@ If no file has been included, the user will be asked for a file."
          (progn
            (end-of-line)
            (insert (format " (0/%d)" parts))))
-      (gnus-inews-news))
+      (message-send))
 
     (save-excursion
       (setq i 1)
@@ -1958,7 +1938,8 @@ If no file has been included, the user will be asked for a file."
              (forward-line 1)))
        (insert beg-line)
        (insert "\n")
-       (gnus-inews-news)))
+       (let (message-sent-message-via)
+         (message-send))))
 
     (and (setq buf (get-buffer send-buffer-name))
         (kill-buffer buf))