make all d-pointer * const pointers

[apt.git] / apt-pkg / cacheiterators.h

diff --git a/apt-pkg/cacheiterators.h b/apt-pkg/cacheiterators.h
index dcd3531197798885f524c05c61f54666435e04e4..f3b107699e35c68a20a1f6f975a4fb6b33e88916 100644 (file)
--- a/apt-pkg/cacheiterators.h
+++ b/apt-pkg/cacheiterators.h
@@ -29,7 +29,12 @@
                                                                        /*}}}*/
 #ifndef PKGLIB_CACHEITERATORS_H
 #define PKGLIB_CACHEITERATORS_H
                                                                        /*}}}*/
 #ifndef PKGLIB_CACHEITERATORS_H
 #define PKGLIB_CACHEITERATORS_H

+#include<apt-pkg/pkgcache.h>
+#include<apt-pkg/macros.h>
+

 #include<iterator>
 #include<iterator>

+#include <iosfwd>
+#include <string>

 
 #include<string.h>
 
 
 #include<string.h>
 


@@ -46,7 +51,7 @@ template<typename Str, typename Itr> class pkgCache::Iterator :
         *  The implementation of this method should be pretty short
         *  as it will only return the Pointer into the mmap stored
         *  in the owner but the name of this pointer is different for
         *  The implementation of this method should be pretty short
         *  as it will only return the Pointer into the mmap stored
         *  in the owner but the name of this pointer is different for

-        *  each stucture and we want to abstract here at least for the
+        *  each structure and we want to abstract here at least for the

         *  basic methods from the actual structure.
         *  \return Pointer to the first structure of this type
         */
         *  basic methods from the actual structure.
         *  \return Pointer to the first structure of this type
         */


@@ -55,36 +60,36 @@ template<typename Str, typename Itr> class pkgCache::Iterator :
        public:
        // Iteration
        virtual void operator ++(int) = 0;
        public:
        // Iteration
        virtual void operator ++(int) = 0;

-       virtual void operator ++() = 0; // Should be {operator ++(0);};
-       inline bool end() const {return Owner == 0 || S == OwnerPointer();};
+       virtual void operator ++() = 0; // Should be {operator ++(0);}
+       inline bool end() const {return Owner == 0 || S == OwnerPointer();}

 
        // Comparison
 
        // Comparison

-       inline bool operator ==(const Itr &B) const {return S == B.S;};
-       inline bool operator !=(const Itr &B) const {return S != B.S;};
+       inline bool operator ==(const Itr &B) const {return S == B.S;}
+       inline bool operator !=(const Itr &B) const {return S != B.S;}

 
        // Accessors
 
        // Accessors

-       inline Str *operator ->() {return S;};
-       inline Str const *operator ->() const {return S;};
-       inline operator Str *() {return S == OwnerPointer() ? 0 : S;};
-       inline operator Str const *() const {return S == OwnerPointer() ? 0 : S;};
-       inline Str &operator *() {return *S;};
-       inline Str const &operator *() const {return *S;};
-       inline pkgCache *Cache() const {return Owner;};
+       inline Str *operator ->() {return S;}
+       inline Str const *operator ->() const {return S;}
+       inline operator Str *() {return S == OwnerPointer() ? 0 : S;}
+       inline operator Str const *() const {return S == OwnerPointer() ? 0 : S;}
+       inline Str &operator *() {return *S;}
+       inline Str const &operator *() const {return *S;}
+       inline pkgCache *Cache() const {return Owner;}

 
        // Mixed stuff
 
        // Mixed stuff

-       inline void operator =(const Itr &B) {S = B.S; Owner = B.Owner;};
-       inline bool IsGood() const { return S && Owner && ! end();};
-       inline unsigned long Index() const {return S - OwnerPointer();};
+       inline void operator =(const Itr &B) {S = B.S; Owner = B.Owner;}
+       inline bool IsGood() const { return S && Owner && ! end();}
+       inline unsigned long Index() const {return S - OwnerPointer();}

 
        void ReMap(void const * const oldMap, void const * const newMap) {
                if (Owner == 0 || S == 0)
                        return;
 
        void ReMap(void const * const oldMap, void const * const newMap) {
                if (Owner == 0 || S == 0)
                        return;

-               S += (Str*)(newMap) - (Str*)(oldMap);
+               S += (Str const * const)(newMap) - (Str const * const)(oldMap);

        }
 
        // Constructors - look out for the variable assigning
        }
 
        // Constructors - look out for the variable assigning

-       inline Iterator() : S(0), Owner(0) {};
-       inline Iterator(pkgCache &Owner,Str *T = 0) : S(T), Owner(&Owner) {};
+       inline Iterator() : S(0), Owner(0) {}
+       inline Iterator(pkgCache &Owner,Str *T = 0) : S(T), Owner(&Owner) {}

 };
                                                                        /*}}}*/
 // Group Iterator                                                      /*{{{*/
 };
                                                                        /*}}}*/
 // Group Iterator                                                      /*{{{*/


