D10629: Avoid a double emitResult()

Jaime Torres Amate noreply at phabricator.kde.org
Sun Feb 18 10:16:28 UTC 2018


jtamate created this revision.
jtamate added reviewers: Frameworks, Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
jtamate requested review of this revision.

REVISION SUMMARY
  ServiceJob::setResult already does a emitResult.
  Discovered with the assert created in https://phabricator.kde.org/D9862

REPOSITORY
  R120 Plasma Workspace

BRANCH
  double_finish (branched from master)

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

AFFECTED FILES
  dataengines/notifications/notificationaction.cpp

To: jtamate, #frameworks, #plasma
Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180218/557f0d8c/attachment.html>


More information about the Kde-frameworks-devel mailing list