[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdepim/kpilot/conduits/notepadconduit
From:       Adriaan de Groot <groot () kde ! org>
Date:       2005-04-22 20:17:19
Message-ID: 20050422201719.A6BFC496 () office ! kde ! org
[Download RAW message or body]

CVS commit by adridg: 

Check for zero notes in the notepad and don't enter the for loop
if there are none -- possibly also triggers a bug in the idList()
function, already committed.

BUG: 104357


  M +15 -11    notepad-conduit.cc   1.14


--- kdepim/kpilot/conduits/notepadconduit/notepad-conduit.cc  #1.13:1.14
@@ -133,5 +133,8 @@ NotepadActionThread::NotepadActionThread
         PilotSerialDatabase *db = new PilotSerialDatabase(fPilotSocket, "npadDB");
 
-        // int n = db->recordCount();
+        int n = db->recordCount();
+
+        if ( n > 0 )
+        {
         QValueList<recordid_t> vl = db->idList();
         QValueList<recordid_t>::iterator it;
@@ -144,4 +147,5 @@ NotepadActionThread::NotepadActionThread
                 }
         }
+        }
     KPILOT_DELETE(db);
         QApplication::postEvent(fParent, new QEvent(QEvent::User));


[prev in list] [next in list] [prev in thread] [next in thread]