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. > > 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
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
