Re-add show audio button and analyse just that content when it is clicked.

[dcpomatic.git] / src / wx / audio_dialog.cc

diff --git a/src/wx/audio_dialog.cc b/src/wx/audio_dialog.cc
index 0034e793d020934b7e3c9e5e31f9c88a01625fe3..cd59693eab61ae8bfbaecefbcf8af99610e39d5a 100644 (file)
--- a/src/wx/audio_dialog.cc
+++ b/src/wx/audio_dialog.cc
@@ -23,15 +23,17 @@
 #include "lib/audio_analysis.h"
 #include "lib/film.h"
 #include "lib/analyse_audio_job.h"
+#include "lib/audio_content.h"
 #include "lib/job_manager.h"
-#include "lib/playlist.h"
 #include <boost/filesystem.hpp>
 
 using boost::shared_ptr;
 using boost::bind;
 using boost::optional;
+using boost::const_pointer_cast;
 
-AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film)
+/** @param content Content to analyse, or 0 to analyse all of the film's audio */
+AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film, shared_ptr<AudioContent> content)
        : wxDialog (parent, wxID_ANY, _("Audio"), wxDefaultPosition, wxSize (640, 512), wxDEFAULT_DIALOG_STYLE | wxRESIZE_BORDER | wxFULL_REPAINT_ON_RESIZE)
        , _film (film)
        , _plot (0)
@@ -39,8 +41,9 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film)
        wxFont subheading_font (*wxNORMAL_FONT);
        subheading_font.SetWeight (wxFONTWEIGHT_BOLD);
 
-       wxBoxSizer* sizer = new wxBoxSizer (wxHORIZONTAL);
-       
+       wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
+       wxBoxSizer* lr_sizer = new wxBoxSizer (wxHORIZONTAL);
+
        wxBoxSizer* left = new wxBoxSizer (wxVERTICAL);
 
        _plot = new AudioPlot (this);
@@ -48,7 +51,7 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film)
        _peak_time = new wxStaticText (this, wxID_ANY, wxT (""));
        left->Add (_peak_time, 0, wxALL, 12);
 
-       sizer->Add (left, 1, wxALL, 12);
+       lr_sizer->Add (left, 1, wxALL, 12);
 
        wxBoxSizer* right = new wxBoxSizer (wxVERTICAL);
 
@@ -69,7 +72,7 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film)
                m->SetFont (subheading_font);
                right->Add (m, 1, wxALIGN_CENTER_VERTICAL | wxTOP, 16);
        }
-       
+
        wxString const types[] = {
                _("Peak"),
                _("RMS")
@@ -86,26 +89,35 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film)
                m->SetFont (subheading_font);
                right->Add (m, 1, wxALIGN_CENTER_VERTICAL | wxTOP, 16);
        }
-       
+
        _smoothing = new wxSlider (this, wxID_ANY, AudioPlot::max_smoothing / 2, 1, AudioPlot::max_smoothing);
        _smoothing->Bind (wxEVT_SCROLL_THUMBTRACK, boost::bind (&AudioDialog::smoothing_changed, this));
        right->Add (_smoothing, 0, wxEXPAND);
 
-       sizer->Add (right, 0, wxALL, 12);
+       lr_sizer->Add (right, 0, wxALL, 12);
 
-       SetSizer (sizer);
-       sizer->Layout ();
-       sizer->SetSizeHints (this);
-}
+       overall_sizer->Add (lr_sizer);
 
-void
-AudioDialog::set_playlist (shared_ptr<const Playlist> p)
-{
-       _playlist_connection.disconnect ();
-       _playlist = p;
-       _playlist_connection = _playlist->ContentChanged.connect (boost::bind (&AudioDialog::try_to_load_analysis, this));
-       try_to_load_analysis ();
+#ifdef DCPOMATIC_LINUX
+       wxSizer* buttons = CreateSeparatedButtonSizer (wxCLOSE);
+       if (buttons) {
+               overall_sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
+       }
+#endif
+
+       SetSizer (overall_sizer);
+       overall_sizer->Layout ();
+       overall_sizer->SetSizeHints (this);
+
+       _film_connection = film->ContentChanged.connect (boost::bind (&AudioDialog::try_to_load_analysis, this));
        SetTitle (_("DCP-o-matic audio"));
+
+       if (content) {
+               _playlist.reset (new Playlist ());
+               const_pointer_cast<Playlist> (_playlist)->add (content);
+       } else {
+               _playlist = film->playlist ();
+       }
 }
 
 void
@@ -138,12 +150,12 @@ AudioDialog::try_to_load_analysis ()
                JobManager::instance()->add (job);
                return;
         }
-       
+
        _plot->set_analysis (_analysis);
        setup_peak_time ();
 
        /* Set up some defaults if no check boxes are checked */
-       
+
        int i = 0;
        while (i < MAX_DCP_AUDIO_CHANNELS && (!_channel_checkbox[i] || !_channel_checkbox[i]->GetValue ())) {
                ++i;
@@ -174,7 +186,7 @@ AudioDialog::analysis_finished ()
 {
        shared_ptr<const Film> film = _film.lock ();
        DCPOMATIC_ASSERT (film);
-       
+
        if (!boost::filesystem::exists (film->audio_analysis_path (_playlist))) {
                /* We analysed and still nothing showed up, so maybe it was cancelled or it failed.
                   Give up.
@@ -232,14 +244,14 @@ AudioDialog::setup_peak_time ()
        if (!_analysis || !_analysis->peak ()) {
                return;
        }
-       
+
        shared_ptr<Film> film = _film.lock ();
        if (!film) {
                return;
        }
-       
+
        float peak_dB = 20 * log10 (_analysis->peak().get());
-       
+
        _peak_time->SetLabel (
                wxString::Format (
                        _("Peak is %.2fdB at %s"),
@@ -247,10 +259,18 @@ AudioDialog::setup_peak_time ()
                        time_to_timecode (_analysis->peak_time().get(), film->video_frame_rate ()).data ()
                        )
                );
-       
+
        if (peak_dB > -3) {
                _peak_time->SetForegroundColour (wxColour (255, 0, 0));
        } else {
                _peak_time->SetForegroundColour (wxColour (0, 0, 0));
        }
 }
+
+bool
+AudioDialog::Show (bool show)
+{
+       bool const r = wxDialog::Show (show);
+       try_to_load_analysis ();
+       return r;
+}