[cfe-dev] [PATCH] Default implementation for ExternalASTSource's pure virtual functions

Douglas Gregor dgregor at apple.com
Thu Feb 24 11:46:09 PST 2011



Sent from my iPhone

On Feb 24, 2011, at 9:37 AM, Axel Naumann <Axel.Naumann at cern.ch> wrote:

> Hi,
> 
> I'd suggest to provide default implementations for ExternalASTSource's
> pure virtual functions. Derived classes should actually not be required
> to implement them. See e.g. r126387 (offering an orthogonal lookup
> source through ExternalASTSource::LookupUnqualified()), and lldb.

Please put these definitions into a .cpp file. They'll never be inlined anyway. 

> Attached patch does that (+ cleanup of forward declarations).
> 
> I think it's a semi-minor patch, so if there are no comments I'll commit
> it in 24 hours :-)
> 
> Cheers, Axel.
> <ExternalASTSource_default_impl.diff>
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev



More information about the cfe-dev mailing list