Fix MidiBuffer::merge_in_place and add aggressive correctness checking.
[ardour.git] / libs / ardour / midi_region.cc
index c3ce6db327c97ad912fa267403b425ee8d03db5d..9b7a4b87480471c17a01a30d01c4d72d356559f1 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2000-2006 Paul Davis 
+    Copyright (C) 2000-2006 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 
 #include <glibmm/thread.h>
 
-#include <pbd/basename.h>
-#include <pbd/xml++.h>
-#include <pbd/enumwriter.h>
+#include "pbd/basename.h"
+#include "pbd/xml++.h"
+#include "pbd/enumwriter.h"
 
-#include <ardour/midi_region.h>
-#include <ardour/session.h>
-#include <ardour/gain.h>
-#include <ardour/dB.h>
-#include <ardour/playlist.h>
-#include <ardour/midi_source.h>
-#include <ardour/types.h>
-#include <ardour/midi_ring_buffer.h>
+#include "ardour/midi_region.h"
+#include "ardour/session.h"
+#include "ardour/gain.h"
+#include "ardour/dB.h"
+#include "ardour/playlist.h"
+#include "ardour/midi_source.h"
+#include "ardour/types.h"
+#include "ardour/midi_ring_buffer.h"
 
 #include "i18n.h"
 #include <locale.h>
