[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    extragear/plasma/applets/notes
From:       Sebastian Kügler <sebas () kde ! nl>
Date:       2008-05-12 1:58:00
Message-ID: 1210557480.458030.20361.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 806682 by sebas:

Don't leak the widgets


 M  +11 -8     notes.cpp  


--- trunk/extragear/plasma/applets/notes/notes.cpp #806681:806682
@@ -50,6 +50,17 @@
     updateTextGeometry();
 }
 
+Notes::~Notes()
+{
+    //FIXME is it really ok to save from here?
+    //also, this has a really weird effect: if I remove a note then add a new one, I can get the old
+    //text back. it was useful when there were load/save issues but it's silly now.
+    saveNote();
+    delete m_textEdit;
+    delete m_layout;
+    delete m_proxy;
+}
+
 void Notes::init()
 {
     m_notes_theme.setContainsMultipleImages(false);
@@ -133,14 +144,6 @@
     }
 }
 
-Notes::~Notes()
-{
-    //FIXME is it really ok to save from here?
-    //also, this has a really weird effect: if I remove a note then add a new one, I can get the old
-    //text back. it was useful when there were load/save issues but it's silly now.
-    saveNote();
-}
-
 void Notes::paintInterface(QPainter *p,
                            const QStyleOptionGraphicsItem *option,
                            const QRect &contentsRect)
[prev in list] [next in list] [prev in thread] [next in thread]