@@ -98,19 +103,19 @@ class pkgCache::GrpIterator: public Iterator<Group, GrpIterator> {
        protected:
        inline Group* OwnerPointer() const {
                return (Owner != 0) ? Owner->GrpP : 0;
        protected:
        inline Group* OwnerPointer() const {
                return (Owner != 0) ? Owner->GrpP : 0;

-       };
+       }

 
        public:
        // This constructor is the 'begin' constructor, never use it.
 
        public:
        // This constructor is the 'begin' constructor, never use it.

-       inline GrpIterator(pkgCache &Owner) : Iterator<Group, GrpIterator>(Owner), HashIndex(-1) {
+       explicit inline GrpIterator(pkgCache &Owner) : Iterator<Group, GrpIterator>(Owner), HashIndex(-1) {

                S = OwnerPointer();
                operator ++(0);
                S = OwnerPointer();
                operator ++(0);

-       };
+       }

 
        virtual void operator ++(int);
 
        virtual void operator ++(int);

-       virtual void operator ++() {operator ++(0);};
+       virtual void operator ++() {operator ++(0);}

 
 

-       inline const char *Name() const {return S->Name == 0?0:Owner->StrP + S->Name;};
+       inline const char *Name() const {return S->Name == 0?0:Owner->StrP + S->Name;}

        inline PkgIterator PackageList() const;
        PkgIterator FindPkg(std::string Arch = "any") const;
        /** \brief find the package with the "best" architecture
        inline PkgIterator PackageList() const;
        PkgIterator FindPkg(std::string Arch = "any") const;
        /** \brief find the package with the "best" architecture


@@ -127,8 +132,8 @@ class pkgCache::GrpIterator: public Iterator<Group, GrpIterator> {
        inline GrpIterator(pkgCache &Owner, Group *Trg) : Iterator<Group, GrpIterator>(Owner, Trg), HashIndex(0) {
                if (S == 0)
                        S = OwnerPointer();
        inline GrpIterator(pkgCache &Owner, Group *Trg) : Iterator<Group, GrpIterator>(Owner, Trg), HashIndex(0) {
                if (S == 0)
                        S = OwnerPointer();

-       };
-       inline GrpIterator() : Iterator<Group, GrpIterator>(), HashIndex(0) {};
+       }
+       inline GrpIterator() : Iterator<Group, GrpIterator>(), HashIndex(0) {}

 
 };
                                                                        /*}}}*/
 
 };
                                                                        /*}}}*/


@@ -139,35 +144,41 @@ class pkgCache::PkgIterator: public Iterator<Package, PkgIterator> {
        protected:
        inline Package* OwnerPointer() const {
                return (Owner != 0) ? Owner->PkgP : 0;
        protected:
        inline Package* OwnerPointer() const {
                return (Owner != 0) ? Owner->PkgP : 0;

-       };
+       }

 
        public:
        // This constructor is the 'begin' constructor, never use it.
 
        public:
        // This constructor is the 'begin' constructor, never use it.

-       inline PkgIterator(pkgCache &Owner) : Iterator<Package, PkgIterator>(Owner), HashIndex(-1) {
+       explicit inline PkgIterator(pkgCache &Owner) : Iterator<Package, PkgIterator>(Owner), HashIndex(-1) {

                S = OwnerPointer();
                operator ++(0);
                S = OwnerPointer();
                operator ++(0);

-       };
+       }

 
        virtual void operator ++(int);
 
        virtual void operator ++(int);

-       virtual void operator ++() {operator ++(0);};
+       virtual void operator ++() {operator ++(0);}

 
        enum OkState {NeedsNothing,NeedsUnpack,NeedsConfigure};
 
        // Accessors
 
        enum OkState {NeedsNothing,NeedsUnpack,NeedsConfigure};
 
        // Accessors

-       inline const char *Name() const {return S->Name == 0?0:Owner->StrP + S->Name;};
-       inline const char *Section() const {return S->Section == 0?0:Owner->StrP + S->Section;};
+       inline const char *Name() const { return Group().Name(); }
+       // Versions have sections - and packages can have different versions with different sections
+       // so this interface is broken by design. Run as fast as you can to Version.Section().
+       APT_DEPRECATED inline const char *Section() const {
+          APT_IGNORE_DEPRECATED_PUSH
+          return S->Section == 0?0:Owner->StrP + S->Section;
+          APT_IGNORE_DEPRECATED_POP
+       }

        inline bool Purge() const {return S->CurrentState == pkgCache::State::Purge ||
        inline bool Purge() const {return S->CurrentState == pkgCache::State::Purge ||

-               (S->CurrentVer == 0 && S->CurrentState == pkgCache::State::NotInstalled);};
-       inline const char *Arch() const {return S->Arch == 0?0:Owner->StrP + S->Arch;};
-       inline GrpIterator Group() const { return GrpIterator(*Owner, Owner->GrpP + S->Group);};
-
-       inline VerIterator VersionList() const;
-       inline VerIterator CurrentVer() const;
-       inline DepIterator RevDependsList() const;
-       inline PrvIterator ProvidesList() const;
-       OkState State() const;
-       const char *CandVersion() const;
-       const char *CurVersion() const;
+               (S->CurrentVer == 0 && S->CurrentState == pkgCache::State::NotInstalled);}
+       inline const char *Arch() const {return S->Arch == 0?0:Owner->StrP + S->Arch;}
+       inline APT_PURE GrpIterator Group() const { return GrpIterator(*Owner, Owner->GrpP + S->Group);}
+
+       inline VerIterator VersionList() const APT_PURE;
+       inline VerIterator CurrentVer() const APT_PURE;
+       inline DepIterator RevDependsList() const APT_PURE;
+       inline PrvIterator ProvidesList() const APT_PURE;
+       OkState State() const APT_PURE;
+       const char *CandVersion() const APT_PURE;
+       const char *CurVersion() const APT_PURE;

 
        //Nice printable representation
        friend std::ostream& operator <<(std::ostream& out, PkgIterator i);
 
        //Nice printable representation
        friend std::ostream& operator <<(std::ostream& out, PkgIterator i);


@@ -177,8 +188,8 @@ class pkgCache::PkgIterator: public Iterator<Package, PkgIterator> {
        inline PkgIterator(pkgCache &Owner,Package *Trg) : Iterator<Package, PkgIterator>(Owner, Trg), HashIndex(0) {
                if (S == 0)
                        S = OwnerPointer();
        inline PkgIterator(pkgCache &Owner,Package *Trg) : Iterator<Package, PkgIterator>(Owner, Trg), HashIndex(0) {
                if (S == 0)
                        S = OwnerPointer();

-       };
-       inline PkgIterator() : Iterator<Package, PkgIterator>(), HashIndex(0) {};
+       }
+       inline PkgIterator() : Iterator<Package, PkgIterator>(), HashIndex(0) {}

 };
                                                                        /*}}}*/
 // Version Iterator                                                    /*{{{*/
 };
                                                                        /*}}}*/
 // Version Iterator                                                    /*{{{*/


