I have tested these changes (btw: I had to compile 5.0 branch instead of master, it's a bit confusing that the newest fixes are firstly introduced on a release branch and then merged into master from time to time).

Well, I did not check the other proposed solution of stopping importing the semantic parent. That could fix this issue, but in my opinion would be more of a hack, unless the importing stuff should really be a synonym for inheritance.

Pushed by accident, ignore. The real diff is D583.

In D583#11138, @mwolff wrote:

Just a question: Is the clang builder correct in importing the base for the case you have in your unit test?

class foo { class inner {}; };

Why is inner importing foo - does anyone know? If that wouldn't be the case, then this patch here wouldn't be required. So before we accept this patch, I'd appreciate if someone could dig in a bit deeper and clarify whether it isn't a bug in kdev-clang that triggers the issue, instead of a generic DUChain issue here.

Perform the requested changes.

In D584#11132, @mwolff wrote:

nice, lgtm! Do you have commit rights?