- Huge pile of cleanups for compiler warnings mostly from Philipp Thomas and
authorpmatilai <pmatilai>
Mon, 27 Mar 2006 21:29:46 +0000 (21:29 +0000)
committerpmatilai <pmatilai>
Mon, 27 Mar 2006 21:29:46 +0000 (21:29 +0000)
  Ralf Corsepius
- Add missing Size() implementation for repomd index

24 files changed:
apt-pkg/acquire-item.cc
apt-pkg/clean.h
apt-pkg/contrib/hashes.cc
apt-pkg/contrib/md5.cc
apt-pkg/contrib/sha1.cc
apt-pkg/indexfile.h
apt-pkg/luaiface.cc
apt-pkg/luaiface.h
apt-pkg/pkgrecords.cc
apt-pkg/pkgsystem.h
apt-pkg/repository.h
apt-pkg/rpm/rpmhandler.cc
apt-pkg/rpm/rpmindexfile.cc
apt-pkg/rpm/rpmindexfile.h
apt-pkg/rpm/rpmlistparser.cc
apt-pkg/rpm/rpmpackagedata.cc
apt-pkg/rpm/rpmpm.cc
apt-pkg/rpm/rpmrecords.cc
apt-pkg/rpm/rpmsrcrecords.cc
cmdline/apt-get.cc
lua/llimits.h
lua/local/linit.c
methods/ftp.cc
methods/gpg.cc

index 33f8b8b..b0beec4 100644 (file)
@@ -59,7 +59,7 @@ bool VerifyChecksums(string File,unsigned long Size,string MD5, string method)
    // LORG:2006-03-09
    // XXX hack alert: repomd doesn't have index sizes so ignore it and
    // rely on checksum
