Merge master and multifarious hackery.
[dcpomatic.git] / src / lib / film.cc
index d22f67e6765c55b242a1077b041dd870ec331f50..fc1d2d8a4ee2fa7c98ee7d34ad06a639cf42b38e 100644 (file)
@@ -1,3 +1,5 @@
+/* -*- c-basic-offset: 8; default-tab-width: 8; -*- */
+
 /*
     Copyright (C) 2012 Carl Hetherington <cth@carlh.net>
 
 #include <boost/algorithm/string.hpp>
 #include <boost/lexical_cast.hpp>
 #include <boost/date_time.hpp>
+#include <libxml++/libxml++.h>
+#include <libcxml/cxml.h>
 #include "film.h"
-#include "format.h"
+#include "container.h"
 #include "job.h"
 #include "filter.h"
-#include "transcoder.h"
 #include "util.h"
 #include "job_manager.h"
 #include "ab_transcode_job.h"
 #include "exceptions.h"
 #include "examine_content_job.h"
 #include "scaler.h"
-#include "decoder_factory.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 "i18n.h"
 
@@ -67,6 +71,8 @@ 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::to_upper_copy;
@@ -77,35 +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)
-       : _use_dci_name (true)
-       , _trust_content_headers (true)
-       , _dcp_content_type (0)
-       , _format (0)
+Film::Film (string d)
+       : _playlist (new Playlist)
+       , _use_dci_name (true)
+       , _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)
-       , _dcp_ab (false)
-       , _audio_gain (0)
-       , _audio_delay (0)
+       , _ab (false)
        , _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)
@@ -126,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")));
 }
 
@@ -146,66 +136,47 @@ 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 (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)
-       , _dcp_ab            (o._dcp_ab)
-       , _audio_gain        (o._audio_gain)
-       , _audio_delay       (o._audio_delay)
+       , _ab                (o._ab)
        , _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)
 {
-       
-}
-
-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 (dcp_ab()) {
+       if (ab()) {
                pair<string, string> fa = Filter::ffmpeg_strings (Config::instance()->reference_filters());
                s << "ab_" << Config::instance()->reference_scaler()->id() << "_" << fa.first << "_" << fa.second;
        }
 
        return s.str ();
-#endif 
 }
          
 /** @return The path to the directory to write video frame info files to */
@@ -219,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 ());
 }
 
@@ -250,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];
@@ -266,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.");
@@ -279,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) {
@@ -297,14 +296,14 @@ Film::make_dcp ()
 
        shared_ptr<Job> r;
 
-       if (dcp_ab()) {
+       if (ab()) {
                r = JobManager::instance()->add (shared_ptr<Job> (new ABTranscodeJob (shared_from_this())));
        } else {
                r = JobManager::instance()->add (shared_ptr<Job> (new TranscodeJob (shared_from_this())));
        }
 }
 
-/** 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 ()
 {
@@ -321,8 +320,7 @@ Film::analyse_audio ()
 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));
+       shared_ptr<Job> j (new ExamineContentJob (shared_from_this(), c));
        JobManager::instance()->add (j);
 }
 
@@ -338,12 +336,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 ()
@@ -358,7 +350,7 @@ Film::send_dcp_to_tms ()
 int
 Film::encoded_frames () const
 {
-       if (format() == 0) {
+       if (container() == 0) {
                return 0;
        }
 
@@ -375,77 +367,44 @@ Film::encoded_frames () const
 void
 Film::write_metadata () const
 {
+       if (!boost::filesystem::exists (directory())) {
+               boost::filesystem::create_directory (directory());
+       }
+       
        boost::mutex::scoped_lock lm (_state_mutex);
+       LocaleGuard lg;
 
        boost::filesystem::create_directories (directory());
 
-       string const m = file ("metadata");
-       ofstream f (m.c_str ());
-       if (!f.good ()) {
-               throw CreateFileError (m);
-       }
+       xmlpp::Document doc;
+       xmlpp::Element* root = doc.create_root_node ("Metadata");
 
-       f << "version " << state_version << endl;
+       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");
 
-       /* User stuff */
-       f << "name " << _name << endl;
-       f << "use_dci_name " << _use_dci_name << endl;
-//     f << "content " << _content << endl;
-       f << "trust_content_headers " << (_trust_content_headers ? "1" : "0") << endl;
        if (_dcp_content_type) {
-               f << "dcp_content_type " << _dcp_content_type->dci_name () << endl;
-       }
-       if (_format) {
-               f << "format " << _format->as_metadata () << endl;
-       }
-       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 () << endl;
+               root->add_child("DCPContentType")->add_child_text (_dcp_content_type->dci_name ());
        }
