Changeset 25568

Timestamp:

11/12/2008 02:58:39 AM (2 months ago)

Author:

catfish_man

Message:

So, the theory I have as to why this assertion is being hit is that we aren't resorting in all situations we need to. This results in invisible people being mixed in with visible ones. I'm attempting to catch it happening by resorting in response to failure and trying again... we'll see if it turns up anything interesting.

Files:

• trunk/Frameworks/Adium Framework/Source/AIListGroup.m (modified) (1 diff)

trunk/Frameworks/Adium Framework/Source/AIListGroup.m

@@ -95 +95 @@
-{
-        AIListObject *obj = [self.containedObjects objectAtIndex:index];
-
+ {
-                AILog(@"Attempted to get visible object at index %i of %@, but %@ is not visible. With contained objects %@, visibility count is %i", index, self, obj, self.containedObjects, self.visibleCount);
+                [self sort];
+                AIListObject *obj = [self.containedObjects objectAtIndex:index];
+                if(!obj.visible)
+                        AILog(@"Failed to correct for messed up visibleObjectAtIndex by sorting");
+                else
+                        AILog(@"Successfully corrected for messed up visibleObjectAtIndex by sorting");
+        }
+
-
-        return obj;