Tidy up creation of analysis a bit.
[dcpomatic.git] / src / wx / film_editor.cc
index 3a22ab67036adf32be6db81c70de582786b71a2e..c9f83677c6da5db1cf7ef2168890538d4d810e5b 100644 (file)
 #include "lib/ab_transcode_job.h"
 #include "lib/job_manager.h"
 #include "lib/filter.h"
-#include "lib/screen.h"
 #include "lib/config.h"
+#include "lib/ffmpeg_decoder.h"
+#include "lib/external_audio_decoder.h"
 #include "filter_dialog.h"
 #include "wx_util.h"
 #include "film_editor.h"
 #include "gain_calculator_dialog.h"
 #include "sound_processor.h"
-#include "dci_name_dialog.h"
+#include "dci_metadata_dialog.h"
 #include "scaler.h"
+#include "audio_dialog.h"
 
 using std::string;
+using std::cout;
 using std::stringstream;
 using std::pair;
 using std::fixed;
@@ -53,20 +56,24 @@ using std::setprecision;
 using std::list;
 using std::vector;
 using boost::shared_ptr;
+using boost::dynamic_pointer_cast;
 
 /** @param f Film to edit */
 FilmEditor::FilmEditor (shared_ptr<Film> f, wxWindow* parent)
        : wxPanel (parent)
        , _film (f)
        , _generally_sensitive (true)
+       , _audio_dialog (0)
 {
-       wxSizer* s = new wxBoxSizer (wxVERTICAL);
+       wxBoxSizer* s = new wxBoxSizer (wxVERTICAL);
        SetSizer (s);
        _notebook = new wxNotebook (this, wxID_ANY);
        s->Add (_notebook, 1);
-       
-       make_general_panel ();
-       _notebook->AddPage (_general_panel, _("General"), true);
+
+       make_film_panel ();
+       _notebook->AddPage (_film_panel, _("Film"), true);
+       make_video_panel ();
+       _notebook->AddPage (_video_panel, _("Video"), false);
        make_audio_panel ();
        _notebook->AddPage (_audio_panel, _("Audio"), false);
        make_subtitle_panel ();
@@ -74,145 +81,98 @@ FilmEditor::FilmEditor (shared_ptr<Film> f, wxWindow* parent)
 
        set_film (_film);
        connect_to_widgets ();
+
+       JobManager::instance()->ActiveJobsChanged.connect (
+               bind (&FilmEditor::active_jobs_changed, this, _1)
+               );
        
        setup_visibility ();
        setup_formats ();
 }
 
 void
