X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Ftempo_dialog.cc;h=3557727394daddc6595ed3fa5829126644d4cffa;hb=04416e2d1df3cc8d9f014765e5ca5ce818b7b4d7;hp=f014832f2ababefe314d06b2a26bd2ffc4614a7d;hpb=e0cf3b6354e2c92a6adabe85cb6d6d3c547c287c;p=ardour.git

diff --git a/gtk2_ardour/tempo_dialog.cc b/gtk2_ardour/tempo_dialog.cc
index f014832f2a..3557727394 100644
--- a/gtk2_ardour/tempo_dialog.cc
+++ b/gtk2_ardour/tempo_dialog.cc
@@ -20,7 +20,10 @@
 #include <cstdio> // for snprintf, grrr
 
 #include <gtkmm/stock.h>
-#include <gtkmm2ext/utils.h>
+
+#include "gtkmm2ext/utils.h"
+
+#include "ardour/rc_configuration.h"
 
 #include "tempo_dialog.h"
 #include "utils.h"
@@ -33,14 +36,13 @@ using namespace Gtkmm2ext;
 using namespace ARDOUR;
 using namespace PBD;
 
-TempoDialog::TempoDialog (TempoMap& map, framepos_t frame, const string & action)
-	: ArdourDialog (_("New Tempo")),
-	  bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0),
-	  bpm_spinner (bpm_adjustment),
-	  ok_button (action),
-	  cancel_button (_("Cancel")),
-	  when_bar_label (_("bar:"), ALIGN_LEFT, ALIGN_CENTER),
-	  when_beat_label (_("beat:"), ALIGN_LEFT, ALIGN_CENTER)
+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)
 {
 	Timecode::BBT_Time when;
 	Tempo tempo (map.tempo_at (frame));
@@ -49,14 +51,13 @@ 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"),
-	  bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0),
-	  bpm_spinner (bpm_adjustment),
-	  ok_button (action),
-	  cancel_button (_("Cancel")),
-	  when_bar_label (_("bar:"), ALIGN_LEFT, ALIGN_CENTER),
-	  when_beat_label (_("beat:"), ALIGN_LEFT, ALIGN_CENTER)
+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)
 {
 	init (section.start(), section.beats_per_minute(), section.note_type(), section.movable());
 }
@@ -64,66 +65,92 @@ TempoDialog::TempoDialog (TempoSection& section, const string & action)
 void
 TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type, bool movable)
 {
+	vector<string> strings;
+	NoteTypes::iterator x;
+
 	bpm_spinner.set_numeric (true);
 	bpm_spinner.set_digits (2);
 	bpm_spinner.set_wrap (true);
 	bpm_spinner.set_value (bpm);
 
-	strings.push_back (_("whole (1)"));
-	strings.push_back (_("second (2)"));
-	strings.push_back (_("third (3)"));
-	strings.push_back (_("quarter (4)"));
-	strings.push_back (_("eighth (8)"));
-	strings.push_back (_("sixteenth (16)"));
-	strings.push_back (_("thirty-second (32)"));
-
-	set_popdown_strings (note_types, strings);
-
-	if (note_type == 1.0f) {
-		note_types.set_active_text (_("whole (1)"));
-	} else if (note_type == 2.0f) {
-		note_types.set_active_text (_("second (2)"));
-	} else if (note_type == 3.0f) {
-		note_types.set_active_text (_("third (3)"));
-	} else if (note_type == 4.0f) {
-		note_types.set_active_text (_("quarter (4)"));
-	} else if (note_type == 8.0f) {
-		note_types.set_active_text (_("eighth (8)"));
-	} else if (note_type == 16.0f) {
-		note_types.set_active_text (_("sixteenth (16)"));
-	} else if (note_type == 32.0f) {
-		note_types.set_active_text (_("thirty-second (32)"));
+	note_types.insert (make_pair (_("whole"), 1.0));
+	strings.push_back (_("whole"));
+	note_types.insert (make_pair (_("second"), 2.0));
+	strings.push_back (_("second"));
+	note_types.insert (make_pair (_("third"), 3.0));
+	strings.push_back (_("third"));
+	note_types.insert (make_pair (_("quarter"), 4.0));
+	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));
+	strings.push_back (_("sixty-fourth"));
+	note_types.insert (make_pair (_("one-hundred-twenty-eighth"), 128.0));
+	strings.push_back (_("one-hundred-twenty-eighth"));
+
+	set_popdown_strings (pulse_selector, strings);
+
+	for (x = note_types.begin(); x != note_types.end(); ++x) {
+		if (x->second == note_type) {
+			pulse_selector.set_active_text (x->first);
+			break;
+		}
+	}
+
+	if (x == note_types.end()) {
+		pulse_selector.set_active_text (strings[3]); // "quarter"
+	}
+	
+	Table* table;
+
+	if (Config->get_allow_non_quarter_pulse()) {
+		table = manage (new Table (5, 5));
 	} else {
-		note_types.set_active_text (_("quarter (4)"));
+		table = manage (new Table (5, 4));
 	}
 
