[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