Put some up/down buttons to the right of the summary. Might help with #3786.
[ardour.git] / gtk2_ardour / audio_clock.cc
index 68df057b953ed903779c35a373e236f01b92b3df..6e65e512b204b2f5c4d1b5032efa3c0723a0bdae 100644 (file)
@@ -23,6 +23,7 @@
 #include "pbd/convert.h"
 #include "pbd/enumwriter.h"
 
+#include <gtkmm/style.h>
 #include <gtkmm2ext/utils.h>
 
 #include "ardour/ardour.h"
@@ -35,6 +36,7 @@
 #include "audio_clock.h"
 #include "utils.h"
 #include "keyboard.h"
+#include "gui_thread.h"
 #include "i18n.h"
 
 using namespace ARDOUR;
@@ -64,9 +66,8 @@ const uint32_t AudioClock::field_length[(int) AudioClock::AudioFrames+1] = {
        10   /* Audio Frame */
 };
 
-AudioClock::AudioClock (
-       std::string clock_name, bool transient, std::string widget_name, bool allow_edit, bool follows_playhead, bool duration, bool with_info
-       )
+AudioClock::AudioClock (const string& clock_name, bool transient, const string& widget_name, 
+                       bool allow_edit, bool follows_playhead, bool duration, bool with_info)
        : _name (clock_name),
          is_transient (transient),
          is_duration (duration),
