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/ardour/ardour/buffer_set.h | |
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/ardour/ardour/buffer_set.h')
-rw-r--r-- | libs/ardour/ardour/buffer_set.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/libs/ardour/ardour/buffer_set.h b/libs/ardour/ardour/buffer_set.h index 85041b6728..3a4b9c8344 100644 --- a/libs/ardour/ardour/buffer_set.h +++ b/libs/ardour/ardour/buffer_set.h @@ -144,6 +144,8 @@ public: template <typename BS, typename B> class iterator_base { public: + iterator_base(const iterator_base& other) + : _set(other._set), _type(other._type), _index(other._index) {} B& operator*() { return (B&)_set.get_available(_type, _index); } B* operator->() { return &(B&)_set.get_available(_type, _index); } iterator_base<BS,B>& operator++() { ++_index; return *this; } // yes, prefix only |