adium 2338:bc3569e7ac9e: Okay, this assert won't work; we need t...

commits at adium.im commits at adium.im
Tue May 26 03:01:25 UTC 2009


details:	http://hg.adium.im/adium/rev/bc3569e7ac9e
revision:	2338:bc3569e7ac9e
author:		Zachary West <zacw at adium.im>
date:		Mon May 25 23:01:17 2009 -0400

Okay, this assert won't work; we need to set index *as* we move, but it's not an atomic operation. Blahhhhhh.

diffstat:

 Frameworks/Adium Framework/Source/AIListObject.m |  3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diffs (13 lines):

diff -r 7d40f9f80e9e -r bc3569e7ac9e Frameworks/Adium Framework/Source/AIListObject.m
--- a/Frameworks/Adium Framework/Source/AIListObject.m	Mon May 25 22:30:29 2009 -0400
+++ b/Frameworks/Adium Framework/Source/AIListObject.m	Mon May 25 23:01:17 2009 -0400
@@ -207,7 +207,8 @@
 {
 	AIListObject<AIContainingObject> *container = (AIListObject<AIContainingObject> *)self;
 	
-	NSAssert([container.containedObjects containsObject:listObject], @"Asked to set an index for an object which doesn't exist.");
+	// We can't enforce this, since we're asked to set it for objects we don't yet *officially* contain.
+	//NSAssert([container.containedObjects containsObject:listObject], @"Asked to set an index for an object which doesn't exist.");
 	
 	if (index == 0) {
 		//Moved to the top of a group.  New index is between 0 and the lowest current index




More information about the commits mailing list