[Differential] [Commented On] D3110: Periodically update progress bar

[brauch (Sven Brauch)]

brauch added a comment.


  Good change in general, but why don't you simply start the timer at the place where showProgress() was emitted before, in case it's not running yet? Then you wouldn't need the "toggled on/off" state for the timer.

REPOSITORY
  rKDEVPLATFORM KDevPlatform

REVISION DETAIL
  https://phabricator.kde.org/D3110

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: wcpan, kfunk, #kdevelop
Cc: brauch, kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20161028/a8c2a0fd/attachment.html>