-       f << "scaler " << _scaler->id () << endl;
-       f << "trim_start " << _trim_start << endl;
-       f << "trim_end " << _trim_end << endl;
-       f << "dcp_ab " << (_dcp_ab ? "1" : "0") << endl;
-//     if (_content_audio_stream) {
-//             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 << endl;
-//     }
-//     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() << endl;
-//     }
-       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) << 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 () << endl;
-//     }
 
-//     f << "external_audio_stream " << _sndfile_stream->to_string() << endl;
+       if (_container) {
+               root->add_child("Container")->add_child_text (_container->id ());
+       }
 
-//     for (vector<shared_ptr<SubtitleStream> >::const_iterator i = _subtitle_streams.begin(); i != _subtitle_streams.end(); ++i) {
-//             f << "subtitle_stream " << (*i)->to_string () << endl;
-//     }
+       root->add_child("Scaler")->add_child_text (_scaler->id ());
+       root->add_child("AB")->add_child_text (_ab ? "1" : "0");
+       root->add_child("WithSubtitles")->add_child_text (_with_subtitles ? "1" : "0");
+       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("DCPAudioChannels")->add_child_text (lexical_cast<string> (_dcp_audio_channels));
+       _playlist->as_xml (root->add_child ("Playlist"));
 
-//     f << "source_frame_rate " << _source_frame_rate << endl;
+       doc.write_to_file_formatted (file ("metadata.xml"));
        
        _dirty = false;
 }
