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

fix behavior of Region::mid_thaw() to match pre-nutempo behavior

Browse source 
We no longer have a distinct "position" property, because the "length" property
defines both extent and position. But we must not call ::recompute_at_start()
when the position has not changed, so here we use Region::last_length() to
simulate the position property.
This commit is contained in:
Paul Davis 2022-08-25 15:13:24 -06:00
parent 00a1b7cb24
commit 13d10582f0
1 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

5
libs/ardour/region.cc
View file

@ -999,6 +999,7 @@ Region::trim_to_internal (timepos_t const & pos, timecnt_t const & len)
if (!property_changes_suspended()) { if (!property_changes_suspended()) {
_last_length = _length; _last_length = _length;
} }
std::cerr << "change len\n";
set_length_internal (nl); set_length_internal (nl);
what_changed.add (Properties::length); what_changed.add (Properties::length);
} }
@ -1402,7 +1403,9 @@ void
Region::mid_thaw (const PropertyChange& what_changed) Region::mid_thaw (const PropertyChange& what_changed)
{ {
if (what_changed.contains (Properties::length)) { if (what_changed.contains (Properties::length)) {
recompute_at_start (); if (length().position() != last_position()) {
recompute_at_start ();
}
recompute_at_end (); recompute_at_end ();
} }
} }