AFAIK the target isn't retained. Interesting point, though...if I remove the target/action ivars from SKColorSwatch and use super's implementation, things work (no exception). Are the overrides needed?
adam On Friday, November 30, 2007, at 01:42PM, "Christiaan Hofman" <[EMAIL PROTECTED]> wrote: >Is the target of a control retained? That may be the problem (in >[SKColorSwatch initWithCoder:]). I assumed it's not retained, as the >target can be the "owner" (of a nib or something). > >Christiaan > >On 30 Nov 2007, at 10:21 PM, Adam R. Maxwell wrote: > >> >> On Friday, November 30, 2007, at 01:08PM, "Christiaan Hofman" >> <[EMAIL PROTECTED]> wrote: >>> >>> On 30 Nov 2007, at 9:15 PM, Adam R. Maxwell wrote: >>> >>>> >>>> On Friday, November 30, 2007, at 11:53AM, "Christiaan Hofman" >>>> <[EMAIL PROTECTED]> wrote: >>>> >>>>> And what about the note adorns, in particular in the optional Add >>>>> Note segmented control? >>>> >>>> I'm not sure which one that is, but I can't customize the toolbar. >>>> The console logs this: >>>> >>>> 11/30/07 12:10:19 Skim[3127] Cannot remove an observer >>>> <SKMainWindowController 0x5658e20> for the key path >>>> "values.SKBackgroundColor" from <NSUserDefaultsController 0x251880> >>>> because it is not registered as an observer. >>>> >>>> I'll look at it in the debugger later. >>> >>> Strange log, it does register as an observer, unless the nib is never >>> loaded. But I doubt it has anything to d with toolbar customization, >>> as it should be logged at dealloc. >> >> Looks like it's copying a view. The item identifier is >> SKDocumentToolbarColorSwatchItemIdentifier. >> >> #0 0x91064680 in objc_exception_throw >> #1 0x90a4fe50 in +[NSException raise:format:arguments:] >> #2 0x90a4fe88 in +[NSException raise:format:] >> #3 0x954fbf18 in -[NSObject(NSKeyValueObserverRegistration) >> _removeObserver:forProperty:] >> #4 0x954fbc2c in -[NSObject(NSKeyValueObserverRegistration) >> removeObserver:forKeyPath:] >> #5 0x93aedf94 in -[NSController removeObserver:forKeyPath:] >> #6 0x000d3380 in -[NSUserDefaultsController(SKExtensions) >> removeObserver:forKeys:] at NSUserDefaultsController_SKExtensions.m:64 >> #7 0x0001f5b4 in -[SKMainWindowController unregisterAsObserver] at >> SKMainWindowController.m:599 >> #8 0x0001bd88 in -[SKMainWindowController dealloc] at >> SKMainWindowController.m:279 >> #9 0x9399f8c0 in -[NSWindowController release] >> #10 0x909b443c in __CFDictionaryDeallocate >> #11 0x909e48c4 in _CFRelease >> #12 0x954a7c7c in -[NSKeyedUnarchiver dealloc] >> #13 0x955016e8 in +[NSKeyedUnarchiver unarchiveObjectWithData:] >> #14 0x93e86208 in -[NSToolbarItem _copyOfCustomView] >> #15 0x93e862c0 in -[NSToolbarItem copyWithZone:] >> #16 0x0005235c in -[SKMainWindowController >> toolbar:itemForItemIdentifier:willBeInsertedIntoToolbar:] at >> SKMainWindowController.m:5067 >> #17 0x93a81750 in -[NSToolbar >> _newItemFromDelegateWithItemIdentifier:willBeInsertedIntoToolbar:] >> #18 0x93a7eb6c in -[NSToolbar >> _newItemFromItemIdentifier:requireImmediateLoad:willBeInsertedIntoTool >> bar:] >> #19 0x93e8115c in -[NSToolbar >> toolbar:itemForItemIdentifier:willBeInsertedIntoToolbar:] >> #20 0x93a81750 in -[NSToolbar >> _newItemFromDelegateWithItemIdentifier:willBeInsertedIntoToolbar:] >> #21 0x93a7eb6c in -[NSToolbar >> _newItemFromItemIdentifier:requireImmediateLoad:willBeInsertedIntoTool >> bar:] >> #22 0x93a7e97c in -[NSToolbar >> _insertNewItemWithItemIdentifier:atIndex:notifyDelegate:notifyView:not >> ifyFamilyAndUpdateDefaults:] >> #23 0x93a80338 in -[NSToolbar >> _setCurrentItemsToItemIdentifiers:notifyDelegate:notifyView:notifyFami >> lyAndUpdateDefaults:] >> #24 0x93e7f57c in -[NSToolbar >> _loadInitialItemIdentifiers:requireImmediateLoad:] >> #25 0x93e84428 in -[NSToolbarConfigPanel _loadData] >> #26 0x93e83ed8 in -[NSToolbarConfigPanel initForToolbar:withWidth:] >> #27 0x93e7fa98 in -[NSToolbar _runCustomizationPanel] >> #28 0x939d635c in -[NSApplication sendAction:to:from:] >> #29 0x93a71994 in -[NSMenu performActionForItemAtIndex:] >> #30 0x93a716c4 in -[NSCarbonMenuImpl >> performActionWithHighlightingForItemAtIndex:] >> >> >> >> ---------------------------------------------------------------------- >> --- >> SF.Net email is sponsored by: The Future of Linux Business White Paper >> from Novell. From the desktop to the data center, Linux is going >> mainstream. Let it simplify your IT future. >> http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 >> _______________________________________________ >> skim-app-develop mailing list >> skim-app-develop@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/skim-app-develop > > >------------------------------------------------------------------------- >SF.Net email is sponsored by: The Future of Linux Business White Paper >from Novell. From the desktop to the data center, Linux is going >mainstream. Let it simplify your IT future. >http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 >_______________________________________________ >skim-app-develop mailing list >skim-app-develop@lists.sourceforge.net >https://lists.sourceforge.net/lists/listinfo/skim-app-develop > > ------------------------------------------------------------------------- SF.Net email is sponsored by: The Future of Linux Business White Paper from Novell. From the desktop to the data center, Linux is going mainstream. Let it simplify your IT future. http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 _______________________________________________ skim-app-develop mailing list skim-app-develop@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/skim-app-develop