summaryrefslogtreecommitdiff
path: root/libs/ardour/ardour
diff options
context:
space:
mode:
authorRobin Gareus <robin@gareus.org>2020-02-05 22:31:22 +0100
committerRobin Gareus <robin@gareus.org>2020-02-06 17:30:22 +0100
commit01a75c04b1912856d60629250b80873199449811 (patch)
tree6d809a8b53259a91fdf6a3b163c98332ab58841c /libs/ardour/ardour
parent016970df25ef27f9d638bb3cd70204da0ff144de (diff)
Fix rt-safety (const reference, not const copy)
PluginInsert (and a few other places) iterate over channel mappings using a const iterator. However mappings() allocated memory since it was copied
Diffstat (limited to 'libs/ardour/ardour')
-rw-r--r--libs/ardour/ardour/chan_mapping.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/libs/ardour/ardour/chan_mapping.h b/libs/ardour/ardour/chan_mapping.h
index 64ffc32890..07b623f8e1 100644
--- a/libs/ardour/ardour/chan_mapping.h
+++ b/libs/ardour/ardour/chan_mapping.h
@@ -106,8 +106,8 @@ public:
typedef std::map<uint32_t, uint32_t> TypeMapping;
typedef std::map<DataType, TypeMapping> Mappings;
- Mappings mappings() { return _mappings; }
- const Mappings mappings() const { return _mappings; }
+ Mappings mappings() { return _mappings; }
+ const Mappings& mappings() const { return _mappings; }
bool operator==(const ChanMapping& other) const {
return (_mappings == other._mappings);