Re: [Emacs-diffs] master 3431e82: Ignore directory symlinks in directory-files-recursively

[Lars Ingebrigtsen <larsi@gnus.org>]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> And I don't like passing in the DIR and LEAF separately.
>
> It can avoid calling expand-file-name only to call file-name-directory
> (or file-name-nondirectory) right after.

It can, but in the use cases I've typically seen, you wouldn't do that,
I think?  You'd say `(directory-name-p file)' and then...  something...

>> -(defun directory-files-recursively (dir match &optional include-directories)
>> +(defun directory-files-recursively (dir match &optional
>> +					include-directories predicate)
>
> Can we reduce the number of arguments?

That would be nice, but I'm not sure what to remove.  The canonical,
that most of the callers use (from the other versions of this function
included in various packages in Emacs) is

(directory-files-recursively "~/" "~\\'")

or something.  And then there are a couple that want the directories,
too.  And then there's `file-tree-walk', which is the new thing.

Uhm...  we could make MATCH be the predicate if it's not a string?

>>  		   (path (expand-file-name leaf dir)))
>
> This is not a "path", it's a file name.  Remember: by convention
> a "path" is a list of directories, as in load-path, $PATH, etc...

Right.  But do we have a word for "string that designates either a file
or a directory"?  That's usually called "a path"...

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