fixes for 98% of all the warnings/errors reported by OS X gcc on tiger
[ardour.git] / gtk2_ardour / editor_rulers.cc
index 873e4919583a893962a43ce9bc46892d16eac154..b7a3f2ba5faa74ad10a484225edacac8392bf228 100644 (file)
 
 */
 
-#define __STDC_FORMAT_MACROS 1
+#ifdef WAF_BUILD
+#include "gtk2ardour-config.h"
+#endif
+
 #include <cstdio> // for sprintf, grrr
 #include <cmath>
 #include <inttypes.h>
@@ -38,6 +41,7 @@
 #include "gui_thread.h"
 #include "time_axis_view.h"
 #include "editor_drag.h"
+#include "editor_cursors.h"
 
 #include "i18n.h"
 
@@ -53,7 +57,7 @@ Editor *Editor::ruler_editor;
 GtkCustomMetric Editor::ruler_metrics[4] = {
        {1, Editor::_metric_get_timecode },
        {1, Editor::_metric_get_bbt },
-       {1, Editor::_metric_get_frames },
+       {1, Editor::_metric_get_samples },
        {1, Editor::_metric_get_minsec }
 };
 
@@ -94,13 +98,13 @@ Editor::initialize_rulers ()
        bbt_ruler->set_no_show_all();
        bbt_nmarks = 0;
 
-       _frames_ruler = gtk_custom_hruler_new ();
-       frames_ruler = Glib::wrap (_frames_ruler);
-       frames_ruler->set_name ("FramesRuler");
-       frames_ruler->set_size_request (-1, (int)timebar_height);
-       gtk_custom_ruler_set_metric (GTK_CUSTOM_RULER(_frames_ruler), &ruler_metrics[ruler_metric_frames]);
-       frames_ruler->hide ();
-       frames_ruler->set_no_show_all();
+       _samples_ruler = gtk_custom_hruler_new ();
+       samples_ruler = Glib::wrap (_samples_ruler);
+       samples_ruler->set_name ("SamplesRuler");
+       samples_ruler->set_size_request (-1, (int) timebar_height);
+       gtk_custom_ruler_set_metric (GTK_CUSTOM_RULER (_samples_ruler), &ruler_metrics[ruler_metric_samples]);
+       samples_ruler->hide ();
+       samples_ruler->set_no_show_all ();
 
        _bbt_ruler = gtk_custom_hruler_new ();
        bbt_ruler = Glib::wrap (_bbt_ruler);
@@ -131,34 +135,34 @@ Editor::initialize_rulers ()
        ruler_children.insert (canvaspos, Element(*minsec_ruler, PACK_SHRINK, PACK_START));
        ruler_lab_children.push_back (Element(timecode_label, PACK_SHRINK, PACK_START));
        ruler_children.insert (canvaspos, Element(*timecode_ruler, PACK_SHRINK, PACK_START));
-       ruler_lab_children.push_back (Element(frame_label, PACK_SHRINK, PACK_START));
-       ruler_children.insert (canvaspos, Element(*frames_ruler, PACK_SHRINK, PACK_START));
+       ruler_lab_children.push_back (Element(samples_label, PACK_SHRINK, PACK_START));
+       ruler_children.insert (canvaspos, Element (*samples_ruler, PACK_SHRINK, PACK_START));
        ruler_lab_children.push_back (Element(bbt_label, PACK_SHRINK, PACK_START));
        ruler_children.insert (canvaspos, Element(*bbt_ruler, PACK_SHRINK, PACK_START));
 
        timecode_ruler->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::SCROLL_MASK);
        bbt_ruler->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::SCROLL_MASK);
-       frames_ruler->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::SCROLL_MASK);
+       samples_ruler->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::SCROLL_MASK);
        minsec_ruler->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::SCROLL_MASK);
 
        timecode_ruler->signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_release));
        bbt_ruler->signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_release));
-       frames_ruler->signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_release));
+       samples_ruler->signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_release));
        minsec_ruler->signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_release));
 
        timecode_ruler->signal_button_press_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_press));
        bbt_ruler->signal_button_press_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_press));
-       frames_ruler->signal_button_press_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_press));
+       samples_ruler->signal_button_press_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_press));
        minsec_ruler->signal_button_press_event().connect (sigc::mem_fun(*this, &Editor::ruler_button_press));
 
        timecode_ruler->signal_motion_notify_event().connect (sigc::mem_fun(*this, &Editor::ruler_mouse_motion));
        bbt_ruler->signal_motion_notify_event().connect (sigc::mem_fun(*this, &Editor::ruler_mouse_motion));
