D7628: Clear previous actions on update

2017-08-31 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R308:d727165b08c9: Clear previous actions on update (authored by davidedmundson). REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7628?vs=19008&id=19009 REVISION DETAIL

D7628: Clear previous actions on update

2017-08-31 Thread Kai Uwe Broulik
broulik accepted this revision. This revision is now accepted and ready to land. REPOSITORY R308 KRunner BRANCH master REVISION DETAIL https://phabricator.kde.org/D7628 To: davidedmundson, #plasma, broulik Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensre

D7628: Clear previous actions on update

2017-08-31 Thread David Edmundson
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