(native) Linux VST support from LinuxDSP
[ardour.git] / gtk2_ardour / audio_region_view.cc
index eea828937943e44a1601c8faf557cf14560a5120..4729024159e45febf5fb2c8fb484ac82ad7eff31 100644 (file)
@@ -68,7 +68,6 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
                                  Gdk::Color const & basic_color)
        : RegionView (parent, tv, r, spu, basic_color)
        , sync_mark(0)
-       , zero_line(0)
        , fade_in_shape(0)
        , fade_out_shape(0)
        , fade_in_handle(0)
@@ -86,7 +85,6 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
                                  Gdk::Color const & basic_color, bool recording, TimeAxisViewItem::Visibility visibility)
        : RegionView (parent, tv, r, spu, basic_color, recording, visibility)
        , sync_mark(0)
-       , zero_line(0)
        , fade_in_shape(0)
        , fade_out_shape(0)
        , fade_in_handle(0)
@@ -99,41 +97,16 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
 {
 }
 
-
-AudioRegionView::AudioRegionView (const AudioRegionView& other)
-       : sigc::trackable(other)
-       , RegionView (other)
-       , zero_line(0)
-       , fade_in_shape(0)
-       , fade_out_shape(0)
-       , fade_in_handle(0)
-       , fade_out_handle(0)
-       , fade_position_line(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);
-}
-
 AudioRegionView::AudioRegionView (const AudioRegionView& other, boost::shared_ptr<AudioRegion> other_region)
        : RegionView (other, boost::shared_ptr<Region> (other_region))
-       , zero_line(0)
        , fade_in_shape(0)
        , fade_out_shape(0)
        , fade_in_handle(0)
        , fade_out_handle(0)
        , fade_position_line(0)
        , gain_line(0)
-       , _amplitude_above_axis(1.0)
-       , _flags(0)
+       , _amplitude_above_axis (other._amplitude_above_axis)
+       , _flags (other._flags)
        , fade_color(0)
 {
        Gdk::Color c;
@@ -156,7 +129,6 @@ AudioRegionView::init (Gdk::Color const & basic_color, bool wfd)
        XMLNode *node;
 
        _amplitude_above_axis = 1.0;
-       zero_line             = 0;
        _flags                = 0;
 
        if ((node = _region->extra_xml ("GUI")) != 0) {
@@ -185,21 +157,18 @@ AudioRegionView::init (Gdk::Color const & basic_color, bool wfd)
        fade_out_shape->set_data ("regionview", this);
 
        if (!_recregion) {
-               uint32_t r,g,b,a;
-               UINT_TO_RGBA(fill_color,&r,&g,&b,&a);
-
                fade_in_handle = new ArdourCanvas::SimpleRect (*group);
-               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,0);
+               fade_in_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fill_color, 0);
                fade_in_handle->property_outline_pixels() = 0;
 
                fade_in_handle->set_data ("regionview", this);
 
                fade_out_handle = new ArdourCanvas::SimpleRect (*group);
-               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,0);
+               fade_out_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fill_color, 0);
                fade_out_handle->property_outline_pixels() = 0;
 
                fade_out_handle->set_data ("regionview", this);
-               
+
                fade_position_line = new ArdourCanvas::SimpleLine (*group);
                fade_position_line->property_color_rgba() = 0xBBBBBBAA;
                fade_position_line->property_y1() = 7;
@@ -272,10 +241,10 @@ AudioRegionView::~AudioRegionView ()
                delete *i;
        }
 
