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
[email protected]
https://lists.sourceforge.net/lists/listinfo/skim-app-commit
