Another engine dialog edge-case fix to set the samplerate
[ardour.git] / gtk2_ardour / editor_tempodisplay.cc
index 44d0154dd48ad493bd83dbaf3c9e7a5d58b6ebd8..4630f1fe43829b48447b106499baab2b5d7d7571 100644 (file)
 #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_canvasvar_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_canvasvar_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()));
        }
 }