X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Freel_writer.cc;h=c560909a9f5c1aa2d189a8b3dccc23a3fcd3a313;hb=60d528017a3ac1afdc30a3a000b28dda96a2175e;hp=e6533d2cc292ca0e9c49f8da78a41f0604b5a276;hpb=a8a0dfd1b21de6c0facf965ab119833ff6f790bf;p=dcpomatic.git

diff --git a/src/lib/reel_writer.cc b/src/lib/reel_writer.cc
index e6533d2cc..c560909a9 100644
--- a/src/lib/reel_writer.cc
+++ b/src/lib/reel_writer.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2016 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -41,6 +41,7 @@
 #include <dcp/certificate_chain.h>
 #include <dcp/interop_subtitle_asset.h>
 #include <dcp/smpte_subtitle_asset.h>
+#include <dcp/raw_convert.h>
 #include <boost/foreach.hpp>
 
 #include "i18n.h"
@@ -57,16 +58,20 @@ using boost::shared_ptr;
 using boost::optional;
 using boost::dynamic_pointer_cast;
 using dcp::Data;
+using dcp::raw_convert;
 
 int const ReelWriter::_info_size = 48;
 
-ReelWriter::ReelWriter (shared_ptr<const Film> film, DCPTimePeriod period, shared_ptr<Job> job)
+ReelWriter::ReelWriter (
+	shared_ptr<const Film> film, DCPTimePeriod period, shared_ptr<Job> job, int reel_index, int reel_count, optional<string> content_summary
+	)
 	: _film (film)
 	, _period (period)
-	, _first_nonexistant_frame (0)
 	, _last_written_video_frame (-1)
 	, _last_written_eyes (EYES_RIGHT)
-	, _total_written_audio_frames (0)
+	, _reel_index (reel_index)
+	, _reel_count (reel_count)
+	, _content_summary (content_summary)
 {
 	/* Create our picture asset in a subdirectory, named according to those
 	   film's parameters which affect the video output.  We will hard-link
@@ -83,6 +88,7 @@ ReelWriter::ReelWriter (shared_ptr<const Film> film, DCPTimePeriod period, share
 
 	if (_film->encrypted ()) {
 		_picture_asset->set_key (_film->key ());
+		_picture_asset->set_context_id (_film->context_id ());
 	}
 
 	_picture_asset->set_file (
@@ -90,7 +96,7 @@ ReelWriter::ReelWriter (shared_ptr<const Film> film, DCPTimePeriod period, share
 		);
 
 	job->sub (_("Checking existing image data"));
-	check_existing_picture_asset ();
+	_first_nonexistant_frame = check_existing_picture_asset ();
 
 	_picture_asset_writer = _picture_asset->start_write (
 		_film->internal_video_asset_dir() / _film->internal_video_asset_filename(_period),
@@ -107,11 +113,13 @@ ReelWriter::ReelWriter (shared_ptr<const Film> film, DCPTimePeriod period, share
 			_sound_asset->set_key (_film->key ());
 		}
 
+		DCPOMATIC_ASSERT (_film->directory());
+
 		/* Write the sound asset into the film directory so that we leave the creation
 		   of the DCP directory until the last minute.
 		*/
 		_sound_asset_writer = _sound_asset->start_write (
-			_film->directory() / audio_asset_filename (_sound_asset),
+			_film->directory().get() / audio_asset_filename (_sound_asset, _reel_index, _reel_count, _content_summary),
 			_film->interop() ? dcp::INTEROP : dcp::SMPTE
 			);
 	}
