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

don't waste time with sending MTC or MMC after stop if we're disconnected from the engine.

Browse Source 
Also, add note about threading oddities with send_mmc_locate() being called from ::non_realtime_stop() which gets
called from a *realtime* context.
This commit is contained in:
Paul Davis 2013-08-09 18:06:08 -04:00
parent cec2a3da36
commit 4b9289e17e
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
libs/ardour/session_transport.cc
View File

@ -609,12 +609,23 @@ Session::non_realtime_stop (bool abort, int on_entry, bool& finished)
have_looped = false;
if (!_engine.freewheeling()) {
/* don't bother with this stuff if we're disconnected from the engine,
because there will be no process callbacks to deliver stuff from
*/
if (_engine.connected() && !_engine.freewheeling()) {
// need to queue this in the next RT cycle
_send_timecode_update = true;
if (!dynamic_cast<MTC_Slave*>(_slave)) {
AudioEngine::instance()->mmc().send (MIDI::MachineControlCommand (MIDI::MachineControl::cmdStop));
/* This (::non_realtime_stop()) gets called by main
process thread, which will lead to confusion
when calling AsyncMIDIPort::write().
Something must be done. XXX
*/
send_mmc_locate (_transport_frame);
}
}