harvester/server/src/harvesterqueue.cpp
branchRCL_3
changeset 8 6752808b2036
parent 3 b73a2e62868f
child 15 3cebc1a84278
--- a/harvester/server/src/harvesterqueue.cpp	Tue Feb 02 00:24:33 2010 +0200
+++ b/harvester/server/src/harvesterqueue.cpp	Fri Feb 19 23:14:48 2010 +0200
@@ -115,7 +115,8 @@
         item = iItemQueue[0];
         iItemQueue.Remove( 0 );	
         }
-   	else
+   	   
+   	if( iItemQueue.Count() == 0 )
         {
         WRITELOG( "Harvester queue items zero!" );
         iItemQueue.Compress();
@@ -134,7 +135,7 @@
 void CHarvesterQueue::Append( CHarvesterData* aItem )
 	{
     WRITELOG( "CHarvesterQueue::Append()" );
-    TInt err = KErrNone;
+    TInt err( KErrNone );
 
     if ( iBlacklist )
         {
@@ -152,24 +153,20 @@
             }
         }
 
-    if ( err == KErrNone )
-        {
-		// check if fast harvest file and add to start of queue
-    	if ( aItem->ObjectType() == EFastHarvest || aItem->Origin() == MdeConstants::Object::ECamera )
-    		{
-    		err = iItemQueue.Insert( aItem, 0 );
-    		}
-    	else
-    		{
-    		err = iItemQueue.Append( aItem );
-    		}
+	// check if fast harvest file and add to start of queue
+    if ( aItem->ObjectType() == EFastHarvest || aItem->Origin() == MdeConstants::Object::ECamera )
+    	{
+    	err = iItemQueue.Insert( aItem, 0 );
+    	}
+    else
+    	{
+    	err = iItemQueue.Append( aItem );
+    	}
     	
-    	if( err != KErrNone )
-			{
-			delete aItem;
-			aItem = NULL;
-			return;
-			}
+    if( err != KErrNone )
+		{
+		delete aItem;
+		aItem = NULL;
         }
     }