Layout the audio mapping's sizer after it changes (#597).
[dcpomatic.git] / src / wx / audio_panel.cc
index b7d6979d35f20e0988f0b3135c5f0e0acc6fac5f..006b12398a6148f3e7738042dcd824b2429bdbba 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2014 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 
 */
 
-#include <boost/lexical_cast.hpp>
-#include <wx/spinctrl.h>
 #include "lib/config.h"
 #include "lib/ffmpeg_content.h"
-#include "lib/ffmpeg_audio_stream.h"
-#include "lib/audio_processor.h"
 #include "lib/cinema_sound_processor.h"
-#include "audio_dialog.h"
 #include "audio_panel.h"
 #include "audio_mapping_view.h"
 #include "wx_util.h"
 #include "gain_calculator_dialog.h"
 #include "content_panel.h"
+#include <wx/spinctrl.h>
+#include <boost/lexical_cast.hpp>
+#include <boost/foreach.hpp>
 
 using std::vector;
 using std::cout;
 using std::string;
 using std::list;
+using std::pair;
 using boost::dynamic_pointer_cast;
 using boost::lexical_cast;
 using boost::shared_ptr;
 
 AudioPanel::AudioPanel (ContentPanel* p)
        : ContentSubPanel (p, _("Audio"))
-       , _audio_dialog (0)
 {
        wxGridBagSizer* grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        _sizer->Add (grid, 0, wxALL, 8);
 
        int r = 0;
 
-       _show = new wxButton (this, wxID_ANY, _("Show Audio..."));
-       grid->Add (_show, wxGBPosition (r, 0));
-       ++r;
-
        add_label_to_grid_bag_sizer (grid, this, _("Gain"), true, wxGBPosition (r, 0));
        _gain = new ContentSpinCtrlDouble<AudioContent> (
                this,
@@ -81,17 +75,6 @@ AudioPanel::AudioPanel (ContentPanel* p)
        add_label_to_grid_bag_sizer (grid, this, _("ms"), false, wxGBPosition (r, 2));
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, this, _("Stream"), true, wxGBPosition (r, 0));
-       _stream = new wxChoice (this, wxID_ANY);
-       grid->Add (_stream, wxGBPosition (r, 1), wxGBSpan (1, 3), wxEXPAND);
-       ++r;
-
-       add_label_to_grid_bag_sizer (grid, this, _("Process with"), true, wxGBPosition (r, 0));
-       _processor = new wxChoice (this, wxID_ANY);
-       setup_processors ();
-       grid->Add (_processor, wxGBPosition (r, 1), wxGBSpan (1, 3), wxEXPAND);
-       ++r;
-       
        _mapping = new AudioMappingView (this);
        _sizer->Add (_mapping, 1, wxEXPAND | wxALL, 6);
        ++r;
@@ -109,12 +92,9 @@ AudioPanel::AudioPanel (ContentPanel* p)
        _gain->wrapped()->SetIncrement (0.5);
        _delay->wrapped()->SetRange (-1000, 1000);
 
-       _stream->Bind                (wxEVT_COMMAND_CHOICE_SELECTED, boost::bind (&AudioPanel::stream_changed, this));
-       _show->Bind                  (wxEVT_COMMAND_BUTTON_CLICKED,  boost::bind (&AudioPanel::show_clicked, this));
        _gain_calculate_button->Bind (wxEVT_COMMAND_BUTTON_CLICKED,  boost::bind (&AudioPanel::gain_calculate_button_clicked, this));
-       _processor->Bind             (wxEVT_COMMAND_CHOICE_SELECTED, boost::bind (&AudioPanel::processor_changed, this));
 
-       _mapping->Changed.connect (boost::bind (&AudioPanel::mapping_changed, this, _1));
+       _mapping_connection = _mapping->Changed.connect (boost::bind (&AudioPanel::mapping_changed, this, _1));
 }
 
 