@@ -186,32 +197,40 @@ class pkgCache::VerIterator : public Iterator<Version, VerIterator> {
        protected:
        inline Version* OwnerPointer() const {
                return (Owner != 0) ? Owner->VerP : 0;
        protected:
        inline Version* OwnerPointer() const {
                return (Owner != 0) ? Owner->VerP : 0;

-       };
+       }

 
        public:
        // Iteration
 
        public:
        // Iteration

-       void operator ++(int) {if (S != Owner->VerP) S = Owner->VerP + S->NextVer;};
-       inline void operator ++() {operator ++(0);};
+       void operator ++(int) {if (S != Owner->VerP) S = Owner->VerP + S->NextVer;}
+       inline void operator ++() {operator ++(0);}

 
        // Comparison
        int CompareVer(const VerIterator &B) const;
        /** \brief compares two version and returns if they are similar
 
            This method should be used to identify if two pseudo versions are
 
        // Comparison
        int CompareVer(const VerIterator &B) const;
        /** \brief compares two version and returns if they are similar
 
            This method should be used to identify if two pseudo versions are

-           refering to the same "real" version */
+           referring to the same "real" version */

        inline bool SimilarVer(const VerIterator &B) const {
                return (B.end() == false && S->Hash == B->Hash && strcmp(VerStr(), B.VerStr()) == 0);
        inline bool SimilarVer(const VerIterator &B) const {
                return (B.end() == false && S->Hash == B->Hash && strcmp(VerStr(), B.VerStr()) == 0);

-       };
+       }

 
        // Accessors
 
        // Accessors

-       inline const char *VerStr() const {return S->VerStr == 0?0:Owner->StrP + S->VerStr;};
-       inline const char *Section() const {return S->Section == 0?0:Owner->StrP + S->Section;};
+       inline const char *VerStr() const {return S->VerStr == 0?0:Owner->StrP + S->VerStr;}
+       inline const char *Section() const {return S->Section == 0?0:Owner->StrP + S->Section;}
+#if APT_PKG_ABI >= 413
+       /** \brief source package name this version comes from
+          Always contains the name, even if it is the same as the binary name */
+       inline const char *SourcePkgName() const {return Owner->StrP + S->SourcePkgName;}
+       /** \brief source version this version comes from
+          Always contains the version string, even if it is the same as the binary version */
+       inline const char *SourceVerStr() const {return Owner->StrP + S->SourceVerStr;}
+#endif

        inline const char *Arch() const {
                if ((S->MultiArch & pkgCache::Version::All) == pkgCache::Version::All)
                        return "all";
                return S->ParentPkg == 0?0:Owner->StrP + ParentPkg()->Arch;
        inline const char *Arch() const {
                if ((S->MultiArch & pkgCache::Version::All) == pkgCache::Version::All)
                        return "all";
                return S->ParentPkg == 0?0:Owner->StrP + ParentPkg()->Arch;

-       };
-       inline PkgIterator ParentPkg() const {return PkgIterator(*Owner,Owner->PkgP + S->ParentPkg);};
+       }
+       inline PkgIterator ParentPkg() const {return PkgIterator(*Owner,Owner->PkgP + S->ParentPkg);}

 
        inline DescIterator DescriptionList() const;
        DescIterator TranslatedDescription() const;
 
        inline DescIterator DescriptionList() const;
        DescIterator TranslatedDescription() const;


@@ -219,7 +238,8 @@ class pkgCache::VerIterator : public Iterator<Version, VerIterator> {
        inline PrvIterator ProvidesList() const;
        inline VerFileIterator FileList() const;
        bool Downloadable() const;
        inline PrvIterator ProvidesList() const;
        inline VerFileIterator FileList() const;
        bool Downloadable() const;

-       inline const char *PriorityType() const {return Owner->Priority(S->Priority);};
+       inline const char *PriorityType() const {return Owner->Priority(S->Priority);}
+       const char *MultiArchType() const APT_PURE;

        std::string RelStr() const;
 
        bool Automatic() const;
        std::string RelStr() const;
 
        bool Automatic() const;


@@ -228,8 +248,8 @@ class pkgCache::VerIterator : public Iterator<Version, VerIterator> {
        inline VerIterator(pkgCache &Owner,Version *Trg = 0) : Iterator<Version, VerIterator>(Owner, Trg) {
                if (S == 0)
                        S = OwnerPointer();
        inline VerIterator(pkgCache &Owner,Version *Trg = 0) : Iterator<Version, VerIterator>(Owner, Trg) {
                if (S == 0)
                        S = OwnerPointer();

-       };
-       inline VerIterator() : Iterator<Version, VerIterator>() {};
+       }
+       inline VerIterator() : Iterator<Version, VerIterator>() {}

 };
                                                                        /*}}}*/
 // Description Iterator                                                        /*{{{*/
 };
                                                                        /*}}}*/
 // Description Iterator                                                        /*{{{*/


@@ -237,26 +257,26 @@ class pkgCache::DescIterator : public Iterator<Description, DescIterator> {
        protected:
        inline Description* OwnerPointer() const {
                return (Owner != 0) ? Owner->DescP : 0;
        protected:
        inline Description* OwnerPointer() const {
                return (Owner != 0) ? Owner->DescP : 0;

-       };
+       }

 
        public:
        // Iteration
 
        public:
        // Iteration

-       void operator ++(int) {if (S != Owner->DescP) S = Owner->DescP + S->NextDesc;};
-       inline void operator ++() {operator ++(0);};
+       void operator ++(int) {if (S != Owner->DescP) S = Owner->DescP + S->NextDesc;}
+       inline void operator ++() {operator ++(0);}

 
        // Comparison
        int CompareDesc(const DescIterator &B) const;
 
        // Accessors
 
        // Comparison
        int CompareDesc(const DescIterator &B) const;
 
        // Accessors

-       inline const char *LanguageCode() const {return Owner->StrP + S->language_code;};
-       inline const char *md5() const {return Owner->StrP + S->md5sum;};
+       inline const char *LanguageCode() const {return Owner->StrP + S->language_code;}
+       inline const char *md5() const {return Owner->StrP + S->md5sum;}

        inline DescFileIterator FileList() const;
 
        inline DescFileIterator FileList() const;
 

-       inline DescIterator() : Iterator<Description, DescIterator>() {};
+       inline DescIterator() : Iterator<Description, DescIterator>() {}

        inline DescIterator(pkgCache &Owner,Description *Trg = 0) : Iterator<Description, DescIterator>(Owner, Trg) {
                if (S == 0)
                        S = Owner.DescP;
        inline DescIterator(pkgCache &Owner,Description *Trg = 0) : Iterator<Description, DescIterator>(Owner, Trg) {
                if (S == 0)
                        S = Owner.DescP;

-       };
+       }

 };
                                                                        /*}}}*/
 // Dependency iterator                                                 /*{{{*/
 };
                                                                        /*}}}*/
 // Dependency iterator                                                 /*{{{*/


