Differential D1382

remove duplicate Declaration entry for DUChainUtils::getOverriders(...) and DUChainUtils::getInheriters(...)
AbandonedPublic
Actions

Authored by zhanghuijie on Apr 11 2016, 4:53 AM.

Details

Reviewers
kfunk
Group Reviewers
KDevelop
Summary

The current impl of DUChainUtils::getOverriders(...) and DUChainUtils::getInheriters(...) may return some duplicate items in some case. This can be seen while we open the class browser or use the declaration navigation.

Diff Detail

Repository
R33 KDevPlatform
Lint
Lint Skipped
Unit
Unit Tests Skipped
zhanghuijie updated this revision to Diff 3269.Apr 11 2016, 4:53 AM
zhanghuijie retitled this revision from to remove duplicate Declaration entry for DUChainUtils::getOverriders(...) and DUChainUtils::getInheriters(...).
zhanghuijie updated this object.
zhanghuijie edited the test plan for this revision. (Show Details)
zhanghuijie added reviewers: KDevelop, kfunk.
zhanghuijie set the repository for this revision to R33 KDevPlatform.
zhanghuijie added a project: KDevelop.
zhanghuijie added a subscriber: KDevelop.
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptApr 11 2016, 4:53 AM
mwolff added a subscriber: mwolff.Apr 12 2016, 12:47 AM

this is for 1.7, right? wasn't this fixed in master already?

zhanghuijie added a comment.Apr 12 2016, 1:12 AM
In D1382#25792, @mwolff wrote:

this is for 1.7, right? wasn't this fixed in master already?

Yes, this is for 1.7. And I will have a look at the master branch.

zhanghuijie added a comment.Apr 12 2016, 5:38 AM

This was fixed in the master branch, and I'd like to port it to 1.7 :).

zhanghuijie abandoned this revision.Apr 12 2016, 5:54 AM

Revision Contents
Changeset List

Diff 3269

View Options

language/duchain/navigation/abstractdeclarationnavigationcontext.cpp

Loading...
View Options

language/duchain/navigation/usescollector.cpp

Loading...
View Options

plugins/classbrowser/classmodelnode.cpp

Loading...