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, 4 insertions, 1 deletions

diff --git a/gtk2_ardour/processor_selection.h b/gtk2_ardour/processor_selection.h
index fc3a133ef1..7de1c4735e 100644
--- a/gtk2_ardour/processor_selection.h
+++ b/gtk2_ardour/processor_selection.h
@@ -67,7 +67,6 @@ class ProcessorSelection : public PBD::ScopedConnectionList, public sigc::tracka
 	XMLProcessorSelection processors;
 	sigc::signal<void> ProcessorsChanged;
 
-	ProcessorSelection& operator= (const ProcessorSelection& other);
 
 	void clear ();
 	bool empty();
@@ -76,6 +75,10 @@ class ProcessorSelection : public PBD::ScopedConnectionList, public sigc::tracka
 	void add (XMLNode* node);
 
 	void clear_processors ();
+
+	private:
+	ProcessorSelection& operator= (const ProcessorSelection& other);
+	ProcessorSelection (ProcessorSelection const&);
 };
 
 bool operator==(const ProcessorSelection& a, const ProcessorSelection& b);