Monotone-Parent: b63f19817044870bee75b439abf67bc6b7e8ceca

Monotone-Revision: b4655ce05ca4a96336269d9d01dd4866945c8841

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2008-07-04T18:23:11
Monotone-Branch: ca.inverse.sogo

SOPE/NGCards/CardGroup.m

@@ -24,7 +24,6 @@
 #import <Foundation/NSDictionary.h>
 #import <Foundation/NSEnumerator.h>
 #import <Foundation/NSString.h>
-
 #import <SaxObjC/SaxXMLReader.h>
 #import <SaxObjC/SaxXMLReaderFactory.h>
 
@@ -70,9 +69,7 @@ static NGCardsSaxHandler *sax = nil;
 + (NSArray *) parseFromSource: (id) source
 {
   static id <NSObject,SaxXMLReader> cardParser;
-  NSMutableArray *cardGroups;
-  NSEnumerator *cards;
-  CardGroup *currentCard;
+  NSArray *cardGroups;
 
   cardGroups = nil;
 
@@ -83,18 +80,9 @@ static NGCardsSaxHandler *sax = nil;
 
       if (parser)
 	{
-	  cardGroups = [NSMutableArray new];
-	  [cardGroups autorelease];
-	  
 	  [parser parseFromSource: source];
-	  cards = [[sax cards] objectEnumerator];
-	  
-	  currentCard = [cards nextObject];
-	  while (currentCard)
-	    {
-	      [cardGroups addObject: currentCard];
-	      currentCard = [cards nextObject];
-	    }
+
+	  cardGroups = [sax cards];
 	}
     }
 

SOPE/NGCards/ChangeLog

@@ -1,3 +1,8 @@
+2008-07-04  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
+
+	* CardGroup.m ([CardGroup +parseFromSource:source]): directly
+	return the result from [sax cards] instead of creating a new array.
+
 2008-07-03  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
 
 	* iCalEvent.m ([iCalEvent -endDate]): if no DTEND is specified,