@@ -455,151 +414,46 @@ void
 Film::read_metadata ()
 {
        boost::mutex::scoped_lock lm (_state_mutex);
+       LocaleGuard lg;
 
-//     _external_audio.clear ();
-//     _content_audio_streams.clear ();
-//     _subtitle_streams.clear ();
-
-       boost::optional<int> version;
-
-       /* Backward compatibility things */
-       boost::optional<int> audio_sample_rate;
-       boost::optional<int> audio_stream_index;
-       boost::optional<int> subtitle_stream_index;
-
-       ifstream f (file ("metadata").c_str());
-       if (!f.good()) {
-               throw OpenFileError (file ("metadata"));
+       if (boost::filesystem::exists (file ("metadata")) && !boost::filesystem::exists (file ("metadata.xml"))) {
+               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!"));
        }
-       
-       multimap<string, string> kv = read_key_value (f);
 
-       /* We need version before anything else */
-       multimap<string, string>::iterator v = kv.find ("version");
-       if (v != kv.end ()) {
-               version = atoi (v->second.c_str());
-       }
+       cxml::File f (file ("metadata.xml"), "Metadata");
        
-       for (multimap<string, string>::const_iterator i = kv.begin(); i != kv.end(); ++i) {
-               string const k = i->first;
-               string const v = i->second;
+       _name = f.string_child ("Name");
+       _use_dci_name = f.bool_child ("UseDCIName");
 
-               if (k == "audio_sample_rate") {
-                       audio_sample_rate = atoi (v.c_str());
-               }
-
-               /* User-specified stuff */
-               if (k == "name") {
-                       _name = v;
-               } else if (k == "use_dci_name") {
-                       _use_dci_name = (v == "1");
-               } else if (k == "content") {
-//                     _content = v;
-               } else if (k == "trust_content_headers") {
-                       _trust_content_headers = (v == "1");
-               } else if (k == "dcp_content_type") {
-                       if (version < 3) {
-                               _dcp_content_type = DCPContentType::from_pretty_name (v);
-                       } else {
-                               _dcp_content_type = DCPContentType::from_dci_name (v);
-                       }
-               } else if (k == "format") {
-                       _format = Format::from_metadata (v);
-               } else if (k == "left_crop") {
-                       _crop.left = atoi (v.c_str ());
-               } else if (k == "right_crop") {
-                       _crop.right = atoi (v.c_str ());
-               } else if (k == "top_crop") {
-                       _crop.top = atoi (v.c_str ());
-               } else if (k == "bottom_crop") {
-                       _crop.bottom = atoi (v.c_str ());
-               } else if (k == "filter") {
-                       _filters.push_back (Filter::from_id (v));
-               } else if (k == "scaler") {
-                       _scaler = Scaler::from_id (v);
-               } else if ( ((!version || version < 2) && k == "dcp_trim_start") || k == "trim_start") {
-                       _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 == "dcp_ab") {
-                       _dcp_ab = (v == "1");
-               } else if (k == "selected_content_audio_stream" || (!version && k == "selected_audio_stream")) {
-                       if (!version) {
-                               audio_stream_index = atoi (v.c_str ());
-                       } else {
-//                             _content_audio_stream = audio_stream_factory (v, version);
-                       }
-               } else if (k == "external_audio") {
-//                     _external_audio.push_back (v);
-               } else if (k == "use_content_audio") {
-//                     _use_content_audio = (v == "1");
-               } else if (k == "audio_gain") {
-                       _audio_gain = atof (v.c_str ());
-               } else if (k == "audio_delay") {
-                       _audio_delay = atoi (v.c_str ());
-               } else if (k == "still_duration") {
-//                     _still_duration = atoi (v.c_str ());
-               } else if (k == "selected_subtitle_stream") {
-                       if (!version) {
-                               subtitle_stream_index = atoi (v.c_str ());
-                       } else {
-//                             _subtitle_stream = subtitle_stream_factory (v, version);
-                       }
-               } else if (k == "with_subtitles") {
-                       _with_subtitles = (v == "1");
-               } else if (k == "subtitle_offset") {
-                       _subtitle_offset = atoi (v.c_str ());
-               } else if (k == "subtitle_scale") {
-                       _subtitle_scale = atof (v.c_str ());
-               } else if (k == "colour_lut") {
-                       _colour_lut = atoi (v.c_str ());
-               } else if (k == "j2k_bandwidth") {
-                       _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 ());
+       {
+               optional<string> c = f.optional_string_child ("DCPContentType");
+               if (c) {
+                       _dcp_content_type = DCPContentType::from_dci_name (c.get ());
                }
+       }
 
-               _dci_metadata.read (k, v);
-               
-               /* Cached stuff */
-               if (k == "width") {
-//                     _size.width = atoi (v.c_str ());
-               } else if (k == "height") {
-//                     _size.height = atoi (v.c_str ());
-               } else if (k == "length") {
-                       int const vv = atoi (v.c_str ());
-                       if (vv) {
-//                             _length = 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") {
-//                     _sndfile_stream = audio_stream_factory (v, version);
-               } else if (k == "subtitle_stream") {
-//                     _subtitle_streams.push_back (subtitle_stream_factory (v, version));
-               } 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);
+       {
+               optional<string> c = f.optional_string_child ("Container");
+               if (c) {
+                       _container = Container::from_id (c.get ());
                }
        }
 
-       _dirty = false;
-}
+       _scaler = Scaler::from_id (f.string_child ("Scaler"));
+       _ab = f.bool_child ("AB");
+       _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_audio_channels = f.number_child<int> ("DCPAudioChannels");
 
-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;
+       _playlist->set_from_xml (shared_from_this(), f.node_child ("Playlist"));
+
+       _dirty = false;
 }
 
 /** Given a directory name, return its full path within the Film's directory.
@@ -637,35 +491,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
-{
-       /* XXX: how often is this method called? */
-       
-       boost::shared_ptr<Playlist> p = playlist ();
-       if (p->has_audio ()) {
-               return 0;
-       }
-       
-       /* Resample to a DCI-approved sample rate */
-       double t = dcp_audio_sample_rate (p->audio_frame_rate());
-
-       FrameRateConversion frc (p->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 *= p->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
@@ -690,8 +515,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 ();
@@ -712,23 +537,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;
@@ -791,23 +600,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)
 {
@@ -819,90 +611,13 @@ Film::set_dcp_content_type (DCPContentType const * t)
 }
 
 void
-Film::set_format (Format const * f)
+Film::set_container (Container const * c)
 {
        {
                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)
-{
-       {
-               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
@@ -916,53 +631,13 @@ Film::set_scaler (Scaler const * s)
 }
 
 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_dcp_ab (bool a)
-{
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _dcp_ab = a;
-       }
-       signal_changed (DCP_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)
+Film::set_ab (bool a)
 {
        {
                boost::mutex::scoped_lock lm (_state_mutex);
-               _audio_delay = d;
+               _ab = a;
        }
-       signal_changed (AUDIO_DELAY);
+       signal_changed (AB);
 }
 
 void
@@ -1027,13 +702,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
@@ -1044,6 +719,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));
        }