-       frames_ruler->signal_motion_notify_event().connect (sigc::mem_fun(*this, &Editor::ruler_mouse_motion));
+       samples_ruler->signal_motion_notify_event().connect (sigc::mem_fun(*this, &Editor::ruler_mouse_motion));
        minsec_ruler->signal_motion_notify_event().connect (sigc::mem_fun(*this, &Editor::ruler_mouse_motion));
 
        timecode_ruler->signal_scroll_event().connect (sigc::mem_fun(*this, &Editor::ruler_scroll));
        bbt_ruler->signal_scroll_event().connect (sigc::mem_fun(*this, &Editor::ruler_scroll));
-       frames_ruler->signal_scroll_event().connect (sigc::mem_fun(*this, &Editor::ruler_scroll));
+       samples_ruler->signal_scroll_event().connect (sigc::mem_fun(*this, &Editor::ruler_scroll));
        minsec_ruler->signal_scroll_event().connect (sigc::mem_fun(*this, &Editor::ruler_scroll));
 
        visible_timebars = 0; /*this will be changed below */
@@ -168,7 +172,7 @@ Editor::initialize_rulers ()
 bool
 Editor::ruler_scroll (GdkEventScroll* event)
 {
-       nframes64_t xdelta;
+       framepos_t xdelta;
        int direction = event->direction;
        bool handled = false;
 
@@ -195,10 +199,10 @@ Editor::ruler_scroll (GdkEventScroll* event)
 
        case GDK_SCROLL_RIGHT:
                xdelta = (current_page_frames() / 2);
-               if (max_frames - xdelta > leftmost_frame) {
+               if (max_framepos - xdelta > leftmost_frame) {
                        reset_x_origin (leftmost_frame + xdelta);
                } else {
-                       reset_x_origin (max_frames - current_page_frames());
+                       reset_x_origin (max_framepos - current_page_frames());
                }
                handled = true;
                break;
@@ -212,11 +216,11 @@ Editor::ruler_scroll (GdkEventScroll* event)
 }
 
 
-gint
+bool
 Editor::ruler_button_press (GdkEventButton* ev)
 {
        if (_session == 0) {
-               return FALSE;
+               return false;
        }
 
        // jlc: grab ev->window ?
@@ -227,8 +231,8 @@ Editor::ruler_button_press (GdkEventButton* ev)
                grab_widget = timecode_ruler;
        } else if (bbt_ruler->is_realized() && ev->window == bbt_ruler->get_window()->gobj()) {
                grab_widget = bbt_ruler;
-       } else if (frames_ruler->is_realized() && ev->window == frames_ruler->get_window()->gobj()) {
-               grab_widget = frames_ruler;
+       } else if (samples_ruler->is_realized() && ev->window == samples_ruler->get_window()->gobj()) {
+               grab_widget = samples_ruler;
        } else if (minsec_ruler->is_realized() && ev->window == minsec_ruler->get_window()->gobj()) {
                grab_widget = minsec_ruler;
        }
@@ -250,14 +254,14 @@ Editor::ruler_button_press (GdkEventButton* ev)
                _dragging_playhead = true;
        }
 
-       return TRUE;
+       return true;
 }
 
-gint
+bool
 Editor::ruler_button_release (GdkEventButton* ev)
 {
        if (_session == 0) {
-               return FALSE;
+               return false;
        }
 
        gint x,y;
@@ -274,7 +278,7 @@ Editor::ruler_button_release (GdkEventButton* ev)
 
                stop_canvas_autoscroll();
 
-               nframes64_t where = leftmost_frame + pixel_to_frame (x);
+               framepos_t where = leftmost_frame + pixel_to_frame (x);
                snap_to (where);
                popup_ruler_menu (where);
        }
@@ -284,40 +288,40 @@ Editor::ruler_button_release (GdkEventButton* ev)
                ruler_grabbed_widget = 0;
        }
 
-       return TRUE;
+       return true;
 }
 
-gint
+bool
 Editor::ruler_label_button_release (GdkEventButton* ev)
 {
        if (ev->button == 3) {
-               Gtk::Menu* m= dynamic_cast<Gtk::Menu*> (ActionManager::get_widget (X_("/RulerMenuPopup")));
+               Gtk::Menu* m = dynamic_cast<Gtk::Menu*> (ActionManager::get_widget (X_("/RulerMenuPopup")));
                if (m) {
                        m->popup (1, ev->time);
                }
        }
 
-       return TRUE;
+       return true;
 }
 
 
