Merge master.
[dcpomatic.git] / src / lib / film.cc
index 19e900784775277c1475fac5ac4232481e18cc88..75ec700e0092e5bb35487f43556e4c93b1600b4e 100644 (file)
 #include <libxml++/libxml++.h>
 #include <libcxml/cxml.h>
 #include "film.h"
-#include "format.h"
 #include "job.h"
 #include "filter.h"
-#include "transcoder.h"
 #include "util.h"
 #include "job_manager.h"
 #include "ab_transcode_job.h"
 #include "config.h"
 #include "version.h"
 #include "ui_signaller.h"
-#include "video_decoder.h"
-#include "audio_decoder.h"
-#include "sndfile_decoder.h"
 #include "analyse_audio_job.h"
 #include "playlist.h"
+#include "player.h"
 #include "ffmpeg_content.h"
 #include "imagemagick_content.h"
 #include "sndfile_content.h"
+#include "dcp_content_type.h"
+#include "ratio.h"
+#include "cross.h"
 
 #include "i18n.h"
 
@@ -71,9 +70,11 @@ using std::setfill;
 using std::min;
 using std::make_pair;
 using std::endl;
+using std::cout;
 using std::list;
 using boost::shared_ptr;
 using boost::lexical_cast;
+using boost::dynamic_pointer_cast;
 using boost::to_upper_copy;
 using boost::ends_with;
 using boost::starts_with;
@@ -82,36 +83,32 @@ using libdcp::Size;
 
 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
- *  must_exist is true and the specified directory does not exist.
+/** Construct a Film object in a given directory.
  *
  *  @param d Film directory.
- *  @param must_exist true to throw an exception if does not exist.
  */
 
-Film::Film (string d, bool must_exist)
+Film::Film (string d)
        : _playlist (new Playlist)
        , _use_dci_name (true)
-       , _trust_content_headers (true)
-       , _dcp_content_type (0)
-       , _format (0)
+       , _dcp_content_type (Config::instance()->default_dcp_content_type ())
+       , _container (Config::instance()->default_container ())
        , _scaler (Scaler::from_id ("bicubic"))
-       , _trim_start (0)
-       , _trim_end (0)
        , _ab (false)
-       , _audio_gain (0)
-       , _audio_delay (0)
        , _with_subtitles (false)
        , _subtitle_offset (0)
        , _subtitle_scale (1)
        , _colour_lut (0)
        , _j2k_bandwidth (200000000)
        , _dci_metadata (Config::instance()->default_dci_metadata ())
