adium-1.4 2574:28d5a4352944: Cherry-picked 34bbe221bb80 from adi...
commits at adium.im
commits at adium.im
Wed Aug 12 07:08:23 UTC 2009
details: http://hg.adium.im/adium-1.4/rev/28d5a4352944
revision: 2574:28d5a4352944
author: Peter Hosey <hg at boredzo.org>
date: Tue Aug 11 23:51:29 2009 -0700
Cherry-picked 34bbe221bb80 from adium: Fixed retaining delegate property, introduced in 1d5619b64c61.
Believe it or not, this caused an *under*-release crash when a FT finished with the progress window closed: The delegate is a progress row, which is normally owned solely by the progress window controller. With this retention keeping the row alive, Adium crashed when the row went to talk to the now-dead progress window controller.
diffs (12 lines):
diff -r 4c8b116ca530 -r 28d5a4352944 Frameworks/Adium Framework/Source/ESFileTransfer.h
--- a/Frameworks/Adium Framework/Source/ESFileTransfer.h Tue Aug 11 11:14:29 2009 -0400
+++ b/Frameworks/Adium Framework/Source/ESFileTransfer.h Tue Aug 11 23:51:29 2009 -0700
@@ -72,7 +72,7 @@
@property (readwrite, nonatomic, retain) id accountData;
- at property (readwrite, nonatomic, retain) id <FileTransferDelegate> delegate;
+ at property (readwrite, nonatomic, assign) id <FileTransferDelegate> delegate;
@property (readonly, nonatomic) BOOL isStopped;
More information about the commits
mailing list