Merge branch 'master' of ssh://carlh.dnsalias.org/home/carl/git/dvdomatic
[dcpomatic.git] / src / lib / film.cc
index ab636bdfcc3a651fe79cd1b87dd94d15693e93ed..6bc503dc1c3b4683df14ed09db1e13bc7e2ca684 100644 (file)
 #include "ui_signaller.h"
 #include "video_decoder.h"
 #include "audio_decoder.h"
-#include "external_audio_decoder.h"
+#include "sndfile_decoder.h"
 #include "analyse_audio_job.h"
+#include "cross.h"
+
+#include "i18n.h"
 
 using std::string;
 using std::stringstream;
@@ -64,6 +67,8 @@ using std::ofstream;
 using std::setfill;
 using std::min;
 using std::make_pair;
+using std::endl;
+using std::list;
 using boost::shared_ptr;
 using boost::lexical_cast;
 using boost::to_upper_copy;
@@ -72,7 +77,7 @@ using boost::starts_with;
 using boost::optional;
 using libdcp::Size;
 
-int const Film::state_version = 3;
+int const Film::state_version = 4;
 
 /** Construct a Film object in a given directory, reading any metadata
  *  file that exists in that directory.  An exception will be thrown if
@@ -85,11 +90,12 @@ int const Film::state_version = 3;
 Film::Film (string d, bool must_exist)
        : _use_dci_name (true)
        , _trust_content_header (true)
-       , _dcp_content_type (0)
-       , _format (0)
+       , _dcp_content_type (Config::instance()->default_dcp_content_type ())
+       , _format (Config::instance()->default_format ())
        , _scaler (Scaler::from_id ("bicubic"))
        , _trim_start (0)
        , _trim_end (0)
+       , _trim_type (CPL)
        , _dcp_ab (false)
        , _use_content_audio (true)
        , _audio_gain (0)
@@ -101,7 +107,8 @@ Film::Film (string d, bool must_exist)
        , _colour_lut (0)
        , _j2k_bandwidth (200000000)
        , _dci_metadata (Config::instance()->default_dci_metadata ())
-       , _frames_per_second (0)
+       , _dcp_frame_rate (0)
+       , _source_frame_rate (0)
        , _dirty (false)
 {
        set_dci_date_today ();
@@ -134,18 +141,21 @@ Film::Film (string d, bool must_exist)
                }
        }
 
-       _external_audio_stream = ExternalAudioStream::create ();
+       _sndfile_stream = SndfileStream::create ();
        
        if (must_exist) {
                read_metadata ();
+       } else {
+               write_metadata ();
        }
 
-       _log = new FileLog (file ("log"));
+       _log.reset (new FileLog (file ("log")));
 }
 
 Film::Film (Film const & o)
        : boost::enable_shared_from_this<Film> (o)
-       , _log (0)
+       /* note: the copied film shares the original's log */
+       , _log               (o._log)
        , _directory         (o._directory)
        , _name              (o._name)
        , _use_dci_name      (o._use_dci_name)
@@ -158,6 +168,7 @@ Film::Film (Film const & o)
        , _scaler            (o._scaler)
        , _trim_start        (o._trim_start)
        , _trim_end          (o._trim_end)
+       , _trim_type         (o._trim_type)
        , _dcp_ab            (o._dcp_ab)
        , _content_audio_stream (o._content_audio_stream)
        , _external_audio    (o._external_audio)
@@ -173,28 +184,29 @@ Film::Film (Film const & o)
        , _j2k_bandwidth     (o._j2k_bandwidth)
        , _dci_metadata      (o._dci_metadata)
        , _dci_date          (o._dci_date)
+       , _dcp_frame_rate    (o._dcp_frame_rate)
        , _size              (o._size)
        , _length            (o._length)
-       , _dcp_intrinsic_duration (o._dcp_intrinsic_duration)
        , _content_digest    (o._content_digest)
        , _content_audio_streams (o._content_audio_streams)
-       , _external_audio_stream (o._external_audio_stream)
+       , _sndfile_stream    (o._sndfile_stream)
        , _subtitle_streams  (o._subtitle_streams)
