D7628: Clear previous actions on update

David Edmundson noreply at phabricator.kde.org
Thu Aug 31 13:09:26 UTC 2017


davidedmundson created this revision.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.

REVISION SUMMARY
  addAction() doesn't check if an action already exists with that name so
  we have a tiny memory leak.
  
  Also I wanted to try and support actions potentially changing if a
  remote service gets restarted. So we should clear our current actions
  before repopulating

TEST PLAN
  Builds

REPOSITORY
  R308 KRunner

BRANCH
  master

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

AFFECTED FILES
  src/dbusrunner.cpp

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


More information about the Kde-frameworks-devel mailing list