[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