-       , _frames_per_second (o._frames_per_second)
+       , _source_frame_rate (o._source_frame_rate)
        , _dirty             (o._dirty)
 {
-
+       
 }
 
 Film::~Film ()
 {
-       delete _log;
+
 }
 
 string
 Film::video_state_identifier () const
 {
        assert (format ());
+       LocaleGuard lg;
 
        pair<string, string> f = Filter::ffmpeg_strings (filters());
 
@@ -202,11 +214,16 @@ Film::video_state_identifier () const
        s << format()->id()
          << "_" << content_digest()
          << "_" << crop().left << "_" << crop().right << "_" << crop().top << "_" << crop().bottom
+         << "_" << _dcp_frame_rate
          << "_" << f.first << "_" << f.second
          << "_" << scaler()->id()
          << "_" << j2k_bandwidth()
          << "_" << boost::lexical_cast<int> (colour_lut());
 
+       if (trim_type() == ENCODE) {
+               s << "_" << trim_start() << "_" << trim_end();
+       }
+
        if (dcp_ab()) {
                pair<string, string> fa = Filter::ffmpeg_strings (Config::instance()->reference_filters());
                s << "ab_" << Config::instance()->reference_scaler()->id() << "_" << fa.first << "_" << fa.second;
@@ -226,18 +243,46 @@ Film::info_dir () const
 }
 
 string
-Film::video_mxf_dir () const
+Film::internal_video_mxf_dir () const
 {
        boost::filesystem::path p;
        return dir ("video");
 }
 
 string
-Film::video_mxf_filename () const
+Film::internal_video_mxf_filename () const
 {
        return video_state_identifier() + ".mxf";
 }
 
+string
+Film::dcp_video_mxf_filename () const
+{
+       return filename_safe_name() + "_video.mxf";
+}
+
+string
+Film::dcp_audio_mxf_filename () const
+{
+       return filename_safe_name() + "_audio.mxf";
+}
+
+string
+Film::filename_safe_name () const
+{
+       string const n = name ();
+       string o;
+       for (size_t i = 0; i < n.length(); ++i) {
+               if (isalnum (n[i])) {
+                       o += n[i];
+               } else {
+                       o += "_";
+               }
+       }
+
+       return o;
+}
+
 string
 Film::audio_analysis_path () const
 {
@@ -254,7 +299,7 @@ Film::make_dcp ()
        set_dci_date_today ();
        
        if (dcp_name().find ("/") != string::npos) {
-               throw BadSettingError ("name", "cannot contain slashes");
+               throw BadSettingError (_("name"), _("cannot contain slashes"));
        }
        
        log()->log (String::compose ("DVD-o-matic %1 git %2 using %3", dvdomatic_version, dvdomatic_git_commit, dependency_version_summary()));
@@ -265,11 +310,12 @@ Film::make_dcp ()
                log()->log (String::compose ("Starting to make DCP on %1", buffer));
        }
        
-       log()->log (String::compose ("Content is %1; type %2", content_path(), (content_type() == STILL ? "still" : "video")));
+       log()->log (String::compose ("Content is %1; type %2", content_path(), (content_type() == STILL ? _("still") : _("video"))));
        if (length()) {
                log()->log (String::compose ("Content length %1", length().get()));
        }
        log()->log (String::compose ("Content digest %1", content_digest()));
+       log()->log (String::compose ("Content at %1 fps, DCP at %2 fps", source_frame_rate(), dcp_frame_rate()));
        log()->log (String::compose ("%1 threads", Config::instance()->num_local_encoding_threads()));
        log()->log (String::compose ("J2K bandwidth %1", j2k_bandwidth()));
 #ifdef DVDOMATIC_DEBUG
@@ -284,21 +330,25 @@ Film::make_dcp ()
 #endif
        pair<string, int> const c = cpu_info ();
        log()->log (String::compose ("CPU: %1, %2 processors", c.first, c.second));
+       list<pair<string, string> > const m = mount_info ();
+       for (list<pair<string, string> >::const_iterator i = m.begin(); i != m.end(); ++i) {
+               log()->log (String::compose ("Mount: %1 %2", i->first, i->second));
+       }
        
        if (format() == 0) {
-               throw MissingSettingError ("format");
+               throw MissingSettingError (_("format"));
        }
 
        if (content().empty ()) {
-               throw MissingSettingError ("content");
+               throw MissingSettingError (_("content"));
        }
 
        if (dcp_content_type() == 0) {
-               throw MissingSettingError ("content type");
+               throw MissingSettingError (_("content type"));
        }
 
        if (name().empty()) {
-               throw MissingSettingError ("name");
+               throw MissingSettingError (_("name"));
        }
 
        DecodeOptions od;
@@ -343,9 +393,12 @@ void
 Film::analyse_audio_finished ()
 {
        ensure_ui_thread ();
-       _analyse_audio_job.reset ();
 
-       AudioAnalysisFinished ();
+       if (_analyse_audio_job->finished_ok ()) {
+               AudioAnalysisSucceeded ();
+       }
+       
+       _analyse_audio_job.reset ();
 }
 
 void
@@ -386,6 +439,7 @@ void
 Film::write_metadata () const
 {
        boost::mutex::scoped_lock lm (_state_mutex);
+       LocaleGuard lg;
 
        boost::filesystem::create_directories (directory());
 
@@ -395,67 +449,75 @@ Film::write_metadata () const
                throw CreateFileError (m);
        }
 
-       f << "version " << state_version << "\n";
+       f << "version " << state_version << endl;
 
        /* User stuff */
-       f << "name " << _name << "\n";
-       f << "use_dci_name " << _use_dci_name << "\n";
-       f << "content " << _content << "\n";
-       f << "trust_content_header " << (_trust_content_header ? "1" : "0") << "\n";
+       f << "name " << _name << endl;
+       f << "use_dci_name " << _use_dci_name << endl;
+       f << "content " << _content << endl;
+       f << "trust_content_header " << (_trust_content_header ? "1" : "0") << endl;
        if (_dcp_content_type) {
-               f << "dcp_content_type " << _dcp_content_type->dci_name () << "\n";
+               f << "dcp_content_type " << _dcp_content_type->dci_name () << endl;
        }
        if (_format) {
-               f << "format " << _format->as_metadata () << "\n";
+               f << "format " << _format->as_metadata () << endl;
        }
-       f << "left_crop " << _crop.left << "\n";
-       f << "right_crop " << _crop.right << "\n";
-       f << "top_crop " << _crop.top << "\n";
-       f << "bottom_crop " << _crop.bottom << "\n";
+       f << "left_crop " << _crop.left << endl;
+       f << "right_crop " << _crop.right << endl;
+       f << "top_crop " << _crop.top << endl;
+       f << "bottom_crop " << _crop.bottom << endl;
        for (vector<Filter const *>::const_iterator i = _filters.begin(); i != _filters.end(); ++i) {
-               f << "filter " << (*i)->id () << "\n";
+               f << "filter " << (*i)->id () << endl;
+       }
+       f << "scaler " << _scaler->id () << endl;
+       f << "trim_start " << _trim_start << endl;
+       f << "trim_end " << _trim_end << endl;
+       switch (_trim_type) {
+       case CPL:
+               f << "trim_type cpl\n";
+               break;
+       case ENCODE:
+               f << "trim_type encode\n";
+               break;
        }
-       f << "scaler " << _scaler->id () << "\n";
-       f << "trim_start " << _trim_start << "\n";
-       f << "trim_end " << _trim_end << "\n";
-       f << "dcp_ab " << (_dcp_ab ? "1" : "0") << "\n";
+       f << "dcp_ab " << (_dcp_ab ? "1" : "0") << endl;
        if (_content_audio_stream) {
-               f << "selected_content_audio_stream " << _content_audio_stream->to_string() << "\n";
+               f << "selected_content_audio_stream " << _content_audio_stream->to_string() << endl;
        }
        for (vector<string>::const_iterator i = _external_audio.begin(); i != _external_audio.end(); ++i) {
-               f << "external_audio " << *i << "\n";
+               f << "external_audio " << *i << endl;
        }
-       f << "use_content_audio " << (_use_content_audio ? "1" : "0") << "\n";
-       f << "audio_gain " << _audio_gain << "\n";
-       f << "audio_delay " << _audio_delay << "\n";
-       f << "still_duration " << _still_duration << "\n";
+       f << "use_content_audio " << (_use_content_audio ? "1" : "0") << endl;
+       f << "audio_gain " << _audio_gain << endl;
+       f << "audio_delay " << _audio_delay << endl;
+       f << "still_duration " << _still_duration << endl;
        if (_subtitle_stream) {
-               f << "selected_subtitle_stream " << _subtitle_stream->to_string() << "\n";
+               f << "selected_subtitle_stream " << _subtitle_stream->to_string() << endl;
        }
-       f << "with_subtitles " << _with_subtitles << "\n";
-       f << "subtitle_offset " << _subtitle_offset << "\n";
-       f << "subtitle_scale " << _subtitle_scale << "\n";
-       f << "colour_lut " << _colour_lut << "\n";
-       f << "j2k_bandwidth " << _j2k_bandwidth << "\n";
+       f << "with_subtitles " << _with_subtitles << endl;
+       f << "subtitle_offset " << _subtitle_offset << endl;
+       f << "subtitle_scale " << _subtitle_scale << endl;
+       f << "colour_lut " << _colour_lut << endl;
+       f << "j2k_bandwidth " << _j2k_bandwidth << endl;
        _dci_metadata.write (f);
-       f << "dci_date " << boost::gregorian::to_iso_string (_dci_date) << "\n";
-       f << "width " << _size.width << "\n";
-       f << "height " << _size.height << "\n";
-       f << "length " << _length.get_value_or(0) << "\n";
-       f << "dcp_intrinsic_duration " << _dcp_intrinsic_duration.get_value_or(0) << "\n";
-       f << "content_digest " << _content_digest << "\n";
+       f << "dci_date " << boost::gregorian::to_iso_string (_dci_date) << endl;
+       f << "dcp_frame_rate " << _dcp_frame_rate << endl;
+       f << "width " << _size.width << endl;
+       f << "height " << _size.height << endl;
+       f << "length " << _length.get_value_or(0) << endl;
+       f << "content_digest " << _content_digest << endl;
 
        for (vector<shared_ptr<AudioStream> >::const_iterator i = _content_audio_streams.begin(); i != _content_audio_streams.end(); ++i) {
-               f << "content_audio_stream " << (*i)->to_string () << "\n";
+               f << "content_audio_stream " << (*i)->to_string () << endl;
        }
 
-       f << "external_audio_stream " << _external_audio_stream->to_string() << "\n";
+       f << "external_audio_stream " << _sndfile_stream->to_string() << endl;
 
        for (vector<shared_ptr<SubtitleStream> >::const_iterator i = _subtitle_streams.begin(); i != _subtitle_streams.end(); ++i) {
-               f << "subtitle_stream " << (*i)->to_string () << "\n";
+               f << "subtitle_stream " << (*i)->to_string () << endl;
        }
 
-       f << "frames_per_second " << _frames_per_second << "\n";
+       f << "source_frame_rate " << _source_frame_rate << endl;
        
        _dirty = false;
 }
