fix crash when copy'ing latent plugins

[ardour.git] / libs / canvas / line.cc

diff --git a/libs/canvas/line.cc b/libs/canvas/line.cc
index 09f9061c8593653b601c3d6791c7c42918f06982..49d2987e3d6b1c4a9adb5a59198ffcc25cd1e1a0 100644 (file)
--- a/libs/canvas/line.cc
+++ b/libs/canvas/line.cc
@@ -29,18 +29,21 @@
 using namespace std;
 using namespace ArdourCanvas;
 
-Line::Line (Group* parent)
-       : Item (parent)
-       , Outline (parent)
+Line::Line (Canvas* c)
+       : Item (c)
 {
+}
 
+Line::Line (Item* parent)
+       : Item (parent)
+{
 }
 
 void
 Line::compute_bounding_box () const
 {
        Rect bbox;
-       
+
        bbox.x0 = min (_points[0].x, _points[1].x);
        bbox.y0 = min (_points[0].y, _points[1].y);
        bbox.x1 = max (_points[0].x, _points[1].x);
@@ -63,7 +66,7 @@ Line::render (Rect const & /*area*/, Cairo::RefPtr<Cairo::Context> context) cons
        if (_outline_width <= 1.0) {
                /* See Cairo FAQ on single pixel lines to understand why we add 0.5
                 */
-               
+
                const Duple half_a_pixel (0.5, 0.5);
                p0 = p0.translate (half_a_pixel);
                p1 = p1.translate (half_a_pixel);
@@ -77,53 +80,55 @@ Line::render (Rect const & /*area*/, Cairo::RefPtr<Cairo::Context> context) cons
 void
 Line::set (Duple a, Duple b)
 {
-       begin_change ();
+       if (a != _points[0] || b != _points[1]) {
+               begin_change ();
 
-       _points[0] = a;
-       _points[1] = b;
+               _points[0] = a;
+               _points[1] = b;
 
-       _bounding_box_dirty = true;
-       end_change ();
-
-       DEBUG_TRACE (PBD::DEBUG::CanvasItemsDirtied, "canvas item dirty: line change\n");
+               _bounding_box_dirty = true;
+               end_change ();
+       }
 }
 
 void
 Line::set_x (Coord x0, Coord x1)
 {
-       begin_change ();
-       
-       _points[0].x = x0;
-       _points[1].x = x1;
+       if (x0 != _points[0].x || x1 != _points[1].x) {
+               begin_change ();
 
-       _bounding_box_dirty = true;
-       end_change ();
+               _points[0].x = x0;
+               _points[1].x = x1;
 
-       DEBUG_TRACE (PBD::DEBUG::CanvasItemsDirtied, "canvas item dirty: line change\n");
-}      
+               _bounding_box_dirty = true;
+               end_change ();
+       }
+}
 
 void
 Line::set_x0 (Coord x0)
 {
-       begin_change ();
-       
-       _points[0].x = x0;
+       if (x0 != _points[0].x) {
+               begin_change ();
 
-       _bounding_box_dirty = true;
-       end_change ();
+               _points[0].x = x0;
 
-       DEBUG_TRACE (PBD::DEBUG::CanvasItemsDirtied, "canvas item dirty: line change\n");
+               _bounding_box_dirty = true;
+               end_change ();
+       }
 }
 
 void
 Line::set_y0 (Coord y0)
 {
-       begin_change ();
+       if (y0 != _points[0].y) {
+               begin_change ();
 
-       _points[0].y = y0;
+               _points[0].y = y0;
 
-       _bounding_box_dirty = true;
-       end_change ();
+               _bounding_box_dirty = true;
+               end_change ();
+       }
 
        DEBUG_TRACE (PBD::DEBUG::CanvasItemsDirtied, "canvas item dirty: line change\n");
 }
@@ -131,33 +136,33 @@ Line::set_y0 (Coord y0)
 void
 Line::set_x1 (Coord x1)
 {
-       begin_change ();
-
-       _points[1].x = x1;
+       if (x1 != _points[1].x) {
+               begin_change ();
 
-       _bounding_box_dirty = true;
-       end_change ();
+               _points[1].x = x1;
 
-       DEBUG_TRACE (PBD::DEBUG::CanvasItemsDirtied, "canvas item dirty: line change\n");
+               _bounding_box_dirty = true;
+               end_change ();
+       }
 }
 
 void
 Line::set_y1 (Coord y1)
 {
-       begin_change ();
-
-       _points[1].y = y1;
+       if (y1 != _points[1].y) {
+               begin_change ();
 
-       _bounding_box_dirty = true;
-       end_change ();
+               _points[1].y = y1;
 
-       DEBUG_TRACE (PBD::DEBUG::CanvasItemsDirtied, "canvas item dirty: line change\n");
+               _bounding_box_dirty = true;
+               end_change ();
+       }
 }
 
 bool
 Line::covers (Duple const & point) const
 {
-       const Duple p = canvas_to_item (point);
+       const Duple p = window_to_item (point);
        static const Distance threshold = 2.0;
 
        /* this quick check works for vertical and horizontal lines, which are
@@ -178,7 +183,7 @@ Line::covers (Duple const & point) const
        double t;
        Duple a (_points[0]);
        Duple b (_points[1]);
-       const Rect visible (_canvas->visible_area());
+       const Rect visible (window_to_item (_canvas->visible_area()));
 
        /*
           Clamp the line endpoints to the visible area of the canvas. If we do