- size_t related bug fixes, addressing to "unsigned vs. signed types" issues
authorpmatilai <pmatilai>
Mon, 12 Jun 2006 15:16:34 +0000 (15:16 +0000)
committerpmatilai <pmatilai>
Mon, 12 Jun 2006 15:16:34 +0000 (15:16 +0000)
  (Ralf)

apt-pkg/contrib/strutl.cc
apt-pkg/rpm/rpmlistparser.cc
apt-pkg/rpm/rpmrecords.cc

index a360821..83fa22d 100644 (file)
@@ -968,7 +968,7 @@ void ioprintf(ostream &out,const char *format,...)
 char *safe_snprintf(char *Buffer,char *End,const char *Format,...)
 {
    va_list args;
-   unsigned long Did;
+   int Did;
 
    va_start(args,Format);
 
index cf65184..2ce0429 100644 (file)
@@ -518,14 +518,14 @@ xmlNode *rpmRepomdParser::FindNode(xmlNode *n, const string Name)
 bool rpmRepomdParser::LoadReleaseInfo(pkgCache::PkgFileIterator FileI,
                                       const string File, const string Dist)
 {
-   size_t start, stop, size;
+   string::size_type start, stop, size;
    string comp;
 
    // Munge sources.list distribution into something that can be used 
    // as Component to allow repository pinning with repomd
    start = Dist.find_first_not_of("/");
    size = Dist.length();
-   while ((start >= 0) && (start < size)) {
+   while ((start != string::npos) && (start < size)) {
       stop = Dist.find_first_of("/", start);
       string part = Dist.substr(start, stop - start);
       if (comp.empty()) {
@@ -533,7 +533,7 @@ bool rpmRepomdParser::LoadReleaseInfo(pkgCache::PkgFileIterator FileI,
       } else {
         comp += "-" + part;
       }
-      if ((stop < 0) || (stop > size)) stop = size;
+      if ((stop == string::npos) || (stop > size)) stop = size;
       start = Dist.find_first_not_of("/", stop + 1);
    }
    FileI->Component = WriteUniqString(comp);
index 7bc9245..b1f9445 100644 (file)
@@ -141,7 +141,7 @@ string rpmRecordParser::SourcePkg()
    string::size_type idx1 = srpm.find(versarch);
 
 // not found
-   if ( idx1 < 0 )
+   if ( idx1 == string::npos )
      return "";
 
 // check if the first dot in "srpm" is the dot at the end of versarch