From 2131adec9d53427689c63e2abb676cb63c45659c Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Fri, 8 Jan 2021 13:54:26 -0700 Subject: [PATCH] remove another NUTEMPO warning by replacing it with an explanatory comment --- libs/ardour/location.cc | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/libs/ardour/location.cc b/libs/ardour/location.cc index 8078352980..e93fca4f06 100644 --- a/libs/ardour/location.cc +++ b/libs/ardour/location.cc @@ -82,7 +82,17 @@ Location::Location (Session& s, timepos_t const & start, timepos_t const & end, , _locked (false) , _timestamp (time (0)) { -#warning NUTEMPO FIXME drop this and ensure that timepos start/end use correct domain in caller for this constructor + + /* it would be nice if the caller could ensure that the start and end + values simply use the correct domain, but that would involve + enforcing/checking that at every place where we create a + Location. So instead we centralize this here. + + NUTEMPO: it might make sense to switch time domains when + happens, but it's not clear what the might be? Maybe + changing some setting of the tempo map. + */ + if (s.config.get_glue_new_markers_to_bars_and_beats()) { set_position_time_domain (Temporal::BeatTime); } else {