remove debugging message
[ardour.git] / gtk2_ardour / audio_region_view.cc
index 915fa05750987be93a563c36364bb266d34445ee..ac0d317a13a14e5f3fe2186555c487c3ccb46020 100644 (file)
@@ -15,7 +15,6 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <cmath>
@@ -31,6 +30,7 @@
 #include <ardour/audiosource.h>
 #include <ardour/audio_diskstream.h>
 #include <pbd/memento_command.h>
+#include <pbd/stacktrace.h>
 
 #include "streamview.h"
 #include "audio_region_view.h"
@@ -57,7 +57,7 @@ using namespace ArdourCanvas;
 
 static const int32_t sync_mark_width = 9;
 
-AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView &tv, AudioRegion& r, double spu,
+AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView &tv, boost::shared_ptr<AudioRegion> r, double spu,
                                  Gdk::Color& basic_color)
        : RegionView (parent, tv, r, spu, basic_color)
        , sync_mark(0)
@@ -73,7 +73,7 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
 {
 }
 
-AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView &tv, AudioRegion& r, double spu, 
+AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView &tv, boost::shared_ptr<AudioRegion> r, double spu, 
                                  Gdk::Color& basic_color, TimeAxisViewItem::Visibility visibility)
        : RegionView (parent, tv, r, spu, basic_color, visibility)
        , sync_mark(0)
@@ -89,21 +89,44 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
 {
 }
 
+
+AudioRegionView::AudioRegionView (const AudioRegionView& other)
+       : RegionView (other)
+       , zero_line(0)
+       , fade_in_shape(0)
+       , fade_out_shape(0)
+       , fade_in_handle(0)
+       , fade_out_handle(0)
+       , gain_line(0)
+       , _amplitude_above_axis(1.0)
+       , _flags(0)
+       , fade_color(0)
+
+{
+       Gdk::Color c;
+       int r,g,b,a;
+
+       UINT_TO_RGBA (other.fill_color, &r, &g, &b, &a);
+       c.set_rgb_p (r/255.0, g/255.0, b/255.0);
+       
+       init (c, false);
+}
+
 void
 AudioRegionView::init (Gdk::Color& basic_color, bool wfd)
 {
        // FIXME: Some redundancy here with RegionView::init.  Need to figure out
        // where order is important and where it isn't...
        
-       RegionView::init(basic_color, wfd);
-
+       RegionView::init(basic_color, false);
+       
        XMLNode *node;
 
        _amplitude_above_axis = 1.0;
        zero_line             = 0;
        _flags                = 0;
 
-       if ((node = _region.extra_xml ("GUI")) != 0) {
+       if ((node = _region->extra_xml ("GUI")) != 0) {
                set_flags (node);
        } else {
                _flags = WaveformVisible;
@@ -149,11 +172,11 @@ AudioRegionView::init (Gdk::Color& basic_color, bool wfd)
                fade_out_handle->set_data ("regionview", this);
        }
 
-       string foo = _region.name();
+       string foo = _region->name();
        foo += ':';
        foo += "gain";
 
-       gain_line = new AudioRegionGainLine (foo, trackview.session(), *this, *group, audio_region().envelope());
+       gain_line = new AudioRegionGainLine (foo, trackview.session(), *this, *group, audio_region()->envelope());
 
        if (!(_flags & EnvelopeVisible)) {
                gain_line->hide ();
@@ -161,8 +184,6 @@ AudioRegionView::init (Gdk::Color& basic_color, bool wfd)
                gain_line->show ();
        }
 
-       reset_width_dependent_items ((double) _region.length() / samples_per_unit);
-
        gain_line->reset ();
 
        set_height (trackview.height);
@@ -176,7 +197,7 @@ AudioRegionView::init (Gdk::Color& basic_color, bool wfd)
        fade_in_active_changed ();
        fade_out_active_changed ();
 
-       _region.StateChanged.connect (mem_fun(*this, &AudioRegionView::region_changed));
+       _region->StateChanged.connect (mem_fun(*this, &AudioRegionView::region_changed));
 
        fade_in_shape->signal_event().connect (bind (mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_event), fade_in_shape, this));
        fade_in_handle->signal_event().connect (bind (mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_handle_event), fade_in_handle, this));