@@ -465,6 +527,7 @@ void
 Film::read_metadata ()
 {
        boost::mutex::scoped_lock lm (_state_mutex);
+       LocaleGuard lg;
 
        _external_audio.clear ();
        _content_audio_streams.clear ();
@@ -531,6 +594,12 @@ Film::read_metadata ()
                        _trim_start = atoi (v.c_str ());
                } else if ( ((!version || version < 2) && k == "dcp_trim_end") || k == "trim_end") {
                        _trim_end = atoi (v.c_str ());
+               } else if (k == "trim_type") {
+                       if (v == "cpl") {
+                               _trim_type = CPL;
+                       } else if (v == "encode") {
+                               _trim_type = ENCODE;
+                       }
                } else if (k == "dcp_ab") {
                        _dcp_ab = (v == "1");
                } else if (k == "selected_content_audio_stream" || (!version && k == "selected_audio_stream")) {
@@ -567,6 +636,8 @@ Film::read_metadata ()
                        _j2k_bandwidth = atoi (v.c_str ());
                } else if (k == "dci_date") {
                        _dci_date = boost::gregorian::from_undelimited_string (v);
+               } else if (k == "dcp_frame_rate") {
+                       _dcp_frame_rate = atoi (v.c_str ());
                }
 
                _dci_metadata.read (k, v);
