GNU bug report logs - #7112
24.0.50; [PATCH] `ls-lisp-insert-directory' should be no-op for empty FILE

Previous Next

Full log

Message #14 received at 7112 <at> debbugs.gnu.org (full text, mbox):

From: Lars Magne Ingebrigtsen <larsi <at> gnus.org>
To: "Drew Adams" <drew.adams <at> oracle.com>
Cc: 7112 <at> debbugs.gnu.org
Subject: Re: 24.0.50;
	[PATCH] `ls-lisp-insert-directory' should be no-op for empty FILE
Date: Tue, 02 Aug 2011 22:06:46 +0200

"Drew Adams" <drew.adams <at> oracle.com> writes:

>> > (error "Args out of range: \"\", -1")
>> > The reason is the following code near the end of
>> > `ls-lisp-insert-directory':
>> 
>> I think this should most likely be fixed in the caller (i.e., in
>> dired.el).
>
> What does that mean?  dired.el is a file, not a caller.

dired.el should not be asking ls-lisp to list a directory called "",
which is obviously something that should generate an error.

>> What's the backtrace for this bug?
>
> Why not try it yourself, using the emacs -Q recipe:
>
> M-: (dired '("foobar" "111.el" ""))

Why would you call this function with an empty string as a parameter?

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.