fix mute automation for busses & consolidate code.
[ardour.git] / libs / ardour / ardour / tempo.h
index 813c96e8afffea9a28543bb4c7888c8b3ca1abc5..a04f1bb072b19033f68f10d7b304a962ae0b0a74 100644 (file)
@@ -55,16 +55,22 @@ class LIBARDOUR_API Tempo {
        Tempo (double bpm, double type=4.0) // defaulting to quarter note
                : _beats_per_minute (bpm), _note_type(type) {}
 
+       /* ..or more aptly 'pulse divisions per minute'.
+          Nothing to do with actual beats, which are defined by the meter and tempo.
+       */
        double beats_per_minute () const { return _beats_per_minute; }
-
-       double ticks_per_minute () const { return _beats_per_minute * Timecode::BBT_Time::ticks_per_beat;}
-       double note_type () const { return _note_type;}
+       void set_beats_per_minute (double bpm) { _beats_per_minute = bpm; }
+       double note_type () const { return _note_type; }
+       double pulses_per_minute () const { return _beats_per_minute / _note_type; }
        /** audio samples per beat
         * @param sr samplerate
         */
        double frames_per_beat (framecnt_t sr) const {
                return (60.0 * sr) / _beats_per_minute;
        }
+       double frames_per_pulse (framecnt_t sr) const {
+               return (_note_type * 60.0 * sr) / _beats_per_minute;
+       }
 
   protected:
        double _beats_per_minute;
@@ -99,16 +105,13 @@ class LIBARDOUR_API Meter {
 /** A section of timeline with a certain Tempo or Meter. */
 class LIBARDOUR_API MetricSection {
   public:
-       MetricSection (double beat)
-               : _beat (beat), _frame (0), _movable (true), _position_lock_style (PositionLockStyle::MusicTime) {}
-       MetricSection (framepos_t frame)
-               : _beat (0.0), _frame (frame), _movable (true), _position_lock_style (PositionLockStyle::AudioTime) {}
+       MetricSection (double pulse, framepos_t frame, PositionLockStyle pls)
+               : _pulse (pulse), _frame (frame), _movable (true), _position_lock_style (pls) {}
 
        virtual ~MetricSection() {}
 
-       const double& beat () const { return _beat; }
-       const double tick () const { return _beat * Timecode::BBT_Time::ticks_per_beat; }
-       void set_beat (double beat) { _beat = beat;}
+       const double& pulse () const { return _pulse; }
+       void set_pulse (double pulse) { _pulse = pulse; }
 
        framepos_t frame() const { return _frame; }
        virtual void set_frame (framepos_t f) {
@@ -128,7 +131,7 @@ class LIBARDOUR_API MetricSection {
        void set_position_lock_style (PositionLockStyle ps) { _position_lock_style = ps; }
 
 private:
-       double             _beat;
+       double             _pulse;
        framepos_t         _frame;
        bool               _movable;
        PositionLockStyle  _position_lock_style;
@@ -137,10 +140,9 @@ private:
 /** A section of timeline with a certain Meter. */
 class LIBARDOUR_API MeterSection : public MetricSection, public Meter {
   public:
-       MeterSection (double beat, const Timecode::BBT_Time& bbt, double bpb, double note_type)
-               : MetricSection (beat), Meter (bpb, note_type), _bbt (bbt) {}
-       MeterSection (framepos_t frame, double bpb, double note_type)
-               : MetricSection (frame), Meter (bpb, note_type), _bbt (1, 1, 0) {}
+       MeterSection (double pulse, framepos_t frame, double beat, const Timecode::BBT_Time& bbt, double bpb, double note_type, PositionLockStyle pls)
+               : MetricSection (pulse, frame, pls), Meter (bpb, note_type), _bbt (bbt),  _beat (beat) {}
+
        MeterSection (const XMLNode&);
 
        static const std::string xml_state_node_name;
@@ -148,13 +150,17 @@ class LIBARDOUR_API MeterSection : public MetricSection, public Meter {
        XMLNode& get_state() const;
 
        void set_beat (std::pair<double, Timecode::BBT_Time>& w) {
-               MetricSection::set_beat (w.first);
+               _beat = w.first;
                _bbt = w.second;
        }
 
        const Timecode::BBT_Time& bbt() const { return _bbt; }
+        const double& beat () const { return _beat; }
+       void set_beat (double beat) { _beat = beat; }
+
 private:
        Timecode::BBT_Time _bbt;
+       double _beat;
 };
 
 /** A section of timeline with a certain Tempo. */
@@ -165,47 +171,45 @@ class LIBARDOUR_API TempoSection : public MetricSection, public Tempo {
                Constant,
        };
 
-       TempoSection (const double& beat, double qpm, double note_type, Type tempo_type)
-               : MetricSection (beat), Tempo (qpm, note_type), _bar_offset (-1.0), _type (tempo_type), _c_func (0.0)  {}
-       TempoSection (framepos_t frame, double qpm, double note_type, Type tempo_type)
-               : MetricSection (frame), Tempo (qpm, note_type), _bar_offset (-1.0), _type (tempo_type), _c_func (0.0) {}
+       TempoSection (const double& pulse, const framepos_t& frame, double qpm, double note_type, Type tempo_type, PositionLockStyle pls)
+               : MetricSection (pulse, frame, pls), Tempo (qpm, note_type), _type (tempo_type), _c_func (0.0), _active (true), _locked_to_meter (false)  {}
+
        TempoSection (const XMLNode&);
 
        static const std::string xml_state_node_name;
 
        XMLNode& get_state() const;
 
-       void update_bar_offset_from_bbt (const Meter&);
-       void update_bbt_time_from_bar_offset (const Meter&);
-       double bar_offset() const { return _bar_offset; }
+       double c_func () const { return _c_func; }
+       void set_c_func (double c_func) { _c_func = c_func; }
 
        void set_type (Type type);
        Type type () const { return _type; }
 
-       double tempo_at_frame (framepos_t frame, framecnt_t frame_rate) const;
-       framepos_t frame_at_tempo (double tempo, double beat, framecnt_t frame_rate) const;
+       bool active () const { return _active; }
+       void set_active (bool yn) { _active = yn; }
 
-       double tempo_at_beat (double beat) const;
-       double beat_at_tempo (double tempo, framepos_t frame, framecnt_t frame_rate) const;
+       bool locked_to_meter ()  const { return _locked_to_meter; }
+       void set_locked_to_meter (bool yn) { _locked_to_meter = yn; }
 
-       double beat_at_frame (framepos_t frame, framecnt_t frame_rate) const;
-       framepos_t frame_at_beat (double beat, framecnt_t frame_rate) const;
+       double tempo_at_frame (const framepos_t& frame, const framecnt_t& frame_rate) const;
+       framepos_t frame_at_tempo (const double& ppm, const double& beat, const framecnt_t& frame_rate) const;
 
-       double tick_at_frame (framepos_t frame, framecnt_t frame_rate) const;
-       framepos_t frame_at_tick (double tick, framecnt_t frame_rate) const;
+       double tempo_at_pulse (const double& pulse) const;
+       double pulse_at_tempo (const double& ppm, const framepos_t& frame, const framecnt_t& frame_rate) const;
 
-       double compute_c_func_beat (double end_bpm, double end_beat, framecnt_t frame_rate);
-       double compute_c_func_frame (double end_bpm, framepos_t end_frame, framecnt_t frame_rate) const;
+       double pulse_at_frame (const framepos_t& frame, const framecnt_t& frame_rate) const;
+       frameoffset_t frame_at_pulse (const double& pulse, const framecnt_t& frame_rate) const;
 
-       double get_c_func () const { return _c_func; }
-       void set_c_func (double c_func) { _c_func = c_func; }
+       double compute_c_func_pulse (const double& end_bpm, const double& end_pulse, const framecnt_t& frame_rate);
+       double compute_c_func_frame (const double& end_bpm, const framepos_t& end_frame, const framecnt_t& frame_rate) const;
 
        Timecode::BBT_Time legacy_bbt () { return _legacy_bbt; }
 
   private:
 
-       framecnt_t minute_to_frame (double time, framecnt_t frame_rate) const;
-       double frame_to_minute (framecnt_t frame, framecnt_t frame_rate) const;
+       framepos_t minute_to_frame (const double& time, const framecnt_t& frame_rate) const;
+       double frame_to_minute (const framepos_t& frame, const framecnt_t& frame_rate) const;
 
        /*  tempo ramp functions. zero-based with time in minutes,
         * 'tick tempo' in ticks per minute and tempo in bpm.
@@ -214,17 +218,14 @@ class LIBARDOUR_API TempoSection : public MetricSection, public Tempo {
        double a_func (double end_tpm, double c_func) const;
        double c_func (double end_tpm, double end_time) const;
 
-       double tick_tempo_at_time (double time) const;
-       double time_at_tick_tempo (double tick_tempo) const;
-
-       double tick_tempo_at_tick (double tick) const;
-       double tick_at_tick_tempo (double tick_tempo) const;
+       double pulse_tempo_at_time (const double& time) const;
+       double time_at_pulse_tempo (const double& pulse_tempo) const;
 
-       double tick_at_time (double time) const;
-       double time_at_tick (double tick) const;
+       double pulse_tempo_at_pulse (const double& pulse) const;
+       double pulse_at_pulse_tempo (const double& pulse_tempo) const;
 
-       double beat_at_time (double time) const;
-       double time_at_beat (double beat) const;
+       double pulse_at_time (const double& time) const;
+       double time_at_pulse (const double& pulse) const;
 
        /* this value provides a fractional offset into the bar in which
           the tempo section is located in. A value of 0.0 indicates that
@@ -234,9 +235,10 @@ class LIBARDOUR_API TempoSection : public MetricSection, public Tempo {
           this enables us to keep the tempo change at the same relative
           position within the bar if/when the meter changes.
        */
-       double _bar_offset;
        Type _type;
        double _c_func;
+       bool _active;
+       bool _locked_to_meter;
        Timecode::BBT_Time _legacy_bbt;
 };
 
@@ -253,7 +255,7 @@ class LIBARDOUR_API TempoMetric {
        void set_tempo (const Tempo& t)              { _tempo = &t; }
        void set_meter (const Meter& m)              { _meter = &m; }
        void set_frame (framepos_t f)                { _frame = f; }
-       void set_beat (const double& t)              { _beat = t; }
+       void set_pulse (const double& p)             { _pulse = p; }
 
        void set_metric (const MetricSection* section) {
                const MeterSection* meter;
@@ -264,20 +266,20 @@ class LIBARDOUR_API TempoMetric {
                        set_tempo(*tempo);
                }
 
-               set_frame(section->frame());
-               set_beat(section->beat());
+               set_frame (section->frame());
+               set_pulse (section->pulse());
        }
 
        const Meter&              meter() const { return *_meter; }
        const Tempo&              tempo() const { return *_tempo; }
        framepos_t                frame() const { return _frame; }
-       const double&             beat() const { return _beat; }
+       const double&             pulse() const { return _pulse; }
 
   private:
        const Meter*       _meter;
        const Tempo*       _tempo;
        framepos_t         _frame;
-       double             _beat;
+       double             _pulse;
 };
 
 /** Tempo Map - mapping of timecode to musical time.
@@ -298,14 +300,15 @@ class LIBARDOUR_API TempoMap : public PBD::StatefulDestructible
 
        struct BBTPoint {
                framepos_t          frame;
-               const MeterSection* meter;
-               const Tempo* tempo;
+               Meter               meter;
+               Tempo               tempo;
+               double              c;
                uint32_t            bar;
                uint32_t            beat;
 
                BBTPoint (const MeterSection& m, const Tempo& t, framepos_t f,
-                         uint32_t b, uint32_t e)
-                       : frame (f), meter (&m), tempo (&t), bar (b), beat (e) {}
+                         uint32_t b, uint32_t e, double func_c)
+               : frame (f), meter (m.divisions_per_bar(), m.note_divisor()), tempo (t.beats_per_minute(), t.note_type()), c (func_c), bar (b), beat (e) {}
 
                Timecode::BBT_Time bbt() const { return Timecode::BBT_Time (bar, beat, 0); }
                operator Timecode::BBT_Time() const { return bbt(); }
@@ -321,76 +324,38 @@ class LIBARDOUR_API TempoMap : public PBD::StatefulDestructible
        void get_grid (std::vector<BBTPoint>&,
                       framepos_t start, framepos_t end);
 
-       /* TEMPO- AND METER-SENSITIVE FUNCTIONS
-
-          bbt_time(), beat_at_frame(), frame_at_beat(), tick_at_frame(),
-          frame_at_tick(),frame_time() and bbt_duration_at()
-          are all sensitive to tempo and meter, and will give answers
-          that align with the grid formed by tempo and meter sections.
-
-          They SHOULD NOT be used to determine the position of events
-          whose location is canonically defined in beats.
-       */
-
-       void bbt_time (framepos_t when, Timecode::BBT_Time&);
-
-       double beat_at_frame (framecnt_t frame) const;
-       framecnt_t frame_at_beat (double beat) const;
-
-       framepos_t frame_time (const Timecode::BBT_Time&);
-       framecnt_t bbt_duration_at (framepos_t, const Timecode::BBT_Time&, int dir);
-
-       /* TEMPO-SENSITIVE FUNCTIONS
-
-          These next 4 functions will all take tempo in account and should be
-          used to determine position (and in the last case, distance in beats)
-          when tempo matters but meter does not.
-
-          They SHOULD be used to determine the position of events
-          whose location is canonically defined in beats.
-       */
-
-       framepos_t framepos_plus_bbt (framepos_t pos, Timecode::BBT_Time b) const;
-       framepos_t framepos_plus_beats (framepos_t, Evoral::Beats) const;
-       framepos_t framepos_minus_beats (framepos_t, Evoral::Beats) const;
-       Evoral::Beats framewalk_to_beats (framepos_t pos, framecnt_t distance) const;
-
        static const Tempo& default_tempo() { return _default_tempo; }
        static const Meter& default_meter() { return _default_meter; }
 
-       const Tempo tempo_at (framepos_t) const;
-       double frames_per_beat_at (framepos_t, framecnt_t sr) const;
-
-       const Meter& meter_at (framepos_t) const;
-
-       const TempoSection& tempo_section_at (framepos_t) const;
-       const MeterSection& meter_section_at (framepos_t) const;
-       const MeterSection& meter_section_at (double) const;
+       /* because tempos may be ramped, this is only valid for the instant requested.*/
+       double frames_per_beat_at (const framepos_t&, const framecnt_t& sr) const;
 
+       const TempoSection& tempo_section_at_frame (framepos_t frame) const;
+       const MeterSection& meter_section_at_frame (framepos_t frame) const;
+       const MeterSection& meter_section_at_beat (double beat) const;
 
-       void add_tempo (const Tempo&, double where, TempoSection::Type type);
-       void add_tempo (const Tempo&, framepos_t frame, TempoSection::Type type);
+       /** add a tempo section locked to pls. ignored values will be set in recompute_tempos()
+        * @param pulse pulse position of new section. ignored if pls == AudioTime
+        * @param frame frame position of new section. ignored if pls == MusicTime
+        * @param type type of new tempo section (Ramp, Constant)
+        */
+       TempoSection* add_tempo (const Tempo&, const double& pulse, const framepos_t& frame, TempoSection::Type type, PositionLockStyle pls);
 
-       void add_meter (const Meter&, double beat, Timecode::BBT_Time where);
-       void add_meter (const Meter&, framepos_t frame);
+       /** add an meter section locked to pls.. ignored values will be set in recompute_meters()
+        * @param beat beat position of new section
+        * @param where bbt position of new section
+        * @param frame frame position of new section. ignored if pls == MusicTime
+        */
+       MeterSection* add_meter (const Meter&, const double& beat, const Timecode::BBT_Time& where, const framepos_t& frame, PositionLockStyle pls);
 
        void remove_tempo (const TempoSection&, bool send_signal);
        void remove_meter (const MeterSection&, bool send_signal);
 
-       framepos_t predict_tempo_frame (TempoSection* section, const Tempo& bpm, const Timecode::BBT_Time& bbt);
-
-       void replace_tempo (const TempoSection&, const Tempo&, const double& where, TempoSection::Type type);
-       void replace_tempo (const TempoSection&, const Tempo&, const framepos_t& frame, TempoSection::Type type);
-
-       void gui_move_tempo_frame (TempoSection*, const Tempo& bpm, const framepos_t& frame);
-       void gui_move_tempo_beat (TempoSection*, const Tempo& bpm, const double& frame);
-       void gui_move_meter (MeterSection*, const Meter& mt, const framepos_t& frame);
-       void gui_move_meter (MeterSection*, const Meter& mt, const double& beat);
-
-       bool can_solve_bbt (TempoSection* section, const Tempo& bpm, const Timecode::BBT_Time& bbt);
+       void replace_tempo (const TempoSection&, const Tempo&, const double& pulse, const framepos_t& frame
+                           , TempoSection::Type type, PositionLockStyle pls);
 
-       void replace_meter (const MeterSection&, const Meter&, const Timecode::BBT_Time& where);
-       void replace_meter (const MeterSection&, const Meter&, const framepos_t& frame);
+       void replace_meter (const MeterSection&, const Meter&, const Timecode::BBT_Time& where, const framepos_t& frame
+                           , PositionLockStyle pls);
 
        framepos_t round_to_bar  (framepos_t frame, RoundMode dir);
        framepos_t round_to_beat (framepos_t frame, RoundMode dir);
@@ -401,7 +366,7 @@ class LIBARDOUR_API TempoMap : public PBD::StatefulDestructible
        XMLNode& get_state (void);
        int set_state (const XMLNode&, int version);
 
-       void dump (Metrics& metrics, std::ostream&) const;
+       void dump (const Metrics& metrics, std::ostream&) const;
        void clear ();
 
        TempoMetric metric_at (Timecode::BBT_Time bbt) const;
@@ -424,29 +389,109 @@ class LIBARDOUR_API TempoMap : public PBD::StatefulDestructible
 
        framecnt_t frame_rate () const { return _frame_rate; }
 
-       double bbt_to_beats (Timecode::BBT_Time bbt);
-       Timecode::BBT_Time beats_to_bbt (double beats);
+       /* TEMPO- AND METER-SENSITIVE FUNCTIONS
+
+          bbt_at_frame(), frame_at_bbt(), beat_at_frame(), frame_at_beat(), tempo_at_beat()
+          and bbt_duration_at()
+          are all sensitive to tempo and meter, and will give answers
+          that align with the grid formed by tempo and meter sections.
+
+          They SHOULD NOT be used to determine the position of events
+          whose location is canonically defined in beats.
+       */
+
+       double beat_at_frame (const framecnt_t& frame) const;
+       framecnt_t frame_at_beat (const double& beat) const;
+
+       Tempo tempo_at_frame (const framepos_t& frame) const;
+       framepos_t frame_at_tempo (const Tempo& tempo) const;
+
+       Tempo tempo_at_beat (const double& beat) const;
+
+       const Meter& meter_at_frame (framepos_t) const;
+
+       /* you probably only need to use pulses when moving tempos */
+       double pulse_at_beat (const double& beat) const;
+       double beat_at_pulse (const double& pulse) const;
+
+       double pulse_at_frame (const framecnt_t& frame) const;
+       framecnt_t frame_at_pulse (const double& pulse) const;
+
+       /* bbt - it's nearly always better to use beats.*/
+       Timecode::BBT_Time bbt_at_frame (framepos_t when);
+       Timecode::BBT_Time bbt_at_frame_rt (framepos_t when);
+       framepos_t frame_at_bbt (const Timecode::BBT_Time&);
+
+       double beat_at_bbt (const Timecode::BBT_Time& bbt);
+       Timecode::BBT_Time bbt_at_beat (const double& beats);
+
+       double pulse_at_bbt (const Timecode::BBT_Time& bbt);
+       Timecode::BBT_Time bbt_at_pulse (const double& pulse);
+
+       framecnt_t bbt_duration_at (framepos_t, const Timecode::BBT_Time&, int dir);
+
+       /* TEMPO-SENSITIVE FUNCTIONS
+
+          These next 4 functions will all take tempo in account and should be
+          used to determine position (and in the last case, distance in beats)
+          when tempo matters but meter does not.
+
+          They SHOULD be used to determine the position of events
+          whose location is canonically defined in beats.
+       */
+
+       framepos_t framepos_plus_bbt (framepos_t pos, Timecode::BBT_Time b) const;
+       framepos_t framepos_plus_beats (framepos_t, Evoral::Beats) const;
+       framepos_t framepos_minus_beats (framepos_t, Evoral::Beats) const;
+       Evoral::Beats framewalk_to_beats (framepos_t pos, framecnt_t distance) const;
+
+       void gui_move_tempo (TempoSection*, const framepos_t& frame);
+       void gui_move_meter (MeterSection*, const framepos_t& frame);
+       bool gui_change_tempo (TempoSection*, const Tempo& bpm);
+       void gui_dilate_tempo (TempoSection* tempo, const framepos_t& frame, const framepos_t& end_frame, const double& pulse);
+
+       std::pair<double, framepos_t> predict_tempo_position (TempoSection* section, const Timecode::BBT_Time& bbt);
+       bool can_solve_bbt (TempoSection* section, const Timecode::BBT_Time& bbt);
 
        PBD::Signal0<void> MetricPositionChanged;
+       void fix_legacy_session();
 
 private:
-       double bbt_to_beats_locked (const Metrics& metrics, Timecode::BBT_Time bbt);
-       Timecode::BBT_Time beats_to_bbt_locked (Metrics& metrics, double beats);
-       double beat_at_frame_locked (const Metrics& metrics, framecnt_t frame) const;
-       framecnt_t frame_at_beat_locked (const Metrics& metrics, double beat) const;
-       double tick_at_frame_locked (const Metrics& metrics, framecnt_t frame) const;
 
-       double beat_offset_at (const Metrics& metrics, double beat) const;
-       frameoffset_t frame_offset_at (const Metrics& metrics, framepos_t frame) const;
+       double beat_at_frame_locked (const Metrics& metrics, const framecnt_t& frame) const;
+       framecnt_t frame_at_beat_locked (const Metrics& metrics, const double& beat) const;
+
+       double pulse_at_beat_locked (const Metrics& metrics, const double& beat) const;
+       double beat_at_pulse_locked (const Metrics& metrics, const double& pulse) const;
+
+       double pulse_at_frame_locked (const Metrics& metrics, const framecnt_t& frame) const;
+       framecnt_t frame_at_pulse_locked (const Metrics& metrics, const double& pulse) const;
+
+       Tempo tempo_at_frame_locked (const Metrics& metrics, const framepos_t& frame) const;
+       framepos_t frame_at_tempo_locked (const Metrics& metrics, const Tempo& tempo) const;
+
+       Timecode::BBT_Time bbt_at_frame_locked (const Metrics& metrics, const framepos_t& frame) const;
+       framepos_t frame_at_bbt_locked (const Metrics& metrics, const Timecode::BBT_Time&) const;
+
+       double beat_at_bbt_locked (const Metrics& metrics, const Timecode::BBT_Time& bbt) const ;
+       Timecode::BBT_Time bbt_at_beat_locked (const Metrics& metrics, const double& beats) const;
+
+       double pulse_at_bbt_locked (const Metrics& metrics, const Timecode::BBT_Time& bbt) const;
+       Timecode::BBT_Time bbt_at_pulse_locked (const Metrics& metrics, const double& pulse) const;
+
+       const TempoSection& tempo_section_at_frame_locked (const Metrics& metrics, framepos_t frame) const;
+       const TempoSection& tempo_section_at_beat_locked (const Metrics& metrics, const double& beat) const;
+
+       const MeterSection& meter_section_at_frame_locked (const Metrics& metrics, framepos_t frame) const;
+       const MeterSection& meter_section_at_beat_locked (const Metrics& metrics, const double& beat) const;
 
-       framecnt_t frame_at_tick_locked (const Metrics& metrics, double tick) const;
-       framepos_t frame_time_locked (const Metrics& metrics, const Timecode::BBT_Time&);
+       bool check_solved (const Metrics& metrics) const;
+       bool set_active_tempos (const Metrics& metrics, const framepos_t& frame);
 
-       bool check_solved (Metrics& metrics, bool by_frame);
-       bool solve_map (Metrics& metrics, TempoSection* section, const Tempo& bpm, const framepos_t& frame);
-       bool solve_map (Metrics& metrics, TempoSection* section, const Tempo& bpm, const double& beat);
-       void solve_map (Metrics& metrics, MeterSection* section, const Meter& mt, const framepos_t& frame);
-       void solve_map (Metrics& metrics, MeterSection* section, const Meter& mt, const double& beat);
+       bool solve_map_frame (Metrics& metrics, TempoSection* section, const framepos_t& frame);
+       bool solve_map_pulse (Metrics& metrics, TempoSection* section, const double& pulse);
+       bool solve_map_frame (Metrics& metrics, MeterSection* section, const framepos_t& frame);
+       bool solve_map_bbt (Metrics& metrics, MeterSection* section, const Timecode::BBT_Time& bbt);
 
        friend class ::BBTTest;
        friend class ::FrameposPlusBeatsTest;
@@ -472,16 +517,17 @@ private:
 
        void do_insert (MetricSection* section);
 
-       void add_tempo_locked (const Tempo&, double where, bool recompute, TempoSection::Type type);
-       void add_tempo_locked (const Tempo&, framepos_t frame, bool recompute, TempoSection::Type type);
+       TempoSection* add_tempo_locked (const Tempo&, double pulse, framepos_t frame
+                              , TempoSection::Type type, PositionLockStyle pls, bool recompute, bool locked_to_meter = false);
 
-       void add_meter_locked (const Meter&, double beat, Timecode::BBT_Time where, bool recompute);
-       void add_meter_locked (const Meter&, framepos_t frame, bool recompute);
+       MeterSection* add_meter_locked (const Meter&, double beat, const Timecode::BBT_Time& where, framepos_t frame
+                                       , PositionLockStyle pls, bool recompute);
 
        bool remove_tempo_locked (const TempoSection&);
        bool remove_meter_locked (const MeterSection&);
 
-       TempoSection* copy_metrics_and_point (Metrics& copy, TempoSection* section);
+       TempoSection* copy_metrics_and_point (const Metrics& metrics, Metrics& copy, TempoSection* section);
+       MeterSection* copy_metrics_and_point (const Metrics& metrics, Metrics& copy, MeterSection* section);
 };
 
 }; /* namespace ARDOUR */