Fix deprecated-copy warnings

It's long been a guideline (and IIRC a Weff-c++ warning) that either all, or
none, of the copy methods should be defined, but this became a standard warning
in GCC9.  Presumably to account for a later language change though I'm not sure
which.

I don't remember why the ChanMapping copy constructor can't just be a simple
copy (it's just a map of POD), but figure it's safer to just copy what that
does.

1 files changed, 6 insertions, 0 deletions

diff --git a/libs/surfaces/contourdesign/contourdesign.h b/libs/surfaces/contourdesign/contourdesign.h
index c1d6ad9bcc..f24da723da 100644
--- a/libs/surfaces/contourdesign/contourdesign.h
+++ b/libs/surfaces/contourdesign/contourdesign.h
@@ -53,6 +53,12 @@ struct JumpDistance {
 	JumpDistance () : value (1.0), unit (BEATS) {}
 	JumpDistance (double v, JumpUnit u) : value (v), unit (u) {}
 	JumpDistance (const JumpDistance& o) : value (o.value), unit (o.unit) {}
+	JumpDistance& operator= (const JumpDistance& o) {
+		value = o.value;
+		unit = o.unit;
+		return *this;
+	}
+
 	double value;
 	JumpUnit unit;
 };