make AutomationList::operator==() private and abort() as its core, because it should never be called/used.

Also make AutomationList::operator=() use ControlList::operator=() rather than try to reimplement it

1 files changed, 4 insertions, 14 deletions

diff --git a/libs/ardour/automation_list.cc b/libs/ardour/automation_list.cc
index 32e69af7ae..e5a4493618 100644
--- a/libs/ardour/automation_list.cc
+++ b/libs/ardour/automation_list.cc
@@ -137,26 +137,16 @@ AutomationList::create_curve_if_necessary()
 	}
 }
 
-bool
-AutomationList::operator== (const AutomationList& other)
-{
-	return _events == other._events;
-}
-
 AutomationList&
 AutomationList::operator= (const AutomationList& other)
 {
 	if (this != &other) {
 
-		_events.clear ();
-
-		for (const_iterator i = other._events.begin(); i != other._events.end(); ++i) {
-			_events.push_back (new Evoral::ControlEvent (**i));
-		}
 
-		_min_yval = other._min_yval;
-		_max_yval = other._max_yval;
-		_default_value = other._default_value;
+		ControlList::operator= (other);
+		_state = other._state;
+		_style = other._style;
+		_touching = other._touching;
 
 		mark_dirty ();
 		maybe_signal_changed ();