Skip to content

Commit 4f9f249

Browse files
committed
Just changed comments and warnings
1 parent 760447e commit 4f9f249

File tree

1 file changed

+22
-0
lines changed

1 file changed

+22
-0
lines changed

SSYSearchField.m

+22
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,28 @@ bug because of the objc_msgSend_corrupt_cache_error().
8080
#13 0x00000001002feea0 in -[CntntViewController search:] at /Users/jk/Documents/Programming/Projects/BkmkMgrs/CntntViewController.m:351
8181
#14 0x0000000100110b24 in -[BkmxDocWinCon search:] at /Users/jk/Documents/Programming/Projects/BkmkMgrs/BkmxDocWinCon.m:2907
8282
#15 0x000000010023b2cd in -[SSYToolbarButton doDaClick:] at /Users/jk/Documents/Programming/ClassesObjC/SSYToolbarButton.m:115
83+
84+
85+
86+
On 2014 Nov 07, at 16:02, Greg Parker <[email protected]> wrote:
87+
88+
You may have better luck tracing it from the other side. Run to that line in appendToRecentSearches:, set a breakpoint on -[NSNotificationCenter postNotificationName:object:userInfo:], and step over your line. At those breakpoints you should be able to see the notification parameters in the parameter registers, assuming the notification is sent every time that line runs.
89+
90+
Great idea, Greg. It worked.
91+
92+
It told me that that setting recent searches in this menu posts 122 notifications, 120 of which are due to building the search field’s popup menu.
93+
94+
• 40 NSMenuDidAddItemNotification, one for each item in the search field’s popup menu
95+
• 20 NSMenuDidChangeItemNotification, which are mixed in with the above, for items that changed.
96+
• 1 NSUserDefaultsDidChangeNotification.
97+
• 1 NSAutosavedRecentsChangedNotification
98+
• 40 NSMenuDidAddItemNotification, same as before.
99+
• 20 NSMenuDidChangeItemNotification, same as before
100+
101+
However, nowhere in *my* code do I directly create an observer of any of those four notification names. So I’m still head-scratching.
102+
103+
If anyone has any idea what disappearing objects in AppKit might be observing these notifications, let us know.
104+
83105
*/
84106
}
85107

0 commit comments

Comments
 (0)