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