- more size_t fixes (Ralf)
authorpmatilai <pmatilai>
Sat, 17 Jun 2006 14:48:41 +0000 (14:48 +0000)
committerpmatilai <pmatilai>
Sat, 17 Jun 2006 14:48:41 +0000 (14:48 +0000)
apt-pkg/contrib/mmap.cc
apt-pkg/contrib/mmap.h
apt-pkg/tagfile.cc
cmdline/apt-cache.cc
methods/ftp.cc
methods/ftp.h
methods/rsh.cc
methods/rsh.h

index d0e584a..23477e2 100644 (file)
@@ -270,10 +270,10 @@ unsigned long DynamicMMap::Allocate(unsigned long ItemSize)
 // DynamicMMap::WriteString - Write a string to the file               /*{{{*/
 // ---------------------------------------------------------------------
 /* Strings are not aligned to anything */
 // DynamicMMap::WriteString - Write a string to the file               /*{{{*/
 // ---------------------------------------------------------------------
 /* Strings are not aligned to anything */
-unsigned long DynamicMMap::WriteString(const char *String,
-                                      unsigned long Len)
+size_t DynamicMMap::WriteString(const char *String,
+                                      size_t Len)
 {
 {
-   unsigned long Result = iSize;
+   size_t Result = iSize;
    // Just in case error check
    if (Result + Len > WorkSpace)
    {
    // Just in case error check
    if (Result + Len > WorkSpace)
    {
@@ -281,7 +281,7 @@ unsigned long DynamicMMap::WriteString(const char *String,
       return 0;
    }   
    
       return 0;
    }   
    
-   if (Len == (unsigned long)-1)
+   if (Len == (size_t)-1)
       Len = strlen(String);
    iSize += Len + 1;
    memcpy((char *)Base + Result,String,Len);
       Len = strlen(String);
    iSize += Len + 1;
    memcpy((char *)Base + Result,String,Len);
index 0e785d3..d1c0560 100644 (file)
@@ -93,8 +93,8 @@ class DynamicMMap : public MMap
    // Allocation
    unsigned long RawAllocate(unsigned long Size,unsigned long Aln = 0);
    unsigned long Allocate(unsigned long ItemSize);
    // Allocation
    unsigned long RawAllocate(unsigned long Size,unsigned long Aln = 0);
    unsigned long Allocate(unsigned long ItemSize);
-   unsigned long WriteString(const char *String,unsigned long Len = (unsigned long)-1);
-   inline unsigned long WriteString(string S) {return WriteString(S.c_str(),S.length());};
+   size_t WriteString(const char *String,size_t Len = (size_t)-1);
+   inline size_t WriteString(string S) {return WriteString(S.c_str(),S.length());};
    void UsePools(Pool &P,unsigned int Count) {Pools = &P; PoolCount = Count;};
    
    DynamicMMap(FileFd &F,unsigned long Flags,unsigned long WorkSpace = 2*1024*1024);
    void UsePools(Pool &P,unsigned int Count) {Pools = &P; PoolCount = Count;};
    
    DynamicMMap(FileFd &F,unsigned long Flags,unsigned long WorkSpace = 2*1024*1024);
index 306f9f9..f0bc8ec 100644 (file)
@@ -84,7 +84,7 @@ bool pkgTagFile::Step(pkgTagSection &Tag)
    then fills the rest from the file */
 bool pkgTagFile::Fill()
 {
    then fills the rest from the file */
 bool pkgTagFile::Fill()
 {
-   unsigned long EndSize = End - Start;
+   size_t EndSize = End - Start;
    unsigned long Actual = 0;
    
    memmove(Buffer,Start,EndSize);
    unsigned long Actual = 0;
    
    memmove(Buffer,Start,EndSize);
index 01fdd43..abd2350 100644 (file)
@@ -308,7 +308,7 @@ bool Stats(CommandLine &Cmd)
       SizeToStr(Cache.Head().ProvidesCount*Cache.Head().ProvidesSz) << ')' << endl;
    
    // String list stats
       SizeToStr(Cache.Head().ProvidesCount*Cache.Head().ProvidesSz) << ')' << endl;
    
    // String list stats
-   unsigned long Size = 0;
+   size_t Size = 0;
    unsigned long Count = 0;
    for (pkgCache::StringItem *I = Cache.StringItemP + Cache.Head().StringList;
         I!= Cache.StringItemP; I = Cache.StringItemP + I->NextItem)
    unsigned long Count = 0;
    for (pkgCache::StringItem *I = Cache.StringItemP + Cache.Head().StringList;
         I!= Cache.StringItemP; I = Cache.StringItemP + I->NextItem)
@@ -318,7 +318,7 @@ bool Stats(CommandLine &Cmd)
    }
    cout << _("Total Globbed Strings: ") << Count << " (" << SizeToStr(Size) << ')' << endl;
 
    }
    cout << _("Total Globbed Strings: ") << Count << " (" << SizeToStr(Size) << ')' << endl;
 
-   unsigned long DepVerSize = 0;
+   size_t DepVerSize = 0;
    for (pkgCache::PkgIterator P = Cache.PkgBegin(); P.end() == false; P++)
    {
       for (pkgCache::VerIterator V = P.VersionList(); V.end() == false; V++)
    for (pkgCache::PkgIterator P = Cache.PkgBegin(); P.end() == false; P++)
    {
       for (pkgCache::VerIterator V = P.VersionList(); V.end() == false; V++)
@@ -1672,7 +1672,7 @@ bool Search(CommandLine &CmdL)
            cout << string(Start,End-Start) << endl;
         }       
         else
            cout << string(Start,End-Start) << endl;
         }       
         else
-           printf("%s - %s\n",P.Name().c_str(),P.ShortDesc().c_str());
+           cout << P.Name() << " - " << P.ShortDesc() << endl;
       }
    }
    
       }
    }
    
