new automation state model, sort of working, but not really
[ardour.git] / libs / ardour / playlist.cc
index 3ef817cc5557bedd90118197442d4a72c78ad8f3..139f62cb91d3bdefff3a8be1df263bd88d951206 100644 (file)
@@ -99,25 +99,24 @@ Playlist::Playlist (const Playlist& other, string namestr, bool hide)
        RegionList tmp;
        other.copy_regions (tmp);
        
-       in_set_state = true;
+       in_set_state++;
 
        for (list<boost::shared_ptr<Region> >::iterator x = tmp.begin(); x != tmp.end(); ++x) {
                add_region_internal( (*x), (*x)->position() );
        }
 
-       in_set_state = false;
+       in_set_state--;
 
        _splicing  = other._splicing;
        _nudging   = other._nudging;
        _edit_mode = other._edit_mode;
 
-       in_set_state = false;
+       in_set_state = 0;
        in_flush = false;
        in_partition = false;
        subcnt = 0;
        _read_data_count = 0;
        _frozen = other._frozen;
-       save_on_thaw = false;
        
        layer_op_counter = other.layer_op_counter;
        freeze_length = other.freeze_length;
@@ -231,14 +230,13 @@ Playlist::init (bool hide)
        _hidden = hide;
        _splicing = false;
        _nudging = false;
-       in_set_state = false;
+       in_set_state = 0;
        _edit_mode = Config->get_edit_mode();
        in_flush = false;
        in_partition = false;
        subcnt = 0;
        _read_data_count = 0;
        _frozen = false;
-       save_on_thaw = false;
        layer_op_counter = 0;
        freeze_length = 0;
 
