more fun and games with meter and the tempo map: rename Meter::beats_per_bar() to...
authorPaul Davis <paul@linuxaudiosystems.com>
Tue, 13 Dec 2011 02:46:36 +0000 (02:46 +0000)
committerPaul Davis <paul@linuxaudiosystems.com>
Tue, 13 Dec 2011 02:46:36 +0000 (02:46 +0000)
git-svn-id: svn://localhost/ardour2/branches/3.0@10995 d708f5d6-7413-0410-9779-e7cbd77b26cf

16 files changed:
gtk2_ardour/audio_clock.cc
gtk2_ardour/editor.cc
gtk2_ardour/editor_drag.cc
gtk2_ardour/editor_ops.cc
gtk2_ardour/editor_rulers.cc
gtk2_ardour/editor_tempodisplay.cc
gtk2_ardour/tempo_dialog.cc
gtk2_ardour/verbose_cursor.cc
libs/ardour/ardour/tempo.h
libs/ardour/midi_buffer.cc
libs/ardour/midi_clock_slave.cc
libs/ardour/session_time.cc
libs/ardour/session_vst.cc
libs/ardour/tempo.cc
libs/ardour/ticker.cc
libs/surfaces/mackie/mackie_control_protocol.cc

index 757d5bc846c727c2128b88e407e2c228ae4230a1..aa150795875f16eeb241a19c292cf7e8362afd1c 100644 (file)
@@ -1171,7 +1171,7 @@ AudioClock::set_bbt (framepos_t when, bool force)
                sprintf (buf, "%-5.2f", m.tempo().beats_per_minute());
                _left_layout->set_text (buf);
 
-               sprintf (buf, "%g/%g", m.meter().beats_per_bar(), m.meter().note_divisor());
+               sprintf (buf, "%g/%g", m.meter().divisions_per_bar(), m.meter().note_divisor());
                _right_layout->set_text (buf);
        }
 }
index 28659d077d3c1283f9295f4123bacc07925aff4f..d4e0e60ef80e1e3da441f7f08f9a9433562fec48 100644 (file)
@@ -3745,7 +3745,7 @@ Editor::get_grid_type_as_beats (bool& success, framepos_t position)
 
        case SnapToBar:
                if (_session) {
-                       return _session->tempo_map().meter_at (position).beats_per_bar();
+                       return _session->tempo_map().meter_at (position).divisions_per_bar();
                }
                break;
 
index ef00b011c16f3913c9a981f6ec8c9dc65078d938..20ebe7f98d51998978b4f7b1c0037f5c5ef6bff9 100644 (file)
@@ -1853,35 +1853,37 @@ MeterMarkerDrag::MeterMarkerDrag (Editor* e, ArdourCanvas::Item* i, bool c)
 void
 MeterMarkerDrag::start_grab (GdkEvent* event, Gdk::Cursor* cursor)
 {
-       if (_copy) {
-               // create a dummy marker for visual representation of moving the copy.
-               // The actual copying is not done before we reach the finish callback.
-               char name[64];
-               snprintf (name, sizeof(name), "%g/%g", _marker->meter().beats_per_bar(), _marker->meter().note_divisor ());
-
-               MeterMarker* new_marker = new MeterMarker (
-                       *_editor,
-                       *_editor->meter_group,
-                       ARDOUR_UI::config()->canvasvar_MeterMarker.get(),
-                       name,
-                       *new MeterSection (_marker->meter())
-                       );
-
-               _item = &new_marker->the_item ();
-               _marker = new_marker;
-
-       } else {
-
-               MetricSection& section (_marker->meter());
-
-               if (!section.movable()) {
-                       return;
-               }
-
+       // create a dummy marker for visual representation of moving the
+       // section, because whether its a copy or not, we're going to 
+       // leave or lose the original marker (leave if its a copy; lose if its
+       // not, because we'll remove it from the map).
+       
+       MeterSection section (_marker->meter());
+       
+       if (!section.movable()) {
+               return;
+       }
+       
+       char name[64];
+       snprintf (name, sizeof(name), "%g/%g", _marker->meter().divisions_per_bar(), _marker->meter().note_divisor ());
+       
+       _marker = new MeterMarker (
+               *_editor,
+               *_editor->meter_group,
+               ARDOUR_UI::config()->canvasvar_MeterMarker.get(),
+               name,
+               *new MeterSection (_marker->meter())
+               );
+       
+       _item = &_marker->the_item ();
+       
+       if (!_copy) {
+               TempoMap& map (_editor->session()->tempo_map());
+               /* remove the section while we drag it */
+               map.remove_meter (section);
        }
 
        Drag::start_grab (event, cursor);
-
        show_verbose_cursor_time (adjusted_current_frame(event));
 }
 
@@ -1914,7 +1916,7 @@ MeterMarkerDrag::finished (GdkEvent* event, bool movement_occurred)
 
        TempoMap& map (_editor->session()->tempo_map());
        map.bbt_time (last_pointer_frame(), when);
-
+       
        if (_copy == true) {
                _editor->begin_reversible_command (_("copy meter mark"));
                XMLNode &before = map.get_state();
@@ -1923,17 +1925,21 @@ MeterMarkerDrag::finished (GdkEvent* event, bool movement_occurred)
                _editor->session()->add_command(new MementoCommand<TempoMap>(map, &before, &after));
                _editor->commit_reversible_command ();
 
-               // delete the dummy marker we used for visual representation of copying.
-               // a new visual marker will show up automatically.
-               delete _marker;
        } else {
                _editor->begin_reversible_command (_("move meter mark"));
                XMLNode &before = map.get_state();
-               map.move_meter (_marker->meter(), when);
+
+               /* we removed it before, so add it back now */
+               
+               map.add_meter (_marker->meter(), when);
                XMLNode &after = map.get_state();
                _editor->session()->add_command(new MementoCommand<TempoMap>(map, &before, &after));
                _editor->commit_reversible_command ();
        }
+
+       // delete the dummy marker we used for visual representation while moving.
+       // a new visual marker will show up automatically.
+       delete _marker;
 }
 
 void
