dfaure requested changes to this revision. dfaure added a comment. This revision now requires changes to proceed.
endless refactoring, for the better :-) INLINE COMMENTS > slaveinterface.cpp:119 > + d->transfer_details.clear(); > + d->transfer_details.append({elapsed_time, (d->filesize - > d->offset)}); > } .append(last) would do the same in a more compact way. ... and then this means we could reduce code size even further by calculating lspeed *before* doing the append. first = ... last = ... lspeed = ... if (!lspeed) { d->transfer_details.clear(); } d->transfer_details.append(last); REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D17528 To: chinmoyr, dfaure Cc: bruns, kde-frameworks-devel, michaelh, ngraham