@@ -581,21 +652,20 @@ Film::read_metadata ()
                        if (vv) {
                                _length = vv;
                        }
-               } else if (k == "dcp_intrinsic_duration") {
-                       int const vv = atoi (v.c_str ());
-                       if (vv) {
-                               _dcp_intrinsic_duration = vv;
-                       }
                } else if (k == "content_digest") {
                        _content_digest = v;
                } else if (k == "content_audio_stream" || (!version && k == "audio_stream")) {
                        _content_audio_streams.push_back (audio_stream_factory (v, version));
                } else if (k == "external_audio_stream") {
-                       _external_audio_stream = audio_stream_factory (v, version);
+                       _sndfile_stream = audio_stream_factory (v, version);
                } else if (k == "subtitle_stream") {
                        _subtitle_streams.push_back (subtitle_stream_factory (v, version));
-               } else if (k == "frames_per_second") {
-                       _frames_per_second = atof (v.c_str ());
+               } else if (k == "source_frame_rate") {
+                       _source_frame_rate = atof (v.c_str ());
+               } else if (version < 4 && k == "frames_per_second") {
+                       _source_frame_rate = atof (v.c_str ());
+                       /* Fill in what would have been used for DCP frame rate by the older version */
+                       _dcp_frame_rate = best_dcp_frame_rate (_source_frame_rate);
                }
        }
 
