[cfe-commits] [PATCH] Make CompilationDatabase work with symlinks and relative paths

[Daniel Jasper]

  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