Cleanup: remove unnecessary variable.
authorCarl Hetherington <cth@carlh.net>
Sat, 24 Dec 2022 23:22:16 +0000 (00:22 +0100)
committerCarl Hetherington <cth@carlh.net>
Sat, 24 Dec 2022 23:22:16 +0000 (00:22 +0100)
src/lib/job_manager.cc
src/lib/job_manager.h

index 7a6bda8405b32240ae18bd9211e65abfe228a060..2b4e872998f72478614557014fe241a2d320d7f1 100644 (file)
@@ -331,7 +331,7 @@ JobManager::pause ()
 {
        boost::mutex::scoped_lock lm (_mutex);
 
-       if (_paused) {
+       if (_paused_job) {
                return;
        }
 
@@ -340,8 +340,6 @@ JobManager::pause ()
                        _paused_job = i;
                }
        }
-
-       _paused = true;
 }
 
 
@@ -349,14 +347,11 @@ void
 JobManager::resume ()
 {
        boost::mutex::scoped_lock lm (_mutex);
-       if (!_paused) {
-               return;
-       }
 
-       if (_paused_job) {
-               _paused_job->resume ();
+       if (!_paused_job) {
+               return;
        }
 
-       _paused_job.reset ();
-       _paused = false;
+       _paused_job->resume();
+       _paused_job.reset();
 }
index 71db33fd6bf9d2b14fe8037ec1c56a5d7de1de67..6802b511597586de15119df0656c11e6e7faf422 100644 (file)
@@ -62,7 +62,7 @@ public:
        void resume ();
        bool paused () const {
                boost::mutex::scoped_lock lm (_mutex);
-               return _paused;
+               return static_cast<bool>(_paused_job);
        }
 
        void analyse_audio (
@@ -104,7 +104,6 @@ private:
        std::list<std::shared_ptr<Job>> _jobs;
        std::list<boost::signals2::connection> _connections;
        bool _terminate = false;
-       bool _paused = false;
        std::shared_ptr<Job> _paused_job;
 
        boost::optional<std::string> _last_active_job;