LGTM Want me to commit in the interim?
On Thu, Jun 7, 2012 at 10:29 AM, James Dennett <[email protected]> wrote: > On Wed, Jun 6, 2012 at 9:57 PM, James Dennett <[email protected]>wrote: > >> On Wed, Jun 6, 2012 at 5:49 PM, Chandler Carruth <[email protected]>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 > > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