-FilmEditor::make_general_panel ()
+FilmEditor::make_film_panel ()
 {
-       _general_panel = new wxPanel (_notebook);
-       _general_sizer = new wxFlexGridSizer (2, 4, 4);
-       _general_panel->SetSizer (_general_sizer);
-
-       add_label_to_sizer (_general_sizer, _general_panel, "Name");
-       _name = new wxTextCtrl (_general_panel, wxID_ANY);
-       _general_sizer->Add (_name, 1, wxEXPAND);
-
-       add_label_to_sizer (_general_sizer, _general_panel, "DCP Name");
-       _dcp_name = new wxStaticText (_general_panel, wxID_ANY, wxT (""));
-       _general_sizer->Add (_dcp_name, 0, wxALIGN_CENTER_VERTICAL | wxSHRINK);
-
-       _use_dci_name = new wxCheckBox (_general_panel, wxID_ANY, wxT ("Use DCI name"));
-       _general_sizer->Add (_use_dci_name, 1, wxEXPAND);
-       _edit_dci_button = new wxButton (_general_panel, wxID_ANY, wxT ("Details..."));
-       _general_sizer->Add (_edit_dci_button, 0);
-
-       add_label_to_sizer (_general_sizer, _general_panel, "Content");
-       _content = new wxFilePickerCtrl (_general_panel, wxID_ANY, wxT (""), wxT ("Select Content File"), wxT("*.*"));
-       _general_sizer->Add (_content, 1, wxEXPAND);
-
-       add_label_to_sizer (_general_sizer, _general_panel, "Content Type");
-       _dcp_content_type = new wxComboBox (_general_panel, wxID_ANY);
-       _general_sizer->Add (_dcp_content_type);
-
-       add_label_to_sizer (_general_sizer, _general_panel, "Format");
-       _format = new wxComboBox (_general_panel, wxID_ANY);
-       _general_sizer->Add (_format);
-
-       {
-               add_label_to_sizer (_general_sizer, _general_panel, "Crop");
-               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-
-               add_label_to_sizer (s, _general_panel, "L");
-               _left_crop = new wxSpinCtrl (_general_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_left_crop, 0);
-               add_label_to_sizer (s, _general_panel, "R");
-               _right_crop = new wxSpinCtrl (_general_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_right_crop, 0);
-               add_label_to_sizer (s, _general_panel, "T");
-               _top_crop = new wxSpinCtrl (_general_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_top_crop, 0);
-               add_label_to_sizer (s, _general_panel, "B");
-               _bottom_crop = new wxSpinCtrl (_general_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_bottom_crop, 0);
-
-               _general_sizer->Add (s);
-       }
-
-       /* VIDEO-only stuff */
-       {
-               video_control (add_label_to_sizer (_general_sizer, _general_panel, "Filters"));
-               wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               _filters = new wxStaticText (_general_panel, wxID_ANY, wxT ("None"));
-               video_control (_filters);
-               s->Add (_filters, 1, wxEXPAND | wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM | wxRIGHT, 6);
-               _filters_button = new wxButton (_general_panel, wxID_ANY, wxT ("Edit..."));
-               video_control (_filters_button);
-               s->Add (_filters_button, 0);
-               _general_sizer->Add (s, 1);
-       }
-
-       video_control (add_label_to_sizer (_general_sizer, _general_panel, "Scaler"));
-       _scaler = new wxComboBox (_general_panel, wxID_ANY);
-       _general_sizer->Add (video_control (_scaler), 1);
-
-       video_control (add_label_to_sizer (_general_sizer, _general_panel, "Frames Per Second"));
-       _frames_per_second = new wxStaticText (_general_panel, wxID_ANY, wxT (""));
-       _general_sizer->Add (video_control (_frames_per_second), 1, wxALIGN_CENTER_VERTICAL);
+       _film_panel = new wxPanel (_notebook);
+       _film_sizer = new wxBoxSizer (wxVERTICAL);
+       _film_panel->SetSizer (_film_sizer);
+
+       wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
+       _film_sizer->Add (grid, 0, wxALL, 8);
+
+       add_label_to_sizer (grid, _film_panel, _("Name"));
+       _name = new wxTextCtrl (_film_panel, wxID_ANY);
+       grid->Add (_name, 1, wxEXPAND);
+
+       add_label_to_sizer (grid, _film_panel, _("DCP Name"));
+       _dcp_name = new wxStaticText (_film_panel, wxID_ANY, wxT (""));
+       grid->Add (_dcp_name, 0, wxALIGN_CENTER_VERTICAL | wxSHRINK);
+
+       _use_dci_name = new wxCheckBox (_film_panel, wxID_ANY, _("Use DCI name"));
+       grid->Add (_use_dci_name, 1, wxEXPAND);
+       _edit_dci_button = new wxButton (_film_panel, wxID_ANY, _("Details..."));
+       grid->Add (_edit_dci_button, 0);
+
+       add_label_to_sizer (grid, _film_panel, _("Content"));
+       _content = new wxFilePickerCtrl (_film_panel, wxID_ANY, wxT (""), _("Select Content File"), wxT("*.*"));
+       grid->Add (_content, 1, wxEXPAND);
+
+       _trust_content_header = new wxCheckBox (_film_panel, wxID_ANY, _("Trust content's header"));
+       video_control (_trust_content_header);
+       grid->Add (_trust_content_header, 1);
+       grid->AddSpacer (0);
+
+       add_label_to_sizer (grid, _film_panel, _("Content Type"));
+       _dcp_content_type = new wxChoice (_film_panel, wxID_ANY);
+       grid->Add (_dcp_content_type);
+
+       video_control (add_label_to_sizer (grid, _film_panel, _("Original Frame Rate")));
+       _frames_per_second = new wxStaticText (_film_panel, wxID_ANY, wxT (""));
+       grid->Add (video_control (_frames_per_second), 1, wxALIGN_CENTER_VERTICAL);
        
-       video_control (add_label_to_sizer (_general_sizer, _general_panel, "Original Size"));
-       _original_size = new wxStaticText (_general_panel, wxID_ANY, wxT (""));
-       _general_sizer->Add (video_control (_original_size), 1, wxALIGN_CENTER_VERTICAL);
+       video_control (add_label_to_sizer (grid, _film_panel, _("Original Size")));
+       _original_size = new wxStaticText (_film_panel, wxID_ANY, wxT (""));
+       grid->Add (video_control (_original_size), 1, wxALIGN_CENTER_VERTICAL);
        
-       video_control (add_label_to_sizer (_general_sizer, _general_panel, "Length"));
-       _length = new wxStaticText (_general_panel, wxID_ANY, wxT (""));
-       _general_sizer->Add (video_control (_length), 1, wxALIGN_CENTER_VERTICAL);
+       video_control (add_label_to_sizer (grid, _film_panel, _("Length")));
+       _length = new wxStaticText (_film_panel, wxID_ANY, wxT (""));
+       grid->Add (video_control (_length), 1, wxALIGN_CENTER_VERTICAL);
 
 
        {
-               video_control (add_label_to_sizer (_general_sizer, _general_panel, "Trim frames"));
+               video_control (add_label_to_sizer (grid, _film_panel, _("Trim frames")));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               add_label_to_sizer (s, _general_panel, "Start");
-               _dcp_trim_start = new wxSpinCtrl (_general_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_dcp_trim_start);
-               add_label_to_sizer (s, _general_panel, "End");
-               _dcp_trim_end = new wxSpinCtrl (_general_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_dcp_trim_end);
-
-               _general_sizer->Add (s);
+               video_control (add_label_to_sizer (s, _film_panel, _("Start")));
+               _trim_start = new wxSpinCtrl (_film_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
+               s->Add (video_control (_trim_start));
+               video_control (add_label_to_sizer (s, _film_panel, _("End")));
+               _trim_end = new wxSpinCtrl (_film_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
+               s->Add (video_control (_trim_end));
+
+               grid->Add (s);
        }
 
-       _dcp_ab = new wxCheckBox (_general_panel, wxID_ANY, wxT ("A/B"));
+       _dcp_ab = new wxCheckBox (_film_panel, wxID_ANY, _("A/B"));
        video_control (_dcp_ab);
-       _general_sizer->Add (_dcp_ab, 1);
-       _general_sizer->AddSpacer (0);
+       grid->Add (_dcp_ab, 1);
+       grid->AddSpacer (0);
 
        /* STILL-only stuff */
        {
-               still_control (add_label_to_sizer (_general_sizer, _general_panel, "Duration"));
+               still_control (add_label_to_sizer (grid, _film_panel, _("Duration")));
                wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               _still_duration = new wxSpinCtrl (_general_panel);
+               _still_duration = new wxSpinCtrl (_film_panel);
                still_control (_still_duration);
                s->Add (_still_duration, 1, wxEXPAND);
-               still_control (add_label_to_sizer (s, _general_panel, "s"));
-               _general_sizer->Add (s);
+               /* TRANSLATORS: `s' here is an abbreviation for seconds, the unit of time */
+               still_control (add_label_to_sizer (s, _film_panel, _("s")));
+               grid->Add (s);
        }
 
-       /* Set up our editing widgets */
-       
-       _left_crop->SetRange (0, 1024);
-       _top_crop->SetRange (0, 1024);
-       _right_crop->SetRange (0, 1024);
-       _bottom_crop->SetRange (0, 1024);
-       _still_duration->SetRange (0, 60 * 60);
-       _dcp_trim_start->SetRange (0, 100);
-       _dcp_trim_end->SetRange (0, 100);
-
        vector<DCPContentType const *> const ct = DCPContentType::all ();
        for (vector<DCPContentType const *>::const_iterator i = ct.begin(); i != ct.end(); ++i) {
                _dcp_content_type->Append (std_to_wx ((*i)->pretty_name ()));
        }
-
-       vector<Scaler const *> const sc = Scaler::all ();
-       for (vector<Scaler const *>::const_iterator i = sc.begin(); i != sc.end(); ++i) {
-               _scaler->Append (std_to_wx ((*i)->name()));
-       }
-
-       JobManager::instance()->ActiveJobsChanged.connect (
-               bind (&FilmEditor::active_jobs_changed, this, _1)
-               );
 }
 
 void
@@ -221,67 +181,176 @@ FilmEditor::connect_to_widgets ()
        _name->Connect (wxID_ANY, wxEVT_COMMAND_TEXT_UPDATED, wxCommandEventHandler (FilmEditor::name_changed), 0, this);
        _use_dci_name->Connect (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler (FilmEditor::use_dci_name_toggled), 0, this);
        _edit_dci_button->Connect (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler (FilmEditor::edit_dci_button_clicked), 0, this);
-       _format->Connect (wxID_ANY, wxEVT_COMMAND_COMBOBOX_SELECTED, wxCommandEventHandler (FilmEditor::format_changed), 0, this);
+       _format->Connect (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED, wxCommandEventHandler (FilmEditor::format_changed), 0, this);
        _content->Connect (wxID_ANY, wxEVT_COMMAND_FILEPICKER_CHANGED, wxCommandEventHandler (FilmEditor::content_changed), 0, this);
+       _trust_content_header->Connect (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler (FilmEditor::trust_content_header_changed), 0, this);
        _left_crop->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::left_crop_changed), 0, this);
        _right_crop->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::right_crop_changed), 0, this);
        _top_crop->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::top_crop_changed), 0, this);
        _bottom_crop->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::bottom_crop_changed), 0, this);
        _filters_button->Connect (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler (FilmEditor::edit_filters_clicked), 0, this);
