adium 3056:3066b0ccb207: Retaining this string here makes next t...

commits at adium.im commits at adium.im
Tue Dec 22 19:10:44 UTC 2009


details:	http://hg.adium.im/adium/rev/3066b0ccb207
revision:	3056:3066b0ccb207
author:		Stephen Holt <sholt at adium.im>
date:		Tue Dec 22 14:07:15 2009 -0500

Retaining this string here makes next to no sense.  Removing the retain fixes a leak of a NSString object on every MUC user count update.
(transplanted from 5a2622ab6a4a7eefafe22819b829e600d623652f)

diffs (12 lines):

diff -r 750a88e761fb -r 3066b0ccb207 Source/KNShelfSplitView.m
--- a/Source/KNShelfSplitView.m	Sat Dec 12 13:32:00 2009 -0500
+++ b/Source/KNShelfSplitView.m	Tue Dec 22 14:07:15 2009 -0500
@@ -720,7 +720,7 @@
 				nil];
 			
 			stringHeight = [NSAttributedString stringHeightForAttributes:attributes];
-			attributedStringValue = [[NSAttributedString alloc] initWithString:[inString retain]
+			attributedStringValue = [[NSAttributedString alloc] initWithString:inString
 																	attributes:attributes];
 		} else {
 			attributedStringValue = nil;




More information about the commits mailing list