Revision: 2533
          http://skim-app.svn.sourceforge.net/skim-app/?rev=2533&view=rev
Author:   hofman
Date:     2007-07-19 06:33:06 -0700 (Thu, 19 Jul 2007)

Log Message:
-----------
No need to retain the left/rightSideView as they are now top-level objects in 
the nib and therefore implicitly retained by the nib's owner.

Modified Paths:
--------------
    trunk/SKMainWindowController.m

Modified: trunk/SKMainWindowController.m
===================================================================
--- trunk/SKMainWindowController.m      2007-07-19 13:22:31 UTC (rev 2532)
+++ trunk/SKMainWindowController.m      2007-07-19 13:33:06 UTC (rev 2533)
@@ -256,14 +256,10 @@
     [[rightSideButton cell] setToolTip:NSLocalizedString(@"View Notes", @"Tool 
tip message") forSegment:SKNoteSidePaneState];
     [[rightSideButton cell] setToolTip:NSLocalizedString(@"View Snapshots", 
@"Tool tip message") forSegment:SKSnapshotSidePaneState];
     
-    [leftSideContentView retain];
     [leftSideContentView setFrame:[leftSideContentBox bounds]];
     [leftSideContentBox addSubview:leftSideContentView];
-    [leftSideContentView release];
-    [rightSideContentView retain];
     [rightSideContentView setFrame:[rightSideContentBox bounds]];
     [rightSideContentBox addSubview:rightSideContentView];
-    [rightSideContentView release];
     
     [pdfView setFrame:[[pdfContentBox contentView] bounds]];
     
@@ -1899,10 +1895,8 @@
         
         if ([[leftSideWindow firstResponder] 
isDescendantOf:leftSideContentView])
             [leftSideWindow makeFirstResponder:nil];
-        [leftSideContentView retain]; // leftSideContentView is removed from 
its old superview in the process
         [leftSideContentView setFrame:[leftSideContentBox bounds]];
         [leftSideContentBox addSubview:leftSideContentView];
-        [leftSideContentView release];
         
         [leftSideEdgeView setEdges:BDSKMinXEdgeMask | BDSKMaxXEdgeMask];
         [findEdgeView setEdges:BDSKMinXEdgeMask | BDSKMaxXEdgeMask];
@@ -1922,10 +1916,8 @@
         
         if ([[rightSideWindow firstResponder] 
isDescendantOf:rightSideContentView])
             [rightSideWindow makeFirstResponder:nil];
-        [rightSideContentView retain]; // rightSideContentView is removed from 
its old superview in the process
         [rightSideContentView setFrame:[rightSideContentBox bounds]];
         [rightSideContentBox addSubview:rightSideContentView];
-        [rightSideContentView release];
         
         [rightSideEdgeView setEdges:BDSKMinXEdgeMask | BDSKMaxXEdgeMask];
         


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Skim-app-commit mailing list
Skim-app-commit@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/skim-app-commit

Reply via email to