@@ -123,8 +103,8 @@ AudioPanel::film_changed (Film::Property property)
 {
        switch (property) {
        case Film::AUDIO_CHANNELS:
-               _mapping->set_channels (_parent->film()->audio_channels ());
-               _sizer->Layout ();
+       case Film::AUDIO_PROCESSOR:
+               _mapping->set_output_channels (_parent->film()->audio_output_names ());
                break;
        case Film::VIDEO_FRAME_RATE:
                setup_description ();
@@ -137,40 +117,16 @@ AudioPanel::film_changed (Film::Property property)
 void
 AudioPanel::film_content_changed (int property)
 {
-       AudioContentList ac = _parent->selected_audio ();
-       shared_ptr<AudioContent> acs;
-       shared_ptr<FFmpegContent> fcs;
-       if (ac.size() == 1) {
-               acs = ac.front ();
-               fcs = dynamic_pointer_cast<FFmpegContent> (acs);
-       }
-       
-       if (property == AudioContentProperty::AUDIO_MAPPING) {
-               _mapping->set (acs ? acs->audio_mapping () : AudioMapping ());
-               _sizer->Layout ();
-       } else if (property == AudioContentProperty::AUDIO_FRAME_RATE) {
-               setup_description ();
-       } else if (property == FFmpegContentProperty::AUDIO_STREAM) {
-               _mapping->set (acs ? acs->audio_mapping () : AudioMapping ());
-               _sizer->Layout ();
-       } else if (property == FFmpegContentProperty::AUDIO_STREAMS) {
-               _stream->Clear ();
-               if (fcs) {
-                       vector<shared_ptr<FFmpegAudioStream> > a = fcs->audio_streams ();
-                       for (vector<shared_ptr<FFmpegAudioStream> >::iterator i = a.begin(); i != a.end(); ++i) {
-                               _stream->Append (std_to_wx ((*i)->name), new wxStringClientData (std_to_wx ((*i)->identifier ())));
-                       }
-                       
-                       if (fcs->audio_stream()) {
-                               checked_set (_stream, fcs->audio_stream()->identifier ());
-                       }
-               }
-       } else if (property == AudioContentProperty::AUDIO_PROCESSOR) {
-               if (acs) {
-                       checked_set (_processor, acs->audio_processor() ? acs->audio_processor()->id() : N_("none"));
+       if (property == AudioContentProperty::AUDIO_STREAMS) {
+               AudioContentList ac = _parent->selected_audio ();
+               if (ac.size() == 1) {
+                       _mapping->set (ac.front()->audio_mapping());
+                       _mapping->set_input_channels (ac.front()->audio_channel_names ());
                } else {
-                       checked_set (_processor, N_("none"));
+                       _mapping->set (AudioMapping ());
                }
+               setup_description ();
+               _sizer->Layout ();
        }
 }
 
@@ -178,9 +134,9 @@ void
 AudioPanel::gain_calculate_button_clicked ()
 {
        GainCalculatorDialog* d = new GainCalculatorDialog (this);
-       d->ShowModal ();
+       int const r = d->ShowModal ();
 
-       if (d->wanted_fader() == 0 || d->actual_fader() == 0) {
+       if (r == wxID_CANCEL || d->wanted_fader() == 0 || d->actual_fader() == 0) {
                d->Destroy ();
                return;
        }
@@ -200,84 +156,16 @@ AudioPanel::gain_calculate_button_clicked ()
        d->Destroy ();
 }
 
-void
-AudioPanel::show_clicked ()
-{
-       if (_audio_dialog) {
-               _audio_dialog->Destroy ();
-               _audio_dialog = 0;
-       }
-
-       AudioContentList ac = _parent->selected_audio ();
-       if (ac.size() != 1) {
-               return;
-       }
-       
-       _audio_dialog = new AudioDialog (this);
-       _audio_dialog->Show ();
-       _audio_dialog->set_content (ac.front ());
-}
-
-void
-AudioPanel::stream_changed ()
-{
-       FFmpegContentList fc = _parent->selected_ffmpeg ();
-       if (fc.size() != 1) {
-               return;
-       }
-
-       shared_ptr<FFmpegContent> fcs = fc.front ();
-       
-       if (_stream->GetSelection() == -1) {
-               return;
-       }
-       
-       vector<shared_ptr<FFmpegAudioStream> > a = fcs->audio_streams ();
-       vector<shared_ptr<FFmpegAudioStream> >::iterator i = a.begin ();
-       string const s = string_client_data (_stream->GetClientObject (_stream->GetSelection ()));
-       while (i != a.end() && (*i)->identifier () != s) {
-               ++i;
-       }
-
-       if (i != a.end ()) {
-               fcs->set_audio_stream (*i);
-       }
-}
-
-void
-AudioPanel::processor_changed ()
-{
-       string const s = string_client_data (_processor->GetClientObject (_processor->GetSelection ()));
-       AudioProcessor const * p = 0;
-       if (s != wx_to_std (N_("none"))) {
-               p = AudioProcessor::from_id (s);
-       }
-               
-       AudioContentList c = _parent->selected_audio ();
-       for (AudioContentList::const_iterator i = c.begin(); i != c.end(); ++i) {
-               (*i)->set_audio_processor (p);
-       }
-}
-
 void
 AudioPanel::setup_description ()
 {
        AudioContentList ac = _parent->selected_audio ();
        if (ac.size () != 1) {
-               _description->SetLabel ("");
+               checked_set (_description, wxT (""));
                return;
        }
 
-       shared_ptr<AudioContent> acs = ac.front ();
-       if (acs->audio_frame_rate() != acs->resampled_audio_frame_rate ()) {
-               _description->SetLabel (wxString::Format (
-                                               _("Audio will be resampled from %.3fkHz to %.3fkHz."),
-                                               acs->audio_frame_rate() / 1000.0,
-                                               acs->resampled_audio_frame_rate() / 1000.0
-                                               ));
-       } else {
-               _description->SetLabel (_("Audio will not be resampled."));
-       }
+       checked_set (_description, ac.front()->processing_description ());
 }
 
 void
@@ -294,45 +182,11 @@ AudioPanel::content_selection_changed ()
 {
        AudioContentList sel = _parent->selected_audio ();
 
-       if (_audio_dialog && sel.size() == 1) {
-               _audio_dialog->set_content (sel.front ());
-       }
-       
        _gain->set_content (sel);
        _delay->set_content (sel);
 
        _gain_calculate_button->Enable (sel.size() == 1);
-       _show->Enable (sel.size() == 1);
-       _stream->Enable (sel.size() == 1);
-       _processor->Enable (!sel.empty());
        _mapping->Enable (sel.size() == 1);
 
-       setup_processors ();
-
-       film_content_changed (AudioContentProperty::AUDIO_MAPPING);
-       film_content_changed (AudioContentProperty::AUDIO_PROCESSOR);
-       film_content_changed (AudioContentProperty::AUDIO_FRAME_RATE);
-       film_content_changed (FFmpegContentProperty::AUDIO_STREAM);
-       film_content_changed (FFmpegContentProperty::AUDIO_STREAMS);
-}
-
-void
-AudioPanel::setup_processors ()
-{
-       AudioContentList sel = _parent->selected_audio ();
-
-       _processor->Clear ();
-       list<AudioProcessor const *> ap = AudioProcessor::all ();
-       _processor->Append (_("None"), new wxStringClientData (N_("none")));
-       for (list<AudioProcessor const *>::const_iterator i = ap.begin(); i != ap.end(); ++i) {
-
-               AudioContentList::const_iterator j = sel.begin();
-               while (j != sel.end() && (*i)->in_channels().includes ((*j)->audio_channels ())) {
-                       ++j;
-               }
-
-               if (j == sel.end ()) {
-                       _processor->Append (std_to_wx ((*i)->name ()), new wxStringClientData (std_to_wx ((*i)->id ())));
-               }
-       }
+       film_content_changed (AudioContentProperty::AUDIO_STREAMS);
 }