-       for (list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator i = feature_lines.begin(); i != feature_lines.end(); ++i) {
+       for (list<std::pair<framepos_t, ArdourCanvas::Line*> >::iterator i = feature_lines.begin(); i != feature_lines.end(); ++i) {
                delete ((*i).second);
        }
-       
+
        /* all waveviews etc will be destroyed when the group is destroyed */
 
        delete gain_line;
@@ -292,7 +261,7 @@ void
 AudioRegionView::region_changed (const PropertyChange& what_changed)
 {
        ENSURE_GUI_THREAD (*this, &AudioRegionView::region_changed, what_changed);
-        // cerr << "AudioRegionView::region_changed() called" << endl;
+       // cerr << "AudioRegionView::region_changed() called" << endl;
 
        RegionView::region_changed (what_changed);
 
@@ -300,7 +269,7 @@ AudioRegionView::region_changed (const PropertyChange& what_changed)
                region_scale_amplitude_changed ();
        }
        if (what_changed.contains (ARDOUR::Properties::fade_in)) {
-                fade_in_changed ();
+               fade_in_changed ();
        }
        if (what_changed.contains (ARDOUR::Properties::fade_out)) {
                fade_out_changed ();
@@ -369,7 +338,7 @@ AudioRegionView::region_scale_amplitude_changed ()
 void
 AudioRegionView::region_renamed ()
 {
-       Glib::ustring str = RegionView::make_name ();
+       std::string str = RegionView::make_name ();
 
        if (audio_region()->speed_mismatch (trackview.session()->frame_rate())) {
                str = string ("*") + str;
@@ -408,17 +377,16 @@ AudioRegionView::region_resized (const PropertyChange& what_changed)
                                }
                        }
                }
-               
+
                /* hide transient lines that extend beyond the region end */
-               
-               list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
-               
+
+               list<std::pair<framepos_t, ArdourCanvas::Line*> >::iterator l;
+
                for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
-                       if ((*l).first > _region->length()- 1){
-                         (*l).second->hide();
-                       }
-                       else {
-                         (*l).second->show();
+                       if (l->first > _region->length() - 1) {
+                               l->second->hide();
+                       } else {
+                               l->second->show();
                        }
                }
        }
@@ -430,10 +398,6 @@ AudioRegionView::reset_width_dependent_items (double pixel_width)
        RegionView::reset_width_dependent_items(pixel_width);
        assert(_pixel_width == pixel_width);
 
-       if (zero_line) {
-               zero_line->property_x2() = pixel_width - 1.0;
-       }
-
        if (fade_in_handle) {
                if (pixel_width <= 6.0 || _height < 5.0 || !trackview.session()->config.get_show_region_fades()) {
                        fade_in_handle->hide();
@@ -441,19 +405,27 @@ AudioRegionView::reset_width_dependent_items (double pixel_width)
                }
                else {
                        fade_in_handle->show();
-                       fade_out_handle->show();                  
+                       fade_out_handle->show();
                }
        }
 
        AnalysisFeatureList analysis_features = _region->transients();
        AnalysisFeatureList::const_iterator i;
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+
+       list<std::pair<framepos_t, ArdourCanvas::Line*> >::iterator l;
 
        for (i = analysis_features.begin(), l = feature_lines.begin(); i != analysis_features.end() && l != feature_lines.end(); ++i, ++l) {
-               (*l).second->property_x1() = trackview.editor().frame_to_pixel (*i);
-               (*l).second->property_x2() = trackview.editor().frame_to_pixel (*i);
+
+               float x_pos = trackview.editor().frame_to_pixel (*i);
+
+               ArdourCanvas::Points points;
+               points.push_back(Gnome::Art::Point(x_pos, 2.0)); // first x-coord needs to be a non-normal value
+               points.push_back(Gnome::Art::Point(x_pos, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
+
+               (*l).first = *i;
+               (*l).second->property_points() = points;
        }
-       
+
        reset_fade_shapes ();
 }
 
@@ -525,35 +497,35 @@ AudioRegionView::set_height (gdouble height)
                gain_line->set_height ((uint32_t) rint (height - NAME_HIGHLIGHT_SIZE) - 2);
        }
 
-       manage_zero_line ();
        reset_fade_shapes ();
-       
+
        /* Update hights for any active feature lines */
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::Line*> >::iterator l;
 
        for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
-               (*l).second->property_y2() = _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1;
-       }       
 