-       _scaler->Connect (wxID_ANY, wxEVT_COMMAND_COMBOBOX_SELECTED, wxCommandEventHandler (FilmEditor::scaler_changed), 0, this);
-       _dcp_content_type->Connect (wxID_ANY, wxEVT_COMMAND_COMBOBOX_SELECTED, wxCommandEventHandler (FilmEditor::dcp_content_type_changed), 0, this);
+       _scaler->Connect (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED, wxCommandEventHandler (FilmEditor::scaler_changed), 0, this);
+       _dcp_content_type->Connect (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED, wxCommandEventHandler (FilmEditor::dcp_content_type_changed), 0, this);
        _dcp_ab->Connect (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler (FilmEditor::dcp_ab_toggled), 0, this);
        _still_duration->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::still_duration_changed), 0, this);
-       _dcp_trim_start->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::dcp_trim_start_changed), 0, this);
-       _dcp_trim_end->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::dcp_trim_end_changed), 0, this);
+       _trim_start->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::trim_start_changed), 0, this);
+       _trim_end->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::trim_end_changed), 0, this);
        _with_subtitles->Connect (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler (FilmEditor::with_subtitles_toggled), 0, this);
        _subtitle_offset->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::subtitle_offset_changed), 0, this);
        _subtitle_scale->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::subtitle_scale_changed), 0, this);
-       _subtitle_stream->Connect (wxID_ANY, wxEVT_COMMAND_COMBOBOX_SELECTED, wxCommandEventHandler (FilmEditor::subtitle_stream_changed), 0, this);
-       _audio_stream->Connect (wxID_ANY, wxEVT_COMMAND_COMBOBOX_SELECTED, wxCommandEventHandler (FilmEditor::audio_stream_changed), 0, this);
+       _colour_lut->Connect (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED, wxCommandEventHandler (FilmEditor::colour_lut_changed), 0, this);
+       _j2k_bandwidth->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::j2k_bandwidth_changed), 0, this);
+       _subtitle_stream->Connect (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED, wxCommandEventHandler (FilmEditor::subtitle_stream_changed), 0, this);
+       _audio_stream->Connect (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED, wxCommandEventHandler (FilmEditor::audio_stream_changed), 0, this);
        _audio_gain->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::audio_gain_changed), 0, this);
        _audio_gain_calculate_button->Connect (
                wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler (FilmEditor::audio_gain_calculate_button_clicked), 0, this
                );
+       _show_audio->Connect (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler (FilmEditor::show_audio_clicked), 0, this);
        _audio_delay->Connect (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED, wxCommandEventHandler (FilmEditor::audio_delay_changed), 0, this);
