ardour/livetrax
13
0
Fork 0
Code

Prevent accumulating events during freewheel export

Browse Source 
When exporting many ranges, locate and restarting transport for
every ranged a SessionEvent::TransportStateChange event is queued.
Since freewheel export does not process events. Those events
can accumulate:
https://discourse.ardour.org/t/pipewire-jack-export-freeze/107383/18?u=x42

During export those requests can be safely ignored because the only
purpose is to emit a TransportStateChange() signal when the playhead
passes a given location and notify the UI.
This commit is contained in:
Robin Gareus 2022-07-07 18:28:27 +02:00
parent baeb5cbafd
commit ef2c1d990b
Signed by: rgareus
GPG Key ID: A090BCE02CF57F04
1 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
libs/ardour/session_export.cc
View File

@ -346,6 +346,16 @@ Session::process_export_fw (pframes_t nframes)
butler_completed_transport_work (); butler_completed_transport_work ();
} }
{
SessionEvent* ev;
while (pending_events.read (&ev, 1) == 1) {
merge_event (ev);
}
/* remove TransportStateChange events (which otherwise accumulate with each exported range) */
ev = new SessionEvent (SessionEvent::TransportStateChange, SessionEvent::Clear, SessionEvent::Immediate, 0, 0);
merge_event (ev);
}
if (_remaining_latency_preroll > 0) { if (_remaining_latency_preroll > 0) {
samplepos_t remain = std::min ((samplepos_t)nframes, _remaining_latency_preroll); samplepos_t remain = std::min ((samplepos_t)nframes, _remaining_latency_preroll);