[PATCH] Use DiagnosticRenderer to convert clang diagnostics to clang-tidy ones.

Alexander Kornienko alexfh at google.com
Mon Mar 10 06:15:50 PDT 2014



================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:71
@@ +70,3 @@
+    for (auto FixIt : Hints) {
+      // FIXME: Find out if const_cast is valid here.
+      Output.Fix.insert(tooling::Replacement(const_cast<SourceManager &>(SM),
----------------
Manuel Klimek wrote:
> Replacement should take a const SourceManager.
Already done.

================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:95
@@ +94,3 @@
+private:
+  ClangTidyError &Output;
+};
----------------
Manuel Klimek wrote:
> I'd probably call it "Error". I got confused as to what this is when I read "Output" in the code above.
Done.


http://llvm-reviews.chandlerc.com/D2950

BRANCH
  svn

ARCANIST PROJECT
  clang-tools-extra



More information about the cfe-commits mailing list