No-op; fix GPL address and use the explicit-program-name version.
[dcpomatic.git] / src / wx / dcp_panel.cc
index b6fca4604bc6946582ddfd478ff0db95d1e4a8cb..66a0842596cd50d1c85ab544ab8e59c6ac3e71c1 100644 (file)
@@ -1,19 +1,20 @@
 /*
-    Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2016 Carl Hetherington <cth@carlh.net>
 
-    This program is free software; you can redistribute it and/or modify
+    This file is part of DCP-o-matic.
+
+    DCP-o-matic is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     the Free Software Foundation; either version 2 of the License, or
     (at your option) any later version.
 
-    This program is distributed in the hope that it will be useful,
+    DCP-o-matic is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
     GNU General Public License for more details.
 
     You should have received a copy of the GNU General Public License
-    along with this program; if not, write to the Free Software
-    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+    along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
 
 */
 
@@ -21,6 +22,7 @@
 #include "wx_util.h"
 #include "key_dialog.h"
 #include "isdcf_metadata_dialog.h"
+#include "audio_dialog.h"
 #include "lib/ratio.h"
 #include "lib/config.h"
 #include "lib/dcp_content_type.h"
 #include "lib/film.h"
 #include "lib/ffmpeg_content.h"
 #include "lib/audio_processor.h"
+#include "lib/video_content.h"
+#include "lib/subtitle_content.h"
+#include "lib/dcp_content.h"
+#include "lib/audio_content.h"
 #include <dcp/key.h>
+#include <dcp/raw_convert.h>
 #include <wx/wx.h>
 #include <wx/notebook.h>
 #include <wx/gbsizer.h>
 #include <wx/spinctrl.h>
 #include <boost/lexical_cast.hpp>
 #include <boost/foreach.hpp>
+#include <iostream>
 
 using std::cout;
 using std::list;
 using std::string;
 using std::vector;
+using std::pair;
+using std::max;
+using std::make_pair;
 using boost::lexical_cast;
 using boost::shared_ptr;
 
