X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Feditor_tempodisplay.cc;h=177f82bd44b24946236f8031fc7d88cd951b3cc0;hb=2592a320d42dd4a157ee16101c042d875d3142be;hp=e9222999929f185f80c07f576e402025862fcede;hpb=e513d106d7ecf7e629f55c6eb554a2638f9c6443;p=ardour.git diff --git a/gtk2_ardour/editor_tempodisplay.cc b/gtk2_ardour/editor_tempodisplay.cc index e922299992..177f82bd44 100644 --- a/gtk2_ardour/editor_tempodisplay.cc +++ b/gtk2_ardour/editor_tempodisplay.cc @@ -44,6 +44,7 @@ #include "rgb_macros.h" #include "gui_thread.h" #include "color.h" +#include "time_axis_view.h" #include "i18n.h" @@ -100,25 +101,23 @@ Editor::tempo_map_changed (Change ignored) BBT_Time previous_beat, next_beat; // the beats previous to the leftmost frame and after the rightmost frame session->bbt_time(leftmost_frame, previous_beat); + session->bbt_time(leftmost_frame + current_page_frames(), next_beat); - previous_beat.ticks = 0; if (previous_beat.beats > 1) { previous_beat.beats -= 1; } else if (previous_beat.bars > 1) { previous_beat.bars--; previous_beat.beats += 1; } - - session->bbt_time(leftmost_frame + current_page_frames(), next_beat); + previous_beat.ticks = 0; if (session->tempo_map().meter_at(leftmost_frame + current_page_frames()).beats_per_bar () > next_beat.beats + 1) { next_beat.beats += 1; - next_beat.ticks = 0; } else { next_beat.bars += 1; next_beat.beats = 1; - next_beat.ticks = 0; } + next_beat.ticks = 0; if (current_bbt_points) { delete current_bbt_points; @@ -127,16 +126,7 @@ Editor::tempo_map_changed (Change ignored) if (session) { current_bbt_points = session->tempo_map().get_points (session->tempo_map().frame_time (previous_beat), session->tempo_map().frame_time (next_beat)); - - /* - TempoMap::BBTPointList::iterator i; - cerr << "******************" << endl << "current bbt points dump: " << endl; - for (i = current_bbt_points->begin(); i != current_bbt_points->end(); i++) { - cerr << (*i).bar << " : " << (*i).beat << endl; - } - cerr << "******************" << endl; - */ - + update_tempo_based_rulers (); } else { current_bbt_points = 0; } @@ -148,11 +138,10 @@ void Editor::redisplay_tempo () { - hide_measures (); - if (session && current_bbt_points) { - Glib::signal_idle().connect (mem_fun (*this, &Editor::draw_measures)); - update_tempo_based_rulers (); + Glib::signal_idle().connect (mem_fun (*this, &Editor::lazy_hide_and_draw_measures)); + } else { + hide_measures (); } } @@ -160,7 +149,7 @@ void Editor::hide_measures () { for (TimeLineList::iterator i = used_measure_lines.begin(); i != used_measure_lines.end(); ++i) { - (*i)->hide(); + (*i)->hide(); free_measure_lines.push_back (*i); } used_measure_lines.clear (); @@ -184,93 +173,79 @@ Editor::get_time_line () } bool +Editor::lazy_hide_and_draw_measures () +{ + hide_measures (); + draw_measures (); + return false; +} + +void Editor::draw_measures () { if (session == 0 || _show_measures == false) { - return false; + return; } TempoMap::BBTPointList::iterator i; ArdourCanvas::SimpleLine *line; - gdouble xpos, last_xpos; - uint32_t cnt; + gdouble xpos; + double x1, x2, y1, y2, beat_density; + + uint32_t beats = 0; + uint32_t bars = 0; uint32_t color; if (current_bbt_points == 0 || current_bbt_points->empty()) { - return false; + return; } - cnt = 0; - last_xpos = 0; + track_canvas.get_scroll_region (x1, y1, x2, y2); + y2 = TimeAxisView::hLargest*5000; // five thousand largest tracks should be enough.. :) /* get the first bar spacing */ - gdouble last_beat = DBL_MAX; - gdouble beat_spacing = 0; + i = current_bbt_points->end(); + i--; + bars = (*i).bar - (*current_bbt_points->begin()).bar; + beats = current_bbt_points->size() - bars; - for (i = current_bbt_points->begin(); i != current_bbt_points->end() && beat_spacing == 0; ++i) { - TempoMap::BBTPoint& p = (*i); - - switch (p.type) { - case TempoMap::Bar: - break; + beat_density = (beats * 10.0f) / track_canvas.get_width (); - case TempoMap::Beat: - xpos = frame_to_unit (p.frame); - if (last_beat < xpos) { - beat_spacing = xpos - last_beat; - } - last_beat = xpos; - } - } - - if (beat_spacing < 12.0) { - /* - if the lines are too close together, - they become useless - */ - return false; + if (beat_density > 4.0f) { + /* if the lines are too close together, they become useless + */ + return; } - double x1, x2, y1, y2; - track_canvas.get_scroll_region (x1, y1, x2, y2); - for (i = current_bbt_points->begin(); i != current_bbt_points->end(); ++i) { - TempoMap::BBTPoint& p = (*i); - - switch (p.type) { + switch ((*i).type) { case TempoMap::Bar: break; case TempoMap::Beat: - xpos = frame_to_unit (p.frame); - if (p.beat == 1) { + if ((*i).beat == 1) { color = color_map[cMeasureLineBeat]; } else { color = color_map[cMeasureLineBar]; - /* only draw beat lines if the gaps between beats - are large. - */ - - if (beat_spacing < 25.0) { + if (beat_density > 2.0) { + /* only draw beat lines if the gaps between beats are large. + */ break; } } - - if (cnt == 0 || xpos - last_xpos > 4.0) { - line = get_time_line (); - line->property_x1() = xpos; - line->property_x2() = xpos; - line->property_y2() = y2; - line->property_color_rgba() = color; - line->raise_to_top(); - line->show(); - last_xpos = xpos; - ++cnt; - } + + xpos = frame_to_unit ((*i).frame); + line = get_time_line (); + line->property_x1() = xpos; + line->property_x2() = xpos; + line->property_y2() = y2; + line->property_color_rgba() = color; + //line->raise_to_top(); + line->show(); break; } } @@ -279,11 +254,11 @@ Editor::draw_measures () cursor_group->raise_to_top(); time_line_group->lower_to_bottom(); - return false; + return; } void -Editor::mouse_add_new_tempo_event (jack_nframes_t frame) +Editor::mouse_add_new_tempo_event (nframes_t frame) { if (session == 0) { return; @@ -325,7 +300,7 @@ Editor::mouse_add_new_tempo_event (jack_nframes_t frame) } void -Editor::mouse_add_new_meter_event (jack_nframes_t frame) +Editor::mouse_add_new_meter_event (nframes_t frame) { if (session == 0) { return;