X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Ftranscoder.cc;h=48cf402d77bac4f36ababbac36adff4fb880d5e9;hb=003de48cd0371a60d095dc9d02ed5763c410cf5e;hp=3beda2b8b31a7625d6e45efdf614c789bb0ff250;hpb=18614dda0d53b713ace5ad1df57298d049dba87f;p=dcpomatic.git

diff --git a/src/lib/transcoder.cc b/src/lib/transcoder.cc
index 3beda2b8b..48cf402d7 100644
--- a/src/lib/transcoder.cc
+++ b/src/lib/transcoder.cc
@@ -36,9 +36,9 @@
 #include "gain.h"
 #include "video_decoder.h"
 #include "audio_decoder.h"
+#include "trimmer.h"
 
 using std::string;
-using std::cout;
 using boost::shared_ptr;
 using boost::dynamic_pointer_cast;
 
@@ -55,33 +55,45 @@ Transcoder::Transcoder (shared_ptr<Film> f, DecodeOptions o, Job* j, shared_ptr<
 {
 	assert (_encoder);
 
-	if (f->audio_stream()) {
-		shared_ptr<AudioStream> st = f->audio_stream();
+	shared_ptr<AudioStream> st = f->audio_stream();
+	if (st && st->sample_rate ()) {
 		_matcher.reset (new Matcher (f->log(), st->sample_rate(), f->source_frame_rate()));
-		_delay_line.reset (new DelayLine (f->log(), st->channels(), f->audio_delay() * st->sample_rate() / 1000));
-		_gain.reset (new Gain (f->log(), f->audio_gain()));
 	}
+	_delay_line.reset (new DelayLine (f->log(), f->audio_delay() / 1000.0f));
+	_gain.reset (new Gain (f->log(), f->audio_gain()));
+
+	int const sr = st ? st->sample_rate() : 0;
+	int const trim_start = f->trim_type() == Film::ENCODE ? f->trim_start() : 0;
+	int const trim_end = f->trim_type() == Film::ENCODE ? f->trim_end() : 0;
+	_trimmer.reset (new Trimmer (
+				f->log(), trim_start, trim_end, f->length().get_value_or(0),
+				sr, f->source_frame_rate(), f->dcp_frame_rate()
+				));
 
 	/* Set up the decoder to use the film's set streams */
 	_decoders.video->set_subtitle_stream (f->subtitle_stream ());
-	if (_decoders.audio) {
-		_decoders.audio->set_audio_stream (f->audio_stream ());
+	if (f->audio_stream ()) {
+	    _decoders.audio->set_audio_stream (f->audio_stream ());
 	}
 
+	_decoders.video->connect_video (_delay_line);
 	if (_matcher) {
-		_decoders.video->connect_video (_matcher);
-		_matcher->connect_video (_encoder);
+		_delay_line->connect_video (_matcher);
+		_matcher->connect_video (_trimmer);
 	} else {
-		/* Discard timestamps here */
-		_decoders.video->Video.connect (boost::bind (&Encoder::process_video, _encoder, _1, _2, _3));
+		_delay_line->connect_video (_trimmer);
 	}
+	_trimmer->connect_video (_encoder);
 	
-	if (_matcher && _delay_line && _decoders.audio) {
-		_decoders.audio->connect_audio (_delay_line);
+	_decoders.audio->connect_audio (_delay_line);
+	if (_matcher) {
 		_delay_line->connect_audio (_matcher);
 		_matcher->connect_audio (_gain);
-		_gain->connect_audio (_encoder);
+	} else {
+		_delay_line->connect_audio (_gain);
 	}
+	_gain->connect_audio (_trimmer);
+	_trimmer->connect_audio (_encoder);
 }
 
 /** Run the decoder, passing its output to the encoder, until the decoder
@@ -91,41 +103,32 @@ void
 Transcoder::go ()
 {
 	_encoder->process_begin ();
-	try {
-		bool done[2] = { false, false };
-		
-		while (1) {
-			if (!done[0]) {
-				done[0] = _decoders.video->pass ();
-				if (_job) {
-					_decoders.video->set_progress (_job);
-				}
-			}
-
-			if (!done[1] && _decoders.audio && dynamic_pointer_cast<Decoder> (_decoders.audio) != dynamic_pointer_cast<Decoder> (_decoders.video)) {
-				done[1] = _decoders.audio->pass ();
-			} else {
-				done[1] = true;
-			}
 
-			if (done[0] && done[1]) {
-				break;
+	bool done[2] = { false, false };
+	
+	while (1) {
+		if (!done[0]) {
+			done[0] = _decoders.video->pass ();
+			if (_job) {
+				_decoders.video->set_progress (_job);
 			}
 		}
 		
-	} catch (...) {
-		_encoder->process_end ();
-		throw;
+		if (!done[1] && _decoders.audio && dynamic_pointer_cast<Decoder> (_decoders.audio) != dynamic_pointer_cast<Decoder> (_decoders.video)) {
+			done[1] = _decoders.audio->pass ();
+		} else {
+			done[1] = true;
+		}
+		
+		if (done[0] && done[1]) {
+			break;
+		}
 	}
 	
-	if (_delay_line) {
-		_delay_line->process_end ();
-	}
+	_delay_line->process_end ();
 	if (_matcher) {
 		_matcher->process_end ();
 	}
-	if (_gain) {
-		_gain->process_end ();
-	}
+	_gain->process_end ();
 	_encoder->process_end ();
 }