-DCPPanel::DCPPanel (wxNotebook* n, boost::shared_ptr<Film> f)
-       : _film (f)
+DCPPanel::DCPPanel (wxNotebook* n, boost::shared_ptr<Film> film)
+       : _audio_dialog (0)
+       , _film (film)
        , _generally_sensitive (true)
 {
        _panel = new wxPanel (n);
@@ -55,16 +67,16 @@ DCPPanel::DCPPanel (wxNotebook* n, boost::shared_ptr<Film> f)
        _sizer->Add (grid, 0, wxEXPAND | wxALL, 8);
 
        int r = 0;
-       
-       add_label_to_grid_bag_sizer (grid, _panel, _("Name"), true, wxGBPosition (r, 0));
+
+       add_label_to_sizer (grid, _panel, _("Name"), true, wxGBPosition (r, 0));
        _name = new wxTextCtrl (_panel, wxID_ANY);
        grid->Add (_name, wxGBPosition(r, 1), wxDefaultSpan, wxEXPAND | wxLEFT | wxRIGHT);
        ++r;
-       
+
        int flags = wxALIGN_CENTER_VERTICAL;
 #ifdef __WXOSX__
        flags |= wxALIGN_RIGHT;
-#endif 
+#endif
 
        _use_isdcf_name = new wxCheckBox (_panel, wxID_ANY, _("Use ISDCF name"));
        grid->Add (_use_isdcf_name, wxGBPosition (r, 0), wxDefaultSpan, flags);
@@ -90,7 +102,7 @@ DCPPanel::DCPPanel (wxNotebook* n, boost::shared_ptr<Film> f)
        grid->Add (_dcp_name, wxGBPosition(r, 0), wxGBSpan (1, 2), wxALIGN_CENTER_VERTICAL | wxEXPAND);
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _panel, _("Content Type"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (grid, _panel, _("Content Type"), true, wxGBPosition (r, 0));
        _dcp_content_type = new wxChoice (_panel, wxID_ANY);
        grid->Add (_dcp_content_type, wxGBPosition (r, 1));
        ++r;
@@ -100,11 +112,11 @@ DCPPanel::DCPPanel (wxNotebook* n, boost::shared_ptr<Film> f)
 
        _notebook->AddPage (make_video_panel (), _("Video"), false);
        _notebook->AddPage (make_audio_panel (), _("Audio"), false);
-       
+
        _signed = new wxCheckBox (_panel, wxID_ANY, _("Signed"));
        grid->Add (_signed, wxGBPosition (r, 0), wxGBSpan (1, 2));
        ++r;
-       
+
        _encrypted = new wxCheckBox (_panel, wxID_ANY, _("Encrypted"));
        grid->Add (_encrypted, wxGBPosition (r, 0), wxGBSpan (1, 2));
        ++r;
@@ -114,7 +126,7 @@ DCPPanel::DCPPanel (wxNotebook* n, boost::shared_ptr<Film> f)
         size.SetHeight (-1);
 
        {
-               add_label_to_grid_bag_sizer (grid, _panel, _("Key"), true, wxGBPosition (r, 0));
+               add_label_to_sizer (grid, _panel, _("Key"), true, wxGBPosition (r, 0));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _key = new wxStaticText (_panel, wxID_ANY, "", wxDefaultPosition, size);
                s->Add (_key, 1, wxALIGN_CENTER_VERTICAL);
@@ -123,27 +135,57 @@ DCPPanel::DCPPanel (wxNotebook* n, boost::shared_ptr<Film> f)
                grid->Add (s, wxGBPosition (r, 1));
                ++r;
        }
-       
-       add_label_to_grid_bag_sizer (grid, _panel, _("Standard"), true, wxGBPosition (r, 0));
+
+       add_label_to_sizer (grid, _panel, _("Reels"), true, wxGBPosition (r, 0));
+       _reel_type = new wxChoice (_panel, wxID_ANY);
+       grid->Add (_reel_type, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
+       ++r;
+
+       add_label_to_sizer (grid, _panel, _("Reel length"), true, wxGBPosition (r, 0));
+
+       {
+               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+               _reel_length = new wxSpinCtrl (_panel, wxID_ANY);
+               s->Add (_reel_length);
+               add_label_to_sizer (s, _panel, _("GB"), false);
+               grid->Add (s, wxGBPosition (r, 1));
+               ++r;
+       }
+
+       add_label_to_sizer (grid, _panel, _("Standard"), true, wxGBPosition (r, 0));
        _standard = new wxChoice (_panel, wxID_ANY);
        grid->Add (_standard, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
        ++r;
 
-       _name->Bind              (wxEVT_COMMAND_TEXT_UPDATED,         boost::bind (&DCPPanel::name_changed, this));
-       _use_isdcf_name->Bind    (wxEVT_COMMAND_CHECKBOX_CLICKED,     boost::bind (&DCPPanel::use_isdcf_name_toggled, this));
-       _edit_isdcf_button->Bind (wxEVT_COMMAND_BUTTON_CLICKED,       boost::bind (&DCPPanel::edit_isdcf_button_clicked, this));
+       _upload_after_make_dcp = new wxCheckBox (_panel, wxID_ANY, _("Upload DCP to TMS after it is made"));
+       grid->Add (_upload_after_make_dcp, wxGBPosition (r, 0), wxGBSpan (1, 2));
+       ++r;
+
+       _name->Bind                  (wxEVT_COMMAND_TEXT_UPDATED,     boost::bind (&DCPPanel::name_changed, this));
+       _use_isdcf_name->Bind        (wxEVT_COMMAND_CHECKBOX_CLICKED, boost::bind (&DCPPanel::use_isdcf_name_toggled, this));
+       _edit_isdcf_button->Bind     (wxEVT_COMMAND_BUTTON_CLICKED,   boost::bind (&DCPPanel::edit_isdcf_button_clicked, this));
        _copy_isdcf_name_button->Bind(wxEVT_COMMAND_BUTTON_CLICKED,   boost::bind (&DCPPanel::copy_isdcf_name_button_clicked, this));
-       _dcp_content_type->Bind  (wxEVT_COMMAND_CHOICE_SELECTED,      boost::bind (&DCPPanel::dcp_content_type_changed, this));
-       _signed->Bind            (wxEVT_COMMAND_CHECKBOX_CLICKED,     boost::bind (&DCPPanel::signed_toggled, this));
-       _encrypted->Bind         (wxEVT_COMMAND_CHECKBOX_CLICKED,     boost::bind (&DCPPanel::encrypted_toggled, this));
-       _edit_key->Bind          (wxEVT_COMMAND_BUTTON_CLICKED,       boost::bind (&DCPPanel::edit_key_clicked, this));
-       _standard->Bind          (wxEVT_COMMAND_CHOICE_SELECTED,      boost::bind (&DCPPanel::standard_changed, this));
+       _dcp_content_type->Bind      (wxEVT_COMMAND_CHOICE_SELECTED,  boost::bind (&DCPPanel::dcp_content_type_changed, this));
+       _signed->Bind                (wxEVT_COMMAND_CHECKBOX_CLICKED, boost::bind (&DCPPanel::signed_toggled, this));
+       _encrypted->Bind             (wxEVT_COMMAND_CHECKBOX_CLICKED, boost::bind (&DCPPanel::encrypted_toggled, this));
+       _edit_key->Bind              (wxEVT_COMMAND_BUTTON_CLICKED,   boost::bind (&DCPPanel::edit_key_clicked, this));
+       _reel_type->Bind             (wxEVT_COMMAND_CHOICE_SELECTED,  boost::bind (&DCPPanel::reel_type_changed, this));
+       _reel_length->Bind           (wxEVT_COMMAND_SPINCTRL_UPDATED, boost::bind (&DCPPanel::reel_length_changed, this));
+       _standard->Bind              (wxEVT_COMMAND_CHOICE_SELECTED,  boost::bind (&DCPPanel::standard_changed, this));
+       _upload_after_make_dcp->Bind (wxEVT_COMMAND_CHECKBOX_CLICKED, boost::bind (&DCPPanel::upload_after_make_dcp_changed, this));
 
        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 ()));
        }
 
+       _reel_type->Append (_("Single reel"));
+       _reel_type->Append (_("Split by video content"));
+       /// TRANSLATORS: translate the word "Custom" here; do not include the "Reel|" prefix
+       _reel_type->Append (S_("Reel|Custom"));
+
+       _reel_length->SetRange (1, 64);
+
        _standard->Append (_("SMPTE"));
        _standard->Append (_("Interop"));
 
@@ -176,7 +218,7 @@ DCPPanel::j2k_bandwidth_changed ()
        if (!_film) {
                return;
        }
-       
+
        _film->set_j2k_bandwidth (_j2k_bandwidth->GetValue() * 1000000);
 }
 
