missing drag-end (fix possible crash)
[ardour.git] / gtk2_ardour / tempo_dialog.cc
index edac3bfbefad8cbf76b429d18f24f993bf33a3a5..0b570abdaa2fcc780de2b5dc78beb8ee8411db21 100644 (file)
 
 #include "gtkmm2ext/utils.h"
 
-#include "ardour/rc_configuration.h"
-
 #include "tempo_dialog.h"
-#include "utils.h"
+#include "ui_config.h"
 
 #include "i18n.h"
 
@@ -36,13 +34,14 @@ using namespace Gtkmm2ext;
 using namespace ARDOUR;
 using namespace PBD;
 
-TempoDialog::TempoDialog (TempoMap& map, framepos_t frame, const string & action)
+TempoDialog::TempoDialog (TempoMap& map, framepos_t frame, const string&)
        : ArdourDialog (_("New Tempo"))
        , bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0)
        , bpm_spinner (bpm_adjustment)
        , when_bar_label (_("bar:"), ALIGN_LEFT, ALIGN_CENTER)
        , when_beat_label (_("beat:"), ALIGN_LEFT, ALIGN_CENTER)
        , pulse_selector_label (_("Pulse note"), ALIGN_LEFT, ALIGN_CENTER)
+       , tap_tempo_button (_("Tap tempo"))
 {
        Timecode::BBT_Time when;
        Tempo tempo (map.tempo_at (frame));
@@ -51,13 +50,14 @@ TempoDialog::TempoDialog (TempoMap& map, framepos_t frame, const string & action
        init (when, tempo.beats_per_minute(), tempo.note_type(), true);
 }
 
-TempoDialog::TempoDialog (TempoSection& section, const string & action)
-       : ArdourDialog ("Edit Tempo")
+TempoDialog::TempoDialog (TempoSection& section, const string&)
+       : ArdourDialog (_("Edit Tempo"))
        , bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0)
        , bpm_spinner (bpm_adjustment)
        , when_bar_label (_("bar:"), ALIGN_LEFT, ALIGN_CENTER)
        , when_beat_label (_("beat:"), ALIGN_LEFT, ALIGN_CENTER)
        , pulse_selector_label (_("Pulse note"), ALIGN_LEFT, ALIGN_CENTER)
+       , tap_tempo_button (_("Tap tempo"))
 {
        init (section.start(), section.beats_per_minute(), section.note_type(), section.movable());
 }
@@ -72,6 +72,7 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
        bpm_spinner.set_digits (2);
        bpm_spinner.set_wrap (true);
        bpm_spinner.set_value (bpm);
+       bpm_spinner.set_alignment (1.0);
 
        note_types.insert (make_pair (_("whole"), 1.0));
        strings.push_back (_("whole"));
@@ -83,6 +84,8 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
        strings.push_back (_("quarter"));
        note_types.insert (make_pair (_("eighth"), 8.0));
        strings.push_back (_("eighth"));
+       note_types.insert (make_pair (_("sixteenth"), 16.0));
+       strings.push_back (_("sixteenth"));
        note_types.insert (make_pair (_("thirty-second"), 32.0));
        strings.push_back (_("thirty-second"));
        note_types.insert (make_pair (_("sixty-fourth"), 64.0));
@@ -102,10 +105,10 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
        if (x == note_types.end()) {
                pulse_selector.set_active_text (strings[3]); // "quarter"
        }
-       
+
        Table* table;
 
-       if (Config->get_allow_non_quarter_pulse()) {
+       if (UIConfiguration::instance().get_allow_non_quarter_pulse()) {
                table = manage (new Table (5, 5));
        } else {
                table = manage (new Table (5, 4));
@@ -119,7 +122,7 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
        table->attach (*bpm_label, 0, 1, 0, 1);
        table->attach (bpm_spinner, 1, 5, 0, 1);
 
-       if (Config->get_allow_non_quarter_pulse()) {
+       if (UIConfiguration::instance().get_allow_non_quarter_pulse()) {
                table->attach (pulse_selector_label, 0, 1, 1, 2);
                table->attach (pulse_selector, 1, 5, 1, 2);
                row = 2;
@@ -127,19 +130,18 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
                row = 1;
        }
 
-       if (movable) {
-               char buf[64];
-
-               snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
-               when_bar_entry.set_text (buf);
-               snprintf (buf, sizeof (buf), "%" PRIu32, when.beats);
-               when_beat_entry.set_text (buf);
+       char buf[64];
 
-               when_bar_entry.set_name ("MetricEntry");
-               when_beat_entry.set_name ("MetricEntry");
+       snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
+       when_bar_entry.set_text (buf);
+       snprintf (buf, sizeof (buf), "%" PRIu32, when.beats);
+       when_beat_entry.set_text (buf);
 
+       if (movable) {
                when_bar_entry.set_width_chars(4);
                when_beat_entry.set_width_chars (4);
+               when_bar_entry.set_alignment (1.0);
+               when_beat_entry.set_alignment (1.0);
 
                when_bar_label.set_name ("MetricLabel");
                when_beat_label.set_name ("MetricLabel");
@@ -160,10 +162,13 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
 
        add_button (Stock::CANCEL, RESPONSE_CANCEL);
        add_button (Stock::APPLY, RESPONSE_ACCEPT);
-       set_response_sensitive (RESPONSE_ACCEPT, false);
+       set_response_sensitive (RESPONSE_ACCEPT, true);
        set_default_response (RESPONSE_ACCEPT);
 
        bpm_spinner.show ();
+       tap_tempo_button.show ();
+       get_vbox()->pack_end (tap_tempo_button);
+       bpm_spinner.grab_focus ();
 
        set_name ("MetricDialog");
 
@@ -176,12 +181,24 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
        when_beat_entry.signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &TempoDialog::response), RESPONSE_ACCEPT));
        when_beat_entry.signal_key_release_event().connect (sigc::mem_fun (*this, &TempoDialog::entry_key_release), false);
        pulse_selector.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::pulse_change));
