Re: NSSavePanel fragile ABI - do we care?

[Luboš Doležel]

On 08/11/2013 03:03 PM, Fred Kiefer wrote:
> At the current state of gui breaking the ABI is fine, we do it with
> almost every release. The only thing we should think about is whether we
> want to make a 0.23.2 release of gui before that.
> 
> I can see what you need the _completionHandler ivar for, but what is the
> purpose of the _modelessWindowController?
> 
> One other way to implement this would be to extend the sheet code in
> NSApplication. At the moment sheets are just modal windows.
> One simple way would be to implement a sheet delegate class that just
> executes the completion block. In the long run, the other way around,
> implementing sheet delegates with completion blocks will be more
> elegant, but for now this could do.
> 
> Fred

That is used to save a reference of an NSWindowController responsible
for that modeless window until it's closed. See the attached patch.

RELEASE()ing it here would close the window immediately, and not keeping
the reference somewhere would leak it.

If I'm missing something and it can be done differently, please let me know.

-- 
Luboš Doležel

nssavepanel.m.patch
Description: Text Data