@@ -190,16 +232,6 @@ DCPPanel::signed_toggled ()
        _film->set_signed (_signed->GetValue ());
 }
 
-void
-DCPPanel::burn_subtitles_toggled ()
-{
-       if (!_film) {
-               return;
-       }
-
-       _film->set_burn_subtitles (_burn_subtitles->GetValue ());
-}
-
 void
 DCPPanel::encrypted_toggled ()
 {
@@ -209,7 +241,7 @@ DCPPanel::encrypted_toggled ()
 
        _film->set_encrypted (_encrypted->GetValue ());
 }
-                              
+
 /** Called when the frame rate choice widget has been changed */
 void
 DCPPanel::frame_rate_choice_changed ()
@@ -243,7 +275,7 @@ DCPPanel::audio_channels_changed ()
                return;
        }
 
-       _film->set_audio_channels ((_audio_channels->GetSelection () + 1) * 2);
+       _film->set_audio_channels (dcp::raw_convert<int> (string_client_data (_audio_channels->GetClientObject (_audio_channels->GetSelection ()))));
 }
 
 void
@@ -266,6 +298,16 @@ DCPPanel::standard_changed ()
        _film->set_interop (_standard->GetSelection() == 1);
 }
 
+void
+DCPPanel::upload_after_make_dcp_changed ()
+{
+       if (!_film) {
+               return;
+       }
+
+       _film->set_upload_after_make_dcp (_upload_after_make_dcp->GetValue ());
+}
+
 void
 DCPPanel::film_changed (int p)
 {
@@ -283,9 +325,6 @@ DCPPanel::film_changed (int p)
                checked_set (_dcp_content_type, DCPContentType::as_index (_film->dcp_content_type ()));
                setup_dcp_name ();
                break;
-       case Film::BURN_SUBTITLES:
-               checked_set (_burn_subtitles, _film->burn_subtitles ());
-               break;
        case Film::SIGNED:
                checked_set (_signed, _film->is_signed ());
                break;
@@ -338,14 +377,18 @@ DCPPanel::film_changed (int p)
                        checked_set (_frame_rate_choice, -1);
                }
 