-       if (name_pixbuf) {
-               name_pixbuf->raise_to_top();
+               float pos_x = trackview.editor().frame_to_pixel((*l).first);
+
+               ArdourCanvas::Points points;
+
+               points.push_back(Gnome::Art::Point(pos_x, 2.0)); // first x-coord needs to be a non-normal value
+               points.push_back(Gnome::Art::Point(pos_x, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
+
+               (*l).second->property_points() = points;
        }
-}
 
-void
-AudioRegionView::manage_zero_line ()
-{
-       if (!zero_line) {
-               return;
+       if (fade_position_line) {
+
+               if (height < NAME_HIGHLIGHT_THRESH) {
+                       fade_position_line->property_y2() = _height - 1;
+               }
+               else {
+                       fade_position_line->property_y2() = _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1;
+               }
        }
 
-       if (_height >= 100) {
-               double const wave_midpoint = (_height - NAME_HIGHLIGHT_SIZE) / 2.0;
-               zero_line->property_y1() = wave_midpoint;
-               zero_line->property_y2() = wave_midpoint;
-               zero_line->show();
-       } else {
-               zero_line->hide();
+       if (name_pixbuf) {
+               name_pixbuf->raise_to_top();
        }
 }
 
@@ -567,11 +539,11 @@ AudioRegionView::reset_fade_shapes ()
 void
 AudioRegionView::reset_fade_in_shape ()
 {
-       reset_fade_in_shape_width ((nframes_t) audio_region()->fade_in()->back()->when);
+       reset_fade_in_shape_width ((framecnt_t) audio_region()->fade_in()->back()->when);
 }
 
 void
-AudioRegionView::reset_fade_in_shape_width (nframes_t width)
+AudioRegionView::reset_fade_in_shape_width (framecnt_t width)
 {
        if (fade_in_handle == 0) {
                return;
@@ -579,11 +551,12 @@ AudioRegionView::reset_fade_in_shape_width (nframes_t width)
 
        /* smallest size for a fade is 64 frames */
 
-       width = std::max ((nframes_t) 64, width);
+       width = std::max ((framecnt_t) 64, width);
 
        Points* points;
-       
-       double pwidth = width / samples_per_unit;
+
+       /* round here to prevent little visual glitches with sub-pixel placement */
+       double const pwidth = rint (width / samples_per_unit);
        uint32_t npoints = std::min (gdk_screen_width(), (int) pwidth);
        double h;
 
@@ -593,17 +566,11 @@ AudioRegionView::reset_fade_in_shape_width (nframes_t width)
                return;
        }
 
-       double handle_center;
-       handle_center = pwidth;
+       double const handle_center = pwidth;
 
-       if (handle_center > 7.0) {
-               handle_center -= 2.0;
-       } else {
-               handle_center = 3.0;
-       }
-
-       fade_in_handle->property_x1() =  handle_center - 3.0;
-       fade_in_handle->property_x2() =  handle_center + 3.0;
+       /* Put the fade in handle so that its left side is at the end-of-fade line */
+       fade_in_handle->property_x1() = handle_center;
+       fade_in_handle->property_x2() = handle_center + 6;
 
        if (pwidth < 5) {
                fade_in_shape->hide();
@@ -659,11 +626,11 @@ AudioRegionView::reset_fade_in_shape_width (nframes_t width)
 void
 AudioRegionView::reset_fade_out_shape ()
 {
-       reset_fade_out_shape_width ((nframes_t) audio_region()->fade_out()->back()->when);
+       reset_fade_out_shape_width ((framecnt_t) audio_region()->fade_out()->back()->when);
 }
 
 void
-AudioRegionView::reset_fade_out_shape_width (nframes_t width)
+AudioRegionView::reset_fade_out_shape_width (framecnt_t width)
 {
        if (fade_out_handle == 0) {
                return;
@@ -671,10 +638,12 @@ AudioRegionView::reset_fade_out_shape_width (nframes_t width)
 
        /* smallest size for a fade is 64 frames */
 
-       width = std::max ((nframes_t) 64, width);
+       width = std::max ((framecnt_t) 64, width);
 
        Points* points;
-       double pwidth = width / samples_per_unit;
+
+       /* round here to prevent little visual glitches with sub-pixel placement */
+       double const pwidth = rint (width / samples_per_unit);
        uint32_t npoints = std::min (gdk_screen_width(), (int) pwidth);
        double h;
 
@@ -684,17 +653,13 @@ AudioRegionView::reset_fade_out_shape_width (nframes_t width)
                return;
        }
 
-       double handle_center;
-       handle_center = (_region->length() - width) / samples_per_unit;
-
-       if (handle_center > 7.0) {
-               handle_center -= 2.0;
-       } else {
-               handle_center = 3.0;
-       }
+       double const handle_center = (_region->length() - width) / samples_per_unit;
 
-       fade_out_handle->property_x1() =  handle_center - 3.0;
-       fade_out_handle->property_x2() =  handle_center + 3.0;
+       /* Put the fade out handle so that its right side is at the end-of-fade line;
+        * it's `one out' for precise pixel accuracy.
+        */
+       fade_out_handle->property_x1() = handle_center - 5;
+       fade_out_handle->property_x2() = handle_center + 1;
 
        /* don't show shape if its too small */
 
@@ -780,12 +745,9 @@ AudioRegionView::compute_colors (Gdk::Color const & basic_color)
 {
        RegionView::compute_colors (basic_color);
 
-       uint32_t r, g, b, a;
-
        /* gain color computed in envelope_active_changed() */
 
-       UINT_TO_RGBA (fill_color, &r, &g, &b, &a);
-       fade_color = RGBA_TO_UINT(r,g,b,120);
+       fade_color = UINT_RGBA_CHANGE_A (fill_color, 120);
 }
 
 void
@@ -964,6 +926,7 @@ AudioRegionView::create_one_wave (uint32_t which, bool /*direct*/)
 
        wave->property_clip_color() = ARDOUR_UI::config()->canvasvar_WaveFormClip.get();
        wave->property_zero_color() = ARDOUR_UI::config()->canvasvar_ZeroLine.get();
+       wave->property_zero_line() = true;
        wave->property_region_start() = _region->start();
        wave->property_rectified() = (bool) (_flags & WaveformRectified);
        wave->property_logscaled() = (bool) (_flags & WaveformLogScaled);
@@ -1002,16 +965,6 @@ AudioRegionView::create_one_wave (uint32_t which, bool /*direct*/)
                /* all waves created, don't hook into peaks ready anymore */
                delete _data_ready_connections[which];
                _data_ready_connections[which] = 0;
-
-#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_color_rgba() = (guint) ARDOUR_UI::config()->canvasvar_ZeroLine.get();
-                       manage_zero_line ();
-               }
-#endif
        }
 }
 
@@ -1040,7 +993,7 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 
        item->w2i (x, y);
 
-       nframes_t fx = trackview.editor().pixel_to_frame (x);
+       framepos_t fx = trackview.editor().pixel_to_frame (x);
 
        if (fx > _region->length()) {
                return;
@@ -1054,9 +1007,9 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 
        gain_line->view_to_model_coord (x, y);
 
-        /* XXX STATEFUL: can't convert to stateful diff until we 
-           can represent automation data with it.
-        */
+       /* XXX STATEFUL: can't convert to stateful diff until we
+          can represent automation data with it.
+       */
 
        trackview.session()->begin_reversible_command (_("add gain control point"));
        XMLNode &before = audio_region()->envelope()->get_state();
@@ -1149,14 +1102,6 @@ AudioRegionView::set_waveform_shape (WaveformShape shape)
                        (*wave)->property_rectified() = yn;
                }
 
-               if (zero_line) {
-                       if (yn) {
-                               zero_line->hide();
-                       } else {
-                               zero_line->show();
-                       }
-               }
-
                if (yn) {
                        _flags |= WaveformRectified;
                } else {
@@ -1233,34 +1178,32 @@ AudioRegionView::add_ghost (TimeAxisView& tv)
 void
 AudioRegionView::entered (bool internal_editing)
 {
+       trackview.editor().set_current_trimmable (_region);
+       trackview.editor().set_current_movable (_region);
+
        if (gain_line && _flags & EnvelopeVisible) {
                gain_line->show_all_control_points ();
        }
 
-       uint32_t r,g,b,a;
-       UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
-       a=255;
-
        if (fade_in_handle && !internal_editing) {
-               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
-               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
+               fade_in_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fade_color, 255);
+               fade_out_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fade_color, 255);
        }
 }
 
 void
 AudioRegionView::exited ()
 {
+       trackview.editor().set_current_trimmable (boost::shared_ptr<Trimmable>());
+       trackview.editor().set_current_movable (boost::shared_ptr<Movable>());
+
        if (gain_line) {
                gain_line->hide_all_but_selected_control_points ();
        }
 
-       uint32_t r,g,b,a;
-       UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
-       a=0;
-
        if (fade_in_handle) {
-               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
-               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
+               fade_in_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fade_color, 0);
+               fade_out_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fade_color, 0);
        }
 }
 