-       , _dcp_frame_rate (0)
+       , _dcp_video_frame_rate (0)
+       , _dcp_audio_channels (MAX_AUDIO_CHANNELS)
        , _dirty (false)
 {
        set_dci_date_today ();
+
+       _playlist->Changed.connect (bind (&Film::playlist_changed, this));
+       _playlist->ContentChanged.connect (bind (&Film::playlist_content_changed, this, _1, _2));
        
        /* Make state.directory a complete path without ..s (where possible)
           (Code swiped from Adam Bowen on stackoverflow)
@@ -132,19 +129,6 @@ Film::Film (string d, bool must_exist)
        }
 
        set_directory (result.string ());
-       
-       if (!boost::filesystem::exists (directory())) {
-               if (must_exist) {
-                       throw OpenFileError (directory());
-               } else {
-                       boost::filesystem::create_directory (directory());
-               }
-       }
-
-       if (must_exist) {
-               read_metadata ();
-       }
-
        _log.reset (new FileLog (file ("log")));
 }
 
@@ -152,59 +136,40 @@ Film::Film (Film const & o)
        : boost::enable_shared_from_this<Film> (o)
        /* note: the copied film shares the original's log */
        , _log               (o._log)
-       , _playlist          (new Playlist)
+       , _playlist          (new Playlist (o._playlist))
        , _directory         (o._directory)
        , _name              (o._name)
        , _use_dci_name      (o._use_dci_name)
-       , _trust_content_headers (o._trust_content_headers)
        , _dcp_content_type  (o._dcp_content_type)
-       , _format            (o._format)
-       , _crop              (o._crop)
-       , _filters           (o._filters)
+       , _container         (o._container)
        , _scaler            (o._scaler)
-       , _trim_start        (o._trim_start)
-       , _trim_end          (o._trim_end)
        , _ab                (o._ab)
-       , _audio_gain        (o._audio_gain)
-       , _audio_delay       (o._audio_delay)
        , _with_subtitles    (o._with_subtitles)
        , _subtitle_offset   (o._subtitle_offset)
        , _subtitle_scale    (o._subtitle_scale)
        , _colour_lut        (o._colour_lut)
        , _j2k_bandwidth     (o._j2k_bandwidth)
        , _dci_metadata      (o._dci_metadata)
+       , _dcp_video_frame_rate (o._dcp_video_frame_rate)
        , _dci_date          (o._dci_date)
-       , _dcp_frame_rate    (o._dcp_frame_rate)
        , _dirty             (o._dirty)
 {
-       _playlist->setup (_content);
-}
-
-Film::~Film ()
-{
-
+       _playlist->ContentChanged.connect (bind (&Film::playlist_content_changed, this, _1, _2));
 }
 
 string
 Film::video_state_identifier () const
 {
-       assert (format ());
-
-       return "XXX";
-
-#if 0  
-
-       pair<string, string> f = Filter::ffmpeg_strings (filters());
+       assert (container ());
+       LocaleGuard lg;
 
        stringstream s;
-       s << format()->id()
-         << "_" << content_digest()
-         << "_" << crop().left << "_" << crop().right << "_" << crop().top << "_" << crop().bottom
-         << "_" << _dcp_frame_rate
-         << "_" << f.first << "_" << f.second
+       s << container()->id()
+         << "_" << _playlist->video_digest()
+         << "_" << _dcp_video_frame_rate
          << "_" << scaler()->id()
          << "_" << j2k_bandwidth()
-         << "_" << boost::lexical_cast<int> (colour_lut());
+         << "_" << lexical_cast<int> (colour_lut());
 
        if (ab()) {
                pair<string, string> fa = Filter::ffmpeg_strings (Config::instance()->reference_filters());
@@ -212,7 +177,6 @@ Film::video_state_identifier () const
        }
 
        return s.str ();
-#endif 
 }
          
 /** @return The path to the directory to write video frame info files to */
@@ -226,24 +190,52 @@ 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
 {
        boost::filesystem::path p;
        p /= "analysis";
-       p /= "XXX";//content_digest();
+       p /= _playlist->audio_digest();
        return file (p.string ());
 }
 
@@ -257,7 +249,7 @@ Film::make_dcp ()
                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()));
