Interop support.

4 files changed, 32 insertions, 7 deletions

diff --git a/src/lib/film.cc b/src/lib/film.cc
index 5dc808c02..e6bf3bd5e 100644
--- a/src/lib/film.cc
+++ b/src/lib/film.cc
@@ -97,6 +97,7 @@ Film::Film (boost::filesystem::path dir)
 	, _audio_channels (MAX_AUDIO_CHANNELS)
 	, _three_d (false)
 	, _sequence_video (true)
+	, _interop (false)
 	, _dirty (false)
 {
 	set_dci_date_today ();
@@ -142,6 +143,12 @@ Film::video_identifier () const
 	  << "_" << scaler()->id()
 	  << "_" << j2k_bandwidth();
 
+	if (_interop) {
+		s << "_I";
+	} else {
+		s << "_S";
+	}
+
 	if (_three_d) {
 		s << "_3D";
 	}
@@ -331,6 +338,7 @@ Film::write_metadata () const
 	root->add_child("AudioChannels")->add_child_text (lexical_cast<string> (_audio_channels));
 	root->add_child("ThreeD")->add_child_text (_three_d ? "1" : "0");
 	root->add_child("SequenceVideo")->add_child_text (_sequence_video ? "1" : "0");
+	root->add_child("Interop")->add_child_text (_interop ? "1" : "0");
 	_playlist->as_xml (root->add_child ("Playlist"));
 
 	doc.write_to_file_formatted (file ("metadata.xml"));
@@ -377,6 +385,7 @@ Film::read_metadata ()
 	_audio_channels = f.number_child<int> ("AudioChannels");
 	_sequence_video = f.bool_child ("SequenceVideo");
 	_three_d = f.bool_child ("ThreeD");
+	_interop = f.bool_child ("Interop");
 
 	_playlist->set_from_xml (shared_from_this(), f.node_child ("Playlist"));
 
@@ -615,6 +624,13 @@ Film::set_three_d (bool t)
 }
 
 void
+Film::set_interop (bool i)
+{
+	_interop = i;
+	signal_changed (INTEROP);
+}
+
+void
 Film::signal_changed (Property p)
 {
 	_dirty = true;
diff --git a/src/lib/film.h b/src/lib/film.h
index df26a3ae5..f5b29466a 100644
--- a/src/lib/film.h
+++ b/src/lib/film.h
@@ -134,6 +134,7 @@ public:
 		/** The setting of _three_d has been changed */
 		THREE_D,
 		SEQUENCE_VIDEO,
+		INTEROP,
 	};
 
 
@@ -195,6 +196,10 @@ public:
 	bool sequence_video () const {
 		return _sequence_video;
 	}
+
+	bool interop () const {
+		return _interop;
+	}
 	
 
 	/* SET */
@@ -217,6 +222,7 @@ public:
 	void set_three_d (bool);
 	void set_dci_date_today ();
 	void set_sequence_video (bool);
+	void set_interop (bool);
 
 	/** Emitted when some property has of the Film has changed */
 	mutable boost::signals2::signal<void (Property)> Changed;
@@ -274,6 +280,7 @@ private:
 	*/
 	bool _three_d;
 	bool _sequence_video;
+	bool _interop;
 
 	/** true if our state has changed since we last saved it */
 	mutable bool _dirty;
diff --git a/src/lib/still_image_decoder.cc b/src/lib/still_image_decoder.cc
index 1dd684639..6e82f9a55 100644
--- a/src/lib/still_image_decoder.cc
+++ b/src/lib/still_image_decoder.cc
@@ -55,18 +55,20 @@ StillImageDecoder::pass ()
 	Magick::Image* magick_image = new Magick::Image (_still_image_content->path().string ());
 	_video_size = libdcp::Size (magick_image->columns(), magick_image->rows());
 	
-	_image.reset (new Image (PIX_FMT_RGB24, _video_size.get(), false));
+	_image.reset (new Image (PIX_FMT_RGB24, _video_size.get(), true));
 
 	using namespace MagickCore;
 	
 	uint8_t* p = _image->data()[0];
 	for (int y = 0; y < _video_size->height; ++y) {
+		uint8_t* q = p;
 		for (int x = 0; x < _video_size->width; ++x) {
 			Magick::Color c = magick_image->pixelColor (x, y);
-			*p++ = c.redQuantum() * 255 / QuantumRange;
-			*p++ = c.greenQuantum() * 255 / QuantumRange;
-			*p++ = c.blueQuantum() * 255 / QuantumRange;
+			*q++ = c.redQuantum() * 255 / QuantumRange;
+			*q++ = c.greenQuantum() * 255 / QuantumRange;
+			*q++ = c.blueQuantum() * 255 / QuantumRange;
 		}
+		p += _image->stride()[0];
 	}
 
 	delete magick_image;
diff --git a/src/lib/writer.cc b/src/lib/writer.cc
index 0d66df3e3..2e0ffd833 100644
--- a/src/lib/writer.cc
+++ b/src/lib/writer.cc
@@ -95,7 +95,7 @@ Writer::Writer (shared_ptr<const Film> f, shared_ptr<Job> j)
 
 	}
 
-	_picture_asset_writer = _picture_asset->start_write (_first_nonexistant_frame > 0);
+	_picture_asset_writer = _picture_asset->start_write (_first_nonexistant_frame > 0, _film->interop ());
 	
 	_sound_asset.reset (
 		new libdcp::SoundAsset (
@@ -107,7 +107,7 @@ Writer::Writer (shared_ptr<const Film> f, shared_ptr<Job> j)
 			)
 		);
 	
-	_sound_asset_writer = _sound_asset->start_write ();
+	_sound_asset_writer = _sound_asset->start_write (_film->interop ());
 
 	_thread = new boost::thread (boost::bind (&Writer::thread, this));
 }
@@ -391,7 +391,7 @@ Writer::finish ()
 
 	libdcp::XMLMetadata meta = Config::instance()->dcp_metadata ();
 	meta.set_issue_date_now ();
-	dcp.write_xml (meta);
+	dcp.write_xml (_film->interop (), meta);
 
 	_film->log()->log (String::compose (N_("Wrote %1 FULL, %2 FAKE, %3 REPEAT; %4 pushed to disk"), _full_written, _fake_written, _repeat_written, _pushed_to_disk));
 }