-	Table* table = manage (new Table (3, 3));
 	table->set_spacings (6);
+	table->set_homogeneous (false);
 
+	int row;
 	Label* bpm_label = manage (new Label(_("Beats per minute:"), ALIGN_LEFT, ALIGN_CENTER));
-	table->attach (*bpm_label, 0, 2, 0, 1);
-	table->attach (bpm_spinner, 2, 3, 0, 1);
+	table->attach (*bpm_label, 0, 1, 0, 1);
+	table->attach (bpm_spinner, 1, 5, 0, 1);
+
+	if (Config->get_allow_non_quarter_pulse()) {
+		table->attach (pulse_selector_label, 0, 1, 1, 2);
+		table->attach (pulse_selector, 1, 5, 1, 2);
+		row = 2;
+	} else {
+		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);
 
-		when_bar_entry.set_name ("MetricEntry");
-		when_beat_entry.set_name ("MetricEntry");
+		when_bar_entry.set_width_chars(4);
+		when_beat_entry.set_width_chars (4);
 
 		when_bar_label.set_name ("MetricLabel");
 		when_beat_label.set_name ("MetricLabel");
 
-		table->attach (when_bar_label, 1, 2, 2, 3);
-		table->attach (when_bar_entry, 2, 3, 2, 3);
+		table->attach (when_bar_label, 1, 2, row, row+1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
+		table->attach (when_bar_entry, 2, 3, row, row+1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
 
-		table->attach (when_beat_label, 1, 2, 1, 2);
-		table->attach (when_beat_entry, 2, 3, 1, 2);
+		table->attach (when_beat_label, 3, 4, row, row+1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
+		table->attach (when_beat_entry, 4, 5, row, row+1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
 
 		Label* when_label = manage (new Label(_("Tempo begins at"), ALIGN_LEFT, ALIGN_CENTER));
-		table->attach (*when_label, 0, 1, 1, 2);
+		table->attach (*when_label, 0, 1, row, row+1);
 	}
 
 	get_vbox()->set_border_width (12);
@@ -147,7 +174,7 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
 	when_bar_entry.signal_key_release_event().connect (sigc::mem_fun (*this, &TempoDialog::entry_key_release), false);
 	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);
-	note_types.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::note_types_change));
+	pulse_selector.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::pulse_change));
 }
 
 void
@@ -207,42 +234,25 @@ TempoDialog::get_bbt_time (Timecode::BBT_Time& requested)
 double
 TempoDialog::get_note_type ()
 {
-	double note_type = 0;
-	vector<string>::iterator i;
-	string text = note_types.get_active_text();
-
-	for (i = strings.begin(); i != strings.end(); ++i) {
-		if (text == *i) {
-			if (sscanf (text.c_str(), "%*[^0-9]%lf", &note_type) != 1) {
-				error << string_compose(_("garbaged note type entry (%1)"), text) << endmsg;
-				return 0;
-			} else {
-				break;
-			}
-		}
-	}
-
-	if (i == strings.end()) {
-		if (sscanf (text.c_str(), "%lf", &note_type) != 1) {
-			error << string_compose(_("incomprehensible note type entry (%1)"), text) << endmsg;
-			return 0;
-		}
+	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;
 	}
 
-	return note_type;
+	return x->second;
 }
 
 void
-TempoDialog::note_types_change ()
+TempoDialog::pulse_change ()
 {
         set_response_sensitive (RESPONSE_ACCEPT, true);
 }
 
 
