diff options
author | David Robillard <d@drobilla.net> | 2019-12-09 22:59:40 +0100 |
---|---|---|
committer | David Robillard <d@drobilla.net> | 2019-12-09 23:25:59 +0100 |
commit | 8ec3e5fb540f7d79a6f6ca2bd113bbc3743e954b (patch) | |
tree | 80bbdd3c69b2d8d1de6584a2493f3774db3edcf0 /libs/midi++2/midi++ | |
parent | 39bdde42504984b7367a0709789c4785e37bf6ea (diff) |
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.
Diffstat (limited to 'libs/midi++2/midi++')
-rw-r--r-- | libs/midi++2/midi++/midnam_patch.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/libs/midi++2/midi++/midnam_patch.h b/libs/midi++2/midi++/midnam_patch.h index e24ba169b5..d95a9e1ff0 100644 --- a/libs/midi++2/midi++/midnam_patch.h +++ b/libs/midi++2/midi++/midnam_patch.h @@ -52,6 +52,11 @@ public: , _program(std::max(0, std::min(program_num, 127))) {} + inline PatchPrimaryKey(const PatchPrimaryKey& id) + : _bank(id._bank) + , _program(id._program) + {} + inline PatchPrimaryKey& operator=(const PatchPrimaryKey& id) { _bank = id._bank; _program = id._program; |