X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Ftranscode_job.cc;h=b260bc44b1d852c5c23ce0870a7a3c2594a2318f;hb=3f6a9b5e51ec39a66f61705278cbd8995a32b052;hp=3502c3346c48072824f04f22936dba0fcc04ba83;hpb=689fa55d1529ad88449ca464e9107c4dcc54d1cb;p=dcpomatic.git

diff --git a/src/lib/transcode_job.cc b/src/lib/transcode_job.cc
index 3502c3346..b260bc44b 100644
--- a/src/lib/transcode_job.cc
+++ b/src/lib/transcode_job.cc
@@ -26,10 +26,12 @@
 
 #include "analytics.h"
 #include "compose.hpp"
+#include "content.h"
 #include "config.h"
 #include "dcp_encoder.h"
 #include "dcpomatic_log.h"
 #include "encoder.h"
+#include "examine_content_job.h"
 #include "film.h"
 #include "job_manager.h"
 #include "log.h"
@@ -52,8 +54,9 @@ using std::dynamic_pointer_cast;
 
 
 /** @param film Film to use */
-TranscodeJob::TranscodeJob (shared_ptr<const Film> film)
+TranscodeJob::TranscodeJob (shared_ptr<const Film> film, ChangedBehaviour changed)
 	: Job (film)
+	, _changed (changed)
 {
 
 }
@@ -90,37 +93,54 @@ void
 TranscodeJob::run ()
 {
 	try {
-		struct timeval start;
-		gettimeofday (&start, 0);
+		auto content = _film->content();
+		std::vector<shared_ptr<Content>> changed;
+		std::copy_if (content.begin(), content.end(), std::back_inserter(changed), [](shared_ptr<Content> c) { return c->changed(); });
+
+		if (!changed.empty()) {
+			switch (_changed) {
+			case ChangedBehaviour::EXAMINE_THEN_STOP:
+				for (auto i: changed) {
+					JobManager::instance()->add(make_shared<ExamineContentJob>(_film, i));
+				}
+				set_progress (1);
+				set_message (_("Some files have been changed since they were added to the project.\n\nThese files will now be re-examined, so you may need to check their settings before trying again."));
+				set_error (_("Files have changed since they were added to the project."), _("Check their new settings, then try again."));
+				set_state (FINISHED_ERROR);
+				return;
+			case ChangedBehaviour::STOP:
+				set_progress (1);
+				set_error (_("Files have changed since they were added to the project."), _("Open the project in DCP-o-matic, check the settings, then save it before trying again."));
+				set_state (FINISHED_ERROR);
+				return;
+			default:
+				LOG_GENERAL_NC (_("Some files have been changed since they were added to the project."));
+				break;
+			}
+		}
+
 		LOG_GENERAL_NC (N_("Transcode job starting"));
 
 		DCPOMATIC_ASSERT (_encoder);
 		_encoder->go ();
 
-		struct timeval finish;
-		gettimeofday (&finish, 0);
-
-		float fps = 0;
-		if (finish.tv_sec != start.tv_sec) {
-			fps = _encoder->frames_done() / (finish.tv_sec - start.tv_sec);
-		}
+		set_progress (1);
+		set_state (FINISHED_OK);
 
-		LOG_GENERAL (N_("Transcode job completed successfully: %1 fps"), fps);
+		LOG_GENERAL(N_("Transcode job completed successfully: %1 fps"), dcp::locale_convert<string>(frames_per_second(), 2, true));
 
 		if (dynamic_pointer_cast<DCPEncoder>(_encoder)) {
-			Analytics::instance()->successful_dcp_encode();
+			try {
+				Analytics::instance()->successful_dcp_encode();
+			} catch (FileError& e) {
+				LOG_WARNING (N_("Failed to write analytics (%1)"), e.what());
+			}
 		}
 
-		/* XXX: this shouldn't be here */
-		if (Config::instance()->upload_after_make_dcp() && dynamic_pointer_cast<DCPEncoder>(_encoder)) {
-			JobManager::instance()->add(make_shared<UploadJob>(_film));
-		}
+		post_transcode ();
 
 		_encoder.reset ();
 
-		set_progress (1);
-		set_state (FINISHED_OK);
-
 	} catch (...) {
 		_encoder.reset ();
 		throw;
@@ -128,6 +148,20 @@ TranscodeJob::run ()
 }
 
 
+void
+TranscodeJob::pause()
+{
+	_encoder->pause();
+}
+
+
+void TranscodeJob::resume()
+{
+	_encoder->resume();
+	Job::resume();
+}
+
+
 string
 TranscodeJob::status () const
 {
@@ -135,29 +169,17 @@ TranscodeJob::status () const
 		return Job::status ();
 	}
 
-
-	char buffer[256];
 	if (finished() || _encoder->finishing()) {
-		strncpy (buffer, Job::status().c_str(), 255);
-		buffer[255] = '\0';
-	} else {
-		snprintf (
-			buffer, sizeof(buffer), "%s; %" PRId64 "/%" PRId64 " frames",
-			Job::status().c_str(),
-			_encoder->frames_done(),
-			_film->length().frames_round (_film->video_frame_rate ())
-			);
-
-		optional<float> const fps = _encoder->current_rate ();
-		if (fps) {
-			char fps_buffer[64];
-			/// TRANSLATORS: fps here is an abbreviation for frames per second
-			snprintf (fps_buffer, sizeof(fps_buffer), _("; %.1f fps"), *fps);
-			strncat (buffer, fps_buffer, strlen(buffer) - 1);
-		}
+		return Job::status();
+	}
+
+	auto status = String::compose(_("%1; %2/%3 frames"), Job::status(), _encoder->frames_done(), _film->length().frames_round(_film->video_frame_rate()));
+	if (auto const fps = _encoder->current_rate()) {
+		/// TRANSLATORS: fps here is an abbreviation for frames per second
+		status += String::compose(_("; %1 fps"), dcp::locale_convert<string>(*fps, 1, true));
 	}
 
-	return buffer;
+	return status;
 }
 
 
@@ -184,3 +206,15 @@ TranscodeJob::remaining_time () const
 	/* Compute approximate proposed length here, as it's only here that we need it */
 	return (_film->length().frames_round(_film->video_frame_rate()) - e->frames_done()) / *fps;
 }
+
+
+float
+TranscodeJob::frames_per_second() const
+{
+	if (_finish_time != _start_time) {
+		return _encoder->frames_done() / (_finish_time - _start_time);
+	} else {
+		return 0;
+	}
+}
+