final for #1600
[ardour.git] / gtk2_ardour / editor_rulers.cc
index ab43306b5ee247138ebad6dd110aa9259302589b..6f4f4bdb6876a4000e1569ae2f79d950b826ec37 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 
@@ -35,6 +34,7 @@
 
 using namespace sigc;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace Gtk;
 using namespace Editing;
 
@@ -55,6 +55,10 @@ Editor::initialize_rulers ()
        ruler_editor = this;
        ruler_grabbed_widget = 0;
        
+       _ruler_separator = new Gtk::HSeparator();
+       _ruler_separator->set_size_request(-1, 2);
+       _ruler_separator->show();
+
        _smpte_ruler = gtk_custom_hruler_new ();
        smpte_ruler = Glib::wrap (_smpte_ruler);
        smpte_ruler->set_name ("SMPTERuler");
@@ -137,6 +141,40 @@ Editor::ruler_button_press (GdkEventButton* ev)
                ruler_grabbed_widget = grab_widget;
        }
 
+       gint x,y;
+       Gdk::ModifierType state;
+
+       /* need to use the correct x,y, the event lies */
+       time_canvas_event_box.get_window()->get_pointer (x, y, state);
+
+       nframes_t where = leftmost_frame + pixel_to_frame (x);
+
+       switch (ev->button) {
+       case 1:
+               // Since we are about to move the playhead, cancel any running
+               // auditions.
+               if (session->is_auditioning()) {
+                       session->cancel_audition ();
+               }
+               /* transport playhead */
+               snap_to (where);
+               session->request_locate (where);
+               _dragging_playhead = true;
+               break;
+
+       case 2:
+               /* edit cursor */
+               if (snap_type != Editing::SnapToEditCursor) {
+                       snap_to (where);
+               }
+               edit_cursor->set_position (where);
+               edit_cursor_clock.set (where);
+               break;
+
+       default:
+               break;
+       }
+
        return TRUE;
 }
 
@@ -149,21 +187,20 @@ Editor::ruler_button_release (GdkEventButton* ev)
        /* need to use the correct x,y, the event lies */
        time_canvas_event_box.get_window()->get_pointer (x, y, state);
 
-
        ruler_pressed_button = 0;
        
        if (session == 0) {
                return FALSE;
        }
 
-       hide_verbose_canvas_cursor();
        stop_canvas_autoscroll();
        
