[cfe-dev] No virtual destructor in ExternalSemaSource
vasil.georgiev.vasilev at cern.ch
Wed Feb 2 01:02:53 PST 2011
On 02/01/2011 09:01 PM, John McCall wrote:
> On Feb 1, 2011, at 3:35 AM, Vassil Vassilev wrote:
>> I found out that there is no virtual destructor for ExternalSemaSource:
>> include/clang/Sema/ExternalSemaSource.h:33: ~ExternalSemaSource();
>> include/clang/AST/ExternalASTSource.h:56: virtual
>> Is that bug or I am misunderstanding something? Shall we (Axel
>> Naumann and me) check in patch.
> Methods with the same signature as virtual methods in a base class are
> implicitly virtual, and this applies to destructors, too.
Oh crap... I forgot that ExternalSemaSource derives from ExternalASTSource.
More information about the cfe-dev