@@ -1324,47 +1267,44 @@ AudioRegionView::set_frame_color ()
                fill_opacity = 0;
        }
 
-       uint32_t r,g,b,a;
+       TimeAxisViewItem::set_frame_color ();
 
-       if (_selected && should_show_selection) {
-               UINT_TO_RGBA(ARDOUR_UI::config()->canvasvar_SelectedFrameBase.get(), &r, &g, &b, &a);
-               frame->property_fill_color_rgba() = RGBA_TO_UINT(r, g, b, fill_opacity ? fill_opacity : a);
+        uint32_t wc;
+        uint32_t fc;
 
-               for (vector<ArdourCanvas::WaveView*>::iterator w = waves.begin(); w != waves.end(); ++w) {
-                       if (_region->muted()) {
-                               (*w)->property_wave_color() = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_SelectedWaveForm.get(), MUTED_ALPHA);
-                       } else {
-                               (*w)->property_wave_color() = ARDOUR_UI::config()->canvasvar_SelectedWaveForm.get();
-                               (*w)->property_fill_color() = ARDOUR_UI::config()->canvasvar_SelectedWaveFormFill.get();
-                       }
-               }
+       if (_selected) {
+                if (_region->muted()) {
+                        wc = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_SelectedWaveForm.get(), MUTED_ALPHA);
+                } else {
+                        wc = ARDOUR_UI::config()->canvasvar_SelectedWaveForm.get();
+                }
+                fc = ARDOUR_UI::config()->canvasvar_SelectedWaveFormFill.get();
        } else {
                if (_recregion) {
-                       UINT_TO_RGBA(ARDOUR_UI::config()->canvasvar_RecordingRect.get(), &r, &g, &b, &a);
-                       frame->property_fill_color_rgba() = RGBA_TO_UINT(r, g, b, a);
-
-                       for (vector<ArdourCanvas::WaveView*>::iterator w = waves.begin(); w != waves.end(); ++w) {
-                               if (_region->muted()) {
-                                       (*w)->property_wave_color() = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_RecWaveForm.get(), MUTED_ALPHA);
-                               } else {
-                                       (*w)->property_wave_color() = ARDOUR_UI::config()->canvasvar_RecWaveForm.get();
-                                       (*w)->property_fill_color() = ARDOUR_UI::config()->canvasvar_RecWaveFormFill.get();
-                               }
-                       }
+                        if (_region->muted()) {
+                                wc = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_RecWaveForm.get(), MUTED_ALPHA);
+                        } else {
+                                wc = ARDOUR_UI::config()->canvasvar_RecWaveForm.get();
+                        }
+                        fc = ARDOUR_UI::config()->canvasvar_RecWaveFormFill.get();
                } else {
-                       UINT_TO_RGBA(ARDOUR_UI::config()->canvasvar_FrameBase.get(), &r, &g, &b, &a);
-                       frame->property_fill_color_rgba() = RGBA_TO_UINT(r, g, b, fill_opacity ? fill_opacity : a);
-
-                       for (vector<ArdourCanvas::WaveView*>::iterator w = waves.begin(); w != waves.end(); ++w) {
-                               if (_region->muted()) {
-                                       (*w)->property_wave_color() = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA);
-                               } else {
-                                       (*w)->property_wave_color() = ARDOUR_UI::config()->canvasvar_WaveForm.get();
-                                       (*w)->property_fill_color() = ARDOUR_UI::config()->canvasvar_WaveFormFill.get();
-                               }
-                       }
+                        if (_region->muted()) {
+                                wc = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA);
+                        } else {
+                                wc = ARDOUR_UI::config()->canvasvar_WaveForm.get();
+                        }
+                        fc = ARDOUR_UI::config()->canvasvar_WaveFormFill.get();
                }
        }
