[PATCH] Fix doxygen warnings

Edwin Vane edwin.vane at intel.com
Mon Jul 22 10:05:23 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(
----------------
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...


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



More information about the cfe-commits mailing list