+       log()->log (String::compose ("DCP-o-matic %1 git %2 using %3", dcpomatic_version, dcpomatic_git_commit, dependency_version_summary()));
 
        {
                char buffer[128];
@@ -273,10 +265,10 @@ Film::make_dcp ()
 //     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
-       log()->log ("DVD-o-matic built in debug mode.");
+#ifdef DCPOMATIC_DEBUG
+       log()->log ("DCP-o-matic built in debug mode.");
 #else
-       log()->log ("DVD-o-matic built in optimised mode.");
+       log()->log ("DCP-o-matic built in optimised mode.");
 #endif
 #ifdef LIBDCP_DEBUG
        log()->log ("libdcp built in debug mode.");
@@ -286,12 +278,12 @@ Film::make_dcp ()
        pair<string, int> const c = cpu_info ();
        log()->log (String::compose ("CPU: %1, %2 processors", c.first, c.second));
        
-       if (format() == 0) {
-               throw MissingSettingError (_("format"));
+       if (container() == 0) {
+               throw MissingSettingError (_("container"));
        }
 
-       if (content().empty ()) {
-               throw MissingSettingError (_("content"));
+       if (_playlist->content().empty ()) {
+               throw StringError (_("You must add some content to the DCP before creating it"));
        }
 
        if (dcp_content_type() == 0) {
@@ -311,7 +303,7 @@ Film::make_dcp ()
        }
 }
 
-/** Start a job to analyse the audio of our content file */
+/** Start a job to analyse the audio in our Playlist */
 void
 Film::analyse_audio ()
 {
@@ -324,15 +316,6 @@ Film::analyse_audio ()
        JobManager::instance()->add (_analyse_audio_job);
 }
 
-/** Start a job to examine a piece of content */
-void
-Film::examine_content (shared_ptr<Content> c)
-{
-       shared_ptr<Job> j (new ExamineContentJob (shared_from_this(), c, trust_content_headers ()));
-       j->Finished.connect (bind (&Film::examine_content_finished, this));
-       JobManager::instance()->add (j);
-}
-
 void
 Film::analyse_audio_finished ()
 {
@@ -345,12 +328,6 @@ Film::analyse_audio_finished ()
        _analyse_audio_job.reset ();
 }
 
-void
-Film::examine_content_finished ()
-{
-       /* XXX */
-}
-
 /** Start a job to send our DCP to the configured TMS */
 void
 Film::send_dcp_to_tms ()
@@ -365,7 +342,7 @@ Film::send_dcp_to_tms ()
 int
 Film::encoded_frames () const
 {
-       if (format() == 0) {
+       if (container() == 0) {
                return 0;
        }
 
@@ -382,52 +359,42 @@ Film::encoded_frames () const
 void
 Film::write_metadata () const
 {
-       ContentList the_content = content ();
+       if (!boost::filesystem::exists (directory())) {
+               boost::filesystem::create_directory (directory());
+       }
        
        boost::mutex::scoped_lock lm (_state_mutex);
+       LocaleGuard lg;
 
        boost::filesystem::create_directories (directory());
 
        xmlpp::Document doc;
        xmlpp::Element* root = doc.create_root_node ("Metadata");
 
-       root->add_child("Version")->add_child_text (boost::lexical_cast<string> (state_version));
+       root->add_child("Version")->add_child_text (lexical_cast<string> (state_version));
        root->add_child("Name")->add_child_text (_name);
        root->add_child("UseDCIName")->add_child_text (_use_dci_name ? "1" : "0");
-       root->add_child("TrustContentHeaders")->add_child_text (_trust_content_headers ? "1" : "0");
+
        if (_dcp_content_type) {
                root->add_child("DCPContentType")->add_child_text (_dcp_content_type->dci_name ());
        }
-       if (_format) {
-               root->add_child("Format")->add_child_text (_format->id ());
-       }
-       root->add_child("LeftCrop")->add_child_text (boost::lexical_cast<string> (_crop.left));
-       root->add_child("RightCrop")->add_child_text (boost::lexical_cast<string> (_crop.right));
-       root->add_child("TopCrop")->add_child_text (boost::lexical_cast<string> (_crop.top));
-       root->add_child("BottomCrop")->add_child_text (boost::lexical_cast<string> (_crop.bottom));
 
-       for (vector<Filter const *>::const_iterator i = _filters.begin(); i != _filters.end(); ++i) {
-               root->add_child("Filter")->add_child_text ((*i)->id ());
+       if (_container) {
+               root->add_child("Container")->add_child_text (_container->id ());
        }
-       
+
        root->add_child("Scaler")->add_child_text (_scaler->id ());
-       root->add_child("TrimStart")->add_child_text (boost::lexical_cast<string> (_trim_start));
-       root->add_child("TrimEnd")->add_child_text (boost::lexical_cast<string> (_trim_end));
        root->add_child("AB")->add_child_text (_ab ? "1" : "0");
-       root->add_child("AudioGain")->add_child_text (boost::lexical_cast<string> (_audio_gain));
-       root->add_child("AudioDelay")->add_child_text (boost::lexical_cast<string> (_audio_delay));
        root->add_child("WithSubtitles")->add_child_text (_with_subtitles ? "1" : "0");
-       root->add_child("SubtitleOffset")->add_child_text (boost::lexical_cast<string> (_subtitle_offset));
-       root->add_child("SubtitleScale")->add_child_text (boost::lexical_cast<string> (_subtitle_scale));
-       root->add_child("ColourLUT")->add_child_text (boost::lexical_cast<string> (_colour_lut));
-       root->add_child("J2KBandwidth")->add_child_text (boost::lexical_cast<string> (_j2k_bandwidth));
+       root->add_child("SubtitleOffset")->add_child_text (lexical_cast<string> (_subtitle_offset));
+       root->add_child("SubtitleScale")->add_child_text (lexical_cast<string> (_subtitle_scale));
+       root->add_child("ColourLUT")->add_child_text (lexical_cast<string> (_colour_lut));
+       root->add_child("J2KBandwidth")->add_child_text (lexical_cast<string> (_j2k_bandwidth));
        _dci_metadata.as_xml (root->add_child ("DCIMetadata"));
+       root->add_child("DCPVideoFrameRate")->add_child_text (lexical_cast<string> (_dcp_video_frame_rate));
        root->add_child("DCIDate")->add_child_text (boost::gregorian::to_iso_string (_dci_date));
-       root->add_child("DCPFrameRate")->add_child_text (boost::lexical_cast<string> (_dcp_frame_rate));
-
-       for (ContentList::iterator i = the_content.begin(); i != the_content.end(); ++i) {
-               (*i)->as_xml (root->add_child ("Content"));
-       }
+       root->add_child("DCPAudioChannels")->add_child_text (lexical_cast<string> (_dcp_audio_channels));
+       _playlist->as_xml (root->add_child ("Playlist"));
 
        doc.write_to_file_formatted (file ("metadata.xml"));
        
@@ -439,16 +406,16 @@ void
 Film::read_metadata ()
 {
        boost::mutex::scoped_lock lm (_state_mutex);
+       LocaleGuard lg;
 
        if (boost::filesystem::exists (file ("metadata")) && !boost::filesystem::exists (file ("metadata.xml"))) {
-               throw StringError (_("This film was created with an older version of DVD-o-matic, and unfortunately it cannot be loaded into this version.  You will need to create a new Film, re-add your content and set it up again.  Sorry!"));
+               throw StringError (_("This film was created with an older version of DCP-o-matic, and unfortunately it cannot be loaded into this version.  You will need to create a new Film, re-add your content and set it up again.  Sorry!"));
        }
 
        cxml::File f (file ("metadata.xml"), "Metadata");
        
        _name = f.string_child ("Name");
        _use_dci_name = f.bool_child ("UseDCIName");
-       _trust_content_headers = f.bool_child ("TrustContentHeaders");
 
        {
                optional<string> c = f.optional_string_child ("DCPContentType");
@@ -458,63 +425,27 @@ Film::read_metadata ()
        }
 
        {
-               optional<string> c = f.optional_string_child ("Format");
+               optional<string> c = f.optional_string_child ("Container");
                if (c) {
-                       _format = Format::from_id (c.get ());
-               }
-       }
-
-       _crop.left = f.number_child<int> ("LeftCrop");
-       _crop.right = f.number_child<int> ("RightCrop");
-       _crop.top = f.number_child<int> ("TopCrop");
-       _crop.bottom = f.number_child<int> ("BottomCrop");
-
-       {
-               list<shared_ptr<cxml::Node> > c = f.node_children ("Filter");
-               for (list<shared_ptr<cxml::Node> >::iterator i = c.begin(); i != c.end(); ++i) {
-                       _filters.push_back (Filter::from_id ((*i)->content ()));
+                       _container = Ratio::from_id (c.get ());
                }
        }
 
        _scaler = Scaler::from_id (f.string_child ("Scaler"));
-       _trim_start = f.number_child<int> ("TrimStart");
-       _trim_end = f.number_child<int> ("TrimEnd");
        _ab = f.bool_child ("AB");
-       _audio_gain = f.number_child<float> ("AudioGain");
-       _audio_delay = f.number_child<int> ("AudioDelay");
        _with_subtitles = f.bool_child ("WithSubtitles");
        _subtitle_offset = f.number_child<float> ("SubtitleOffset");
        _subtitle_scale = f.number_child<float> ("SubtitleScale");
        _colour_lut = f.number_child<int> ("ColourLUT");
        _j2k_bandwidth = f.number_child<int> ("J2KBandwidth");
        _dci_metadata = DCIMetadata (f.node_child ("DCIMetadata"));
+       _dcp_video_frame_rate = f.number_child<int> ("DCPVideoFrameRate");
        _dci_date = boost::gregorian::from_undelimited_string (f.string_child ("DCIDate"));
-       _dcp_frame_rate = f.number_child<int> ("DCPFrameRate");
-
-       list<shared_ptr<cxml::Node> > c = f.node_children ("Content");
-       for (list<shared_ptr<cxml::Node> >::iterator i = c.begin(); i != c.end(); ++i) {
-
-               string const type = (*i)->string_child ("Type");
-               
-               if (type == "FFmpeg") {
-                       _content.push_back (shared_ptr<Content> (new FFmpegContent (*i)));
-               } else if (type == "ImageMagick") {
-                       _content.push_back (shared_ptr<Content> (new ImageMagickContent (*i)));
-               } else if (type == "Sndfile") {
-                       _content.push_back (shared_ptr<Content> (new SndfileContent (*i)));
-               }
-       }
+       _dcp_audio_channels = f.number_child<int> ("DCPAudioChannels");
 
-       _dirty = false;
-}
+       _playlist->set_from_xml (shared_from_this(), f.node_child ("Playlist"));
 
-libdcp::Size
-Film::cropped_size (libdcp::Size s) const
-{
-       boost::mutex::scoped_lock lm (_state_mutex);
-       s.width -= _crop.left + _crop.right;
-       s.height -= _crop.top + _crop.bottom;
-       return s;
+       _dirty = false;
 }
 
 /** Given a directory name, return its full path within the Film's directory.
@@ -552,32 +483,6 @@ Film::file (string f) const
        return p.string ();
 }
 
-/** @return The sampling rate that we will resample the audio to */
-int
-Film::target_audio_sample_rate () const
-{
-       if (has_audio ()) {
-               return 0;
-       }
-       
-       /* Resample to a DCI-approved sample rate */
-       double t = dcp_audio_sample_rate (audio_frame_rate());
-
-       FrameRateConversion frc (video_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
-          look different in the DCP compared to the source (slower or faster).
-          skip/repeat doesn't come into effect here.
-       */
-
-       if (frc.change_speed) {
-               t *= video_frame_rate() * frc.factor() / dcp_frame_rate();
-       }
-
-       return rint (t);
-}
-
 /** @return a DCI-compliant name for a DCP of this film */
 string
 Film::dci_name (bool if_created_now) const
@@ -602,8 +507,8 @@ Film::dci_name (bool if_created_now) const
                d << "_" << dcp_content_type()->dci_name();
        }
 
-       if (format()) {
-               d << "_" << format()->dci_name();
+       if (container()) {
+               d << "_" << container()->dci_name();
        }
 
        DCIMetadata const dm = dci_metadata ();
@@ -624,23 +529,7 @@ Film::dci_name (bool if_created_now) const
                }
        }
 
-       /* XXX */
-       switch (2) {
-       case 1:
-               d << "_10";
-               break;
-       case 2:
-               d << "_20";
-               break;
-       case 6:
-               d << "_51";
-               break;
-       case 8:
-               d << "_71";
-               break;
-       }
-
-       d << "_2K";
+       d << "_51_2K";
 
        if (!dm.studio.empty ()) {
                d << "_" << dm.studio;
@@ -703,23 +592,6 @@ Film::set_use_dci_name (bool u)
        signal_changed (USE_DCI_NAME);
 }
 
-void
-Film::set_trust_content_headers (bool t)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _trust_content_headers = t;
-       }
-       
-       signal_changed (TRUST_CONTENT_HEADERS);
-
-       if (!_trust_content_headers && !content().empty()) {
-               /* We just said that we don't trust the content's header */
-               /* XXX */
-//             examine_content ();
-       }
-}
-              
 void
 Film::set_dcp_content_type (DCPContentType const * t)
 {
@@ -731,90 +603,13 @@ Film::set_dcp_content_type (DCPContentType const * t)
 }
 
 void
-Film::set_format (Format const * f)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _format = f;
-       }
-       signal_changed (FORMAT);
-}
-
-void
-Film::set_crop (Crop c)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _crop = c;
-       }
-       signal_changed (CROP);
-}
-
-void
-Film::set_left_crop (int c)
+Film::set_container (Ratio const * c)
 {
        {
                boost::mutex::scoped_lock lm (_state_mutex);
-               
-               if (_crop.left == c) {
-                       return;
-               }
-               
-               _crop.left = c;
+               _container = c;
        }
-       signal_changed (CROP);
-}
-
-void
-Film::set_right_crop (int c)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               if (_crop.right == c) {
-                       return;
-               }
-               
-               _crop.right = c;
-       }
-       signal_changed (CROP);
-}
-
-void
-Film::set_top_crop (int c)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               if (_crop.top == c) {
-                       return;
-               }
-               
-               _crop.top = c;
-       }
-       signal_changed (CROP);
-}
-
-void
-Film::set_bottom_crop (int c)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               if (_crop.bottom == c) {
-                       return;
-               }
-               
-               _crop.bottom = c;
-       }
-       signal_changed (CROP);
-}
-
-void
-Film::set_filters (vector<Filter const *> f)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _filters = f;
-       }
-       signal_changed (FILTERS);
+       signal_changed (CONTAINER);
 }
 
 void
