Numerous fixes to A/B mode so that at least it doesn't crash (#72).

[dcpomatic.git] / src / lib / transcoder.cc

diff --git a/src/lib/transcoder.cc b/src/lib/transcoder.cc
index 114803fb533892d020a03b64f18b6152cbe08eb0..e0f3a03a2319c02de63918a6201647c48fe478cc 100644 (file)
--- a/src/lib/transcoder.cc
+++ b/src/lib/transcoder.cc
@@ -38,33 +38,34 @@
 #include "audio_decoder.h"
 
 using std::string;
-using std::cout;
 using boost::shared_ptr;
 using boost::dynamic_pointer_cast;
 
 /** Construct a transcoder using a Decoder that we create and a supplied Encoder.
  *  @param f Film that we are transcoding.
- *  @param o Options.
+ *  @param o Decode options.
  *  @param j Job that we are running under, or 0.
  *  @param e Encoder to use.
  */
-Transcoder::Transcoder (shared_ptr<Film> f, shared_ptr<const Options> o, Job* j, shared_ptr<Encoder> e)
+Transcoder::Transcoder (shared_ptr<Film> f, DecodeOptions o, Job* j, shared_ptr<Encoder> e)
        : _job (j)
        , _encoder (e)
-       , _decoders (decoder_factory (f, o, j))
+       , _decoders (decoder_factory (f, o))
 {
        assert (_encoder);
 
        if (f->audio_stream()) {
                shared_ptr<AudioStream> st = f->audio_stream();
-               _matcher.reset (new Matcher (f->log(), st->sample_rate(), f->frames_per_second()));
+               _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()));
        }
 
        /* Set up the decoder to use the film's set streams */
        _decoders.video->set_subtitle_stream (f->subtitle_stream ());
-       _decoders.audio->set_audio_stream (f->audio_stream ());
+       if (_decoders.audio) {
+               _decoders.audio->set_audio_stream (f->audio_stream ());
+       }
 
        if (_matcher) {
                _decoders.video->connect_video (_matcher);
@@ -73,7 +74,7 @@ Transcoder::Transcoder (shared_ptr<Film> f, shared_ptr<const Options> o, Job* j,
                _decoders.video->connect_video (_encoder);
        }
        
-       if (_matcher && _delay_line) {
+       if (_matcher && _delay_line && _decoders.audio) {
                _decoders.audio->connect_audio (_delay_line);
                _delay_line->connect_audio (_matcher);
                _matcher->connect_audio (_gain);
@@ -94,10 +95,12 @@ Transcoder::go ()
                while (1) {
                        if (!done[0]) {
                                done[0] = _decoders.video->pass ();
-                               _decoders.video->set_progress ();
+                               if (_job) {
+                                       _decoders.video->set_progress (_job);
+                               }
                        }
 
-                       if (!done[1] && dynamic_pointer_cast<Decoder> (_decoders.audio) != dynamic_pointer_cast<Decoder> (_decoders.video)) {
+                       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;