From 0e427086857037fb224082708117223934c728be Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Wed, 8 Jan 2020 22:21:08 -0700 Subject: expand comment to include (more of?) the full story about uncombining and copies --- libs/ardour/playlist.cc | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'libs/ardour/playlist.cc') diff --git a/libs/ardour/playlist.cc b/libs/ardour/playlist.cc index 0474fc0096..3508eaf0b3 100644 --- a/libs/ardour/playlist.cc +++ b/libs/ardour/playlist.cc @@ -3238,7 +3238,7 @@ Playlist::uncombine (boost::shared_ptr target) RegionFactory::CompoundAssociations& cassocs (RegionFactory::compound_associations()); sampleoffset_t move_offset = 0; - /* there are two possibilities here: + /* there are three possibilities here: 1) the playlist that the playlist source was based on is us, so just add the originals (which belonged to us anyway) back in the right place. @@ -3247,6 +3247,10 @@ Playlist::uncombine (boost::shared_ptr target) is NOT us, so we need to make copies of each of the original regions that we find, and add them instead. + + 3) target region is a copy of a compount region previously + created. In this case we will also need to make copies ot each of + the original regions, and add them instead. */ const bool need_copies = (boost::dynamic_pointer_cast (pls)->owner() != target->id()) || -- cgit v1.2.3