+       tap_tempo_button.signal_button_press_event().connect (sigc::mem_fun (*this, &TempoDialog::tap_tempo_button_press), false);
+       tap_tempo_button.signal_focus_out_event().connect (sigc::mem_fun (*this, &TempoDialog::tap_tempo_focus_out));
+
+       tapped = false;
+}
+
+bool
+TempoDialog::is_user_input_valid() const
+{
+       return (when_beat_entry.get_text() != "")
+               && (when_bar_entry.get_text() != "")
+               && (when_bar_entry.get_text() != "0");
 }
 
 void
 TempoDialog::bpm_changed ()
 {
-       set_response_sensitive (RESPONSE_ACCEPT, true);
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
 }
 
 bool
@@ -195,18 +212,14 @@ TempoDialog::bpm_button_release (GdkEventButton*)
 {
        /* the value has been modified, accept should work now */
 
-       set_response_sensitive (RESPONSE_ACCEPT, true);
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
        return false;
 }
 
 bool
 TempoDialog::entry_key_release (GdkEventKey*)
 {
-       if (when_beat_entry.get_text() != "" && when_bar_entry.get_text() != "") {
-               set_response_sensitive (RESPONSE_ACCEPT, true);
-       } else {
-               set_response_sensitive (RESPONSE_ACCEPT, false);
-       }
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
        return false;
 }
 
@@ -236,7 +249,7 @@ double
 TempoDialog::get_note_type ()
 {
        NoteTypes::iterator x = note_types.find (pulse_selector.get_active_text());
-       
+
        if (x == note_types.end()) {
                error << string_compose(_("incomprehensible pulse note type (%1)"), pulse_selector.get_active_text()) << endmsg;
                return 0;
@@ -248,23 +261,69 @@ TempoDialog::get_note_type ()
 void
 TempoDialog::pulse_change ()
 {
-        set_response_sensitive (RESPONSE_ACCEPT, true);
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
+}
+
+bool
+TempoDialog::tap_tempo_button_press (GdkEventButton *ev)
+{
+       gint64 now;
+       now = g_get_monotonic_time (); // microseconds
+
+       if (tapped) {
+               double interval, bpm;
+               static const double decay = 0.5;
+
+               interval = (now - last_tap) * 1.0e-6;
+               if (interval <= 6.0) {
+                       // <= 6 seconds (say): >= 10 bpm
+                       if (average_interval > 0) {
+                               if (average_interval > interval / 1.2 && average_interval < interval * 1.2) {
+                               average_interval = interval * decay
+                                       + average_interval * (1.0-decay);
+                               } else {
+                                       average_interval = 0;
+                               }
+                       } else {
+                               average_interval = interval;
+                       }
+
+                       if (average_interval > 0) {
+                               bpm = 60.0 / average_interval;
+                               bpm_spinner.set_value (bpm);
+                       }
+               } else {
+                       average_interval = 0;
+               }
+       } else {
+               average_interval = 0;
+               tapped = true;
+       }
+       last_tap = now;
+
+       return true;
 }
 
+bool
+TempoDialog::tap_tempo_focus_out (GdkEventFocus* )
+{
+       tapped = false;
+       return false;
+}
 
-MeterDialog::MeterDialog (TempoMap& map, framepos_t frame, const string & action)
-       : ArdourDialog ("New Meter")
+MeterDialog::MeterDialog (TempoMap& map, framepos_t frame, const string&)
+       : ArdourDialog (_("New Meter"))
 {
        Timecode::BBT_Time when;
-       frame = map.round_to_bar(frame,0);
+       frame = map.round_to_bar(frame, RoundNearest);
        Meter meter (map.meter_at(frame));
 
        map.bbt_time (frame, when);
        init (when, meter.divisions_per_bar(), meter.note_divisor(), true);
 }
 
-MeterDialog::MeterDialog (MeterSection& section, const string & action)
-       : ArdourDialog ("Edit Meter")
+MeterDialog::MeterDialog (MeterSection& section, const string&)
+       : ArdourDialog (_("Edit Meter"))
 {
        init (section.start(), section.divisions_per_bar(), section.note_divisor(), section.movable());
 }
@@ -279,6 +338,7 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double divisor, b
        snprintf (buf, sizeof (buf), "%.2f", bpb);
        bpb_entry.set_text (buf);
        bpb_entry.select_region (0, -1);
+       bpb_entry.set_alignment (1.0);
 
        note_types.insert (make_pair (_("whole"), 1.0));
        strings.push_back (_("whole"));
@@ -290,6 +350,8 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double divisor, b
        strings.push_back (_("quarter"));
        note_types.insert (make_pair (_("eighth"), 8.0));
        strings.push_back (_("eighth"));
+       note_types.insert (make_pair (_("sixteenth"), 16.0));
+       strings.push_back (_("sixteenth"));
        note_types.insert (make_pair (_("thirty-second"), 32.0));
        strings.push_back (_("thirty-second"));
        note_types.insert (make_pair (_("sixty-fourth"), 64.0));
@@ -305,7 +367,7 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double divisor, b
                        break;
                }
        }