@@ -827,26 +622,6 @@ Film::set_scaler (Scaler const * s)
        signal_changed (SCALER);
 }
 
-void
-Film::set_trim_start (int t)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _trim_start = t;
-       }
-       signal_changed (TRIM_START);
-}
-
-void
-Film::set_trim_end (int t)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _trim_end = t;
-       }
-       signal_changed (TRIM_END);
-}
-
 void
 Film::set_ab (bool a)
 {
@@ -857,26 +632,6 @@ Film::set_ab (bool a)
        signal_changed (AB);
 }
 
-void
-Film::set_audio_gain (float g)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _audio_gain = g;
-       }
-       signal_changed (AUDIO_GAIN);
-}
-
-void
-Film::set_audio_delay (int d)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _audio_delay = d;
-       }
-       signal_changed (AUDIO_DELAY);
-}
-
 void
 Film::set_with_subtitles (bool w)
 {
@@ -939,13 +694,13 @@ Film::set_dci_metadata (DCIMetadata m)
 
 
 void
-Film::set_dcp_frame_rate (int f)
+Film::set_dcp_video_frame_rate (int f)
 {
        {
                boost::mutex::scoped_lock lm (_state_mutex);
-               _dcp_frame_rate = f;
+               _dcp_video_frame_rate = f;
        }
-       signal_changed (DCP_FRAME_RATE);
+       signal_changed (DCP_VIDEO_FRAME_RATE);
 }
 
 void
@@ -956,6 +711,14 @@ Film::signal_changed (Property p)
                _dirty = true;
        }
 