-       jack_nframes_t where = leftmost_frame + pixel_to_frame (x);
+       nframes_t where = leftmost_frame + pixel_to_frame (x);
 
        switch (ev->button) {
        case 1:
                /* transport playhead */
+               _dragging_playhead = false;
                snap_to (where);
                session->request_locate (where);
                break;
@@ -214,8 +251,8 @@ Editor::ruler_mouse_motion (GdkEventMotion* ev)
        if (session == 0 || !ruler_pressed_button) {
                return FALSE;
        }
-
-       double wcx=0,wcy=0;
+       
+               double wcx=0,wcy=0;
        double cx=0,cy=0;
 
        gint x,y;
@@ -224,30 +261,28 @@ Editor::ruler_mouse_motion (GdkEventMotion* ev)
        /* need to use the correct x,y, the event lies */
        time_canvas_event_box.get_window()->get_pointer (x, y, state);
 
-       
+
        track_canvas.c2w (x, y, wcx, wcy);
        track_canvas.w2c (wcx, wcy, cx, cy);
        
-       jack_nframes_t where = leftmost_frame + pixel_to_frame (x);
-
-       /// ripped from maybe_autoscroll
-       jack_nframes_t one_page = (jack_nframes_t) rint (canvas_width * frames_per_unit);
-       jack_nframes_t rightmost_frame = leftmost_frame + one_page;
+       nframes_t where = leftmost_frame + pixel_to_frame (x);
 
-       jack_nframes_t frame = pixel_to_frame (cx);
+       /// ripped from maybe_autoscroll, and adapted to work here
+       nframes_t one_page = (nframes_t) rint (canvas_width * frames_per_unit);
+       nframes_t rightmost_frame = leftmost_frame + one_page;
 
        if (autoscroll_timeout_tag < 0) {
-               if (frame > rightmost_frame) {
+               if (where > rightmost_frame) {
                        if (rightmost_frame < max_frames) {
                                start_canvas_autoscroll (1);
                        }
-               } else if (frame < leftmost_frame) {
+               } else if (where <= leftmost_frame) {
                        if (leftmost_frame > 0) {
                                start_canvas_autoscroll (-1);
                        }
                } 
        } else {
-               if (frame >= leftmost_frame && frame < rightmost_frame) {
+               if (where >= leftmost_frame && where < rightmost_frame) {
                        stop_canvas_autoscroll ();
                }
        }
@@ -272,15 +307,14 @@ Editor::ruler_mouse_motion (GdkEventMotion* ev)
                break;
        }
 
-       if (cursor)
-       {
+       if (cursor) {
                cursor->set_position (where);
                
                if (cursor == edit_cursor) {
                        edit_cursor_clock.set (where);
+               } else if (cursor == playhead_cursor) {
+                       UpdateAllTransportClocks (cursor->current_frame);
                }
-               
-               show_verbose_time_cursor (where, 10, cx, 0);
        }
        
        return TRUE;
@@ -288,7 +322,7 @@ Editor::ruler_mouse_motion (GdkEventMotion* ev)
 
 
 void
-Editor::popup_ruler_menu (jack_nframes_t where, ItemType t)
+Editor::popup_ruler_menu (nframes_t where, ItemType t)
 {
        using namespace Menu_Helpers;
 
@@ -310,11 +344,13 @@ Editor::popup_ruler_menu (jack_nframes_t where, ItemType t)
        case MarkerBarItem:
                ruler_items.push_back (MenuElem (_("New location marker"), bind ( mem_fun(*this, &Editor::mouse_add_new_marker), where)));
                ruler_items.push_back (MenuElem (_("Clear all locations"), mem_fun(*this, &Editor::clear_markers)));
+               ruler_items.push_back (MenuElem (_("Unhide locations"), mem_fun(*this, &Editor::unhide_markers)));
                ruler_items.push_back (SeparatorElem ());
                break;
        case RangeMarkerBarItem:
                //ruler_items.push_back (MenuElem (_("New Range")));
                ruler_items.push_back (MenuElem (_("Clear all ranges"), mem_fun(*this, &Editor::clear_ranges)));
+               ruler_items.push_back (MenuElem (_("Unhide ranges"), mem_fun(*this, &Editor::unhide_ranges)));
                ruler_items.push_back (SeparatorElem ());
 
                break;
@@ -344,7 +380,7 @@ Editor::popup_ruler_menu (jack_nframes_t where, ItemType t)
                mitem->set_active(true);
        }
 
-       ruler_items.push_back (CheckMenuElem (X_("SMPTE"), bind (mem_fun(*this, &Editor::ruler_toggled), (int)ruler_metric_smpte)));
+       ruler_items.push_back (CheckMenuElem (_("Timecode"), bind (mem_fun(*this, &Editor::ruler_toggled), (int)ruler_metric_smpte)));
        mitem = (CheckMenuItem *) &ruler_items.back(); 
        if (ruler_shown[ruler_metric_smpte]) {
                mitem->set_active(true);
@@ -394,7 +430,7 @@ Editor::popup_ruler_menu (jack_nframes_t where, ItemType t)
                mitem->set_active(true);
        }
        
-        editor_ruler_menu->popup (1, 0);
+        editor_ruler_menu->popup (1, gtk_get_current_event_time());
 
        no_ruler_shown_update = false;
 }
@@ -440,7 +476,7 @@ Editor::store_ruler_visibility ()
        session->add_extra_xml (*node);
        session->set_dirty ();
 }
+
 void 
 Editor::restore_ruler_visibility ()
 {
@@ -520,7 +556,7 @@ Editor::update_ruler_visibility ()
 
        lab_children.clear();
 
-       // leave the last one (the time_canvas_scroller) intact
+       // leave the last one (the time_canvas) intact
        while (ruler_children.size() > 1) {
                ruler_children.pop_front();
        }
@@ -572,6 +608,7 @@ Editor::update_ruler_visibility ()
        frames_ruler->signal_motion_notify_event().connect (mem_fun(*this, &Editor::ruler_mouse_motion));
        minsec_ruler->signal_motion_notify_event().connect (mem_fun(*this, &Editor::ruler_mouse_motion));
 
+       ruler_children.insert (canvaspos, Element(*_ruler_separator, PACK_SHRINK, PACK_START));
        
        if (ruler_shown[ruler_metric_minsec]) {
                lab_children.push_back (Element(minsec_label, PACK_SHRINK, PACK_START));
@@ -597,7 +634,7 @@ Editor::update_ruler_visibility ()
                visible_timebars++;
        }
 
-       double tbpos = 0.0;
+       double tbpos = 1.0;
        double old_unit_pos ;
        
        if (ruler_shown[ruler_time_meter]) {
@@ -617,7 +654,7 @@ Editor::update_ruler_visibility ()
        
        if (ruler_shown[ruler_time_tempo]) {
                lab_children.push_back (Element(tempo_label, PACK_SHRINK, PACK_START));
-               gtk_object_get (GTK_OBJECT(tempo_group), "y", &old_unit_pos);
+               old_unit_pos = tempo_group->property_y();
                if (tbpos != old_unit_pos) {
                        tempo_group->move(0.0, tbpos - old_unit_pos);
                }
@@ -676,10 +713,10 @@ Editor::update_ruler_visibility ()
        
        update_fixed_rulers();
        //update_tempo_based_rulers();
-       tempo_map_changed(Change (0));
+       redisplay_tempo (false);
 
        time_canvas_event_box.show_all();
-       time_button_event_box.show_all();
+       time_button_frame.show_all();
 }
 
 void
@@ -695,8 +732,8 @@ Editor::update_just_smpte ()
           an uint32_t (or larger) to a float ... what to do ?
        */
 
-       jack_nframes_t page = (jack_nframes_t) floor (canvas_width * frames_per_unit);
-       jack_nframes_t rightmost_frame = leftmost_frame + page;
+       nframes_t page = (nframes_t) floor (canvas_width * frames_per_unit);
+       nframes_t rightmost_frame = leftmost_frame + page;
 
        if (ruler_shown[ruler_metric_smpte]) {
                gtk_custom_ruler_set_range (GTK_CUSTOM_RULER(_smpte_ruler), leftmost_frame, rightmost_frame,
@@ -707,7 +744,7 @@ Editor::update_just_smpte ()
 void
 Editor::update_fixed_rulers ()
 {
-       jack_nframes_t rightmost_frame;
+       nframes_t rightmost_frame;
 
        if (session == 0) {
                return;
@@ -717,7 +754,7 @@ Editor::update_fixed_rulers ()
           an uint32_t (or larger) to a float ... what to do ?
        */
 
-       jack_nframes_t page = (jack_nframes_t) floor (canvas_width * frames_per_unit);
+       nframes_t page = (nframes_t) floor (canvas_width * frames_per_unit);
 
        ruler_metrics[ruler_metric_smpte].units_per_pixel = frames_per_unit;
        ruler_metrics[ruler_metric_frames].units_per_pixel = frames_per_unit;
@@ -756,7 +793,7 @@ Editor::update_tempo_based_rulers ()
           an uint32_t (or larger) to a float ... what to do ?
        */
 
-       jack_nframes_t page = (jack_nframes_t) floor (canvas_width * frames_per_unit);
+       nframes_t page = (nframes_t) floor (canvas_width * frames_per_unit);
        ruler_metrics[ruler_metric_bbt].units_per_pixel = frames_per_unit;
 
        if (ruler_shown[ruler_metric_bbt]) {
@@ -768,37 +805,37 @@ Editor::update_tempo_based_rulers ()
 /* Mark generation */
 
 gint
-Editor::_metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper, gint maxchars)
+Editor::_metric_get_smpte (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
        return ruler_editor->metric_get_smpte (marks, lower, upper, maxchars);
 }
 
 gint
-Editor::_metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper, gint maxchars)
+Editor::_metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
        return ruler_editor->metric_get_bbt (marks, lower, upper, maxchars);
 }
 
 gint
-Editor::_metric_get_frames (GtkCustomRulerMark **marks, gulong lower, gulong upper, gint maxchars)
+Editor::_metric_get_frames (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
        return ruler_editor->metric_get_frames (marks, lower, upper, maxchars);
 }
 
 gint
-Editor::_metric_get_minsec (GtkCustomRulerMark **marks, gulong lower, gulong upper, gint maxchars)
+Editor::_metric_get_minsec (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
        return ruler_editor->metric_get_minsec (marks, lower, upper, maxchars);
 }
 
 gint
-Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper, gint maxchars)
+Editor::metric_get_smpte (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
-       jack_nframes_t range;
-       jack_nframes_t pos;
-       jack_nframes_t spacer;
-       jack_nframes_t fr;
-       SMPTE_Time smpte;
+       nframes_t range;
+       nframes_t pos;
+       nframes_t spacer;
+       nframes_t fr;
+       SMPTE::Time smpte;
        gchar buf[16];
        gint nmarks = 0;
        gint n;
@@ -815,34 +852,34 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
 
        fr = session->frame_rate();
 
-       if (lower > (spacer = (jack_nframes_t)(128 * Editor::get_current_zoom ()))) {
+       if (lower > (spacer = (nframes_t)(128 * Editor::get_current_zoom ()))) {
                lower = lower - spacer;
        } else {
-               upper = upper + spacer - lower;
                lower = 0;
        }
-       range = upper - lower;
+       upper = upper + spacer;
+       range = (nframes_t) floor (upper - lower);
 
        if (range < (2 * session->frames_per_smpte_frame())) { /* 0 - 2 frames */
                show_bits = true;
                mark_modulo = 20;
-               nmarks = 1 + 160;
+               nmarks = 1 + (2 * Config->get_subframes_per_frame());
        } else if (range <= (fr / 4)) { /* 2 frames - 0.250 second */
                show_frames = true;
                mark_modulo = 1;
-               nmarks = 1 + (range / (jack_nframes_t)session->frames_per_smpte_frame());
+               nmarks = 1 + (range / (nframes_t)session->frames_per_smpte_frame());
        } else if (range <= (fr / 2)) { /* 0.25-0.5 second */
                show_frames = true;
                mark_modulo = 2;
-               nmarks = 1 + (range / (jack_nframes_t)session->frames_per_smpte_frame());
+               nmarks = 1 + (range / (nframes_t)session->frames_per_smpte_frame());
        } else if (range <= fr) { /* 0.5-1 second */
                show_frames = true;
                mark_modulo = 5;
-               nmarks = 1 + (range / (jack_nframes_t)session->frames_per_smpte_frame());
+               nmarks = 1 + (range / (nframes_t)session->frames_per_smpte_frame());
        } else if (range <= 2 * fr) { /* 1-2 seconds */
                show_frames = true;
                mark_modulo = 10;
-               nmarks = 1 + (range / (jack_nframes_t)session->frames_per_smpte_frame());
+               nmarks = 1 + (range / (nframes_t)session->frames_per_smpte_frame());
        } else if (range <= 8 * fr) { /* 2-8 seconds */
                show_seconds = true;
                mark_modulo = 1;
@@ -893,15 +930,15 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
                nmarks = 1 + 24;
        } else {
     
-               /* not possible if jack_nframes_t is a 32 bit quantity */
+               /* not possible if nframes_t is a 32 bit quantity */
     
                show_hours = true;
                mark_modulo = 4;
                nmarks = 1 + 24;
        }
   
-       pos = lower;
-
+       pos = (nframes_t) floor (lower);
+       
        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);  
        
        if (show_bits) {
@@ -913,10 +950,10 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
                        if ((smpte.subframes % mark_modulo) == 0) {
                                if (smpte.subframes == 0) {
                                        (*marks)[n].style = GtkCustomRulerMarkMajor;
-                                       snprintf (buf, sizeof(buf), "%s%02ld:%02ld:%02ld:%02ld", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
+                                       snprintf (buf, sizeof(buf), "%s%02u:%02u:%02u:%02u", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
                                } else {
                                        (*marks)[n].style = GtkCustomRulerMarkMinor;
-                                       snprintf (buf, sizeof(buf), ".%02ld", smpte.subframes);
+                                       snprintf (buf, sizeof(buf), ".%02u", smpte.subframes);
                                }
                        } else {
                                snprintf (buf, sizeof(buf)," ");
@@ -927,13 +964,13 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
                        (*marks)[n].position = pos;
 
                        // Increment subframes by one
-                       session->smpte_increment_subframes( smpte );
+                       SMPTE::increment_subframes( smpte );
                }
        } else if (show_seconds) {
                // Find smpte time of this sample (pos)
                session->sample_to_smpte(pos, smpte, true /* use_offset */, false /* use_subframes */ );
                // Go to next whole second down
-               session->smpte_seconds_floor( smpte );
+               SMPTE::seconds_floor( smpte );
 
                for (n = 0; n < nmarks; n++) {
                        session->smpte_to_sample(smpte, pos, true /* use_offset */, false /* use_subframes */ );
@@ -945,7 +982,7 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
                                        (*marks)[n].style = GtkCustomRulerMarkMinor;
                                        (*marks)[n].position = pos;
                                }
-                               snprintf (buf, sizeof(buf), "%s%02ld:%02ld:%02ld:%02ld", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
+                               snprintf (buf, sizeof(buf), "%s%02u:%02u:%02u:%02u", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
                        } else {
                                snprintf (buf, sizeof(buf)," ");
                                (*marks)[n].style = GtkCustomRulerMarkMicro;
@@ -953,13 +990,13 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
         
                        }
                        (*marks)[n].label = g_strdup (buf);
-                       session->smpte_increment_seconds( smpte );
+                       SMPTE::increment_seconds( smpte );
                }
        } else if (show_minutes) {
                // Find smpte time of this sample (pos)
                session->sample_to_smpte(pos, smpte, true /* use_offset */, false /* use_subframes */ );
                // Go to next whole minute down
-               session->smpte_minutes_floor( smpte );
+               SMPTE::minutes_floor( smpte );
 
                for (n = 0; n < nmarks; n++) {
                        session->smpte_to_sample(smpte, pos, true /* use_offset */, false /* use_subframes */ );
@@ -969,7 +1006,7 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
                                } else {
                                        (*marks)[n].style = GtkCustomRulerMarkMinor;
                                }
-                               snprintf (buf, sizeof(buf), "%s%02ld:%02ld:%02ld:%02ld", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
+                               snprintf (buf, sizeof(buf), "%s%02u:%02u:%02u:%02u", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
                        } else {
                                snprintf (buf, sizeof(buf)," ");
                                (*marks)[n].style = GtkCustomRulerMarkMicro;
@@ -977,19 +1014,19 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
                        }
                        (*marks)[n].label = g_strdup (buf);
                        (*marks)[n].position = pos;
-                       session->smpte_increment_minutes( smpte );
+                       SMPTE::increment_minutes( smpte );
                }
        } else if (show_hours) {
                // Find smpte time of this sample (pos)
                session->sample_to_smpte(pos, smpte, true /* use_offset */, false /* use_subframes */ );
                // Go to next whole hour down
-               session->smpte_hours_floor( smpte );
+               SMPTE::hours_floor( smpte );
 
                for (n = 0; n < nmarks; n++) {
                        session->smpte_to_sample(smpte, pos, true /* use_offset */, false /* use_subframes */ );
                        if ((smpte.hours % mark_modulo) == 0) {
                                (*marks)[n].style = GtkCustomRulerMarkMajor;
-                               snprintf (buf, sizeof(buf), "%s%02ld:%02ld:%02ld:%02ld", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
+                               snprintf (buf, sizeof(buf), "%s%02u:%02u:%02u:%02u", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
                        } else {
                                snprintf (buf, sizeof(buf)," ");
                                (*marks)[n].style = GtkCustomRulerMarkMicro;
@@ -998,20 +1035,20 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
                        (*marks)[n].label = g_strdup (buf);
                        (*marks)[n].position = pos;
 
-                       session->smpte_increment_hours( smpte );
+                       SMPTE::increment_hours( smpte );
                }
        } else { // show_frames
                // Find smpte time of this sample (pos)
                session->sample_to_smpte(pos, smpte, true /* use_offset */, false /* use_subframes */ );
                // Go to next whole frame down
-               session->smpte_frames_floor( smpte );
+               SMPTE::frames_floor( smpte );
 
                for (n = 0; n < nmarks; n++) {
                        session->smpte_to_sample(smpte, pos, true /* use_offset */, false /* use_subframes */ );
                        if ((smpte.frames % mark_modulo) == 0)  {
                                (*marks)[n].style = GtkCustomRulerMarkMajor;
                                (*marks)[n].position = pos;
-                               snprintf (buf, sizeof(buf), "%s%02ld:%02ld:%02ld:%02ld", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
+                               snprintf (buf, sizeof(buf), "%s%02u:%02u:%02u:%02u", smpte.negative ? "-" : "", smpte.hours, smpte.minutes, smpte.seconds, smpte.frames);
                        } else {
                                snprintf (buf, sizeof(buf)," ");
                                (*marks)[n].style = GtkCustomRulerMarkMicro;
@@ -1019,7 +1056,7 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
         
                        }
                        (*marks)[n].label = g_strdup (buf);
-                       session->smpte_increment( smpte );
+                       SMPTE::increment( smpte );
                }
        }
   
@@ -1028,118 +1065,93 @@ Editor::metric_get_smpte (GtkCustomRulerMark **marks, gulong lower, gulong upper
 
 
 gint
-Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper, gint maxchars)
+Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
         if (session == 0) {
                 return 0;
         }
 
-        TempoMap::BBTPointList::iterator i;
-       TempoMap::BBTPointList *zoomed_bbt_points;
+       TempoMap::BBTPointList::iterator i;
+
         uint32_t beats = 0;
         uint32_t bars = 0;
-       uint32_t tick = 0;
-       uint32_t skip;
-       uint32_t t;
-        uint32_t zoomed_beats = 0;
-        uint32_t zoomed_bars = 0;
         uint32_t desirable_marks;
        uint32_t magic_accent_number = 1;
        gint nmarks;
         char buf[64];
-        gint n;
-       jack_nframes_t pos;
-        jack_nframes_t frame_one_beats_worth;
-        jack_nframes_t frame_skip;
-       double frame_skip_error;
-       double accumulated_error;
-        bool bar_helper_on = true;
-
+        gint  n = 0;
+       nframes_t pos;
+       bool bar_helper_on = true;
        
-       BBT_Time previous_beat;
        BBT_Time next_beat;
-       jack_nframes_t next_beat_pos;
+       nframes_t next_beat_pos;
 
-       if ((desirable_marks = maxchars / 6) == 0) {
+       if ((desirable_marks = maxchars / 7) == 0) {
                return 0;
         }
 
         /* align the tick marks to whatever we're snapping to... */
-                                                                                                             
-        if (snap_type == SnapToAThirdBeat) {
+
+       switch (snap_type) {
+       case SnapToAThirdBeat:
                 bbt_beat_subdivision = 3;
-        } else if (snap_type == SnapToAQuarterBeat) {
+               break;
+       case SnapToAQuarterBeat:
                 bbt_beat_subdivision = 4;
-        } else if (snap_type == SnapToAEighthBeat) {
+               break;
+       case SnapToAEighthBeat:
                 bbt_beat_subdivision = 8;
                magic_accent_number = 2;
-        } else if (snap_type == SnapToASixteenthBeat) {
+               break;
+       case SnapToASixteenthBeat:
                 bbt_beat_subdivision = 16;
                magic_accent_number = 4;
-        } else if (snap_type == SnapToAThirtysecondBeat) {
+               break;
+       case SnapToAThirtysecondBeat:
                 bbt_beat_subdivision = 32;
                magic_accent_number = 8;
-        } else {
+               break;
+       default:
               bbt_beat_subdivision = 4;
+               break;
        }
 
-       /* First find what a beat's distance is, so we can start plotting stuff before the beginning of the ruler */
-
-       session->bbt_time(lower,previous_beat);
-       previous_beat.ticks = 0;
-       next_beat = previous_beat;
-
-       if (session->tempo_map().meter_at(lower).beats_per_bar() < (next_beat.beats + 1)) {
-              next_beat.bars += 1;
-              next_beat.beats = 1;
-       } else {
-              next_beat.beats += 1;
-       }
-
-       frame_one_beats_worth = session->tempo_map().frame_time(next_beat) - session->tempo_map().frame_time(previous_beat);
-
-
-       zoomed_bbt_points = session->tempo_map().get_points((lower >= frame_one_beats_worth) ? lower - frame_one_beats_worth : 0, upper);
-
-       if (current_bbt_points == 0 || zoomed_bbt_points == 0 || zoomed_bbt_points->empty()) {
+       if (current_bbt_points == 0 || current_bbt_points->empty()) {
                return 0;
        }
 
-       for (i = current_bbt_points->begin(); i != current_bbt_points->end(); i++) {
-               if ((*i).type == TempoMap::Beat) {
-                       beats++;
-               } else if ((*i).type == TempoMap::Bar) {
-                       bars++;
-               }
-       }
+       i = current_bbt_points->end();
+       i--;
+       bars = (*i).bar - (*current_bbt_points->begin()).bar;
+       beats = current_bbt_points->size() - bars;
+
        /*Only show the bar helper if there aren't many bars on the screen */
        if (bars > 1) {
                bar_helper_on = false;
        }
 
-       for (i = zoomed_bbt_points->begin(); i != zoomed_bbt_points->end(); i++) {
-                       if ((*i).type == TempoMap::Beat) {
-                               zoomed_beats++;
-                       } else if ((*i).type == TempoMap::Bar) {
-                               zoomed_bars++;
-                       }
-               }
-
        if (desirable_marks > (beats / 4)) {
 
                /* we're in beat land...*/
 
+               uint32_t tick = 0;
+               uint32_t skip;
+               uint32_t t;
+               nframes_t frame_skip;
+               double frame_skip_error;
+               double accumulated_error;
                double position_of_helper;
                bool i_am_accented = false;
                bool we_need_ticks = false;
+               bool helper_active = false;
        
                position_of_helper = lower + (30 * Editor::get_current_zoom ());
 
-               if (desirable_marks >= (beats * 2)) {
-                               nmarks = (zoomed_beats * bbt_beat_subdivision) + 1;
+               if (desirable_marks >= (beats)) {
+                       nmarks = (beats * bbt_beat_subdivision) + 1;
                        we_need_ticks = true;
                } else {
-                       nmarks = zoomed_beats + 1;
+                       nmarks = beats + 1;
                }
 
                *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
@@ -1148,26 +1160,26 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper,
                (*marks)[0].position = lower;
                (*marks)[0].style = GtkCustomRulerMarkMicro;
                
-               for (n = 1, i = zoomed_bbt_points->begin(); i != zoomed_bbt_points->end() && n < nmarks; ++i) {
-               
-                       if ((*i).frame <= lower && (bar_helper_on)) {
-                       
+               for (n = 1,   i = current_bbt_points->begin(); n < nmarks && i != current_bbt_points->end(); ++i) {
+
+                       if ((*i).frame < lower && (bar_helper_on)) {
                                        snprintf (buf, sizeof(buf), "<%" PRIu32 "|%" PRIu32, (*i).bar, (*i).beat);
                                        (*marks)[0].label = g_strdup (buf); 
+                                       helper_active = true;
                        } else {
 
-              
                          if ((*i).type == TempoMap::Bar)  {
-                           tick = 0;
-                           (((*i).frame < position_of_helper) && bar_helper_on) ?
-                             snprintf (buf, sizeof(buf), " ") : snprintf (buf, sizeof(buf), "%" PRIu32, (*i).bar);
+                           if (((*i).frame < position_of_helper) && helper_active) {
+                             snprintf (buf, sizeof(buf), " ");
+                           } else {
+                             snprintf (buf, sizeof(buf), "%" PRIu32, (*i).bar);
+                           }
                            (*marks)[n].label = g_strdup (buf);
                            (*marks)[n].position = (*i).frame;
                            (*marks)[n].style = GtkCustomRulerMarkMajor;
                            n++;
                            
                          } else if (((*i).type == TempoMap::Beat) && ((*i).beat > 1)) {
-                           tick = 0;
                            ((((*i).frame < position_of_helper) && bar_helper_on) || !we_need_ticks) ?
                              snprintf (buf, sizeof(buf), " ") : snprintf (buf, sizeof(buf), "%" PRIu32, (*i).beat);
                            if (((*i).beat % 2 == 1) || we_need_ticks) {
@@ -1181,33 +1193,36 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper,
                          }
 
                        }
-                       /* Find the next beat */
-                       
-                       session->bbt_time((*i).frame, next_beat);
 
-                       if (session->tempo_map().meter_at((*i).frame).beats_per_bar() > (next_beat.beats + 1)) {
-                         next_beat.beats += 1;
-                       } else {
-                         next_beat.bars += 1;
-                         next_beat.beats = 1;
-                       }
-               
-                       next_beat_pos = session->tempo_map().frame_time(next_beat);
 
                        /* Add the tick marks */
 
-                       if (we_need_ticks) {
+                       if (we_need_ticks && (*i).type == TempoMap::Beat) {
+
+                               /* Find the next beat */
 
-                               frame_skip = (jack_nframes_t) floor ((session->frame_rate() *  60) / (bbt_beat_subdivision * (*i).tempo->beats_per_minute()));
-                               frame_skip_error =  ((session->frame_rate() *  60.0f) / (bbt_beat_subdivision * (*i).tempo->beats_per_minute())) - frame_skip;
+                               next_beat.beats = (*i).beat;
+                               next_beat.bars = (*i).bar;
+
+                               if ((*i).meter->beats_per_bar() > (next_beat.beats + 1)) {
+                                 next_beat.beats += 1;
+                               } else {
+                                 next_beat.bars += 1;
+                                 next_beat.beats = 1;
+                               }
+                               
+                               next_beat_pos = session->tempo_map().frame_time(next_beat);
+
+                               frame_skip = (nframes_t) floor (frame_skip_error = (session->frame_rate() *  60) / (bbt_beat_subdivision * (*i).tempo->beats_per_minute()));
+                               frame_skip_error -= frame_skip;
                                skip = (uint32_t) (Meter::ticks_per_beat / bbt_beat_subdivision);
 
                                pos = (*i).frame + frame_skip;
                                accumulated_error = frame_skip_error;
 
-                               tick += skip;
+                               tick = skip;
 
-                               for (t = 0; tick < Meter::ticks_per_beat && pos <= next_beat_pos ; pos += frame_skip, tick += skip, ++t) {
+                               for (t = 0; (tick < Meter::ticks_per_beat) && (n < nmarks) && (pos < next_beat_pos) ; pos += frame_skip, tick += skip, ++t) {
 
                                        if (t % magic_accent_number == (magic_accent_number - 1)) {
                                                i_am_accented = true;
@@ -1224,7 +1239,7 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper,
 
                                        (*marks)[n].label = g_strdup (buf);
 
-                                       /* Error compensation for float to jack_nframes_t*/
+                                       /* Error compensation for float to nframes_t*/
                                        accumulated_error += frame_skip_error;
                                        if (accumulated_error > 1) {
                                                pos += 1;
@@ -1244,23 +1259,22 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper,
                                }
                        }
                }
-               delete zoomed_bbt_points;
                return n; //return the actual number of marks made, since we might have skipped some fro fractional time signatures 
 
        } else {
 
                /* we're in bar land */
 
-               if (desirable_marks < (uint32_t) (zoomed_bars / 256)) {
+               if (desirable_marks < (bars / 256)) {
                        nmarks = 1;
                        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
-                       snprintf (buf, sizeof(buf), "too many bars... (currently %" PRIu32 ")", zoomed_bars );
+                       snprintf (buf, sizeof(buf), "too many bars... (currently %" PRIu32 ")", bars );
                        (*marks)[0].style = GtkCustomRulerMarkMajor;
                        (*marks)[0].label = g_strdup (buf);
                        (*marks)[0].position = lower;
-               } else if (desirable_marks < (uint32_t) (nmarks = (gint) (zoomed_bars / 64))) {
+               } else if (desirable_marks < (uint32_t)(nmarks = (gint) (bars / 64) + 1)) {
                        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
-                       for (n = 0, i = zoomed_bbt_points->begin(); i != zoomed_bbt_points->end() && n < nmarks; i++) {
+                       for (n = 0,   i = current_bbt_points->begin(); i != current_bbt_points->end() && n < nmarks; i++) {
                                if ((*i).type == TempoMap::Bar)  {
                                        if ((*i).bar % 64 == 1) {
                                                if ((*i).bar % 256 == 1) {
@@ -1280,9 +1294,9 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper,
                                        }
                                }
                        }
-               } else if (desirable_marks < (uint32_t) (nmarks = (gint)(zoomed_bars / 16))) {
+               } else if (desirable_marks < (uint32_t)(nmarks = (bars / 16) + 1)) {
                        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
-                       for (n = 0, i = zoomed_bbt_points->begin(); i != zoomed_bbt_points->end() && n < nmarks; i++) {
+                       for (n = 0,  i = current_bbt_points->begin(); i != current_bbt_points->end() && n < nmarks; i++) {
                                if ((*i).type == TempoMap::Bar)  {
                                        if ((*i).bar % 16 == 1) {
                                                if ((*i).bar % 64 == 1) {
@@ -1302,9 +1316,9 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper,
                                        }
                                }
                        }
-               } else if (desirable_marks < (uint32_t) (nmarks = (gint)(zoomed_bars / 4))){
+               } else if (desirable_marks < (uint32_t)(nmarks = (bars / 4) + 1)){
                        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
-                       for (n = 0, i = zoomed_bbt_points->begin(); i != zoomed_bbt_points->end() && n < nmarks; ++i) {
+                       for (n = 0,   i = current_bbt_points->begin(); i != current_bbt_points->end() && n < nmarks; ++i) {
                                if ((*i).type == TempoMap::Bar)  {
                                        if ((*i).bar % 4 == 1) {
                                                if ((*i).bar % 16 == 1) {
@@ -1325,9 +1339,9 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper,
                                }
                        }
                } else {
-                       nmarks = zoomed_bars;
-                       *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
-                       for (n = 0, i = zoomed_bbt_points->begin(); i != zoomed_bbt_points->end() && n < nmarks; i++) {
+                       nmarks = bars + 1;
+                       *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks );
+                       for (n = 0,  i = current_bbt_points->begin(); i != current_bbt_points->end() && n < nmarks; i++) {
                                if ((*i).type == TempoMap::Bar)  {
                                        if ((*i).bar % 4 == 1) {
                                                snprintf (buf, sizeof(buf), "%" PRIu32, (*i).bar);
@@ -1346,16 +1360,17 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gulong lower, gulong upper,
                                }
                        }
                }
-               delete zoomed_bbt_points;
-               return nmarks;
+               return n;
        }
 }
 
 gint
-Editor::metric_get_frames (GtkCustomRulerMark **marks, gulong lower, gulong upper, gint maxchars)
+Editor::metric_get_frames (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
-       jack_nframes_t mark_interval;
-       jack_nframes_t pos;
+       nframes_t mark_interval;
+       nframes_t pos;
+       nframes_t ilower = (nframes_t) floor (lower);
+       nframes_t iupper = (nframes_t) floor (upper);
        gchar buf[16];
        gint nmarks;
        gint n;
@@ -1364,7 +1379,7 @@ Editor::metric_get_frames (GtkCustomRulerMark **marks, gulong lower, gulong uppe
                return 0;
        }
 
-       mark_interval = (upper - lower) / 5;
+       mark_interval = (iupper - ilower) / 5;
        if (mark_interval > session->frame_rate()) {
                mark_interval -= mark_interval % session->frame_rate();
        } else {
@@ -1372,7 +1387,7 @@ Editor::metric_get_frames (GtkCustomRulerMark **marks, gulong lower, gulong uppe
        }
        nmarks = 5;
        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
-       for (n = 0, pos = lower; n < nmarks; pos += mark_interval, ++n) {
+       for (n = 0, pos = ilower; n < nmarks; pos += mark_interval, ++n) {
                snprintf (buf, sizeof(buf), "%u", pos);
                (*marks)[n].label = g_strdup (buf);
                (*marks)[n].position = pos;
@@ -1383,15 +1398,15 @@ Editor::metric_get_frames (GtkCustomRulerMark **marks, gulong lower, gulong uppe
 }
 
 static void
-sample_to_clock_parts ( jack_nframes_t sample,
-                       jack_nframes_t sample_rate, 
+sample_to_clock_parts ( nframes_t sample,
+                       nframes_t sample_rate, 
                        long *hrs_p,
                        long *mins_p,
                        long *secs_p,
                        long *millisecs_p)
 
 {
-       jack_nframes_t left;
+       nframes_t left;
        long hrs;
        long mins;
        long secs;
@@ -1415,13 +1430,13 @@ sample_to_clock_parts ( jack_nframes_t sample,
 }
 
 gint
-Editor::metric_get_minsec (GtkCustomRulerMark **marks, gulong lower, gulong upper, gint maxchars)
+Editor::metric_get_minsec (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
-       jack_nframes_t range;
-       jack_nframes_t fr;
-       jack_nframes_t mark_interval;
-       jack_nframes_t pos;
-       jack_nframes_t spacer;
+       nframes_t range;
+       nframes_t fr;
+       nframes_t mark_interval;
+       nframes_t pos;
+       nframes_t spacer;
        long hrs, mins, secs, millisecs;
        gchar buf[16];
        gint nmarks;
@@ -1430,6 +1445,8 @@ Editor::metric_get_minsec (GtkCustomRulerMark **marks, gulong lower, gulong uppe
        bool show_seconds = false;
        bool show_minutes = false;
        bool show_hours = false;
+       nframes_t ilower = (nframes_t) floor (lower);
+       nframes_t iupper = (nframes_t) floor (upper);
 
        if (session == 0) {
                return 0;
@@ -1438,13 +1455,13 @@ Editor::metric_get_minsec (GtkCustomRulerMark **marks, gulong lower, gulong uppe
        fr = session->frame_rate();
 
        /* to prevent 'flashing' */
-       if (lower > (spacer = (jack_nframes_t)(128 * Editor::get_current_zoom ()))) {
+       if (lower > (spacer = (nframes_t)(128 * Editor::get_current_zoom ()))) {
                lower = lower - spacer;
        } else {
-               upper = upper + spacer;
                lower = 0;
        }
-       range = upper - lower;
+       upper = upper + spacer;
+       range = iupper - ilower;
 
        if (range <  (fr / 50)) {
                mark_interval =  fr / 100; /* show 1/100 seconds */
@@ -1510,14 +1527,14 @@ Editor::metric_get_minsec (GtkCustomRulerMark **marks, gulong lower, gulong uppe
                mark_modulo = 2;
         } else {
                                                                                                                    
-                /* not possible if jack_nframes_t is a 32 bit quantity */
+                /* not possible if nframes_t is a 32 bit quantity */
                                                                                                                    
                 mark_interval = 4 * 60 * 60 * fr; /* show 4 hrs */
         }
 
        nmarks = 1 + (range / mark_interval);
        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
-       pos = ((lower + (mark_interval/2))/mark_interval) * mark_interval;
+       pos = ((ilower + (mark_interval/2))/mark_interval) * mark_interval;
        
        if (show_seconds) {
                for (n = 0; n < nmarks; pos += mark_interval, ++n) {