Ben Langmuir <blangmuir at apple.com> writes: >> + bool Continue = true; >> + if (First->needsInputFileVisitation() && > > The above notwithstanding, Continue should start false, or we will > never return true :-) If you fix that, then LGTM! Right, good catch! Committed with that fixed in r209394.