+       switch (p) {
+       case Film::CONTENT:
+               set_dcp_video_frame_rate (_playlist->best_dcp_frame_rate ());
+               break;
+       default:
+               break;
+       }
+
        if (ui_signaller) {
                ui_signaller->emit (boost::bind (boost::ref (Changed), p));
        }
@@ -1025,125 +788,133 @@ Film::have_dcp () const
 shared_ptr<Player>
 Film::player () const
 {
-       boost::mutex::scoped_lock lm (_state_mutex);
        return shared_ptr<Player> (new Player (shared_from_this (), _playlist));
 }
 
-void
-Film::add_content (shared_ptr<Content> c)
+shared_ptr<Playlist>
+Film::playlist () const
 {
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _content.push_back (c);
-               _playlist->setup (_content);
-       }
+       boost::mutex::scoped_lock lm (_state_mutex);
+       return _playlist;
+}
 
-       signal_changed (CONTENT);
+Playlist::ContentList
+Film::content () const
+{
+       return _playlist->content ();
+}
 
-       examine_content (c);
+void
+Film::examine_and_add_content (shared_ptr<Content> c)
+{
+       shared_ptr<Job> j (new ExamineContentJob (shared_from_this(), c));
+       JobManager::instance()->add (j);
 }
 
 void
-Film::remove_content (shared_ptr<Content> c)
+Film::add_content (shared_ptr<Content> c)
 {
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               ContentList::iterator i = find (_content.begin(), _content.end(), c);
-               if (i != _content.end ()) {
-                       _content.erase (i);
-               }
+       /* Add video content after any existing content */
+       if (dynamic_pointer_cast<VideoContent> (c)) {
+               c->set_start (_playlist->video_end ());
        }
 
-       signal_changed (CONTENT);
+       _playlist->add (c);
 }
 
 void
