> On Dec 16, 2016, at 11:33 AM, Jeremy Hughes <moon.rab...@virginmedia.com> > wrote: >> On 16 Dec 2016, at 19:29, John McCall <rjmcc...@apple.com> wrote: >> >>> On Dec 16, 2016, at 11:24 AM, Jeremy Hughes <moon.rab...@virginmedia.com> >>> wrote: >>> Thanks for the link. >>> >>> I’ve looked at it, and I don’t think it applies in this case, because I’m >>> not actually overriding the document(_:didPrint:contextInfo:) method, just >>> providing it as a callback. There isn’t an override keyword before the >>> method, and there isn’t a superclass method that I can call through to. I >>> suppose it would apply if I had a subclass (of my document class) that >>> needed to override the callback. Does that seem right to you? >> >> You're overriding printDocumentWithSettings to ignore the passed-in delegate >> and selector. Unless you're certain that the passed-in delegate and >> selector are never useful, you're probably breaking something. >> >> John. > > OK - I misunderstood what Quincey was saying. > > The passed-in delegate and selector are nil, but I obviously can’t be sure > that they will always be nil. > > All I’m actually trying to do is to clean up some objects that need to exist > during the print operation. Is this the best way for me to be doing that?
I'm not an expert in this part of Cocoa. Are there implicit system *callers* of this method, or is it more of a system *utility* that you're expected to call from your own code? If it's the latter, then maybe instead of overriding it you should just provide a different method that calls it, doing whatever set up you need first and then passing in the appropriate delegate and selector to do cleanup. John. _______________________________________________ Cocoa-dev mailing list (Cocoa-dev@lists.apple.com) Please do not post admin requests or moderator comments to the list. Contact the moderators at cocoa-dev-admins(at)lists.apple.com Help/Unsubscribe/Update your Subscription: https://lists.apple.com/mailman/options/cocoa-dev/archive%40mail-archive.com This email sent to arch...@mail-archive.com
