Draw all automation points, rather than trying to use a heuristic to show some

at any given zoom level. 

This works because (1) automation data is obtained by sampling, and we control
the sampling interval (2) automation data is filtered to remove co-linear
points before becoming part of a ControlList. 

This commit removes some of the hairiest code in Ardour's GUI, which was
responsible for figuring out which invisible control points were affected by an
edit. The change is based on an experiment in mixbus done by Ben Loftis.


git-svn-id: svn://localhost/ardour2/branches/3.0@11798 d708f5d6-7413-0410-9779-e7cbd77b26cf

1 files changed, 8 insertions, 10 deletions

diff --git a/gtk2_ardour/control_point.h b/gtk2_ardour/control_point.h
index 40eb7b9179..603d062ec4 100644
--- a/gtk2_ardour/control_point.h
+++ b/gtk2_ardour/control_point.h
@@ -84,20 +84,18 @@ class ControlPoint : public Selectable
 	AutomationLine&                  line()  const { return _line; }
 
   private:
-	ArdourCanvas::SimpleRect* _item;
-
-	AutomationLine& _line;
-
+	ArdourCanvas::SimpleRect*        _item;
+	AutomationLine&                  _line;
 	ARDOUR::AutomationList::iterator _model;
-	uint32_t _view_index;
-	bool     _can_slide;
+	uint32_t                         _view_index;
+	bool                             _can_slide;
+	double                           _x;
+	double                           _y;
+	double                           _size;
+	ShapeType                        _shape;
 
 	virtual bool event_handler (GdkEvent*);
 
-	double    _x;
-	double    _y;
-	double    _size;
-	ShapeType _shape;
 };