-gint
+bool
 Editor::ruler_mouse_motion (GdkEventMotion* ev)
 {
        if (_session == 0) {
-               return FALSE;
+               return false;
        }
 
        if (_drags->active ()) {
                _drags->motion_handler (reinterpret_cast<GdkEvent*> (ev), false);
        }
 
-       return TRUE;
+       return true;
 }
 
 
 void
-Editor::popup_ruler_menu (nframes64_t where, ItemType t)
+Editor::popup_ruler_menu (framepos_t where, ItemType t)
 {
        using namespace Menu_Helpers;
 
@@ -357,13 +361,11 @@ Editor::popup_ruler_menu (nframes64_t where, ItemType t)
 
        case TempoBarItem:
                ruler_items.push_back (MenuElem (_("New Tempo"), sigc::bind ( sigc::mem_fun(*this, &Editor::mouse_add_new_tempo_event), where)));
-               ruler_items.push_back (MenuElem (_("Clear tempo")));
                ruler_items.push_back (SeparatorElem ());
                break;
 
        case MeterBarItem:
                ruler_items.push_back (MenuElem (_("New Meter"), sigc::bind ( sigc::mem_fun(*this, &Editor::mouse_add_new_meter_event), where)));
-               ruler_items.push_back (MenuElem (_("Clear meter")));
                ruler_items.push_back (SeparatorElem ());
                break;
 
@@ -430,7 +432,7 @@ Editor::store_ruler_visibility ()
 
        node->add_property (X_("timecode"), ruler_timecode_action->get_active() ? "yes": "no");
        node->add_property (X_("bbt"), ruler_bbt_action->get_active() ? "yes": "no");
-       node->add_property (X_("frames"), ruler_samples_action->get_active() ? "yes": "no");
+       node->add_property (X_("samples"), ruler_samples_action->get_active() ? "yes": "no");
        node->add_property (X_("minsec"), ruler_minsec_action->get_active() ? "yes": "no");
        node->add_property (X_("tempo"), ruler_tempo_action->get_active() ? "yes": "no");
        node->add_property (X_("meter"), ruler_meter_action->get_active() ? "yes": "no");
@@ -466,7 +468,7 @@ Editor::restore_ruler_visibility ()
                                ruler_bbt_action->set_active (false);
                        }
                }
