Change in plasma-framework[master]: Fix leak in ContainmentActions

David Edmundson (Code Review) noreply at kde.org
Wed Feb 11 13:50:05 UTC 2015


David Edmundson has uploaded a new change for review.

  https://gerrit.vesnicky.cesnet.cz/r/366

Change subject: Fix leak in ContainmentActions
......................................................................

Fix leak in ContainmentActions

Change-Id: I41c9b9c224b36b045034755b07a92f2bdfe9f40e
---
M src/plasma/containmentactions.cpp
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.vesnicky.cesnet.cz:29418/plasma-framework refs/changes/66/366/1

diff --git a/src/plasma/containmentactions.cpp b/src/plasma/containmentactions.cpp
index 3abb7fe..f24bdac 100644
--- a/src/plasma/containmentactions.cpp
+++ b/src/plasma/containmentactions.cpp
@@ -50,6 +50,8 @@
     : d(new ContainmentActionsPrivate(KService::serviceByStorageId(args.count() > 0 ?
                                       args[0].toString() : QString()), this))
 {
+    setParent(parentObject);
+
     // now remove first item since those are managed by Wallpaper and subclasses shouldn't
     // need to worry about them. yes, it violates the constness of this var, but it lets us add
     // or remove items later while applets can just pretend that their args always start at 0

-- 
To view, visit https://gerrit.vesnicky.cesnet.cz/r/366
To unsubscribe, visit https://gerrit.vesnicky.cesnet.cz/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I41c9b9c224b36b045034755b07a92f2bdfe9f40e
Gerrit-PatchSet: 1
Gerrit-Project: plasma-framework
Gerrit-Branch: master
Gerrit-Owner: David Edmundson <david at davidedmundson.co.uk>


More information about the Plasma-devel mailing list