-Sorry for the whitespaces

-Delete endDate which is used by queryByExample

Eilers can you look if a QDate::isValid makes more sense?

1 files changed, 11 insertions, 9 deletions

diff --git a/libopie2/opiepim/backend/ocontactaccessbackend_sql.cpp b/libopie2/opiepim/backend/ocontactaccessbackend_sql.cpp
index 221e977..3d284f7 100644
--- a/libopie2/opiepim/backend/ocontactaccessbackend_sql.cpp
+++ b/libopie2/opiepim/backend/ocontactaccessbackend_sql.cpp
@@ -304,9 +304,9 @@ namespace {
 		    return single();
 	    else
 		    return multi();
     }
-        
+
     QString FindQuery::multi()const {
 	    QString qu = "select * from addressbook where";
 	    for (uint i = 0; i < m_uids.count(); i++ ) {
 		    qu += " uid = " + QString::number( m_uids[i] ) + " OR";
@@ -319,9 +319,9 @@ namespace {
 
     QString FindQuery::single()const{
 	    QString qu = "select *";
 	    qu += " from addressbook where uid = " + QString::number(m_uid);
-	    
+
 	    // owarn << "find query: " << qu << "" << oendl;
 	    return qu;
     }
 
@@ -491,12 +491,12 @@ OPimContact OPimContactAccessBackend_SQL::find( int uid, const QArray<int>& quer
     odebug << "searching for " << uid << "" << oendl;
 
     QTime t;
     t.start();
-	
+
     uint numReadAhead = readAhead();
     QArray<int> searchList( numReadAhead );
- 
+
     uint size =0;
 
     // Build an array with all elements which should be requested and cached
     // We will just request "numReadAhead" elements, starting from "current" position in
@@ -623,8 +623,10 @@ QArray<int> OPimContactAccessBackend_SQL::queryByExample ( const OPimContact &qu
                         + queryStr + "'" + ")";
 
             }
         }
+
+        delete endDate;
     }
     // Skip trailing "AND"
 //  if ( isAnyFieldSelected )
 //      qu = qu.left( qu.length() - 4 );
@@ -824,24 +826,24 @@ QMap<int, QString>  OPimContactAccessBackend_SQL::requestNonCustom( int uid ) co
     QTime t3;
     t3.start();
     nonCustomMap = fillNonCustomMap( resItem );
     t3needed = t3.elapsed();
-    
+
 
     // odebug << "Adding UID: " << resItem.data( "uid" ) << "" << oendl;
     odebug << "RequestNonCustom needed: insg.:" << t.elapsed() << " ms, query: " << t2needed
            << " ms, mapping: " << t3needed << " ms" << oendl;
 
     return nonCustomMap;
 }
 
-/* Returns contact requested by uid and fills cache with contacts requested by uids in the cachelist */ 
+/* Returns contact requested by uid and fills cache with contacts requested by uids in the cachelist */
 OPimContact OPimContactAccessBackend_SQL::requestContactsAndCache( int uid, const QArray<int>& uidlist )const
 {
 	// We want to get all contacts with one query.
 	// We don't have to add the given uid to the uidlist, it is expected to be there already (see opimrecordlist.h).
 	// All contacts will be stored in the cache, afterwards the contact with the user id "uid" will be returned
-	// by using the cache..  
+	// by using the cache..
 	QArray<int> cachelist = uidlist;
 	OPimContact retContact;
 
 	odebug << "Reqest and cache" << cachelist.size() << "elements !" << oendl;
@@ -870,14 +872,14 @@ OPimContact OPimContactAccessBackend_SQL::requestContactsAndCache( int uid, cons
 			retContact = contact;
 		resItem = res_noncustom.next();
 	} while ( ! res_noncustom.atEnd() ); //atEnd() is true if we are past(!) the list !!
 	t3needed = t3.elapsed();
-    
+
 
 	// odebug << "Adding UID: " << resItem.data( "uid" ) << "" << oendl;
 	odebug << "RequestContactsAndCache needed: insg.:" << t.elapsed() << " ms, query: " << t2needed
 	       << " ms, mapping: " << t3needed << " ms" << oendl;
-	
+
 	return retContact;
 }
 
 QMap<int, QString> OPimContactAccessBackend_SQL::fillNonCustomMap( const OSQLResultItem& resultItem ) const