On Wed, Feb 6, 2013 at 11:48 PM, Richard Smith <[email protected]>wrote:

> On Wed, Feb 6, 2013 at 8:41 PM, Sean Silva <[email protected]> wrote:
>
>>    /// \brief Finds all the visible declarations with a given name.
>>    /// The current implementation of this method just loads the entire
>>    /// lookup table as unmaterialized references.
>> -  virtual DeclContext::lookup_result
>> +  virtual bool
>>
>> Bool return arguments probably deserve a mention in the comment, due to
>> their notorious ambiguity.
>>
>
> This is an overload of a virtual function; the return value is described
> in the base class.
>

Ah ok.

Do we have a policy on how to document virtual overrides?
>

Not that I know of.

-- Sean Silva
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to