+       _use_content_audio->Connect (wxID_ANY, wxEVT_COMMAND_RADIOBUTTON_SELECTED, wxCommandEventHandler (FilmEditor::use_audio_changed), 0, this);
+       _use_external_audio->Connect (wxID_ANY, wxEVT_COMMAND_RADIOBUTTON_SELECTED, wxCommandEventHandler (FilmEditor::use_audio_changed), 0, this);
+       for (int i = 0; i < MAX_AUDIO_CHANNELS; ++i) {
+               _external_audio[i]->Connect (
+                       wxID_ANY, wxEVT_COMMAND_FILEPICKER_CHANGED, wxCommandEventHandler (FilmEditor::external_audio_changed), 0, this
+                       );
+       }
+}
+
+void
+FilmEditor::make_video_panel ()
+{
+       _video_panel = new wxPanel (_notebook);
+       _video_sizer = new wxBoxSizer (wxVERTICAL);
+       _video_panel->SetSizer (_video_sizer);
+       
+       wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
+       _video_sizer->Add (grid, 0, wxALL, 8);
+
+       add_label_to_sizer (grid, _video_panel, _("Format"));
+       _format = new wxChoice (_video_panel, wxID_ANY);
+       grid->Add (_format);
+
+       add_label_to_sizer (grid, _video_panel, _("Left crop"));
+       _left_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
+       grid->Add (_left_crop);
+
+       add_label_to_sizer (grid, _video_panel, _("Right crop"));
+       _right_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
+       grid->Add (_right_crop);
+       
+       add_label_to_sizer (grid, _video_panel, _("Top crop"));
+       _top_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
+       grid->Add (_top_crop);
+       
+       add_label_to_sizer (grid, _video_panel, _("Bottom crop"));
+       _bottom_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
+       grid->Add (_bottom_crop);
+
+       /* VIDEO-only stuff */
+       {
+               video_control (add_label_to_sizer (grid, _video_panel, _("Filters")));
+               wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+               _filters = new wxStaticText (_video_panel, wxID_ANY, _("None"));
+               video_control (_filters);
+               s->Add (_filters, 1, wxEXPAND | wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM | wxRIGHT, 6);
+               _filters_button = new wxButton (_video_panel, wxID_ANY, _("Edit..."));
+               video_control (_filters_button);
+               s->Add (_filters_button, 0);
+               grid->Add (s, 1);
+       }
+
+       video_control (add_label_to_sizer (grid, _video_panel, _("Scaler")));
+       _scaler = new wxChoice (_video_panel, wxID_ANY);
+       grid->Add (video_control (_scaler), 1);
+
+       vector<Scaler const *> const sc = Scaler::all ();
+       for (vector<Scaler const *>::const_iterator i = sc.begin(); i != sc.end(); ++i) {
+               _scaler->Append (std_to_wx ((*i)->name()));
+       }
+
+       add_label_to_sizer (grid, _video_panel, _("Colour look-up table"));
+       _colour_lut = new wxChoice (_video_panel, wxID_ANY);
+       for (int i = 0; i < 2; ++i) {
+               _colour_lut->Append (std_to_wx (colour_lut_index_to_name (i)));
+       }
+       _colour_lut->SetSelection (0);
+       grid->Add (_colour_lut, 1, wxEXPAND);
+
+       {
+               add_label_to_sizer (grid, _video_panel, _("JPEG2000 bandwidth"));
+               wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+               _j2k_bandwidth = new wxSpinCtrl (_video_panel, wxID_ANY);
+               s->Add (_j2k_bandwidth, 1);
+               add_label_to_sizer (s, _video_panel, _("MBps"));
+               grid->Add (s, 1);
+       }
+
+       _left_crop->SetRange (0, 1024);
+       _top_crop->SetRange (0, 1024);
+       _right_crop->SetRange (0, 1024);
+       _bottom_crop->SetRange (0, 1024);
+       _still_duration->SetRange (1, 60 * 60);
+       _trim_start->SetRange (0, 100);
+       _trim_end->SetRange (0, 100);
+       _j2k_bandwidth->SetRange (50, 250);
 }
 
 void
 FilmEditor::make_audio_panel ()
 {
        _audio_panel = new wxPanel (_notebook);
-       _audio_sizer = new wxFlexGridSizer (2, 4, 4);
+       _audio_sizer = new wxBoxSizer (wxVERTICAL);
        _audio_panel->SetSizer (_audio_sizer);
+       
+       wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
+       _audio_sizer->Add (grid, 0, wxALL, 8);
 
-       {
-               video_control (add_label_to_sizer (_audio_sizer, _audio_panel, "Audio Stream"));
-               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               _audio_stream = new wxComboBox (_audio_panel, wxID_ANY);
-               s->Add (video_control (_audio_stream), 1);
-               _audio = new wxStaticText (_audio_panel, wxID_ANY, wxT (""));
-               s->Add (video_control (_audio), 1, wxALIGN_CENTER_VERTICAL | wxLEFT, 8);
-               _audio_sizer->Add (s, 1, wxEXPAND);
-       }
+       _show_audio = new wxButton (_audio_panel, wxID_ANY, _("Show Audio..."));
+       grid->Add (_show_audio, 1);
+       grid->AddSpacer (0);
 
        {
-               video_control (add_label_to_sizer (_audio_sizer, _audio_panel, "Audio Gain"));
+               video_control (add_label_to_sizer (grid, _audio_panel, _("Audio Gain")));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _audio_gain = new wxSpinCtrl (_audio_panel);
                s->Add (video_control (_audio_gain), 1);
-               video_control (add_label_to_sizer (s, _audio_panel, "dB"));
+               video_control (add_label_to_sizer (s, _audio_panel, _("dB")));
                _audio_gain_calculate_button = new wxButton (_audio_panel, wxID_ANY, _("Calculate..."));
                video_control (_audio_gain_calculate_button);
                s->Add (_audio_gain_calculate_button, 1, wxEXPAND);
-               _audio_sizer->Add (s);
+               grid->Add (s);
        }
 
        {
-               video_control (add_label_to_sizer (_audio_sizer, _audio_panel, "Audio Delay"));
+               video_control (add_label_to_sizer (grid, _audio_panel, _("Audio Delay")));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _audio_delay = new wxSpinCtrl (_audio_panel);
                s->Add (video_control (_audio_delay), 1);
-               video_control (add_label_to_sizer (s, _audio_panel, "ms"));
-               _audio_sizer->Add (s);
+               /* TRANSLATORS: this is an abbreviation for milliseconds, the unit of time */
+               video_control (add_label_to_sizer (s, _audio_panel, _("ms")));
+               grid->Add (s);
+       }
+
+       {
+               _use_content_audio = new wxRadioButton (_audio_panel, wxID_ANY, _("Use content's audio"), wxDefaultPosition, wxDefaultSize, wxRB_GROUP);
+               grid->Add (video_control (_use_content_audio));
+               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+               _audio_stream = new wxChoice (_audio_panel, wxID_ANY);
+               s->Add (video_control (_audio_stream), 1);
+               _audio = new wxStaticText (_audio_panel, wxID_ANY, wxT (""));
+               s->Add (video_control (_audio), 1, wxALIGN_CENTER_VERTICAL | wxLEFT, 8);
+               grid->Add (s, 1, wxEXPAND);
+       }
+
+       _use_external_audio = new wxRadioButton (_audio_panel, wxID_ANY, _("Use external audio"));
+       grid->Add (_use_external_audio);
+       grid->AddSpacer (0);
+
+       for (int i = 0; i < MAX_AUDIO_CHANNELS; ++i) {
+               add_label_to_sizer (grid, _audio_panel, audio_channel_name (i));
+               _external_audio[i] = new wxFilePickerCtrl (_audio_panel, wxID_ANY, wxT (""), _("Select Audio File"), wxT ("*.wav"));
+               grid->Add (_external_audio[i], 1, wxEXPAND);
        }
 
        _audio_gain->SetRange (-60, 60);
@@ -292,27 +361,29 @@ void
 FilmEditor::make_subtitle_panel ()
 {
        _subtitle_panel = new wxPanel (_notebook);
-       _subtitle_sizer = new wxFlexGridSizer (2, 4, 4);
+       _subtitle_sizer = new wxBoxSizer (wxVERTICAL);
        _subtitle_panel->SetSizer (_subtitle_sizer);
+       wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
+       _subtitle_sizer->Add (grid, 0, wxALL, 8);
 
-       _with_subtitles = new wxCheckBox (_subtitle_panel, wxID_ANY, wxT("With Subtitles"));
+       _with_subtitles = new wxCheckBox (_subtitle_panel, wxID_ANY, _("With Subtitles"));
        video_control (_with_subtitles);
-       _subtitle_sizer->Add (_with_subtitles, 1);
+       grid->Add (_with_subtitles, 1);
        
-       _subtitle_stream = new wxComboBox (_subtitle_panel, wxID_ANY);
-       _subtitle_sizer->Add (_subtitle_stream);
+       _subtitle_stream = new wxChoice (_subtitle_panel, wxID_ANY);
+       grid->Add (video_control (_subtitle_stream));
 
-       video_control (add_label_to_sizer (_subtitle_sizer, _subtitle_panel, "Subtitle Offset"));
+       video_control (add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Offset")));
        _subtitle_offset = new wxSpinCtrl (_subtitle_panel);
-       _subtitle_sizer->Add (video_control (_subtitle_offset), 1);
+       grid->Add (video_control (_subtitle_offset), 1);
 
        {
-               video_control (add_label_to_sizer (_subtitle_sizer, _subtitle_panel, "Subtitle Scale"));
+               video_control (add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Scale")));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _subtitle_scale = new wxSpinCtrl (_subtitle_panel);
                s->Add (video_control (_subtitle_scale));
-               video_control (add_label_to_sizer (s, _subtitle_panel, "%"));
-               _subtitle_sizer->Add (s);
+               video_control (add_label_to_sizer (s, _subtitle_panel, _("%")));
+               grid->Add (s);
        }
 
        _subtitle_offset->SetRange (-1024, 1024);
@@ -375,8 +446,18 @@ FilmEditor::content_changed (wxCommandEvent &)
                _film->set_content (wx_to_std (_content->GetPath ()));
        } catch (std::exception& e) {
                _content->SetPath (std_to_wx (_film->directory ()));
-               error_dialog (this, String::compose ("Could not set content: %1", e.what ()));
+               error_dialog (this, wxString::Format (_("Could not set content: %s"), e.what ()));
+       }
+}
+
+void
+FilmEditor::trust_content_header_changed (wxCommandEvent &)
+{
+       if (!_film) {
+               return;
        }
+
+       _film->set_trust_content_header (_trust_content_header->GetValue ());
 }
 
 /** Called when the DCP A/B switch has been toggled */
@@ -421,6 +502,26 @@ FilmEditor::subtitle_scale_changed (wxCommandEvent &)
        _film->set_subtitle_scale (_subtitle_scale->GetValue() / 100.0);
 }
 
+void
+FilmEditor::colour_lut_changed (wxCommandEvent &)
+{
+       if (!_film) {
+               return;
+       }
+       
+       _film->set_colour_lut (_colour_lut->GetSelection ());
+}
+
+void
+FilmEditor::j2k_bandwidth_changed (wxCommandEvent &)
+{
+       if (!_film) {
+               return;
+       }
+       
+       _film->set_j2k_bandwidth (_j2k_bandwidth->GetValue() * 1e6);
+}      
+
 
 /** Called when the metadata stored in the Film object has changed;
  *  so that we can update the GUI.
@@ -447,12 +548,14 @@ FilmEditor::film_changed (Film::Property p)
                setup_subtitle_control_sensitivity ();
                setup_streams ();
                break;
-       case Film::HAS_SUBTITLES:
+       case Film::TRUST_CONTENT_HEADER:
+               checked_set (_trust_content_header, _film->trust_content_header ());
+               break;
+       case Film::SUBTITLE_STREAMS:
                setup_subtitle_control_sensitivity ();
                setup_streams ();
                break;
-       case Film::AUDIO_STREAMS:
-       case Film::SUBTITLE_STREAMS:
+       case Film::CONTENT_AUDIO_STREAMS:
                setup_streams ();
                break;
        case Film::FORMAT:
@@ -463,8 +566,12 @@ FilmEditor::film_changed (Film::Property p)
                        ++i;
                        ++n;
                }
-               checked_set (_format, n);
-               _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
+               if (i == _formats.end()) {
+                       checked_set (_format, -1);
+               } else {
+                       checked_set (_format, n);
+               }
+               setup_dcp_name ();
                break;
        }
        case Film::CROP:
@@ -482,21 +589,17 @@ FilmEditor::film_changed (Film::Property p)
                        string const b = p.first + " " + p.second;
                        _filters->SetLabel (std_to_wx (b));
                }
-               _general_sizer->Layout ();
+               _film_sizer->Layout ();
                break;
        }
        case Film::NAME:
                checked_set (_name, _film->name());
-               _film->set_dci_date_today ();
-               _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
+               setup_dcp_name ();
                break;
        case Film::FRAMES_PER_SECOND:
                s << fixed << setprecision(2) << _film->frames_per_second();
                _frames_per_second->SetLabel (std_to_wx (s.str ()));
                break;
-       case Film::AUDIO_SAMPLE_RATE:
-               setup_audio_details ();
-               break;
        case Film::SIZE:
                if (_film->size().width == 0 && _film->size().height == 0) {
                        _original_size->SetLabel (wxT (""));
@@ -513,15 +616,15 @@ FilmEditor::film_changed (Film::Property p)
                } 
                _length->SetLabel (std_to_wx (s.str ()));
                if (_film->length()) {
-                       _dcp_trim_start->SetRange (0, _film->length().get());
-                       _dcp_trim_end->SetRange (0, _film->length().get());
+                       _trim_start->SetRange (0, _film->length().get());
+                       _trim_end->SetRange (0, _film->length().get());
                }
                break;
+       case Film::DCP_INTRINSIC_DURATION:
+               break;
        case Film::DCP_CONTENT_TYPE:
                checked_set (_dcp_content_type, DCPContentType::as_index (_film->dcp_content_type ()));
-               _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
-               break;
-       case Film::THUMBS:
+               setup_dcp_name ();
                break;
        case Film::DCP_AB:
                checked_set (_dcp_ab, _film->dcp_ab ());
@@ -529,11 +632,11 @@ FilmEditor::film_changed (Film::Property p)
        case Film::SCALER:
                checked_set (_scaler, Scaler::as_index (_film->scaler ()));
                break;
-       case Film::DCP_TRIM_START:
-               checked_set (_dcp_trim_start, _film->dcp_trim_start());
+       case Film::TRIM_START:
+               checked_set (_trim_start, _film->trim_start());
                break;
-       case Film::DCP_TRIM_END:
-               checked_set (_dcp_trim_end, _film->dcp_trim_end());
+       case Film::TRIM_END:
+               checked_set (_trim_end, _film->trim_end());
                break;
        case Film::AUDIO_GAIN:
                checked_set (_audio_gain, _film->audio_gain ());
@@ -546,10 +649,8 @@ FilmEditor::film_changed (Film::Property p)
                break;
        case Film::WITH_SUBTITLES:
                checked_set (_with_subtitles, _film->with_subtitles ());
-               _subtitle_stream->Enable (_film->with_subtitles ());
-               _subtitle_scale->Enable (_film->with_subtitles ());
-               _subtitle_offset->Enable (_film->with_subtitles ());
-               _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
+               setup_subtitle_control_sensitivity ();
+               setup_dcp_name ();
                break;
        case Film::SUBTITLE_OFFSET:
                checked_set (_subtitle_offset, _film->subtitle_offset ());
@@ -557,21 +658,48 @@ FilmEditor::film_changed (Film::Property p)
        case Film::SUBTITLE_SCALE:
                checked_set (_subtitle_scale, _film->subtitle_scale() * 100);
                break;
+       case Film::COLOUR_LUT:
+               checked_set (_colour_lut, _film->colour_lut ());
+               break;
+       case Film::J2K_BANDWIDTH:
+               checked_set (_j2k_bandwidth, double (_film->j2k_bandwidth()) / 1e6);
+               break;
        case Film::USE_DCI_NAME:
                checked_set (_use_dci_name, _film->use_dci_name ());
-               _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
+               setup_dcp_name ();
                break;
        case Film::DCI_METADATA:
-               _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
+               setup_dcp_name ();
+               break;
+       case Film::CONTENT_AUDIO_STREAM:
+               if (_film->content_audio_stream()) {
+                       checked_set (_audio_stream, _film->content_audio_stream()->to_string());
+               }
+               setup_dcp_name ();
+               setup_audio_details ();
+               setup_audio_control_sensitivity ();
                break;
-       case Film::AUDIO_STREAM:
-               checked_set (_audio_stream, _film->audio_stream_index ());
-               _dcp_name->SetLabel (std_to_wx (_film->dcp_name ()));
+       case Film::USE_CONTENT_AUDIO:
+               checked_set (_use_content_audio, _film->use_content_audio());
+               checked_set (_use_external_audio, !_film->use_content_audio());
+               setup_dcp_name ();
                setup_audio_details ();
+               setup_audio_control_sensitivity ();
                break;
        case Film::SUBTITLE_STREAM:
-               checked_set (_subtitle_stream, _film->subtitle_stream_index ());
+               if (_film->subtitle_stream()) {
+                       checked_set (_subtitle_stream, _film->subtitle_stream()->to_string());
+               }
                break;
+       case Film::EXTERNAL_AUDIO:
+       {
+               vector<string> a = _film->external_audio ();
+               for (size_t i = 0; i < a.size() && i < MAX_AUDIO_CHANNELS; ++i) {
+                       checked_set (_external_audio[i], a[i]);
+               }
+               setup_audio_details ();
+               break;
+       }
        }
 }
 
@@ -599,7 +727,7 @@ FilmEditor::dcp_content_type_changed (wxCommandEvent &)
        }
 
        int const n = _dcp_content_type->GetSelection ();
-       if (n >= 0) {
+       if (n != wxNOT_FOUND) {
                _film->set_dcp_content_type (DCPContentType::from_index (n));
        }
 }
@@ -621,30 +749,40 @@ FilmEditor::set_film (shared_ptr<Film> f)
        } else {
                FileChanged ("");
        }
+
+       if (_audio_dialog) {
+               _audio_dialog->set_film (_film);
+       }
        
        film_changed (Film::NAME);
+       film_changed (Film::USE_DCI_NAME);
        film_changed (Film::CONTENT);
+       film_changed (Film::TRUST_CONTENT_HEADER);
        film_changed (Film::DCP_CONTENT_TYPE);
        film_changed (Film::FORMAT);
        film_changed (Film::CROP);
        film_changed (Film::FILTERS);
-       film_changed (Film::DCP_TRIM_START);
-       film_changed (Film::DCP_TRIM_END);
-       film_changed (Film::DCP_AB);
-       film_changed (Film::SIZE);
-       film_changed (Film::LENGTH);
-       film_changed (Film::FRAMES_PER_SECOND);
-       film_changed (Film::AUDIO_SAMPLE_RATE);
        film_changed (Film::SCALER);
+       film_changed (Film::TRIM_START);
+       film_changed (Film::TRIM_END);
+       film_changed (Film::DCP_AB);
+       film_changed (Film::CONTENT_AUDIO_STREAM);
+       film_changed (Film::EXTERNAL_AUDIO);
+       film_changed (Film::USE_CONTENT_AUDIO);
        film_changed (Film::AUDIO_GAIN);
        film_changed (Film::AUDIO_DELAY);
        film_changed (Film::STILL_DURATION);
        film_changed (Film::WITH_SUBTITLES);
-       film_changed (Film::HAS_SUBTITLES);
        film_changed (Film::SUBTITLE_OFFSET);
        film_changed (Film::SUBTITLE_SCALE);
-       film_changed (Film::USE_DCI_NAME);
+       film_changed (Film::COLOUR_LUT);
+       film_changed (Film::J2K_BANDWIDTH);
        film_changed (Film::DCI_METADATA);
+       film_changed (Film::SIZE);
+       film_changed (Film::LENGTH);
+       film_changed (Film::CONTENT_AUDIO_STREAMS);
+       film_changed (Film::SUBTITLE_STREAMS);
+       film_changed (Film::FRAMES_PER_SECOND);
 }
 
 /** Updates the sensitivity of lots of widgets to a given value.
@@ -660,6 +798,7 @@ FilmEditor::set_things_sensitive (bool s)
        _edit_dci_button->Enable (s);
        _format->Enable (s);
        _content->Enable (s);
+       _trust_content_header->Enable (s);
        _left_crop->Enable (s);
        _right_crop->Enable (s);
        _top_crop->Enable (s);
@@ -668,15 +807,19 @@ FilmEditor::set_things_sensitive (bool s)
        _scaler->Enable (s);
        _audio_stream->Enable (s);
        _dcp_content_type->Enable (s);
-       _dcp_trim_start->Enable (s);
-       _dcp_trim_end->Enable (s);
+       _trim_start->Enable (s);
+       _trim_end->Enable (s);
        _dcp_ab->Enable (s);
+       _colour_lut->Enable (s);
+       _j2k_bandwidth->Enable (s);
        _audio_gain->Enable (s);
        _audio_gain_calculate_button->Enable (s);
+       _show_audio->Enable (s);
        _audio_delay->Enable (s);
        _still_duration->Enable (s);
 
        setup_subtitle_control_sensitivity ();
+       setup_audio_control_sensitivity ();
 }
 
 /** Called when the `Edit filters' button has been clicked */
@@ -754,7 +897,19 @@ FilmEditor::setup_visibility ()
                (*i)->Show (c == STILL);
        }
 