-   if (Size > 0 && Buf.st_size != Size)
+   if (Size > 0 && (unsigned long)Buf.st_size != Size)
    {
       if (_config->FindB("Acquire::Verbose", false) == true)
         cout << "Size of "<<File<<" did not match what's in the checksum list and was redownloaded."<<endl;
@@ -309,7 +309,7 @@ void pkgAcqIndex::Done(string Message,unsigned long Size,string MD5,
            ErrorText = _("Size mismatch");
            Rename(DestFile,DestFile + ".FAILED");
            if (_config->FindB("Acquire::Verbose",false) == true) 
-              _error->Warning("Size mismatch of index file %s: %ul was supposed to be %ul",
+              _error->Warning("Size mismatch of index file %s: %lu was supposed to be %lu",
                               RealURI.c_str(), Size, FSize);
            return;
         }
@@ -592,7 +592,7 @@ void pkgAcqIndexRel::Done(string Message,unsigned long Size,string MD5,
         ErrorText = _("Size mismatch");
         Rename(DestFile,DestFile + ".FAILED");
         if (_config->FindB("Acquire::Verbose",false) == true) 
-           _error->Warning("Size mismatch of index file %s: %ul was supposed to be %ul",
+           _error->Warning("Size mismatch of index file %s: %lu was supposed to be %lu",
                            RealURI.c_str(), Size, FSize);
         return;
       }
index 3aa8194..e576120 100644 (file)
@@ -21,6 +21,7 @@ class pkgArchiveCleaner
    protected:
    
    virtual void Erase(const char * /*File*/,string /*Pkg*/,string /*Ver*/,struct stat & /*St*/) {};
+   virtual ~pkgArchiveCleaner() {};
 
    public:   
    
index 6a11c42..6aca183 100644 (file)
@@ -30,8 +30,8 @@ bool Hashes::AddFD(int Fd,unsigned long Size)
    int Res = 0;
    while (Size != 0)
    {
-      Res = read(Fd,Buf,MIN(Size,sizeof(Buf)));
-      if (Res < 0 || (unsigned)Res != MIN(Size,sizeof(Buf)))
+      Res = read(Fd,Buf,std::min(Size,sizeof(Buf)));
+      if (Res < 0 || (unsigned)Res != std::min(Size,sizeof(Buf)))
         return false;
       Size -= Res;
       MD5.Add(Buf,Res);
index 029e336..d753546 100644 (file)
@@ -299,8 +299,8 @@ bool MD5Summation::AddFD(int Fd,unsigned long Size)
    int Res = 0;
    while (Size != 0)
    {
-      Res = read(Fd,Buf,MIN(Size,sizeof(Buf)));
-      if (Res < 0 || (unsigned)Res != MIN(Size,sizeof(Buf)))
+      Res = read(Fd,Buf,std::min(Size,sizeof(Buf)));
+      if (Res < 0 || (unsigned)Res != std::min(Size,sizeof(Buf)))
         return false;
       Size -= Res;
       Add(Buf,Res);
index 7934a47..ca445f2 100644 (file)
@@ -345,8 +345,8 @@ bool SHA1Summation::AddFD(int Fd,unsigned long Size)
    int Res = 0;
    while (Size != 0)
    {
-      Res = read(Fd,Buf,MIN(Size,sizeof(Buf)));
-      if (Res < 0 || (unsigned) Res != MIN(Size,sizeof(Buf)))
+      Res = read(Fd,Buf,std::min(Size,sizeof(Buf)));
+      if (Res < 0 || (unsigned) Res != std::min(Size,sizeof(Buf)))
         return false;
       Size -= Res;
       Add(Buf,Res);
index 4892ef1..9d0322c 100644 (file)
@@ -51,6 +51,7 @@ class pkgIndexFile
       const char *Label;
 
       virtual pkgRecords::Parser *CreatePkgParser(pkgCache::PkgFileIterator /*File*/) const {return 0;};
+      virtual ~Type() {};
       Type();
    };
 
index 28d3c17..451fbc6 100644 (file)
@@ -378,7 +378,7 @@ void Lua::SetGlobal(const char *Name, vector<const char *> &Value,
 {
    lua_pushstring(L, Name);
    lua_newtable(L);
-   if (Total == -1 || Total > Value.size())
+   if (Total == -1 || (size_t)Total > Value.size())
       Total = Value.size();
    for (int i=0; i != Total && Value[i] != NULL; i++) {
       lua_pushstring(L, Value[i]);
@@ -393,7 +393,7 @@ void Lua::SetGlobal(const char *Name, vector<string> &Value,
 {
    lua_pushstring(L, Name);
    lua_newtable(L);
-   if (Total == -1 || Total > Value.size())
+   if (Total == -1 || (size_t)Total > Value.size())
       Total = Value.size();
    for (int i=0; i != Total; i++) {
       lua_pushstring(L, Value[i].c_str());
@@ -408,7 +408,7 @@ void Lua::SetGlobal(const char *Name, vector<pkgCache::Package*> &Value,
 {
    lua_pushstring(L, Name);
    lua_newtable(L);
-   if (Total == -1 || Total > Value.size())
+   if (Total == -1 || (size_t)Total > Value.size())
       Total = Value.size();
    for (int i=0; i != Total && Value[i] != NULL; i++) {
       pushudata(pkgCache::Package*, Value[i]);
@@ -1161,21 +1161,11 @@ static int AptLua_markremove(lua_State *L)
    return AptAux_mark(L, MARK_REMOVE);
 }
 
-static int AptLua_marksimplekeep(lua_State *L)
-{
-   return AptAux_marksimple(L, MARK_KEEP);
-}
-
 static int AptLua_marksimpleinstall(lua_State *L)
 {
    return AptAux_marksimple(L, MARK_INSTALL);
 }
 
-static int AptLua_marksimpleremove(lua_State *L)
-{
-   return AptAux_marksimple(L, MARK_REMOVE);
-}
-
 static int AptLua_markdistupgrade(lua_State *L)
 {
    if (lua_gettop(L) != 0) {
index ef783b3..f0cf2c5 100644 (file)
@@ -55,7 +55,7 @@ class Lua {
         { RunInteractive(PlaceHint.c_str()); };
 
    bool RunScript(const string &Script, const string &ChunkCacheKey="")
-        { RunScript(Script.c_str(), (ChunkCacheKey.length() == 0) ?
+        { return RunScript(Script.c_str(), (ChunkCacheKey.length() == 0) ?
                                      NULL : ChunkCacheKey.c_str()); };
    bool RunScript(const char *Script, const char *ChunkCacheKey=NULL);
    bool RunScripts(const char *ConfListKey, bool CacheChunks=false);
index 105fad9..62f94e2 100644 (file)
@@ -57,7 +57,7 @@ pkgRecords::~pkgRecords()
 {
    // CNC:2002-11-28
    // See comments above.
-   for (unsigned I = 0; I != PackageFileCount; I++)
+   for (unsigned I = 0; I != (unsigned)PackageFileCount; I++)
       delete Files[I];
    delete [] Files;
 }
index 59dbd2d..a296e69 100644 (file)
@@ -90,7 +90,7 @@ class pkgSystem
    // Return a list of system index files..
    virtual bool AddStatusFiles(std::vector<pkgIndexFile *> &List) = 0;   
    // CNC:2003-11-21
-   virtual bool AddSourceFiles(std::vector<pkgIndexFile *> &List) {};
+   virtual bool AddSourceFiles(std::vector<pkgIndexFile *> &List) {return true;};
    virtual bool FindIndex(pkgCache::PkgFileIterator File,
                          pkgIndexFile *&Found) const = 0;
    
index 48a5843..9f27ba9 100644 (file)
@@ -60,6 +60,7 @@ class pkgRepository
       CheckMethod = "MD5-Hash";
    };
 
+   virtual ~pkgRepository() {};
 };
 
 #endif
index f925684..0e5f02f 100644 (file)
@@ -58,7 +58,6 @@ string RPMHandler::Epoch()
 
 unsigned long RPMHandler::GetITag(rpmTag Tag)
 {
-   char *str;
    int_32 count, type, *num;
    assert(HeaderP != NULL);
    int rc = headerGetEntry(HeaderP, Tag,
@@ -711,7 +710,7 @@ string RPMDirHandler::MD5Sum()
 
 
 RPMDBHandler::RPMDBHandler(bool WriteLock)
-   : WriteLock(WriteLock), Handler(0)
+   : Handler(0), WriteLock(WriteLock)
 {
 #if RPM_VERSION >= 0x040000
    RpmIter = NULL;
@@ -1121,7 +1120,6 @@ bool RPMRepomdHandler::HasFile(const char *File)
    if (*File == '\0')
       return false;
 
-   bool inprimary = false;
    bool found = false;
 
    xmlNode *format = FindNode("format");
@@ -1247,7 +1245,6 @@ bool RPMRepomdHandler::Provides(vector<Dependency*> &Provs)
 {
    xmlNode *format = FindNode("format");
    xmlNode *provides = FindNode(format, "provides");
-   bool ret = true;
 
    if (! provides)
       return true;
index d0da012..9002f3f 100644 (file)
@@ -536,7 +536,6 @@ string rpmSingleSrcIndex::ArchiveURI(string File) const
 
 string rpmRepomdIndex::ArchiveURI(string File) const
 {
-   RPMPackageData *rpmdata = RPMPackageData::Singleton();
    string Res;
 
    //cout << Dist << File << endl;
@@ -586,7 +585,6 @@ pkgCache::PkgFileIterator rpmRepomdIndex::FindInCache(pkgCache &Cache) const
 }
 string rpmRepomdIndex::ReleaseURI(string Type) const
 {
-   RPMPackageData *rpmdata = RPMPackageData::Singleton();
    string Res;
    Res = URI + Dist + "/repodata/" + "repomd.xml";
 
@@ -629,7 +627,6 @@ string rpmRepomdIndex::Info(string Type) const
 
 string rpmRepomdIndex::IndexURI(string Type) const
 {
-   RPMPackageData *rpmdata = RPMPackageData::Singleton();
    string Res = URI + Dist;
    if (Dist[Dist.size() - 1] != '/') {
         Res += "/";
@@ -694,6 +691,14 @@ string rpmRepomdIndex::ReleasePath() const
    return _config->FindDir("Dir::State::lists")+URItoFileName(ReleaseURI("release.xml"));
 }
 
+unsigned long rpmRepomdIndex::Size() const
+{
+   // ugh.. all this just to get the package count ;(
+   RPMHandler *Handler = CreateHandler();
+   unsigned long Res = Handler->Size();
+   delete Handler;
+   return Res;
+}
 
 bool rpmRepomdIndex::Merge(pkgCacheGenerator &Gen,OpProgress &Prog) const
 {
index 59c8ace..7e189c6 100644 (file)
@@ -308,7 +308,7 @@ class rpmRepomdIndex : public rpmIndexFile
 
    // Interface for the Cache Generator
    virtual bool Exists() const;
-   virtual unsigned long Size() const {};
+   virtual unsigned long Size() const;
 
    // Interface for acquire
    virtual string Describe(bool Short) const;
index 6ab4646..3704083 100644 (file)
@@ -83,7 +83,6 @@ string rpmListParser::Package()
 #endif
 
    string Name = Handler->Name();
-   int type, count;
    
    Duplicated = false;
    
@@ -184,9 +183,6 @@ string rpmListParser::Version()
 /* */
 bool rpmListParser::NewVersion(pkgCache::VerIterator Ver)
 {
-   int count, type;
-   int_32 *num;
-
 #if WITH_VERSION_CACHING
    // Cache it for future usage.
    RpmData->SetVersion(Handler->GetID(), Offset(), Ver);
index 0ce30fd..718a16b 100644 (file)
@@ -296,7 +296,6 @@ int RPMPackageData::RpmArchScore(const char *Arch)
 
 bool RPMPackageData::IsCompatArch(string Arch)
 {
-   bool compat = false;
    for (vector<string>::iterator I = CompatArch[BaseArch].begin();
         I != CompatArch[BaseArch].end(); I++) {
       if (Arch == *I)
index c5a13dc..5794c67 100644 (file)
@@ -392,13 +392,13 @@ pkgRPMExtPM::~pkgRPMExtPM()
 bool pkgRPMExtPM::ExecRPM(Item::RPMOps op, vector<const char*> &files)
 {
    const char *Args[10000];      
-   const char *operation;
+   const char *operation = NULL;
    unsigned int n = 0;
    bool Interactive = _config->FindB("RPM::Interactive",true);
    
    Args[n++] = _config->Find("Dir::Bin::rpm","rpm").c_str();
 
-   bool nodeps;
+   bool nodeps = false;
 
    switch (op)
    {
@@ -662,8 +662,7 @@ pkgRPMLibPM::~pkgRPMLibPM()
 
 bool pkgRPMLibPM::AddToTransaction(Item::RPMOps op, vector<const char*> &files)
 {
-   int debug = _config->FindB("Debug::pkgRPMPM", false);
-   int n = 0, rc, xx;
+   int rc;
    FD_t fd;
    rpmHeader hdr;
 
@@ -744,7 +743,6 @@ bool pkgRPMLibPM::Process(vector<const char*> &install,
    int rc = 0;
    bool Success = false;
    bool Interactive = _config->FindB("RPM::Interactive",true);
-   int debug = _config->FindB("Debug::pkgRPMPM", false);
    string Dir = _config->Find("RPM::RootDir");
    rpmReadConfigFiles(NULL, NULL);
 
index eb99b2d..49df2e1 100644 (file)
@@ -226,11 +226,6 @@ void rpmRecordParser::BufCatDescr(const char *descr)
 void rpmRecordParser::GetRec(const char *&Start,const char *&Stop) 
 {
    // FIXME: This method is leaking memory from headerGetEntry().
-   int type, type2, type3, count;
-   char *str;
-   char **strv;
-   char **strv2;
-   int_32 *numv;
    char buf[32];
 
    BufUsed = 0;
@@ -239,7 +234,7 @@ void rpmRecordParser::GetRec(const char *&Start,const char *&Stop)
 
    BufCatTag("\nSection: ", Handler->Group().c_str());
 
-   snprintf(buf, sizeof(buf), "%d", Handler->InstalledSize());
+   snprintf(buf, sizeof(buf), "%lu", Handler->InstalledSize());
    BufCatTag("\nInstalled Size: ", buf);
 
    BufCatTag("\nPackager: ", Handler->Packager().c_str());
@@ -325,7 +320,7 @@ void rpmRecordParser::GetRec(const char *&Start,const char *&Stop)
 
    BufCatTag("\nArchitecture: ", Handler->Arch().c_str());
    
-   snprintf(buf, sizeof(buf), "%d", Handler->FileSize());
+   snprintf(buf, sizeof(buf), "%lu", Handler->FileSize());
    BufCatTag("\nSize: ", buf);
 
    BufCatTag("\nMD5Sum: ", Handler->MD5Sum().c_str());
index 4a96ea5..f294f94 100644 (file)
@@ -251,11 +251,6 @@ void rpmSrcRecordParser::BufCatDescr(const char *descr)
 // -----------------------------------------------
 string rpmSrcRecordParser::AsStr() 
 {
-   int type, type2, type3, count;
-   char *str;
-   char **strv;
-   char **strv2;
-   int_32 *numv;
    char buf[32];
 
    BufUsed = 0;
@@ -264,7 +259,7 @@ string rpmSrcRecordParser::AsStr()
 
    BufCatTag("\nSection: ", Handler->Group().c_str());
 
-   snprintf(buf, sizeof(buf), "%d", Handler->InstalledSize());
+   snprintf(buf, sizeof(buf), "%lu", Handler->InstalledSize());
    BufCatTag("\nInstalled Size: ", buf);
 
    BufCatTag("\nPackager: ", Handler->Packager().c_str());
@@ -318,7 +313,7 @@ string rpmSrcRecordParser::AsStr()
 
    BufCatTag("\nArchitecture: ", Handler->Arch().c_str());
 
-   snprintf(buf, sizeof(buf), "%d", Handler->FileSize());
+   snprintf(buf, sizeof(buf), "%lu", Handler->FileSize());
    BufCatTag("\nSize: ", buf);
 
    BufCatTag("\nMD5Sum: ", Handler->MD5Sum().c_str());
index c0749e6..697c4ba 100644 (file)
@@ -1223,7 +1223,7 @@ bool TryToInstall(pkgCache::PkgIterator Pkg,pkgDepCache &Cache,
         // Check if it's a different version of a package already
         // considered as a good solution.
         bool AlreadySeen = false;
-        for (int i = 0; i != GoodSolutions.size(); i++)
+        for (size_t i = 0; i != GoodSolutions.size(); i++)
         {
            pkgCache::PkgIterator GoodPkg(Cache, GoodSolutions[i]);
            if (PrvPkg == GoodPkg)
@@ -1259,7 +1259,7 @@ bool TryToInstall(pkgCache::PkgIterator Pkg,pkgDepCache &Cache,
         }
       }
       vector<string> GoodSolutionNames;
-      for (int i = 0; i != GoodSolutions.size(); i++)
+      for (size_t i = 0; i != GoodSolutions.size(); i++)
       {
         pkgCache::PkgIterator GoodPkg(Cache, GoodSolutions[i]);
         GoodSolutionNames.push_back(GoodPkg.Name());
@@ -1307,7 +1307,7 @@ bool TryToInstall(pkgCache::PkgIterator Pkg,pkgDepCache &Cache,
       {
         ioprintf(c1out,_("Package %s is a virtual package provided by:\n"),
                  Pkg.Name());
-        for (int i = 0; i != GoodSolutions.size(); i++)
+        for (size_t i = 0; i != GoodSolutions.size(); i++)
         {
            pkgCache::PkgIterator GoodPkg(Cache, GoodSolutions[i]);
            if (GoodPkg.CurrentVer().end() == false)
@@ -1615,6 +1615,8 @@ class UpdateLogCleaner : public pkgArchiveCleaner
       c1out << "Del " << Pkg << " " << Ver << " [" << SizeToStr(St.st_size) << "B]" << endl;
       unlink(File);      
    };
+   public:
+   virtual ~UpdateLogCleaner() {};
 };
 
 bool DoUpdate(CommandLine &CmdL)
@@ -2374,6 +2376,8 @@ class LogCleaner : public pkgArchiveCleaner
       if (_config->FindB("APT::Get::Simulate") == false)
         unlink(File);      
    };
+   public:
+   virtual ~LogCleaner() {};
 };
 
 bool DoAutoClean(CommandLine &CmdL)
index 343c922..e4c1247 100644 (file)
@@ -69,7 +69,7 @@ typedef unsigned char lu_byte;
 ** this is for hashing only; there is no problem if the integer
 ** cannot hold the whole pointer value
 */
-#define IntPoint(p)  ((lu_hash)(p))
+#define IntPoint(p)  ((lu_hash)(unsigned long)(p))
 
 
 
index 79bfeac..0168a16 100644 (file)
@@ -4,4 +4,5 @@
 LUA_API int luaopen_init(lua_State *L)
 {
 #include "linit.lch"
+       return 0;
 }
index fda89cb..4351ac2 100644 (file)
@@ -1104,7 +1104,7 @@ int main(int argc,const char *argv[])
         
         // Run the http method
         string Path = flNotFile(argv[0]) + "http";
-        execl(Path.c_str(),Path.c_str(),0);
+        execl(Path.c_str(),Path.c_str(),NULL);
         cerr << _("Unable to invoke ") << Path << endl;
         exit(100);
       }      
index 932397f..9789c37 100644 (file)
@@ -255,7 +255,7 @@ char *getFileSigner(const char *file, const char *sigfile,
       {
         char *sig;
         ptr = sig = ptr1 + sizeof(SIGPACK);
-        while (isxdigit(*ptr) && (ptr-sig) < sizeof(keyid)) ptr++;
+        while (isxdigit(*ptr) && (size_t)(ptr-sig) < sizeof(keyid)) ptr++;
         *ptr = 0;
         strcpy(keyid, sig);
       }