Better fix for reading feed Packages files.

1 files changed, 24 insertions, 32 deletions

diff --git a/noncore/settings/aqpkg/server.cpp b/noncore/settings/aqpkg/server.cpp
index 64a9c26..9a239a5 100644
--- a/noncore/settings/aqpkg/server.cpp
+++ b/noncore/settings/aqpkg/server.cpp
@@ -10,35 +10,30 @@
  ***************************************************************************/
 
 /***************************************************************************
  *                                                                         *
  *   This program is free software; you can redistribute it and/or modify  *
  *   it under the terms of the GNU General Public License as published by  *
  *   the Free Software Foundation; either version 2 of the License, or     *
  *   (at your option) any later version.                                   *
  *                                                                         *
  ***************************************************************************/
 
 
-#include <stdio.h>
-#include <string.h>
-#include <stdlib.h>
-
-#include <iostream>
-#include <fstream>
-using namespace std;
-
 #include "server.h"
 #include "datamgr.h"
 
+#include <qfile.h>
+#include <qtextstream.h>
+
 #ifdef QWS
 #include <qpe/global.h>
 #include <qpe/applnk.h>
 #include <qlist.h>
 #endif
 
 #include "utils.h"
 
 #include "global.h"
 
 Server :: Server( const char *name, const char *url )
 {
@@ -64,40 +59,41 @@ void Server :: readStatusFile(  QList<Destination> &destList )
     cleanUp();
 
     Destination *dest;
     QListIterator<Destination> dit( destList );
     bool rootRead = false;
     for ( ; dit.current(); ++dit )
     {
         dest = dit.current();
         bool installingToRoot = false;
         
         QString path = dest->getDestinationPath();
         if ( path.right( 1 ) != "/" )
-            path += "/";
+            path.append( "/" );
             
         if ( path == "/" )
         {
             rootRead = true;
             installingToRoot = true;
         }   
         
-        packageFile = path + "usr/lib/ipkg/status";
+        packageFile = path;
+        packageFile.append( "usr/lib/ipkg/status" );
         readPackageFile( 0, false, installingToRoot, &( *dest ) );
     }
 
     // Ensure that the root status file is read
     if ( !rootRead )
     {
-        cout << "Reading status file " << "/usr/lib/ipkg/status" << endl;
+        //cout << "Reading status file " << "/usr/lib/ipkg/status" << endl;
         packageFile = "/usr/lib/ipkg/status";
         readPackageFile( 0, false, true );
     }
 }
 
 void Server :: readLocalIpks( Server *local )
 {
     cleanUp();
 
 #ifdef QWS
     // First, get any local IPKGs in the documents area
     // Only applicable to Qtopie/Opie
@@ -137,54 +133,50 @@ void Server :: readLocalIpks( Server *local )
                 tmp = file.mid( p+1, p2-(p+1) );
                 packageList.back().setVersion( tmp );
                 packageList.back().setPackageStoredLocally( true );
         }
 #endif
 
     // build local packages
     buildLocalPackages( local );
 }
 
 void Server :: readPackageFile( Server *local, bool clearAll, bool installingToRoot, Destination *dest )
 {
-    ifstream in( packageFile );
-        if ( !in.is_open() )
+    QFile f( packageFile );
+    if ( !f.open( IO_ReadOnly ) )
         return;
+    QTextStream t( &f );
 
-    char line[5001];
-    char k[21];
-    char v[5001];
+    QString line;
     QString key;
     QString value;
+    int pos;
 
     if ( clearAll )
         cleanUp();
     Package *currPackage = 0;
 
     bool newPackage = true;
-    do
+    while ( !t.eof() )
     {
-        in.getline( line, 5000 );
-        if ( in.eof() )
-            continue;
-
-        k[0] = '\0';
-        v[0] = '\0';
-
-        sscanf( line, "%[^:]: %[^\n]", k, v );
+        line = t.readLine();
         
-        key = k;
-        value = v;
-        key = key.stripWhiteSpace();
-        value = value.stripWhiteSpace();
+        pos = line.find( ':', 0 );
+        if ( pos > -1 )
+            key = line.mid( 0, pos ).stripWhiteSpace();
+        else
+            key = QString::null;
+        value = line.mid( pos+1, line.length()-pos ).stripWhiteSpace();
+
         if ( key == "Package" && newPackage )
         {
             newPackage = false;
 
             currPackage = getPackage( value );
             if ( !currPackage )
             {
                 Package *package = new Package( value );
                 packageList.append( package );
                 currPackage = package;
                 currPackage->setInstalledTo( dest );
                 if ( installingToRoot )
@@ -219,31 +211,31 @@ void Server :: readPackageFile( Server *local, bool clearAll, bool installingToR
         else if ( key == "Size" )
         {
             if ( currPackage )
                 currPackage->setPackageSize( value );
         }
         else if ( key == "Section" )
         {
             if ( currPackage )
                 currPackage->setSection( value );
 
             DataManager::setAvailableCategories( value );
         }
-        else if ( key == "" )
+        else if ( key == QString::null )
         {
             newPackage = true;
         }
-    } while ( !in.eof() );
+    }
 
-    in.close();
+    f.close();
 
     // build local packages
     buildLocalPackages( local );
 }
 
 void Server :: buildLocalPackages( Server *local )
 {
     Package *curr;
     QListIterator<Package> it( packageList );
     for ( ; it.current(); ++it )
     {
         curr = it.current();