[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: prompt of cvs-do-removal

From: Kevin Rodgers
Subject: Re: prompt of cvs-do-removal
Date: Mon, 25 Aug 2003 16:27:46 -0600
User-agent: Mozilla/5.0 (X11; U; SunOS i86pc; en-US; rv: Gecko/20020406 Netscape6/6.2.2

Masatake YAMATO wrote:

P.S. Thank you for reviewing my patch every time.
-                    (yes-or-no-p (format "Delete %d files? " (length files)))
+                    (if (eq 1 (length files))
+                        (yes-or-no-p (format "Delete file: \"%s\" ? " 
(cvs-fileinfo->file (car files))))
+                      (yes-or-no-p (format "Delete %d files? " (length 

Wouldn't = be more appropriate than eq there?  And you may want to bind (length 
files) to a local

variable, so you don't have to re-evaluate it.

Kevin Rodgers

reply via email to

[Prev in Thread] Current Thread [Next in Thread]