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

diff --git a/src/lib/transcode_job.cc b/src/lib/transcode_job.cc
index 3502c3346..6b1563b9b 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,6 +93,32 @@ void
 TranscodeJob::run ()
 {
 	try {
+		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;
+			}
+		}
+
 		struct timeval start;
 		gettimeofday (&start, 0);
 		LOG_GENERAL_NC (N_("Transcode job starting"));
@@ -105,16 +134,17 @@ TranscodeJob::run ()
 			fps = _encoder->frames_done() / (finish.tv_sec - start.tv_sec);
 		}
 
-		LOG_GENERAL (N_("Transcode job completed successfully: %1 fps"), fps);
+		LOG_GENERAL (N_("Transcode job completed successfully: %1 fps"), dcp::locale_convert<string>(fps, 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 ();
 
@@ -135,29 +165,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;
 }