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).
- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
Feb 14 2016
Feb 2 2016
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.
Nov 24 2015
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?