introduce the notion that note additions and property changes can cause the removal...
[ardour.git] / gtk2_ardour / audio_clock.cc
index 347a3f68483c827ea10b8445c8f19d44a830c4e3..be1bb97713b763e5d9e97cc5facda2a2e7b4ac90 100644 (file)
 #include "audio_clock.h"
 #include "utils.h"
 #include "keyboard.h"
+#include "gui_thread.h"
 #include "i18n.h"
 
 using namespace ARDOUR;
 using namespace PBD;
-using namespace sigc;
 using namespace Gtk;
 using namespace std;
 
@@ -65,9 +65,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),
@@ -82,7 +81,6 @@ AudioClock::AudioClock (
          b2 ("|"),
          last_when(0)
 {
-       session = 0;
        last_when = 0;
        last_pdelta = 0;
        last_sdelta = 0;
@@ -217,7 +215,7 @@ 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));
+       Session::TimecodeOffsetChanged.connect (_session_connections, invalidator (*this), boost::bind (&AudioClock::timecode_offset_changed, this), gui_context());
 
        if (editable) {
                setup_events ();
@@ -420,7 +418,7 @@ void
 AudioClock::set (nframes_t when, bool force, nframes_t offset, char which)
 {
 
-       if ((!force && !is_visible()) || session == 0) {
+       if ((!force && !is_visible()) || _session == 0) {
                return;
        }
 
@@ -502,7 +500,7 @@ AudioClock::set_frames (nframes_t when, bool /*force*/)
        audio_frames_label.set_text (buf);
 
        if (frames_upper_info_label) {
-               nframes_t rate = session->frame_rate();
+               nframes_t rate = _session->frame_rate();
 
                if (fmod (rate, 1000.0) == 0.000) {
                        sprintf (buf, "%uK", rate/1000);
@@ -514,7 +512,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")) {
@@ -539,11 +537,11 @@ AudioClock::set_minsec (nframes_t when, bool force)
        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 -= (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();
 
        if (force || hrs != ms_last_hrs) {
                sprintf (buf, "%02d", hrs);
@@ -571,9 +569,9 @@ AudioClock::set_timecode (nframes_t when, bool force)
        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) {
@@ -606,7 +604,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));
@@ -619,7 +617,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");
@@ -648,12 +646,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);
@@ -678,7 +676,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) {
@@ -694,12 +692,12 @@ AudioClock::set_bbt (nframes_t when, bool force)
 void
 AudioClock::set_session (Session *s)
 {
-       session = s;
+       SessionHandlePtr::set_session (s);
 
-       if (s) {
+       if (_session) {
 
                XMLProperty* prop;
-               XMLNode* node = session->extra_xml (X_("ClockModes"));
+               XMLNode* node = _session->extra_xml (X_("ClockModes"));
                AudioClock::Mode amode;
 
                if (node) {
@@ -1191,7 +1189,7 @@ 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;
        }
 
@@ -1237,7 +1235,7 @@ 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;
        }
 
@@ -1284,7 +1282,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;
        }
 
@@ -1343,16 +1341,16 @@ AudioClock::get_frames (Field field,nframes_t pos,int dir)
        BBT_Time bbt;
        switch (field) {
        case Timecode_Hours:
-               frames = (nframes_t) floor (3600.0 * session->frame_rate());
+               frames = (nframes_t) floor (3600.0 * _session->frame_rate());
                break;
        case Timecode_Minutes:
-               frames = (nframes_t) floor (60.0 * session->frame_rate());
+               frames = (nframes_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 = (nframes_t) floor (_session->frame_rate() / _session->timecode_frames_per_second());
                break;
 
        case AudioFrames:
@@ -1360,32 +1358,32 @@ AudioClock::get_frames (Field field,nframes_t pos,int dir)
                break;
 
        case MS_Hours:
-               frames = (nframes_t) floor (3600.0 * session->frame_rate());
+               frames = (nframes_t) floor (3600.0 * _session->frame_rate());
                break;
        case MS_Minutes:
-               frames = (nframes_t) floor (60.0 * session->frame_rate());
+               frames = (nframes_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;
        }
 
@@ -1460,7 +1458,7 @@ AudioClock::timecode_sanitize_display()
                seconds_label.set_text("59");
        }
 
-       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");
@@ -1480,7 +1478,7 @@ AudioClock::timecode_sanitize_display()
                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");
                }
@@ -1490,7 +1488,7 @@ AudioClock::timecode_sanitize_display()
 nframes_t
 AudioClock::timecode_frame_from_display () const
 {
-       if (session == 0) {
+       if (_session == 0) {
                return 0;
        }
 
@@ -1501,10 +1499,10 @@ AudioClock::timecode_frame_from_display () const
        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
@@ -1523,14 +1521,14 @@ AudioClock::timecode_frame_from_display () const
        Timecode::Time timecode2;
        nframes_t sample_increment;
 
-       sample_increment = (long)rint(session->frame_rate() / session->timecode_frames_per_second);
+       sample_increment = (long)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;
@@ -1550,7 +1548,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_increment( timecode1 );
+               _session->timecode_increment( timecode1 );
        }
 
        cout << "sample_increment: " << sample_increment << endl;
@@ -1569,13 +1567,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;
@@ -1600,7 +1598,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_increment( timecode1 );
+               _session->timecode_increment( timecode1 );
        }
 
        cout << "sample_increment: " << sample_increment << endl;
@@ -1612,13 +1610,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;
@@ -1643,7 +1641,7 @@ AudioClock::timecode_frame_from_display () const
                        break;
                }
                oldsample = sample1;
-               session->timecode_decrement( timecode1 );
+               _session->timecode_decrement( timecode1 );
        }
 
        cout << "sample_decrement: " << sample_increment << endl;
@@ -1664,14 +1662,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;
@@ -1691,7 +1689,7 @@ AudioClock::timecode_frame_from_display () const
                                break;
                        }
                        oldsample = sample1;
-                       session->timecode_increment( timecode1 );
+                       _session->timecode_increment( timecode1 );
                }
 
                cout << "sample_increment: " << sample_increment << endl;
@@ -1699,8 +1697,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;
@@ -1720,7 +1718,7 @@ AudioClock::timecode_frame_from_display () const
                                break;
                        }
                        oldsample = sample1;
