[cfe-commits] Fwd: PATCH: Fix a bunch of Doxygen-formatting gremlins (no more "class A" page etc.)

James Dennett 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:
> http://llvm.org/docs/DeveloperPolicy.html#commitaccess
> 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.

-- James
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120606/c903dc70/attachment.html>

More information about the cfe-commits mailing list