use home-grown solution for path_expand(), rather than wordexp() which is broken...

[ardour.git] / gtk2_ardour / strip_silence_dialog.cc

diff --git a/gtk2_ardour/strip_silence_dialog.cc b/gtk2_ardour/strip_silence_dialog.cc
index 6a591c08b9eb2afb382e515ed3d675d17451ae39..5f9dc6441d131b078ea1ba4030fa8f21b8228dd9 100644 (file)
--- a/gtk2_ardour/strip_silence_dialog.cc
+++ b/gtk2_ardour/strip_silence_dialog.cc
@@ -29,6 +29,7 @@
 #include "ardour_ui.h"
 #include "ardour/session.h"
 
+#include "audio_clock.h"
 #include "gui_thread.h"
 #include "strip_silence_dialog.h"
 #include "canvas_impl.h"
@@ -48,8 +49,8 @@ using namespace ArdourCanvas;
 StripSilenceDialog::StripSilenceDialog (Session* s, list<RegionView*> const & v)
        : ArdourDialog (_("Strip Silence"))
        , ProgressReporter ()
-        , _minimum_length (X_("silence duration"), true, "SilenceDurationClock", true, false, true, false)
-        , _fade_length (X_("silence duration"), true, "SilenceDurationClock", true, false, true, false)
+        , _minimum_length (new AudioClock (X_("silence duration"), true, "SilenceDurationClock", true, false, true, false))
+        , _fade_length (new AudioClock (X_("silence duration"), true, "SilenceDurationClock", true, false, true, false))
        , _peaks_ready_connection (0)
        , _destroying (false)
 {
@@ -60,7 +61,7 @@ StripSilenceDialog::StripSilenceDialog (Session* s, list<RegionView*> const & v)
         }
 
        Gtk::HBox* hbox = Gtk::manage (new Gtk::HBox);
-        
+
        Gtk::Table* table = Gtk::manage (new Gtk::Table (3, 3));
        table->set_spacings (6);
 
@@ -70,27 +71,27 @@ StripSilenceDialog::StripSilenceDialog (Session* s, list<RegionView*> const & v)
        table->attach (_threshold, 1, 2, n, n + 1, Gtk::FILL);
        table->attach (*Gtk::manage (new Gtk::Label (_("dbFS"))), 2, 3, n, n + 1, Gtk::FILL);
        ++n;
-        
+
        _threshold.set_digits (1);
        _threshold.set_increments (1, 10);
        _threshold.set_range (-120, 0);
        _threshold.set_value (-60);
 
        table->attach (*Gtk::manage (new Gtk::Label (_("Minimum length"), 1, 0.5)), 0, 1, n, n + 1, Gtk::FILL);
-       table->attach (_minimum_length, 1, 2, n, n + 1, Gtk::FILL);
+       table->attach (*_minimum_length, 1, 2, n, n + 1, Gtk::FILL);
        ++n;
-       
-        _minimum_length.set_session (s);
-        _minimum_length.set_mode (AudioClock::Frames);
-        _minimum_length.set (1000, true);
+
+        _minimum_length->set_session (s);
+        _minimum_length->set_mode (AudioClock::Frames);
+        _minimum_length->set (1000, true);
 
        table->attach (*Gtk::manage (new Gtk::Label (_("Fade length"), 1, 0.5)), 0, 1, n, n + 1, Gtk::FILL);
-        table->attach (_fade_length, 1, 2, n, n + 1, Gtk::FILL);
+        table->attach (*_fade_length, 1, 2, n, n + 1, Gtk::FILL);
        ++n;
 
-        _fade_length.set_session (s);
-        _fade_length.set_mode (AudioClock::Frames);
-        _fade_length.set (64, true);
+        _fade_length->set_session (s);
+        _fade_length->set_mode (AudioClock::Frames);
+        _fade_length->set (64, true);
 
        hbox->pack_start (*table);
 
@@ -104,7 +105,7 @@ StripSilenceDialog::StripSilenceDialog (Session* s, list<RegionView*> const & v)
        show_all ();
 
         _threshold.get_adjustment()->signal_value_changed().connect (sigc::mem_fun (*this, &StripSilenceDialog::threshold_changed));
-        _minimum_length.ValueChanged.connect (sigc::mem_fun (*this, &StripSilenceDialog::restart_thread));
+        _minimum_length->ValueChanged.connect (sigc::mem_fun (*this, &StripSilenceDialog::restart_thread));
 
        update_silence_rects ();
        update_threshold_line ();
@@ -119,9 +120,9 @@ StripSilenceDialog::StripSilenceDialog (Session* s, list<RegionView*> const & v)
 StripSilenceDialog::~StripSilenceDialog ()
 {
        _destroying = true;
-       
+
        /* Terminate our thread */
-       
+
        _lock.lock ();
        _interthread_info.cancel = true;
        _thread_should_finish = true;
@@ -129,7 +130,10 @@ StripSilenceDialog::~StripSilenceDialog ()
 
        _run_cond.signal ();
        pthread_join (_thread, 0);
-       
+
+       delete _minimum_length;
+       delete _fade_length;
+
        delete _peaks_ready_connection;
 }
 
@@ -161,7 +165,7 @@ StripSilenceDialog::update_threshold_line ()
        for (list<Wave*>::iterator i = _waves.begin(); i != _waves.end(); ++i) {
                (*i)->threshold_line->property_x1() = 0;
                (*i)->threshold_line->property_x2() = _wave_width;
-               
+
                double const y = alt_log_meter (_threshold.get_value());
 
                (*i)->threshold_line->property_y1() = (n + 1 - y) * _wave_height;
@@ -206,7 +210,7 @@ StripSilenceDialog::detection_thread_work ()
 
        /* Hold this lock when we are doing work */
        _lock.lock ();
-       
+
        while (1) {
                for (list<ViewInterval>::iterator i = views.begin(); i != views.end(); ++i) {
                         boost::shared_ptr<AudioRegion> ar = boost::dynamic_pointer_cast<AudioRegion> ((*i).view->region());
@@ -249,7 +253,7 @@ StripSilenceDialog::restart_thread ()
                */
                return;
        }
-       
+
        /* Cancel any current run */
        _interthread_info.cancel = true;
 
@@ -273,15 +277,15 @@ StripSilenceDialog::threshold_changed ()
 framecnt_t
 StripSilenceDialog::minimum_length () const
 {
-        return _minimum_length.current_duration (views.front().view->region()->position());
+        return _minimum_length->current_duration (views.front().view->region()->position());
 }
 
 framecnt_t
 StripSilenceDialog::fade_length () const
 {
-        return _fade_length.current_duration (views.front().view->region()->position());
+        return _fade_length->current_duration (views.front().view->region()->position());
 }
-               
+
 void
 StripSilenceDialog::update_progress_gui (float p)
 {