X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fjob_manager.cc;h=3992e685ed4fe76c9ee4c5912ae7cd41fcaac873;hb=c180f317d5a8b27dd191c1f2228ceb6fc4039393;hp=08c744e0738a8faa4155d0bdf7ac083b8fcba134;hpb=3828baf56467224f5d44049bf1e7a7ed11f43a05;p=dcpomatic.git diff --git a/src/lib/job_manager.cc b/src/lib/job_manager.cc index 08c744e07..3992e685e 100644 --- a/src/lib/job_manager.cc +++ b/src/lib/job_manager.cc @@ -63,8 +63,12 @@ JobManager::~JobManager () } if (_scheduler) { - DCPOMATIC_ASSERT (_scheduler->joinable ()); - _scheduler->join (); + /* Ideally this would be a DCPOMATIC_ASSERT(_scheduler->joinable()) but we + can't throw exceptions from a destructor. + */ + if (_scheduler->joinable ()) { + _scheduler->join (); + } } delete _scheduler; @@ -206,3 +210,72 @@ JobManager::analyse_audio ( emit (boost::bind (boost::ref (JobAdded), weak_ptr (job))); } + +void +JobManager::increase_priority (shared_ptr job) +{ + bool changed = false; + + { + boost::mutex::scoped_lock lm (_mutex); + list >::iterator last = _jobs.end (); + for (list >::iterator i = _jobs.begin(); i != _jobs.end(); ++i) { + if (*i == job && last != _jobs.end()) { + swap (*i, *last); + changed = true; + break; + } + last = i; + } + } + + if (changed) { + priority_changed (); + } +} + +void +JobManager::priority_changed () +{ + { + boost::mutex::scoped_lock lm (_mutex); + + bool first = true; + BOOST_FOREACH (shared_ptr i, _jobs) { + if (first) { + if (i->is_new ()) { + i->start (); + } else if (i->paused_by_priority ()) { + i->resume (); + } + first = false; + } else { + if (i->running ()) { + i->pause_by_priority (); + } + } + } + } + + emit (boost::bind (boost::ref (JobsReordered))); +} + +void +JobManager::decrease_priority (shared_ptr job) +{ + bool changed = false; + + for (list >::iterator i = _jobs.begin(); i != _jobs.end(); ++i) { + list >::iterator next = i; + ++next; + if (*i == job && next != _jobs.end()) { + swap (*i, *next); + changed = true; + break; + } + } + + if (changed) { + priority_changed (); + } +}