X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Feditor_tempodisplay.cc;h=4630f1fe43829b48447b106499baab2b5d7d7571;hb=8f216afa245da108d1617dedb2a0a5197250eb18;hp=fec380ee358c5f56e1220e2a1250bfc6ad4904ff;hpb=ba4d1cd1c1d5e4b47e288cd83c81011e7399207b;p=ardour.git

diff --git a/gtk2_ardour/editor_tempodisplay.cc b/gtk2_ardour/editor_tempodisplay.cc
index fec380ee35..4630f1fe43 100644
--- a/gtk2_ardour/editor_tempodisplay.cc
+++ b/gtk2_ardour/editor_tempodisplay.cc
@@ -48,10 +48,10 @@
 #include "rgb_macros.h"
 #include "gui_thread.h"
 #include "time_axis_view.h"
-#include "ardour_ui.h"
-#include "tempo_lines.h"
+#include "grid_lines.h"
+#include "ui_config.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace std;
 using namespace ARDOUR;
@@ -69,38 +69,108 @@ Editor::remove_metric_marks ()
 		delete_when_idle (*x);
 	}
 	metric_marks.clear ();
-}
 
+	for (Curves::iterator x = tempo_curves.begin(); x != tempo_curves.end(); ++x) {
+		delete (*x);
+	}
+	tempo_curves.clear ();
+}
+struct CurveComparator {
+	bool operator() (TempoCurve const * a, TempoCurve const * b) {
+		return a->tempo().sample() < b->tempo().sample();
+	}
+};
 void
 Editor::draw_metric_marks (const Metrics& metrics)
 {
-
-	const MeterSection *ms;
-	const TempoSection *ts;
 	char buf[64];
+	TempoSection* prev_ts = 0;
+	double max_tempo = 0.0;
+	double min_tempo = DBL_MAX;
 
-	remove_metric_marks ();
+	remove_metric_marks (); // also clears tempo curves
 
 	for (Metrics::const_iterator i = metrics.begin(); i != metrics.end(); ++i) {
+		const MeterSection *ms;
+		TempoSection *ts;
 
 		if ((ms = dynamic_cast<const MeterSection*>(*i)) != 0) {
 			snprintf (buf, sizeof(buf), "%g/%g", ms->divisions_per_bar(), ms->note_divisor ());
-			metric_marks.push_back (new MeterMarker (*this, *meter_group, ARDOUR_UI::config()->get_MeterMarker(), buf,
-								 *(const_cast<MeterSection*>(ms))));
-		} else if ((ts = dynamic_cast<const TempoSection*>(*i)) != 0) {
-			if (Config->get_allow_non_quarter_pulse()) {
-				snprintf (buf, sizeof (buf), "%.2f/%.0f", ts->beats_per_minute(), ts->note_type());
+			if (ms->position_lock_style() == MusicTime) {
+				metric_marks.push_back (new MeterMarker (*this, *meter_group, UIConfiguration::instance().color ("meter marker music"), buf,
+									 *(const_cast<MeterSection*>(ms))));
 			} else {
-				snprintf (buf, sizeof (buf), "%.2f", ts->beats_per_minute());
+				metric_marks.push_back (new MeterMarker (*this, *meter_group, UIConfiguration::instance().color ("meter marker"), buf,
+									 *(const_cast<MeterSection*>(ms))));
 			}
-			metric_marks.push_back (new TempoMarker (*this, *tempo_group, ARDOUR_UI::config()->get_TempoMarker(), buf,
+		} else if ((ts = dynamic_cast<TempoSection*>(*i)) != 0) {
+
+			max_tempo = max (max_tempo, ts->note_types_per_minute());
+			max_tempo = max (max_tempo, ts->end_note_types_per_minute());
+			min_tempo = min (min_tempo, ts->note_types_per_minute());
+			min_tempo = min (min_tempo, ts->end_note_types_per_minute());
+			uint32_t const tc_color = UIConfiguration::instance().color ("tempo curve");
+
+			tempo_curves.push_back (new TempoCurve (*this, *tempo_group, tc_color,
+								*(const_cast<TempoSection*>(ts)), ts->sample(), false));
+
+			const std::string tname (X_(""));
+			if (ts->position_lock_style() == MusicTime) {
+				metric_marks.push_back (new TempoMarker (*this, *tempo_group, UIConfiguration::instance().color ("tempo marker music"), tname,
 								 *(const_cast<TempoSection*>(ts))));
+			} else {
+				metric_marks.push_back (new TempoMarker (*this, *tempo_group, UIConfiguration::instance().color ("tempo marker"), tname,
+								 *(const_cast<TempoSection*>(ts))));
+			}
+			if (prev_ts && abs (prev_ts->end_note_types_per_minute() - ts->note_types_per_minute()) < 1.0) {
+				metric_marks.back()->set_points_color (UIConfiguration::instance().color ("tempo marker music"));
+			} else {
+				metric_marks.back()->set_points_color (UIConfiguration::instance().color ("tempo marker"));
+			}
+			prev_ts = ts;
 		}
 
 	}
+	tempo_curves.sort (CurveComparator());
 
+	const double min_tempo_range = 5.0;
+	const double tempo_delta = fabs (max_tempo - min_tempo);
+
+	if (tempo_delta < min_tempo_range) {
+		max_tempo += min_tempo_range - tempo_delta;
+		min_tempo += tempo_delta - min_tempo_range;
+	}
+
+	for (Curves::iterator x = tempo_curves.begin(); x != tempo_curves.end(); ) {
+		Curves::iterator tmp = x;
+		(*x)->set_max_tempo (max_tempo);
+		(*x)->set_min_tempo (min_tempo);
+		++tmp;
+		if (tmp != tempo_curves.end()) {
+			(*x)->set_position ((*x)->tempo().sample(), (*tmp)->tempo().sample());
+		} else {
+			(*x)->set_position ((*x)->tempo().sample(), UINT32_MAX);
+		}
+
+		if (!(*x)->tempo().active()) {
+			(*x)->hide();
+		} else {
+			(*x)->show();
+		}
+
+		++x;
+	}
+
+	for (Marks::iterator x = metric_marks.begin(); x != metric_marks.end(); ++x) {
+		TempoMarker* tempo_marker;
+
+		if ((tempo_marker = dynamic_cast<TempoMarker*> (*x)) != 0) {
+			tempo_marker->update_height_mark ((tempo_marker->tempo().note_types_per_minute() - min_tempo) / max (10.0, max_tempo - min_tempo));
+		}
+	}
 }
 
+
 void
 Editor::tempo_map_changed (const PropertyChange& /*ignored*/)
 {
@@ -110,44 +180,142 @@ Editor::tempo_map_changed (const PropertyChange& /*ignored*/)
 
 	ENSURE_GUI_THREAD (*this, &Editor::tempo_map_changed, ignored);
 
-	if (tempo_lines) {
-		tempo_lines->tempo_map_changed();
+	compute_bbt_ruler_scale (_leftmost_sample, _leftmost_sample + current_page_samples());
+
+	_session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks); // redraw metric markers
+	update_tempo_based_rulers ();
+
+	maybe_draw_grid_lines ();
+}
+
+void
+Editor::tempometric_position_changed (const PropertyChange& /*ignored*/)
+{
+	if (!_session) {
+		return;
 	}
 
-	ARDOUR::TempoMap::BBTPointList::const_iterator begin;
-	ARDOUR::TempoMap::BBTPointList::const_iterator end;
+	ENSURE_GUI_THREAD (*this, &Editor::tempo_map_changed);
 
-	compute_current_bbt_points (leftmost_frame, leftmost_frame + current_page_samples(), begin, end);
-	_session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks); // redraw metric markers
-	draw_measures (begin, end);
-	update_tempo_based_rulers (begin, end);
+	TempoSection* prev_ts = 0;
+	double max_tempo = 0.0;
+	double min_tempo = DBL_MAX;
+
+	for (Marks::iterator x = metric_marks.begin(); x != metric_marks.end(); ++x) {
+		TempoMarker* tempo_marker;
+		MeterMarker* meter_marker;
+		TempoSection *ts;
+		const MeterSection *ms;
+
+		if ((tempo_marker = dynamic_cast<TempoMarker*> (*x)) != 0) {
+			if ((ts = &tempo_marker->tempo()) != 0) {
+
+				tempo_marker->set_position (ts->sample ());
+
+				if (prev_ts && abs (prev_ts->end_note_types_per_minute() - ts->note_types_per_minute()) < 1.0) {
+					tempo_marker->set_points_color (UIConfiguration::instance().color ("tempo marker music"));
+				} else {
+					tempo_marker->set_points_color (UIConfiguration::instance().color ("tempo marker"));
+				}
+
+				max_tempo = max (max_tempo, ts->note_types_per_minute());
+				max_tempo = max (max_tempo, ts->end_note_types_per_minute());
+				min_tempo = min (min_tempo, ts->note_types_per_minute());
+				min_tempo = min (min_tempo, ts->end_note_types_per_minute());
+
+				prev_ts = ts;
+			}
+		}
+		if ((meter_marker = dynamic_cast<MeterMarker*> (*x)) != 0) {
+			if ((ms = &meter_marker->meter()) != 0) {
+				meter_marker->set_position (ms->sample ());
+			}
+		}
+	}
+
+	tempo_curves.sort (CurveComparator());
+
+	const double min_tempo_range = 5.0;
+	const double tempo_delta = fabs (max_tempo - min_tempo);
+
+	if (tempo_delta < min_tempo_range) {
+		max_tempo += min_tempo_range - tempo_delta;
+		min_tempo += tempo_delta - min_tempo_range;
+	}
+
+	for (Curves::iterator x = tempo_curves.begin(); x != tempo_curves.end(); ) {
+		Curves::iterator tmp = x;
+		(*x)->set_max_tempo (max_tempo);
+		(*x)->set_min_tempo (min_tempo);
+		++tmp;
+		if (tmp != tempo_curves.end()) {
+			(*x)->set_position ((*x)->tempo().sample(), (*tmp)->tempo().sample());
+		} else {
+			(*x)->set_position ((*x)->tempo().sample(), UINT32_MAX);
+		}
+
+		if (!(*x)->tempo().active()) {
+			(*x)->hide();
+		} else {
+			(*x)->show();
+		}
+
+		++x;
+	}
+
+	for (Marks::iterator x = metric_marks.begin(); x != metric_marks.end(); ++x) {
+		TempoMarker* tempo_marker;
+		if ((tempo_marker = dynamic_cast<TempoMarker*> (*x)) != 0) {
+			tempo_marker->update_height_mark ((tempo_marker->tempo().note_types_per_minute() - min_tempo) / max (max_tempo - min_tempo, 10.0));
+		}
+	}
+
+	compute_bbt_ruler_scale (_leftmost_sample, _leftmost_sample + current_page_samples());
+
+	update_tempo_based_rulers ();
+
+	maybe_draw_grid_lines ();
 }
 
 void
-Editor::redisplay_tempo (bool immediate_redraw)
+Editor::redisplay_grid (bool immediate_redraw)
 {
 	if (!_session) {
 		return;
 	}
 
 	if (immediate_redraw) {
-		ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_begin;
-		ARDOUR::TempoMap::BBTPointList::const_iterator current_bbt_points_end;
+
+		update_tempo_based_rulers ();
+
+		update_grid();
 		
-		compute_current_bbt_points (leftmost_frame, leftmost_frame + current_page_samples(),
-					    current_bbt_points_begin, current_bbt_points_end);
-		draw_measures (current_bbt_points_begin, current_bbt_points_end);
-		update_tempo_based_rulers (current_bbt_points_begin, current_bbt_points_end); // redraw rulers and measures
-	
 	} else {
-		Glib::signal_idle().connect (sigc::bind_return (sigc::bind (sigc::mem_fun (*this, &Editor::redisplay_tempo), true), false));
+		Glib::signal_idle().connect (sigc::bind_return (sigc::bind (sigc::mem_fun (*this, &Editor::redisplay_grid), true), false));
+	}
+}
+void
+Editor::tempo_curve_selected (TempoSection* ts, bool yn)
+{
+	if (ts == 0) {
+		return;
+	}
+
+	for (Curves::iterator x = tempo_curves.begin(); x != tempo_curves.end(); ++x) {
+		if (&(*x)->tempo() == ts) {
+			if (yn) {
+				(*x)->set_color_rgba (UIConfiguration::instance().color ("location marker"));
+			} else {
+				(*x)->set_color_rgba (UIConfiguration::instance().color ("tempo curve"));
+			}
+			break;
+		}
 	}
 }
 
+/* computes a grid starting a beat before and ending a beat after leftmost and rightmost respectively */
 void
-Editor::compute_current_bbt_points (framepos_t leftmost, framepos_t rightmost,
-				    ARDOUR::TempoMap::BBTPointList::const_iterator& begin,
-				    ARDOUR::TempoMap::BBTPointList::const_iterator& end)
+Editor::compute_current_bbt_points (std::vector<TempoMap::BBTPoint>& grid, samplepos_t leftmost, samplepos_t rightmost)
 {
 	if (!_session) {
 		return;
@@ -155,76 +323,102 @@ Editor::compute_current_bbt_points (framepos_t leftmost, framepos_t rightmost,
 
 	/* prevent negative values of leftmost from creeping into tempomap
 	 */
+	const double lower_beat = floor (max (0.0, _session->tempo_map().beat_at_sample (leftmost))) - 1.0;
+	switch (bbt_ruler_scale) {
+
+	case bbt_show_quarters:
+	case bbt_show_eighths:
+	case bbt_show_sixteenths:
+	case bbt_show_thirtyseconds:
+		_session->tempo_map().get_grid (grid, max (_session->tempo_map().sample_at_beat (lower_beat), (samplepos_t) 0), rightmost);
+		break;
+
+	case bbt_show_1:
+		_session->tempo_map().get_grid (grid, max (_session->tempo_map().sample_at_beat (lower_beat), (samplepos_t) 0), rightmost, 1);
+		break;
 
-	_session->tempo_map().get_grid (begin, end, max (leftmost, (framepos_t) 0), rightmost);
+	case bbt_show_4:
+		_session->tempo_map().get_grid (grid, max (_session->tempo_map().sample_at_beat (lower_beat), (samplepos_t) 0), rightmost, 4);
+		break;
+
+	case bbt_show_16:
+		_session->tempo_map().get_grid (grid, max (_session->tempo_map().sample_at_beat (lower_beat), (samplepos_t) 0), rightmost, 16);
+		break;
+
+	case bbt_show_64:
+		_session->tempo_map().get_grid (grid, max (_session->tempo_map().sample_at_beat (lower_beat), (samplepos_t) 0), rightmost, 64);
+		break;
+
+	default:
+		/* bbt_show_many */
+		_session->tempo_map().get_grid (grid, max (_session->tempo_map().sample_at_beat (lower_beat), (samplepos_t) 0), rightmost, 128);
+		break;
+	}
 }
 
 void
-Editor::hide_measures ()
+Editor::hide_grid_lines ()
 {
-	if (tempo_lines) {
-		tempo_lines->hide();
+	if (grid_lines) {
+		grid_lines->hide();
 	}
 }
 
 void
-Editor::draw_measures (ARDOUR::TempoMap::BBTPointList::const_iterator& begin,
-		       ARDOUR::TempoMap::BBTPointList::const_iterator& end)
+Editor::maybe_draw_grid_lines ()
 {
-	if (_session == 0 || _show_measures == false || distance (begin, end) == 0) {
+	if ( _session == 0 ) {
 		return;
 	}
 
-	if (tempo_lines == 0) {
-		tempo_lines = new TempoLines (time_line_group, ArdourCanvas::LineSet::Vertical);
+	if (grid_lines == 0) {
+		grid_lines = new GridLines (time_line_group, ArdourCanvas::LineSet::Vertical);
+	}
+
+	grid_marks.clear();
+	samplepos_t rightmost_sample = _leftmost_sample + current_page_samples();
+
+	if ( grid_musical() ) {
+		 metric_get_bbt (grid_marks, _leftmost_sample, rightmost_sample, 12);
+	} else if (_grid_type== GridTypeTimecode) {
+		 metric_get_timecode (grid_marks, _leftmost_sample, rightmost_sample, 12);
+	} else if (_grid_type == GridTypeCDFrame) {
+		metric_get_minsec (grid_marks, _leftmost_sample, rightmost_sample, 12);
+	} else if (_grid_type == GridTypeMinSec) {
+		metric_get_minsec (grid_marks, _leftmost_sample, rightmost_sample, 12);
 	}
-	
-	tempo_lines->draw (begin, end);
+
+	grid_lines->draw ( grid_marks );
+	grid_lines->show();
 }
 
 void
-Editor::mouse_add_new_tempo_event (framepos_t frame)
+Editor::mouse_add_new_tempo_event (samplepos_t sample)
 {
 	if (_session == 0) {
 		return;
 	}
 
 	TempoMap& map(_session->tempo_map());
-	TempoDialog tempo_dialog (map, frame, _("add"));
 
-	//this causes compiz to display no border.
-	//tempo_dialog.signal_realize().connect (sigc::bind (sigc::ptr_fun (set_decoration), &tempo_dialog, Gdk::WMDecoration (Gdk::DECOR_BORDER|Gdk::DECOR_RESIZEH)));
+	begin_reversible_command (_("add tempo mark"));
+	const double pulse = map.exact_qn_at_sample (sample, get_grid_music_divisions (0)) / 4.0;
 
-	ensure_float (tempo_dialog);
+	if (pulse > 0.0) {
+		XMLNode &before = map.get_state();
+		/* add music-locked ramped (?) tempo using the bpm/note type at sample*/
+		map.add_tempo (map.tempo_at_sample (sample), pulse, 0, MusicTime);
 
-	switch (tempo_dialog.run()) {
-	case RESPONSE_ACCEPT:
-		break;
-	default:
-		return;
+		XMLNode &after = map.get_state();
+		_session->add_command(new MementoCommand<TempoMap>(map, &before, &after));
+		commit_reversible_command ();
 	}
 
-	double bpm = 0;
-	Timecode::BBT_Time requested;
-
-	bpm = tempo_dialog.get_bpm ();
-	double nt = tempo_dialog.get_note_type();
-	bpm = max (0.01, bpm);
-
-	tempo_dialog.get_bbt_time (requested);
-
-	begin_reversible_command (_("add tempo mark"));
-        XMLNode &before = map.get_state();
-	map.add_tempo (Tempo (bpm,nt), requested);
-        XMLNode &after = map.get_state();
-	_session->add_command(new MementoCommand<TempoMap>(map, &before, &after));
-	commit_reversible_command ();
-
 	//map.dump (cerr);
 }
 
 void
-Editor::mouse_add_new_meter_event (framepos_t frame)
+Editor::mouse_add_new_meter_event (samplepos_t sample)
 {
 	if (_session == 0) {
 		return;
@@ -232,12 +426,7 @@ Editor::mouse_add_new_meter_event (framepos_t frame)
 
 
 	TempoMap& map(_session->tempo_map());
-	MeterDialog meter_dialog (map, frame, _("add"));
-
-	//this causes compiz to display no border..
-	//meter_dialog.signal_realize().connect (sigc::bind (sigc::ptr_fun (set_decoration), &meter_dialog, Gdk::WMDecoration (Gdk::DECOR_BORDER|Gdk::DECOR_RESIZEH)));
-
-	ensure_float (meter_dialog);
+	MeterDialog meter_dialog (map, sample, _("add"));
 
 	switch (meter_dialog.run ()) {
 	case RESPONSE_ACCEPT:
@@ -250,13 +439,20 @@ Editor::mouse_add_new_meter_event (framepos_t frame)
 	bpb = max (1.0, bpb); // XXX is this a reasonable limit?
 
 	double note_type = meter_dialog.get_note_type ();
-	Timecode::BBT_Time requested;
 
+	Timecode::BBT_Time requested;
 	meter_dialog.get_bbt_time (requested);
 
+	const double al_sample = map.sample_at_bbt (requested);
 	begin_reversible_command (_("add meter mark"));
-        XMLNode &before = map.get_state();
-	map.add_meter (Meter (bpb, note_type), requested);
+	XMLNode &before = map.get_state();
+
+	if (meter_dialog.get_lock_style() == MusicTime) {
+		map.add_meter (Meter (bpb, note_type), requested, 0, MusicTime);
+	} else {
+		map.add_meter (Meter (bpb, note_type), requested, al_sample, AudioTime);
+	}
+
 	_session->add_command(new MementoCommand<TempoMap>(map, &before, &map.get_state()));
 	commit_reversible_command ();
 
@@ -266,20 +462,20 @@ Editor::mouse_add_new_meter_event (framepos_t frame)
 void
 Editor::remove_tempo_marker (ArdourCanvas::Item* item)
 {
-	Marker* marker;
+	ArdourMarker* marker;
 	TempoMarker* tempo_marker;
 
-	if ((marker = reinterpret_cast<Marker *> (item->get_data ("marker"))) == 0) {
+	if ((marker = reinterpret_cast<ArdourMarker *> (item->get_data ("marker"))) == 0) {
 		fatal << _("programming error: tempo marker canvas item has no marker object pointer!") << endmsg;
-		/*NOTREACHED*/
+		abort(); /*NOTREACHED*/
 	}
 
 	if ((tempo_marker = dynamic_cast<TempoMarker*> (marker)) == 0) {
 		fatal << _("programming error: marker for tempo is not a tempo marker!") << endmsg;
-		/*NOTREACHED*/
+		abort(); /*NOTREACHED*/
 	}
 
-	if (tempo_marker->tempo().movable()) {
+	if (!tempo_marker->tempo().locked_to_meter() && tempo_marker->tempo().active()) {
 		Glib::signal_idle().connect (sigc::bind (sigc::mem_fun(*this, &Editor::real_remove_tempo_marker), &tempo_marker->tempo()));
 	}
 }
@@ -287,9 +483,7 @@ Editor::remove_tempo_marker (ArdourCanvas::Item* item)
 void
 Editor::edit_meter_section (MeterSection* section)
 {
-	MeterDialog meter_dialog (*section, _("done"));
-
-	ensure_float (meter_dialog);
+	MeterDialog meter_dialog (_session->tempo_map(), *section, _("done"));
 
 	switch (meter_dialog.run()) {
 	case RESPONSE_ACCEPT:
@@ -301,15 +495,20 @@ Editor::edit_meter_section (MeterSection* section)
 	double bpb = meter_dialog.get_bpb ();
 	bpb = max (1.0, bpb); // XXX is this a reasonable limit?
 
-	double note_type = meter_dialog.get_note_type ();
+	double const note_type = meter_dialog.get_note_type ();
+	const Meter meter (bpb, note_type);
 
 	Timecode::BBT_Time when;
-	meter_dialog.get_bbt_time(when);
+	meter_dialog.get_bbt_time (when);
+	const samplepos_t sample = _session->tempo_map().sample_at_bbt (when);
+	const PositionLockStyle pls = (meter_dialog.get_lock_style() == AudioTime) ? AudioTime : MusicTime;
 
-	begin_reversible_command (_("replace tempo mark"));
-        XMLNode &before = _session->tempo_map().get_state();
-	_session->tempo_map().replace_meter (*section, Meter (bpb, note_type), when);
-        XMLNode &after = _session->tempo_map().get_state();
+	begin_reversible_command (_("replace meter mark"));
+	XMLNode &before = _session->tempo_map().get_state();
+
+	_session->tempo_map().replace_meter (*section, meter, when, sample, pls);
+
+	XMLNode &after = _session->tempo_map().get_state();
 	_session->add_command(new MementoCommand<TempoMap>(_session->tempo_map(), &before, &after));
 	commit_reversible_command ();
 }
@@ -317,9 +516,7 @@ Editor::edit_meter_section (MeterSection* section)
 void
 Editor::edit_tempo_section (TempoSection* section)
 {
-	TempoDialog tempo_dialog (*section, _("done"));
-
-	ensure_float (tempo_dialog);
+	TempoDialog tempo_dialog (_session->tempo_map(), *section, _("done"));
 
 	switch (tempo_dialog.run ()) {
 	case RESPONSE_ACCEPT:
@@ -329,14 +526,25 @@ Editor::edit_tempo_section (TempoSection* section)
 	}
 
 	double bpm = tempo_dialog.get_bpm ();
+	double end_bpm = tempo_dialog.get_end_bpm ();
 	double nt = tempo_dialog.get_note_type ();
-	Timecode::BBT_Time when;
-	tempo_dialog.get_bbt_time(when);
 	bpm = max (0.01, bpm);
+	const Tempo tempo (bpm, nt, end_bpm);
+
+	Timecode::BBT_Time when;
+	tempo_dialog.get_bbt_time (when);
 
 	begin_reversible_command (_("replace tempo mark"));
 	XMLNode &before = _session->tempo_map().get_state();
-	_session->tempo_map().replace_tempo (*section, Tempo (bpm, nt), when);
+
+	if (tempo_dialog.get_lock_style() == AudioTime) {
+		samplepos_t const f = _session->tempo_map().predict_tempo_position (section, when).second;
+		_session->tempo_map().replace_tempo (*section, tempo, 0.0, f, AudioTime);
+	} else {
+		double const p = _session->tempo_map().predict_tempo_position (section, when).first;
+		_session->tempo_map().replace_tempo (*section, tempo, p, 0, MusicTime);
+	}
+
 	XMLNode &after = _session->tempo_map().get_state();
 	_session->add_command (new MementoCommand<TempoMap>(_session->tempo_map(), &before, &after));
 	commit_reversible_command ();
@@ -370,20 +578,20 @@ Editor::real_remove_tempo_marker (TempoSection *section)
 void
 Editor::remove_meter_marker (ArdourCanvas::Item* item)
 {
-	Marker* marker;
+	ArdourMarker* marker;
 	MeterMarker* meter_marker;
 
-	if ((marker = reinterpret_cast<Marker *> (item->get_data ("marker"))) == 0) {
+	if ((marker = reinterpret_cast<ArdourMarker *> (item->get_data ("marker"))) == 0) {
 		fatal << _("programming error: meter marker canvas item has no marker object pointer!") << endmsg;
-		/*NOTREACHED*/
+		abort(); /*NOTREACHED*/
 	}
 
 	if ((meter_marker = dynamic_cast<MeterMarker*> (marker)) == 0) {
 		fatal << _("programming error: marker for meter is not a meter marker!") << endmsg;
-		/*NOTREACHED*/
+		abort(); /*NOTREACHED*/
 	}
 
-	if (meter_marker->meter().movable()) {
+	if (!meter_marker->meter().initial()) {
 	  Glib::signal_idle().connect (sigc::bind (sigc::mem_fun(*this, &Editor::real_remove_meter_marker), &meter_marker->meter()));
 	}
 }