slightly improved fixes for MIDI issues
[ardour.git] / libs / ardour / midi_playlist.cc
index 7f2781d707779c840a50551460f6c9a914c3ef0a..27410462d1b85a3ed6778a4c6cabf33f87b99ea1 100644 (file)
@@ -40,42 +40,26 @@ using namespace ARDOUR;
 using namespace sigc;
 using namespace std;
 
-MidiPlaylist::State::~State ()
-{}
-
 MidiPlaylist::MidiPlaylist (Session& session, const XMLNode& node, bool hidden)
                : Playlist (session, node, DataType::MIDI, hidden)
 {
        const XMLProperty* prop = node.property("type");
        assert(prop && DataType(prop->value()) == DataType::MIDI);
 
-       in_set_state = true;
+       in_set_state++;
        set_state (node);
-       in_set_state = false;
-
-       save_state (_("initial state"));
-
-       if (!hidden) {
-               PlaylistCreated (this); /* EMIT SIGNAL */
-       }
+       in_set_state--;
 }
 
 MidiPlaylist::MidiPlaylist (Session& session, string name, bool hidden)
                : Playlist (session, name, DataType::MIDI, hidden)
 {
-       save_state (_("initial state"));
-
-       if (!hidden) {
-               PlaylistCreated (this); /* EMIT SIGNAL */
-       }
-
 }
 
-MidiPlaylist::MidiPlaylist (const MidiPlaylist& other, string name, bool hidden)
+MidiPlaylist::MidiPlaylist (boost::shared_ptr<const MidiPlaylist> other, string name, bool hidden)
                : Playlist (other, name, hidden)
 {
        throw; // nope
-       save_state (_("initial state"));
 
        /*
        list<Region*>::const_iterator in_o  = other.regions.begin();
@@ -114,28 +98,21 @@ MidiPlaylist::MidiPlaylist (const MidiPlaylist& other, string name, bool hidden)
                in_n++;
        }
 */
-       if (!hidden) {
-               PlaylistCreated (this); /* EMIT SIGNAL */
-       }
 }
 