-       _general_sizer->Layout ();
+       _notebook->InvalidateBestSize ();
+       
+       _film_sizer->Layout ();
+       _film_sizer->SetSizeHints (_film_panel);
+       _video_sizer->Layout ();
+       _video_sizer->SetSizeHints (_video_panel);
+       _audio_sizer->Layout ();
+       _audio_sizer->SetSizeHints (_audio_panel);
+       _subtitle_sizer->Layout ();
+       _subtitle_sizer->SetSizeHints (_subtitle_panel);
+
+       _notebook->Fit ();
+       Fit ();
 }
 
 void
@@ -768,23 +923,23 @@ FilmEditor::still_duration_changed (wxCommandEvent &)
 }
 
 void
-FilmEditor::dcp_trim_start_changed (wxCommandEvent &)
+FilmEditor::trim_start_changed (wxCommandEvent &)
 {
        if (!_film) {
                return;
        }
 
-       _film->set_dcp_trim_start (_dcp_trim_start->GetValue ());
+       _film->set_trim_start (_trim_start->GetValue ());
 }
 
 void
-FilmEditor::dcp_trim_end_changed (wxCommandEvent &)
+FilmEditor::trim_end_changed (wxCommandEvent &)
 {
        if (!_film) {
                return;
        }
 
-       _film->set_dcp_trim_end (_dcp_trim_end->GetValue ());
+       _film->set_trim_end (_trim_end->GetValue ());
 }
 
 void
