On Apr 11, 2013, at 10:02 , Argyrios Kyrtzidis <[email protected]> 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.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to