-Film::move_content_earlier (shared_ptr<Content> c)
+Film::remove_content (shared_ptr<Content> c)
 {
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               ContentList::iterator i = find (_content.begin(), _content.end(), c);
-               if (i == _content.begin () || i == _content.end()) {
-                       return;
-               }
+       _playlist->remove (c);
+}
 
-               ContentList::iterator j = i;
-               --j;
+Time
+Film::length () const
+{
+       return _playlist->length ();
+}
 
-               swap (*i, *j);
-       }
+bool
+Film::has_subtitles () const
+{
+       return _playlist->has_subtitles ();
+}
 
-       signal_changed (CONTENT);
+OutputVideoFrame
+Film::best_dcp_video_frame_rate () const
+{
+       return _playlist->best_dcp_frame_rate ();
 }
 
 void
-Film::move_content_later (shared_ptr<Content> c)
+Film::playlist_content_changed (boost::weak_ptr<Content> c, int p)
 {
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               ContentList::iterator i = find (_content.begin(), _content.end(), c);
-               if (i == _content.end()) {
-                       return;
-               }
+       if (p == VideoContentProperty::VIDEO_FRAME_RATE) {
+               set_dcp_video_frame_rate (_playlist->best_dcp_frame_rate ());
+       } 
 
-               ContentList::iterator j = i;
-               ++j;
-               if (j == _content.end ()) {
-                       return;
-               }
-
-               swap (*i, *j);
+       if (ui_signaller) {
+               ui_signaller->emit (boost::bind (boost::ref (ContentChanged), c, p));
        }
+}
 
