In SSYDooDoUndoManager, removed the kludgey methods to 'couple' to document and managed object context, because now thanks to BSManagedDocument commit aa76e79, this can be done without kludging, and it seems to have fixed some edge case hangs.

Author: jerrykrinock

SSYDooDooUndoManager.h

@@ -99,45 +99,6 @@ extern NSString* const SSYUndoManagerDocumentWillSaveNotification ;
 	NSManagedObjectContext* m_managedObjectContext ;
 }
 
-/*!
- @brief    Gets a newSSYDooDooUndoManager instance and assigns it
- to a given document and its managed object context.
-
- @details  The document is required for three reasons:
- * The returned SSYDooDooUndoManager instance is set as its undo manager.
- * The returned SSYDooDooUndoManager instance is set as its 
- managed object context's undo manager.
- * The returned SSYDooDooUndoManager begins observing it for
- SSYUndoManagerDocumentWillSaveNotification.
- * A weak reference to the document's undo manager is kept
- in order to -processPendingChanges before ending an undo group.
- 
- Although the document could possibly be obtained by iterating
- through the shared NSDocumentController's documents, that will fail
- if you invoke this method during document initialization, before it
- has been added to NSDocumentController, which you usually do.
- 
- Note that NSDocument creates an NSUndoManager and assigns it during
- its initialization.  Thus, the document you pass will probably already
- have an undo manager.  That's OK, though.  We just set our own
- instead, and the original NSUndoManager instance is discarded.
- 
- We access the managed object context of the given document by
- sending -managedObjectContext to it.  If the document has never
- has never before had its managed object context accessed in this
- way, this will cause one to be created.  Thus, if your document
- did not have a managed object context prior to being passed to 
- this method, it will have one after this method executes.
- It needs one sooner or later anyhow.
- 
- @param    document  The document to be associated with the new
- SSYDooDooUndoManager instance.
- @result   The new SSYDooDooUndoManager instance.  Typically, you
- don't need this.  You can always get it later by sending -undoManager
- to your document.
-*/
-+ (SSYDooDooUndoManager*)makeUndoManagerForDocument:(NSDocument*)document ;
-
 /*!
  @brief    Begins an undo grouping which is highly resistant to
  being screwed up by Core Data and closes itself automatically.
@@ -189,18 +150,6 @@ extern NSString* const SSYUndoManagerDocumentWillSaveNotification ;
 */
 - (void)endUndoGrouping ;
 
-/*!
- @brief    Performs operations necessary to make the receiver act as the undo
- manager for a given document
- @details  You typically send this after initializing the receiver, if the
- receiver is to be the undo manager for a document.  This method does not
- associate a document's managed object context, if any. For Core Data documents,
- you must also -coupleToManagedObjectContext:.
- */
-- (void)coupleToDocument:(NSDocument*)document ;
-
-- (void)coupleToManagedObjectContext:(NSManagedObjectContext*)managedObjectContext ;
-
 @end
 
 

SSYDooDooUndoManager.m

@@ -35,50 +35,12 @@ - (NSInteger)nLivingManualGroups {
 
 @synthesize managedObjectContext = m_managedObjectContext ;
 
-- (void)coupleToManagedObjectContext:(NSManagedObjectContext*)managedObjectContext {
-    [managedObjectContext setUndoManager:(id)self] ;
-    [self setManagedObjectContext:managedObjectContext] ;
-}
-
-
-- (void)coupleToDocument:(NSDocument*)document {
-    //	[self setGroupsByEvent:NO] ;  // Causes all hell to break loose with Core Data.
-    
-    /* The following line has no effect if document defines its
-     -setUndoManager method to be a noop.  Check with your document class :)
-
-     We cast to an id since the compiler expects these to methods
-     to get something which inherits from NSUndoManager, which
-     GCUndoManager does not. */
-    [document setUndoManager:(id)self] ; // Causes moc to be created, which causes persisten store to be created ??
-    
-    [[NSNotificationCenter defaultCenter] addObserver:self
-                                             selector:@selector(endAnyUndoGroupings:)
-                                                 name:SSYUndoManagerDocumentWillSaveNotification
-                                               object:document] ;
-    [[NSNotificationCenter defaultCenter] addObserver:self
-                                             selector:@selector(endAnyUndoGroupings:)
-                                                 name:SSYUndoManagerDocumentWillCloseNotification
-                                               object:document] ;
-}
-
 - (void)dealloc {
 	[[NSNotificationCenter defaultCenter] removeObserver:self] ;
 
 	[super dealloc] ;
 }
 
-+ (SSYDooDooUndoManager*)makeUndoManagerForDocument:(BSManagedDocument*)document {
-	SSYDooDooUndoManager* undoManager = [[SSYDooDooUndoManager alloc] init] ;
-    /* The following line has no effect if document defines its
-     -setUndoManager method to be a noop.  Check with your document class :)
-. */
-    [undoManager coupleToDocument:document] ;
-    [undoManager coupleToManagedObjectContext:[document managedObjectContext]] ;
-    [undoManager autorelease] ;
-    return undoManager ;
-}
-
 - (void)beginManualUndoGrouping {
 	if ([self isUndoRegistrationEnabled]) {
 		nLivingManualGroups++ ;