+
+        for (vector<ArdourCanvas::WaveView*>::iterator w = waves.begin(); w != waves.end(); ++w) {
+                if (_region->muted()) {
+                        (*w)->property_wave_color() = wc;
+                } else {
+                        (*w)->property_wave_color() = wc;
+                        (*w)->property_fill_color() = fc;
+                }
+        }
 }
 
 void
@@ -1424,7 +1364,7 @@ AudioRegionView::show_region_editor ()
 
 
 void
-AudioRegionView::show_fade_line (nframes64_t pos)
+AudioRegionView::show_fade_line (framepos_t pos)
 {
        fade_position_line->property_x1() = trackview.editor().frame_to_pixel (pos);
        fade_position_line->property_x2() = trackview.editor().frame_to_pixel (pos);
@@ -1445,31 +1385,55 @@ AudioRegionView::transients_changed ()
        AnalysisFeatureList analysis_features = _region->transients();
 
        while (feature_lines.size() < analysis_features.size()) {
-               ArdourCanvas::SimpleLine* l = new ArdourCanvas::SimpleLine (*group);
-               l->property_color_rgba() = (guint) ARDOUR_UI::config()->canvasvar_ZeroLine.get();
-               feature_lines.push_back (make_pair(0, l));
+
+               ArdourCanvas::Line* canvas_item = new ArdourCanvas::Line(*group);
+
+               ArdourCanvas::Points points;
+
+               points.push_back(Gnome::Art::Point(-1.0, 2.0)); // first x-coord needs to be a non-normal value
+               points.push_back(Gnome::Art::Point(1.0, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
+
+               canvas_item->property_points() = points;
+               canvas_item->property_width_pixels() = 1;
+               canvas_item->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_ZeroLine.get();
+               canvas_item->property_first_arrowhead() = TRUE;
+               canvas_item->property_last_arrowhead() = TRUE;
+               canvas_item->property_arrow_shape_a() = 11.0;
+               canvas_item->property_arrow_shape_b() = 0.0;
+               canvas_item->property_arrow_shape_c() = 4.0;
+
+               canvas_item->raise_to_top ();
+               canvas_item->show ();
+
+               canvas_item->set_data ("regionview", this);
+               canvas_item->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_feature_line_event), canvas_item, this));
+
+               feature_lines.push_back (make_pair(0, canvas_item));
        }
 
        while (feature_lines.size() > analysis_features.size()) {
-               ArdourCanvas::SimpleLine *line = feature_lines.back().second;
+               ArdourCanvas::Line* line = feature_lines.back().second;
                feature_lines.pop_back ();
                delete line;
        }
 
        AnalysisFeatureList::const_iterator i;
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::Line*> >::iterator l;
 
        for (i = analysis_features.begin(), l = feature_lines.begin(); i != analysis_features.end() && l != feature_lines.end(); ++i, ++l) {
+
+               ArdourCanvas::Points points;
+
+               float *pos = new float;
+               *pos = trackview.editor().frame_to_pixel (*i);
+
+               points.push_back(Gnome::Art::Point(*pos, 2.0)); // first x-coord needs to be a non-normal value
+               points.push_back(Gnome::Art::Point(*pos, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
+
+               (*l).second->property_points() = points;
+               (*l).second->set_data ("position", pos);
+
                (*l).first = *i;
-               (*l).second->property_x1() = trackview.editor().frame_to_pixel (*i);
-               (*l).second->property_x2() = trackview.editor().frame_to_pixel (*i);            
-               (*l).second->property_y1() = 2;
-               (*l).second->property_y2() = _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1;
-               (*l).second->set_data("regionview", this);
-               (*l).second->show ();
-               (*l).second->raise_to_top ();
-               
-               (*l).second->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_feature_line_event), (*l).second, this));
        }
 }
 