-               if ((prop = node->property ("frames")) != 0) {
+               if ((prop = node->property ("samples")) != 0) {
                        if (string_is_affirmative (prop->value())) {
                                ruler_samples_action->set_active (true);
                        } else {
@@ -575,11 +577,11 @@ Editor::update_ruler_visibility ()
 
        if (ruler_samples_action->get_active()) {
                visible_rulers++;
-               frame_label.show ();
-               frames_ruler->show ();
+               samples_label.show ();
+               samples_ruler->show ();
        } else {
-               frame_label.hide ();
-               frames_ruler->hide ();
+               samples_label.hide ();
+               samples_ruler->hide ();
        }
 
        if (ruler_bbt_action->get_active()) {
@@ -761,8 +763,14 @@ Editor::update_ruler_visibility ()
        ruler_label_vbox.set_size_request (-1, (int)(timebar_height * visible_rulers));
        time_canvas_vbox.set_size_request (-1,-1);
 
+       compute_fixed_ruler_scale ();
        update_fixed_rulers();
        redisplay_tempo (false);
+
+       /* Changing ruler visibility means that any lines on markers might need updating */
+       for (LocationMarkerMap::iterator i = location_markers.begin(); i != location_markers.end(); ++i) {
+               i->second->setup_lines ();
+       }
 }
 
 void
@@ -774,7 +782,7 @@ Editor::update_just_timecode ()
                return;
        }
 
-       nframes64_t rightmost_frame = leftmost_frame + current_page_frames();
+       framepos_t rightmost_frame = leftmost_frame + current_page_frames();
 
        if (ruler_timecode_action->get_active()) {
                gtk_custom_ruler_set_range (GTK_CUSTOM_RULER(_timecode_ruler), leftmost_frame, rightmost_frame,
@@ -790,25 +798,29 @@ Editor::compute_fixed_ruler_scale ()
        }
 
        if (ruler_timecode_action->get_active()) {
-               set_timecode_ruler_scale (leftmost_frame, leftmost_frame + current_page_frames() );
+               set_timecode_ruler_scale (leftmost_frame, leftmost_frame + current_page_frames());
        }
 
        if (ruler_minsec_action->get_active()) {
-               set_minsec_ruler_scale (leftmost_frame, leftmost_frame + current_page_frames() );
+               set_minsec_ruler_scale (leftmost_frame, leftmost_frame + current_page_frames());
+       }
+
+       if (ruler_samples_action->get_active()) {
+               set_samples_ruler_scale (leftmost_frame, leftmost_frame + current_page_frames());
        }
 }
 
 void
 Editor::update_fixed_rulers ()
 {
-       nframes64_t rightmost_frame;
+       framepos_t rightmost_frame;
 
        if (_session == 0) {
                return;
        }
 
        ruler_metrics[ruler_metric_timecode].units_per_pixel = frames_per_unit;
-       ruler_metrics[ruler_metric_frames].units_per_pixel = frames_per_unit;
+       ruler_metrics[ruler_metric_samples].units_per_pixel = frames_per_unit;
        ruler_metrics[ruler_metric_minsec].units_per_pixel = frames_per_unit;
 
        rightmost_frame = leftmost_frame + current_page_frames();
@@ -823,7 +835,7 @@ Editor::update_fixed_rulers ()
        }
 
        if (ruler_samples_action->get_active()) {
-               gtk_custom_ruler_set_range (GTK_CUSTOM_RULER(_frames_ruler), leftmost_frame, rightmost_frame,
+               gtk_custom_ruler_set_range (GTK_CUSTOM_RULER (_samples_ruler), leftmost_frame, rightmost_frame,
                                            leftmost_frame, _session->current_end_frame());
        }
 
@@ -863,9 +875,9 @@ Editor::_metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble uppe
 }
 
 gint
-Editor::_metric_get_frames (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
+Editor::_metric_get_samples (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint maxchars)
 {
-       return ruler_editor->metric_get_frames (marks, lower, upper, maxchars);
+       return ruler_editor->metric_get_samples (marks, lower, upper, maxchars);
 }
 
 gint
@@ -875,11 +887,10 @@ Editor::_metric_get_minsec (GtkCustomRulerMark **marks, gdouble lower, gdouble u
 }
 
 void
-Editor::set_timecode_ruler_scale (gdouble lower, gdouble upper)
+Editor::set_timecode_ruler_scale (framepos_t lower, framepos_t upper)
 {
-       nframes64_t range;
-       nframes64_t spacer;
-       nframes64_t fr;
+       framepos_t spacer;
+       framepos_t fr;
 
        if (_session == 0) {
                return;
@@ -887,13 +898,13 @@ Editor::set_timecode_ruler_scale (gdouble lower, gdouble upper)
 
        fr = _session->frame_rate();
 
-       if (lower > (spacer = (nframes64_t)(128 * Editor::get_current_zoom ()))) {
+       if (lower > (spacer = (framepos_t) (128 * Editor::get_current_zoom ()))) {
                lower = lower - spacer;
        } else {
                lower = 0;
        }
        upper = upper + spacer;
-       range = (nframes64_t) floor (upper - lower);
+       framecnt_t const range = upper - lower;
 
        if (range < (2 * _session->frames_per_timecode_frame())) { /* 0 - 2 frames */
                timecode_ruler_scale = timecode_show_bits;
@@ -902,19 +913,19 @@ Editor::set_timecode_ruler_scale (gdouble lower, gdouble upper)
        } else if (range <= (fr / 4)) { /* 2 frames - 0.250 second */
                timecode_ruler_scale = timecode_show_frames;
                timecode_mark_modulo = 1;
-               timecode_nmarks = 2 + (range / (nframes64_t)_session->frames_per_timecode_frame());
+               timecode_nmarks = 2 + (range / (framepos_t)_session->frames_per_timecode_frame());
        } else if (range <= (fr / 2)) { /* 0.25-0.5 second */
                timecode_ruler_scale = timecode_show_frames;
                timecode_mark_modulo = 2;
-               timecode_nmarks = 2 + (range / (nframes64_t)_session->frames_per_timecode_frame());
+               timecode_nmarks = 2 + (range / (framepos_t)_session->frames_per_timecode_frame());
        } else if (range <= fr) { /* 0.5-1 second */
                timecode_ruler_scale = timecode_show_frames;
                timecode_mark_modulo = 5;
-               timecode_nmarks = 2 + (range / (nframes64_t)_session->frames_per_timecode_frame());
+               timecode_nmarks = 2 + (range / (framepos_t)_session->frames_per_timecode_frame());
        } else if (range <= 2 * fr) { /* 1-2 seconds */
                timecode_ruler_scale = timecode_show_frames;
                timecode_mark_modulo = 10;
-               timecode_nmarks = 2 + (range / (nframes64_t)_session->frames_per_timecode_frame());
+               timecode_nmarks = 2 + (range / (framepos_t)_session->frames_per_timecode_frame());
        } else if (range <= 8 * fr) { /* 2-8 seconds */
                timecode_ruler_scale = timecode_show_seconds;
                timecode_mark_modulo = 1;
@@ -965,7 +976,7 @@ Editor::set_timecode_ruler_scale (gdouble lower, gdouble upper)
                timecode_nmarks = 2 + 24;
        } else {
 
-               /* not possible if nframes64_t is a 32 bit quantity */
+               /* not possible if framepos_t is a 32 bit quantity */
 
                timecode_ruler_scale = timecode_show_hours;
                timecode_mark_modulo = 4;
@@ -977,8 +988,8 @@ Editor::set_timecode_ruler_scale (gdouble lower, gdouble upper)
 gint
 Editor::metric_get_timecode (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upper*/, gint /*maxchars*/)
 {
-       nframes_t pos;
-       nframes64_t spacer;
+       framepos_t pos;
+       framecnt_t spacer;
        Timecode::Time timecode;
        gchar buf[16];
        gint n;
@@ -987,13 +998,13 @@ Editor::metric_get_timecode (GtkCustomRulerMark **marks, gdouble lower, gdouble
                return 0;
        }
 
-       if (lower > (spacer = (nframes64_t)(128 * Editor::get_current_zoom ()))) {
+       if (lower > (spacer = (framecnt_t)(128 * Editor::get_current_zoom ()))) {
                lower = lower - spacer;
        } else {
                lower = 0;
        }
 
-       pos = (nframes_t) floor (lower);
+       pos = (framecnt_t) floor (lower);
 
        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * timecode_nmarks);
        switch (timecode_ruler_scale) {
@@ -1133,22 +1144,22 @@ Editor::metric_get_timecode (GtkCustomRulerMark **marks, gdouble lower, gdouble
 
 
 void
-Editor::compute_bbt_ruler_scale (nframes64_t lower, nframes64_t upper)
+Editor::compute_bbt_ruler_scale (framepos_t lower, framepos_t upper)
 {
-        if (_session == 0) {
-                return;
-        }
+       if (_session == 0) {
+               return;
+       }
        TempoMap::BBTPointList::iterator i;
-        BBT_Time lower_beat, upper_beat; // the beats at each end of the ruler
+       Timecode::BBT_Time lower_beat, upper_beat; // the beats at each end of the ruler
 
-        _session->bbt_time((jack_nframes_t) lower, lower_beat);
-        _session->bbt_time((jack_nframes_t) upper, upper_beat);
-        uint32_t beats = 0;
+       _session->bbt_time (lower, lower_beat);
+       _session->bbt_time (upper, upper_beat);
+       uint32_t beats = 0;
 
        bbt_accent_modulo = 1;
        bbt_bar_helper_on = false;
-        bbt_bars = 0;
-        bbt_nmarks = 1;
+       bbt_bars = 0;
+       bbt_nmarks = 1;
 
        bbt_ruler_scale =  bbt_over;
 
@@ -1194,6 +1205,10 @@ Editor::compute_bbt_ruler_scale (nframes64_t lower, nframes64_t upper)
                 bbt_beat_subdivision = 16;
                bbt_accent_modulo = 4;
                break;
+       case SnapToBeatDiv20:
+                bbt_beat_subdivision = 20;
+               bbt_accent_modulo = 5;
+               break;
        case SnapToBeatDiv24:
                 bbt_beat_subdivision = 24;
                bbt_accent_modulo = 6;
@@ -1218,36 +1233,36 @@ Editor::compute_bbt_ruler_scale (nframes64_t lower, nframes64_t upper)
        i = current_bbt_points->end();
        i--;
        if ((*i).beat >= (*current_bbt_points->begin()).beat) {
-         bbt_bars = (*i).bar - (*current_bbt_points->begin()).bar;
+               bbt_bars = (*i).bar - (*current_bbt_points->begin()).bar;
        } else {
-         bbt_bars = (*i).bar - (*current_bbt_points->begin()).bar - 1;
+               bbt_bars = (*i).bar - (*current_bbt_points->begin()).bar - 1;
        }
        beats = current_bbt_points->size() - bbt_bars;
 
-       /*Only show the bar helper if there aren't many bars on the screen */
+       /* Only show the bar helper if there aren't many bars on the screen */
        if ((bbt_bars < 2) || (beats < 5)) {
                bbt_bar_helper_on = true;
        }
 
        if (bbt_bars > 8192) {
-         bbt_ruler_scale =  bbt_over;
+               bbt_ruler_scale =  bbt_over;
        } else if (bbt_bars > 1024) {
-         bbt_ruler_scale = bbt_show_64;
+               bbt_ruler_scale = bbt_show_64;
        } else if (bbt_bars > 256) {
-         bbt_ruler_scale = bbt_show_16;
+               bbt_ruler_scale = bbt_show_16;
        } else if (bbt_bars > 64) {
-         bbt_ruler_scale = bbt_show_4;
+               bbt_ruler_scale = bbt_show_4;
        } else if (bbt_bars > 10) {
-         bbt_ruler_scale =  bbt_show_1;
+               bbt_ruler_scale =  bbt_show_1;
        } else if (bbt_bars > 2) {
-         bbt_ruler_scale =  bbt_show_beats;
+               bbt_ruler_scale =  bbt_show_beats;
        } else  if (bbt_bars > 0) {
-         bbt_ruler_scale =  bbt_show_ticks;
+               bbt_ruler_scale =  bbt_show_ticks;
        } else {
-         bbt_ruler_scale =  bbt_show_ticks_detail;
+               bbt_ruler_scale =  bbt_show_ticks_detail;
        }
 
-       if ((bbt_ruler_scale == bbt_show_ticks_detail) && (lower_beat.beats == upper_beat.beats) && (upper_beat.ticks - lower_beat.ticks <= Meter::ticks_per_beat / 4)) {
+       if ((bbt_ruler_scale == bbt_show_ticks_detail) && (lower_beat.beats == upper_beat.beats) && (upper_beat.ticks - lower_beat.ticks <= Timecode::BBT_Time::ticks_per_beat / 4)) {
                bbt_ruler_scale =  bbt_show_ticks_super_detail;
        }
 }
@@ -1255,23 +1270,23 @@ Editor::compute_bbt_ruler_scale (nframes64_t lower, nframes64_t upper)
 gint
 Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upper*/, gint /*maxchars*/)
 {
-        if (_session == 0) {
-                return 0;
-        }
+       if (_session == 0) {
+               return 0;
+       }
 
        TempoMap::BBTPointList::iterator i;
 
-        char buf[64];
-        gint  n = 0;
-       nframes64_t pos;
-       BBT_Time next_beat;
-       nframes64_t next_beat_pos;
-        uint32_t beats = 0;
+       char buf[64];
+       gint  n = 0;
+       framepos_t pos;
+       Timecode::BBT_Time next_beat;
+       framepos_t next_beat_pos;
+       uint32_t beats = 0;
 
        uint32_t tick = 0;
        uint32_t skip;
        uint32_t t;
-       nframes64_t frame_skip;
+       framepos_t frame_skip;
        double frame_skip_error;
        double bbt_position_of_helper;
        double accumulated_error;
@@ -1374,16 +1389,16 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
 
                        next_beat_pos = _session->tempo_map().frame_time(next_beat);
 
-                       frame_skip = (nframes64_t) floor (frame_skip_error = (_session->frame_rate() *  60) / (bbt_beat_subdivision * (*i).tempo->beats_per_minute()));
+                       frame_skip = (framepos_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);
+                       skip = (uint32_t) (Timecode::BBT_Time::ticks_per_beat / bbt_beat_subdivision);
 
                        pos = (*i).frame + frame_skip;
                        accumulated_error = frame_skip_error;
 
                        tick = skip;
 
-                       for (t = 0; (tick < Meter::ticks_per_beat) && (n < bbt_nmarks) && (pos < next_beat_pos) ; pos += frame_skip, tick += skip, ++t) {
+                       for (t = 0; (tick < Timecode::BBT_Time::ticks_per_beat) && (n < bbt_nmarks) && (pos < next_beat_pos) ; pos += frame_skip, tick += skip, ++t) {
 
                                if (t % bbt_accent_modulo == (bbt_accent_modulo - 1)) {
                                        i_am_accented = true;
@@ -1392,7 +1407,7 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
                                snprintf (buf, sizeof(buf), " ");
                                (*marks)[n].label = g_strdup (buf);
 
-                               /* Error compensation for float to nframes64_t*/
+                               /* Error compensation for float to framepos_t*/
                                accumulated_error += frame_skip_error;
                                if (accumulated_error > 1) {
                                        pos += 1;
@@ -1466,16 +1481,16 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
 
                        next_beat_pos = _session->tempo_map().frame_time(next_beat);
 
-                       frame_skip = (nframes64_t) floor (frame_skip_error = (_session->frame_rate() *  60) / (bbt_beat_subdivision * (*i).tempo->beats_per_minute()));
+                       frame_skip = (framepos_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);
+                       skip = (uint32_t) (Timecode::BBT_Time::ticks_per_beat / bbt_beat_subdivision);
 
                        pos = (*i).frame + frame_skip;
                        accumulated_error = frame_skip_error;
 
                        tick = skip;
 
-                       for (t = 0; (tick < Meter::ticks_per_beat) && (n < bbt_nmarks) && (pos < next_beat_pos) ; pos += frame_skip, tick += skip, ++t) {
+                       for (t = 0; (tick < Timecode::BBT_Time::ticks_per_beat) && (n < bbt_nmarks) && (pos < next_beat_pos) ; pos += frame_skip, tick += skip, ++t) {
 
                                if (t % bbt_accent_modulo == (bbt_accent_modulo - 1)) {
                                        i_am_accented = true;
@@ -1489,7 +1504,7 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
 
                                (*marks)[n].label = g_strdup (buf);
 
-                               /* Error compensation for float to nframes64_t*/
+                               /* Error compensation for float to framepos_t*/
                                accumulated_error += frame_skip_error;
                                if (accumulated_error > 1) {
                                        pos += 1;
@@ -1563,16 +1578,16 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
 
                        next_beat_pos = _session->tempo_map().frame_time(next_beat);
 
-                       frame_skip = (nframes64_t) floor (frame_skip_error = (_session->frame_rate() *  60) / (bbt_beat_subdivision * (*i).tempo->beats_per_minute()));
+                       frame_skip = (framepos_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);
+                       skip = (uint32_t) (Timecode::BBT_Time::ticks_per_beat / bbt_beat_subdivision);
 
                        pos = (*i).frame + frame_skip;
                        accumulated_error = frame_skip_error;
 
                        tick = skip;
 
-                       for (t = 0; (tick < Meter::ticks_per_beat) && (n < bbt_nmarks) && (pos < next_beat_pos) ; pos += frame_skip, tick += skip, ++t) {
+                       for (t = 0; (tick < Timecode::BBT_Time::ticks_per_beat) && (n < bbt_nmarks) && (pos < next_beat_pos) ; pos += frame_skip, tick += skip, ++t) {
 
                                  if (t % bbt_accent_modulo == (bbt_accent_modulo - 1)) {
                                          i_am_accented = true;
@@ -1586,7 +1601,7 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
 
                                  (*marks)[n].label = g_strdup (buf);
 
-                                 /* Error compensation for float to nframes64_t*/
+                                 /* Error compensation for float to framepos_t*/
                                  accumulated_error += frame_skip_error;
                                  if (accumulated_error > 1) {
                                          pos += 1;
@@ -1724,13 +1739,17 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
 
 }
 
+void
+Editor::set_samples_ruler_scale (framepos_t lower, framepos_t upper)
+{
+       _samples_ruler_interval = (upper - lower) / 5;
+}
+
 gint
-Editor::metric_get_frames (GtkCustomRulerMark **marks, gdouble lower, gdouble upper, gint /*maxchars*/)
+Editor::metric_get_samples (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upper*/, gint /*maxchars*/)
 {
-       nframes64_t mark_interval;
-       nframes64_t pos;
-       nframes64_t ilower = (nframes64_t) floor (lower);
-       nframes64_t iupper = (nframes64_t) floor (upper);
+       framepos_t pos;
+       framepos_t const ilower = (framepos_t) floor (lower);
        gchar buf[16];
        gint nmarks;
        gint n;
@@ -1739,15 +1758,9 @@ Editor::metric_get_frames (GtkCustomRulerMark **marks, gdouble lower, gdouble up
                return 0;
        }
 
-       mark_interval = (iupper - ilower) / 5;
-       if (mark_interval > _session->frame_rate()) {
-               mark_interval -= mark_interval % _session->frame_rate();
-       } else {
-               mark_interval = _session->frame_rate() / (_session->frame_rate() / mark_interval ) ;
-       }
        nmarks = 5;
        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * nmarks);
-       for (n = 0, pos = ilower; n < nmarks; pos += mark_interval, ++n) {
+       for (n = 0, pos = ilower; n < nmarks; pos += _samples_ruler_interval, ++n) {
                snprintf (buf, sizeof(buf), "%" PRIi64, pos);
                (*marks)[n].label = g_strdup (buf);
                (*marks)[n].position = pos;
@@ -1758,15 +1771,15 @@ Editor::metric_get_frames (GtkCustomRulerMark **marks, gdouble lower, gdouble up
 }
 
 static void
-sample_to_clock_parts ( nframes64_t sample,
-                       nframes64_t sample_rate,
+sample_to_clock_parts ( framepos_t sample,
+                       framepos_t sample_rate,
                        long *hrs_p,
                        long *mins_p,
                        long *secs_p,
                        long *millisecs_p)
 
 {
-       nframes64_t left;
+       framepos_t left;
        long hrs;
        long mins;
        long secs;
@@ -1790,11 +1803,10 @@ sample_to_clock_parts ( nframes64_t sample,
 }
 
 void
-Editor::set_minsec_ruler_scale (gdouble lower, gdouble upper)
+Editor::set_minsec_ruler_scale (framepos_t lower, framepos_t upper)
 {
-       nframes64_t range;
-       nframes64_t fr;
-       nframes64_t spacer;
+       framepos_t fr;
+       framepos_t spacer;
 
        if (_session == 0) {
                return;
@@ -1803,13 +1815,13 @@ Editor::set_minsec_ruler_scale (gdouble lower, gdouble upper)
        fr = _session->frame_rate();
 
        /* to prevent 'flashing' */
-       if (lower > (spacer = (nframes64_t)(128 * Editor::get_current_zoom ()))) {
+       if (lower > (spacer = (framepos_t)(128 * Editor::get_current_zoom ()))) {
                lower -= spacer;
        } else {
                lower = 0;
        }
        upper += spacer;
-       range = (nframes64_t) (upper - lower);
+       framecnt_t const range = upper - lower;
 
        if (range <  (fr / 50)) {
                minsec_mark_interval =  fr / 1000; /* show 1/1000 seconds */
@@ -1881,7 +1893,7 @@ Editor::set_minsec_ruler_scale (gdouble lower, gdouble upper)
                minsec_mark_modulo = 2;
         } else {
 
-                /* not possible if nframes64_t is a 32 bit quantity */
+                /* not possible if framepos_t is a 32 bit quantity */
 
                 minsec_mark_interval = 4 * 60 * 60 * fr; /* show 4 hrs */
         }
@@ -1891,8 +1903,8 @@ Editor::set_minsec_ruler_scale (gdouble lower, gdouble upper)
 gint
 Editor::metric_get_minsec (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upper*/, gint /*maxchars*/)
 {
-       nframes64_t pos;
-       nframes64_t spacer;
+       framepos_t pos;
+       framepos_t spacer;
        long hrs, mins, secs, millisecs;
        gchar buf[16];
        gint n;
@@ -1902,14 +1914,14 @@ Editor::metric_get_minsec (GtkCustomRulerMark **marks, gdouble lower, gdouble /*
        }
 
        /* to prevent 'flashing' */
-       if (lower > (spacer = (nframes64_t)(128 * Editor::get_current_zoom ()))) {
+       if (lower > (spacer = (framepos_t) (128 * Editor::get_current_zoom ()))) {
                lower = lower - spacer;
        } else {
                lower = 0;
        }
 
        *marks = (GtkCustomRulerMark *) g_malloc (sizeof(GtkCustomRulerMark) * minsec_nmarks);
-       pos = ((((nframes64_t) floor(lower)) + (minsec_mark_interval/2))/minsec_mark_interval) * minsec_mark_interval;
+       pos = ((((framepos_t) floor(lower)) + (minsec_mark_interval/2))/minsec_mark_interval) * minsec_mark_interval;
        switch (minsec_ruler_scale) {
        case minsec_show_seconds:
                for (n = 0; n < minsec_nmarks; pos += minsec_mark_interval, ++n) {