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