Просмотр исходного кода

rename APT::Progress::Packagemanager Started->Start, Finished->Stop to match pkgAcquireStatus

Michael Vogt лет назад: 12
Родитель
Сommit
a22fdebf63
3 измененных файлов с 14 добавлено и 17 удалено
  1. 4 6
      apt-pkg/deb/dpkgpm.cc
  2. 4 4
      apt-pkg/iprogress.cc
  3. 6 7
      apt-pkg/iprogress.h

+ 4 - 6
apt-pkg/deb/dpkgpm.cc

@@ -1264,11 +1264,9 @@ bool pkgDPkgPM::Go(APT::Progress::PackageManager *progress)
 	 _error->DumpErrors(std::cerr);
       _error->RevertToStack();
 
-       // Fork dpkg
-      pid_t Child;
-      d->progress->Started();
-
-      Child = ExecFork();
+       // Tell the progress that its starting and fork dpkg 
+      d->progress->Start();
+      pid_t Child = ExecFork();
       // This is the child
       if (Child == 0)
       {
@@ -1398,7 +1396,7 @@ bool pkgDPkgPM::Go(APT::Progress::PackageManager *progress)
       signal(SIGHUP,old_SIGHUP);
 
       // tell the progress
-      d->progress->Finished();
+      d->progress->Stop();
 
       if(master >= 0) 
       {

+ 4 - 4
apt-pkg/iprogress.cc

@@ -40,7 +40,7 @@ void PackageManagerProgressFd::WriteToStatusFd(std::string s)
    FileFd::Write(OutStatusFd, s.c_str(), s.size());   
 }
 
-void PackageManagerProgressFd::Started()
+void PackageManagerProgressFd::Start()
 {
    _config->Set("APT::Keep-Fds::", OutStatusFd);
 
@@ -53,7 +53,7 @@ void PackageManagerProgressFd::Started()
    WriteToStatusFd(status.str());
 }
 
-void PackageManagerProgressFd::Finished()
+void PackageManagerProgressFd::Stop()
 {
    // clear the Keep-Fd again
    _config->Clear("APT::Keep-Fds", OutStatusFd);
@@ -134,13 +134,13 @@ PackageManagerFancy::PackageManagerFancy()
    }
 }
 
-void PackageManagerFancy::Started()
+void PackageManagerFancy::Start()
 {
    if (nr_terminal_rows > 0)
       SetupTerminalScrollArea(nr_terminal_rows);
 }
 
-void PackageManagerFancy::Finished()
+void PackageManagerFancy::Stop()
 {
    if (nr_terminal_rows > 0)
    {

+ 6 - 7
apt-pkg/iprogress.h

@@ -24,8 +24,8 @@ namespace Progress {
        : percentage(0.0), last_reported_progress(0) {};
     virtual ~PackageManager() {};
 
-    virtual void Started() {};
-    virtual void Finished() {};
+    virtual void Start() {};
+    virtual void Stop() {};
 
     virtual pid_t fork() {return fork(); };
 
@@ -59,9 +59,8 @@ namespace Progress {
  public:
     PackageManagerProgressFd(int progress_fd);
 
-    // FIXME: rename to Start/Stop to match the pkgAcquireStatus
-    virtual void Started();
-    virtual void Finished();
+    virtual void Start();
+    virtual void Stop();
 
     virtual bool StatusChanged(std::string PackageName, 
                                unsigned int StepsDone,
@@ -86,8 +85,8 @@ namespace Progress {
 
  public:
     PackageManagerFancy();
-    virtual void Started();
-    virtual void Finished();
+    virtual void Start();
+    virtual void Stop();
     virtual bool StatusChanged(std::string PackageName, 
                                unsigned int StepsDone,
                                unsigned int TotalSteps,