@@ -839,7 +994,7 @@ FilmEditor::setup_formats ()
                _format->Append (std_to_wx ((*i)->name ()));
        }
 
-       _general_sizer->Layout ();
+       _film_sizer->Layout ();
 }
 
 void
@@ -857,13 +1012,34 @@ FilmEditor::setup_subtitle_control_sensitivity ()
 {
        bool h = false;
        if (_generally_sensitive && _film) {
-               h = _film->has_subtitles();
+               h = !_film->subtitle_streams().empty();
        }
        
        _with_subtitles->Enable (h);
-       _subtitle_stream->Enable (h);
-       _subtitle_offset->Enable (h);
-       _subtitle_scale->Enable (h);
+
+       bool j = false;
+       if (_film) {
+               j = _film->with_subtitles ();
+       }
+       
+       _subtitle_stream->Enable (j);
+       _subtitle_offset->Enable (j);
+       _subtitle_scale->Enable (j);
+}
+
+void
+FilmEditor::setup_audio_control_sensitivity ()
+{
+       _use_content_audio->Enable (_generally_sensitive);
+       _use_external_audio->Enable (_generally_sensitive);
+       
+       bool const source = _generally_sensitive && _use_content_audio->GetValue();
+       bool const external = _generally_sensitive && _use_external_audio->GetValue();
+
+       _audio_stream->Enable (source);
+       for (int i = 0; i < MAX_AUDIO_CHANNELS; ++i) {
+               _external_audio[i]->Enable (external);
+       }
 }
 
 void