@@ -266,31 +286,33 @@ class pkgCache::DepIterator : public Iterator<Dependency, DepIterator> {
        protected:
        inline Dependency* OwnerPointer() const {
                return (Owner != 0) ? Owner->DepP : 0;
        protected:
        inline Dependency* OwnerPointer() const {
                return (Owner != 0) ? Owner->DepP : 0;

-       };
+       }

 
        public:
        // Iteration
        void operator ++(int) {if (S != Owner->DepP) S = Owner->DepP +
 
        public:
        // Iteration
        void operator ++(int) {if (S != Owner->DepP) S = Owner->DepP +

-               (Type == DepVer ? S->NextDepends : S->NextRevDepends);};
-       inline void operator ++() {operator ++(0);};
+               (Type == DepVer ? S->NextDepends : S->NextRevDepends);}
+       inline void operator ++() {operator ++(0);}

 
        // Accessors
 
        // Accessors

-       inline const char *TargetVer() const {return S->Version == 0?0:Owner->StrP + S->Version;};
-       inline PkgIterator TargetPkg() const {return PkgIterator(*Owner,Owner->PkgP + S->Package);};
-       inline PkgIterator SmartTargetPkg() const {PkgIterator R(*Owner,0);SmartTargetPkg(R);return R;};
-       inline VerIterator ParentVer() const {return VerIterator(*Owner,Owner->VerP + S->ParentVer);};
-       inline PkgIterator ParentPkg() const {return PkgIterator(*Owner,Owner->PkgP + Owner->VerP[S->ParentVer].ParentPkg);};
-       inline bool Reverse() const {return Type == DepRev;};
-       bool IsCritical() const;
-       bool IsNegative() const;
-       bool IsIgnorable(PrvIterator const &Prv) const;
-       bool IsIgnorable(PkgIterator const &Pkg) const;
-       bool IsMultiArchImplicit() const;
+       inline const char *TargetVer() const {return S->Version == 0?0:Owner->StrP + S->Version;}
+       inline PkgIterator TargetPkg() const {return PkgIterator(*Owner,Owner->PkgP + S->Package);}
+       inline PkgIterator SmartTargetPkg() const {PkgIterator R(*Owner,0);SmartTargetPkg(R);return R;}
+       inline VerIterator ParentVer() const {return VerIterator(*Owner,Owner->VerP + S->ParentVer);}
+       inline PkgIterator ParentPkg() const {return PkgIterator(*Owner,Owner->PkgP + Owner->VerP[S->ParentVer].ParentPkg);}
+       inline bool Reverse() const {return Type == DepRev;}
+       bool IsCritical() const APT_PURE;
+       bool IsNegative() const APT_PURE;
+       bool IsIgnorable(PrvIterator const &Prv) const APT_PURE;
+       bool IsIgnorable(PkgIterator const &Pkg) const APT_PURE;
+       bool IsMultiArchImplicit() const APT_PURE;
+       bool IsSatisfied(VerIterator const &Ver) const APT_PURE;
+       bool IsSatisfied(PrvIterator const &Prv) const APT_PURE;

        void GlobOr(DepIterator &Start,DepIterator &End);
        Version **AllTargets() const;
        bool SmartTargetPkg(PkgIterator &Result) const;
        void GlobOr(DepIterator &Start,DepIterator &End);
        Version **AllTargets() const;
        bool SmartTargetPkg(PkgIterator &Result) const;

-       inline const char *CompType() const {return Owner->CompType(S->CompareOp);};
-       inline const char *DepType() const {return Owner->DepType(S->Type);};
+       inline const char *CompType() const {return Owner->CompType(S->CompareOp);}
+       inline const char *DepType() const {return Owner->DepType(S->Type);}

 
        //Nice printable representation
        friend std::ostream& operator <<(std::ostream& out, DepIterator D);
 
        //Nice printable representation
        friend std::ostream& operator <<(std::ostream& out, DepIterator D);


@@ -299,13 +321,13 @@ class pkgCache::DepIterator : public Iterator<Dependency, DepIterator> {
                Iterator<Dependency, DepIterator>(Owner, Trg), Type(DepVer) {
                if (S == 0)
                        S = Owner.DepP;
                Iterator<Dependency, DepIterator>(Owner, Trg), Type(DepVer) {
                if (S == 0)
                        S = Owner.DepP;

-       };
+       }

        inline DepIterator(pkgCache &Owner, Dependency *Trg, Package*) :
                Iterator<Dependency, DepIterator>(Owner, Trg), Type(DepRev) {
                if (S == 0)
                        S = Owner.DepP;
        inline DepIterator(pkgCache &Owner, Dependency *Trg, Package*) :
                Iterator<Dependency, DepIterator>(Owner, Trg), Type(DepRev) {
                if (S == 0)
                        S = Owner.DepP;

-       };
-       inline DepIterator() : Iterator<Dependency, DepIterator>(), Type(DepVer) {};
+       }
+       inline DepIterator() : Iterator<Dependency, DepIterator>(), Type(DepVer) {}

 };
                                                                        /*}}}*/
 // Provides iterator                                                   /*{{{*/
 };
                                                                        /*}}}*/
 // Provides iterator                                                   /*{{{*/


