Re: Why I attempted to port dired/VC from emacs
Mats Lidell wrote:
>>>>> Rodney Sparapani <rsparapa(a)mcw.edu>
writes:
>>>>>
Rodney> After a little more testing, the vc.el change does not seem to
Rodney> be necessary any more. Therefore, the minimal change
Rodney> necessary is:
During testing of this patch I got a little confused. You see I'm new
to the vc-directory mode so I suspected erroneously(!?) that when I
got no files listed in an unmodified directory that there was
something wrong. I was expecting a full directory list but with all
files listed as unmodified. But after checking with Emacs if found
that the empty listing is just how it is supposed to be. Right?
Right.
If that is the case I don't seem to need the patch to
dired-sort-other
at all. It works for me without it. Do you have a use case that
illustrates when the patch is needed so that I can recreate the
problem?
Yours
To list all files requires a 2nd call to vc-directory (as you noted;
I never would have thought of that). Without this patch, then you get
the error that I reported (and solved) on 4/22:
Debugger entered--Lisp error: (wrong-type-argument natnump -1)
dired-sort-other((?a ?l ?R))
dired-internal-noselect("/home/rsparapa/svn/ess/" "-alR"
vc-dired-mode)
vc-directory("/home/rsparapa/svn/ess/" nil)
call-interactively(vc-directory)
command-execute(vc-directory t)
execute-extended-command(nil)
call-interactively(execute-extended-command)
And, I think that this is really the problem that Issue 491 is all about.
The "mangled" list is simply nil which it should never be.
--
Rodney Sparapani Center for Patient Care & Outcomes Research (PCOR)
Sr. Biostatistician http://www.mcw.edu/pcor
4 wheels good, 2 wheels better! Medical College of Wisconsin (MCW)
WWLD?: What Would Lombardi Do? Milwaukee, WI, USA
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-beta