zwabel at googlemail.com
Sat Jul 31 20:06:22 UTC 2010
The patch looks good. The appended-list stuff is only required if you
have declared such lists.
The DUContext.:Language enumerator is redundant with the type itself.
You think it's really required? You can just do
"dynamic_cast<LanguageDUContext*>(..)" after all.
What do you mean by integrating it into the abstract context builder,
is there anything you need to do? I anyway think the abstract builders
are much too complicated in design. It would be nicer to have "helper"
classes instead that would not be part of the builder inheritance
More information about the KDevelop-devel