-MidiPlaylist::MidiPlaylist (const MidiPlaylist& other, jack_nframes_t start, jack_nframes_t dur, string name, bool hidden)
+MidiPlaylist::MidiPlaylist (boost::shared_ptr<const MidiPlaylist> other, nframes_t start, nframes_t dur, string name, bool hidden)
                : Playlist (other, start, dur, name, hidden)
 {
-       save_state (_("initial state"));
-
        /* this constructor does NOT notify others (session) */
 }
 
 MidiPlaylist::~MidiPlaylist ()
 {
        GoingAway (); /* EMIT SIGNAL */
-
-       for (StateMap::iterator i = states.begin(); i != states.end(); ++i) {
-
-               MidiPlaylist::State* apstate = dynamic_cast<MidiPlaylist::State*> (*i);
-               delete apstate;
-       }
+       
+       /* drop connections to signals */
+       
+       notify_callbacks ();
 }
 
 struct RegionSortByLayer {
@@ -145,9 +122,9 @@ struct RegionSortByLayer {
 };
 
 /** Returns the number of frames in time duration read (eg could be large when 0 events are read) */
-jack_nframes_t
-MidiPlaylist::read (MidiRingBuffer& dst, jack_nframes_t start,
-                     jack_nframes_t dur, unsigned chan_n)
+nframes_t
+MidiPlaylist::read (MidiRingBuffer& dst, nframes_t start,
+                     nframes_t dur, unsigned chan_n)
 {
        /* this function is never called from a realtime thread, so
           its OK to block (for short intervals).
@@ -155,12 +132,9 @@ MidiPlaylist::read (MidiRingBuffer& dst, jack_nframes_t start,
 
        Glib::Mutex::Lock rm (region_lock);
 
-       jack_nframes_t ret         = 0;
-       jack_nframes_t end         = start + dur - 1;
-       //jack_nframes_t read_frames = 0;
-       //jack_nframes_t skip_frames = 0;
+       nframes_t end = start + dur - 1;
 
-       //_read_data_count = 0;
+       _read_data_count = 0;
 
        // relevent regions overlapping start <--> end
        vector<boost::shared_ptr<Region> > regs;
@@ -176,15 +150,14 @@ MidiPlaylist::read (MidiRingBuffer& dst, jack_nframes_t start,
        sort(regs.begin(), regs.end(), layer_cmp);
 
        for (vector<boost::shared_ptr<Region> >::iterator i = regs.begin(); i != regs.end(); ++i) {
-               // FIXME: ensure time is monotonic here
+               // FIXME: ensure time is monotonic here?
                boost::shared_ptr<MidiRegion> mr = boost::dynamic_pointer_cast<MidiRegion>(*i);
-               mr->read_at (dst, start, dur, chan_n, 0, 0);// FIXME read_frames, skip_frames);
-               ret += mr->read_data_count();
+               if (mr) {
+                       mr->read_at (dst, start, dur, chan_n, _note_mode);
+                       _read_data_count += mr->read_data_count();
+               }
        }
 
-       _read_data_count += ret;
-       
-       //return ret; FIXME?
        return dur;
 }
 
@@ -192,26 +165,14 @@ MidiPlaylist::read (MidiRingBuffer& dst, jack_nframes_t start,
 void
 MidiPlaylist::remove_dependents (boost::shared_ptr<Region> region)
 {
+       /* MIDI regions have no dependents (crossfades) */
 }
 
 
-void
-MidiPlaylist::flush_notifications ()
-{
-       Playlist::flush_notifications();
-
-       if (in_flush) {
-               return;
-       }
-
-       in_flush = true;
-
-       in_flush = false;
-}
-
 void
 MidiPlaylist::refresh_dependents (boost::shared_ptr<Region> r)
 {
+       /* MIDI regions have no dependents (crossfades) */
 }
 
 void
@@ -259,116 +220,24 @@ MidiPlaylist::finalize_split_region (boost::shared_ptr<Region> original, boost::
 void
 MidiPlaylist::check_dependents (boost::shared_ptr<Region> r, bool norefresh)
 {
+       /* MIDI regions have no dependents (crossfades) */
 }
 
 
 int
 MidiPlaylist::set_state (const XMLNode& node)
 {
-       if (!in_set_state) {
-               Playlist::set_state (node);
-       }
-
-       // Actually Charles, I don't much care for children
-       
-       /*
-       XMLNodeList nlist = node.children();
+       in_set_state++;
+       freeze ();
 
-       for (XMLNodeConstIterator niter = nlist.begin(); niter != nlist.end(); ++niter) {
+       Playlist::set_state (node);
 
-               XMLNode* const child = *niter;
-
-       }*/
+       thaw();
+       in_set_state--;
 
        return 0;
 }
 
-void
-MidiPlaylist::drop_all_states ()
-{
-       set<boost::shared_ptr<Region> > all_regions;
-
-       /* find every region we've ever used, and add it to the set of 
-          all regions. same for xfades;
-       */
-
-       for (StateMap::iterator i = states.begin(); i != states.end(); ++i) {
-               
-               MidiPlaylist::State* apstate = dynamic_cast<MidiPlaylist::State*> (*i);
-
-               for (RegionList::iterator r = apstate->regions.begin(); r != apstate->regions.end(); ++r) {
-                       all_regions.insert (*r);
-               }
-       }
-
-       /* now remove from the "all" lists every region that is in the current list. */
-
-       for (RegionList::iterator i = regions.begin(); i != regions.end(); ++i) {
-               set<boost::shared_ptr<Region> >::iterator x = all_regions.find (*i);
-               if (x != all_regions.end()) {
-                       all_regions.erase (x);
-               }
-       }
-       
-       /* Now do the generic thing ... */
-
-       StateManager::drop_all_states ();
-}
-
-StateManager::State*
-MidiPlaylist::state_factory (std::string why) const
-{
-       State* state = new State (why);
-
-       state->regions = regions;
-       state->region_states.clear ();
-       for (RegionList::const_iterator i = regions.begin(); i != regions.end(); ++i) {
-               state->region_states.push_back ((*i)->get_memento());
-       }
-
-       return state;
-}
-
-Change
-MidiPlaylist::restore_state (StateManager::State& state)
-{
-       {
-               RegionLock rlock (this);
-               State* apstate = dynamic_cast<State*> (&state);
-
-               in_set_state = true;
-
-               regions = apstate->regions;
-
-               for (list<UndoAction>::iterator s = apstate->
-                                                   region_states.begin();
-                       s != apstate->region_states.end();
-                       ++s) {
-                       (*s) ();
-               }
-
-               in_set_state = false;
-       }
-
-       notify_length_changed ();
-       return Change (~0);
-}
-
-UndoAction
-MidiPlaylist::get_memento () const
-{
-       return sigc::bind (mem_fun (*(const_cast<MidiPlaylist*> (this)), &StateManager::use_state), _current_state_id);
-}
-
-
-XMLNode&
-MidiPlaylist::state (bool full_state)
-{
-       XMLNode& node = Playlist::state (full_state);
-
-       return node;
-}
-
 void
 MidiPlaylist::dump () const
 {
@@ -422,39 +291,6 @@ MidiPlaylist::destroy_region (boost::shared_ptr<Region> region)
                }
        }
 
-       for (StateMap::iterator s = states.begin(); s != states.end(); ) {
-               StateMap::iterator tmp;
-
-               tmp = s;
-               ++tmp;
-
-               State* astate = dynamic_cast<State*> (*s);
-
-               list<UndoAction>::iterator rsi, rsitmp;
-               RegionList::iterator ri, ritmp;
-
-               for (ri = astate->regions.begin(), rsi = astate->region_states.begin();
-                       ri != astate->regions.end() && rsi != astate->region_states.end();) {
-
-
-                       ritmp = ri;
-                       ++ritmp;
-
-                       rsitmp = rsi;
-                       ++rsitmp;
-
-                       if (region == (*ri)) {
-                               astate->regions.erase (ri);
-                               astate->region_states.erase (rsi);
-                       }
-
-                       ri = ritmp;
-                       rsi = rsitmp;
-               }
-
-               s = tmp;
-       }
-
 
        if (changed) {
                /* overload this, it normally means "removed", not destroyed */
@@ -484,8 +320,6 @@ MidiPlaylist::region_changed (Change what_changed, boost::shared_ptr<Region> reg
 
        parent_wants_notify = Playlist::region_changed (what_changed, region);
 
-       maybe_save_state (_("region modified"));
-
        if ((parent_wants_notify || (what_changed & our_interests))) {
                notify_modified ();
        }