-MeterDialog::MeterDialog (TempoMap& map, framepos_t frame, const string & action)
-	: ArdourDialog ("New Meter"),
-	  ok_button (action),
-	  cancel_button (_("Cancel"))
+MeterDialog::MeterDialog (TempoMap& map, framepos_t frame, const string&)
+	: ArdourDialog ("New Meter")
 {
 	Timecode::BBT_Time when;
 	frame = map.round_to_bar(frame,0);
@@ -252,47 +262,53 @@ MeterDialog::MeterDialog (TempoMap& map, framepos_t frame, const string & action
 	init (when, meter.divisions_per_bar(), meter.note_divisor(), true);
 }
 
-MeterDialog::MeterDialog (MeterSection& section, const string & action)
-	: ArdourDialog ("Edit Meter"),
-	  ok_button (action),
-	  cancel_button (_("Cancel"))
+MeterDialog::MeterDialog (MeterSection& section, const string&)
+	: ArdourDialog ("Edit Meter")
 {
 	init (section.start(), section.divisions_per_bar(), section.note_divisor(), section.movable());
 }
 
 void
-MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double note_type, bool movable)
+MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double divisor, bool movable)
 {
+	char buf[64];
+	vector<string> strings;
+	NoteTypes::iterator x;
+
 	snprintf (buf, sizeof (buf), "%.2f", bpb);
 	bpb_entry.set_text (buf);
 	bpb_entry.select_region (0, -1);
 
-	strings.push_back (_("whole (1)"));
-	strings.push_back (_("second (2)"));
-	strings.push_back (_("third (3)"));
-	strings.push_back (_("quarter (4)"));
-	strings.push_back (_("eighth (8)"));
-	strings.push_back (_("sixteenth (16)"));
-	strings.push_back (_("thirty-second (32)"));
-
-	set_popdown_strings (note_types, strings);
-
-	if (note_type == 1.0f) {
-		note_types.set_active_text (_("whole (1)"));
-	} else if (note_type == 2.0f) {
-		note_types.set_active_text (_("second (2)"));
-	} else if (note_type == 3.0f) {
-		note_types.set_active_text (_("third (3)"));
-	} else if (note_type == 4.0f) {
-		note_types.set_active_text (_("quarter (4)"));
-	} else if (note_type == 8.0f) {
-		note_types.set_active_text (_("eighth (8)"));
-	} else if (note_type == 16.0f) {
-		note_types.set_active_text (_("sixteenth (16)"));
-	} else if (note_type == 32.0f) {
-		note_types.set_active_text (_("thirty-second (32)"));
-	} else {
-		note_types.set_active_text (_("quarter (4)"));
+	note_types.insert (make_pair (_("whole"), 1.0));
+	strings.push_back (_("whole"));
+	note_types.insert (make_pair (_("second"), 2.0));
+	strings.push_back (_("second"));
+	note_types.insert (make_pair (_("third"), 3.0));
+	strings.push_back (_("third"));
+	note_types.insert (make_pair (_("quarter"), 4.0));
+	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));
+	strings.push_back (_("sixty-fourth"));
+	note_types.insert (make_pair (_("one-hundred-twenty-eighth"), 128.0));
+	strings.push_back (_("one-hundred-twenty-eighth"));
+
+	set_popdown_strings (note_type, strings);
+
+	for (x = note_types.begin(); x != note_types.end(); ++x) {
+		if (x->second == divisor) {
+			note_type.set_active_text (x->first);
+			break;
+		}
+	}
+	
+	if (x == note_types.end()) {
+		note_type.set_active_text (strings[3]); // "quarter"
 	}
 
 	Label* note_label = manage (new Label (_("Note value:"), ALIGN_LEFT, ALIGN_CENTER));
@@ -303,12 +319,13 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double note_type,
 	table->attach (*bpb_label, 0, 1, 0, 1, FILL|EXPAND, FILL|EXPAND);
 	table->attach (bpb_entry, 1, 2, 0, 1, FILL|EXPAND, FILL|EXPAND);
 	table->attach (*note_label, 0, 1, 1, 2, FILL|EXPAND, FILL|EXPAND);
-	table->attach (note_types, 1, 2, 1, 2, FILL|EXPAND, SHRINK);
+	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");
 
 		Label* when_label = manage (new Label(_("Meter begins at bar:"), ALIGN_LEFT, ALIGN_CENTER));
 
@@ -321,8 +338,6 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double note_type,
 	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);
@@ -337,8 +352,7 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double note_type,
 	when_bar_entry.signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &MeterDialog::response), RESPONSE_ACCEPT));
 	when_bar_entry.signal_key_press_event().connect (sigc::mem_fun (*this, &MeterDialog::entry_key_press), false);
 	when_bar_entry.signal_key_release_event().connect (sigc::mem_fun (*this, &MeterDialog::entry_key_release));
-
-	note_types.signal_changed().connect (sigc::mem_fun (*this, &MeterDialog::note_types_change));
+	note_type.signal_changed().connect (sigc::mem_fun (*this, &MeterDialog::note_type_change));
 }
 
 bool
@@ -401,7 +415,7 @@ MeterDialog::entry_key_release (GdkEventKey*)
 }
 
 void
-MeterDialog::note_types_change ()
+MeterDialog::note_type_change ()
 {
         set_response_sensitive (RESPONSE_ACCEPT, true);
 }
@@ -421,29 +435,14 @@ MeterDialog::get_bpb ()
 double
 MeterDialog::get_note_type ()
 {
-	double note_type = 0;
-	vector<string>::iterator i;
-	string text = note_types.get_active_text();
-
-	for (i = strings.begin(); i != strings.end(); ++i) {
-		if (text == *i) {
-			if (sscanf (text.c_str(), "%*[^0-9]%lf", &note_type) != 1) {
-				error << string_compose(_("garbaged note type entry (%1)"), text) << endmsg;
-				return 0;
-			} else {
-				break;
-			}
-		}
-	}
-
-	if (i == strings.end()) {
-		if (sscanf (text.c_str(), "%lf", &note_type) != 1) {
-			error << string_compose(_("incomprehensible note type entry (%1)"), text) << endmsg;
-			return 0;
-		}
+	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;
 	}
 
-	return note_type;
+	return x->second;
 }
 
 bool
@@ -454,7 +453,6 @@ MeterDialog::get_bbt_time (Timecode::BBT_Time& requested)
 	}
 
 	requested.beats = 1;
-
 	requested.ticks = 0;
 
 	return true;