-               _frame_rate_spin->SetValue (_film->video_frame_rate ());
+               checked_set (_frame_rate_spin, _film->video_frame_rate ());
 
                _best_frame_rate->Enable (_film->best_video_frame_rate () != _film->video_frame_rate ());
                break;
        }
        case Film::AUDIO_CHANNELS:
-               checked_set (_audio_channels, (_film->audio_channels () / 2) - 1);
-               setup_dcp_name ();
+               if (_film->audio_channels () < minimum_allowed_audio_channels ()) {
+                       _film->set_audio_channels (minimum_allowed_audio_channels ());
+               } else {
+                       checked_set (_audio_channels, dcp::raw_convert<string> (max (minimum_allowed_audio_channels(), _film->audio_channels ())));
+                       setup_dcp_name ();
+               }
                break;
        case Film::THREE_D:
                checked_set (_three_d, _film->three_d ());
@@ -355,6 +398,28 @@ DCPPanel::film_changed (int p)
                checked_set (_standard, _film->interop() ? 1 : 0);
                setup_dcp_name ();
                break;
+       case Film::AUDIO_PROCESSOR:
+               if (_film->audio_processor ()) {
+                       checked_set (_audio_processor, _film->audio_processor()->id());
+               } else {
+                       checked_set (_audio_processor, 0);
+               }
+               setup_audio_channels_choice ();
+               film_changed (Film::AUDIO_CHANNELS);
+               break;
+       case Film::REEL_TYPE:
+               checked_set (_reel_type, _film->reel_type ());
+               _reel_length->Enable (_film->reel_type() == REELTYPE_BY_LENGTH);
+               break;
+       case Film::REEL_LENGTH:
+               checked_set (_reel_length, _film->reel_length() / 1000000000LL);
+               break;
+       case Film::UPLOAD_AFTER_MAKE_DCP:
+               checked_set (_upload_after_make_dcp, _film->upload_after_make_dcp ());
+               break;
+       case Film::CONTENT:
+               setup_dcp_name ();
+               break;
        default:
                break;
        }
@@ -363,9 +428,13 @@ DCPPanel::film_changed (int p)
 void
 DCPPanel::film_content_changed (int property)
 {
-       if (property == FFmpegContentProperty::AUDIO_STREAMS ||
-           property == SubtitleContentProperty::USE_SUBTITLES ||
-           property == VideoContentProperty::VIDEO_SCALE) {
+       if (property == AudioContentProperty::STREAMS ||
+           property == SubtitleContentProperty::USE ||
+           property == SubtitleContentProperty::BURN ||
+           property == VideoContentProperty::SCALE ||
+           property == DCPContentProperty::REFERENCE_VIDEO ||
+           property == DCPContentProperty::REFERENCE_AUDIO ||
+           property == DCPContentProperty::REFERENCE_SUBTITLE) {
                setup_dcp_name ();
        }
 }
@@ -381,7 +450,7 @@ DCPPanel::setup_container ()
                ++i;
                ++n;
        }
