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

Reply via email to