[cfe-commits] r165771 - in /cfe/trunk: include/clang/AST/ASTContext.h include/clang/AST/Comment.h lib/AST/ASTContext.cpp lib/AST/Comment.cpp lib/AST/CommentSema.cpp test/Index/overriding-method-comments.mm tools/libclang/CXComment.cpp

jahanian fjahanian at apple.com
Mon Oct 15 12:02:50 PDT 2012


On Oct 12, 2012, at 11:06 AM, Douglas Gregor <dgregor at apple.com> wrote:

> 
> On Oct 11, 2012, at 4:52 PM, Fariborz Jahanian <fjahanian at apple.com> wrote:
> 
>> Author: fjahanian
>> Date: Thu Oct 11 18:52:50 2012
>> New Revision: 165771
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=165771&view=rev
>> Log:
>> search for overridden methods with comment when overriding method
>> has none of its own. Factor in Doug's comments.
>> // rdar://12378793
> 
> Looks good. Shouldn't we do the same for \tparam as well, e.g.,
> 
> /// \brief Documentation
> /// \tparam T The type, silly.
> template<typename T>
> void foo(T);
> 
> template<typename U>
> void foo(U);

In r 165953.
- Fariborz

> 
> ?
> 
> 	- Doug




More information about the cfe-commits mailing list