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

Chandler Carruth chandlerc at google.com
Thu Jun 7 10:32:42 PDT 2012


LGTM

Want me to commit in the interim?

On Thu, Jun 7, 2012 at 10:29 AM, James Dennett <jdennett at google.com> wrote:

> On Wed, Jun 6, 2012 at 9:57 PM, James Dennett <jdennett at googlers.com>wrote:
>
>> 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.
>>
>
> Revised patch attached.  (No, really.  I checked this time.)
>
> -- James
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120607/2425013a/attachment.html>


More information about the cfe-commits mailing list