On Mon, 2016-05-30 at 09:55 +0200, Michael Haggerty wrote: > +struct dir_iterator_level { > + int initialized; > + > + DIR *dir; > + > + /* > + * The length of the directory part of refname at this level "refname"? Isn't this was for directories? > + if (lstat(iter->base.path.buf, &iter ->base.st) < 0) > + continue; /* silently skip */ Should we warn on non-ENOENT errors here? > +/* > + * End the iteration before it has been exhausted. Free the > reference s/reference/directory/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html