-       
+
        if (i == ratios.end()) {
                checked_set (_container, -1);
                checked_set (_container_size, wxT (""));
@@ -390,9 +459,9 @@ DCPPanel::setup_container ()
                dcp::Size const size = fit_ratio_within (_film->container()->ratio(), _film->full_frame ());
                checked_set (_container_size, wxString::Format ("%dx%d", size.width, size.height));
        }
-       
+
        setup_dcp_name ();
-}      
+}
 
 /** Called when the container widget has been changed */
 void
@@ -428,7 +497,7 @@ void
 DCPPanel::set_film (shared_ptr<Film> film)
 {
        _film = film;
-       
+
        film_changed (Film::NAME);
        film_changed (Film::USE_ISDCF_NAME);
        film_changed (Film::CONTENT);
@@ -436,16 +505,19 @@ DCPPanel::set_film (shared_ptr<Film> film)
        film_changed (Film::CONTAINER);
        film_changed (Film::RESOLUTION);
        film_changed (Film::SIGNED);
-       film_changed (Film::BURN_SUBTITLES);
        film_changed (Film::ENCRYPTED);
        film_changed (Film::KEY);
        film_changed (Film::J2K_BANDWIDTH);
        film_changed (Film::ISDCF_METADATA);
        film_changed (Film::VIDEO_FRAME_RATE);
        film_changed (Film::AUDIO_CHANNELS);
-       film_changed (Film::SEQUENCE_VIDEO);
+       film_changed (Film::SEQUENCE);
        film_changed (Film::THREE_D);
        film_changed (Film::INTEROP);
+       film_changed (Film::AUDIO_PROCESSOR);
+       film_changed (Film::REEL_TYPE);
+       film_changed (Film::REEL_LENGTH);
+       film_changed (Film::UPLOAD_AFTER_MAKE_DCP);
 }
 
 void
@@ -461,12 +533,14 @@ DCPPanel::set_general_sensitivity (bool s)
        if (_film && _film->encrypted ()) {
                si = false;
        }
-       _burn_subtitles->Enable (s);
        _signed->Enable (si);
-       
+
        _encrypted->Enable (s);
        _key->Enable (s && _film && _film->encrypted ());
        _edit_key->Enable (s && _film && _film->encrypted ());
+       _reel_type->Enable (s);
+       _reel_length->Enable (s && _film && _film->reel_type() == REELTYPE_BY_LENGTH);
+       _upload_after_make_dcp->Enable (s);
        _frame_rate_choice->Enable (s);
        _frame_rate_spin->Enable (s);
        _audio_channels->Enable (s);
@@ -496,7 +570,7 @@ DCPPanel::edit_isdcf_button_clicked ()
                return;
        }
 
-       ISDCFMetadataDialog* d = new ISDCFMetadataDialog (_panel, _film->isdcf_metadata ());
+       ISDCFMetadataDialog* d = new ISDCFMetadataDialog (_panel, _film->isdcf_metadata (), _film->three_d ());
        d->ShowModal ();
        _film->set_isdcf_metadata (d->isdcf_metadata ());
        d->Destroy ();
@@ -514,7 +588,7 @@ DCPPanel::best_frame_rate_clicked ()
        if (!_film) {
                return;
        }
-       
+
        _film->set_video_frame_rate (_film->best_video_frame_rate ());
 }
 
@@ -559,8 +633,8 @@ DCPPanel::make_video_panel ()
        panel->SetSizer (sizer);
 
        int r = 0;
-       
-       add_label_to_grid_bag_sizer (grid, panel, _("Container"), true, wxGBPosition (r, 0));
+
+       add_label_to_sizer (grid, panel, _("Container"), true, wxGBPosition (r, 0));
        {
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _container = new wxChoice (panel, wxID_ANY);
@@ -571,8 +645,8 @@ DCPPanel::make_video_panel ()
                ++r;
        }
 
