ardour/ardour
ardour
/
ardour
13
1
Fork 0
Code Pull Requests Releases Activity

fix exception when loading a region whose length exceeds its source and the time domains do not match 

See code comment for more info
This commit is contained in:
Paul Davis 2022-08-04 09:50:39 -06:00
parent 32d99aa64f
commit d10557fc8c
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
libs/ardour/region.cc
View File

@ -1346,7 +1346,14 @@ Region::_set_state (const XMLNode& node, int version, PropertyChange& what_chang
*/
if (!_sources.empty() && _type == DataType::AUDIO) {
if ((length().time_domain() == Temporal::AudioTime) && (length().distance() > _sources.front()->length())) {
/* both region and source length must be audio time for this to
actually be a case of a destructive track/region. And also
for the operator>() in the 3rd conditional clause to be
legal, since these values are timepos_t IS-A int62_t and
that requires the same "flagged" status (i.e. domain) to be
match.
*/
if ((length().time_domain() == Temporal::AudioTime) && (_sources.front()->length().time_domain() == Temporal::AudioTime) && (length().distance() > _sources.front()->length())) {
_length = timecnt_t (start().distance (_sources.front()->length()), _length.val().position());
}
}