D10874: Make sure we don't crash when stopping all jobs

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon Feb 26 17:01:17 UTC 2018


apol created this revision.
apol added a reviewer: KDevelop.
Restricted Application added a subscriber: kdevelop-devel.
apol requested review of this revision.

REVISION SUMMARY
  If we don't emit result, the ExecuteCompositeJob doesn't relise that
  it's failing and keeps the dangling object. Calling stop all processes
  would result in a crash then.

TEST PLAN
  - run app
  - run app again
  - choose to kill the first instance
  - stop all processes

REPOSITORY
  R32 KDevelop

BRANCH
  killallcrash

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

AFFECTED FILES
  plugins/execute/nativeappjob.cpp

To: apol, #kdevelop
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180226/4aec6deb/attachment.html>


More information about the KDevelop-devel mailing list