From 78c89fcbaeee60e8a85f6f9bb5d8b8aa66bbc663 Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Sun, 19 Feb 2023 22:49:23 +0100 Subject: [PATCH] Fix MIDI Clock transport master (#8750) MIDI Start and Position messages need to leave the clock timestamp alone. They are not relevant to compute BPM. --- libs/ardour/midi_clock_slave.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libs/ardour/midi_clock_slave.cc b/libs/ardour/midi_clock_slave.cc index 19e3aa3df0..b11b2710e0 100644 --- a/libs/ardour/midi_clock_slave.cc +++ b/libs/ardour/midi_clock_slave.cc @@ -324,7 +324,7 @@ MIDIClock_TransportMaster::start (Parser& /*parser*/, samplepos_t timestamp) } _running = true; - current.update (0, timestamp, 0); + current.update (0, current.timestamp, 0); } void @@ -399,7 +399,7 @@ MIDIClock_TransportMaster::position (Parser& /*parser*/, MIDI::byte* message, si DEBUG_TRACE (DEBUG::MidiClock, string_compose ("Song Position: %1 samples: %2\n", position_in_sixteenth_notes, position_in_samples)); - current.update (position_in_samples + midi_port_latency.max, timestamp, 0); + current.update (position_in_samples + midi_port_latency.max, current.timestamp, 0); } bool