@@ -1110,22 +793,113 @@ Film::have_dcp () const
        return true;
 }
 
+shared_ptr<Player>
+Film::player () const
+{
+       return shared_ptr<Player> (new Player (shared_from_this (), _playlist));
+}
+
 shared_ptr<Playlist>
 Film::playlist () const
 {
        boost::mutex::scoped_lock lm (_state_mutex);
-       return shared_ptr<Playlist> (new Playlist (shared_from_this (), _content));
+       return _playlist;
+}
+
+Playlist::ContentList
+Film::content () const
+{
+       return _playlist->content ();
 }
 
 void
 Film::add_content (shared_ptr<Content> c)
 {
-       {
-               boost::mutex::scoped_lock lm (_state_mutex);
-               _content.push_back (c);
+       _playlist->add (c);
+       examine_content (c);
+}
+
+void
+Film::remove_content (shared_ptr<Content> c)
+{
+       _playlist->remove (c);
+}
+
+Time
+Film::length () const
+{
+       return _playlist->length ();
+}
+
+bool
+Film::has_subtitles () const
+{
+       return _playlist->has_subtitles ();
+}
+
+OutputVideoFrame
+Film::best_dcp_video_frame_rate () const
+{
+       return _playlist->best_dcp_frame_rate ();
+}
+
+void
+Film::playlist_content_changed (boost::weak_ptr<Content> c, int p)
+{
+       if (p == VideoContentProperty::VIDEO_FRAME_RATE) {
+               set_dcp_video_frame_rate (_playlist->best_dcp_frame_rate ());
+       } 
+
+       if (ui_signaller) {
+               ui_signaller->emit (boost::bind (boost::ref (ContentChanged), c, p));
        }
+}
 
+void
+Film::playlist_changed ()
+{
        signal_changed (CONTENT);
+}      
 
-       examine_content (c);
+int
+Film::loop () const
+{
+       return _playlist->loop ();
+}
+
+void
+Film::set_loop (int c)
+{
+       _playlist->set_loop (c);
+}
+
+OutputAudioFrame
+Film::time_to_audio_frames (Time t) const
+{
+       return t * dcp_audio_frame_rate () / TIME_HZ;
+}
+
+OutputVideoFrame
+Film::time_to_video_frames (Time t) const
+{
+       return t * dcp_video_frame_rate () / TIME_HZ;
+}
+
+Time
+Film::audio_frames_to_time (OutputAudioFrame f) const
+{
+       return f * TIME_HZ / dcp_audio_frame_rate ();
+}
+
+Time
+Film::video_frames_to_time (OutputVideoFrame f) const
+{
+       return f * TIME_HZ / dcp_video_frame_rate ();
+}
+
+OutputAudioFrame
+Film::dcp_audio_frame_rate () const
+{
+       /* XXX */
+       return 48000;
 }