@@ -315,34 +337,65 @@ class pkgCache::PrvIterator : public Iterator<Provides, PrvIterator> {
        protected:
        inline Provides* OwnerPointer() const {
                return (Owner != 0) ? Owner->ProvideP : 0;
        protected:
        inline Provides* OwnerPointer() const {
                return (Owner != 0) ? Owner->ProvideP : 0;

-       };
+       }

 
        public:
        // Iteration
        void operator ++(int) {if (S != Owner->ProvideP) S = Owner->ProvideP +
 
        public:
        // Iteration
        void operator ++(int) {if (S != Owner->ProvideP) S = Owner->ProvideP +

-               (Type == PrvVer?S->NextPkgProv:S->NextProvides);};
-       inline void operator ++() {operator ++(0);};
+               (Type == PrvVer?S->NextPkgProv:S->NextProvides);}
+       inline void operator ++() {operator ++(0);}

 
        // Accessors
 
        // Accessors

-       inline const char *Name() const {return Owner->StrP + Owner->PkgP[S->ParentPkg].Name;};
-       inline const char *ProvideVersion() const {return S->ProvideVersion == 0?0:Owner->StrP + S->ProvideVersion;};
-       inline PkgIterator ParentPkg() const {return PkgIterator(*Owner,Owner->PkgP + S->ParentPkg);};
-       inline VerIterator OwnerVer() const {return VerIterator(*Owner,Owner->VerP + S->Version);};
-       inline PkgIterator OwnerPkg() const {return PkgIterator(*Owner,Owner->PkgP + Owner->VerP[S->Version].ParentPkg);};
+       inline const char *Name() const {return ParentPkg().Name();}
+       inline const char *ProvideVersion() const {return S->ProvideVersion == 0?0:Owner->StrP + S->ProvideVersion;}
+       inline PkgIterator ParentPkg() const {return PkgIterator(*Owner,Owner->PkgP + S->ParentPkg);}
+       inline VerIterator OwnerVer() const {return VerIterator(*Owner,Owner->VerP + S->Version);}
+       inline PkgIterator OwnerPkg() const {return PkgIterator(*Owner,Owner->PkgP + Owner->VerP[S->Version].ParentPkg);}

 
 

-       bool IsMultiArchImplicit() const;
+       bool IsMultiArchImplicit() const APT_PURE;

 
 

-       inline PrvIterator() : Iterator<Provides, PrvIterator>(), Type(PrvVer) {};
+       inline PrvIterator() : Iterator<Provides, PrvIterator>(), Type(PrvVer) {}

        inline PrvIterator(pkgCache &Owner, Provides *Trg, Version*) :
                Iterator<Provides, PrvIterator>(Owner, Trg), Type(PrvVer) {
                if (S == 0)
                        S = Owner.ProvideP;
        inline PrvIterator(pkgCache &Owner, Provides *Trg, Version*) :
                Iterator<Provides, PrvIterator>(Owner, Trg), Type(PrvVer) {
                if (S == 0)
                        S = Owner.ProvideP;

-       };
+       }

        inline PrvIterator(pkgCache &Owner, Provides *Trg, Package*) :
                Iterator<Provides, PrvIterator>(Owner, Trg), Type(PrvPkg) {
                if (S == 0)
                        S = Owner.ProvideP;
        inline PrvIterator(pkgCache &Owner, Provides *Trg, Package*) :
                Iterator<Provides, PrvIterator>(Owner, Trg), Type(PrvPkg) {
                if (S == 0)
                        S = Owner.ProvideP;

-       };
+       }
+};
+                                                                       /*}}}*/
+// Release file                                                                /*{{{*/
+class pkgCache::RlsFileIterator : public Iterator<ReleaseFile, RlsFileIterator> {
+       protected:
+       inline ReleaseFile* OwnerPointer() const {
+               return (Owner != 0) ? Owner->RlsFileP : 0;
+       }
+
+       public:
+       // Iteration
+       void operator ++(int) {if (S != Owner->RlsFileP) S = Owner->RlsFileP + S->NextFile;}
+       inline void operator ++() {operator ++(0);}
+
+       // Accessors
+       inline const char *FileName() const {return S->FileName == 0?0:Owner->StrP + S->FileName;}
+       inline const char *Archive() const {return S->Archive == 0?0:Owner->StrP + S->Archive;}
+       inline const char *Version() const {return S->Version == 0?0:Owner->StrP + S->Version;}
+       inline const char *Origin() const {return S->Origin == 0?0:Owner->StrP + S->Origin;}
+       inline const char *Codename() const {return S->Codename ==0?0:Owner->StrP + S->Codename;}
+       inline const char *Label() const {return S->Label == 0?0:Owner->StrP + S->Label;}
+       inline const char *Site() const {return S->Site == 0?0:Owner->StrP + S->Site;}
+       inline bool Flagged(pkgCache::Flag::ReleaseFileFlags const flag) const {return (S->Flags & flag) == flag; }
+
+       bool IsOk();
+       std::string RelStr();
+
+       // Constructors
+       inline RlsFileIterator() : Iterator<ReleaseFile, RlsFileIterator>() {}
+       explicit inline RlsFileIterator(pkgCache &Owner) : Iterator<ReleaseFile, RlsFileIterator>(Owner, Owner.RlsFileP) {}
+       inline RlsFileIterator(pkgCache &Owner,ReleaseFile *Trg) : Iterator<ReleaseFile, RlsFileIterator>(Owner, Trg) {}

 };
                                                                        /*}}}*/
 // Package file                                                                /*{{{*/
 };
                                                                        /*}}}*/
 // Package file                                                                /*{{{*/