+       add_label_to_sizer (grid, panel, _("Frame Rate"), true, wxGBPosition (r, 0));
        {
-               add_label_to_grid_bag_sizer (grid, panel, _("Frame Rate"), true, wxGBPosition (r, 0));
                _frame_rate_sizer = new wxBoxSizer (wxHORIZONTAL);
                _frame_rate_choice = new wxChoice (panel, wxID_ANY);
                _frame_rate_sizer->Add (_frame_rate_choice, 1, wxALIGN_CENTER_VERTICAL);
@@ -582,24 +656,20 @@ DCPPanel::make_video_panel ()
                _best_frame_rate = new wxButton (panel, wxID_ANY, _("Use best"));
                _frame_rate_sizer->Add (_best_frame_rate, 1, wxALIGN_CENTER_VERTICAL);
                grid->Add (_frame_rate_sizer, wxGBPosition (r, 1));
+               ++r;
        }
-       ++r;
-
-       _burn_subtitles = new wxCheckBox (panel, wxID_ANY, _("Burn subtitles into image"));
-       grid->Add (_burn_subtitles, wxGBPosition (r, 0), wxGBSpan (1, 2));
-       ++r;
 
        _three_d = new wxCheckBox (panel, wxID_ANY, _("3D"));
        grid->Add (_three_d, wxGBPosition (r, 0), wxGBSpan (1, 2));
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, panel, _("Resolution"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (grid, panel, _("Resolution"), true, wxGBPosition (r, 0));
        _resolution = new wxChoice (panel, wxID_ANY);
        grid->Add (_resolution, wxGBPosition (r, 1));
        ++r;
 
        {
-               add_label_to_grid_bag_sizer (grid, panel, _("JPEG2000 bandwidth"), true, wxGBPosition (r, 0));
+               add_label_to_sizer (grid, panel, _("JPEG2000 bandwidth"), true, wxGBPosition (r, 0));
                wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _j2k_bandwidth = new wxSpinCtrl (panel, wxID_ANY);
                s->Add (_j2k_bandwidth, 1);
@@ -612,7 +682,6 @@ DCPPanel::make_video_panel ()
        _frame_rate_choice->Bind(wxEVT_COMMAND_CHOICE_SELECTED,       boost::bind (&DCPPanel::frame_rate_choice_changed, this));
        _frame_rate_spin->Bind  (wxEVT_COMMAND_SPINCTRL_UPDATED,      boost::bind (&DCPPanel::frame_rate_spin_changed, this));
        _best_frame_rate->Bind  (wxEVT_COMMAND_BUTTON_CLICKED,        boost::bind (&DCPPanel::best_frame_rate_clicked, this));
-       _burn_subtitles->Bind   (wxEVT_COMMAND_CHECKBOX_CLICKED,      boost::bind (&DCPPanel::burn_subtitles_toggled, this));
        _j2k_bandwidth->Bind    (wxEVT_COMMAND_SPINCTRL_UPDATED,      boost::bind (&DCPPanel::j2k_bandwidth_changed, this));
        /* Also listen to wxEVT_COMMAND_TEXT_UPDATED so that typing numbers directly in is always noticed */
        _j2k_bandwidth->Bind    (wxEVT_COMMAND_TEXT_UPDATED,          boost::bind (&DCPPanel::j2k_bandwidth_changed, this));
@@ -638,6 +707,32 @@ DCPPanel::make_video_panel ()
        return panel;
 }
 
+int
+DCPPanel::minimum_allowed_audio_channels () const
+{
+       int min = 2;
+       if (_film && _film->audio_processor ()) {
+               min = _film->audio_processor()->out_channels ();
+       }
+
+       if (min % 2 == 1) {
+               ++min;
+       }
+
+       return min;
+}
+
+void
+DCPPanel::setup_audio_channels_choice ()
+{
+       vector<pair<string, string> > items;
+       for (int i = minimum_allowed_audio_channels(); i <= 16; i += 2) {
+               items.push_back (make_pair (dcp::raw_convert<string> (i), dcp::raw_convert<string> (i)));
+       }
+
+       checked_set (_audio_channels, items);
+}
+
 wxPanel *
 DCPPanel::make_audio_panel ()
 {
@@ -648,26 +743,29 @@ DCPPanel::make_audio_panel ()
        panel->SetSizer (sizer);
 
        int r = 0;
-       
-       add_label_to_grid_bag_sizer (grid, panel, _("Channels"), true, wxGBPosition (r, 0));
+
+       add_label_to_sizer (grid, panel, _("Channels"), true, wxGBPosition (r, 0));
        _audio_channels = new wxChoice (panel, wxID_ANY);
-       for (int i = 2; i <= 12; i += 2) {
-               _audio_channels->Append (wxString::Format ("%d", i));
-       }
+       setup_audio_channels_choice ();
        grid->Add (_audio_channels, wxGBPosition (r, 1));
+       ++r;
 
-       add_label_to_grid_bag_sizer (grid, panel, _("Processor"), true, wxGBPosition (r, 0));
+       add_label_to_sizer (grid, panel, _("Processor"), true, wxGBPosition (r, 0));
        _audio_processor = new wxChoice (panel, wxID_ANY);
        _audio_processor->Append (_("None"), new wxStringClientData (N_("none")));
        BOOST_FOREACH (AudioProcessor const * ap, AudioProcessor::all ()) {
                _audio_processor->Append (std_to_wx (ap->name ()), new wxStringClientData (std_to_wx (ap->id ())));
        }
        grid->Add (_audio_processor, wxGBPosition (r, 1));
-       
+       ++r;
+
+       _show_audio = new wxButton (panel, wxID_ANY, _("Show audio..."));
+       grid->Add (_show_audio, wxGBPosition (r, 0), wxGBSpan (1, 2));
        ++r;
 
        _audio_channels->Bind (wxEVT_COMMAND_CHOICE_SELECTED, boost::bind (&DCPPanel::audio_channels_changed, this));
        _audio_processor->Bind (wxEVT_COMMAND_CHOICE_SELECTED, boost::bind (&DCPPanel::audio_processor_changed, this));
+       _show_audio->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&DCPPanel::show_audio_clicked, this));
 
        return panel;
 }
