[Differential] [Commandeered] D583: Fix DUChainUtils::getInheriters.

[mwolff (Milian Wolff)]

mwolff commandeered this revision.
mwolff added a reviewer: gsmolarczyk.
mwolff added a comment.


  I've now pushed a proper fix for this issue to kdev-clang. No DUChain feature needs to be changed, it was a misuse of that API that triggered this issue. Olivier was right btw, we simply should not import the semantic parent context if it equals the lexical parent.
  
  commit https://phabricator.kde.org/rKDEVELOPf945e0f03ee5bc49a0728e4bd7fdd239351745cc
  Author: Milian Wolff <mail at milianw.de>
  Date:   Sun Feb 14 18:38:00 2016 +0100
  
    Don't import the semantic parent if it equals the lexical parent.
    
    We to import the semantic parent only if it differs from the lexical
    parent, as then we deal with out-of-line declarations. For the other
    case, the normal DUChain tree works fine. And importing the parent
    again can lead to subtle differences, like DUChainUtils::getInheriters
    misbehaving for nested classes.
    
    This change should also have a nice positive performance impact.

REPOSITORY
  rKDEVPLATFORM KDevPlatform

REVISION DETAIL
  https://phabricator.kde.org/D583

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mwolff, KDevelop, kfunk, gsmolarczyk
Cc: kfunk, olivierjg, mwolff, kdevelop-devel, arrowdodger