KDE/kdevelop/plugins/languages/cpp

[Andreas Pakulat]

On 05.01.09 00:15:12, Alex Merry wrote:
> On Sunday 04 January 2009 23:51:18 David Nolden wrote:
> > Am Monday 05 January 2009 00:42:56 schrieb Andreas Pakulat:
> > It might be loaded when the class-browser or something similar requests to
> > laod some C++ duchain objects, and if it happens in foreground, this will
> > trigger a loading of the language support.
> >
> > Even if class-view is fixed, we would still have to live with this.
> 
> Unless you can guarantee that a Language object is created for "C++" and added 
> to the LanguageController every time the C++ language support plugin is 
> loaded, I would have thought the guard I added needs to be there.
> 
> Actually, I can't quite see why CppLanguageSupport::language() returns 0 
> rather than going into an infinite loop, since it calls core()-
> >languageController()->language("C++"), and 
> LanguageController::language(QString) seems to call 
> ILanguageSupport::language() (presumeably CppLanguageSupport::language() in 
> this case) if it can't find a cached Language object.

I don't see why the C++ language support shouldn't register itself with the
language controller when its loaded, so its guaranteed that language() will
always return a proper value.

Andreas 

--