@@ -675,7 +773,7 @@ DCPPanel::make_audio_panel ()
 void
 DCPPanel::copy_isdcf_name_button_clicked ()
 {
-       _film->set_name (_film->isdcf_name (false));
+       _film->set_name (_film->isdcf_name (true));
        _film->set_use_isdcf_name (false);
 }
 
@@ -687,7 +785,41 @@ DCPPanel::audio_processor_changed ()
        }
 
        string const s = string_client_data (_audio_processor->GetClientObject (_audio_processor->GetSelection ()));
-       if (s != "none") {
-               _film->set_audio_processor (AudioProcessor::from_id (s));
+       _film->set_audio_processor (AudioProcessor::from_id (s));
+}
+
+void
+DCPPanel::show_audio_clicked ()
+{
+       if (!_film) {
+               return;
        }
+
+       if (_audio_dialog) {
+               _audio_dialog->Destroy ();
+               _audio_dialog = 0;
+       }
+
+       AudioDialog* d = new AudioDialog (_panel, _film);
+       d->Show ();
+}
+
+void
+DCPPanel::reel_type_changed ()
+{
+       if (!_film) {
+               return;
+       }
+
+       _film->set_reel_type (static_cast<ReelType> (_reel_type->GetSelection ()));
+}
+
+void
+DCPPanel::reel_length_changed ()
+{
+       if (!_film) {
+               return;
+       }
+
+       _film->set_reel_length (_reel_length->GetValue() * 1000000000LL);
 }