@@ -79,9 +80,10 @@ AudioClock::AudioClock (
          colon5 (":"),
          b1 ("|"),
          b2 ("|"),
-         last_when(0)
+         last_when(0),
+         _canonical_time_is_displayed (true),
+         _canonical_time (0)
 {
-       session = 0;
        last_when = 0;
        last_pdelta = 0;
        last_sdelta = 0;
@@ -216,8 +218,6 @@ AudioClock::AudioClock (
        clock_base.add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK|Gdk::SCROLL_MASK);
        clock_base.signal_button_release_event().connect (sigc::bind (sigc::mem_fun (*this, &AudioClock::field_button_release_event), Timecode_Hours));
 
-       Session::TimecodeOffsetChanged.connect (sigc::mem_fun (*this, &AudioClock::timecode_offset_changed));
-
        if (editable) {
                setup_events ();
        }
@@ -416,19 +416,14 @@ AudioClock::on_realize ()
 }
 
 void
-AudioClock::set (nframes_t when, bool force, nframes_t offset, char which)
+AudioClock::set (framepos_t when, bool force, framecnt_t offset, char which)
 {
-
-       if ((!force && !is_visible()) || session == 0) {
-               return;
-       }
-
-       if (when == last_when && !offset && !force) {
+       if ((!force && !is_visible()) || _session == 0) {
                return;
        }
 
-       bool pdelta = Config->get_primary_clock_delta_edit_cursor();
-       bool sdelta = Config->get_secondary_clock_delta_edit_cursor();
+       bool const pdelta = Config->get_primary_clock_delta_edit_cursor ();
+       bool const sdelta = Config->get_secondary_clock_delta_edit_cursor ();
 
        if (offset && which == 'p' && pdelta) {
                when = (when > offset) ? when - offset : offset - when;
@@ -436,13 +431,17 @@ AudioClock::set (nframes_t when, bool force, nframes_t offset, char which)
                when = (when > offset) ? when - offset : offset - when;
        }
 
+       if (when == last_when && !force) {
+               return;
+       }
+
        if (which == 'p' && pdelta && !last_pdelta) {
                set_widget_name("TransportClockDisplayDelta");
                last_pdelta = true;
        } else if (which == 'p' && !pdelta && last_pdelta) {
                set_widget_name("TransportClockDisplay");
                last_pdelta = false;
-       } else if (which == 's'  && sdelta && !last_sdelta) {
+       } else if (which == 's' && sdelta && !last_sdelta) {
                set_widget_name("SecondaryClockDisplayDelta");
                last_sdelta = true;
        } else if (which == 's' && !sdelta && last_sdelta) {
@@ -472,17 +471,25 @@ AudioClock::set (nframes_t when, bool force, nframes_t offset, char which)
        }
 
        last_when = when;
+
+       /* we're setting the time from a frames value, so keep it as the canonical value */
+       _canonical_time = when;
+       _canonical_time_is_displayed = false;
 }
 
 void
-AudioClock::timecode_offset_changed ()
+AudioClock::session_configuration_changed (std::string p)
 {
-       nframes_t current;
+       if (p != "timecode-offset" && p != "timecode-offset-negative") {
+               return;
+       }
+       
+       framecnt_t current;
 
        switch (_mode) {
        case Timecode:
                if (is_duration) {
-                       current = current_duration();
+                       current = current_duration ();
                } else {
                        current = current_time ();
                }
@@ -494,17 +501,17 @@ AudioClock::timecode_offset_changed ()
 }
 
 void
-AudioClock::set_frames (nframes_t when, bool /*force*/)
+AudioClock::set_frames (framepos_t when, bool /*force*/)
 {
        char buf[32];
-       snprintf (buf, sizeof (buf), "%u", when);
+       snprintf (buf, sizeof (buf), "%" PRId64, when);
        audio_frames_label.set_text (buf);
 
        if (frames_upper_info_label) {
-               nframes_t rate = session->frame_rate();
+               framecnt_t rate = _session->frame_rate();
 
                if (fmod (rate, 1000.0) == 0.000) {
-                       sprintf (buf, "%uK", rate/1000);
+                       sprintf (buf, "%" PRId64 "K", rate/1000);
                } else {
                        sprintf (buf, "%.3fK", rate/1000.0f);
                }
@@ -513,7 +520,7 @@ AudioClock::set_frames (nframes_t when, bool /*force*/)
                        frames_upper_info_label->set_text (buf);
                }
 
-               float vid_pullup = session->config.get_video_pullup();
+               float vid_pullup = _session->config.get_video_pullup();
 
                if (vid_pullup == 0.0) {
                        if (frames_lower_info_label->get_text () != _("none")) {
@@ -529,20 +536,20 @@ AudioClock::set_frames (nframes_t when, bool /*force*/)
 }
 
 void
-AudioClock::set_minsec (nframes_t when, bool force)
+AudioClock::set_minsec (framepos_t when, bool force)
 {
        char buf[32];
-       nframes_t left;
+       framecnt_t left;
        int hrs;
        int mins;
        float secs;
 
        left = when;
-       hrs = (int) floor (left / (session->frame_rate() * 60.0f * 60.0f));
-       left -= (nframes_t) floor (hrs * session->frame_rate() * 60.0f * 60.0f);
-       mins = (int) floor (left / (session->frame_rate() * 60.0f));
-       left -= (nframes_t) floor (mins * session->frame_rate() * 60.0f);
-       secs = left / (float) session->frame_rate();
+       hrs = (int) floor (left / (_session->frame_rate() * 60.0f * 60.0f));
+       left -= (framecnt_t) floor (hrs * _session->frame_rate() * 60.0f * 60.0f);
+       mins = (int) floor (left / (_session->frame_rate() * 60.0f));
+       left -= (framecnt_t) floor (mins * _session->frame_rate() * 60.0f);
+       secs = left / (float) _session->frame_rate();
 
        if (force || hrs != ms_last_hrs) {
                sprintf (buf, "%02d", hrs);
@@ -564,15 +571,15 @@ AudioClock::set_minsec (nframes_t when, bool force)
 }
 
 void
-AudioClock::set_timecode (nframes_t when, bool force)
+AudioClock::set_timecode (framepos_t when, bool force)
 {
        char buf[32];
        Timecode::Time timecode;
 
        if (is_duration) {
-               session->timecode_duration (when, timecode);
+               _session->timecode_duration (when, timecode);
        } else {
-               session->timecode_time (when, timecode);
+               _session->timecode_time (when, timecode);
        }
 
        if (force || timecode.hours != last_hrs || timecode.negative != last_negative) {
@@ -605,7 +612,7 @@ AudioClock::set_timecode (nframes_t when, bool force)
        }
 
        if (timecode_upper_info_label) {
-               double timecode_frames = session->timecode_frames_per_second();
+               double timecode_frames = _session->timecode_frames_per_second();
 
                if ( fmod(timecode_frames, 1.0) == 0.0) {
                        sprintf (buf, "%u", int (timecode_frames));
@@ -618,7 +625,7 @@ AudioClock::set_timecode (nframes_t when, bool force)
                }
 
                if ((fabs(timecode_frames - 29.97) < 0.0001) || timecode_frames == 30) {
-                       if (session->timecode_drop_frames()) {
+                       if (_session->timecode_drop_frames()) {
                                sprintf (buf, "DF");
                        } else {
                                sprintf (buf, "NDF");
@@ -635,10 +642,10 @@ AudioClock::set_timecode (nframes_t when, bool force)
 }
 
 void
-AudioClock::set_bbt (nframes_t when, bool force)
+AudioClock::set_bbt (framepos_t when, bool force)
 {
        char buf[16];
-       BBT_Time bbt;
+       Timecode::BBT_Time bbt;
 
        /* handle a common case */
        if (is_duration) {
@@ -647,12 +654,12 @@ AudioClock::set_bbt (nframes_t when, bool force)
                        bbt.beats = 0;
                        bbt.ticks = 0;
                } else {
-                       session->tempo_map().bbt_time (when, bbt);
+                       _session->tempo_map().bbt_time (when, bbt);
                        bbt.bars--;
                        bbt.beats--;
                }
        } else {
-               session->tempo_map().bbt_time (when, bbt);
+               _session->tempo_map().bbt_time (when, bbt);
        }
 
        sprintf (buf, "%03" PRIu32, bbt.bars);
@@ -669,7 +676,7 @@ AudioClock::set_bbt (nframes_t when, bool force)
        }
 
        if (bbt_upper_info_label) {
-               nframes64_t pos;
+               framepos_t pos;
 
                if (bbt_reference_time < 0) {
                        pos = when;
@@ -677,7 +684,7 @@ AudioClock::set_bbt (nframes_t when, bool force)
                        pos = bbt_reference_time;
                }
 
-               TempoMetric m (session->tempo_map().metric_at (pos));
+               TempoMetric m (_session->tempo_map().metric_at (pos));
 
                sprintf (buf, "%-5.2f", m.tempo().beats_per_minute());
                if (bbt_lower_info_label->get_text() != buf) {
@@ -693,12 +700,14 @@ AudioClock::set_bbt (nframes_t when, bool force)
 void
 AudioClock::set_session (Session *s)
 {
-       session = s;
+       SessionHandlePtr::set_session (s);
 
-       if (s) {
+       if (_session) {
+
+               _session->config.ParameterChanged.connect (_session_connections, invalidator (*this), boost::bind (&AudioClock::session_configuration_changed, this, _1), gui_context());
 
                XMLProperty* prop;
-               XMLNode* node = session->extra_xml (X_("ClockModes"));
+               XMLNode* node = _session->extra_xml (X_("ClockModes"));
                AudioClock::Mode amode;
 
                if (node) {
@@ -887,6 +896,7 @@ AudioClock::field_key_release_event (GdkEventKey *ev, Field field)
 
                new_text += new_char;
                label->set_text (new_text);
+               _canonical_time_is_displayed = true;
                key_entry_state++;
        }
 
@@ -912,10 +922,12 @@ AudioClock::field_key_release_event (GdkEventKey *ev, Field field)
                                // Bars should never be, unless this clock is for a duration
                                if (atoi(bars_label.get_text()) == 0 && !is_duration) {
                                        bars_label.set_text("001");
+                                       _canonical_time_is_displayed = true;
                                }
                                //  beats should never be 0, unless this clock is for a duration
                                if (atoi(beats_label.get_text()) == 0 && !is_duration) {
                                        beats_label.set_text("01");
+                                       _canonical_time_is_displayed = true;
                                }
                                break;
                        default:
@@ -1130,7 +1142,6 @@ AudioClock::field_button_release_event (GdkEventButton *ev, Field field)
        }
 
        if (Keyboard::is_context_menu_event (ev)) {
-               cerr << "Context menu event on clock\n";
                if (ops_menu == 0) {
                        build_ops_menu ();
                }
@@ -1190,11 +1201,11 @@ AudioClock::field_button_release_event (GdkEventButton *ev, Field field)
 bool
 AudioClock::field_button_press_event (GdkEventButton *ev, Field /*field*/)
 {
-       if (session == 0) {
+       if (_session == 0) {
                return false;
        }
 
-       nframes_t frames = 0;
+       framepos_t frames = 0;
 
        switch (ev->button) {
        case 1:
@@ -1203,7 +1214,7 @@ AudioClock::field_button_press_event (GdkEventButton *ev, Field /*field*/)
                        ValueChanged (); /* EMIT_SIGNAL */
                                        }
 
-                /* make absolutely sure that the pointer is grabbed */
+               /* make absolutely sure that the pointer is grabbed */
                gdk_pointer_grab(ev->window,false ,
                                 GdkEventMask( Gdk::POINTER_MOTION_MASK | Gdk::BUTTON_PRESS_MASK |Gdk::BUTTON_RELEASE_MASK),
                                 NULL,NULL,ev->time);
@@ -1236,11 +1247,11 @@ AudioClock::field_button_press_event (GdkEventButton *ev, Field /*field*/)
 bool
 AudioClock::field_button_scroll_event (GdkEventScroll *ev, Field field)
 {
-       if (session == 0) {
+       if (_session == 0) {
                return false;
        }
 
-       nframes_t frames = 0;
+       framepos_t frames = 0;
 
        switch (ev->direction) {
 
@@ -1283,7 +1294,7 @@ AudioClock::field_button_scroll_event (GdkEventScroll *ev, Field field)
 bool
 AudioClock::field_motion_notify_event (GdkEventMotion *ev, Field field)
 {
-       if (session == 0 || !dragging) {
+       if (_session == 0 || !dragging) {
                return false;
        }
 
@@ -1309,8 +1320,8 @@ AudioClock::field_motion_notify_event (GdkEventMotion *ev, Field field)
 
        if (trunc(drag_accum) != 0) {
 
-               nframes_t frames;
-               nframes_t pos ;
+               framepos_t frames;
+               framepos_t pos;
                int dir;
                dir = (drag_accum < 0 ? 1:-1);
                pos = current_time();
@@ -1318,7 +1329,7 @@ AudioClock::field_motion_notify_event (GdkEventMotion *ev, Field field)
 
                if (frames  != 0 &&  frames * drag_accum < current_time()) {
 
-                       set ((nframes_t) floor (pos - drag_accum * frames), false); // minus because up is negative in computer-land
+                       set ((framepos_t) floor (pos - drag_accum * frames), false); // minus because up is negative in computer-land
 
                } else {
                        set (0 , false);
@@ -1334,24 +1345,23 @@ AudioClock::field_motion_notify_event (GdkEventMotion *ev, Field field)
        return true;
 }
 
-nframes_t
-AudioClock::get_frames (Field field,nframes_t pos,int dir)
+framepos_t
+AudioClock::get_frames (Field field, framepos_t pos, int dir)
 {
-
-       nframes_t frames = 0;
-       BBT_Time bbt;
+       framecnt_t frames = 0;
+       Timecode::BBT_Time bbt;
        switch (field) {
        case Timecode_Hours:
-               frames = (nframes_t) floor (3600.0 * session->frame_rate());
+               frames = (framecnt_t) floor (3600.0 * _session->frame_rate());
                break;
        case Timecode_Minutes:
-               frames = (nframes_t) floor (60.0 * session->frame_rate());
+               frames = (framecnt_t) floor (60.0 * _session->frame_rate());
                break;
        case Timecode_Seconds:
-               frames = session->frame_rate();
+               frames = _session->frame_rate();
                break;
        case Timecode_Frames:
-               frames = (nframes_t) floor (session->frame_rate() / session->timecode_frames_per_second());
+               frames = (framecnt_t) floor (_session->frame_rate() / _session->timecode_frames_per_second());
                break;
 
        case AudioFrames:
@@ -1359,42 +1369,46 @@ AudioClock::get_frames (Field field,nframes_t pos,int dir)
                break;
 
        case MS_Hours:
-               frames = (nframes_t) floor (3600.0 * session->frame_rate());
+               frames = (framecnt_t) floor (3600.0 * _session->frame_rate());
                break;
        case MS_Minutes:
-               frames = (nframes_t) floor (60.0 * session->frame_rate());
+               frames = (framecnt_t) floor (60.0 * _session->frame_rate());
                break;
        case MS_Seconds:
-               frames = session->frame_rate();
+               frames = _session->frame_rate();
                break;
 
        case Bars:
                bbt.bars = 1;
                bbt.beats = 0;
                bbt.ticks = 0;
-               frames = session->tempo_map().bbt_duration_at(pos,bbt,dir);
+               frames = _session->tempo_map().bbt_duration_at(pos,bbt,dir);
                break;
        case Beats:
                bbt.bars = 0;
                bbt.beats = 1;
                bbt.ticks = 0;
-               frames = session->tempo_map().bbt_duration_at(pos,bbt,dir);
+               frames = _session->tempo_map().bbt_duration_at(pos,bbt,dir);
                break;
        case Ticks:
                bbt.bars = 0;
                bbt.beats = 0;
                bbt.ticks = 1;
-               frames = session->tempo_map().bbt_duration_at(pos,bbt,dir);
+               frames = _session->tempo_map().bbt_duration_at(pos,bbt,dir);
                break;
        }
 
        return frames;
 }
 
-nframes_t
-AudioClock::current_time (nframes_t pos) const
+framepos_t
+AudioClock::current_time (framepos_t pos) const
 {
-       nframes_t ret = 0;
+       if (!_canonical_time_is_displayed) {
+               return _canonical_time;
+       }
+       
+       framepos_t ret = 0;
 
        switch (_mode) {
        case Timecode:
@@ -1419,10 +1433,10 @@ AudioClock::current_time (nframes_t pos) const
        return ret;
 }
 
-nframes_t
-AudioClock::current_duration (nframes_t pos) const
+framepos_t
+AudioClock::current_duration (framepos_t pos) const
 {
-       nframes_t ret = 0;
+       framepos_t ret = 0;
 
        switch (_mode) {
        case Timecode:
@@ -1453,57 +1467,63 @@ AudioClock::timecode_sanitize_display()
        // Check Timecode fields for sanity, possibly adjusting values
        if (atoi(minutes_label.get_text()) > 59) {
                minutes_label.set_text("59");
+               _canonical_time_is_displayed = true;
        }
 
        if (atoi(seconds_label.get_text()) > 59) {
                seconds_label.set_text("59");
+               _canonical_time_is_displayed = true;
        }
 
-       switch ((long)rint(session->timecode_frames_per_second())) {
+       switch ((long)rint(_session->timecode_frames_per_second())) {
        case 24:
                if (atoi(frames_label.get_text()) > 23) {
                        frames_label.set_text("23");
+                       _canonical_time_is_displayed = true;
                }
                break;
        case 25:
                if (atoi(frames_label.get_text()) > 24) {
                        frames_label.set_text("24");
+                       _canonical_time_is_displayed = true;
                }
                break;
        case 30:
                if (atoi(frames_label.get_text()) > 29) {
                        frames_label.set_text("29");
+                       _canonical_time_is_displayed = true;
                }
                break;
        default:
                break;
        }
 
-       if (session->timecode_drop_frames()) {
+       if (_session->timecode_drop_frames()) {
                if ((atoi(minutes_label.get_text()) % 10) && (atoi(seconds_label.get_text()) == 0) && (atoi(frames_label.get_text()) < 2)) {
                        frames_label.set_text("02");
+                       _canonical_time_is_displayed = true;
                }
        }
 }
 
-nframes_t
+framepos_t
 AudioClock::timecode_frame_from_display () const
 {
-       if (session == 0) {
+       if (_session == 0) {
                return 0;
        }
 
        Timecode::Time timecode;
-       nframes_t sample;
+       framepos_t sample;
 
        timecode.hours = atoi (hours_label.get_text());
        timecode.minutes = atoi (minutes_label.get_text());
        timecode.seconds = atoi (seconds_label.get_text());
        timecode.frames = atoi (frames_label.get_text());
-       timecode.rate = session->timecode_frames_per_second();
-       timecode.drop= session->timecode_drop_frames();
+       timecode.rate = _session->timecode_frames_per_second();
+       timecode.drop= _session->timecode_drop_frames();
 
-       session->timecode_to_sample( timecode, sample, false /* use_offset */, false /* use_subframes */ );
+       _session->timecode_to_sample( timecode, sample, false /* use_offset */, false /* use_subframes */ );
 
 
 #if 0
@@ -1517,19 +1537,19 @@ AudioClock::timecode_frame_from_display () const
 
        // Testcode for timecode<->sample conversions (P.S.)
        Timecode::Time timecode1;
-       nframes_t sample1;
-       nframes_t oldsample = 0;
+       framepos_t sample1;
+       framepos_t oldsample = 0;
        Timecode::Time timecode2;
-       nframes_t sample_increment;
+       framecnt_t sample_increment;
 
-       sample_increment = (long)rint(session->frame_rate() / session->timecode_frames_per_second);
+       sample_increment = (framecnt_t)rint(_session->frame_rate() / _session->timecode_frames_per_second);
 
 #ifdef Timecode_SAMPLE_TEST_1
        // Test 1: use_offset = false, use_subframes = false
        cout << "use_offset = false, use_subframes = false" << endl;
        for (int i = 0; i < 108003; i++) {
-               session->timecode_to_sample( timecode1, sample1, false /* use_offset */, false /* use_subframes */ );
-               session->sample_to_timecode( sample1, timecode2, false /* use_offset */, false /* use_subframes */ );
+               _session->timecode_to_sample( timecode1, sample1, false /* use_offset */, false /* use_subframes */ );
+               _session->sample_to_timecode( sample1, timecode2, false /* use_offset */, false /* use_subframes */ );
 
                if ((i > 0) && ( ((sample1 - oldsample) != sample_increment) && ((sample1 - oldsample) != (sample_increment + 1)) && ((sample1 - oldsample) != (sample_increment - 1)))) {
                        cout << "ERROR: sample increment not right: " << (sample1 - oldsample) << " != " << sample_increment << endl;
@@ -1549,7 +1569,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_increment( timecode1 );
+               _session->timecode_increment( timecode1 );
        }
 
        cout << "sample_increment: " << sample_increment << endl;
@@ -1568,13 +1588,13 @@ AudioClock::timecode_frame_from_display () const
        timecode1.subframes = 0;
        sample1 = oldsample = 0;
 
-       session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
+       _session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
        cout << "Starting at sample: " << sample1 << " -> ";
        cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << endl;
 
        for (int i = 0; i < 108003; i++) {
-               session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
-               session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
+               _session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
+               _session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
 
 //     cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << " -> ";
 //     cout << "sample: " << sample1 << endl;
@@ -1599,7 +1619,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_increment( timecode1 );
+               _session->timecode_increment( timecode1 );
        }
 
        cout << "sample_increment: " << sample_increment << endl;
@@ -1611,13 +1631,13 @@ AudioClock::timecode_frame_from_display () const
        // Test 3: use_offset = true, use_subframes = false, decrement
        cout << "use_offset = true, use_subframes = false, decrement" << endl;
 
-       session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
+       _session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
        cout << "Starting at sample: " << sample1 << " -> ";
        cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << endl;
 
        for (int i = 0; i < 108003; i++) {
-               session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
-               session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
+               _session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
+               _session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
 
 //     cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << " -> ";
 //     cout << "sample: " << sample1 << endl;
@@ -1642,7 +1662,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_decrement( timecode1 );
+               _session->timecode_decrement( timecode1 );
        }
 
        cout << "sample_decrement: " << sample_increment << endl;
@@ -1663,14 +1683,14 @@ AudioClock::timecode_frame_from_display () const
                timecode1.subframes = 0;
                sample1 = oldsample = (sample_increment * sub) / 80;
 
-               session->sample_to_timecode( sample1, timecode1, true /* use_offset */, true /* use_subframes */ );
+               _session->sample_to_timecode( sample1, timecode1, true /* use_offset */, true /* use_subframes */ );
 
                cout << "starting at sample: " << sample1 << " -> ";
                cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << endl;
 
                for (int i = 0; i < 108003; i++) {
-                       session->timecode_to_sample( timecode1, sample1, true /* use_offset */, true /* use_subframes */ );
-                       session->sample_to_timecode( sample1, timecode2, true /* use_offset */, true /* use_subframes */ );
+                       _session->timecode_to_sample( timecode1, sample1, true /* use_offset */, true /* use_subframes */ );
+                       _session->sample_to_timecode( sample1, timecode2, true /* use_offset */, true /* use_subframes */ );
 
                        if ((i > 0) && ( ((sample1 - oldsample) != sample_increment) && ((sample1 - oldsample) != (sample_increment + 1)) && ((sample1 - oldsample) != (sample_increment - 1)))) {
                                cout << "ERROR: sample increment not right: " << (sample1 - oldsample) << " != " << sample_increment << endl;
@@ -1690,7 +1710,7 @@ AudioClock::timecode_frame_from_display () const
                                break;
                        }
                        oldsample = sample1;
-                       session->timecode_increment( timecode1 );
+                       _session->timecode_increment( timecode1 );
                }
 
                cout << "sample_increment: " << sample_increment << endl;
@@ -1698,8 +1718,8 @@ AudioClock::timecode_frame_from_display () const
                cout << "timecode: " << (timecode2.negative ? "-" : "") << timecode2.hours << ":" << timecode2.minutes << ":" << timecode2.seconds << ":" << timecode2.frames << "::" << timecode2.subframes << endl;
 
                for (int i = 0; i < 108003; i++) {
-                       session->timecode_to_sample( timecode1, sample1, true /* use_offset */, true /* use_subframes */ );
-                       session->sample_to_timecode( sample1, timecode2, true /* use_offset */, true /* use_subframes */ );
+                       _session->timecode_to_sample( timecode1, sample1, true /* use_offset */, true /* use_subframes */ );
+                       _session->sample_to_timecode( sample1, timecode2, true /* use_offset */, true /* use_subframes */ );
 
                        if ((i > 0) && ( ((oldsample - sample1) != sample_increment) && ((oldsample - sample1) != (sample_increment + 1)) && ((oldsample - sample1) != (sample_increment - 1)))) {
                                cout << "ERROR: sample increment not right: " << (oldsample - sample1) << " != " << sample_increment << endl;
@@ -1719,7 +1739,7 @@ AudioClock::timecode_frame_from_display () const
                                break;
                        }
                        oldsample = sample1;
-                       session->timecode_decrement( timecode1 );
+                       _session->timecode_decrement( timecode1 );
                }
 
                cout << "sample_decrement: " << sample_increment << endl;
@@ -1739,15 +1759,15 @@ AudioClock::timecode_frame_from_display () const
        timecode1.frames = 0;
        timecode1.subframes = 0;
        sample1 = oldsample = 0;
-       sample_increment = session->frame_rate();
+       sample_increment = _session->frame_rate();
 
-       session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
+       _session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
        cout << "Starting at sample: " << sample1 << " -> ";
        cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << endl;
 
        for (int i = 0; i < 3600; i++) {
-               session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
-               session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
+               _session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
+               _session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
 
 //     cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << " -> ";
 //     cout << "sample: " << sample1 << endl;
@@ -1769,7 +1789,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_increment_seconds( timecode1 );
+               _session->timecode_increment_seconds( timecode1 );
        }
 
        cout << "sample_increment: " << sample_increment << endl;
@@ -1788,15 +1808,15 @@ AudioClock::timecode_frame_from_display () const
        timecode1.frames = 0;
        timecode1.subframes = 0;
        sample1 = oldsample = 0;
-       sample_increment = session->frame_rate() * 60;
+       sample_increment = _session->frame_rate() * 60;
 
-       session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
+       _session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
        cout << "Starting at sample: " << sample1 << " -> ";
        cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << endl;
 
        for (int i = 0; i < 60; i++) {
-               session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
-               session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
+               _session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
+               _session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
 
 //     cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << " -> ";
 //     cout << "sample: " << sample1 << endl;
@@ -1818,7 +1838,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_increment_minutes( timecode1 );
+               _session->timecode_increment_minutes( timecode1 );
        }
 
        cout << "sample_increment: " << sample_increment << endl;
@@ -1836,15 +1856,15 @@ AudioClock::timecode_frame_from_display () const
        timecode1.frames = 0;
        timecode1.subframes = 0;
        sample1 = oldsample = 0;
-       sample_increment = session->frame_rate() * 60 * 60;
+       sample_increment = _session->frame_rate() * 60 * 60;
 
-       session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
+       _session->sample_to_timecode( sample1, timecode1, true /* use_offset */, false /* use_subframes */ );
        cout << "Starting at sample: " << sample1 << " -> ";
        cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << endl;
 
        for (int i = 0; i < 10; i++) {
-               session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
-               session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
+               _session->timecode_to_sample( timecode1, sample1, true /* use_offset */, false /* use_subframes */ );
+               _session->sample_to_timecode( sample1, timecode2, true /* use_offset */, false /* use_subframes */ );
 
 //     cout << "timecode: " << (timecode1.negative ? "-" : "") << timecode1.hours << ":" << timecode1.minutes << ":" << timecode1.seconds << ":" << timecode1.frames << "::" << timecode1.subframes << " -> ";
 //     cout << "sample: " << sample1 << endl;
@@ -1866,7 +1886,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_increment_hours( timecode1 );
+               _session->timecode_increment_hours( timecode1 );
        }
 
        cout << "sample_increment: " << sample_increment << endl;
@@ -1879,10 +1899,10 @@ AudioClock::timecode_frame_from_display () const
        return sample;
 }
 
-nframes_t
+framepos_t
 AudioClock::minsec_frame_from_display () const
 {
-       if (session == 0) {
+       if (_session == 0) {
                return 0;
        }
 
@@ -1890,15 +1910,15 @@ AudioClock::minsec_frame_from_display () const
        int mins = atoi (ms_minutes_label.get_text());
        float secs = atof (ms_seconds_label.get_text());
 
-       nframes_t sr = session->frame_rate();
+       framecnt_t sr = _session->frame_rate();
 
-       return (nframes_t) floor ((hrs * 60.0f * 60.0f * sr) + (mins * 60.0f * sr) + (secs * sr));
+       return (framepos_t) floor ((hrs * 60.0f * 60.0f * sr) + (mins * 60.0f * sr) + (secs * sr));
 }
 
-nframes_t
-AudioClock::bbt_frame_from_display (nframes_t pos) const
+framepos_t
+AudioClock::bbt_frame_from_display (framepos_t pos) const
 {
-       if (session == 0) {
+       if (_session == 0) {
                error << "AudioClock::current_time() called with BBT mode but without session!" << endmsg;
                return 0;
        }
@@ -1910,39 +1930,38 @@ AudioClock::bbt_frame_from_display (nframes_t pos) const
        any.bbt.beats = atoi (beats_label.get_text());
        any.bbt.ticks = atoi (ticks_label.get_text());
 
-       if (is_duration) {
-               any.bbt.bars++;
-               any.bbt.beats++;
-       }
-
-       nframes_t ret = session->convert_to_frames_at (pos, any);
-
-       return ret;
+       if (is_duration) {
+               any.bbt.bars++;
+               any.bbt.beats++;
+                return _session->any_duration_to_frames (pos, any);
+       } else {
+                return _session->convert_to_frames (any);
+        }
 }
 
 
-nframes_t
-AudioClock::bbt_frame_duration_from_display (nframes_t pos) const
+framepos_t
+AudioClock::bbt_frame_duration_from_display (framepos_t pos) const
 {
-       if (session == 0) {
+       if (_session == 0) {
                error << "AudioClock::current_time() called with BBT mode but without session!" << endmsg;
                return 0;
        }
 
-       BBT_Time bbt;
+       Timecode::BBT_Time bbt;
 
 
        bbt.bars = atoi (bars_label.get_text());
        bbt.beats = atoi (beats_label.get_text());
        bbt.ticks = atoi (ticks_label.get_text());
 
-       return session->tempo_map().bbt_duration_at(pos,bbt,1);
+       return _session->tempo_map().bbt_duration_at(pos,bbt,1);
 }
 
-nframes_t
+framepos_t
 AudioClock::audio_frame_from_display () const
 {
-       return (nframes_t) atoi (audio_frames_label.get_text());
+       return (framepos_t) atoi (audio_frames_label.get_text());
 }
 
 void
@@ -1964,28 +1983,29 @@ AudioClock::build_ops_menu ()
        if (editable && !is_duration && !_follows_playhead) {
                ops_items.push_back (SeparatorElem());
                ops_items.push_back (MenuElem (_("Set From Playhead"), sigc::mem_fun(*this, &AudioClock::set_from_playhead)));
-               ops_items.push_back (MenuElem (_("Locate to this time"), sigc::mem_fun(*this, &AudioClock::locate)));
+               ops_items.push_back (MenuElem (_("Locate to This Time"), sigc::mem_fun(*this, &AudioClock::locate)));
        }
 }
 
 void
 AudioClock::set_from_playhead ()
 {
-       if (!session) {
+       if (!_session) {
                return;
        }
-       
-       set (session->transport_frame());
+
+       set (_session->transport_frame());
+       ValueChanged ();
 }
 
 void
 AudioClock::locate ()
 {
-       if (!session || is_duration) {
+       if (!_session || is_duration) {
                return;
        }
        
-       session->request_locate (current_time(), false);
+       _session->request_locate (current_time(), _session->transport_rolling ());
 }
 
 void
@@ -2045,30 +2065,30 @@ AudioClock::set_mode (Mode m)
 void
 AudioClock::set_size_requests ()
 {
-       /* note that in some fonts, "88" is narrower than "00", hence the 2 pixel padding */
+       /* note that in some fonts, "88" is narrower than "00" */
 
        switch (_mode) {
        case Timecode:
-               Gtkmm2ext::set_size_request_to_display_given_text (hours_label, "-00", 5, 5);
-               Gtkmm2ext::set_size_request_to_display_given_text (minutes_label, "00", 5, 5);
-               Gtkmm2ext::set_size_request_to_display_given_text (seconds_label, "00", 5, 5);
-               Gtkmm2ext::set_size_request_to_display_given_text (frames_label, "00", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (hours_label, "-88", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (minutes_label, "88", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (seconds_label, "88", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (frames_label, "88", 5, 5);
                break;
 
        case BBT:
-               Gtkmm2ext::set_size_request_to_display_given_text (bars_label, "-000", 5, 5);
-               Gtkmm2ext::set_size_request_to_display_given_text (beats_label, "00", 5, 5);
-               Gtkmm2ext::set_size_request_to_display_given_text (ticks_label, "0000", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (bars_label, "-888", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (beats_label, "88", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (ticks_label, "8888", 5, 5);
                break;
 
        case MinSec:
-               Gtkmm2ext::set_size_request_to_display_given_text (ms_hours_label, "00", 5, 5);
-               Gtkmm2ext::set_size_request_to_display_given_text (ms_minutes_label, "00", 5, 5);
-               Gtkmm2ext::set_size_request_to_display_given_text (ms_seconds_label, "00.000", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (ms_hours_label, "88", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (ms_minutes_label, "88", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (ms_seconds_label, "88.888", 5, 5);
                break;
 
        case Frames:
-               Gtkmm2ext::set_size_request_to_display_given_text (audio_frames_label, "0000000000", 5, 5);
+               Gtkmm2ext::set_size_request_to_display_given_text (audio_frames_label, "8888888888", 5, 5);
                break;
 
        case Off:
@@ -2079,7 +2099,62 @@ AudioClock::set_size_requests ()
 }
 
 void
-AudioClock::set_bbt_reference (nframes64_t pos)
+AudioClock::set_bbt_reference (framepos_t pos)
 {
        bbt_reference_time = pos;
 }
+
+void
+AudioClock::on_style_changed (const Glib::RefPtr<Gtk::Style>& old_style)
+{
+       HBox::on_style_changed (old_style);
+
+       /* propagate style changes to all component widgets that should inherit the main one */
+
+       Glib::RefPtr<RcStyle> rcstyle = get_modifier_style();
+
+       clock_base.modify_style (rcstyle);
+       audio_frames_label.modify_style (rcstyle);
+       hours_label.modify_style (rcstyle);
+       minutes_label.modify_style (rcstyle);
+       seconds_label.modify_style (rcstyle);
+       frames_label.modify_style (rcstyle);
+       bars_label.modify_style (rcstyle);
+       beats_label.modify_style (rcstyle);
+       ticks_label.modify_style (rcstyle);
+       ms_hours_label.modify_style (rcstyle);
+       ms_minutes_label.modify_style (rcstyle);
+       ms_seconds_label.modify_style (rcstyle);
+       hours_ebox.modify_style (rcstyle);
+       minutes_ebox.modify_style (rcstyle);
+       seconds_ebox.modify_style (rcstyle);
+       frames_ebox.modify_style (rcstyle);
+       audio_frames_ebox.modify_style (rcstyle);
+       bars_ebox.modify_style (rcstyle);
+       beats_ebox.modify_style (rcstyle);
+       ticks_ebox.modify_style (rcstyle);
+       ms_hours_ebox.modify_style (rcstyle);
+       ms_minutes_ebox.modify_style (rcstyle);
+       ms_seconds_ebox.modify_style (rcstyle);
+
+       colon1.modify_style (rcstyle);
+       colon2.modify_style (rcstyle);
+       colon3.modify_style (rcstyle);
+       colon4.modify_style (rcstyle);
+       colon5.modify_style (rcstyle);
+       b1.modify_style (rcstyle);
+       b2.modify_style (rcstyle);
+
+       set_size_requests ();
+}
+
+void
+AudioClock::set_is_duration (bool yn)
+{
+       if (yn == is_duration) {
+               return;
+       }
+        
+       is_duration = yn;
+       set (last_when, true, 0, 's');
+}