libclang: expose ref-qualifier of function type
Argyrios Kyrtzidis
akyrtzi at gmail.com
Fri Oct 11 13:02:25 PDT 2013
Committed in r192493, thanks!
On Oct 9, 2013, at 5:05 PM, Che-Liang Chiou <clchiou at gmail.com> wrote:
> On Mon, Oct 7, 2013 at 8:45 AM, Argyrios Kyrtzidis <akyrtzi at gmail.com> wrote:
>> Hi Che-Liang,
>>
>> On Oct 4, 2013, at 9:53 PM, Che-Liang Chiou <clchiou at gmail.com> wrote:
>>
>> This patch exposes ref-qualifier information of function type.
>> <refqualifier.patch>
>>
>>
>> +enum CXRefQualifierKind {
>> + /** \brief No ref-qualifier was provided. */
>> + CXRQ_None = 0,
>> + /** \brief An lvalue ref-qualifier was provided (\c &). */
>> + CXRQ_LValue,
>> + /** \brief An rvalue ref-qualifier was provided (\c &&). */
>> + CXRQ_RValue
>> +};
>>
>> The general convention used is to repeat the enumerator name without 'kind'
>> in the enumerators:
>
> Done.
>
>> enum CXRefQualifierKind {
>> /** \brief No ref-qualifier was provided. */
>> CXRefQualifier_None = 0,
>> /** \brief An lvalue ref-qualifier was provided (\c &). */
>> CXRefQualifier_LValue,
>> /** \brief An rvalue ref-qualifier was provided (\c &&). */
>> CXRefQualifier_RValue
>> };
>>
>>
>>
>>
>> + * \brief Retrieve the ref-qualifier kind of a function or method.
>> + *
>> + * The ref-qualifier is returned for functions or methods. For other types
>> + * CXRQ_None is returned.
>> + */
>> +CINDEX_LINKAGE enum CXRefQualifierKind clang_Type_getRefQualifier(CXType
>> T);
>>
>> Could you rename it to clang_Type_getCXXRefQualifier and mention in the
>> comments that this is only relevant for C++ ?
>
> Done.
>
>> Finally, could you also add some tests for this ?
>>
>
> Done.
> <refqualifier-v2.patch>
More information about the cfe-commits
mailing list