Move Film::make_kdms to the call sites.

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

diff --git a/src/wx/audio_panel.cc b/src/wx/audio_panel.cc
index 0cb062efd3f2c4cc127b093c29f8df3b12b233f3..37d6f8bb6f410525bb6b0fb37a8c13a043afe017 100644 (file)
--- a/src/wx/audio_panel.cc
+++ b/src/wx/audio_panel.cc
@@ -88,7 +88,7 @@ AudioPanel::AudioPanel (ContentPanel* p)
        /// TRANSLATORS: this is an abbreviation for milliseconds, the unit of time
        _delay_ms_label = create_label (this, _("ms"), false);
 
-       _mapping = new AudioMappingView (this);
+       _mapping = new AudioMappingView (this, _("Content"), _("content"), _("DCP"), _("DCP"));
        _sizer->Add (_mapping, 1, wxEXPAND | wxALL, 6);
 
        _description = new StaticText (this, wxT(" \n"), wxDefaultPosition, wxDefaultSize);
@@ -258,13 +258,22 @@ AudioPanel::gain_calculate_button_clicked ()
                return;
        }
 
-       _gain->wrapped()->SetValue (*c);
+       optional<float> old_peak_dB = peak ();
+       double old_value = _gain->wrapped()->GetValue();
+       _gain->wrapped()->SetValue(old_value + *c);
 
        /* This appears to be necessary, as the change is not signalled,
           I think.
        */
        _gain->view_changed ();
 
+       optional<float> peak_dB = peak ();
+       if (old_peak_dB && *old_peak_dB < -0.5 && peak_dB && *peak_dB > -0.5) {
+               error_dialog (this, _("It is not possible to adjust the content's gain for this fader change as it would cause the DCP's audio to clip.  The gain has not been changed."));
+               _gain->wrapped()->SetValue (old_value);
+               _gain->view_changed ();
+       }
+
        d->Destroy ();
 }
 
@@ -316,7 +325,13 @@ AudioPanel::setup_sensitivity ()
 
        string why_not;
        bool const can_reference = dcp && dcp->can_reference_audio (_parent->film(), why_not);
-       setup_refer_button (_reference, _reference_note, dcp, can_reference, why_not);
+       wxString cannot;
+       if (why_not.empty()) {
+               cannot = _("Cannot reference this DCP's audio.");
+       } else {
+               cannot = _("Cannot reference this DCP's audio: ") + std_to_wx(why_not);
+       }
+       setup_refer_button (_reference, _reference_note, dcp, can_reference, cannot);
 
        if (_reference->GetValue ()) {
                _gain->wrapped()->Enable (false);
@@ -354,22 +369,40 @@ AudioPanel::show_clicked ()
        _audio_dialog->Show ();
 }
 
-void
-AudioPanel::setup_peak ()
+/** @return If there is one selected piece of audio content, return its peak value in dB (if known) */
+optional<float>
+AudioPanel::peak () const
 {
-       ContentList sel = _parent->selected_audio ();
        optional<float> peak_dB;
 
-       if (sel.size() != 1) {
-               _peak->SetLabel (wxT (""));
-       } else {
+       ContentList sel = _parent->selected_audio ();
+       if (sel.size() == 1) {
                shared_ptr<Playlist> playlist (new Playlist);
                playlist->add (_parent->film(), sel.front());
                try {
-                       shared_ptr<AudioAnalysis> analysis (new AudioAnalysis (_parent->film()->audio_analysis_path (playlist)));
-                       peak_dB = 20 * log10 (analysis->overall_sample_peak().first.peak) + analysis->gain_correction (playlist);
-                       _peak->SetLabel (wxString::Format (_("Peak: %.2fdB"), *peak_dB));
+                       shared_ptr<AudioAnalysis> analysis (new AudioAnalysis(_parent->film()->audio_analysis_path(playlist)));
+                       peak_dB = linear_to_db(analysis->overall_sample_peak().first.peak) + analysis->gain_correction(playlist);
                } catch (...) {
+
+               }
+       }
+
+       return peak_dB;
+}
+
+void
+AudioPanel::setup_peak ()
+{
+       ContentList sel = _parent->selected_audio ();
+
+       optional<float> peak_dB = peak ();
+       if (sel.size() != 1) {
+               _peak->SetLabel (wxT(""));
+       } else {
+               peak_dB = peak ();
+               if (peak_dB) {
+                       _peak->SetLabel (wxString::Format(_("Peak: %.2fdB"), *peak_dB));
+               } else {
                        _peak->SetLabel (_("Peak: unknown"));
                }
        }