adium 5023:54c9e016b5ba: Forgot a [pool release]; here.

Thijs Alkemade thijsalkemade at gmail.com
Wed Sep 5 21:50:34 UTC 2012


Nope, that is 10.7+ only. :(

The default branch, so 1.7hg, does use @autoreleasepool instead of NSAutoreleasePool everywhere.

Thijs

On 5 sep. 2012, at 23:47, Peter Hosey <boredzo at adium.im> wrote:

> Can we use @autoreleasepool yet?
> 
> On Sep 5, 2012, at 14:10:36, commits at adium.im wrote:
>> details:	http://hg.adium.im/adium/rev/54c9e016b5ba
>> revision:	5023:54c9e016b5ba
>> branch:		adium-1.5.4
>> date:		Wed Sep 05 23:10:17 2012 +0200
>> 
>> Forgot a [pool release]; here.
>> 
>> diffs (15 lines):
>> 
>> diff -r 932796c0e3bf -r 54c9e016b5ba Plugins/Purple Service/AMPurpleJabberAdHocServer.m
>> --- a/Plugins/Purple Service/AMPurpleJabberAdHocServer.m	Wed Sep 05 22:57:49 2012 +0200
>> +++ b/Plugins/Purple Service/AMPurpleJabberAdHocServer.m	Wed Sep 05 23:10:17 2012 +0200
>> @@ -37,8 +37,10 @@
>>>> 		const char *type = xmlnode_get_attrib(*packet,"type");
>> -		if(!type || strcmp(type,"set"))
>> +		if(!type || strcmp(type,"set")) {
>> +			[pool release];
>> 			return; // doesn't talk to us, probably the user interacting with some other adhoc node
>> +		}
>>> 
> 





More information about the devel mailing list