@@ -350,32 +403,35 @@ class pkgCache::PkgFileIterator : public Iterator<PackageFile, PkgFileIterator>
        protected:
        inline PackageFile* OwnerPointer() const {
                return (Owner != 0) ? Owner->PkgFileP : 0;
        protected:
        inline PackageFile* OwnerPointer() const {
                return (Owner != 0) ? Owner->PkgFileP : 0;

-       };
+       }

 
        public:
        // Iteration
 
        public:
        // Iteration

-       void operator ++(int) {if (S != Owner->PkgFileP) S = Owner->PkgFileP + S->NextFile;};
-       inline void operator ++() {operator ++(0);};
+       void operator ++(int) {if (S != Owner->PkgFileP) S = Owner->PkgFileP + S->NextFile;}
+       inline void operator ++() {operator ++(0);}

 
        // Accessors
 
        // Accessors

-       inline const char *FileName() const {return S->FileName == 0?0:Owner->StrP + S->FileName;};
-       inline const char *Archive() const {return S->Archive == 0?0:Owner->StrP + S->Archive;};
-       inline const char *Component() const {return S->Component == 0?0:Owner->StrP + S->Component;};
-       inline const char *Version() const {return S->Version == 0?0:Owner->StrP + S->Version;};
-       inline const char *Origin() const {return S->Origin == 0?0:Owner->StrP + S->Origin;};
-       inline const char *Codename() const {return S->Codename ==0?0:Owner->StrP + S->Codename;};
-       inline const char *Label() const {return S->Label == 0?0:Owner->StrP + S->Label;};
-       inline const char *Site() const {return S->Site == 0?0:Owner->StrP + S->Site;};
-       inline const char *Architecture() const {return S->Architecture == 0?0:Owner->StrP + S->Architecture;};
-       inline const char *IndexType() const {return S->IndexType == 0?0:Owner->StrP + S->IndexType;};
+       inline const char *FileName() const {return S->FileName == 0?0:Owner->StrP + S->FileName;}
+       inline pkgCache::RlsFileIterator ReleaseFile() const {return RlsFileIterator(*Owner, Owner->RlsFileP + S->Release);}
+       inline const char *Archive() const {return S->Release == 0 ? Component() : ReleaseFile().Archive();}
+       inline const char *Version() const {return S->Release == 0 ? NULL : ReleaseFile().Version();}
+       inline const char *Origin() const {return S->Release == 0 ? NULL : ReleaseFile().Origin();}
+       inline const char *Codename() const {return S->Release == 0 ? NULL : ReleaseFile().Codename();}
+       inline const char *Label() const {return S->Release == 0 ? NULL : ReleaseFile().Label();}
+       inline const char *Site() const {return S->Release == 0 ? NULL : ReleaseFile().Site();}
+       inline bool Flagged(pkgCache::Flag::ReleaseFileFlags const flag) const {return S->Release== 0 ? false : ReleaseFile().Flagged(flag);}
+       inline bool Flagged(pkgCache::Flag::PkgFFlags const flag) const {return (S->Flags & flag) == flag;}
+       inline const char *Component() const {return S->Component == 0?0:Owner->StrP + S->Component;}
+       inline const char *Architecture() const {return S->Architecture == 0?0:Owner->StrP + S->Architecture;}
+       inline const char *IndexType() const {return S->IndexType == 0?0:Owner->StrP + S->IndexType;}

 
        bool IsOk();
        std::string RelStr();
 
        // Constructors
 
        bool IsOk();
        std::string RelStr();
 
        // Constructors

