[Differential] [Request, 96 lines] D2792: fix KDevelop::JobStatus for "short-lived" jobs

[antonanikin (Anton Anikin)]

antonanikin created this revision.
antonanikin added a reviewer: kfunk.
antonanikin added a subscriber: KDevelop.
antonanikin set the repository for this revision to rKDEVPLATFORM KDevPlatform.
Restricted Application added a subscriber: kdevelop-devel.

REVISION SUMMARY
  The patch fixes showing progress status for "short-lived" jobs. By default `StatusbarProgressWidget` blocks progress showing for statuses for 1000 msec. This is a good way to hide annoying progress showing for "small" tasks, like background parsing of saved source file. But this behavior also leads to blocking progress for "short-lived" statuses for different KJob's. Such jobs can show some output to OuputView therefore progress blocking is not problem for them. But if job do some "silent" work it will be very useful for users to indicate what the job is finished.

TEST PLAN
  Tested on master branch with different short- (<1 s.) and long-lived (>1 s.) jobs:
  
  1. Standard builder jobs.
  2. kdev-cppcheck jobs, which emits `percent` signals during work.
  3. kdev-cppcheck jobs, which don't emits `percent` signals during work.

REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

AFFECTED FILES
  util/jobstatus.cpp
  util/jobstatus.h

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

To: antonanikin, kfunk
Cc: kdevelop-devel, #kdevelop
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160916/7a7bec0c/attachment.html>