@@ -883,8 +1059,9 @@ FilmEditor::edit_dci_button_clicked (wxCommandEvent &)
                return;
        }
 
-       DCINameDialog* d = new DCINameDialog (this, _film);
+       DCIMetadataDialog* d = new DCIMetadataDialog (this, _film->dci_metadata ());
        d->ShowModal ();
+       _film->set_dci_metadata (d->dci_metadata ());
        d->Destroy ();
 }
 
@@ -892,18 +1069,27 @@ void
 FilmEditor::setup_streams ()
 {
        _audio_stream->Clear ();
-       vector<AudioStream> a = _film->audio_streams ();
-       for (vector<AudioStream>::iterator i = a.begin(); i != a.end(); ++i) {
-               _audio_stream->Append (std_to_wx (i->name()));
+       vector<shared_ptr<AudioStream> > a = _film->content_audio_streams ();
+       for (vector<shared_ptr<AudioStream> >::iterator i = a.begin(); i != a.end(); ++i) {
+               shared_ptr<FFmpegAudioStream> ffa = dynamic_pointer_cast<FFmpegAudioStream> (*i);
+               assert (ffa);
+               _audio_stream->Append (std_to_wx (ffa->name()), new wxStringClientData (std_to_wx (ffa->to_string ())));
+       }
+       
+       if (_film->use_content_audio() && _film->audio_stream()) {
+               checked_set (_audio_stream, _film->audio_stream()->to_string());
        }
-       _audio_stream->SetSelection (_film->audio_stream_index ());
 
        _subtitle_stream->Clear ();
-       vector<SubtitleStream> s = _film->subtitle_streams ();
-       for (vector<SubtitleStream>::iterator i = s.begin(); i != s.end(); ++i) {
-               _subtitle_stream->Append (std_to_wx (i->name()));
+       vector<shared_ptr<SubtitleStream> > s = _film->subtitle_streams ();
+       for (vector<shared_ptr<SubtitleStream> >::iterator i = s.begin(); i != s.end(); ++i) {
+               _subtitle_stream->Append (std_to_wx ((*i)->name()), new wxStringClientData (std_to_wx ((*i)->to_string ())));
+       }
+       if (_film->subtitle_stream()) {
+               checked_set (_subtitle_stream, _film->subtitle_stream()->to_string());
+       } else {
+               _subtitle_stream->SetSelection (wxNOT_FOUND);
        }
-       _subtitle_stream->SetSelection (_film->subtitle_stream_index ());
 }
 
 void
@@ -913,7 +1099,12 @@ FilmEditor::audio_stream_changed (wxCommandEvent &)
                return;
        }
 
-       _film->set_audio_stream (_audio_stream->GetSelection ());
+       _film->set_content_audio_stream (
+               audio_stream_factory (
+                       string_client_data (_audio_stream->GetClientObject (_audio_stream->GetSelection ())),
+                       Film::state_version
+                       )
+               );
 }
 
 void
