bug#13380: 24.2.91; Annoying change to cvs-cleanup-collection

[Stefan Monnier]

> The following change is just SO annoying.
> I'm accustomed to using 'x' in the *cvs* buffer to get rid of
> all the files that need attention -- but with this change,
> it is practically random what is kept and what is removed.

Oh, it was not meant to affect `x', indeed.  I just installed the patch
below which should fix it.


        Stefan


=== modified file 'lisp/vc/pcvs.el'
--- lisp/vc/pcvs.el     2013-01-01 09:11:05 +0000
+++ lisp/vc/pcvs.el     2013-01-08 17:29:57 +0000
@@ -876,9 +876,10 @@
                     (`DEAD nil)
                     ;; handled also?
                     (`UP-TO-DATE
+                      (not
                       (if (find-buffer-visiting (cvs-fileinfo->full-name fi))
-                          t
-                        (not rm-handled)))
+                           (eq rm-handled 'all)
+                         rm-handled)))
                     ;; keep the rest
                     (_ (not (run-hook-with-args-until-success
                              'cvs-cleanup-functions fi))))))
@@ -2121,7 +2122,7 @@
 Empty directories are removed."
   (interactive)
   (cvs-cleanup-collection cvs-cookies
-                         t (or cvs-auto-remove-directories 'handled) t))
+                         'all (or cvs-auto-remove-directories 'handled) t))
 
 
 (defun-cvs-mode cvs-mode-acknowledge ()