@@ -705,7 +775,7 @@ Film::target_audio_sample_rate () const
        /* Resample to a DCI-approved sample rate */
        double t = dcp_audio_sample_rate (audio_stream()->sample_rate());
 
-       DCPFrameRate dfr (frames_per_second ());
+       FrameRateConversion frc (source_frame_rate(), dcp_frame_rate());
 
        /* Compensate if the DCP is being run at a different frame rate
           to the source; that is, if the video is run such that it will
@@ -713,8 +783,8 @@ Film::target_audio_sample_rate () const
           skip/repeat doesn't come into effect here.
        */
 
-       if (dfr.change_speed) {
-               t *= _frames_per_second * dfr.factor() / dfr.frames_per_second;
+       if (frc.change_speed) {
+               t *= source_frame_rate() * frc.factor() / dcp_frame_rate();
        }
 
        return rint (t);
@@ -723,7 +793,7 @@ Film::target_audio_sample_rate () const
 int
 Film::still_duration_in_frames () const
 {
-       return still_duration() * frames_per_second();
+       return still_duration() * source_frame_rate();
 }
 
 /** @return a DCI-compliant name for a DCP of this film */
@@ -744,70 +814,67 @@ Film::dci_name (bool if_created_now) const
                fixed_name = fixed_name.substr (0, 14);
        }
 
-       d << fixed_name << "_";
+       d << fixed_name;
 
        if (dcp_content_type()) {
-               d << dcp_content_type()->dci_name() << "_";
+               d << "_" << dcp_content_type()->dci_name();
        }
 
        if (format()) {
-               d << format()->dci_name() << "_";
+               d << "_" << format()->dci_name();
        }
 
        DCIMetadata const dm = dci_metadata ();
 
        if (!dm.audio_language.empty ()) {
-               d << dm.audio_language;
-               if (!dm.subtitle_language.empty() && with_subtitles()) {
+               d << "_" << dm.audio_language;
+               if (!dm.subtitle_language.empty()) {
                        d << "-" << dm.subtitle_language;
                } else {
                        d << "-XX";
                }
-                       
-               d << "_";
        }
 
        if (!dm.territory.empty ()) {
-               d << dm.territory;
+               d << "_" << dm.territory;
                if (!dm.rating.empty ()) {
                        d << "-" << dm.rating;
                }
-               d << "_";
        }
 
        switch (audio_channels()) {
        case 1:
-               d << "10_";
+               d << "_10";
                break;
        case 2:
-               d << "20_";
+               d << "_20";
                break;
        case 6:
-               d << "51_";
+               d << "_51";
                break;
        case 8:
-               d << "71_";
+               d << "_71";
                break;
        }
 
-       d << "2K_";
+       d << "_2K";
 
        if (!dm.studio.empty ()) {
-               d << dm.studio << "_";
+               d << "_" << dm.studio;
        }
 
        if (if_created_now) {
-               d << boost::gregorian::to_iso_string (boost::gregorian::day_clock::local_day ()) << "_";
+               d << "_" << boost::gregorian::to_iso_string (boost::gregorian::day_clock::local_day ());
        } else {
-               d << boost::gregorian::to_iso_string (_dci_date) << "_";
+               d << "_" << boost::gregorian::to_iso_string (_dci_date);
        }
 
        if (!dm.facility.empty ()) {
-               d << dm.facility << "_";
+               d << "_" << dm.facility;
        }
 
        if (!dm.package_type.empty ()) {
-               d << dm.package_type;
+               d << "_" << dm.package_type;
        }
 
        return d.str ();
@@ -881,6 +948,9 @@ Film::set_content (string c)
                _content = c;
        }
 
