- more size_t / off_t fixes (Ralf)
authorpmatilai <pmatilai>
Tue, 20 Jun 2006 15:46:49 +0000 (15:46 +0000)
committerpmatilai <pmatilai>
Tue, 20 Jun 2006 15:46:49 +0000 (15:46 +0000)
apt-pkg/contrib/mmap.cc
apt-pkg/contrib/mmap.h
apt-pkg/tagfile.cc
apt-pkg/tagfile.h
cmdline/acqprogress.cc

index 23477e2..d5b6d2d 100644 (file)
@@ -25,7 +25,6 @@
 #pragma implementation "apt-pkg/mmap.h"
 #endif 
 
-#define _BSD_SOURCE
 #include <apt-pkg/mmap.h>
 #include <apt-pkg/error.h>
 
@@ -84,7 +83,7 @@ bool MMap::Map(FileFd &Fd)
    // Map it.
    Base = mmap(0,iSize,Prot,Map,Fd.Fd(),0);
    if (Base == (void *)-1)
-      return _error->Errno("mmap",_("Couldn't make mmap of %lu bytes"),iSize);
+      return _error->Errno("mmap",_("Couldn't make mmap of %lu bytes"),(unsigned long) iSize);
 
    return true;
 }
@@ -146,13 +145,13 @@ bool MMap::Sync(size_t Start,size_t Stop)
 // DynamicMMap::DynamicMMap - Constructor                              /*{{{*/
 // ---------------------------------------------------------------------
 /* */
-DynamicMMap::DynamicMMap(FileFd &F,unsigned long Flags,unsigned long WorkSpace) : 
+DynamicMMap::DynamicMMap(FileFd &F,unsigned long Flags,size_t WorkSpace) : 
              MMap(F,Flags | NoImmMap), Fd(&F), WorkSpace(WorkSpace)
 {
    if (_error->PendingError() == true)
       return;
    
-   unsigned long EndOfFile = Fd->Size();
+   size_t EndOfFile = Fd->Size();
    if (EndOfFile > WorkSpace)
       WorkSpace = EndOfFile;
    else
@@ -169,7 +168,7 @@ DynamicMMap::DynamicMMap(FileFd &F,unsigned long Flags,unsigned long WorkSpace)
 // DynamicMMap::DynamicMMap - Constructor for a non-file backed map    /*{{{*/
 // ---------------------------------------------------------------------
 /* This is just a fancy malloc really.. */
-DynamicMMap::DynamicMMap(unsigned long Flags,unsigned long WorkSpace) :
+DynamicMMap::DynamicMMap(unsigned long Flags,size_t WorkSpace) :
              MMap(Flags | NoImmMap | UnMapped), Fd(0), WorkSpace(WorkSpace)
 {
    if (_error->PendingError() == true)
@@ -204,9 +203,9 @@ DynamicMMap::~DynamicMMap()
 // DynamicMMap::RawAllocate - Allocate a raw chunk of unaligned space  /*{{{*/
 // ---------------------------------------------------------------------
 /* This allocates a block of memory aligned to the given size */
-unsigned long DynamicMMap::RawAllocate(unsigned long Size,unsigned long Aln)
+size_t DynamicMMap::RawAllocate(size_t Size,unsigned long Aln)
 {
-   unsigned long Result = iSize;
+   size_t Result = iSize;
    if (Aln != 0)
       Result += Aln - (iSize%Aln);
    
index d1c0560..f9a3cb4 100644 (file)
@@ -44,7 +44,7 @@ class MMap
    protected:
    
    unsigned long Flags;
-   unsigned long iSize;
+   size_t iSize;
    void *Base;
 
    bool Map(FileFd &Fd);
@@ -58,7 +58,7 @@ class MMap
    // Simple accessors
    inline operator void *() {return Base;};
    inline void *Data() {return Base;}; 
-   inline unsigned long Size() {return iSize;};
+   inline size_t Size() {return iSize;};
    
    // File manipulators
    bool Sync();
@@ -84,21 +84,21 @@ class DynamicMMap : public MMap
    protected:
    
    FileFd *Fd;
-   unsigned long WorkSpace;
+   size_t WorkSpace;
    Pool *Pools;
    unsigned int PoolCount;
    
    public:
 
    // Allocation
-   unsigned long RawAllocate(unsigned long Size,unsigned long Aln = 0);
+   size_t RawAllocate(size_t Size,unsigned long Aln = 0);
    unsigned long Allocate(unsigned long ItemSize);
    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);
-   DynamicMMap(unsigned long Flags,unsigned long WorkSpace = 2*1024*1024);
+   DynamicMMap(FileFd &F,unsigned long Flags,size_t WorkSpace = 2*1024*1024);
+   DynamicMMap(unsigned long Flags,size_t WorkSpace = 2*1024*1024);
    virtual ~DynamicMMap();
 };
 
index f0bc8ec..40ae774 100644 (file)
@@ -31,7 +31,7 @@ using std::string;
 // TagFile::pkgTagFile - Constructor                                   /*{{{*/
 // ---------------------------------------------------------------------
 /* */
-pkgTagFile::pkgTagFile(FileFd *pFd,unsigned long Size) : Fd(*pFd), Size(Size)
+pkgTagFile::pkgTagFile(FileFd *pFd,size_t Size) : Fd(*pFd), Size(Size)
 {
    if (Fd.IsOpen() == false)
    {
@@ -126,7 +126,7 @@ bool pkgTagFile::Fill()
 // ---------------------------------------------------------------------
 /* This jumps to a pre-recorded file location and reads the record
    that is there */
-bool pkgTagFile::Jump(pkgTagSection &Tag,unsigned long Offset)
+bool pkgTagFile::Jump(pkgTagSection &Tag,off_t Offset)
 {
    // We are within a buffer space of the next hit..
    if (Offset >= iOffset && iOffset + (End - Start) > Offset)
@@ -226,7 +226,7 @@ void pkgTagSection::Trim()
 /* This searches the section for a tag that matches the given string. */
 bool pkgTagSection::Find(const char *Tag,unsigned &Pos) const
 {
-   unsigned int Length = strlen(Tag);
+   size_t Length = strlen(Tag);
    unsigned int I = AlphaIndexes[AlphaHash(Tag)];
    if (I == 0)
       return false;
index e9c797e..e691f3a 100644 (file)
@@ -73,7 +73,7 @@ class pkgTagFile
    char *Start;
    char *End;
    bool Done;
-   unsigned long iOffset;
+   off_t iOffset;
    unsigned long Size;
    
    bool Fill();
@@ -81,10 +81,10 @@ class pkgTagFile
    public:
 
    bool Step(pkgTagSection &Section);
-   inline unsigned long Offset() {return iOffset;};
-   bool Jump(pkgTagSection &Tag,unsigned long Offset);
+   inline off_t Offset() {return iOffset;};
+   bool Jump(pkgTagSection &Tag,off_t Offset);
 
-   pkgTagFile(FileFd *F,unsigned long Size = 32*1024);
+   pkgTagFile(FileFd *F,size_t Size = 32*1024);
    ~pkgTagFile();
 };
 
index 8c7264a..fa6ad7f 100644 (file)
@@ -332,8 +332,7 @@ bool AcqTextStatus::Authenticate(string Desc,string &User,string &Pass)
    // Turn echo back on
    tcsetattr(STDIN_FILENO, TCSANOW, &TermIO);
 
-   ioprintf(cout,"\n");
-   cout << flush;
+   cout << endl;
    
    Update = true;
    return true;