Nits and comments fixed.
================ Comment at: lib/Tooling/FileMatchTrie.cpp:79 @@ +78,3 @@ + StringRef Result = MatchingChild->getValue().findEquivalent( + Comparator, FileName, IsAmbiguous, ConsumedLength + Element.size() + 1); + if (!Result.empty() || IsAmbiguous) ---------------- Manuel Klimek wrote: > 80 columns. Done. ================ Comment at: include/clang/Tooling/FileMatchTrie.h:104 @@ +103,3 @@ + + void injectComparator(PathComparator *NewComparator); +private: ---------------- Manuel Klimek wrote: > I have a small preference to overloading the constructor (because we don't > need to ever call this multiple times). Done. ================ Comment at: lib/Tooling/FileMatchTrie.cpp:144 @@ +143,3 @@ + bool IsAmbiguous = false; + StringRef Result = Root->findEquivalent(*Comparator, FileName, IsAmbiguous, 0); + if (IsAmbiguous) ---------------- Manuel Klimek wrote: > 80 columns. Done. http://llvm-reviews.chandlerc.com/D30 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
