[PATCH v3 3/4] new: Merge error checks from add_files and add_files_recursive

Austin Clements amdragon at MIT.EDU
Thu May 24 14:57:06 PDT 2012


Quoth Jani Nikula on May 24 at 11:57 pm:
> On Thu, 24 May 2012, Austin Clements <amdragon at MIT.EDU> wrote:
> > Before XXX, add_files_recursive could have been called on a symlink to
> > a non-directory.  Hence, calling it on a non-directory was not an
> > error, so a separate function, add_files, existed to fail loudly in
> > situations where the path had to be a directory.
> 
> "Before XXX"?

Arg.  I meant to replace that with a commit ID, but now I don't
remember what commit I wanted to reference.

> Otherwise, this 3/4 and following 4/4 patch LGTM. I didn't bother
> looking at 1/4 and 2/4 again, as you say there were no changes.
> 
> 
> BR,
> Jani.


More information about the notmuch mailing list