-       inline PkgFileIterator() : Iterator<PackageFile, PkgFileIterator>() {};
-       inline PkgFileIterator(pkgCache &Owner) : Iterator<PackageFile, PkgFileIterator>(Owner, Owner.PkgFileP) {};
-       inline PkgFileIterator(pkgCache &Owner,PackageFile *Trg) : Iterator<PackageFile, PkgFileIterator>(Owner, Trg) {};
+       inline PkgFileIterator() : Iterator<PackageFile, PkgFileIterator>() {}
+       explicit inline PkgFileIterator(pkgCache &Owner) : Iterator<PackageFile, PkgFileIterator>(Owner, Owner.PkgFileP) {}
+       inline PkgFileIterator(pkgCache &Owner,PackageFile *Trg) : Iterator<PackageFile, PkgFileIterator>(Owner, Trg) {}

 };
                                                                        /*}}}*/
 // Version File                                                                /*{{{*/
 };
                                                                        /*}}}*/
 // Version File                                                                /*{{{*/


@@ -383,18 +439,18 @@ class pkgCache::VerFileIterator : public pkgCache::Iterator<VerFile, VerFileIter
        protected:
        inline VerFile* OwnerPointer() const {
                return (Owner != 0) ? Owner->VerFileP : 0;
        protected:
        inline VerFile* OwnerPointer() const {
                return (Owner != 0) ? Owner->VerFileP : 0;

-       };
+       }

 
        public:
        // Iteration
 
        public:
        // Iteration

-       void operator ++(int) {if (S != Owner->VerFileP) S = Owner->VerFileP + S->NextFile;};
-       inline void operator ++() {operator ++(0);};
+       void operator ++(int) {if (S != Owner->VerFileP) S = Owner->VerFileP + S->NextFile;}
+       inline void operator ++() {operator ++(0);}

 
        // Accessors
 
        // Accessors

-       inline PkgFileIterator File() const {return PkgFileIterator(*Owner,S->File + Owner->PkgFileP);};
+       inline PkgFileIterator File() const {return PkgFileIterator(*Owner,S->File + Owner->PkgFileP);}

 
 

-       inline VerFileIterator() : Iterator<VerFile, VerFileIterator>() {};
-       inline VerFileIterator(pkgCache &Owner,VerFile *Trg) : Iterator<VerFile, VerFileIterator>(Owner, Trg) {};
+       inline VerFileIterator() : Iterator<VerFile, VerFileIterator>() {}
+       inline VerFileIterator(pkgCache &Owner,VerFile *Trg) : Iterator<VerFile, VerFileIterator>(Owner, Trg) {}

 };
                                                                        /*}}}*/
 // Description File                                                    /*{{{*/
 };
                                                                        /*}}}*/
 // Description File                                                    /*{{{*/