@@ -1477,17 +1441,21 @@ void
 AudioRegionView::update_transient(float /*old_pos*/, float new_pos)
 {
        /* Find frame at old pos, calulate new frame then update region transients*/
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::Line*> >::iterator l;
 
        for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
-               /* Simple line x1 has been updated in drag so we compare to new_pos */
-               if (rint(new_pos) == rint((*l).second->property_x1())) {
-                   
-                   nframes64_t old_frame = (*l).first;
-                   nframes64_t new_frame = trackview.editor().pixel_to_frame (new_pos);
+
+               /* Line has been updated in drag so we compare to new_pos */
+
+               float* pos = (float*) (*l).second->get_data ("position");
+
+               if (rint(new_pos) == rint(*pos)) {
+
+                   framepos_t old_frame = (*l).first;
+                   framepos_t new_frame = trackview.editor().pixel_to_frame (new_pos);
 
                    _region->update_transient (old_frame, new_frame);
-                   
+
                    break;
                }
        }
@@ -1497,13 +1465,24 @@ void
 AudioRegionView::remove_transient(float pos)
 {
        /* Find frame at old pos, calulate new frame then update region transients*/
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::Line*> >::iterator l;
 
        for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
-               /* Simple line x1 has been updated in drag so we compare to new_pos */
-               if (rint(pos) == rint((*l).second->property_x1())) {
-                   _region->remove_transient ((*l).first);                 
+
+               /* Line has been updated in drag so we compare to new_pos */
+               float *line_pos = (float*) (*l).second->get_data ("position");
+
+               if (rint(pos) == rint(*line_pos)) {
+                   _region->remove_transient ((*l).first);
                    break;
                }
        }
 }
+
+void
+AudioRegionView::thaw_after_trim ()
+{
+       RegionView::thaw_after_trim ();
+
+       unhide_envelope ();
+}