Use boost::shared_ptr instead of raw pointers for RouteUI track (etc) accessors ...
[ardour.git] / gtk2_ardour / editor_tempodisplay.cc
index e9222999929f185f80c07f576e402025862fcede..7fe87596390436f0a73fae47b9f2216c672535ef 100644 (file)
@@ -15,7 +15,6 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <cstdio> // for sprintf, grrr 
@@ -44,6 +43,7 @@
 #include "rgb_macros.h"
 #include "gui_thread.h"
 #include "color.h"
+#include "time_axis_view.h"
 
 #include "i18n.h"
 
@@ -95,31 +95,48 @@ Editor::draw_metric_marks (const Metrics& metrics)
 void
 Editor::tempo_map_changed (Change ignored)
 {
-        ENSURE_GUI_THREAD(bind (mem_fun(*this, &Editor::tempo_map_changed), ignored));
+       if (!session) {
+               return;
+       }
+
+       ENSURE_GUI_THREAD(bind (mem_fun (*this, &Editor::tempo_map_changed), ignored));
+       
+       redisplay_tempo (false); // redraw rulers and measures
+       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks); // redraw metric markers
+}
+
+/**
+ * This code was originally in tempo_map_changed, but this is called every time the canvas scrolls horizontally. 
+ * That's why this is moved in here. The new tempo_map_changed is called when the ARDOUR::TempoMap actually changed.
+ */
+void
+Editor::redisplay_tempo (bool immediate_redraw)
+{
+       if (!session) {
+               return;
+       }
 
        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;
+               next_beat.beats += 1;
        } else {
-         next_beat.bars += 1;
-         next_beat.beats = 1;
-         next_beat.ticks = 0;
+               next_beat.bars += 1;
+               next_beat.beats = 1;
        }
-
+       next_beat.ticks = 0;
+       
        if (current_bbt_points) {
                delete current_bbt_points;
                current_bbt_points = 0;
@@ -127,32 +144,26 @@ 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;
        }
 
-       redisplay_tempo ();
-}
+       if (immediate_redraw) {
 
-void
-Editor::redisplay_tempo ()
-{      
+               hide_measures ();
 
-       hide_measures ();
+               if (session && current_bbt_points) {
+                       draw_measures ();
+               }
 
-       if (session && current_bbt_points) {
-               Glib::signal_idle().connect (mem_fun (*this, &Editor::draw_measures));
-               update_tempo_based_rulers ();
+       } else {
+
+               if (session && current_bbt_points) {
+                       Glib::signal_idle().connect (mem_fun (*this, &Editor::lazy_hide_and_draw_measures));
+               } else {
+                       hide_measures ();
+               }
        }
 }
 
@@ -160,7 +171,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 ();
@@ -169,10 +180,10 @@ Editor::hide_measures ()
 ArdourCanvas::SimpleLine *
 Editor::get_time_line ()
 {
-         ArdourCanvas::SimpleLine *line;
+       ArdourCanvas::SimpleLine *line;
 
        if (free_measure_lines.empty()) {
-               line = new ArdourCanvas::SimpleLine (*time_line_group);
+               line = new ArdourCanvas::SimpleLine (*time_line_group);
                used_measure_lines.push_back (line);
        } else {
                line = free_measure_lines.front();
@@ -184,93 +195,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);
+       beat_density =  (beats * 10.0f) / track_canvas.get_width ();
 
-               switch (p.type) {
-               case TempoMap::Bar:
-                       break;
-
-               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) {
-                               color = color_map[cMeasureLineBeat];
-                       } else {
+                       if ((*i).beat == 1) {
                                color = color_map[cMeasureLineBar];
+                       } else {
+                               color = color_map[cMeasureLineBeat];
 
-                               /* 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 +276,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;
@@ -320,12 +317,10 @@ Editor::mouse_add_new_tempo_event (jack_nframes_t frame)
        commit_reversible_command ();
        
        map.dump (cerr);
-
-       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 }
 
 void
-Editor::mouse_add_new_meter_event (jack_nframes_t frame)
+Editor::mouse_add_new_meter_event (nframes_t frame)
 {
        if (session == 0) {
                return;
@@ -362,8 +357,6 @@ Editor::mouse_add_new_meter_event (jack_nframes_t frame)
        commit_reversible_command ();
        
        map.dump (cerr);
-
-       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 }
 
 void
@@ -414,8 +407,6 @@ Editor::edit_meter_section (MeterSection* section)
         XMLNode &after = session->tempo_map().get_state();
        session->add_command(new MementoCommand<TempoMap>(session->tempo_map(), &before, &after));
        commit_reversible_command ();
-
-       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 }
 
 void
@@ -446,8 +437,6 @@ Editor::edit_tempo_section (TempoSection* section)
         XMLNode &after = session->tempo_map().get_state();
        session->add_command (new MementoCommand<TempoMap>(session->tempo_map(), &before, &after));
        commit_reversible_command ();
-
-       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
 }
 
 void
@@ -498,8 +487,6 @@ Editor::real_remove_tempo_marker (TempoSection *section)
        session->add_command(new MementoCommand<TempoMap>(session->tempo_map(), &before, &after));
        commit_reversible_command ();
 
-       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
-
        return FALSE;
 }
 
@@ -534,7 +521,5 @@ Editor::real_remove_meter_marker (MeterSection *section)
        session->add_command(new MementoCommand<TempoMap>(session->tempo_map(), &before, &after));
        commit_reversible_command ();
 
-       session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks);
-
        return FALSE;
 }