@@ -205,11 +226,11 @@ AudioRegionView::~AudioRegionView ()
        }
 }
 
-ARDOUR::AudioRegion&
+boost::shared_ptr<ARDOUR::AudioRegion>
 AudioRegionView::audio_region() const
 {
        // "Guaranteed" to succeed...
-       return dynamic_cast<AudioRegion&>(_region);
+       return boost::dynamic_pointer_cast<AudioRegion>(_region);
 }
 
 void
@@ -250,19 +271,6 @@ AudioRegionView::fade_out_changed ()
 {
        reset_fade_out_shape ();
 }
-
-void
-AudioRegionView::set_fade_in_active (bool yn)
-{
-       audio_region().set_fade_in_active (yn);
-}
-
-void
-AudioRegionView::set_fade_out_active (bool yn)
-{
-       audio_region().set_fade_out_active (yn);
-}
-
 void
 AudioRegionView::fade_in_active_changed ()
 {
@@ -270,7 +278,7 @@ AudioRegionView::fade_in_active_changed ()
        uint32_t col;
        UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
 
-       if (audio_region().fade_in_active()) {
+       if (audio_region()->fade_in_active()) {
                col = RGBA_TO_UINT(r,g,b,120);
                fade_in_shape->property_fill_color_rgba() = col;
                fade_in_shape->property_width_pixels() = 0;
@@ -290,7 +298,7 @@ AudioRegionView::fade_out_active_changed ()
        uint32_t col;
        UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
 
-       if (audio_region().fade_out_active()) {
+       if (audio_region()->fade_out_active()) {
                col = RGBA_TO_UINT(r,g,b,120);
                fade_out_shape->property_fill_color_rgba() = col;
                fade_out_shape->property_width_pixels() = 0;
@@ -311,7 +319,7 @@ AudioRegionView::region_scale_amplitude_changed ()
 
        for (uint32_t n = 0; n < waves.size(); ++n) {
                // force a reload of the cache
-               waves[n]->property_data_src() = &_region;
+               waves[n]->property_data_src() = _region.get();
        }
 }
 
@@ -323,13 +331,13 @@ AudioRegionView::region_resized (Change what_changed)
        if (what_changed & Change (StartChanged|LengthChanged)) {
 
                for (uint32_t n = 0; n < waves.size(); ++n) {
-                       waves[n]->property_region_start() = _region.start();
+                       waves[n]->property_region_start() = _region->start();
                }
                
                for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
 
                        for (vector<WaveView*>::iterator w = (*i)->waves.begin(); w != (*i)->waves.end(); ++w) {
-                               (*w)->property_region_start() = _region.start();
+                               (*w)->property_region_start() = _region->start();
                        }
                }
        }
@@ -369,7 +377,7 @@ AudioRegionView::region_muted ()
        RegionView::region_muted();
 
        for (uint32_t n=0; n < waves.size(); ++n) {
-               if (_region.muted()) {
+               if (_region->muted()) {
                        waves[n]->property_wave_color() = color_map[cMutedWaveForm];
                } else {
                        waves[n]->property_wave_color() = color_map[cWaveForm];
@@ -449,11 +457,11 @@ AudioRegionView::reset_fade_shapes ()
 void
 AudioRegionView::reset_fade_in_shape ()
 {
-       reset_fade_in_shape_width ((jack_nframes_t) audio_region().fade_in().back()->when);
+       reset_fade_in_shape_width ((nframes_t) audio_region()->fade_in().back()->when);
 }
        
 void
-AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
+AudioRegionView::reset_fade_in_shape_width (nframes_t width)
 {
        if (fade_in_handle == 0) {
                return;
@@ -461,7 +469,7 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
 
        /* smallest size for a fade is 64 frames */
 
-       width = std::max ((jack_nframes_t) 64, width);
+       width = std::max ((nframes_t) 64, width);
 
        Points* points;
        double pwidth = width / samples_per_unit;
@@ -494,7 +502,7 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
        fade_in_shape->show();
 
        float curve[npoints];
-       audio_region().fade_in().get_vector (0, audio_region().fade_in().back()->when, curve, npoints);
+       audio_region()->fade_in().get_vector (0, audio_region()->fade_in().back()->when, curve, npoints);
 
        points = get_canvas_points ("fade in shape", npoints+3);
 
@@ -533,11 +541,11 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
 void
 AudioRegionView::reset_fade_out_shape ()
 {
-       reset_fade_out_shape_width ((jack_nframes_t) audio_region().fade_out().back()->when);
+       reset_fade_out_shape_width ((nframes_t) audio_region()->fade_out().back()->when);
 }
 
 void
-AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
+AudioRegionView::reset_fade_out_shape_width (nframes_t width)
 {      
        if (fade_out_handle == 0) {
                return;
@@ -545,7 +553,7 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
 
        /* smallest size for a fade is 64 frames */
 
-       width = std::max ((jack_nframes_t) 64, width);
+       width = std::max ((nframes_t) 64, width);
 
        Points* points;
        double pwidth = width / samples_per_unit;
@@ -559,7 +567,7 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
        }
 
        double handle_center;
-       handle_center = (_region.length() - width) / samples_per_unit;
+       handle_center = (_region->length() - width) / samples_per_unit;
        
        if (handle_center > 7.0) {
                handle_center -= 3.0;
@@ -580,7 +588,7 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
        fade_out_shape->show();
 
        float curve[npoints];
-       audio_region().fade_out().get_vector (0, audio_region().fade_out().back()->when, curve, npoints);
+       audio_region()->fade_out().get_vector (0, audio_region()->fade_out().back()->when, curve, npoints);
 
        if (_height > NAME_HIGHLIGHT_THRESH) {
                h = _height - NAME_HIGHLIGHT_SIZE;
@@ -621,13 +629,16 @@ AudioRegionView::set_samples_per_unit (gdouble spu)
 {
        RegionView::set_samples_per_unit (spu);
 
-       for (uint32_t n=0; n < waves.size(); ++n) {
-               waves[n]->property_samples_per_unit() = spu;
+       if (_flags & WaveformVisible) {
+               for (uint32_t n=0; n < waves.size(); ++n) {
+                       waves[n]->property_samples_per_unit() = spu;
+               }
        }
 
        if (gain_line) {
                gain_line->reset ();
        }
+
        reset_fade_shapes ();
 }
 
@@ -658,15 +669,18 @@ AudioRegionView::set_colors ()
        RegionView::set_colors();
        
        if (gain_line) {
-               gain_line->set_line_color (audio_region().envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
+               gain_line->set_line_color (audio_region()->envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
        }
 
        for (uint32_t n=0; n < waves.size(); ++n) {
-               if (_region.muted()) {
+               if (_region->muted()) {
                        waves[n]->property_wave_color() = color_map[cMutedWaveForm];
                } else {
                        waves[n]->property_wave_color() = color_map[cWaveForm];
                }
+
+               waves[n]->property_clip_color() = color_map[cWaveFormClip];
+               waves[n]->property_zero_color() = color_map[cZeroLine];
        }
 }
 
@@ -680,8 +694,8 @@ AudioRegionView::show_region_editor ()
                // trackview.editor.ensure_float (*editor);
        } 
 
-       editor->show_all ();
-       editor->get_window()->raise();
+       editor->present ();
+       editor->show_all();
 }
 
 void
@@ -690,6 +704,10 @@ AudioRegionView::set_waveform_visible (bool yn)
        if (((_flags & WaveformVisible) != yn)) {
                if (yn) {
                        for (uint32_t n=0; n < waves.size(); ++n) {
+                               /* make sure the zoom level is correct, since we don't update
+                                  this when waveforms are hidden.
+                               */
+                               waves[n]->property_samples_per_unit() = samples_per_unit;
                                waves[n]->show();
                        }
                        _flags |= WaveformVisible;
@@ -737,8 +755,6 @@ AudioRegionView::set_envelope_visible (bool yn)
 void
 AudioRegionView::create_waves ()
 {
-       bool create_zero_line = true;
-
        RouteTimeAxisView& atv (*(dynamic_cast<RouteTimeAxisView*>(&trackview))); // ick
 
        if (!atv.get_diskstream()) {
@@ -751,33 +767,24 @@ AudioRegionView::create_waves ()
        for (uint32_t n = 0; n < nchans; ++n) {
                tmp_waves.push_back (0);
        }
-       
+
        for (uint32_t n = 0; n < nchans; ++n) {
                
-               if (n >= audio_region().n_channels()) {
+               if (n >= audio_region()->n_channels()) {
                        break;
                }
                
                wave_caches.push_back (WaveView::create_cache ());
 
                if (wait_for_data) {
-                       if (audio_region().source(n).peaks_ready (bind (mem_fun(*this, &AudioRegionView::peaks_ready_handler), n), data_ready_connection)) {
+                       if (audio_region()->source(n)->peaks_ready (bind (mem_fun(*this, &AudioRegionView::peaks_ready_handler), n), data_ready_connection)) {
                                create_one_wave (n, true);
                        } else {
-                               create_zero_line = false;
                        }
                } else {
                        create_one_wave (n, true);
                }
        }
-
-       if (create_zero_line) {
-               zero_line = new ArdourCanvas::SimpleLine (*group);
-               zero_line->property_x1() = (gdouble) 1.0;
-               zero_line->property_x2() = (gdouble) (_region.length() / samples_per_unit) - 1.0;
-               zero_line->property_color_rgba() = (guint) color_map[cZeroLine];
-               manage_zero_line ();
-       }
 }
 
 void
@@ -786,7 +793,7 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
        RouteTimeAxisView& atv (*(dynamic_cast<RouteTimeAxisView*>(&trackview))); // ick
        uint32_t nchans = atv.get_diskstream()->n_channels();
        uint32_t n;
-       uint32_t nwaves = std::min (nchans, audio_region().n_channels());
+       uint32_t nwaves = std::min (nchans, audio_region()->n_channels());
        gdouble ht;
 
        if (trackview.height < NAME_HIGHLIGHT_SIZE) {
@@ -799,7 +806,7 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
 
        WaveView *wave = new WaveView(*group);
 
-       wave->property_data_src() = (gpointer) &_region;
+       wave->property_data_src() = (gpointer) _region.get();
        wave->property_cache() =  wave_caches[which];
        wave->property_cache_updater() = true;
        wave->property_channel() =  which;
@@ -811,8 +818,12 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
        wave->property_height() =  (double) ht;
        wave->property_samples_per_unit() =  samples_per_unit;
        wave->property_amplitude_above_axis() =  _amplitude_above_axis;
-       wave->property_wave_color() = _region.muted() ? color_map[cMutedWaveForm] : color_map[cWaveForm];
-       wave->property_region_start() = _region.start();
+       wave->property_wave_color() = _region->muted() ? color_map[cMutedWaveForm] : color_map[cWaveForm];
+       wave->property_clip_color() = color_map[cWaveFormClip];
+       wave->property_zero_color() = color_map[cZeroLine];
+       wave->property_region_start() = _region->start();
+       wave->property_rectified() = (bool) (_flags & WaveformRectified);
+       wave->property_logscaled() = (bool) (_flags & WaveformLogScaled);
 
        if (!(_flags & WaveformVisible)) {
                wave->hide();
@@ -837,7 +848,7 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
                        break;
                }
        }
-       
+
        if (n == nwaves && waves.empty()) {
                /* all waves are ready */
                tmp_waves.resize(nwaves);
@@ -845,10 +856,14 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
                waves = tmp_waves;
                tmp_waves.clear ();
 
+               /* all waves created, don't hook into peaks ready anymore */
+               data_ready_connection.disconnect ();            
+
+               if(0)
                if (!zero_line) {
                        zero_line = new ArdourCanvas::SimpleLine (*group);
                        zero_line->property_x1() = (gdouble) 1.0;
-                       zero_line->property_x2() = (gdouble) (_region.length() / samples_per_unit) - 1.0;
+                       zero_line->property_x2() = (gdouble) (_region->length() / samples_per_unit) - 1.0;
                        zero_line->property_color_rgba() = (guint) color_map[cZeroLine];
                        manage_zero_line ();
                }
@@ -859,11 +874,6 @@ void
 AudioRegionView::peaks_ready_handler (uint32_t which)
 {
        Gtkmm2ext::UI::instance()->call_slot (bind (mem_fun(*this, &AudioRegionView::create_one_wave), which, false));
-
-       if (!waves.empty()) {
-               /* all waves created, don't hook into peaks ready anymore */
-               data_ready_connection.disconnect ();            
-       }
 }
 
 void
@@ -884,9 +894,9 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 
        item->w2i (x, y);
 
-       jack_nframes_t fx = trackview.editor.pixel_to_frame (x);
+       nframes_t fx = trackview.editor.pixel_to_frame (x);
 
-       if (fx > _region.length()) {
+       if (fx > _region->length()) {
                return;
        }
 
@@ -899,20 +909,19 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
        gain_line->view_to_model_y (y);
 
        trackview.session().begin_reversible_command (_("add gain control point"));
-       XMLNode &before = audio_region().envelope().get_state();
-
+       XMLNode &before = audio_region()->envelope().get_state();
 
-       if (!audio_region().envelope_active()) {
-               XMLNode &before = audio_region().get_state();
-               audio_region().set_envelope_active(true);
-               XMLNode &after = audio_region().get_state();
-               trackview.session().add_command (new MementoCommand<AudioRegion>(audio_region(), &before, &after));
+       if (!audio_region()->envelope_active()) {
+               XMLNode &region_before = audio_region()->get_state();
+               audio_region()->set_envelope_active(true);
+               XMLNode &region_after = audio_region()->get_state();
+               trackview.session().add_command (new MementoCommand<AudioRegion>(*(audio_region().get()), &region_before, &region_after));
        }
 
-       audio_region().envelope().add (fx, y);
+       audio_region()->envelope().add (fx, y);
        
-       XMLNode &after = audio_region().envelope().get_state();
-       trackview.session().add_command (new MementoCommand<Curve>(audio_region().envelope(), &before, &after));
+       XMLNode &after = audio_region()->envelope().get_state();
+       trackview.session().add_command (new MementoCommand<Curve>(audio_region()->envelope(), &before, &after));
        trackview.session().commit_reversible_command ();
 }
 
@@ -920,7 +929,7 @@ void
 AudioRegionView::remove_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 {
         ControlPoint *cp = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"));
-       audio_region().envelope().erase (cp->model);
+       audio_region()->envelope().erase (cp->model);
 }
 
 void
@@ -930,8 +939,10 @@ AudioRegionView::store_flags()
 
        node->add_property ("waveform-visible", (_flags & WaveformVisible) ? "yes" : "no");
        node->add_property ("envelope-visible", (_flags & EnvelopeVisible) ? "yes" : "no");
+       node->add_property ("waveform-rectified", (_flags & WaveformRectified) ? "yes" : "no");
+       node->add_property ("waveform-logscaled", (_flags & WaveformLogScaled) ? "yes" : "no");
 
-       _region.add_extra_xml (*node);
+       _region->add_extra_xml (*node);
 }
 
 void
@@ -950,6 +961,18 @@ AudioRegionView::set_flags (XMLNode* node)
                        _flags |= EnvelopeVisible;
                }
        }
+
+       if ((prop = node->property ("waveform-rectified")) != 0) {
+               if (prop->value() == "yes") {
+                       _flags |= WaveformRectified;
+               }
+       }
+
+       if ((prop = node->property ("waveform-logscaled")) != 0) {
+               if (prop->value() == "yes") {
+                       _flags |= WaveformLogScaled;
+               }
+       }
 }
        
 void
@@ -989,16 +1012,37 @@ AudioRegionView::set_waveform_shape (WaveformShape shape)
                } else {
                        _flags &= ~WaveformRectified;
                }
+               store_flags ();
+       }
+}
+
+void
+AudioRegionView::set_waveform_scale (WaveformScale scale)
+{
+       bool yn = (scale == LogWaveform);
+
+       if (yn != (bool) (_flags & WaveformLogScaled)) {
+               for (vector<WaveView *>::iterator wave = waves.begin(); wave != waves.end() ; ++wave) {
+                       (*wave)->property_logscaled() = yn;
+               }
+
+               if (yn) {
+                       _flags |= WaveformLogScaled;
+               } else {
+                       _flags &= ~WaveformLogScaled;
+               }
+               store_flags ();
        }
 }
 
+
 GhostRegion*
 AudioRegionView::add_ghost (AutomationTimeAxisView& atv)
 {
        RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(&trackview);
        assert(rtv);
 
-       double unit_position = _region.position () / samples_per_unit;
+       double unit_position = _region->position () / samples_per_unit;
        GhostRegion* ghost = new GhostRegion (atv, unit_position);
        uint32_t nchans;
        
@@ -1006,13 +1050,13 @@ AudioRegionView::add_ghost (AutomationTimeAxisView& atv)
 
        for (uint32_t n = 0; n < nchans; ++n) {
                
-               if (n >= audio_region().n_channels()) {
+               if (n >= audio_region()->n_channels()) {
                        break;
                }
                
                WaveView *wave = new WaveView(*ghost->group);
 
-               wave->property_data_src() =  &_region;
+               wave->property_data_src() = _region.get();
                wave->property_cache() =  wave_caches[n];
                wave->property_cache_updater() = false;
                wave->property_channel() = n;
@@ -1023,13 +1067,15 @@ AudioRegionView::add_ghost (AutomationTimeAxisView& atv)
                wave->property_samples_per_unit() =  samples_per_unit;
                wave->property_amplitude_above_axis() =  _amplitude_above_axis;
                wave->property_wave_color() = color_map[cGhostTrackWave];
-               wave->property_region_start() = _region.start();
+               wave->property_clip_color() = color_map[cGhostTrackWaveClip];
+               wave->property_zero_color() = color_map[cGhostTrackZeroLine];
+               wave->property_region_start() = _region->start();
 
                ghost->waves.push_back(wave);
        }
 
        ghost->set_height ();
-       ghost->set_duration (_region.length() / samples_per_unit);
+       ghost->set_duration (_region->length() / samples_per_unit);
        ghosts.push_back (ghost);
 
        ghost->GoingAway.connect (mem_fun(*this, &AudioRegionView::remove_ghost));
@@ -1075,7 +1121,7 @@ void
 AudioRegionView::envelope_active_changed ()
 {
        if (gain_line) {
-               gain_line->set_line_color (audio_region().envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
+               gain_line->set_line_color (audio_region()->envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
        }
 }
 
@@ -1083,11 +1129,11 @@ void
 AudioRegionView::set_waveview_data_src()
 {
 
-       double unit_length= _region.length() / samples_per_unit;
+       double unit_length= _region->length() / samples_per_unit;
 
        for (uint32_t n = 0; n < waves.size(); ++n) {
                // TODO: something else to let it know the channel
-               waves[n]->property_data_src() = &_region;
+               waves[n]->property_data_src() = _region.get();
        }
        
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
@@ -1095,7 +1141,7 @@ AudioRegionView::set_waveview_data_src()
                (*i)->set_duration (unit_length);
                
                for (vector<WaveView*>::iterator w = (*i)->waves.begin(); w != (*i)->waves.end(); ++w) {
-                       (*w)->property_data_src() = &_region;
+                       (*w)->property_data_src() = _region.get();
                }
        }
 
@@ -1107,6 +1153,8 @@ AudioRegionView::color_handler (ColorID id, uint32_t val)
        switch (id) {
        case cMutedWaveForm:
        case cWaveForm:
+       case cWaveFormClip:
+       case cZeroLine:
                set_colors ();
                break;
 
@@ -1114,15 +1162,6 @@ AudioRegionView::color_handler (ColorID id, uint32_t val)
        case cGainLine:
                envelope_active_changed();
                break;
-               
-       case cZeroLine:
-               if (zero_line) {
-                       zero_line->property_color_rgba() = (guint) color_map[cZeroLine];
-               }
-               break;
-
-       case cGhostTrackWave:
-               break;
 
        default:
                break;