smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-16 03:36:32 +01:00
Code Issues Projects Releases Wiki Activity

add explanatory comment re: region duplication positioning

Browse source 
This is currently an unsolved problem-that-may-not-be-a-problem.
This commit is contained in:
Paul Davis 2022-10-23 22:31:31 -06:00
parent 75a990da8c
commit 570aead3b1
1 changed files with 14 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

18
gtk2_ardour/editor_ops.cc
View file

@ -5266,12 +5266,22 @@ Editor::duplicate_some_regions (RegionSelection& regions, float times)
latest_regionviews.clear (); latest_regionviews.clear ();
sigc::connection c = rtv->view()->RegionViewAdded.connect (sigc::mem_fun(*this, &Editor::collect_new_region_view)); sigc::connection c = rtv->view()->RegionViewAdded.connect (sigc::mem_fun(*this, &Editor::collect_new_region_view));
/* end time is an inclusive end; we need to place the /* XXX problem arew here. When duplicating audio regions, the
* duplicated region after that. * next one must be positioned 1 sample after the end of the
* prior region.
*
* For MIDI, this is not requiredd, because we allow MIDI
* events to be simultaneous without affecting semantics.
*
* HOWEVER, opaque MIDI regions will cause an underlying note
* off at the end of earlier region to not be delivered. In
* reality, the note tracker will fix this by resolving all
* on-notes at the region's end, but that does not seem like a
* good solution.
*/ */
timepos_t position = end_time.increment(); timepos_t position = end_time + (start_time.distance (r->position()));
position += start_time.distance (r->position()); position.increment ();
playlist = (*i)->region()->playlist(); playlist = (*i)->region()->playlist();