-       
+
        if (x == note_types.end()) {
                note_type.set_active_text (strings[3]); // "quarter"
        }
@@ -320,29 +382,23 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double divisor, b
        table->attach (*note_label, 0, 1, 1, 2, FILL|EXPAND, FILL|EXPAND);
        table->attach (note_type, 1, 2, 1, 2, FILL|EXPAND, SHRINK);
 
-       if (movable) {
-               char buf[64];
-
-               snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
-               when_bar_entry.set_text (buf);
-               when_bar_entry.set_name ("MetricEntry");
+       snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
+       when_bar_entry.set_text (buf);
+       when_bar_entry.set_alignment (1.0);
 
+       if (movable) {
                Label* when_label = manage (new Label(_("Meter begins at bar:"), ALIGN_LEFT, ALIGN_CENTER));
 
                table->attach (*when_label, 0, 1, 2, 3, FILL | EXPAND, FILL | EXPAND);
                table->attach (when_bar_entry, 1, 2, 2, 3, FILL | EXPAND, FILL | EXPAND);
-       } else {
-               when_bar_entry.set_text ("0");
        }
 
        get_vbox()->set_border_width (12);
        get_vbox()->pack_start (*table, false, false);
 
-       bpb_entry.set_name ("MetricEntry");
-
        add_button (Stock::CANCEL, RESPONSE_CANCEL);
        add_button (Stock::APPLY, RESPONSE_ACCEPT);
-       set_response_sensitive (RESPONSE_ACCEPT, false);
+       set_response_sensitive (RESPONSE_ACCEPT, true);
        set_default_response (RESPONSE_ACCEPT);
 
        get_vbox()->show_all ();
@@ -357,6 +413,14 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double divisor, b
        note_type.signal_changed().connect (sigc::mem_fun (*this, &MeterDialog::note_type_change));
 }
 
+bool
+MeterDialog::is_user_input_valid() const
+{
+       return (when_bar_entry.get_text() != "")
+               && (when_bar_entry.get_text() != "0")
+               && (bpb_entry.get_text() != "");
+}
+
 bool
 MeterDialog::entry_key_press (GdkEventKey* ev)
 {
@@ -408,18 +472,14 @@ MeterDialog::entry_key_press (GdkEventKey* ev)
 bool
 MeterDialog::entry_key_release (GdkEventKey*)
 {
-        if (when_bar_entry.get_text() != "" && bpb_entry.get_text() != "") {
-               set_response_sensitive (RESPONSE_ACCEPT, true);
-       } else {
-               set_response_sensitive (RESPONSE_ACCEPT, false);
-       }
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
        return false;
 }
 
 void
 MeterDialog::note_type_change ()
 {
-        set_response_sensitive (RESPONSE_ACCEPT, true);
+        set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
 }
 
 double
@@ -438,7 +498,7 @@ double
 MeterDialog::get_note_type ()
 {
        NoteTypes::iterator x = note_types.find (note_type.get_active_text());
-       
+
        if (x == note_types.end()) {
                error << string_compose(_("incomprehensible meter note type (%1)"), note_type.get_active_text()) << endmsg;
                return 0;