[cfe-commits] Fwd: PATCH: Fix a bunch of Doxygen-formatting gremlins (no more "class A" page etc.)
jdennett at googlers.com
Wed Jun 6 21:57:31 PDT 2012
On Wed, Jun 6, 2012 at 5:49 PM, Chandler Carruth <chandlerc at google.com>wrote:
> Minor comment:
> -/// \class The AST-based call graph.
> +/// \class CallGraph
> +/// \brief The AST-based call graph.
> Here and many other places, please nuke the redundant '\class Foo'.
> Doxygen will always correctly associate the comment with a class 'Foo' if
> it immediately preceeds 'class Foo' in the source code. I think most of
> these do.
They do, and I was wondering if we could nuke those. I'll happily do so.
> Have you requested commit access? If not, please do so:
> These types of changes fall into the 'obvious' bucket IMO, and you should
> feel free to directly commit them. We can do any necessary bike shedding in
> post-commit review.
I'll make the request, as it would speed up the documentation cleanups.
Some of my later patches will be sufficiently non-obvious to warrant
review in any case.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits