Another engine dialog edge-case fix to set the samplerate
[ardour.git] / gtk2_ardour / editor_tempodisplay.cc
index 86b1b8d65a54c5b4ed5ce12166fd9530f2c07bbf..4630f1fe43829b48447b106499baab2b5d7d7571 100644 (file)
@@ -48,7 +48,7 @@
 #include "rgb_macros.h"
 #include "gui_thread.h"
 #include "time_axis_view.h"
-#include "tempo_lines.h"
+#include "grid_lines.h"
 #include "ui_config.h"
 
 #include "pbd/i18n.h"
@@ -77,13 +77,14 @@ Editor::remove_metric_marks ()
 }
 struct CurveComparator {
        bool operator() (TempoCurve const * a, TempoCurve const * b) {
-               return a->tempo().frame() < b->tempo().frame();
+               return a->tempo().sample() < b->tempo().sample();
        }
 };
 void
 Editor::draw_metric_marks (const Metrics& metrics)
 {
        char buf[64];
+       TempoSection* prev_ts = 0;
        double max_tempo = 0.0;
        double min_tempo = DBL_MAX;
 
@@ -91,7 +92,7 @@ Editor::draw_metric_marks (const Metrics& metrics)
 
        for (Metrics::const_iterator i = metrics.begin(); i != metrics.end(); ++i) {
                const MeterSection *ms;
-               const TempoSection *ts;
+               TempoSection *ts;
 
                if ((ms = dynamic_cast<const MeterSection*>(*i)) != 0) {
                        snprintf (buf, sizeof(buf), "%g/%g", ms->divisions_per_bar(), ms->note_divisor ());
@@ -102,12 +103,7 @@ Editor::draw_metric_marks (const Metrics& metrics)
                                metric_marks.push_back (new MeterMarker (*this, *meter_group, UIConfiguration::instance().color ("meter marker"), buf,
                                                                         *(const_cast<MeterSection*>(ms))));
                        }
-               } else if ((ts = dynamic_cast<const TempoSection*>(*i)) != 0) {
-                       if (UIConfiguration::instance().get_allow_non_quarter_pulse()) {
-                               snprintf (buf, sizeof (buf), "%.3f/%.0f", ts->note_types_per_minute(), ts->note_type());
-                       } else {
-                               snprintf (buf, sizeof (buf), "%.3f", ts->note_types_per_minute());
-                       }
+               } 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());
@@ -116,15 +112,22 @@ Editor::draw_metric_marks (const Metrics& metrics)
                        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->frame(), false));
+                                                               *(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"), buf,
+                               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"), buf,
+                               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;
                }
 
        }
@@ -144,9 +147,9 @@ Editor::draw_metric_marks (const Metrics& metrics)
                (*x)->set_min_tempo (min_tempo);
                ++tmp;
                if (tmp != tempo_curves.end()) {
-                       (*x)->set_position ((*x)->tempo().frame(), (*tmp)->tempo().frame());
+                       (*x)->set_position ((*x)->tempo().sample(), (*tmp)->tempo().sample());
                } else {
-                       (*x)->set_position ((*x)->tempo().frame(), UINT32_MAX);
+                       (*x)->set_position ((*x)->tempo().sample(), UINT32_MAX);
                }
 
                if (!(*x)->tempo().active()) {
@@ -177,18 +180,12 @@ 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());
 
-       compute_bbt_ruler_scale (leftmost_frame, leftmost_frame + current_page_samples());
-       std::vector<TempoMap::BBTPoint> grid;
-       if (bbt_ruler_scale != bbt_show_many) {
-               compute_current_bbt_points (grid, leftmost_frame, leftmost_frame + current_page_samples());
-       }
        _session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks); // redraw metric markers
-       draw_measures (grid);
        update_tempo_based_rulers ();
+
+       maybe_draw_grid_lines ();
 }
 
 void
@@ -200,41 +197,38 @@ Editor::tempometric_position_changed (const PropertyChange& /*ignored*/)
 
        ENSURE_GUI_THREAD (*this, &Editor::tempo_map_changed);
 
-       if (tempo_lines) {
-               tempo_lines->tempo_map_changed();
-       }
-
+       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;
-               const TempoSection *ts;
+               TempoSection *ts;
                const MeterSection *ms;
 
                if ((tempo_marker = dynamic_cast<TempoMarker*> (*x)) != 0) {
                        if ((ts = &tempo_marker->tempo()) != 0) {
-                               tempo_marker->set_position (ts->frame ());
-                               char buf[64];
 
-                               if (UIConfiguration::instance().get_allow_non_quarter_pulse()) {
-                                       snprintf (buf, sizeof (buf), "%.3f/%.0f", ts->note_types_per_minute(), ts->note_type());
+                               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 {
-                                       snprintf (buf, sizeof (buf), "%.3f", ts->note_types_per_minute());
+                                       tempo_marker->set_points_color (UIConfiguration::instance().color ("tempo marker"));
                                }
 
-                               tempo_marker->set_name (buf);
-
                                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->frame ());
+                               meter_marker->set_position (ms->sample ());
                        }
                }
        }
@@ -255,9 +249,9 @@ Editor::tempometric_position_changed (const PropertyChange& /*ignored*/)
                (*x)->set_min_tempo (min_tempo);
                ++tmp;
                if (tmp != tempo_curves.end()) {
-                       (*x)->set_position ((*x)->tempo().frame(), (*tmp)->tempo().frame());
+                       (*x)->set_position ((*x)->tempo().sample(), (*tmp)->tempo().sample());
                } else {
-                       (*x)->set_position ((*x)->tempo().frame(), UINT32_MAX);
+                       (*x)->set_position ((*x)->tempo().sample(), UINT32_MAX);
                }
 
                if (!(*x)->tempo().active()) {
@@ -276,48 +270,44 @@ Editor::tempometric_position_changed (const PropertyChange& /*ignored*/)
                }
        }
 
-       compute_bbt_ruler_scale (leftmost_frame, leftmost_frame + current_page_samples());
-       std::vector<TempoMap::BBTPoint> grid;
-
-       if (bbt_ruler_scale != bbt_show_many) {
-               compute_current_bbt_points (grid, leftmost_frame, leftmost_frame + current_page_samples());
-       }
+       compute_bbt_ruler_scale (_leftmost_sample, _leftmost_sample + current_page_samples());
 
-       draw_measures (grid);
        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) {
-               compute_bbt_ruler_scale (leftmost_frame, leftmost_frame + current_page_samples());
-               std::vector<TempoMap::BBTPoint> grid;
-
-               if (bbt_ruler_scale != bbt_show_many) {
-                       compute_current_bbt_points (grid, leftmost_frame, leftmost_frame + current_page_samples());
-               }
 
-               draw_measures (grid);
-               update_tempo_based_rulers (); // redraw rulers and measure lines
+               update_tempo_based_rulers ();
 
+               update_grid();
+               
        } 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 && yn) {
-                       (*x)->set_color_rgba (UIConfiguration::instance().color ("location marker"));
-                       break;
-               } else if (&(*x)->tempo() == ts) {
-                       (*x)->set_color_rgba (UIConfiguration::instance().color ("tempo curve"));
+               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;
                }
        }
@@ -325,7 +315,7 @@ Editor::tempo_curve_selected (TempoSection* ts, bool yn)
 
 /* computes a grid starting a beat before and ending a beat after leftmost and rightmost respectively */
 void
-Editor::compute_current_bbt_points (std::vector<TempoMap::BBTPoint>& grid, framepos_t leftmost, framepos_t rightmost)
+Editor::compute_current_bbt_points (std::vector<TempoMap::BBTPoint>& grid, samplepos_t leftmost, samplepos_t rightmost)
 {
        if (!_session) {
                return;
@@ -333,64 +323,77 @@ Editor::compute_current_bbt_points (std::vector<TempoMap::BBTPoint>& grid, frame
 
        /* prevent negative values of leftmost from creeping into tempomap
         */
-       const double lower_beat = floor (max (0.0, _session->tempo_map().beat_at_frame (leftmost))) - 1.0;
+       const double lower_beat = floor (max (0.0, _session->tempo_map().beat_at_sample (leftmost))) - 1.0;
        switch (bbt_ruler_scale) {
 
-       case bbt_show_beats:
-       case bbt_show_ticks:
-       case bbt_show_ticks_detail:
-       case bbt_show_ticks_super_detail:
-               _session->tempo_map().get_grid (grid, max (_session->tempo_map().frame_at_beat (lower_beat), (framepos_t) 0), rightmost);
+       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().frame_at_beat (lower_beat), (framepos_t) 0), rightmost, 1);
+               _session->tempo_map().get_grid (grid, max (_session->tempo_map().sample_at_beat (lower_beat), (samplepos_t) 0), rightmost, 1);
                break;
 
        case bbt_show_4:
-               _session->tempo_map().get_grid (grid, max (_session->tempo_map().frame_at_beat (lower_beat), (framepos_t) 0), rightmost, 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().frame_at_beat (lower_beat), (framepos_t) 0), rightmost, 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().frame_at_beat (lower_beat), (framepos_t) 0), rightmost, 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().frame_at_beat (lower_beat), (framepos_t) 0), rightmost, 128);
+               _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 (std::vector<ARDOUR::TempoMap::BBTPoint>& grid)
+Editor::maybe_draw_grid_lines ()
 {
-       if (_session == 0 || _show_measures == false || distance (grid.begin(), grid.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);
        }
 
-       const unsigned divisions = get_grid_beat_divisions(leftmost_frame);
-       tempo_lines->draw (grid, divisions, leftmost_frame, _session->frame_rate());
+       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;
@@ -399,12 +402,12 @@ Editor::mouse_add_new_tempo_event (framepos_t frame)
        TempoMap& map(_session->tempo_map());
 
        begin_reversible_command (_("add tempo mark"));
-       const double pulse = map.exact_qn_at_frame (frame, get_grid_music_divisions (0)) / 4.0;
+       const double pulse = map.exact_qn_at_sample (sample, get_grid_music_divisions (0)) / 4.0;
 
        if (pulse > 0.0) {
                XMLNode &before = map.get_state();
-               /* add music-locked ramped (?) tempo using the bpm/note type at frame*/
-               map.add_tempo (map.tempo_at_frame (frame), pulse, 0, MusicTime);
+               /* add music-locked ramped (?) tempo using the bpm/note type at sample*/
+               map.add_tempo (map.tempo_at_sample (sample), pulse, 0, MusicTime);
 
                XMLNode &after = map.get_state();
                _session->add_command(new MementoCommand<TempoMap>(map, &before, &after));
@@ -415,7 +418,7 @@ Editor::mouse_add_new_tempo_event (framepos_t frame)
 }
 
 void
-Editor::mouse_add_new_meter_event (framepos_t frame)
+Editor::mouse_add_new_meter_event (samplepos_t sample)
 {
        if (_session == 0) {
                return;
@@ -423,7 +426,7 @@ Editor::mouse_add_new_meter_event (framepos_t frame)
 
 
        TempoMap& map(_session->tempo_map());
-       MeterDialog meter_dialog (map, frame, _("add"));
+       MeterDialog meter_dialog (map, sample, _("add"));
 
        switch (meter_dialog.run ()) {
        case RESPONSE_ACCEPT:
@@ -440,15 +443,14 @@ Editor::mouse_add_new_meter_event (framepos_t frame)
        Timecode::BBT_Time requested;
        meter_dialog.get_bbt_time (requested);
 
-       const double beat = map.beat_at_bbt (requested);
-       const double al_frame = map.frame_at_beat (beat);
+       const double al_sample = map.sample_at_bbt (requested);
        begin_reversible_command (_("add meter mark"));
-        XMLNode &before = map.get_state();
+       XMLNode &before = map.get_state();
 
        if (meter_dialog.get_lock_style() == MusicTime) {
-               map.add_meter (Meter (bpb, note_type), beat, requested, 0, MusicTime);
+               map.add_meter (Meter (bpb, note_type), requested, 0, MusicTime);
        } else {
-               map.add_meter (Meter (bpb, note_type), beat, requested, al_frame, AudioTime);
+               map.add_meter (Meter (bpb, note_type), requested, al_sample, AudioTime);
        }
 
        _session->add_command(new MementoCommand<TempoMap>(map, &before, &map.get_state()));
@@ -498,13 +500,13 @@ Editor::edit_meter_section (MeterSection* section)
 
        Timecode::BBT_Time when;
        meter_dialog.get_bbt_time (when);
-       const framepos_t frame = _session->tempo_map().frame_at_bbt (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 meter mark"));
-        XMLNode &before = _session->tempo_map().get_state();
+       XMLNode &before = _session->tempo_map().get_state();
 
-       _session->tempo_map().replace_meter (*section, meter, when, frame, pls);
+       _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));
@@ -524,9 +526,10 @@ 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 ();
        bpm = max (0.01, bpm);
-       const Tempo tempo (bpm, nt);
+       const Tempo tempo (bpm, nt, end_bpm);
 
        Timecode::BBT_Time when;
        tempo_dialog.get_bbt_time (when);
@@ -535,7 +538,7 @@ Editor::edit_tempo_section (TempoSection* section)
        XMLNode &before = _session->tempo_map().get_state();
 
        if (tempo_dialog.get_lock_style() == AudioTime) {
-               framepos_t const f = _session->tempo_map().predict_tempo_position (section, when).second;
+               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;