@@ -123,13 +131,27 @@ ReelWriter::write_frame_info (Frame frame, Eyes eyes, dcp::FrameInfo info) const
 {
 	FILE* file = 0;
 	boost::filesystem::path info_file = _film->info_file (_period);
-	if (boost::filesystem::exists (info_file)) {
+
+	bool const read = boost::filesystem::exists (info_file);
+
+#ifdef DCPOMATIC_WINDOWS
+	if (read) {
+		LOG_GENERAL (
+			"Checked %1 (which exists) length is %2 perms are %3",
+			info_file, boost::filesystem::file_size (info_file), int(boost::filesystem::status(info_file).permissions())
+			);
+	} else {
+		LOG_GENERAL ("Checked %1 (which does not exist)", info_file);
+	}
+#endif
+
+	if (read) {
 		file = fopen_boost (info_file, "r+b");
 	} else {
 		file = fopen_boost (info_file, "wb");
 	}
 	if (!file) {
-		throw OpenFileError (info_file);
+		throw OpenFileError (info_file, errno, read);
 	}
 	dcpomatic_fseek (file, frame_info_position (frame, eyes), SEEK_SET);
 	fwrite (&info.offset, sizeof (info.offset), 1, file);
@@ -171,16 +193,29 @@ ReelWriter::frame_info_position (Frame frame, Eyes eyes) const
 	DCPOMATIC_ASSERT (false);
 }
 
-void
+Frame
 ReelWriter::check_existing_picture_asset ()
 {
+	DCPOMATIC_ASSERT (_picture_asset->file());
+	boost::filesystem::path asset = _picture_asset->file().get();
+
+	/* If there is an existing asset, break any hard links to it as we are about to change its contents
+	   (if only by changing the IDs); see #1126.
+	*/
+
+	if (boost::filesystem::exists(asset) && boost::filesystem::hard_link_count(asset) > 1) {
+		boost::filesystem::copy_file (asset, asset.string() + ".tmp");
+		boost::filesystem::remove (asset);
+		boost::filesystem::rename (asset.string() + ".tmp", asset);
+	}
+
 	/* Try to open the existing asset */
-	FILE* asset_file = fopen_boost (_picture_asset->file(), "rb");
+	FILE* asset_file = fopen_boost (asset, "rb");
 	if (!asset_file) {
-		LOG_GENERAL ("Could not open existing asset at %1 (errno=%2)", _picture_asset->file().string(), errno);
-		return;
+		LOG_GENERAL ("Could not open existing asset at %1 (errno=%2)", asset.string(), errno);
+		return 0;
 	} else {
-		LOG_GENERAL ("Opened existing asset at %1", _picture_asset->file().string());
+		LOG_GENERAL ("Opened existing asset at %1", asset.string());
 	}
 
 	/* Offset of the last dcp::FrameInfo in the info file */
@@ -191,31 +226,34 @@ ReelWriter::check_existing_picture_asset ()
 	if (!info_file) {
 		LOG_GENERAL_NC ("Could not open film info file");
 		fclose (asset_file);
-		return;
+		return 0;
 	}
 
+	Frame first_nonexistant_frame;
 	if (_film->three_d ()) {
 		/* Start looking at the last left frame */
-		_first_nonexistant_frame = n / 2;
+		first_nonexistant_frame = n / 2;
 	} else {
-		_first_nonexistant_frame = n;
+		first_nonexistant_frame = n;
 	}
 
-	while (!existing_picture_frame_ok(asset_file, info_file) && _first_nonexistant_frame > 0) {
-		--_first_nonexistant_frame;
+	while (!existing_picture_frame_ok(asset_file, info_file, first_nonexistant_frame) && first_nonexistant_frame > 0) {
+		--first_nonexistant_frame;
 	}
 
-	if (!_film->three_d() && _first_nonexistant_frame > 0) {
+	if (!_film->three_d() && first_nonexistant_frame > 0) {
 		/* If we are doing 3D we might have found a good L frame with no R, so only
 		   do this if we're in 2D and we've just found a good B(oth) frame.
 		*/
-		++_first_nonexistant_frame;
+		++first_nonexistant_frame;
 	}
 
-	LOG_GENERAL ("Proceeding with first nonexistant frame %1", _first_nonexistant_frame);
+	LOG_GENERAL ("Proceeding with first nonexistant frame %1", first_nonexistant_frame);
 
 	fclose (asset_file);
 	fclose (info_file);
+
+	return first_nonexistant_frame;
 }
 
 void
@@ -253,6 +291,7 @@ ReelWriter::finish ()
 {
 	if (!_picture_asset_writer->finalize ()) {
 		/* Nothing was written to the picture asset */
+		LOG_GENERAL ("Nothing was written to reel %1 of %2", _reel_index, _reel_count);
 		_picture_asset.reset ();
 	}
 
@@ -263,10 +302,11 @@ ReelWriter::finish ()
 
 	/* Hard-link any video asset file into the DCP */
 	if (_picture_asset) {
-		boost::filesystem::path video_from = _picture_asset->file ();
+		DCPOMATIC_ASSERT (_picture_asset->file());
+		boost::filesystem::path video_from = _picture_asset->file().get();
 		boost::filesystem::path video_to;
 		video_to /= _film->dir (_film->dcp_name());
-		video_to /= video_asset_filename (_picture_asset);
+		video_to /= video_asset_filename (_picture_asset, _reel_index, _reel_count, _content_summary);
 
 		boost::system::error_code ec;
 		boost::filesystem::create_hard_link (video_from, video_to, ec);
@@ -286,13 +326,14 @@ ReelWriter::finish ()
 	if (_sound_asset) {
 		boost::filesystem::path audio_to;
 		audio_to /= _film->dir (_film->dcp_name ());
-		audio_to /= audio_asset_filename (_sound_asset);
+		string const aaf = audio_asset_filename (_sound_asset, _reel_index, _reel_count, _content_summary);
+		audio_to /= aaf;
 
 		boost::system::error_code ec;
-		boost::filesystem::rename (_film->file (audio_asset_filename (_sound_asset)), audio_to, ec);
+		boost::filesystem::rename (_film->file (aaf), audio_to, ec);
 		if (ec) {
 			throw FileError (
-				String::compose (_("could not move audio asset into the DCP (%1)"), ec.value ()), audio_asset_filename (_sound_asset)
+				String::compose (_("could not move audio asset into the DCP (%1)"), ec.value ()), aaf
 				);
 		}
 
@@ -319,30 +360,48 @@ ReelWriter::create_reel (list<ReferencedReelAsset> const & refs, list<shared_ptr
 			reel_picture_asset.reset (new dcp::ReelStereoPictureAsset (stereo, 0));
 		}
 	} else {
+		LOG_GENERAL ("no picture asset of our own; look through %1", refs.size());
 		/* We don't have a picture asset of our own; hopefully we have one to reference */
 		BOOST_FOREACH (ReferencedReelAsset j, refs) {
 			shared_ptr<dcp::ReelPictureAsset> k = dynamic_pointer_cast<dcp::ReelPictureAsset> (j.asset);
+			if (k) {
+				LOG_GENERAL ("candidate picture asset period is %1-%2", j.period.from.get(), j.period.to.get());
+			}
 			if (k && j.period == _period) {
 				reel_picture_asset = k;
 			}
 		}
 	}
 
+	LOG_GENERAL ("create_reel for %1-%2; %3 of %4", _period.from.get(), _period.to.get(), _reel_index, _reel_count);
+
+	Frame const period_duration = _period.duration().frames_round(_film->video_frame_rate());
+
+	DCPOMATIC_ASSERT (reel_picture_asset);
+	if (reel_picture_asset->duration() != period_duration) {
+		throw ProgrammingError (
+			__FILE__, __LINE__,
+			String::compose ("%1 vs %2", reel_picture_asset->duration(), period_duration)
+			);
+	}
 	reel->add (reel_picture_asset);
+
 	/* If we have a hash for this asset in the CPL, assume that it is correct */
 	if (reel_picture_asset->hash()) {
 		reel_picture_asset->asset_ref()->set_hash (reel_picture_asset->hash().get());
 	}
 
+	shared_ptr<dcp::ReelSoundAsset> reel_sound_asset;
+
 	if (_sound_asset) {
 		/* We have made a sound asset of our own.  Put it into the reel */
-		reel->add (shared_ptr<dcp::ReelSoundAsset> (new dcp::ReelSoundAsset (_sound_asset, 0)));
+		reel_sound_asset.reset (new dcp::ReelSoundAsset (_sound_asset, 0));
 	} else {
 		/* We don't have a sound asset of our own; hopefully we have one to reference */
 		BOOST_FOREACH (ReferencedReelAsset j, refs) {
 			shared_ptr<dcp::ReelSoundAsset> k = dynamic_pointer_cast<dcp::ReelSoundAsset> (j.asset);
 			if (k && j.period == _period) {
-				reel->add (k);
+				reel_sound_asset = k;
 				/* If we have a hash for this asset in the CPL, assume that it is correct */
 				if (k->hash()) {
 					k->asset_ref()->set_hash (k->hash().get());
@@ -351,6 +410,25 @@ ReelWriter::create_reel (list<ReferencedReelAsset> const & refs, list<shared_ptr
 		}
 	}
 
+	DCPOMATIC_ASSERT (reel_sound_asset);
+	if (reel_sound_asset->duration() != period_duration) {
+		LOG_ERROR (
+			"Reel sound asset has length %1 but reel period is %2",
+			reel_sound_asset->duration(),
+			period_duration
+			);
+		if (reel_sound_asset->duration() != period_duration) {
+			throw ProgrammingError (
+				__FILE__, __LINE__,
+				String::compose ("%1 vs %2", reel_sound_asset->duration(), period_duration)
+				);
+		}
+
+	}
+	reel->add (reel_sound_asset);
+
+	shared_ptr<dcp::ReelSubtitleAsset> reel_subtitle_asset;
+
 	if (_subtitle_asset) {
 
 		boost::filesystem::path liberation_normal;
@@ -391,20 +469,20 @@ ReelWriter::create_reel (list<ReferencedReelAsset> const & refs, list<shared_ptr
 				);
 		}
 
-		reel->add (shared_ptr<dcp::ReelSubtitleAsset> (
-				   new dcp::ReelSubtitleAsset (
-					   _subtitle_asset,
-					   dcp::Fraction (_film->video_frame_rate(), 1),
-					   reel_picture_asset->intrinsic_duration (),
-					   0
-					   )
-				   ));
+		reel_subtitle_asset.reset (
+			new dcp::ReelSubtitleAsset (
+				_subtitle_asset,
+				dcp::Fraction (_film->video_frame_rate(), 1),
+				reel_picture_asset->intrinsic_duration (),
+				0
+				)
+			);
 	} else {
 		/* We don't have a subtitle asset of our own; hopefully we have one to reference */
 		BOOST_FOREACH (ReferencedReelAsset j, refs) {
 			shared_ptr<dcp::ReelSubtitleAsset> k = dynamic_pointer_cast<dcp::ReelSubtitleAsset> (j.asset);
 			if (k && j.period == _period) {
-				reel->add (k);
+				reel_subtitle_asset = k;
 				/* If we have a hash for this asset in the CPL, assume that it is correct */
 				if (k->hash()) {
 					k->asset_ref()->set_hash (k->hash().get());
@@ -413,20 +491,28 @@ ReelWriter::create_reel (list<ReferencedReelAsset> const & refs, list<shared_ptr
 		}
 	}
 
+	if (reel_subtitle_asset) {
+		if (reel_subtitle_asset->duration() != period_duration) {
+			throw ProgrammingError (
+				__FILE__, __LINE__,
+				String::compose ("%1 vs %2", reel_subtitle_asset->duration(), period_duration)
+				);
+		}
+		reel->add (reel_subtitle_asset);
+	}
+
 	return reel;
 }
 
 void
-ReelWriter::calculate_digests (shared_ptr<Job> job)
+ReelWriter::calculate_digests (boost::function<void (float)> set_progress)
 {
-	job->sub (_("Computing image digest"));
 	if (_picture_asset) {
-		_picture_asset->hash (boost::bind (&Job::set_progress, job.get(), _1, false));
+		_picture_asset->hash (set_progress);
 	}
 
 	if (_sound_asset) {
-		job->sub (_("Computing audio digest"));
-		_sound_asset->hash (boost::bind (&Job::set_progress, job.get(), _1, false));
+		_sound_asset->hash (set_progress);
 	}
 }
 
@@ -447,8 +533,6 @@ ReelWriter::write (shared_ptr<const AudioBuffers> audio)
 	if (audio) {
 		_sound_asset_writer->write (audio->data(), audio->frames());
 	}
-
-	++_total_written_audio_frames;
 }
 
 void
@@ -463,21 +547,24 @@ ReelWriter::write (PlayerSubtitles subs)
 			shared_ptr<dcp::InteropSubtitleAsset> s (new dcp::InteropSubtitleAsset ());
 			s->set_movie_title (_film->name ());
 			s->set_language (lang);
-			s->set_reel_number ("1");
+			s->set_reel_number (raw_convert<string> (_reel_index + 1));
 			_subtitle_asset = s;
 		} else {
 			shared_ptr<dcp::SMPTESubtitleAsset> s (new dcp::SMPTESubtitleAsset ());
 			s->set_content_title_text (_film->name ());
 			s->set_language (lang);
 			s->set_edit_rate (dcp::Fraction (_film->video_frame_rate (), 1));
-			s->set_reel_number (1);
+			s->set_reel_number (_reel_index + 1);
 			s->set_time_code_rate (_film->video_frame_rate ());
 			s->set_start_time (dcp::Time ());
+			if (_film->encrypted ()) {
+				s->set_key (_film->key ());
+			}
 			_subtitle_asset = s;
 		}
 	}
 
-	BOOST_FOREACH (dcp::SubtitleString i, subs.text) {
+	BOOST_FOREACH (SubtitleString i, subs.text) {
 		i.set_in  (i.in()  - dcp::Time (_period.from.seconds(), i.in().tcr));
 		i.set_out (i.out() - dcp::Time (_period.from.seconds(), i.out().tcr));
 		_subtitle_asset->add (i);
@@ -485,14 +572,14 @@ ReelWriter::write (PlayerSubtitles subs)
 }
 
 bool
-ReelWriter::existing_picture_frame_ok (FILE* asset_file, FILE* info_file) const
+ReelWriter::existing_picture_frame_ok (FILE* asset_file, FILE* info_file, Frame frame) const
 {
-	LOG_GENERAL ("Checking existing picture frame %1", _first_nonexistant_frame);
+	LOG_GENERAL ("Checking existing picture frame %1", frame);
 
 	/* Read the data from the info file; for 3D we just check the left
 	   frames until we find a good one.
 	*/
-	dcp::FrameInfo const info = read_frame_info (info_file, _first_nonexistant_frame, _film->three_d () ? EYES_LEFT : EYES_BOTH);
+	dcp::FrameInfo const info = read_frame_info (info_file, frame, _film->three_d () ? EYES_LEFT : EYES_BOTH);
 
 	bool ok = true;
 
@@ -502,14 +589,14 @@ ReelWriter::existing_picture_frame_ok (FILE* asset_file, FILE* info_file) const
 	size_t const read = fread (data.data().get(), 1, data.size(), asset_file);
 	LOG_GENERAL ("Read %1 bytes of asset data; wanted %2", read, info.size);
 	if (read != static_cast<size_t> (data.size ())) {
-		LOG_GENERAL ("Existing frame %1 is incomplete", _first_nonexistant_frame);
+		LOG_GENERAL ("Existing frame %1 is incomplete", frame);
 		ok = false;
 	} else {
 		Digester digester;
 		digester.add (data.data().get(), data.size());
 		LOG_GENERAL ("Hash %1 vs %2", digester.get(), info.hash);
 		if (digester.get() != info.hash) {
-			LOG_GENERAL ("Existing frame %1 failed hash check", _first_nonexistant_frame);
+			LOG_GENERAL ("Existing frame %1 failed hash check", frame);
 			ok = false;
 		}
 	}