[Differential] [Updated] D845: TopContextDynamicData: Fix bug in loadPartialData

mwolff (Milian Wolff) noreply at phabricator.kde.org
Wed Jan 20 12:00:10 UTC 2016


mwolff added a comment.

Ouch! Sorry for that mess... Any idea how we could unit test this?


INLINE COMMENTS
  language/duchain/topducontextdynamicdata.cpp:174 Shouldn't we always use readValue?
  
  Also good gotcha! No compiler warns us in this case that readValue is "set but not used" :(

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


More information about the KDevelop-devel mailing list