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

fix crash in a debug build caused by trying to print debug info about a negative time value that will never be used

Browse source
This commit is contained in:
Paul Davis 2022-07-03 16:53:24 -06:00
parent 0295e05d04
commit d932d9365f
1 changed files with 9 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

17
libs/ardour/triggerbox.cc
View file

@ -3650,6 +3650,15 @@ TriggerBox::run (BufferSet& bufs, samplepos_t start_sample, samplepos_t end_samp
return; return;
} }
/* STEP TWO: if latency compensation tells us that we haven't really
* started yet, do nothing, because we can't make sense of a negative
* start sample time w.r.t the tempo map.
*/
if (start_sample < 0) {
return;
}
#ifndef NDEBUG #ifndef NDEBUG
{ {
Temporal::TempoMap::SharedPtr __tmap (Temporal::TempoMap::use()); Temporal::TempoMap::SharedPtr __tmap (Temporal::TempoMap::use());
@ -3663,14 +3672,6 @@ TriggerBox::run (BufferSet& bufs, samplepos_t start_sample, samplepos_t end_samp
bool allstop = _requests.stop_all.exchange (false); bool allstop = _requests.stop_all.exchange (false);
/* STEP TWO: if latency compensation tells us that we haven't really
* started yet, do nothing, because we can't make sense of a negative
* start sample time w.r.t the tempo map.
*/
if (start_sample < 0) {
return;
}
/* STEP THREE: triggers in audio tracks need a MIDI sidechain to be /* STEP THREE: triggers in audio tracks need a MIDI sidechain to be
* able to receive inbound MIDI for triggering etc. This needs to run * able to receive inbound MIDI for triggering etc. This needs to run