Futures free their handlers as soon as they’re completed/cancelled.

robrix · Mar 22, 2012 · b05578a · b05578a
1 parent 59e8ea1
commit b05578a
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/RXFutures/RXFuture.m b/RXFutures/RXFuture.m
@@ -66,6 +66,12 @@ -(void)cancel {
 			for(void (^block)() in cancellationHandlers) {
 				[self dispatchCallback:block];
 			}
+#if !__has_feature(objc_arc)
+			[completionHandlers release];
+			[cancellationHandlers release];
+#endif
+			completionHandlers = nil;
+			cancellationHandlers = nil;
 		}
 	}];
 }
@@ -93,6 +99,12 @@ -(void)complete {
 			for(void (^block)() in completionHandlers) {
 				[self dispatchCallback:block];
 			}
+#if !__has_feature(objc_arc)
+			[completionHandlers release];
+			[cancellationHandlers release];
+#endif
+			completionHandlers = nil;
+			cancellationHandlers = nil;
 		}
 	}];
 }