index 738737f46d193f8ebff8a0e428c9d196aab455ed..a07f16b5c8f393479e4f221916962870953167ed 100644 (file)
@@ -5475,7 +5475,9 @@ Editor::define_one_bar (framepos_t start, framepos_t end)
           we have frames per bar, and beats per bar, so ...
        */
 
-       double frames_per_beat = length / m.beats_per_bar();
+       /* XXXX METER MATH */
+
+       double frames_per_beat = length / m.divisions_per_bar();
 
        /* beats per minute = */
 
index 3b525ae6aae38ea3d3305d63c72768c58afc51b7..3a54e965f17edd538b2468f59aa1b765eb7d3478 100644 (file)
@@ -1380,7 +1380,7 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
                        next_beat.bars = (*i).bar;
                        next_beat.ticks = 0;
 
-                       if ((*i).meter->beats_per_bar() > (next_beat.beats + 1)) {
+                       if ((*i).meter->divisions_per_bar() > (next_beat.beats + 1)) {
                                  next_beat.beats += 1;
                        } else {
                                  next_beat.bars += 1;
@@ -1472,7 +1472,7 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
                        next_beat.beats = (*i).beat;
                        next_beat.bars = (*i).bar;
 
-                       if ((*i).meter->beats_per_bar() > (next_beat.beats + 1)) {
+                       if ((*i).meter->divisions_per_bar() > (next_beat.beats + 1)) {
                                  next_beat.beats += 1;
                        } else {
                                  next_beat.bars += 1;
@@ -1569,7 +1569,7 @@ Editor::metric_get_bbt (GtkCustomRulerMark **marks, gdouble lower, gdouble /*upp
                        next_beat.beats = (*i).beat;
                        next_beat.bars = (*i).bar;
 
-                       if ((*i).meter->beats_per_bar() > (next_beat.beats + 1)) {
+                       if ((*i).meter->divisions_per_bar() > (next_beat.beats + 1)) {
                                  next_beat.beats += 1;
                        } else {
                                  next_beat.bars += 1;
index 1069e457e635efa1dab9f7576e5b9116c838ff21..28afc3b015ab2354c223e2be7a1062594293f855 100644 (file)
@@ -84,7 +84,7 @@ Editor::draw_metric_marks (const Metrics& metrics)
        for (Metrics::const_iterator i = metrics.begin(); i != metrics.end(); ++i) {
 
                if ((ms = dynamic_cast<const MeterSection*>(*i)) != 0) {
-                       snprintf (buf, sizeof(buf), "%g/%g", ms->beats_per_bar(), ms->note_divisor ());
+                       snprintf (buf, sizeof(buf), "%g/%g", ms->divisions_per_bar(), ms->note_divisor ());
                        metric_marks.push_back (new MeterMarker (*this, *meter_group, ARDOUR_UI::config()->canvasvar_MeterMarker.get(), buf,
                                                                 *(const_cast<MeterSection*>(ms))));
                } else if ((ts = dynamic_cast<const TempoSection*>(*i)) != 0) {
@@ -157,7 +157,7 @@ Editor::compute_current_bbt_points (framepos_t leftmost, framepos_t rightmost)
        }
        previous_beat.ticks = 0;
 
-       if (_session->tempo_map().meter_at(rightmost).beats_per_bar () > next_beat.beats + 1) {
+       if (_session->tempo_map().meter_at(rightmost).divisions_per_bar () > next_beat.beats + 1) {
                next_beat.beats += 1;
        } else {
                next_beat.bars += 1;
index bb9a5c4a8306a766ebe588819e5d822fece6bd51..f014832f2ababefe314d06b2a26bd2ffc4614a7d 100644 (file)
@@ -249,7 +249,7 @@ MeterDialog::MeterDialog (TempoMap& map, framepos_t frame, const string & action
        Meter meter (map.meter_at(frame));
 
        map.bbt_time (frame, when);
-       init (when, meter.beats_per_bar(), meter.note_divisor(), true);
+       init (when, meter.divisions_per_bar(), meter.note_divisor(), true);
 }
 
 MeterDialog::MeterDialog (MeterSection& section, const string & action)
@@ -257,7 +257,7 @@ MeterDialog::MeterDialog (MeterSection& section, const string & action)
          ok_button (action),
          cancel_button (_("Cancel"))
 {
-       init (section.start(), section.beats_per_bar(), section.note_divisor(), section.movable());
+       init (section.start(), section.divisions_per_bar(), section.note_divisor(), section.movable());
 }
 
 void
index 2ae215acecae8fa8d560c1d29e4de0eb6476a57d..f80b74c4f8d7be048a8e4a3d3b8ad6e83a093777 100644 (file)
@@ -215,7 +215,7 @@ VerboseCursor::set_duration (framepos_t start, framepos_t end, double x, double
 
                beats -= sbbt.beats;
                if (beats < 0) {
-                       beats += int (meter_at_start.beats_per_bar());
+                       beats += int (meter_at_start.divisions_per_bar());
                        --bars;
                }
 
index 631b709a28d3a9317b3abe2cea3aa204a99827a0..281f42443e7d0828076707d4aea4de29006d1bfc 100644 (file)
@@ -45,7 +45,7 @@ class Tempo {
 
        double beats_per_minute () const { return _beats_per_minute;}
        double note_type () const { return _note_type;}
-       double frames_per_beat (framecnt_t sr, const Meter& meter) const;
+       double frames_per_beat (framecnt_t sr) const;
 
   protected:
        double _beats_per_minute;
@@ -54,23 +54,23 @@ class Tempo {
 
 class Meter {
   public:
-       Meter (double bpb, double bt)
-               : _beats_per_bar (bpb), _note_type (bt) {}
+       Meter (double dpb, double bt)
+               : _divisions_per_bar (dpb), _note_type (bt) {}
 
-       double beats_per_bar () const { return _beats_per_bar; }
+       double divisions_per_bar () const { return _divisions_per_bar; }
        double note_divisor() const { return _note_type; }
 
        double frames_per_bar (const Tempo&, framecnt_t sr) const;
        double frames_per_division (const Tempo&, framecnt_t sr) const;
 
   protected:
-       /** The number of beats in a bar.  This is a real value because
+       /** The number of divisions in a bar.  This is a floating point value because
            there are musical traditions on our planet that do not limit
            themselves to integral numbers of beats per bar.
        */
-       double _beats_per_bar;
+       double _divisions_per_bar;
 
-       /** The type of "note" that a beat represents.  For example, 4.0 is
+       /** The type of "note" that a division represents.  For example, 4.0 is
            a quarter (crotchet) note, 8.0 is an eighth (quaver) note, etc.
        */
        double _note_type;
@@ -297,4 +297,8 @@ class TempoMap : public PBD::StatefulDestructible
 
 }; /* namespace ARDOUR */
 
+std::ostream& operator<< (std::ostream&, const ARDOUR::Meter&);
+std::ostream& operator<< (std::ostream&, const ARDOUR::Tempo&);
+std::ostream& operator<< (std::ostream&, const ARDOUR::MetricSection&);
+
 #endif /* __ardour_tempo_h__ */
index e944727337ce827ba6779f06537b4d4f4eb7625b..403f0a86ac44b81c6ed9d05d3f5044093756a52d 100644 (file)
@@ -497,6 +497,12 @@ MidiBuffer::merge_in_place(const MidiBuffer &other)
                                                     (int) *(_data + us.offset + sizeof (TimeType)),
                                                     (int) *(other._data + them.offset + sizeof (TimeType))));
 
+                       if ((*us).time() != (*them).time()) {
+                               cerr << " in merge code for two events:\n"
+                                    << '\t' << (*us) << endl
+                                    << '\t' << (*them) << endl
+                                    << "about to crash ...\n";
+                       }
                        assert ((*us).time() == (*them).time());
                                                     
                        uint8_t our_midi_status_byte = *(_data + us.offset + sizeof (TimeType));
index cc717d5fb4d1a158c09c08ece4f0e1a24f6bfa52..892bc800ff11d006f66e8d6b8604d9941e31e19d 100644 (file)
@@ -86,10 +86,7 @@ void
 MIDIClock_Slave::calculate_one_ppqn_in_frames_at(framepos_t time)
 {
        const Tempo& current_tempo = session->tempo_map().tempo_at(time);
-       const Meter& current_meter = session->tempo_map().meter_at(time);
-       double frames_per_beat =
-               current_tempo.frames_per_beat(session->frame_rate(),
-                                             current_meter);
+       double frames_per_beat = current_tempo.frames_per_beat(session->frame_rate());
 
        double quarter_notes_per_beat = 4.0 / current_tempo.note_type();
        double frames_per_quarter_note = frames_per_beat / quarter_notes_per_beat;
index d9c612fbee8ae0bf141fc678775325e0a68756f0..0543fe2d021fea6de9b8fa4251478095f0ccee0c 100644 (file)
@@ -492,7 +492,7 @@ Session::jack_timebase_callback (jack_transport_state_t /*state*/,
 
                // XXX still need to set bar_start_tick
 
-               pos->beats_per_bar = metric.meter().beats_per_bar();
+               pos->beats_per_bar = metric.meter().divisions_per_bar();
                pos->beat_type = metric.meter().note_divisor();
                pos->ticks_per_beat = Timecode::BBT_Time::ticks_per_beat;
                pos->beats_per_minute = metric.tempo().beats_per_minute();
index 9ba3864383a6bd5cc13e0abf69ab61c0d1146642..fa0641a6ab7ef4b12a531b6e78faae6c6e5a4e7e 100644 (file)
@@ -132,7 +132,7 @@ intptr_t Session::vst_callback (
                        }
                        if (value & (kVstBarsValid)) {
                                const Meter& m (session->tempo_map().meter_at (session->transport_frame()));
-                               _timeInfo.timeSigNumerator = m.beats_per_bar ();
+                               _timeInfo.timeSigNumerator = m.divisions_per_bar ();
                                _timeInfo.timeSigDenominator = m.note_divisor ();
                                _timeInfo.flags |= (kVstBarsValid);
                        }
index 3231e563271571f9cdc313a59797b38c524e06dd..1c36f129b4458995e08f4f3fece917277be33766 100644 (file)
@@ -45,7 +45,8 @@ using Timecode::BBT_Time;
 Meter    TempoMap::_default_meter (4.0, 4.0);
 Tempo    TempoMap::_default_tempo (120.0);
 
-double Tempo::frames_per_beat (framecnt_t sr, const Meter& meter) const
+double 
+Tempo::frames_per_beat (framecnt_t sr) const
 {
        return  (60.0 * sr) / _beats_per_minute;
 }
@@ -55,13 +56,13 @@ double Tempo::frames_per_beat (framecnt_t sr, const Meter& meter) const
 double
 Meter::frames_per_bar (const Tempo& tempo, framecnt_t sr) const
 {
-       return ((60.0 * sr * _beats_per_bar) / (tempo.beats_per_minute() * _note_type/tempo.note_type()));
+       return (60.0 * sr * _divisions_per_bar) / tempo.beats_per_minute();
 }
 
 double 
 Meter::frames_per_division (const Tempo& tempo, framecnt_t sr) const
 {
-       return  ((60.0 * sr) / (tempo.beats_per_minute() * _note_type/tempo.note_type()));
+       return (60.0 * sr) / (tempo.beats_per_minute() * _note_type/tempo.note_type());
 }
 
 /***********************************************************************/
@@ -171,7 +172,7 @@ MeterSection::MeterSection (const XMLNode& node)
                throw failed_constructor();
        }
 
-       if (sscanf (prop->value().c_str(), "%lf", &_beats_per_bar) != 1 || _beats_per_bar < 0.0) {
+       if (sscanf (prop->value().c_str(), "%lf", &_divisions_per_bar) != 1 || _divisions_per_bar < 0.0) {
                error << _("MeterSection XML node has an illegal \"beats-per-bar\" value") << endmsg;
                throw failed_constructor();
        }
@@ -208,7 +209,7 @@ MeterSection::get_state() const
        root->add_property ("start", buf);
        snprintf (buf, sizeof (buf), "%f", _note_type);
        root->add_property ("note-type", buf);
-       snprintf (buf, sizeof (buf), "%f", _beats_per_bar);
+       snprintf (buf, sizeof (buf), "%f", _divisions_per_bar);
        root->add_property ("beats-per-bar", buf);
        snprintf (buf, sizeof (buf), "%s", movable()?"yes":"no");
        root->add_property ("movable", buf);
@@ -236,7 +237,7 @@ TempoMap::TempoMap (framecnt_t fr)
        start.ticks = 0;
 
        TempoSection *t = new TempoSection (start, _default_tempo.beats_per_minute(), _default_tempo.note_type());
-       MeterSection *m = new MeterSection (start, _default_meter.beats_per_bar(), _default_meter.note_divisor());
+       MeterSection *m = new MeterSection (start, _default_meter.divisions_per_bar(), _default_meter.note_divisor());
 
        t->set_movable (false);
        m->set_movable (false);
@@ -374,40 +375,23 @@ TempoMap::remove_meter (const MeterSection& tempo)
 void
 TempoMap::do_insert (MetricSection* section, bool with_bbt)
 {
-       /* First of all, check to see if the new MetricSection is in the
-          middle of a bar.  If so, we need to fix the bar that we are in
-          to have a different meter.
-       */
-
        assert (section->start().ticks == 0);
 
-       if (section->start().beats != 1) {
-
-               /* Here's the tempo and metric where we are proposing to insert `section' */
-               TempoMetric tm = metric_at (section->start ());
-
-               /* This is where we will put the `corrective' new meter; at the start of
-                  the bar that we are inserting into the middle of.
-               */
-               BBT_Time where_correction = section->start();
-               where_correction.beats = 1;
-               where_correction.ticks = 0;
+       /* we only allow new meters to be inserted on beat 1 of an existing
+        * measure. 
+        */
 
-               /* Put in the meter change to make the bar before our `section' the right
-                  length.
-               */
-               do_insert (new MeterSection (where_correction, section->start().beats, tm.meter().note_divisor ()), true);
+       if (dynamic_cast<MeterSection*>(section) && 
+           (section->start().beats != 1 || section->start().ticks != 0)) {
 
-               /* This is where the new stuff will now go; the start of the next bar
-                  (after the one whose meter we just fixed).
-               */
-               BBT_Time where_new (where_correction.bars + 1, 1, 0);
+               BBT_Time corrected = section->start();
+               corrected.beats = 1;
+               corrected.ticks = 0;
+               
+               warning << string_compose (_("Meter changes can only be positioned on the first beat of a bar. Moving from %1 to %2"),
+                                          section->start(), corrected) << endmsg;
 
-               /* Change back to the original meter */
-               do_insert (new MeterSection (where_new, tm.meter().beats_per_bar(), tm.meter().note_divisor()), true);
-
-               /* And set up `section' for where it should be, ready to be inserted */
-               section->set_start (where_new);
+               section->set_start (corrected);
        }
 
        Metrics::iterator i;
@@ -541,7 +525,7 @@ TempoMap::add_meter (const Meter& meter, BBT_Time where)
                /* new meters *always* start on a beat. */
                where.ticks = 0;
 
-               do_insert (new MeterSection (where, meter.beats_per_bar(), meter.note_divisor()), true);
+               do_insert (new MeterSection (where, meter.divisions_per_bar(), meter.note_divisor()), true);
        }
 
        PropertyChanged (PropertyChange ());
@@ -552,7 +536,7 @@ TempoMap::add_meter (const Meter& meter, framepos_t where)
 {
        {
                Glib::RWLock::WriterLock lm (lock);
-               do_insert (new MeterSection (where, meter.beats_per_bar(), meter.note_divisor()), false);
+               do_insert (new MeterSection (where, meter.divisions_per_bar(), meter.note_divisor()), false);
        }
 
        PropertyChanged (PropertyChange ());
@@ -688,7 +672,7 @@ TempoMap::timestamp_metrics (bool use_bbt)
        tempo = &first_tempo ();
 
        if (use_bbt) {
-
+               
                // cerr << "\n\n\n ######################\nTIMESTAMP via BBT ##############\n" << endl;
 
                framepos_t current = 0;
@@ -739,10 +723,9 @@ TempoMap::timestamp_metrics (bool use_bbt)
                        }
 
                        bbt_time_with_metric ((*i)->frame(), bbt, metric);
-
+                       
                        // cerr << "timestamp @ " << (*i)->frame() << " with " << bbt.bars << "|" << bbt.beats << "|" << bbt.ticks << " => ";
 
-
                        if (first) {
                                first = false;
                        } else {
@@ -761,10 +744,10 @@ TempoMap::timestamp_metrics (bool use_bbt)
                                }
                        }
 
-                       //s cerr << bbt.bars << "|" << bbt.beats << "|" << bbt.ticks << endl;
+                       // cerr << bbt << endl;
 
                        (*i)->set_start (bbt);
-
+                       
                        if ((t = dynamic_cast<TempoSection*>(*i)) != 0) {
                                tempo = t;
                                // cerr << "NEW TEMPO, frame = " << (*i)->frame() << " start = " << (*i)->start() <<endl;
@@ -801,6 +784,8 @@ TempoMap::metric_at (framepos_t frame) const
 
        for (Metrics::const_iterator i = metrics->begin(); i != metrics->end(); ++i) {
 
+               // cerr << "Looking at a metric section " << **i << endl;
+
                if ((*i)->frame() > frame) {
                        break;
                }
@@ -815,6 +800,7 @@ TempoMap::metric_at (framepos_t frame) const
                m.set_start ((*i)->start ());
        }
 
+       // cerr << "for framepos " << frame << " returning " << m.meter() << " @ " << m.tempo() << endl;
        return m;
 }
 
@@ -873,8 +859,13 @@ TempoMap::bbt_time_with_metric (framepos_t frame, BBT_Time& bbt, const TempoMetr
 {
        framecnt_t frame_diff;
 
-       const double beats_per_bar = metric.meter().beats_per_bar();
-       const double ticks_per_frame = metric.tempo().frames_per_beat (_frame_rate, metric.meter()) / BBT_Time::ticks_per_beat;
+       const double divisions_per_bar = metric.meter().divisions_per_bar();
+       const double ticks_per_frame = metric.meter().frames_per_division (metric.tempo(),_frame_rate) / BBT_Time::ticks_per_beat;
+
+       // cerr << "*** Compute BBT time for " << frame 
+       // <<  " from metric at " << metric.frame() << " tempo = " << metric.tempo().beats_per_minute () << " meter " 
+       // << metric.meter().divisions_per_bar() << '/' << metric.meter().note_divisor() 
+       // << endl; 
 
        /* now compute how far beyond that point we actually are. */
 
@@ -885,13 +876,15 @@ TempoMap::bbt_time_with_metric (framepos_t frame, BBT_Time& bbt, const TempoMetr
        bbt.ticks %= (uint32_t)BBT_Time::ticks_per_beat;
 
        bbt.beats = metric.start().beats + xtra_beats - 1; // correction for 1-based counting, see below for matching operation.
-       bbt.bars = metric.start().bars + (uint32_t)floor((double)bbt.beats / beats_per_bar);
-       bbt.beats = (uint32_t)fmod((double)bbt.beats, beats_per_bar);
+       bbt.bars = metric.start().bars + (uint32_t)floor((double)bbt.beats / divisions_per_bar);
+       bbt.beats = (uint32_t)fmod((double)bbt.beats, divisions_per_bar);
 
        /* if we have a fractional number of beats per bar, we see if
           we're in the last beat (the fractional one).  if so, we
-          round ticks appropriately and bump to the next bar. */
-       double beat_fraction = beats_per_bar - floor(beats_per_bar);
+          round ticks appropriately and bump to the next bar. 
+       */
+       double beat_fraction = divisions_per_bar - floor(divisions_per_bar);
+
        /* XXX one problem here is that I'm not sure how to handle
           fractional beats that don't evenly divide ticks_per_beat.
           If they aren't handled consistently, I would guess we'll
@@ -901,7 +894,7 @@ TempoMap::bbt_time_with_metric (framepos_t frame, BBT_Time& bbt, const TempoMetr
        */
        uint32_t ticks_on_last_beat = (uint32_t)floor(BBT_Time::ticks_per_beat * beat_fraction);
 
-       if (bbt.beats > (uint32_t)floor(beats_per_bar) && bbt.ticks >= ticks_on_last_beat) {
+       if (bbt.beats > (uint32_t)floor(divisions_per_bar) && bbt.ticks >= ticks_on_last_beat) {
                bbt.ticks -= ticks_on_last_beat;
                bbt.beats = 0;
                bbt.bars++;
@@ -928,20 +921,29 @@ TempoMap::count_frames_between (const BBT_Time& start, const BBT_Time& end) cons
 
        uint32_t bar_offset = start.bars - m.start().bars;
 
-       double  beat_offset = bar_offset*m.meter().beats_per_bar() - (m.start().beats-1) + (start.beats -1)
+       double  beat_offset = bar_offset*m.meter().divisions_per_bar() - (m.start().beats-1) + (start.beats -1)
                + start.ticks/BBT_Time::ticks_per_beat;
 
+       start_frame = m.frame() + (framepos_t) rint(beat_offset * m.meter().frames_per_division(m.tempo(),_frame_rate));
 
-       start_frame = m.frame() + (framepos_t) rint(beat_offset * m.tempo().frames_per_beat(_frame_rate, m.meter()));
+       // cerr << "from start " << start << " compute frame = " << start_frame 
+       // <<  " from metric at " << m.frame() << " tempo = " << m.tempo().beats_per_minute () << " meter " 
+       // << m.meter().divisions_per_bar() << '/' << m.meter().note_divisor() 
+       // << endl;
 
        m =  metric_at(end);
 
        bar_offset = end.bars - m.start().bars;
 
-       beat_offset = bar_offset * m.meter().beats_per_bar() - (m.start().beats -1) + (end.beats - 1)
+       beat_offset = bar_offset * m.meter().divisions_per_bar() - (m.start().beats -1) + (end.beats - 1)
                + end.ticks/BBT_Time::ticks_per_beat;
 
-       end_frame = m.frame() + (framepos_t) rint(beat_offset * m.tempo().frames_per_beat(_frame_rate, m.meter()));
+       end_frame = m.frame() + (framepos_t) rint(beat_offset * m.meter().frames_per_division(m.tempo(),_frame_rate));
+
+       // cerr << "from end " << end << " compute frame = " << end_frame 
+       // <<  " from metric at " << m.frame() << " tempo = " << m.tempo().beats_per_minute () << " meter " 
+       // << m.meter().divisions_per_bar() << '/' << m.meter().note_divisor() 
+       // << endl;
 
        frames = end_frame - start_frame;
 
@@ -957,43 +959,43 @@ TempoMap::count_frames_between_metrics (const Meter& meter, const Tempo& tempo,
        framecnt_t frames = 0;
        uint32_t bar = start.bars;
        double beat = (double) start.beats;
-       double beats_counted = 0;
-       double beats_per_bar = 0;
-       double beat_frames = 0;
+       double divisions_counted = 0;
+       double divisions_per_bar = 0;
+       double division_frames = 0;
 
-       beats_per_bar = meter.beats_per_bar();
-       beat_frames = tempo.frames_per_beat (_frame_rate,meter);
+       divisions_per_bar = meter.divisions_per_bar();
+       division_frames = meter.frames_per_division (tempo, _frame_rate);
 
        frames = 0;
 
        while (bar < end.bars || (bar == end.bars && beat < end.beats)) {
 
-               if (beat >= beats_per_bar) {
+               if (beat >= divisions_per_bar) {
                        beat = 1;
                        ++bar;
-                       ++beats_counted;
+                       ++divisions_counted;
 
-                       if (beat > beats_per_bar) {
+                       if (beat > divisions_per_bar) {
 
                                /* this is a fractional beat at the end of a fractional bar
                                   so it should only count for the fraction
                                */
 
-                               beats_counted -= (ceil(beats_per_bar) - beats_per_bar);
+                               divisions_counted -= (ceil(divisions_per_bar) - divisions_per_bar);
                        }
 
                } else {
                        ++beat;
-                       ++beats_counted;
+                       ++divisions_counted;
                }
        }
 
        // cerr << "Counted " << beats_counted << " from " << start << " to " << end
-       // << " bpb were " << beats_per_bar
+       // << " bpb were " << divisions_per_bar
        // << " fpb was " << beat_frames
        // << endl;
 
-       frames = (framecnt_t) llrint (floor (beats_counted * beat_frames));
+       frames = (framecnt_t) llrint (floor (divisions_counted * division_frames));
 
        return frames;
 
@@ -1028,7 +1030,7 @@ TempoMap::bbt_duration_at_unlocked (const BBT_Time& when, const BBT_Time& bbt, i
 {
        framecnt_t frames = 0;
 
-       double beats_per_bar;
+       double divisions_per_bar;
        BBT_Time result;
 
        result.bars = max(1U, when.bars + dir * bbt.bars) ;
@@ -1036,7 +1038,7 @@ TempoMap::bbt_duration_at_unlocked (const BBT_Time& when, const BBT_Time& bbt, i
        result.ticks = 0;
 
        TempoMetric     metric = metric_at(result);
-       beats_per_bar = metric.meter().beats_per_bar();
+       divisions_per_bar = metric.meter().divisions_per_bar();
 
        /* Reduce things to legal bbt values we have to handle possible
          fractional=shorter beats at the end of measures and things like 0|11|9000
@@ -1048,11 +1050,11 @@ TempoMap::bbt_duration_at_unlocked (const BBT_Time& when, const BBT_Time& bbt, i
                result.beats = when.beats +  bbt.beats;
                result.ticks = when.ticks +  bbt.ticks;
 
-               while (result.beats >= (beats_per_bar + 1)) {
+               while (result.beats >= (divisions_per_bar + 1)) {
                        result.bars++;
-                       result.beats -=  (uint32_t) ceil(beats_per_bar);
+                       result.beats -=  (uint32_t) ceil(divisions_per_bar);
                        metric = metric_at(result); // maybe there is a meter change
-                       beats_per_bar = metric.meter().beats_per_bar();
+                       divisions_per_bar = metric.meter().divisions_per_bar();
 
                }
 
@@ -1066,21 +1068,21 @@ TempoMap::bbt_duration_at_unlocked (const BBT_Time& when, const BBT_Time& bbt, i
                /* of course gtk_ardour only allows bar with at least 1.0 beats .....
                 */
 
-               uint32_t ticks_at_beat = (uint32_t) (result.beats == ceil(beats_per_bar) ?
-                                       (1 - (ceil(beats_per_bar) - beats_per_bar))* BBT_Time::ticks_per_beat
+               uint32_t ticks_at_beat = (uint32_t) (result.beats == ceil(divisions_per_bar) ?
+                                       (1 - (ceil(divisions_per_bar) - divisions_per_bar))* BBT_Time::ticks_per_beat
                                           : BBT_Time::ticks_per_beat );
 
                while (result.ticks >= ticks_at_beat) {
                        result.beats++;
                        result.ticks -= ticks_at_beat;
-                       if  (result.beats >= (beats_per_bar + 1)) {
+                       if  (result.beats >= (divisions_per_bar + 1)) {
                                result.bars++;
                                result.beats = 1;
                                metric = metric_at(result); // maybe there is a meter change
-                               beats_per_bar = metric.meter().beats_per_bar();
+                               divisions_per_bar = metric.meter().divisions_per_bar();
                        }
-                       ticks_at_beat= (uint32_t) (result.beats == ceil(beats_per_bar) ?
-                                      (1 - (ceil(beats_per_bar) - beats_per_bar) ) * BBT_Time::ticks_per_beat
+                       ticks_at_beat= (uint32_t) (result.beats == ceil(divisions_per_bar) ?
+                                      (1 - (ceil(divisions_per_bar) - divisions_per_bar) ) * BBT_Time::ticks_per_beat
                                       : BBT_Time::ticks_per_beat);
                }
 
@@ -1093,11 +1095,11 @@ TempoMap::bbt_duration_at_unlocked (const BBT_Time& when, const BBT_Time& bbt, i
                        --result.bars;
                        result.bars = max(1U, result.bars);
                        metric = metric_at(result); // maybe there is a meter change
-                       beats_per_bar = metric.meter().beats_per_bar();
-                       if (b >= ceil(beats_per_bar)) {
-                               b -= (uint32_t) ceil(beats_per_bar);
+                       divisions_per_bar = metric.meter().divisions_per_bar();
+                       if (b >= ceil(divisions_per_bar)) {
+                               b -= (uint32_t) ceil(divisions_per_bar);
                        } else {
-                               b = (uint32_t) ceil(beats_per_bar) - b + when.beats ;
+                               b = (uint32_t) ceil(divisions_per_bar) - b + when.beats ;
                        }
                }
                result.beats = when.beats - b;
@@ -1117,9 +1119,9 @@ TempoMap::bbt_duration_at_unlocked (const BBT_Time& when, const BBT_Time& bbt, i
                                        --result.bars;
                                        result.bars = max(1U, result.bars) ;
                                        metric = metric_at(result); // maybe there is a meter change
-                                       beats_per_bar = metric.meter().beats_per_bar();
-                                       result.beats = (uint32_t) ceil(beats_per_bar);
-                                       ticks_at_beat = (uint32_t) ((1 - (ceil(beats_per_bar) - beats_per_bar)) * BBT_Time::ticks_per_beat) ;
+                                       divisions_per_bar = metric.meter().divisions_per_bar();
+                                       result.beats = (uint32_t) ceil(divisions_per_bar);
+                                       ticks_at_beat = (uint32_t) ((1 - (ceil(divisions_per_bar) - divisions_per_bar)) * BBT_Time::ticks_per_beat) ;
                                } else {
                                        --result.beats;
                                        ticks_at_beat = (uint32_t) BBT_Time::ticks_per_beat;
@@ -1283,7 +1285,7 @@ TempoMap::round_to_type (framepos_t frame, int dir, BBTPointType type)
                        float midbar_beats;
                        float midbar_ticks;
 
-                       midbar_beats = metric.meter().beats_per_bar() / 2 + 1;
+                       midbar_beats = metric.meter().divisions_per_bar() / 2 + 1;
                        midbar_ticks = BBT_Time::ticks_per_beat * fmod (midbar_beats, 1.0f);
                        midbar_beats = floor (midbar_beats);
 
@@ -1370,7 +1372,7 @@ TempoMap::get_points (framepos_t lower, framepos_t upper) const
        const TempoSection* t;
        uint32_t bar;
        uint32_t beat;
-       double beats_per_bar;
+       double divisions_per_bar;
        double beat_frame;
        double beat_frames;
        double frames_per_bar;
@@ -1406,7 +1408,7 @@ TempoMap::get_points (framepos_t lower, framepos_t upper) const
           Now start generating points.
        */
 
-       beats_per_bar = meter->beats_per_bar ();
+       divisions_per_bar = meter->divisions_per_bar ();
        frames_per_bar = meter->frames_per_bar (*tempo, _frame_rate);
        beat_frames = meter->frames_per_division (*tempo,_frame_rate);
 
@@ -1426,7 +1428,7 @@ TempoMap::get_points (framepos_t lower, framepos_t upper) const
        */
 
        delta_bars = (lower-current) / frames_per_bar;
-       delta_beats = modf(delta_bars, &dummy) * beats_per_bar;
+       delta_beats = modf(delta_bars, &dummy) * divisions_per_bar;
        current += (floor(delta_bars) * frames_per_bar) +  (floor(delta_beats) * beat_frames);
 
        // adjust bars and beats too
@@ -1463,7 +1465,7 @@ TempoMap::get_points (framepos_t lower, framepos_t upper) const
 
                        beat_frame = current;
 
-                       while (beat <= ceil(beats_per_bar) && beat_frame < limit) {
+                       while (beat <= ceil(divisions_per_bar) && beat_frame < limit) {
                                if (beat_frame >= lower) {
                                        // cerr << "Add Beat at " << bar << '|' << beat << " @ " << beat_frame << endl;
                                        points->push_back (BBTPoint (*meter, *tempo, (framepos_t) rint(beat_frame), Beat, bar, beat));
@@ -1474,11 +1476,12 @@ TempoMap::get_points (framepos_t lower, framepos_t upper) const
                                beat++;
                        }
 
-                       //  cerr << "out of beats, @ end ? " << (i == metrics->end()) << " out of bpb ? "
-                       // << (beat > ceil(beats_per_bar))
+                       // cerr << "out of beats, @ end ? " << (i == metrics->end()) << " out of bpb ? "
+                       // << (beat > ceil(divisions_per_bar))
+                       // << " beat frame @ " << beat_frame << " vs. " << limit
                        // << endl;
 
-                       if (beat > ceil(beats_per_bar) || i != metrics->end()) {
+                       if (beat > ceil(divisions_per_bar) || i != metrics->end()) {
 
                                /* we walked an entire bar. its
                                   important to move `current' forward
@@ -1496,9 +1499,9 @@ TempoMap::get_points (framepos_t lower, framepos_t upper) const
                                   so we subtract the possible extra fraction from the current
                                */
 
-                               if (beat > ceil (beats_per_bar)) {
+                               if (beat > ceil (divisions_per_bar)) {
                                        /* next bar goes where the numbers suggest */
-                                       current -=  beat_frames * (ceil(beats_per_bar)-beats_per_bar);
+                                       current -=  beat_frames * (ceil(divisions_per_bar)-divisions_per_bar);
                                        // cerr << "++ next bar from numbers\n";
                                } else {
                                        /* next bar goes where the next metric is */
@@ -1530,11 +1533,11 @@ TempoMap::get_points (framepos_t lower, framepos_t upper) const
                                /* new MeterSection, beat always returns to 1 */
                                beat = 1;
                        }
-
+                       
                        current = (*i)->frame ();
                        // cerr << "loop around with current @ " << current << endl;
 
-                       beats_per_bar = meter->beats_per_bar ();
+                       divisions_per_bar = meter->divisions_per_bar ();
                        frames_per_bar = meter->frames_per_bar (*tempo, _frame_rate);
                        beat_frames = meter->frames_per_division (*tempo, _frame_rate);
 
@@ -1672,7 +1675,7 @@ TempoMap::dump (std::ostream& o) const
                        o << "Tempo @ " << *i << ' ' << t->beats_per_minute() << " BPM (denom = " << t->note_type() << ") at " << t->start() << " frame= " << t->frame() << " (move? "
                          << t->movable() << ')' << endl;
                } else if ((m = dynamic_cast<const MeterSection*>(*i)) != 0) {
-                       o << "Meter @ " << *i << ' ' << m->beats_per_bar() << '/' << m->note_divisor() << " at " << m->start() << " frame= " << m->frame()
+                       o << "Meter @ " << *i << ' ' << m->divisions_per_bar() << '/' << m->note_divisor() << " at " << m->start() << " frame= " << m->frame()
                          << " (move? " << m->movable() << ')' << endl;
                }
        }
@@ -1801,7 +1804,7 @@ TempoMap::bbt_add (const BBT_Time& start, const BBT_Time& increment, const Tempo
 
                /* given the current meter, have we gone past the end of the bar ? */
 
-               if (result.beats >= meter->beats_per_bar()) {
+               if (result.beats >= meter->divisions_per_bar()) {
                        /* move to next bar, first beat */
                        result.bars++;
                        result.beats = 1;
@@ -1909,7 +1912,7 @@ TempoMap::bbt_subtract (const BBT_Time& start, const BBT_Time& decrement) const
                        }
 
                        result.bars--;
-                       result.beats = meter->beats_per_bar();
+                       result.beats = meter->divisions_per_bar();
                } else {
 
                        /* back one beat */
@@ -1991,14 +1994,14 @@ TempoMap::framepos_plus_beats (framepos_t pos, Evoral::MusicalTime beats) const
                framecnt_t distance_frames = i == metrics->end() ? max_framepos : ((*i)->frame() - pos);
 
                /* Distance to the end in beats */
-               Evoral::MusicalTime distance_beats = distance_frames / tempo->frames_per_beat (_frame_rate, *meter);
+               Evoral::MusicalTime distance_beats = distance_frames / tempo->frames_per_beat (_frame_rate);
 
                /* Amount to subtract this time */
                double const sub = min (distance_beats, beats);
 
                /* Update */
                beats -= sub;
-               pos += sub * tempo->frames_per_beat (_frame_rate, *meter);
+               pos += sub * tempo->frames_per_beat (_frame_rate);
 
                /* Move on if there's anything to move to */
                if (i != metrics->end ()) {
@@ -2075,14 +2078,14 @@ TempoMap::framepos_minus_beats (framepos_t pos, Evoral::MusicalTime beats) const
                framecnt_t distance_frames = no_more_metrics ? max_framepos : (pos - (*i)->frame());
 
                /* Distance to the end in beats */
-               Evoral::MusicalTime distance_beats = distance_frames / tempo->frames_per_beat (_frame_rate, *meter);
+               Evoral::MusicalTime distance_beats = distance_frames / tempo->frames_per_beat (_frame_rate);
 
                /* Amount to subtract this time */
                double const sub = min (distance_beats, beats);
 
                /* Update */
                beats -= sub;
-               pos -= sub * tempo->frames_per_beat (_frame_rate, *meter);
+               pos -= sub * tempo->frames_per_beat (_frame_rate);
 
                /* Move i, tempo and meter back, if there's anything to move to.
                   This is more complicated than the forward case, as we have to
@@ -2182,7 +2185,7 @@ TempoMap::framepos_plus_bbt (framepos_t pos, BBT_Time op) const
           checking for a new metric on every beat.
        */
 
-       frames_per_beat = tempo->frames_per_beat (_frame_rate, *meter);
+       frames_per_beat = tempo->frames_per_beat (_frame_rate);
 
        uint64_t bars = 0;
 
@@ -2204,7 +2207,7 @@ TempoMap::framepos_plus_bbt (framepos_t pos, BBT_Time op) const
                                 * frames_per_beat value.
                                 */
                                
-                               pos += llrint (frames_per_beat * (bars * meter->beats_per_bar()));
+                               pos += llrint (frames_per_beat * (bars * meter->divisions_per_bar()));
                                bars = 0;
 
                                if ((t = dynamic_cast<const TempoSection*>(*i)) != 0) {
@@ -2213,14 +2216,14 @@ TempoMap::framepos_plus_bbt (framepos_t pos, BBT_Time op) const
                                        meter = m;
                                }
                                ++i;
-                               frames_per_beat = tempo->frames_per_beat (_frame_rate, *meter);
+                               frames_per_beat = tempo->frames_per_beat (_frame_rate);
 
                        }
                }
 
        }
 
-       pos += llrint (frames_per_beat * (bars * meter->beats_per_bar()));
+       pos += llrint (frames_per_beat * (bars * meter->divisions_per_bar()));
 
        uint64_t beats = 0;
 
@@ -2253,7 +2256,7 @@ TempoMap::framepos_plus_bbt (framepos_t pos, BBT_Time op) const
                                        meter = m;
                                }
                                ++i;
-                               frames_per_beat = tempo->frames_per_beat (_frame_rate, *meter);
+                               frames_per_beat = tempo->frames_per_beat (_frame_rate);
                        }
                }
        }
@@ -2324,7 +2327,7 @@ TempoMap::framewalk_to_beats (framepos_t pos, framecnt_t distance) const
                /* Update */
                pos += sub;
                distance -= sub;
-               beats += sub / tempo->frames_per_beat (_frame_rate, *meter);
+               beats += sub / tempo->frames_per_beat (_frame_rate);
 
                /* Move on if there's anything to move to */
                if (i != metrics->end ()) {
@@ -2373,3 +2376,28 @@ MetricSection::compare (MetricSection* other, bool with_bbt) const
        /* NOTREACHED */
        return 0;
 }
+
+std::ostream& 
+operator<< (std::ostream& o, const Meter& m) {
+       return o << m.divisions_per_bar() << '/' << m.note_divisor();
+}
+std::ostream& 
+operator<< (std::ostream& o, const Tempo& t) {
+       return o << t.beats_per_minute() << " (1/" << t.note_type() << " per minute)" << endl;
+}
+std::ostream& 
+operator<< (std::ostream& o, const MetricSection& section) {
+
+       o << "MetricSection @ " << section.frame() << " aka " << section.start() << ' ';
+
+       const TempoSection* ts;
+       const MeterSection* ms;
+
+       if ((ts = dynamic_cast<const TempoSection*> (&section)) != 0) {
+               o << *((Tempo*) ts);
+       } else if ((ms = dynamic_cast<const MeterSection*> (&section)) != 0) {
+               o << *((Meter*) ms);
+       }
+
+       return o;
+}
index a46bc56c2d27bb48931422e96b427c20606f163f..e53d31bff8f5ec791cf16c99142178f722a23730 100644 (file)
@@ -157,10 +157,7 @@ void MidiClockTicker::tick (const framepos_t& transport_frames, const Timecode::
 double MidiClockTicker::one_ppqn_in_frames (framepos_t transport_position)
 {
        const Tempo& current_tempo = _session->tempo_map().tempo_at(transport_position);
-       const Meter& current_meter = _session->tempo_map().meter_at(transport_position);
-       double frames_per_beat =
-               current_tempo.frames_per_beat(_session->nominal_frame_rate(),
-                                             current_meter);
+       double frames_per_beat = current_tempo.frames_per_beat(_session->nominal_frame_rate());
 
        double quarter_notes_per_beat = 4.0 / current_tempo.note_type();
        double frames_per_quarter_note = frames_per_beat / quarter_notes_per_beat;
index 65a03f483debd2f18228a9532979e8fc2f9e06dd..7e208686ba23254774a198a1714b28bf08eb59aa 100644 (file)
@@ -1021,7 +1021,7 @@ MackieControlProtocol::format_bbt_timecode (framepos_t now_frame)
        // figure out subdivisions per beat
        const Meter & meter = session->tempo_map().meter_at (now_frame);
        int subdiv = 2;
-       if (meter.note_divisor() == 8 && (meter.beats_per_bar() == 12.0 || meter.beats_per_bar() == 9.0 || meter.beats_per_bar() == 6.0)) {
+       if (meter.note_divisor() == 8 && (meter.divisions_per_bar() == 12.0 || meter.divisions_per_bar() == 9.0 || meter.divisions_per_bar() == 6.0)) {
                subdiv = 3;
        }