Make static analysis happy..

If the copy c'tor of ProcessorSelection was actually used,
assigning the XMLProcessorSelection
    processors = other.processors;
would lead to duplicate free() of the XMLNode*
XMLProcessorSelection would need a dedicated copy c'tor that
duplicates allocates a new XMLNode.
see also #10 at https://www.viva64.com/en/b/0540/

Anyway, the copy c'tor and assignment is never used. This commit makes
this explicit.

1 files changed, 3 insertions, 2 deletions

diff --git a/gtk2_ardour/route_processor_selection.h b/gtk2_ardour/route_processor_selection.h
index d92436dbc4..80ab9e6adf 100644
--- a/gtk2_ardour/route_processor_selection.h
+++ b/gtk2_ardour/route_processor_selection.h
@@ -39,8 +39,6 @@ public:
 
 	RouteProcessorSelection (ARDOUR::SessionHandlePtr&, AxisViewProvider&);
 
-	RouteProcessorSelection& operator= (const RouteProcessorSelection& other);
-
 	void clear ();
 	bool empty();
 
@@ -58,6 +56,9 @@ private:
 	AxisViewProvider& avp;
 	void removed (AxisView*);
 	std::list<AxisView*> add_grouped_tracks (AxisView*) const;
+
+	RouteProcessorSelection& operator= (const RouteProcessorSelection& other);
+	RouteProcessorSelection (RouteProcessorSelection const&);
 };
 
 bool operator==(const RouteProcessorSelection& a, const RouteProcessorSelection& b);