[Differential] [Commented On] D845: TopContextDynamicData: Fix bug in loadPartialData

Wed Jan 20 12:08:42 UTC 2016

kfunk added inline comments.

INLINE COMMENTS
  language/duchain/topducontextdynamicdata.cpp:174 ASAN is just awesome, I encourage everyone to re-build KDevelop with it :)

  Re. "always use readValue". Well the partial load is clearly an optimization attempt, I'm fine with leaving it as-is. I've noticed we're loading quite a few top context during shutdown; any performance boost there is likely useful.

REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

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

To: kfunk, mwolff
Cc: kdevelop-devel