[Differential] [Changed Subscribers] D758: Introduce IFilterStrategy::progressInLine [1/3]
mwolff (Milian Wolff)
noreply at phabricator.kde.org
Fri Jan 8 16:08:02 UTC 2016
mwolff added a subscriber: mwolff.
INLINE COMMENTS
outputview/ifilterstrategy.h:46 struct
outputview/ifilterstrategy.h:49 float percent?
also save a few chars via
const QString& status = {}
outputview/ifilterstrategy.h:80 how do you differentiate between invalid and valid progress? 0% can be valid, no?
outputview/outputexecutejob.cpp:43 cref
outputview/outputexecutejob.cpp:409 scoped ptr + reset?
outputview/outputfilteringstrategies.h:50 now ots public api you must use the real export macro
outputview/outputfilteringstrategies.h:79 please do
outputview/outputmodel.cpp:104 cref
outputview/outputmodel.h:81 cref
REPOSITORY
rKDEVPLATFORM KDevPlatform
REVISION DETAIL
https://phabricator.kde.org/D758
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: kfunk, apol
Cc: mwolff, apol, kdevelop-devel, arrowdodger
More information about the KDevelop-devel
mailing list