@@ -402,40 +458,40 @@ class pkgCache::DescFileIterator : public Iterator<DescFile, DescFileIterator> {
        protected:
        inline DescFile* OwnerPointer() const {
                return (Owner != 0) ? Owner->DescFileP : 0;
        protected:
        inline DescFile* OwnerPointer() const {
                return (Owner != 0) ? Owner->DescFileP : 0;

-       };
+       }

 
        public:
        // Iteration
 
        public:
        // Iteration

-       void operator ++(int) {if (S != Owner->DescFileP) S = Owner->DescFileP + S->NextFile;};
-       inline void operator ++() {operator ++(0);};
+       void operator ++(int) {if (S != Owner->DescFileP) S = Owner->DescFileP + S->NextFile;}
+       inline void operator ++() {operator ++(0);}

 
        // Accessors
 
        // Accessors

-       inline PkgFileIterator File() const {return PkgFileIterator(*Owner,S->File + Owner->PkgFileP);};
+       inline PkgFileIterator File() const {return PkgFileIterator(*Owner,S->File + Owner->PkgFileP);}

 
 

-       inline DescFileIterator() : Iterator<DescFile, DescFileIterator>() {};
-       inline DescFileIterator(pkgCache &Owner,DescFile *Trg) : Iterator<DescFile, DescFileIterator>(Owner, Trg) {};
+       inline DescFileIterator() : Iterator<DescFile, DescFileIterator>() {}
+       inline DescFileIterator(pkgCache &Owner,DescFile *Trg) : Iterator<DescFile, DescFileIterator>(Owner, Trg) {}

 };
                                                                        /*}}}*/
 };
                                                                        /*}}}*/

-// Inlined Begin functions cant be in the class because of order problems /*{{{*/
+// Inlined Begin functions can't be in the class because of order problems /*{{{*/

 inline pkgCache::PkgIterator pkgCache::GrpIterator::PackageList() const
 inline pkgCache::PkgIterator pkgCache::GrpIterator::PackageList() const

-       {return PkgIterator(*Owner,Owner->PkgP + S->FirstPackage);};
+       {return PkgIterator(*Owner,Owner->PkgP + S->FirstPackage);}

 inline pkgCache::VerIterator pkgCache::PkgIterator::VersionList() const
 inline pkgCache::VerIterator pkgCache::PkgIterator::VersionList() const

-       {return VerIterator(*Owner,Owner->VerP + S->VersionList);};
+       {return VerIterator(*Owner,Owner->VerP + S->VersionList);}

 inline pkgCache::VerIterator pkgCache::PkgIterator::CurrentVer() const
 inline pkgCache::VerIterator pkgCache::PkgIterator::CurrentVer() const

-       {return VerIterator(*Owner,Owner->VerP + S->CurrentVer);};
+       {return VerIterator(*Owner,Owner->VerP + S->CurrentVer);}

 inline pkgCache::DepIterator pkgCache::PkgIterator::RevDependsList() const
 inline pkgCache::DepIterator pkgCache::PkgIterator::RevDependsList() const

-       {return DepIterator(*Owner,Owner->DepP + S->RevDepends,S);};
+       {return DepIterator(*Owner,Owner->DepP + S->RevDepends,S);}

 inline pkgCache::PrvIterator pkgCache::PkgIterator::ProvidesList() const
 inline pkgCache::PrvIterator pkgCache::PkgIterator::ProvidesList() const

-       {return PrvIterator(*Owner,Owner->ProvideP + S->ProvidesList,S);};
+       {return PrvIterator(*Owner,Owner->ProvideP + S->ProvidesList,S);}

 inline pkgCache::DescIterator pkgCache::VerIterator::DescriptionList() const
 inline pkgCache::DescIterator pkgCache::VerIterator::DescriptionList() const

-       {return DescIterator(*Owner,Owner->DescP + S->DescriptionList);};
+       {return DescIterator(*Owner,Owner->DescP + S->DescriptionList);}

 inline pkgCache::PrvIterator pkgCache::VerIterator::ProvidesList() const
 inline pkgCache::PrvIterator pkgCache::VerIterator::ProvidesList() const

-       {return PrvIterator(*Owner,Owner->ProvideP + S->ProvidesList,S);};
+       {return PrvIterator(*Owner,Owner->ProvideP + S->ProvidesList,S);}

 inline pkgCache::DepIterator pkgCache::VerIterator::DependsList() const
 inline pkgCache::DepIterator pkgCache::VerIterator::DependsList() const

-       {return DepIterator(*Owner,Owner->DepP + S->DependsList,S);};
+       {return DepIterator(*Owner,Owner->DepP + S->DependsList,S);}

 inline pkgCache::VerFileIterator pkgCache::VerIterator::FileList() const
 inline pkgCache::VerFileIterator pkgCache::VerIterator::FileList() const

-       {return VerFileIterator(*Owner,Owner->VerFileP + S->FileList);};
+       {return VerFileIterator(*Owner,Owner->VerFileP + S->FileList);}

 inline pkgCache::DescFileIterator pkgCache::DescIterator::FileList() const
 inline pkgCache::DescFileIterator pkgCache::DescIterator::FileList() const

-       {return DescFileIterator(*Owner,Owner->DescFileP + S->FileList);};
+       {return DescFileIterator(*Owner,Owner->DescFileP + S->FileList);}

                                                                        /*}}}*/
 #endif
                                                                        /*}}}*/
 #endif