Generic MIDI control now saves+restores its state; PBD::ID now requires a buffer...
[ardour.git] / libs / ardour / playlist.cc
index 8977135ba7e5b52d97bdd4d49c5296a41331c72f..6d6a07da95f085bfe983877cb2700f8efdd684fd 100644 (file)
@@ -41,7 +41,7 @@
 
 using namespace std;
 using namespace ARDOUR;
-//using namespace sigc;
+using namespace PBD;
 
 sigc::signal<void,Playlist*> Playlist::PlaylistCreated;
 
@@ -55,19 +55,19 @@ struct ShowMeTheList {
 };
 
 struct RegionSortByLayer {
-    bool operator() (Region *a, Region *b) {
+    bool operator() (boost::shared_ptr<Region> a, boost::shared_ptr<Region> b) {
            return a->layer() < b->layer();
     }
 };
 
 struct RegionSortByPosition {
-    bool operator() (Region *a, Region *b) {
+    bool operator() (boost::shared_ptr<Region> a, boost::shared_ptr<Region> b) {
            return a->position() < b->position();
     }
 };
 
 struct RegionSortByLastLayerOp {
-    bool operator() (Region *a, Region *b) {
+    bool operator() (boost::shared_ptr<Region> a, boost::shared_ptr<Region> b) {
            return a->last_layer_op() < b->last_layer_op();
     }
 };
@@ -77,7 +77,6 @@ Playlist::Playlist (Session& sess, string nom, bool hide)
 {
        init (hide);
        _name = nom;
-       _orig_diskstream_id = 0;
        
 }
 
@@ -86,7 +85,6 @@ Playlist::Playlist (Session& sess, const XMLNode& node, bool hide)
 {
        init (hide);
        _name = "unnamed"; /* reset by set_state */
-       _orig_diskstream_id = 0;
        
        if (set_state (node)) {
                throw failed_constructor();
@@ -98,29 +96,50 @@ Playlist::Playlist (const Playlist& other, string namestr, bool hide)
 {
        init (hide);
 
-       other.copy_regions (regions);
+       RegionList tmp;
+       other.copy_regions (tmp);
+       
+       in_set_state = true;
 
-       for (list<Region*>::iterator x = regions.begin(); x != regions.end(); ++x) {
-               (*x)->set_playlist (this);
+       for (list<boost::shared_ptr<Region> >::iterator x = tmp.begin(); x != tmp.end(); ++x) {
+               add_region_internal( (*x), (*x)->position() );
        }
+
+       in_set_state = false;
+
+       _splicing  = other._splicing;
+       _nudging   = other._nudging;
+       _edit_mode = other._edit_mode;
+
+       in_set_state = false;
+       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;
+       
 }
 
-Playlist::Playlist (const Playlist& other, jack_nframes_t start, jack_nframes_t cnt, string str, bool hide)
+Playlist::Playlist (const Playlist& other, nframes_t start, nframes_t cnt, string str, bool hide)
        : _name (str), _session (other._session), _orig_diskstream_id(other._orig_diskstream_id)
 {
        RegionLock rlock2 (&((Playlist&)other));
        
-       jack_nframes_t end = start + cnt - 1;
+       nframes_t end = start + cnt - 1;
 
        init (hide);
 
        for (RegionList::const_iterator i = other.regions.begin(); i != other.regions.end(); i++) {
 
-               Region   *region;
-               Region   *new_region;
-               jack_nframes_t offset = 0;
-               jack_nframes_t position = 0;
-               jack_nframes_t len = 0;
+               boost::shared_ptr<Region> region;
+               boost::shared_ptr<Region> new_region;
+               nframes_t offset = 0;
+               nframes_t position = 0;
+               nframes_t len = 0;
                string    new_name;
                OverlapType overlap;
 
@@ -159,7 +178,7 @@ Playlist::Playlist (const Playlist& other, jack_nframes_t start, jack_nframes_t
 
                _session.region_name (new_name, region->name(), false);
 
-               new_region = createRegion (*region, offset, len, new_name, region->layer(), region->flags());
+               new_region = RegionFactory::RegionFactory::create (region, offset, len, new_name, region->layer(), region->flags());
 
                add_region_internal (new_region, position, true);
        }
@@ -197,15 +216,15 @@ Playlist::copy_regions (RegionList& newlist) const
        RegionLock rlock (const_cast<Playlist *> (this));
 
        for (RegionList::const_iterator i = regions.begin(); i != regions.end(); ++i) {
-               newlist.push_back (createRegion (**i));
+               newlist.push_back (RegionFactory::RegionFactory::create (*i));
        }
 }
 
 void
 Playlist::init (bool hide)
 {
-       atomic_set (&block_notifications, 0);
-       atomic_set (&ignore_state_changes, 0);
+       g_atomic_int_set (&block_notifications, 0);
+       g_atomic_int_set (&ignore_state_changes, 0);
        pending_modified = false;
        pending_length = false;
        _refcnt = 0;
@@ -213,7 +232,7 @@ Playlist::init (bool hide)
        _splicing = false;
        _nudging = false;
        in_set_state = false;
-       _edit_mode = _session.get_edit_mode();
+       _edit_mode = Config->get_edit_mode();
        in_flush = false;
        in_partition = false;
        subcnt = 0;
@@ -223,8 +242,6 @@ Playlist::init (bool hide)
        layer_op_counter = 0;
        freeze_length = 0;
 
-       // _session.LayerModelChanged.connect (slot (*this, &Playlist::relayer));
-
        Modified.connect (mem_fun (*this, &Playlist::mark_session_dirty));
 }
 
@@ -242,6 +259,7 @@ Playlist::Playlist (Playlist& pl)
 
 Playlist::~Playlist ()
 {
+       /* GoingAway must be emitted by derived classes */
 }
 
 void
@@ -273,13 +291,13 @@ void
 Playlist::freeze ()
 {
        delay_notifications ();
-       atomic_inc (&ignore_state_changes);
+       g_atomic_int_inc (&ignore_state_changes);
 }
 
 void
 Playlist::thaw ()
 {
-       atomic_dec (&ignore_state_changes);
+       g_atomic_int_dec_and_test (&ignore_state_changes);
        release_notifications ();
 }
 
@@ -287,14 +305,14 @@ Playlist::thaw ()
 void
 Playlist::delay_notifications ()
 {
-       atomic_inc (&block_notifications);
+       g_atomic_int_inc (&block_notifications);
        freeze_length = _get_maximum_extent();
 }
 
 void
 Playlist::release_notifications ()
 {
-       if (atomic_dec_and_test(&block_notifications)) { 
+       if (g_atomic_int_dec_and_test (&block_notifications)) { 
                flush_notifications ();
        } 
 }
@@ -312,7 +330,7 @@ Playlist::notify_modified ()
 }
 
 void
-Playlist::notify_region_removed (Region *r)
+Playlist::notify_region_removed (boost::shared_ptr<Region> r)
 {
        if (holding_state ()) {
                pending_removals.insert (pending_removals.end(), r);
@@ -327,7 +345,7 @@ Playlist::notify_region_removed (Region *r)
 }
 
 void
-Playlist::notify_region_added (Region *r)
+Playlist::notify_region_added (boost::shared_ptr<Region> r)
 {
        if (holding_state()) {
                pending_adds.insert (pending_adds.end(), r);
@@ -357,7 +375,7 @@ Playlist::flush_notifications ()
 {
        RegionList::iterator r;
        RegionList::iterator a;
-       set<Region*> dependent_checks_needed;
+       set<boost::shared_ptr<Region> > dependent_checks_needed;
        uint32_t n = 0;
 
        if (in_flush) {
@@ -376,8 +394,8 @@ Playlist::flush_notifications ()
        // pending_bounds.sort (cmp);
 
        for (RegionList::iterator r = pending_bounds.begin(); r != pending_bounds.end(); ++r) {
-               if (_session.get_layer_model() == Session::MoveAddHigher) {
-                       timestamp_layer_op (**r);
+               if (Config->get_layer_model() == MoveAddHigher) {
+                       timestamp_layer_op (*r);
                }
                pending_length = true;
                n++;
@@ -394,12 +412,12 @@ Playlist::flush_notifications ()
                n++;
        }
 
-       for (set<Region*>::iterator x = dependent_checks_needed.begin(); x != dependent_checks_needed.end(); ++x) {
-               check_dependents (**x, false);
+       for (set<boost::shared_ptr<Region> >::iterator x = dependent_checks_needed.begin(); x != dependent_checks_needed.end(); ++x) {
+               check_dependents (*x, false);
        }
 
        for (r = pending_removals.begin(); r != pending_removals.end(); ++r) {
-               remove_dependents (**r);
+               remove_dependents (*r);
                RegionRemoved (*r); /* EMIT SIGNAL */
                n++;
        }
@@ -411,8 +429,10 @@ Playlist::flush_notifications ()
        }
 
        if (n || pending_modified) {
-               possibly_splice ();
-               relayer ();
+               if (!in_set_state) {
+                       possibly_splice ();
+                       relayer ();
+               }
                pending_modified = false;
                Modified (); /* EMIT SIGNAL */
        }
@@ -434,7 +454,7 @@ Playlist::flush_notifications ()
  *************************************************************/
 
 void
-Playlist::add_region (const Region& region, jack_nframes_t position, float times, bool with_save) 
+Playlist::add_region (boost::shared_ptr<Region> region, nframes_t position, float times, bool with_save) 
 { 
        RegionLock rlock (this);
        
@@ -442,11 +462,11 @@ Playlist::add_region (const Region& region, jack_nframes_t position, float times
        
        int itimes = (int) floor (times);
 
-       jack_nframes_t pos = position;
+       nframes_t pos = position;
        
        if (itimes >= 1) {
-               add_region_internal (const_cast<Region*>(&region), pos, true);
-               pos += region.length();
+               add_region_internal (region, pos, true);
+               pos += region->length();
                --itimes;
        }
        
@@ -461,16 +481,16 @@ Playlist::add_region (const Region& region, jack_nframes_t position, float times
        */
 
        for (int i = 0; i < itimes; ++i) {
-               Region *copy = createRegion (region);
+               boost::shared_ptr<Region> copy = RegionFactory::create (region);
                add_region_internal (copy, pos, true);
-               pos += region.length();
+               pos += region->length();
        }
        
        if (floor (times) != times) {
-               jack_nframes_t length = (jack_nframes_t) floor (region.length() * (times - floor (times)));
+               nframes_t length = (nframes_t) floor (region->length() * (times - floor (times)));
                string name;
-               _session.region_name (name, region.name(), false);
-               Region *sub = createRegion (region, 0, length, name, region.layer(), region.flags());
+               _session.region_name (name, region->name(), false);
+               boost::shared_ptr<Region> sub = RegionFactory::create (region, 0, length, name, region->layer(), region->flags());
                add_region_internal (sub, pos, true);
        }
        
@@ -480,12 +500,10 @@ Playlist::add_region (const Region& region, jack_nframes_t position, float times
 }
 
 void
-Playlist::add_region_internal (Region *region, jack_nframes_t position, bool delay_sort)
+Playlist::add_region_internal (boost::shared_ptr<Region> region, nframes_t position, bool delay_sort)
 {
        RegionSortByPosition cmp;
-       jack_nframes_t old_length = 0;
-
-       // cerr << "adding region " << region->name() << " at " << position << endl;
+       nframes_t old_length = 0;
 
        if (!holding_state()) {
                 old_length = _get_maximum_extent();
@@ -493,9 +511,8 @@ Playlist::add_region_internal (Region *region, jack_nframes_t position, bool del
 
        region->set_playlist (this);
        region->set_position (position, this);
-       region->lock_sources ();
 
-       timestamp_layer_op (*region);
+       timestamp_layer_op (region);
 
        regions.insert (upper_bound (regions.begin(), regions.end(), region, cmp), region);
 
@@ -509,7 +526,7 @@ Playlist::add_region_internal (Region *region, jack_nframes_t position, bool del
        notify_region_added (region);
        
        if (!holding_state ()) {
-               check_dependents (*region, false);
+               check_dependents (region, false);
                if (old_length != _get_maximum_extent()) {
                        notify_length_changed ();
                }
@@ -519,12 +536,12 @@ Playlist::add_region_internal (Region *region, jack_nframes_t position, bool del
 }
 
 void
-Playlist::replace_region (Region& old, Region& newr, jack_nframes_t pos)
+Playlist::replace_region (boost::shared_ptr<Region> old, boost::shared_ptr<Region> newr, nframes_t pos)
 {
        RegionLock rlock (this);
 
-       remove_region_internal (&old);
-       add_region_internal (&newr, pos);
+       remove_region_internal (old);
+       add_region_internal (newr, pos);
 
        if (!holding_state ()) {
                possibly_splice_unlocked ();
@@ -534,7 +551,7 @@ Playlist::replace_region (Region& old, Region& newr, jack_nframes_t pos)
 }
 
 void
-Playlist::remove_region (Region *region)
+Playlist::remove_region (boost::shared_ptr<Region> region)
 {
        RegionLock rlock (this);
        remove_region_internal (region);
@@ -547,10 +564,10 @@ Playlist::remove_region (Region *region)
 }
 
 int
-Playlist::remove_region_internal (Region *region, bool delay_sort)
+Playlist::remove_region_internal (boost::shared_ptr<Region>region, bool delay_sort)
 {
        RegionList::iterator i;
-       jack_nframes_t old_length = 0;
+       nframes_t old_length = 0;
 
        // cerr << "removing region " << region->name() << endl;
 
@@ -565,7 +582,7 @@ Playlist::remove_region_internal (Region *region, bool delay_sort)
 
                        if (!holding_state ()) {
                                relayer ();
-                               remove_dependents (*region);
+                               remove_dependents (region);
                                
                                if (old_length != _get_maximum_extent()) {
                                        notify_length_changed ();
@@ -580,7 +597,32 @@ Playlist::remove_region_internal (Region *region, bool delay_sort)
 }
 
 void
-Playlist::partition (jack_nframes_t start, jack_nframes_t end, bool just_top_level)
+Playlist::get_equivalent_regions (boost::shared_ptr<Region> other, vector<boost::shared_ptr<Region> >& results)
+{
+       for (RegionList::iterator i = regions.begin(); i != regions.end(); ++i) {
+               if (Config->get_use_overlap_equivalency()) {
+                       if ((*i)->overlap_equivalent (other)) {
+                               results.push_back ((*i));
+                       } else if ((*i)->equivalent (other)) {
+                               results.push_back ((*i));
+                       }
+               }
+       }
+}
+
+void
+Playlist::get_region_list_equivalent_regions (boost::shared_ptr<Region> other, vector<boost::shared_ptr<Region> >& results)
+{
+       for (RegionList::iterator i = regions.begin(); i != regions.end(); ++i) {
+
+               if ((*i) && (*i)->region_list_equivalent (other)) {
+                       results.push_back (*i);
+               }
+       }
+}
+
+void
+Playlist::partition (nframes_t start, nframes_t end, bool just_top_level)
 {
        RegionList thawlist;
 
@@ -594,15 +636,15 @@ Playlist::partition (jack_nframes_t start, jack_nframes_t end, bool just_top_lev
 }
 
 void
-Playlist::partition_internal (jack_nframes_t start, jack_nframes_t end, bool cutting, RegionList& thawlist)
+Playlist::partition_internal (nframes_t start, nframes_t end, bool cutting, RegionList& thawlist)
 {
        RegionLock rlock (this);
-       Region *region;
-       Region *current;
+       boost::shared_ptr<Region> region;
+       boost::shared_ptr<Region> current;
        string new_name;
        RegionList::iterator tmp;
        OverlapType overlap;
-       jack_nframes_t pos1, pos2, pos3, pos4;
+       nframes_t pos1, pos2, pos3, pos4;
        RegionList new_regions;
 
        in_partition = true;
@@ -658,7 +700,7 @@ Playlist::partition_internal (jack_nframes_t start, jack_nframes_t end, bool cut
                                /* "middle" ++++++ */
                                
                                _session.region_name (new_name, current->name(), false);
-                               region = createRegion (*current, pos2 - pos1, pos3 - pos2, new_name,
+                               region = RegionFactory::create (current, pos2 - pos1, pos3 - pos2, new_name,
                                                       regions.size(), Region::Flag(current->flags()|Region::Automatic|Region::LeftOfSplit|Region::RightOfSplit));
                                add_region_internal (region, start, true);
                                new_regions.push_back (region);
@@ -667,7 +709,7 @@ Playlist::partition_internal (jack_nframes_t start, jack_nframes_t end, bool cut
                        /* "end" ====== */
                        
                        _session.region_name (new_name, current->name(), false);
-                       region = createRegion (*current, pos3 - pos1, pos4 - pos3, new_name, 
+                       region = RegionFactory::create (current, pos3 - pos1, pos4 - pos3, new_name, 
                                               regions.size(), Region::Flag(current->flags()|Region::Automatic|Region::RightOfSplit));
 
                        add_region_internal (region, end, true);
@@ -697,7 +739,7 @@ Playlist::partition_internal (jack_nframes_t start, jack_nframes_t end, bool cut
                                /* end +++++ */
                                
                                _session.region_name (new_name, current->name(), false);
-                               region = createRegion (*current, pos2 - pos1, pos4 - pos2, new_name, (layer_t) regions.size(),
+                               region = RegionFactory::create (current, pos2 - pos1, pos4 - pos2, new_name, (layer_t) regions.size(),
                                                       Region::Flag(current->flags()|Region::Automatic|Region::LeftOfSplit));
                                add_region_internal (region, start, true);
                                new_regions.push_back (region);
@@ -731,7 +773,7 @@ Playlist::partition_internal (jack_nframes_t start, jack_nframes_t end, bool cut
                                
                                /* front **** */
                                 _session.region_name (new_name, current->name(), false);
-                                region = createRegion (*current, 0, pos3 - pos1, new_name,
+                                region = RegionFactory::create (current, 0, pos3 - pos1, new_name,
                                                        regions.size(), Region::Flag(current->flags()|Region::Automatic|Region::RightOfSplit));
                                 add_region_internal (region, pos1, true);
                                 new_regions.push_back (region);
@@ -771,16 +813,16 @@ Playlist::partition_internal (jack_nframes_t start, jack_nframes_t end, bool cut
        in_partition = false;
 
        for (RegionList::iterator i = new_regions.begin(); i != new_regions.end(); ++i) {
-               check_dependents (**i, false);
+               check_dependents (*i, false);
        }
 }
 
 Playlist*
-Playlist::cut_copy (Playlist* (Playlist::*pmf)(jack_nframes_t, jack_nframes_t,bool), list<AudioRange>& ranges, bool result_is_hidden)
+Playlist::cut_copy (Playlist* (Playlist::*pmf)(nframes_t, nframes_t,bool), list<AudioRange>& ranges, bool result_is_hidden)
 {
        Playlist* ret;
        Playlist* pl;
-       jack_nframes_t start;
+       nframes_t start;
 
        if (ranges.empty()) {
                return 0;
@@ -820,19 +862,19 @@ Playlist::cut_copy (Playlist* (Playlist::*pmf)(jack_nframes_t, jack_nframes_t,bo
 Playlist*
 Playlist::cut (list<AudioRange>& ranges, bool result_is_hidden)
 {
-       Playlist* (Playlist::*pmf)(jack_nframes_t,jack_nframes_t,bool) = &Playlist::cut;
+       Playlist* (Playlist::*pmf)(nframes_t,nframes_t,bool) = &Playlist::cut;
        return cut_copy (pmf, ranges, result_is_hidden);
 }
 
 Playlist*
 Playlist::copy (list<AudioRange>& ranges, bool result_is_hidden)
 {
-       Playlist* (Playlist::*pmf)(jack_nframes_t,jack_nframes_t,bool) = &Playlist::copy;
+       Playlist* (Playlist::*pmf)(nframes_t,nframes_t,bool) = &Playlist::copy;
        return cut_copy (pmf, ranges, result_is_hidden);
 }
 
 Playlist *
-Playlist::cut (jack_nframes_t start, jack_nframes_t cnt, bool result_is_hidden)
+Playlist::cut (nframes_t start, nframes_t cnt, bool result_is_hidden)
 {
        Playlist *the_copy;
        RegionList thawlist;
@@ -860,7 +902,7 @@ Playlist::cut (jack_nframes_t start, jack_nframes_t cnt, bool result_is_hidden)
 }
 
 Playlist *
-Playlist::copy (jack_nframes_t start, jack_nframes_t cnt, bool result_is_hidden)
+Playlist::copy (nframes_t start, nframes_t cnt, bool result_is_hidden)
 {
        char buf[32];
        
@@ -874,10 +916,10 @@ Playlist::copy (jack_nframes_t start, jack_nframes_t cnt, bool result_is_hidden)
 }
 
 int
-Playlist::paste (Playlist& other, jack_nframes_t position, float times)
+Playlist::paste (Playlist& other, nframes_t position, float times)
 {
        times = fabs (times);
-       jack_nframes_t old_length;
+       nframes_t old_length;
 
        {
                RegionLock rl1 (this);
@@ -886,13 +928,13 @@ Playlist::paste (Playlist& other, jack_nframes_t position, float times)
                old_length = _get_maximum_extent();
        
                int itimes = (int) floor (times);
-               jack_nframes_t pos = position;
-               jack_nframes_t shift = other._get_maximum_extent();
+               nframes_t pos = position;
+               nframes_t shift = other._get_maximum_extent();
                layer_t top_layer = regions.size();
 
                while (itimes--) {
                        for (RegionList::iterator i = other.regions.begin(); i != other.regions.end(); ++i) {
-                               Region *copy_of_region = createRegion (**i);
+                               boost::shared_ptr<Region> copy_of_region = RegionFactory::create (*i);
 
                                /* put these new regions on top of all existing ones, but preserve
                                   the ordering they had in the original playlist.
@@ -922,25 +964,25 @@ Playlist::paste (Playlist& other, jack_nframes_t position, float times)
 
 
 void
-Playlist::duplicate (Region& region, jack_nframes_t position, float times)
+Playlist::duplicate (boost::shared_ptr<Region> region, nframes_t position, float times)
 {
        times = fabs (times);
 
        RegionLock rl (this);
        int itimes = (int) floor (times);
-       jack_nframes_t pos = position;
+       nframes_t pos = position;
 
        while (itimes--) {
-               Region *copy = createRegion (region);
+               boost::shared_ptr<Region> copy = RegionFactory::create (region);
                add_region_internal (copy, pos, true);
-               pos += region.length();
+               pos += region->length();
        }
 
        if (floor (times) != times) {
-               jack_nframes_t length = (jack_nframes_t) floor (region.length() * (times - floor (times)));
+               nframes_t length = (nframes_t) floor (region->length() * (times - floor (times)));
                string name;
-               _session.region_name (name, region.name(), false);
-               Region *sub = createRegion (region, 0, length, name, region.layer(), region.flags());
+               _session.region_name (name, region->name(), false);
+               boost::shared_ptr<Region> sub = RegionFactory::create (region, 0, length, name, region->layer(), region->flags());
                add_region_internal (sub, pos, true);
        }
 
@@ -948,41 +990,56 @@ Playlist::duplicate (Region& region, jack_nframes_t position, float times)
 }
 
 void
-Playlist::split_region (Region& region, jack_nframes_t playlist_position)
+Playlist::split_region (boost::shared_ptr<Region> region, nframes_t playlist_position)
 {
        RegionLock rl (this);
 
-       if (!region.covers (playlist_position)) {
-               return;
-       }
-
-       if (region.position() == playlist_position ||
-           region.last_frame() == playlist_position) {
+       if (!region->covers (playlist_position)) {
                return;
        }
 
-       if (remove_region_internal (&region, true)) {
+       if (region->position() == playlist_position ||
+           region->last_frame() == playlist_position) {
                return;
        }
 
-       Region *left;
-       Region *right;
-       jack_nframes_t before;
-       jack_nframes_t after;
+       boost::shared_ptr<Region> left;
+       boost::shared_ptr<Region> right;
+       nframes_t before;
+       nframes_t after;
        string before_name;
        string after_name;
 
-       before = playlist_position - region.position();
-       after = region.length() - before;
+       before = playlist_position - region->position();
+       after = region->length() - before;
        
-       _session.region_name (before_name, region.name(), false);
-       left = createRegion (region, 0, before, before_name, region.layer(), Region::Flag (region.flags()|Region::LeftOfSplit));
+       
+       _session.region_name (before_name, region->name(), false);
+       left = RegionFactory::create (region, 0, before, before_name, region->layer(), Region::Flag (region->flags()|Region::LeftOfSplit));
+
+       _session.region_name (after_name, region->name(), false);
+       right = RegionFactory::create (region, before, after, after_name, region->layer(), Region::Flag (region->flags()|Region::RightOfSplit));
 
-       _session.region_name (after_name, region.name(), false);
-       right = createRegion (region, before, after, after_name, region.layer(), Region::Flag (region.flags()|Region::RightOfSplit));
+       add_region_internal (left, region->position(), true);
+       add_region_internal (right, region->position() + before);
+       
+       uint64_t orig_layer_op = region->last_layer_op();
+       for (RegionList::iterator i = regions.begin(); i != regions.end(); ++i) {
+               if ((*i)->last_layer_op() > orig_layer_op) {
+                       (*i)->set_last_layer_op( (*i)->last_layer_op() + 1 );
+               }
+       }
        
-       add_region_internal (left, region.position(), true);
-       add_region_internal (right, region.position() + before);
+       left->set_last_layer_op ( orig_layer_op );
+       right->set_last_layer_op ( orig_layer_op + 1);
+
+       layer_op_counter++;
+
+       finalize_split_region (region, left, right);
+       
+       if (remove_region_internal (region, true)) {
+               return;
+       }
 
        maybe_save_state (_("split"));
 }
@@ -1044,7 +1101,7 @@ Playlist::core_splice ()
 }
 
 void
-Playlist::region_bounds_changed (Change what_changed, Region *region)
+Playlist::region_bounds_changed (Change what_changed, boost::shared_ptr<Region> region)
 {
        if (in_set_state || _splicing || _nudging) {
                return;
@@ -1068,8 +1125,7 @@ Playlist::region_bounds_changed (Change what_changed, Region *region)
                }
 
                regions.erase (i);
-               regions.insert (upper_bound (regions.begin(), regions.end(), region, cmp),
-                               region);
+               regions.insert (upper_bound (regions.begin(), regions.end(), region, cmp), region);
 
        }
 
@@ -1078,13 +1134,13 @@ Playlist::region_bounds_changed (Change what_changed, Region *region)
                if (holding_state ()) {
                        pending_bounds.push_back (region);
                } else {
-                       if (_session.get_layer_model() == Session::MoveAddHigher) {
+                       if (Config->get_layer_model() == MoveAddHigher) {
                                /* it moved or changed length, so change the timestamp */
-                               timestamp_layer_op (*region);
+                               timestamp_layer_op (region);
                        }
                        
                        possibly_splice ();
-                       check_dependents (*region, false);
+                       check_dependents (region, false);
                        notify_length_changed ();
                        relayer ();
                }
@@ -1092,7 +1148,7 @@ Playlist::region_bounds_changed (Change what_changed, Region *region)
 }
 
 void
-Playlist::region_changed_proxy (Change what_changed, Region* region)
+Playlist::region_changed_proxy (Change what_changed, boost::shared_ptr<Region> region)
 {
        /* this makes a virtual call to the right kind of playlist ... */
 
@@ -1100,7 +1156,7 @@ Playlist::region_changed_proxy (Change what_changed, Region* region)
 }
 
 bool
-Playlist::region_changed (Change what_changed, Region* region)
+Playlist::region_changed (Change what_changed, boost::shared_ptr<Region> region)
 {
        Change our_interests = Change (Region::MuteChanged|Region::LayerChanged|Region::OpacityChanged);
        bool save = false;
@@ -1117,7 +1173,7 @@ Playlist::region_changed (Change what_changed, Region* region)
                
                if ((what_changed & Region::MuteChanged) && 
                    !(what_changed &  Change (ARDOUR::PositionChanged|ARDOUR::LengthChanged))) {
-                       check_dependents (*region, false);
+                       check_dependents (region, false);
                }
                
                if (what_changed & our_interests) {
@@ -1129,7 +1185,7 @@ Playlist::region_changed (Change what_changed, Region* region)
 }
 
 void
-Playlist::clear (bool with_delete, bool with_save)
+Playlist::clear (bool with_save)
 {
        RegionList::iterator i;
        RegionList tmp;
@@ -1142,9 +1198,6 @@ Playlist::clear (bool with_delete, bool with_save)
        
        for (i = tmp.begin(); i != tmp.end(); ++i) {
                notify_region_removed (*i);
-               if (with_delete) {
-                       delete *i;
-               }
        }
 
        if (with_save) {
@@ -1157,21 +1210,21 @@ Playlist::clear (bool with_delete, bool with_save)
  **********************************************************************/
 
 Playlist::RegionList *
-Playlist::regions_at (jack_nframes_t frame)
+Playlist::regions_at (nframes_t frame)
 
 {
        RegionLock rlock (this);
        return find_regions_at (frame);
 }      
 
-Region *
-Playlist::top_region_at (jack_nframes_t frame)
+boost::shared_ptr<Region>
+Playlist::top_region_at (nframes_t frame)
 
 {
        RegionLock rlock (this);
        RegionList *rlist = find_regions_at (frame);
-       Region *region = 0;
-
+       boost::shared_ptr<Region> region;
+       
        if (rlist->size()) {
                RegionSortByLayer cmp;
                rlist->sort (cmp);
@@ -1183,7 +1236,7 @@ Playlist::top_region_at (jack_nframes_t frame)
 }      
 
 Playlist::RegionList *
-Playlist::find_regions_at (jack_nframes_t frame)
+Playlist::find_regions_at (nframes_t frame)
 {
        RegionList *rlist = new RegionList;
 
@@ -1197,7 +1250,7 @@ Playlist::find_regions_at (jack_nframes_t frame)
 }
 
 Playlist::RegionList *
-Playlist::regions_touched (jack_nframes_t start, jack_nframes_t end)
+Playlist::regions_touched (nframes_t start, nframes_t end)
 {
        RegionLock rlock (this);
        RegionList *rlist = new RegionList;
@@ -1212,19 +1265,18 @@ Playlist::regions_touched (jack_nframes_t start, jack_nframes_t end)
 }
 
 
-Region*
-
-Playlist::find_next_region (jack_nframes_t frame, RegionPoint point, int dir)
+boost::shared_ptr<Region>
+Playlist::find_next_region (nframes_t frame, RegionPoint point, int dir)
 {
        RegionLock rlock (this);
-       Region* ret = 0;
-       jack_nframes_t closest = max_frames;
+       boost::shared_ptr<Region> ret;
+       nframes_t closest = max_frames;
 
        for (RegionList::iterator i = regions.begin(); i != regions.end(); ++i) {
 
-               jack_nframes_t distance;
-               Region* r = (*i);
-               jack_nframes_t pos = 0;
+               nframes_t distance;
+               boost::shared_ptr<Region> r = (*i);
+               nframes_t pos = 0;
 
                switch (point) {
                case Start:
@@ -1288,10 +1340,10 @@ Playlist::set_state (const XMLNode& node)
        XMLPropertyList plist;
        XMLPropertyConstIterator piter;
        XMLProperty *prop;
-       Region *region;
+       boost::shared_ptr<Region> region;
        string region_name;
 
-       clear (false, false);
+       clear (false);
 
        if (node.name() != "Playlist") {
                in_set_state = false;
@@ -1307,7 +1359,7 @@ Playlist::set_state (const XMLNode& node)
                if (prop->name() == X_("name")) {
                        _name = prop->value();
                } else if (prop->name() == X_("orig_diskstream_id")) {
-                       sscanf (prop->value().c_str(), "%" PRIu64, &_orig_diskstream_id);
+                       _orig_diskstream_id = prop->value ();
                } else if (prop->name() == X_("frozen")) {
                        _frozen = (prop->value() == X_("yes"));
                }
@@ -1321,24 +1373,28 @@ Playlist::set_state (const XMLNode& node)
                
                if (child->name() == "Region") {
 
-                       if ((region = createRegion (_session, *child, true)) == 0) {
+                       if ((region = RegionFactory::create (_session, *child, true)) == 0) {
                                error << _("Playlist: cannot create region from state file") << endmsg;
                                continue;
                        }
 
-                       add_region (*region, region->position(), 1.0, false);
+                       add_region (region, region->position(), 1.0, false);
+
+                       // 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 
        */
 
        for (RegionList::iterator r = regions.begin(); r != regions.end(); ++r) {
-               check_dependents (**r, false);
+               check_dependents (*r, false);
        }
-
+       
        in_set_state = false;
 
        return 0;
@@ -1364,7 +1420,7 @@ Playlist::state (bool full_state)
        
        node->add_property (X_("name"), _name);
 
-       snprintf (buf, sizeof(buf), "%" PRIu64, _orig_diskstream_id);
+       _orig_diskstream_id.print (buf, sizeof (buf));
        node->add_property (X_("orig_diskstream_id"), buf);
        node->add_property (X_("frozen"), _frozen ? "yes" : "no");
 
@@ -1389,19 +1445,19 @@ Playlist::empty() const
        return regions.empty();
 }
 
-jack_nframes_t
+nframes_t
 Playlist::get_maximum_extent () const
 {
        RegionLock rlock (const_cast<Playlist *>(this));
        return _get_maximum_extent ();
 }
 
-jack_nframes_t
+nframes_t
 Playlist::_get_maximum_extent () const
 {
        RegionList::const_iterator i;
-       jack_nframes_t max_extent = 0;
-       jack_nframes_t end = 0;
+       nframes_t max_extent = 0;
+       nframes_t end = 0;
 
        for (i = regions.begin(); i != regions.end(); ++i) {
                if ((end = (*i)->position() + (*i)->length()) > max_extent) {
@@ -1481,8 +1537,8 @@ Playlist::relayer ()
 
        freeze ();
 
-       if (_session.get_layer_model() == Session::MoveAddHigher || 
-           _session.get_layer_model() == Session::AddHigher) {
+       if (Config->get_layer_model() == MoveAddHigher || 
+           Config->get_layer_model() == AddHigher) {
 
                RegionSortByLastLayerOp cmp;
                RegionList copy = regions;
@@ -1518,10 +1574,10 @@ Playlist::relayer ()
 /* XXX these layer functions are all deprecated */
 
 void
-Playlist::raise_region (Region& region)
+Playlist::raise_region (boost::shared_ptr<Region> region)
 {
        uint32_t rsz = regions.size();
-       layer_t target = region.layer() + 1U;
+       layer_t target = region->layer() + 1U;
 
        if (target >= rsz) {
                /* its already at the effective top */
@@ -1532,45 +1588,45 @@ Playlist::raise_region (Region& region)
 }
 
 void
-Playlist::lower_region (Region& region)
+Playlist::lower_region (boost::shared_ptr<Region> region)
 {
-       if (region.layer() == 0) {
+       if (region->layer() == 0) {
                /* its already at the bottom */
                return;
        }
 
-       layer_t target = region.layer() - 1U;
+       layer_t target = region->layer() - 1U;
 
        move_region_to_layer (target, region, -1);
 }
 
 void
-Playlist::raise_region_to_top (Region& region)
+Playlist::raise_region_to_top (boost::shared_ptr<Region> region)
 {
        /* does nothing useful if layering mode is later=higher */
-       if ((_session.get_layer_model() == Session::MoveAddHigher) ||
-           (_session.get_layer_model() == Session::AddHigher)) {
+       if ((Config->get_layer_model() == MoveAddHigher) ||
+           (Config->get_layer_model() == AddHigher)) {
                timestamp_layer_op (region);
                relayer ();
        }
 }
 
 void
-Playlist::lower_region_to_bottom (Region& region)
+Playlist::lower_region_to_bottom (boost::shared_ptr<Region> region)
 {
        /* does nothing useful if layering mode is later=higher */
-       if ((_session.get_layer_model() == Session::MoveAddHigher) ||
-           (_session.get_layer_model() == Session::AddHigher)) {
-               region.set_last_layer_op (0);
+       if ((Config->get_layer_model() == MoveAddHigher) ||
+           (Config->get_layer_model() == AddHigher)) {
+               region->set_last_layer_op (0);
                relayer ();
        }
 }
 
 int
-Playlist::move_region_to_layer (layer_t target_layer, Region& region, int dir)
+Playlist::move_region_to_layer (layer_t target_layer, boost::shared_ptr<Region> region, int dir)
 {
        RegionList::iterator i;
-       typedef pair<Region*,layer_t> LayerInfo;
+       typedef pair<boost::shared_ptr<Region>,layer_t> LayerInfo;
        list<LayerInfo> layerinfo;
        layer_t dest;
 
@@ -1579,7 +1635,7 @@ Playlist::move_region_to_layer (layer_t target_layer, Region& region, int dir)
                
                for (i = regions.begin(); i != regions.end(); ++i) {
                        
-                       if (&region == *i) {
+                       if (region == *i) {
                                continue;
                        }
 
@@ -1589,7 +1645,7 @@ Playlist::move_region_to_layer (layer_t target_layer, Region& region, int dir)
                                   down 1
                                */
                                
-                               if ((*i)->layer() > region.layer() && (*i)->layer() <= target_layer) {
+                               if ((*i)->layer() > region->layer() && (*i)->layer() <= target_layer) {
                                        dest = (*i)->layer() - 1;
                                } else {
                                        /* not affected */
@@ -1601,7 +1657,7 @@ Playlist::move_region_to_layer (layer_t target_layer, Region& region, int dir)
                                   up 1
                                */
 
-                               if ((*i)->layer() < region.layer() && (*i)->layer() >= target_layer) {
+                               if ((*i)->layer() < region->layer() && (*i)->layer() >= target_layer) {
                                        dest = (*i)->layer() + 1;
                                } else {
                                        /* not affected */
@@ -1624,12 +1680,12 @@ Playlist::move_region_to_layer (layer_t target_layer, Region& region, int dir)
                x->first->set_layer (x->second);
        }
 
-       region.set_layer (target_layer);
+       region->set_layer (target_layer);
 
        /* now check all dependents */
 
        for (list<LayerInfo>::iterator x = layerinfo.begin(); x != layerinfo.end(); ++x) {
-               check_dependents (*(x->first), false);
+               check_dependents (x->first, false);
        }
        
        check_dependents (region, false);
@@ -1638,10 +1694,10 @@ Playlist::move_region_to_layer (layer_t target_layer, Region& region, int dir)
 }
 
 void
-Playlist::nudge_after (jack_nframes_t start, jack_nframes_t distance, bool forwards)
+Playlist::nudge_after (nframes_t start, nframes_t distance, bool forwards)
 {
        RegionList::iterator i;
-       jack_nframes_t new_pos;
+       nframes_t new_pos;
        bool moved = false;
 
        _nudging = true;
@@ -1684,19 +1740,20 @@ Playlist::nudge_after (jack_nframes_t start, jack_nframes_t distance, bool forwa
 
 }
 
-Region*
-Playlist::find_region (id_t id) const
+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) {
                if ((*i)->id() == id) {
-                       return (*i);
+                       ret = *i;
                }
        }
 
-       return 0;
+       return ret;
 }
        
 void
@@ -1710,7 +1767,7 @@ Playlist::save_state (std::string why)
 void
 Playlist::dump () const
 {
-       Region *r;
+       boost::shared_ptr<Region> r;
 
        cerr << "Playlist \"" << _name << "\" " << endl
             << regions.size() << " regions "
@@ -1735,11 +1792,11 @@ Playlist::set_frozen (bool yn)
 }
 
 void
-Playlist::timestamp_layer_op (Region& region)
+Playlist::timestamp_layer_op (boost::shared_ptr<Region> region)
 {
 //     struct timeval tv;
 //     gettimeofday (&tv, 0);
-       region.set_last_layer_op (++layer_op_counter);
+       region->set_last_layer_op (++layer_op_counter);
 }
 
 void