@@ -923,17 +1114,27 @@ FilmEditor::subtitle_stream_changed (wxCommandEvent &)
                return;
        }
 
-       _film->set_subtitle_stream (_subtitle_stream->GetSelection ());
+       _film->set_subtitle_stream (
+               subtitle_stream_factory (
+                       string_client_data (_subtitle_stream->GetClientObject (_subtitle_stream->GetSelection ())),
+                       Film::state_version
+                       )
+               );
 }
 
 void
 FilmEditor::setup_audio_details ()
 {
-       if (_film->audio_channels() == 0 && _film->audio_sample_rate() == 0) {
+       if (!_film->audio_stream()) {
                _audio->SetLabel (wxT (""));
        } else {
                stringstream s;
-               s << _film->audio_channels () << " channels, " << _film->audio_sample_rate() << "Hz";
+               if (_film->audio_stream()->channels() == 1) {
+                       s << "1 channel";
+               } else {
+                       s << _film->audio_stream()->channels () << " channels";
+               }
+               s << ", " << _film->audio_stream()->sample_rate() << "Hz";
                _audio->SetLabel (std_to_wx (s.str ()));
        }
 }
@@ -943,3 +1144,45 @@ FilmEditor::active_jobs_changed (bool a)
 {
        set_things_sensitive (!a);
 }
+
+void
+FilmEditor::use_audio_changed (wxCommandEvent &)
+{
+       _film->set_use_content_audio (_use_content_audio->GetValue());
+}
+
+void
+FilmEditor::external_audio_changed (wxCommandEvent &)
+{
+       vector<string> a;
+       for (int i = 0; i < MAX_AUDIO_CHANNELS; ++i) {
+               a.push_back (wx_to_std (_external_audio[i]->GetPath()));
+       }
+
+       _film->set_external_audio (a);
+}
+
+void
+FilmEditor::setup_dcp_name ()
+{
+       string s = _film->dcp_name (true);
+       if (s.length() > 28) {
+               _dcp_name->SetLabel (std_to_wx (s.substr (0, 28) + "..."));
+               _dcp_name->SetToolTip (std_to_wx (s));
+       } else {
+               _dcp_name->SetLabel (std_to_wx (s));
+       }
+}
+
+void
+FilmEditor::show_audio_clicked (wxCommandEvent &)
+{
+       if (_audio_dialog) {
+               _audio_dialog->Destroy ();
+               _audio_dialog = 0;
+       }
+       
+       _audio_dialog = new AudioDialog (this);
+       _audio_dialog->Show ();
+       _audio_dialog->set_film (_film);
+}