You're right, I didn't notice that.  But since you mention it, surely that had to have been a bug in the original implementation right?  That flag isn't intended to be a synonym for "non recursive iteration ", because that's what the Next enumeration value is for.  The algorithm would intentionally leave that decision up to the callback.<br><br>Find directories seems to want to mean "should you call my callback with directories?"<br><br>Imagine someone wants all files recursively but nothing else, that was impossible before, they would have to opt in to seeing each directory just so they could return Enter<br><div class="gmail_quote"><div dir="ltr">On Fri, Mar 10, 2017 at 2:27 AM Pavel Labath via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: lldb/source/Host/common/FileSpec.cpp:786<br class="gmail_msg">
+      continue;<br class="gmail_msg">
+    if (!find_directories && fs::is_directory(Status))<br class="gmail_msg">
+      continue;<br class="gmail_msg">
----------------<br class="gmail_msg">
This looks like it changes behavior. Previously, if `find_directories` was false this function would *not* recurse into them, whereas now it will. I guess you did not intend to do that (?)<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D30807" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D30807</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>