@@ -91,7 +91,7 @@ MidiRegion::MidiRegion (boost::shared_ptr<const MidiRegion> other)
 MidiRegion::MidiRegion (boost::shared_ptr<MidiSource> src, const XMLNode& node)
        : Region (src, node)
 {
-       if (set_state (node)) {
+       if (set_state (node, Stateful::loading_state_version)) {
                throw failed_constructor();
        }
 
@@ -103,7 +103,7 @@ MidiRegion::MidiRegion (boost::shared_ptr<MidiSource> src, const XMLNode& node)
 MidiRegion::MidiRegion (const SourceList& srcs, const XMLNode& node)
        : Region (srcs, node)
 {
-       if (set_state (node)) {
+       if (set_state (node, Stateful::loading_state_version)) {
                throw failed_constructor();
        }
 
@@ -116,32 +116,44 @@ MidiRegion::~MidiRegion ()
 {
 }
 
-nframes_t
-MidiRegion::read_at (MidiRingBuffer& out, nframes_t position, nframes_t dur, uint32_t chan_n, NoteMode mode) const
+void
+MidiRegion::set_position_internal (nframes_t pos, bool allow_bbt_recompute)
 {
-       return _read_at (_sources, out, position, dur, chan_n, mode);
+       BeatsFramesConverter old_converter(_session, _position - _start);
+       double length_beats = old_converter.from(_length);
+
+       Region::set_position_internal(pos, allow_bbt_recompute);
+
+       BeatsFramesConverter new_converter(_session, pos - _start);
+
+       set_length(new_converter.to(length_beats), 0);
 }
 
 nframes_t
-MidiRegion::master_read_at (MidiRingBuffer& out, nframes_t position, nframes_t dur, uint32_t chan_n, NoteMode mode) const
+MidiRegion::read_at (MidiRingBuffer<nframes_t>& out, sframes_t position, nframes_t dur, uint32_t chan_n, NoteMode mode, MidiStateTracker* tracker) const
 {
-       return _read_at (_master_sources, out, position, dur, chan_n, mode);
+       return _read_at (_sources, out, position, dur, chan_n, mode, tracker);
 }
 
 nframes_t
-MidiRegion::_read_at (const SourceList& srcs, MidiRingBuffer& dst, nframes_t position, nframes_t dur, uint32_t chan_n, NoteMode mode) const
+MidiRegion::master_read_at (MidiRingBuffer<nframes_t>& out, sframes_t position, nframes_t dur, uint32_t chan_n, NoteMode mode) const
 {
-       cerr << "reading from region " << _name << " position: " << _position << " start: " << _start << endl;
-       cerr << _name << "._read_at(" << position << ") - " << position << " duration: " << dur << endl;
+       return _read_at (_master_sources, out, position, dur, chan_n, mode); /* no tracker */
+}
 
+nframes_t
+MidiRegion::_read_at (const SourceList& /*srcs*/,
+               MidiRingBuffer<nframes_t>& dst, sframes_t position, nframes_t dur, uint32_t chan_n,
+               NoteMode mode, MidiStateTracker* tracker) const
+{
        nframes_t internal_offset = 0;
        nframes_t src_offset      = 0;
        nframes_t to_read         = 0;
-       
+
        /* precondition: caller has verified that we cover the desired section */
 
        assert(chan_n == 0);
-       
+
        if (position < _position) {
                internal_offset = 0;
                src_offset = _position - position;
@@ -154,7 +166,6 @@ MidiRegion::_read_at (const SourceList& srcs, MidiRingBuffer& dst, nframes_t pos
        if (internal_offset >= _length) {
                return 0; /* read nothing */
        }
-       
 
        if ((to_read = min (dur, _length - internal_offset)) == 0) {
                return 0; /* read nothing */
@@ -174,27 +185,33 @@ MidiRegion::_read_at (const SourceList& srcs, MidiRingBuffer& dst, nframes_t pos
 
        nframes_t output_buffer_position = 0;
        nframes_t negative_output_buffer_position = 0;
-       if(_position >= _start) {
+       if (_position >= _start) {
                // handle resizing of beginnings of regions correctly
                output_buffer_position = _position - _start;
        } else {
                // when _start is greater than _position, we have to subtract
                // _start from the note times in the midi source
-               negative_output_buffer_position = _start; 
+               negative_output_buffer_position = _start;
        }
-       
+
+#if 0
+       cerr << "\t\tsource read from " << _position << " - " << _start << " (" << _position - _start << ") "
+            << " start in source " << _start << " + " << internal_offset << " (" << _start + internal_offset << ") "
+            << " dur = " << to_read
+            << " offset = " << output_buffer_position
+            << " negoffset = " << negative_output_buffer_position
+            << endl;
+#endif
+
        if (src->midi_read (
-                       // the destination buffer
-                       dst,  
-                       // where to start reading in the region
-                       _start + internal_offset, 
-                       // how many bytes
-                       to_read, 
-                       // the offset in the output buffer
-                       output_buffer_position,
-                       // what to substract from note times written in the output buffer
-                       negative_output_buffer_position
-               ) != to_read) {
+                       dst, // destination buffer
+                       _position - _start, // start position of the source in this read context
+                       _start + internal_offset, // where to start reading in the source
+                       to_read, // read duration in frames
+                       output_buffer_position, // the offset in the output buffer
+                       negative_output_buffer_position, // amount to substract from note times
+                       tracker
+                   ) != to_read) {
                return 0; /* "read nothing" */
        }
 
@@ -202,7 +219,7 @@ MidiRegion::_read_at (const SourceList& srcs, MidiRingBuffer& dst, nframes_t pos
 
        return to_read;
 }
-       
+
 XMLNode&
 MidiRegion::state (bool full)
 {
@@ -210,11 +227,11 @@ MidiRegion::state (bool full)
        char buf[64];
        char buf2[64];
        LocaleGuard lg (X_("POSIX"));
-       
+
        node.add_property ("flags", enum_2_string (_flags));
 
        // XXX these should move into Region
-       
+
        for (uint32_t n=0; n < _sources.size(); ++n) {
                snprintf (buf2, sizeof(buf2), "source-%d", n);
                _sources[n]->id().print (buf, sizeof(buf));
@@ -235,15 +252,15 @@ MidiRegion::state (bool full)
 }
 
 int
-MidiRegion::set_live_state (const XMLNode& node, Change& what_changed, bool send)
+MidiRegion::set_live_state (const XMLNode& node, int version, Change& what_changed, bool send)
 {
        const XMLProperty *prop;
        LocaleGuard lg (X_("POSIX"));
 
-       Region::set_live_state (node, what_changed, false);
+       Region::set_live_state (node, version, what_changed, false);
 
        uint32_t old_flags = _flags;
-               
+
        if ((prop = node.property ("flags")) != 0) {
                _flags = Flag (string_2_enum (prop->value(), _flags));
 
@@ -271,14 +288,14 @@ MidiRegion::set_live_state (const XMLNode& node, Change& what_changed, bool send
 }
 
 int
-MidiRegion::set_state (const XMLNode& node)
+MidiRegion::set_state (const XMLNode& node, int version)
 {
        /* Region::set_state() calls the virtual set_live_state(),
           which will get us back to AudioRegion::set_live_state()
           to handle the relevant stuff.
        */
 
-       return Region::set_state (node);
+       return Region::set_state (node, version);
 }
 
 void
@@ -288,7 +305,7 @@ MidiRegion::recompute_at_end ()
         * (non destructively) "chop" notes that pass the end boundary, to
         * prevent stuck notes.
         */
-}      
+}
 
 void
 MidiRegion::recompute_at_start ()
@@ -303,33 +320,9 @@ MidiRegion::recompute_at_start ()
 }
 
 int
-MidiRegion::separate_by_channel (Session& session, vector<MidiRegion*>& v) const
+MidiRegion::separate_by_channel (ARDOUR::Session&, vector< boost::shared_ptr<Region> >&) const
 {
-       // Separate by MIDI channel?  bit different from audio since this is separating based
-       // on the actual contained data and destructively modifies and creates new sources..
-       
-#if 0
-       SourceList srcs;
-       string new_name;
-
-       for (SourceList::const_iterator i = _master_sources.begin(); i != _master_sources.end(); ++i) {
-
-               srcs.clear ();
-               srcs.push_back (*i);
-
-               /* generate a new name */
-               
-               if (session.region_name (new_name, _name)) {
-                       return -1;
-               }
-
-               /* create a copy with just one source */
-
-               v.push_back (new MidiRegion (srcs, _start, _length, new_name, _layer, _flags));
-       }
-#endif
-
-       // Actually, I would prefer not if that's alright
+       // TODO
        return -1;
 }