smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-21 22:26:29 +01:00
Code Issues Projects Releases Wiki Activity

AudioPlaylist doesn't double-notify that contents have changed if bounds have changed.

Browse source
This commit is contained in:
nick_m 2016-06-27 04:35:16 +10:00
parent 3ee7972069
commit 4acd365e42
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
libs/ardour/audio_playlist.cc
View file

@ -335,6 +335,11 @@ AudioPlaylist::region_changed (const PropertyChange& what_changed, boost::shared
return false; return false;
} }
PropertyChange bounds;
bounds.add (Properties::start);
bounds.add (Properties::position);
bounds.add (Properties::length);
PropertyChange our_interests; PropertyChange our_interests;
our_interests.add (Properties::fade_in_active); our_interests.add (Properties::fade_in_active);
@ -348,8 +353,8 @@ AudioPlaylist::region_changed (const PropertyChange& what_changed, boost::shared
bool parent_wants_notify; bool parent_wants_notify;
parent_wants_notify = Playlist::region_changed (what_changed, region); parent_wants_notify = Playlist::region_changed (what_changed, region);
/* if bounds changed, we have already done notify_contents_changed ()*/
if (parent_wants_notify || (what_changed.contains (our_interests))) { if ((parent_wants_notify || what_changed.contains (our_interests)) && !what_changed.contains (bounds)) {
notify_contents_changed (); notify_contents_changed ();
} }