|
From: | Riccardo Mottola |
Subject: | Re: Testers wanted for MPDCon and Zipper |
Date: | Wed, 22 Feb 2012 13:10:32 +0100 |
User-agent: | Mozilla/5.0 (X11; OpenBSD i386; rv:5.0) Gecko/20110815 Firefox/5.0 SeaMonkey/2.2 |
Hi, Wolfgang Lux wrote:
You are right, I knew I introduced a memory leak, but better than a crash. I fixed that now by releasing the controller from the AppDelegate who alloc'd it in the first place. This is easy, since it is modal. I think it can stay modal, there is no gain in making it modeless for the preferences it holds, although i generally hate modal panelsYou may like it or not, but this part was correct. What you did then was replacing poor code (using a modal panel where a non-modal panel should be used) by even worse code (you now also have a space leak). Since the problem is the close button of the panel you simply should have disabled the close box in the panel for now.
Riccardo
[Prev in Thread] | Current Thread | [Next in Thread] |