diff options
author | nick_m <mainsbridge@gmail.com> | 2016-07-02 05:38:25 +1000 |
---|---|---|
committer | nick_m <mainsbridge@gmail.com> | 2016-07-10 02:18:37 +1000 |
commit | 828cb52f86410da447cb93e5033ccaf4ea43b4cc (patch) | |
tree | 7053553599429e3dfd7c8d6469df0dd41d0dd469 /libs/ardour/tempo.cc | |
parent | 39b70ceced2635358030b373a6e14a44e81b3816 (diff) |
Fail properly when adding a new audio locked meter's tempo.
Diffstat (limited to 'libs/ardour/tempo.cc')
-rw-r--r-- | libs/ardour/tempo.cc | 24 |
1 files changed, 18 insertions, 6 deletions
diff --git a/libs/ardour/tempo.cc b/libs/ardour/tempo.cc index 2d1c10f73e..6bf734c145 100644 --- a/libs/ardour/tempo.cc +++ b/libs/ardour/tempo.cc @@ -980,18 +980,23 @@ TempoMap::add_tempo_locked (const Tempo& tempo, double pulse, framepos_t frame { TempoSection* t = new TempoSection (pulse, frame, tempo.beats_per_minute(), tempo.note_type(), type, pls); t->set_locked_to_meter (locked_to_meter); - + bool solved = false; do_insert (t); if (recompute) { if (pls == AudioTime) { - solve_map_frame (_metrics, t, t->frame()); + solved = solve_map_frame (_metrics, t, t->frame()); } else { - solve_map_pulse (_metrics, t, t->pulse()); + solved = solve_map_pulse (_metrics, t, t->pulse()); } recompute_meters (_metrics); } + if (!solved) { + remove_tempo_locked (*t); + return 0; + } + return t; } @@ -1046,17 +1051,24 @@ TempoMap::replace_meter (const MeterSection& ms, const Meter& meter, const BBT_T } MeterSection* -TempoMap::add_meter_locked (const Meter& meter, double beat, const Timecode::BBT_Time& where, framepos_t frame, PositionLockStyle pls, bool recompute) +TempoMap::add_meter_locked (const Meter& meter, double beat, const BBT_Time& where, framepos_t frame, PositionLockStyle pls, bool recompute) { const MeterSection& prev_m = meter_section_at_frame_locked (_metrics, frame - 1); const double pulse = ((where.bars - prev_m.bbt().bars) * (prev_m.divisions_per_bar() / prev_m.note_divisor())) + prev_m.pulse(); + const BBT_Time bbt (where); + TempoSection* mlt = 0; if (pls == AudioTime) { /* add meter-locked tempo */ - add_tempo_locked (tempo_at_frame_locked (_metrics, frame), pulse, frame, TempoSection::Ramp, AudioTime, true, true); + mlt = add_tempo_locked (tempo_at_frame_locked (_metrics, frame), pulse, frame, TempoSection::Ramp, AudioTime, true, true); + + if (!mlt) { + return 0; + } + } - MeterSection* new_meter = new MeterSection (pulse, frame, beat, where, meter.divisions_per_bar(), meter.note_divisor(), pls); + MeterSection* new_meter = new MeterSection (pulse, frame, beat, bbt, meter.divisions_per_bar(), meter.note_divisor(), pls); do_insert (new_meter); |