ardour/livetrax
13
0
Fork 0
Code

libardour: change flow control to return early rather than use condition scope in Region::set_length()

Browse Source
This commit is contained in:
Paul Davis 2020-12-08 20:35:40 -07:00
parent 9ca376c232
commit 9e58c43ecd
1 changed files with 29 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

59
libs/ardour/region.cc
View File

@ -465,42 +465,41 @@ Region::set_length (timecnt_t const & len)
return;
}
if (_length != len && !len.zero()) {
/* check that the current _position wouldn't make the new
* length impossible.
*/
if (timepos_t::max (len.time_domain()).earlier (len) < _position) {
return;
}
timecnt_t l = len;
if (!verify_length (l)) {
return;
}
set_length_internal (l);
_whole_file = false;
first_edit ();
maybe_uncopy ();
maybe_invalidate_transients ();
if (!property_changes_suspended()) {
recompute_at_end ();
}
send_change (Properties::length);
if (_length == len || len.zero()) {
return;
}
/* check that the current _position wouldn't make the new
* length impossible.
*/
if (timepos_t::max (len.time_domain()).earlier (len) < _position) {
return;
}
timecnt_t l = len;
if (!verify_length (l)) {
return;
}
set_length_internal (l);
_whole_file = false;
first_edit ();
maybe_uncopy ();
maybe_invalidate_transients ();
if (!property_changes_suspended()) {
recompute_at_end ();
}
send_change (Properties::length);
}
void
Region::set_length_internal (timecnt_t const & len)
{
std::cerr << "Region::set_length_internal() len = " << len << std::endl;
_last_length = _length;
_length = len;
}