Changeset 19865

Show
Ignore:
Timestamp:
03/17/07 12:33:34 (2 years ago)
Author:
patrick
Message:

Fixed two memory leaks.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • juggler/trunk/modules/gadgeteer/gadget/Devices/KeyboardMouseDevice/InputWindowCocoa.mm

    r19859 r19865  
    257257      // Set the delegate for mCocoaWindow. In particular, we do this so that 
    258258      // we can be told if the window gets closed by some external entity. 
    259       InputWindowDelegate* delegate = 
    260          [[InputWindowDelegate alloc] initWithWindow:this]; 
     259      id delegate = 
     260         [[[InputWindowDelegate alloc] initWithWindow:this] autorelease]; 
    261261      [mCocoaWindow setDelegate:delegate]; 
    262262 
    …… 
    266266                                              inputArea:this]; 
    267267 
    268       // Ownership of mMainView gets transferred to mCocoaWindow. 
     268      // mCocoaWindow retains a refernece to mMainView, meaning that the 
     269      // reference count on it is now two. We release our reference to 
     270      // mMainView later when we are done with it. 
    269271      [mCocoaWindow setContentView:mMainView]; 
    270272      [mCocoaWindow setInitialFirstResponder:mMainView]; 
    …… 
    347349   } 
    348350 
     351   // We are done with mMainView. 
     352   [mMainView release]; 
     353 
    349354   mMainView    = nil; 
    350355   mCocoaWindow = nil;