Here the patch.

Peter Kümmel wrote:
> Abdelrazak Younes wrote:
>> Peter Kümmel wrote:
>>> Bennett Helm wrote:
>>>> Well done!
>>>>
>>>> Bennett
>>> Great, we've solved it.
>>> Is in svn now.
>> Congratulations to you both!
>>
>> Abdel.
>>
> 
> Thanks, Abbel.
> But congratulations to you, too.
> Your recent speedups are really impressive.
> 
> 
> Currently we have memory leaks because the closed
> views are not deleted. With attached patch the
> delete on close is enabled, but we get a crash.
> 
> Could you have a quick look at it? The crash is
> because the menu somehow points to a destructed
> instance. Hope you have instantly an idea how
> to fix this, if not we have to investigate it
> in more detail.
> 
> Peter
> 


-- 
Peter Kümmel
Index: src/frontends/qt4/GuiView.C
===================================================================
--- src/frontends/qt4/GuiView.C (Revision 16155)
+++ src/frontends/qt4/GuiView.C (Arbeitskopie)
@@ -159,7 +159,7 @@
        setAttribute(Qt::WA_QuitOnClose, false);
        // FIXME: the following statement avoids memory leaks but produces a
        //        crash after a new window has been closed (click into the menu)
-       //setAttribute(Qt::WA_DeleteOnClose, true);
+       setAttribute(Qt::WA_DeleteOnClose, true);
 
        // hardcode here the platform specific icon size
        d.smallIconSize = 14;   // scaling problems

Reply via email to