From e6c8f104d1ca3da57a3693cb147cde67ab423bfc Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Sat, 28 Oct 2017 15:44:39 +0200 Subject: [PATCH] Fix loading sessions w/new tempo-map. Newly constructed sessions don't save "Tempo-start" property. If there's no "start" node, _legacy_bbt is never explicitly set and the default c'tor is used, which sets bar = 1. The test for legacy session checks bar != 0. All new sessions were processed with fix_legacy_session(), which breaks the tempo-map and makes the session not loadable (duplicate Tempo). --- libs/ardour/tempo.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/libs/ardour/tempo.cc b/libs/ardour/tempo.cc index 93e8a90ee5..ce2d311d39 100644 --- a/libs/ardour/tempo.cc +++ b/libs/ardour/tempo.cc @@ -162,6 +162,7 @@ TempoSection::TempoSection (const XMLNode& node, samplecnt_t sample_rate) { BBT_Time bbt; std::string start_bbt; + _legacy_bbt.bars = 0; // legacy session check compars .bars != 0; default BBT_Time c'tor uses 1. if (node.get_property ("start", start_bbt)) { if (string_to_bbt_time (start_bbt, bbt)) { /* legacy session - start used to be in bbt*/ -- 2.30.2