[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