Fix Coverity findings
ClosedPublic

Authored by ckertesz on Jun 5 2018, 2:05 PM.

Details

Reviewers
mutlaqja

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
ckertesz created this revision.Jun 5 2018, 2:05 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptJun 5 2018, 2:05 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
ckertesz requested review of this revision.Jun 5 2018, 2:05 PM

What's rationale for 1395 change?

What's rationale for 1395 change?

activeJob was already dereferenced a few lines above in an if condition what is not visible in this diff.

mutlaqja accepted this revision.Jun 5 2018, 2:13 PM
This revision is now accepted and ready to land.Jun 5 2018, 2:13 PM
ckertesz closed this revision.Jun 5 2018, 2:51 PM