+void
+Film::playlist_changed ()
+{
        signal_changed (CONTENT);
+}      
 
+int
+Film::loop () const
+{
+       return _playlist->loop ();
 }
 
-ContentAudioFrame
-Film::audio_length () const
+void
+Film::set_loop (int c)
 {
-       return _playlist->audio_length ();
+       _playlist->set_loop (c);
 }
 
-int
-Film::audio_channels () const
+OutputAudioFrame
+Film::time_to_audio_frames (Time t) const
 {
-       return _playlist->audio_channels ();
+       return t * dcp_audio_frame_rate () / TIME_HZ;
 }
 
-int
-Film::audio_frame_rate () const
+OutputVideoFrame
+Film::time_to_video_frames (Time t) const
 {
-       return _playlist->audio_frame_rate ();
+       return t * dcp_video_frame_rate () / TIME_HZ;
 }
 
-int64_t
-Film::audio_channel_layout () const
+Time
+Film::audio_frames_to_time (OutputAudioFrame f) const
 {
-       return _playlist->audio_channel_layout ();
+       return f * TIME_HZ / dcp_audio_frame_rate ();
 }
 
-bool
-Film::has_audio () const
+Time
+Film::video_frames_to_time (OutputVideoFrame f) const
 {
-       return _playlist->has_audio ();
+       return f * TIME_HZ / dcp_video_frame_rate ();
 }
 
-float
-Film::video_frame_rate () const
+OutputAudioFrame
+Film::dcp_audio_frame_rate () const
 {
-       return _playlist->video_frame_rate ();
+       /* XXX */
+       return 48000;
 }
 
-libdcp::Size
-Film::video_size () const
+void
+Film::set_sequence_video (bool s)
 {
-       return _playlist->video_size ();
+       _playlist->set_sequence_video (s);
 }
 
-ContentVideoFrame
-Film::video_length () const
+libdcp::Size
+Film::full_frame () const
 {
-       return _playlist->video_length ();
+       return libdcp::Size (2048, 1080);
 }
-