r179297 - [libclang] Have clang_getCXXAccessSpecifier() also return the access control of a C++ declaration within its parent scope.

Argyrios Kyrtzidis akyrtzi at gmail.com
Thu Apr 11 10:33:00 PDT 2013


On Apr 11, 2013, at 10:22 AM, Jordan Rose <jordan_rose at apple.com> wrote:

> 
> On Apr 11, 2013, at 10:02 , Argyrios Kyrtzidis <akyrtzi at gmail.com> wrote:
> 
>> Author: akirtzidis
>> Date: Thu Apr 11 12:02:10 2013
>> New Revision: 179297
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=179297&view=rev
>> Log:
>> [libclang] Have clang_getCXXAccessSpecifier() also return the access control of a C++ declaration within its parent scope.
>> 
>> Suggested by Stefan Seefeld.
>> 
>> Modified:
>>    cfe/trunk/include/clang-c/Index.h
>>    cfe/trunk/test/Index/load-classes.cpp
>>    cfe/trunk/tools/c-index-test/c-index-test.c
>>    cfe/trunk/tools/libclang/CIndexCXX.cpp
>> 
>> Modified: cfe/trunk/include/clang-c/Index.h
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang-c/Index.h?rev=179297&r1=179296&r2=179297&view=diff
>> ==============================================================================
>> --- cfe/trunk/include/clang-c/Index.h (original)
>> +++ cfe/trunk/include/clang-c/Index.h Thu Apr 11 12:02:10 2013
>> @@ -2995,9 +2995,10 @@ enum CX_CXXAccessSpecifier {
>> };
>> 
>> /**
>> - * \brief Returns the access control level for the C++ base specifier
>> - * represented by a cursor with kind CXCursor_CXXBaseSpecifier or
>> - * CXCursor_AccessSpecifier.
>> + * \brief Returns the access control level for the referenced object.
>> + * If the cursor refers to a C++ declaration, its access control level within its
>> + * parent scope is returned. Otherwise, if the cursor refers to a base specifier or
>> + * access specifier, the specifier itself is returned.
>>  */
> 
> Nitpicking: for best Doxygen results, insert an extra newline after the \brief, so that the explication doesn't get included in the generated summary table.

In r179300, thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130411/44a30d9f/attachment.html>


More information about the cfe-commits mailing list