+       /* Do this before we start using FFmpeg ourselves */
+       run_ffprobe (c, file ("ffprobe.log"), _log);
+       
        /* Reset streams here in case the new content doesn't have one or the other */
        _content_audio_stream = shared_ptr<AudioStream> ();
        _subtitle_stream = shared_ptr<SubtitleStream> ();
@@ -896,7 +966,8 @@ Film::set_content (string c)
                Decoders d = decoder_factory (shared_from_this(), DecodeOptions());
                
                set_size (d.video->native_size ());
-               set_frames_per_second (d.video->frames_per_second ());
+               set_source_frame_rate (d.video->frames_per_second ());
+               set_dcp_frame_rate (best_dcp_frame_rate (source_frame_rate ()));
                set_subtitle_streams (d.video->subtitle_streams ());
                if (d.audio) {
                        set_content_audio_streams (d.audio->audio_streams ());
@@ -929,14 +1000,7 @@ Film::set_content (string c)
        }
 
        /* Default format */
-       switch (content_type()) {
-       case STILL:
-               set_format (Format::from_id ("var-185"));
-               break;
-       case VIDEO:
-               set_format (Format::from_id ("185"));
-               break;
-       }
+       set_format (Config::instance()->default_format ());
 
        /* Still image DCPs must use external audio */
        if (content_type() == STILL) {
@@ -1087,6 +1151,16 @@ Film::set_trim_end (int t)
        signal_changed (TRIM_END);
 }
 
+void
+Film::set_trim_type (TrimType t)
+{
+       {
+               boost::mutex::scoped_lock lm (_state_mutex);
+               _trim_type = t;
+       }
+       signal_changed (TRIM_TYPE);
+}
+
 void
 Film::set_dcp_ab (bool a)
 {
@@ -1115,9 +1189,9 @@ Film::set_external_audio (vector<string> a)
                _external_audio = a;
        }
 
-       shared_ptr<ExternalAudioDecoder> decoder (new ExternalAudioDecoder (shared_from_this(), DecodeOptions()));
+       shared_ptr<SndfileDecoder> decoder (new SndfileDecoder (shared_from_this(), DecodeOptions()));
        if (decoder->audio_stream()) {
-               _external_audio_stream = decoder->audio_stream ();
+               _sndfile_stream = decoder->audio_stream ();
        }
        
        signal_changed (EXTERNAL_AUDIO);
@@ -1234,6 +1308,17 @@ Film::set_dci_metadata (DCIMetadata m)
        signal_changed (DCI_METADATA);
 }
 
+
+void
+Film::set_dcp_frame_rate (int f)
+{
+       {
+               boost::mutex::scoped_lock lm (_state_mutex);
+               _dcp_frame_rate = f;
+       }
+       signal_changed (DCP_FRAME_RATE);
+}
+
 void
 Film::set_size (libdcp::Size s)
 {
@@ -1264,16 +1349,6 @@ Film::unset_length ()
        signal_changed (LENGTH);
 }
 
-void
-Film::set_dcp_intrinsic_duration (int d)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _dcp_intrinsic_duration = d;
-       }
-       signal_changed (DCP_INTRINSIC_DURATION);
-}
-
 void
 Film::set_content_digest (string d)
 {
@@ -1305,13 +1380,13 @@ Film::set_subtitle_streams (vector<shared_ptr<SubtitleStream> > s)
 }
 
 void
-Film::set_frames_per_second (float f)
+Film::set_source_frame_rate (float f)
 {
        {
                boost::mutex::scoped_lock lm (_state_mutex);
-               _frames_per_second = f;
+               _source_frame_rate = f;
        }
-       signal_changed (FRAMES_PER_SECOND);
+       signal_changed (SOURCE_FRAME_RATE);
 }
        
 void
@@ -1351,7 +1426,7 @@ Film::audio_stream () const
                return _content_audio_stream;
        }
 
-       return _external_audio_stream;
+       return _sndfile_stream;
 }
 
 string
@@ -1408,3 +1483,21 @@ Film::have_dcp () const
 
        return true;
 }
+
+bool
+Film::has_audio () const
+{
+       if (use_content_audio()) {
+               return audio_stream();
+       }
+
+       vector<string> const e = external_audio ();
+       for (vector<string>::const_iterator i = e.begin(); i != e.end(); ++i) {
+               if (!i->empty ()) {
+                       return true;
+               }
+       }
+
+       return false;
+}
+