[PATCH] Fix doxygen warnings
Ariel Bernal
ariel.j.bernal at intel.com
Mon Jul 22 10:09:03 PDT 2013
================
Comment at: cpp11-migrate/Core/FileOverrides.cpp:36
@@ -36,1 +35,3 @@
+void
+SourceOverrides::applyReplacements(clang::tooling::Replacements &Replaces) {
llvm::IntrusiveRefCntPtr<clang::DiagnosticOptions> DiagOpts(
----------------
Edwin Vane wrote:
> Ariel Bernal wrote:
> > Edwin Vane wrote:
> > > using namespace clang::tooling is in effect. Why not just remove tooling::? Does this fix Doxygen too?
> > The problem is that the method definition in FileOverrides.h doesn't match the type and doxygen can't find this declaration.
> There's got to be more to this problem than that. If you look at Transform.h and Transform.cpp you'll see the header uses clang:: in the method declaration and the source file doesn't use clang:: because of the using directive. Doxygen doesn't complain about this file...
Yes you are right. I'll try to find out why it doesn't work here.
http://llvm-reviews.chandlerc.com/D1194
More information about the cfe-commits
mailing list