-                       session->timecode_decrement( timecode1 );
+                       _session->timecode_decrement( timecode1 );
                }
 
                cout << "sample_decrement: " << sample_increment << endl;
@@ -1740,15 +1738,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;
@@ -1770,7 +1768,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;
@@ -1789,15 +1787,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;
@@ -1819,7 +1817,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;
@@ -1837,15 +1835,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;
@@ -1867,7 +1865,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;
@@ -1883,7 +1881,7 @@ AudioClock::timecode_frame_from_display () const
 nframes_t
 AudioClock::minsec_frame_from_display () const
 {
-       if (session == 0) {
+       if (_session == 0) {
                return 0;
        }
 
@@ -1891,7 +1889,7 @@ 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();
+       nframes_t sr = _session->frame_rate();
 
        return (nframes_t) floor ((hrs * 60.0f * 60.0f * sr) + (mins * 60.0f * sr) + (secs * sr));
 }
@@ -1899,7 +1897,7 @@ AudioClock::minsec_frame_from_display () const
 nframes_t
 AudioClock::bbt_frame_from_display (nframes_t pos) const
 {
-       if (session == 0) {
+       if (_session == 0) {
                error << "AudioClock::current_time() called with BBT mode but without session!" << endmsg;
                return 0;
        }
@@ -1916,7 +1914,7 @@ AudioClock::bbt_frame_from_display (nframes_t pos) const
                any.bbt.beats++;
        }
 
-       nframes_t ret = session->convert_to_frames_at (pos, any);
+       nframes_t ret = _session->convert_to_frames_at (pos, any);
 
        return ret;
 }
@@ -1925,7 +1923,7 @@ AudioClock::bbt_frame_from_display (nframes_t pos) const
 nframes_t
 AudioClock::bbt_frame_duration_from_display (nframes_t pos) const
 {
-       if (session == 0) {
+       if (_session == 0) {
                error << "AudioClock::current_time() called with BBT mode but without session!" << endmsg;
                return 0;
        }
@@ -1937,7 +1935,7 @@ AudioClock::bbt_frame_duration_from_display (nframes_t pos) const
        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
@@ -1972,21 +1970,21 @@ AudioClock::build_ops_menu ()
 void
 AudioClock::set_from_playhead ()
 {
-       if (!session) {
+       if (!_session) {
                return;
        }
        
-       set (session->transport_frame());
+       set (_session->transport_frame());
 }
 
 void
 AudioClock::locate ()
 {
-       if (!session || is_duration) {
+       if (!_session || is_duration) {
                return;
        }
        
-       session->request_locate (current_time(), false);
+       _session->request_locate (current_time(), false);
 }
 
 void
@@ -2046,30 +2044,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:
@@ -2084,3 +2082,47 @@ AudioClock::set_bbt_reference (nframes64_t pos)
 {
        bbt_reference_time = pos;
 }
+
+void
+AudioClock::on_style_changed (const Glib::RefPtr<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 ();
+}