just use show() and hide() to manage control point visibility in automation lines

No need for the wierd old set_visible()/property_draw() stuff that was a hangover from gnomecanvas.

1 files changed, 7 insertions, 5 deletions

diff --git a/gtk2_ardour/automation_line.cc b/gtk2_ardour/automation_line.cc
index d75243691a..831bdb00f9 100644
--- a/gtk2_ardour/automation_line.cc
+++ b/gtk2_ardour/automation_line.cc
@@ -148,16 +148,19 @@ AutomationLine::show ()
 
 	if (_visible & ControlPoints) {
 		for (vector<ControlPoint*>::iterator i = control_points.begin(); i != control_points.end(); ++i) {
-			(*i)->set_visible (true);
 			(*i)->show ();
 		}
 	} else if (_visible & SelectedControlPoints) {
 		for (vector<ControlPoint*>::iterator i = control_points.begin(); i != control_points.end(); ++i) {
-			(*i)->set_visible ((*i)->get_selected());
+			if ((*i)->get_selected()) {
+				(*i)->show ();
+			} else {
+				(*i)->hide ();
+			}
 		}
 	} else {
 		for (vector<ControlPoint*>::iterator i = control_points.begin(); i != control_points.end(); ++i) {
-			(*i)->set_visible (false);
+			(*i)->hide ();
 		}
 	}
 }
@@ -1215,9 +1218,8 @@ AutomationLine::add_visible_control_point (uint32_t view_index, uint32_t pi, dou
 
 	if (_visible & ControlPoints) {
 		control_points[view_index]->show ();
-		control_points[view_index]->set_visible (true);
 	} else {
-		control_points[view_index]->set_visible (false);
+		control_points[view_index]->hide ();
 	}
 }