- a bit of handler class cleansing
authorPanu Matilainen <pmatilai@turre.koti.laiskiainen.org>
Tue, 6 Mar 2007 19:50:14 +0000 (21:50 +0200)
committerPanu Matilainen <pmatilai@turre.koti.laiskiainen.org>
Tue, 6 Mar 2007 19:50:14 +0000 (21:50 +0200)
apt-pkg/rpm/rpmhandler.cc
apt-pkg/rpm/rpmhandler.h

index f280b76..be4d12f 100644 (file)
@@ -600,6 +600,11 @@ string RPMSingleFileHandler::MD5Sum()
    return MD5.Result().Value();
 }
 
+bool RPMSingleFileHandler::ChangeLog(vector<ChangeLogEntry* > &ChangeLogs)
+{
+   return RPMHdrHandler::ChangeLog(ChangeLogs);
+}
+
 RPMDirHandler::RPMDirHandler(string DirName)
    : sDirName(DirName)
 {
@@ -933,16 +938,14 @@ void RPMDBHandler::Rewind()
 #endif
 
 #ifdef APT_WITH_REPOMD
-RPMRepomdHandler::RPMRepomdHandler(string File)
+RPMRepomdHandler::RPMRepomdHandler(string File): RPMHandler(),
+      Root(NULL), Primary(NULL), PrimaryPath(File)
 {
-   PrimaryPath = File;
    string DBBase = PrimaryPath.substr(0, File.size() - strlen("primary.xml"));
    FilelistPath = DBBase + "filelists.xml";
    OtherPath = DBBase + "other.xml";
 
    ID = File;
-   Root = NULL;
-   Primary = NULL;
    xmlChar *packages = NULL;
    off_t pkgcount = 0;
    
@@ -1311,12 +1314,10 @@ RPMRepomdHandler::~RPMRepomdHandler()
    xmlFreeDoc(Primary);
 }
 
-RPMRepomdReaderHandler::RPMRepomdReaderHandler(string File) : RPMHandler()
+RPMRepomdReaderHandler::RPMRepomdReaderHandler(string File) : RPMHandler(),
+   XmlPath(File), NodeP(NULL), XmlFile(NULL)
 {
-   XmlPath = File;
    ID = File;
-   XmlFile = NULL;
-   NodeP = NULL;
    iOffset = -1;
 
    if (FileExists(XmlPath)) {
index 2a15c32..f9d3ab6 100644 (file)
@@ -101,8 +101,8 @@ class RPMHandler
    virtual bool IsSourceRpm() {return SourceRpm().empty();}
 
    virtual bool PRCO(unsigned int Type, vector<Dependency*> &Deps) = 0;
-   virtual bool FileList(vector<string> &FileList) { return false; };
-   virtual bool ChangeLog(vector<ChangeLogEntry* > &ChangeLogs) { return false; };
+   virtual bool FileList(vector<string> &FileList) = 0;
+   virtual bool ChangeLog(vector<ChangeLogEntry* > &ChangeLogs) = 0;
 
    virtual bool HasFile(const char *File);
 
@@ -195,7 +195,7 @@ class RPMSingleFileHandler : public RPMFileHandler
    virtual off_t FileSize();
    virtual string MD5Sum();
    virtual bool ProvideFileName() {return true;};
-   virtual bool ChangeLog(vector<ChangeLogEntry* > &ChangeLogs) { return RPMHandler::ChangeLog(ChangeLogs); };
+   virtual bool ChangeLog(vector<ChangeLogEntry* > &ChangeLogs);
 
    RPMSingleFileHandler(string File) : RPMFileHandler(File), sFilePath(File) {};
    virtual ~RPMSingleFileHandler() {};
@@ -362,7 +362,6 @@ class RPMRepomdReaderHandler : public RPMHandler
    virtual bool PRCO(unsigned int Type, vector<Dependency*> &Deps)
        {return true;};
 
-   virtual bool FileList(vector<string> &FileList) {return true;};
    RPMRepomdReaderHandler(string File);
    virtual ~RPMRepomdReaderHandler();
 };
@@ -372,13 +371,16 @@ class RPMRepomdFLHandler : public RPMRepomdReaderHandler
    public:
    virtual bool FileList(vector<string> &FileList);
    RPMRepomdFLHandler(string File) : RPMRepomdReaderHandler(File) {};
+   virtual bool ChangeLog(vector<ChangeLogEntry* > &ChangeLogs) {return true;};
    virtual ~RPMRepomdFLHandler() {};
 };
 
 class RPMRepomdOtherHandler : public RPMRepomdReaderHandler
 {
    public:
+   virtual bool FileList(vector<string> &FileList) {return true;};
    virtual bool ChangeLog(vector<ChangeLogEntry* > &ChangeLogs);
+
    RPMRepomdOtherHandler(string File) : RPMRepomdReaderHandler(File) {};
    virtual ~RPMRepomdOtherHandler() {};
 };