@@ -333,9 +331,10 @@ void
 Playlist::notify_region_removed (boost::shared_ptr<Region> r)
 {
        if (holding_state ()) {
-               pending_removals.insert (pending_removals.end(), r);
+               pending_removes.insert (r);
+               pending_modified = true;
+               pending_length = true;
        } else {
-               RegionRemoved (r); /* EMIT SIGNAL */
                /* this might not be true, but we have to act
                   as though it could be.
                */
@@ -347,13 +346,15 @@ Playlist::notify_region_removed (boost::shared_ptr<Region> r)
 void
 Playlist::notify_region_added (boost::shared_ptr<Region> r)
 {
+       /* the length change might not be true, but we have to act
+          as though it could be.
+       */
+
        if (holding_state()) {
-               pending_adds.insert (pending_adds.end(), r);
+               pending_adds.insert (r);
+               pending_modified = true;
+               pending_length = true;
        } else {
-               RegionAdded (r); /* EMIT SIGNAL */
-               /* this might not be true, but we have to act
-                  as though it could be.
-               */
                LengthChanged (); /* EMIT SIGNAL */
                Modified (); /* EMIT SIGNAL */
        }
@@ -373,9 +374,8 @@ Playlist::notify_length_changed ()
 void
 Playlist::flush_notifications ()
 {
-       RegionList::iterator r;
-       RegionList::iterator a;
        set<boost::shared_ptr<Region> > dependent_checks_needed;
+       set<boost::shared_ptr<Region> >::iterator s;
        uint32_t n = 0;
 
        if (in_flush) {
@@ -406,19 +406,17 @@ Playlist::flush_notifications ()
                /* don't increment n again - its the same list */
        }
 
-       for (a = pending_adds.begin(); a != pending_adds.end(); ++a) {
-               dependent_checks_needed.insert (*a);
-               RegionAdded (*a); /* EMIT SIGNAL */
+       for (s = pending_adds.begin(); s != pending_adds.end(); ++s) {
+               dependent_checks_needed.insert (*s);
                n++;
        }
 
-       for (set<boost::shared_ptr<Region> >::iterator x = dependent_checks_needed.begin(); x != dependent_checks_needed.end(); ++x) {
-               check_dependents (*x, false);
+       for (s = dependent_checks_needed.begin(); s != dependent_checks_needed.end(); ++s) {
+               check_dependents (*s, false);
        }
 
-       for (r = pending_removals.begin(); r != pending_removals.end(); ++r) {
-               remove_dependents (*r);
-               RegionRemoved (*r); /* EMIT SIGNAL */
+       for (s = pending_removes.begin(); s != pending_removes.end(); ++s) {
+               remove_dependents (*s);
                n++;
        }
 
@@ -438,14 +436,9 @@ Playlist::flush_notifications ()
        }
 
        pending_adds.clear ();
-       pending_removals.clear ();
+       pending_removes.clear ();
        pending_bounds.clear ();
 
-       if (save_on_thaw) {
-               save_on_thaw = false;
-               save_state (last_save_reason);
-       }
-       
        in_flush = false;
 }
 
@@ -454,7 +447,7 @@ Playlist::flush_notifications ()
  *************************************************************/
 
 void
-Playlist::add_region (boost::shared_ptr<Region> region, nframes_t position, float times, bool with_save
+Playlist::add_region (boost::shared_ptr<Region> region, nframes_t position, float times) 
 { 
        RegionLock rlock (this);
        
@@ -493,10 +486,6 @@ Playlist::add_region (boost::shared_ptr<Region> region, nframes_t position, floa
                boost::shared_ptr<Region> sub = RegionFactory::create (region, 0, length, name, region->layer(), region->flags());
                add_region_internal (sub, pos, true);
        }
-       
-       if (with_save) {
-               maybe_save_state (_("add region"));
-       }
 }
 
 void
@@ -515,6 +504,7 @@ Playlist::add_region_internal (boost::shared_ptr<Region> region, nframes_t posit
        timestamp_layer_op (region);
 
        regions.insert (upper_bound (regions.begin(), regions.end(), region, cmp), region);
+       all_regions.insert (region);
 
        if (!holding_state () && !in_set_state) {
                /* layers get assigned from XML state */
@@ -547,8 +537,6 @@ Playlist::replace_region (boost::shared_ptr<Region> old, boost::shared_ptr<Regio
        if (!holding_state ()) {
                possibly_splice_unlocked ();
        }
-
-       maybe_save_state (_("replace region"));
 }
 
 void
@@ -560,8 +548,6 @@ Playlist::remove_region (boost::shared_ptr<Region> region)
        if (!holding_state ()) {
                possibly_splice_unlocked ();
        }
-
-       maybe_save_state (_("remove region"));
 }
 
 int
@@ -570,7 +556,7 @@ Playlist::remove_region_internal (boost::shared_ptr<Region>region, bool delay_so
        RegionList::iterator i;
        nframes_t old_length = 0;
 
-       // cerr << "removing region " << region->name() << endl;
+       cerr << "removing region " << region->name() << " holding = " << holding_state() << endl;
 
        if (!holding_state()) {
                old_length = _get_maximum_extent();
@@ -632,8 +618,6 @@ Playlist::partition (nframes_t start, nframes_t end, bool just_top_level)
        for (RegionList::iterator i = thawlist.begin(); i != thawlist.end(); ++i) {
                (*i)->thaw ("separation");
        }
-
-       maybe_save_state (_("separate"));
 }
 
 void
@@ -897,8 +881,6 @@ Playlist::cut (nframes_t start, nframes_t cnt, bool result_is_hidden)
                (*i)->thaw ("playlist cut");
        }
 
-       maybe_save_state (_("cut"));
-
        return the_copy;
 }
 
@@ -958,8 +940,6 @@ Playlist::paste (Playlist& other, nframes_t position, float times)
                
        }
 
-       maybe_save_state (_("paste"));
-
        return 0;
 }
 
@@ -986,8 +966,6 @@ Playlist::duplicate (boost::shared_ptr<Region> region, nframes_t position, float
                boost::shared_ptr<Region> sub = RegionFactory::create (region, 0, length, name, region->layer(), region->flags());
                add_region_internal (sub, pos, true);
        }
-
-       maybe_save_state (_("duplicate"));
 }
 
 void
@@ -1041,8 +1019,6 @@ Playlist::split_region (boost::shared_ptr<Region> region, nframes_t playlist_pos
        if (remove_region_internal (region, true)) {
                return;
        }
-
-       maybe_save_state (_("split"));
 }
 
 void
@@ -1192,24 +1168,21 @@ Playlist::region_changed (Change what_changed, boost::shared_ptr<Region> region)
 }
 
 void
-Playlist::clear (bool with_save)
+Playlist::clear (bool with_signals)
 {
-       RegionList::iterator i;
-       RegionList tmp;
-
        { 
                RegionLock rl (this);
-               tmp = regions;
+               for (RegionList::iterator i = regions.begin(); i != regions.end(); ++i) {
+                       pending_removes.insert (*i);
+               }
                regions.clear ();
        }
-       
-       for (i = tmp.begin(); i != tmp.end(); ++i) {
-               notify_region_removed (*i);
-       }
 
-       if (with_save) {
-               maybe_save_state (_("clear"));
+       if (with_signals) {
+               LengthChanged ();
+               Modified ();
        }
+
 }
 
 /***********************************************************************
@@ -1339,8 +1312,6 @@ Playlist::mark_session_dirty ()
 int
 Playlist::set_state (const XMLNode& node)
 {
-       in_set_state = true;
-
        XMLNode *child;
        XMLNodeList nlist;
        XMLNodeConstIterator niter;
@@ -1350,13 +1321,15 @@ Playlist::set_state (const XMLNode& node)
        boost::shared_ptr<Region> region;
        string region_name;
 
-       clear (false);
+       in_set_state++;
 
        if (node.name() != "Playlist") {
-               in_set_state = false;
+               in_set_state--;
                return -1;
        }
 
+       freeze ();
+
        plist = node.properties();
 
        for (piter = plist.begin(); piter != plist.end(); ++piter) {
@@ -1372,6 +1345,8 @@ Playlist::set_state (const XMLNode& node)
                }
        }
 
+       clear (false);
+       
        nlist = node.children();
 
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
@@ -1385,19 +1360,29 @@ Playlist::set_state (const XMLNode& node)
                                continue;
                        }
                        
-                       if ((region = RegionFactory::create (_session, *child, true)) == 0) {
-                               error << _("Playlist: cannot create region from state file") << endmsg;
+                       ID id = prop->value ();
+                       
+                       if ((region = region_by_id (id))) {
+
+                               Change what_changed = Change (0);
+
+                               if (region->set_live_state (*child, what_changed, true)) {
+                                       error << _("Playlist: cannot reset region state from XML") << endmsg;
+                                       continue;
+                               }
+
+                       } else if ((region = RegionFactory::create (_session, *child, true)) == 0) {
+                               error << _("Playlist: cannot create region from XML") << endmsg;
                                continue;
                        }
 
-                       add_region (region, region->position(), 1.0, false);
+                       add_region (region, region->position(), 1.0);
 
                        // So that layer_op ordering doesn't get screwed up
                        region->set_last_layer_op( region->layer());
 
                }                       
        }
-
        
        /* update dependents, which was not done during add_region_internal 
           due to in_set_state being true 
@@ -1406,8 +1391,12 @@ Playlist::set_state (const XMLNode& node)
        for (RegionList::iterator r = regions.begin(); r != regions.end(); ++r) {
                check_dependents (*r, false);
        }
-       
-       in_set_state = false;
+
+       notify_modified ();
+
+       thaw ();
+
+       in_set_state--;
 
        return 0;
 }
@@ -1438,7 +1427,6 @@ Playlist::state (bool full_state)
 
        if (full_state) {
                RegionLock rlock (this, false);
-
                for (RegionList::iterator i = regions.begin(); i != regions.end(); ++i) {
                        node->add_child_nocopy ((*i)->get_state());
                }
@@ -1746,7 +1734,6 @@ Playlist::nudge_after (nframes_t start, nframes_t distance, bool forwards)
 
        if (moved) {
                _nudging = false;
-               maybe_save_state (_("nudged"));
                notify_length_changed ();
        }
 
@@ -1756,26 +1743,31 @@ boost::shared_ptr<Region>
 Playlist::find_region (const ID& id) const
 {
        RegionLock rlock (const_cast<Playlist*> (this));
-       RegionList::const_iterator i;
-       boost::shared_ptr<Region> ret;
 
-       for (i = regions.begin(); i != regions.end(); ++i) {
+       /* searches all regions currently in use by the playlist */
+
+       for (RegionList::const_iterator i = regions.begin(); i != regions.end(); ++i) {
                if ((*i)->id() == id) {
-                       ret = *i;
+                       return *i;
                }
        }
 
-       return ret;
+       return boost::shared_ptr<Region> ();
 }
-       
-void
-Playlist::save_state (std::string why)
+
+boost::shared_ptr<Region>
+Playlist::region_by_id (ID id)
 {
-       if (!in_set_state) {
-               StateManager::save_state (why);
+       /* searches all regions ever added to this playlist */
+
+       for (set<boost::shared_ptr<Region> >::iterator i = all_regions.begin(); i != all_regions.end(); ++i) {
+               if ((*i)->id() == id) {
+                       return *i;
+               }
        }
+       return boost::shared_ptr<Region> ();
 }
-
+       
 void
 Playlist::dump () const
 {
@@ -1811,13 +1803,3 @@ Playlist::timestamp_layer_op (boost::shared_ptr<Region> region)
        region->set_last_layer_op (++layer_op_counter);
 }
 
-void
-Playlist::maybe_save_state (string why)
-{
-       if (holding_state ()) {
-               save_on_thaw = true;
-               last_save_reason = why;
-       } else {
-               save_state (why);
-       }
-}