index 95afe54..98c235a 100644 (file)
@@ -309,7 +309,7 @@ bool FTPConn::ReadLine(string &Text)
    while (Len < sizeof(Buffer))
    {
       // Scan the buffer for a new line
    while (Len < sizeof(Buffer))
    {
       // Scan the buffer for a new line
-      for (unsigned int I = 0; I != Len; I++)
+      for (size_t I = 0; I != Len; I++)
       {
         // Escape some special chars
         if (Buffer[I] == 0)
       {
         // Escape some special chars
         if (Buffer[I] == 0)
@@ -625,7 +625,7 @@ bool FTPConn::ExtGoPasv()
 // FTPConn::Size - Return the size of a file                           /*{{{*/
 // ---------------------------------------------------------------------
 /* Grab the file size from the server, 0 means no size or empty file */
 // FTPConn::Size - Return the size of a file                           /*{{{*/
 // ---------------------------------------------------------------------
 /* Grab the file size from the server, 0 means no size or empty file */
-bool FTPConn::Size(const char *Path,unsigned long &Size)
+bool FTPConn::Size(const char *Path,size_t &Size)
 {
    // Query the size
    unsigned int Tag;
 {
    // Query the size
    unsigned int Tag;
@@ -701,7 +701,7 @@ bool FTPConn::CreateDataFd()
       if (WaitFd(DataFd,true,TimeOut) == false)
         return _error->Error(_("Could not connect data socket, connection timed out"));
       unsigned int Err;
       if (WaitFd(DataFd,true,TimeOut) == false)
         return _error->Error(_("Could not connect data socket, connection timed out"));
       unsigned int Err;
-      unsigned int Len = sizeof(Err);
+      socklen_t Len = sizeof(Err);
       if (getsockopt(DataFd,SOL_SOCKET,SO_ERROR,&Err,&Len) != 0)
         return _error->Errno("getsockopt",_("Failed"));
       if (Err != 0)
       if (getsockopt(DataFd,SOL_SOCKET,SO_ERROR,&Err,&Len) != 0)
         return _error->Errno("getsockopt",_("Failed"));
       if (Err != 0)
@@ -999,7 +999,7 @@ bool FtpMethod::Fetch(FetchItem *Itm)
    
    // Get the files information
    Status(_("Query"));
    
    // Get the files information
    Status(_("Query"));
-   unsigned long Size;
+   size_t Size;
    if (Server->Size(File,Size) == false ||
        Server->ModTime(File,FailTime) == false)
    {
    if (Server->Size(File,Size) == false ||
        Server->ModTime(File,FailTime) == false)
    {
index 94cb2fb..b062da1 100644 (file)
@@ -13,7 +13,7 @@
 class FTPConn
 {
    char Buffer[1024*10];
 class FTPConn
 {
    char Buffer[1024*10];
-   unsigned long Len;
+   size_t Len;
    int ServerFd;
    int DataFd;
    int DataListenFd;
    int ServerFd;
    int DataFd;
    int DataListenFd;
@@ -53,7 +53,7 @@ class FTPConn
    bool ExtGoPasv();
    
    // Query
    bool ExtGoPasv();
    
    // Query
-   bool Size(const char *Path,unsigned long &Size);
+   bool Size(const char *Path,size_t &Size);
    bool ModTime(const char *Path, time_t &Time);
    bool Get(const char *Path,FileFd &To,unsigned long Resume,
            Hashes &MD5,bool &Missing);
    bool ModTime(const char *Path, time_t &Time);
    bool Get(const char *Path,FileFd &To,unsigned long Resume,
            Hashes &MD5,bool &Missing);
index ad2e414..97ba6c3 100644 (file)
@@ -162,7 +162,7 @@ bool RSHConn::ReadLine(string &Text)
    while (Len < sizeof(Buffer))
    {
       // Scan the buffer for a new line
    while (Len < sizeof(Buffer))
    {
       // Scan the buffer for a new line
-      for (unsigned int I = 0; I != Len; I++)
+      for (size_t I = 0; I != Len; I++)
       {
          // Escape some special chars
          if (Buffer[I] == 0)
       {
          // Escape some special chars
          if (Buffer[I] == 0)
@@ -187,7 +187,7 @@ bool RSHConn::ReadLine(string &Text)
       }
 
       // Suck it back
       }
 
       // Suck it back
-      int Res = read(ReadFd,Buffer + Len,sizeof(Buffer) - Len);
+      ssize_t Res = read(ReadFd,Buffer + Len,sizeof(Buffer) - Len);
       if (Res <= 0)
       {
          _error->Errno("read",_("Read error"));
       if (Res <= 0)
       {
          _error->Errno("read",_("Read error"));
@@ -218,7 +218,7 @@ bool RSHConn::WriteMsg(string &Text,bool Sync,const char *Fmt,...)
       strcat(S," 2> /dev/null\n");
 
    // Send it off
       strcat(S," 2> /dev/null\n");
 
    // Send it off
-   unsigned long Len = strlen(S);
+   size_t Len = strlen(S);
    unsigned long Start = 0;
    while (Len != 0)
    {
    unsigned long Start = 0;
    while (Len != 0)
    {
@@ -229,7 +229,7 @@ bool RSHConn::WriteMsg(string &Text,bool Sync,const char *Fmt,...)
         return _error->Error(_("Connection timeout"));
       }      
       
         return _error->Error(_("Connection timeout"));
       }      
       
-      int Res = write(WriteFd,S + Start,Len);
+      ssize_t Res = write(WriteFd,S + Start,Len);
       if (Res <= 0)
       {
          _error->Errno("write",_("Write Error"));
       if (Res <= 0)
       {
          _error->Errno("write",_("Write Error"));
@@ -250,7 +250,7 @@ bool RSHConn::WriteMsg(string &Text,bool Sync,const char *Fmt,...)
 // ---------------------------------------------------------------------
 /* Right now for successfull transfer the file size must be known in 
    advance. */
 // ---------------------------------------------------------------------
 /* Right now for successfull transfer the file size must be known in 
    advance. */
-bool RSHConn::Size(const char *Path,unsigned long &Size)
+bool RSHConn::Size(const char *Path,size_t &Size)
 {
    // Query the size
    string Msg;
 {
    // Query the size
    string Msg;
@@ -326,7 +326,7 @@ bool RSHConn::Get(const char *Path,FileFd &To,unsigned long Resume,
       }
 
       // Read the data..
       }
 
       // Read the data..
-      int Res = read(ReadFd,Buffer,sizeof(Buffer));
+      ssize_t Res = read(ReadFd,Buffer,sizeof(Buffer));
       if (Res == 0)
       {
         Close();
       if (Res == 0)
       {
         Close();
@@ -428,7 +428,7 @@ bool RSHMethod::Fetch(FetchItem *Itm)
    Status(_("Connecting to %s"), Get.Host.c_str());
 
    // Get the files information
    Status(_("Connecting to %s"), Get.Host.c_str());
 
    // Get the files information
-   unsigned long Size;
+   size_t Size;
    if (Server->Size(File,Size) == false ||
        Server->ModTime(File,FailTime) == false)
    {
    if (Server->Size(File,Size) == false ||
        Server->ModTime(File,FailTime) == false)
    {
index 461dbbc..f062bb1 100644 (file)
@@ -41,7 +41,7 @@ class RSHConn
    void Close();
 
    // Query
    void Close();
 
    // Query
-   bool Size(const char *Path,unsigned long &Size);
+   bool Size(const char *Path,size_t &Size);
    bool ModTime(const char *Path, time_t &Time);
    bool Get(const char *Path,FileFd &To,unsigned long Resume,
             Hashes &Hash,bool &Missing, unsigned long Size);
    bool ModTime(const char *Path, time_t &Time);
    bool Get(const char *Path,FileFd &To,unsigned long Resume,
             Hashes &Hash,bool &Missing, unsigned long Size);