more MTC debugging
[ardour.git] / libs / ardour / region.cc
index 20c87fb1defc3c1f43827e44279389bfedf2f2bd..1f4d6f0f912221d78d3fc4739ea8840a82d13af4 100644 (file)
 #include <algorithm>
 #include <sstream>
 
-#include <sigc++/bind.h>
-#include <sigc++/class_slot.h>
 
 #include <glibmm/thread.h>
 #include "pbd/xml++.h"
 #include "pbd/stacktrace.h"
 #include "pbd/enumwriter.h"
 
+#include "ardour/debug.h"
 #include "ardour/region.h"
 #include "ardour/playlist.h"
 #include "ardour/session.h"
@@ -55,7 +54,7 @@ Change Region::LockChanged       = ARDOUR::new_change ();
 Change Region::LayerChanged      = ARDOUR::new_change ();
 Change Region::HiddenChanged     = ARDOUR::new_change ();
 
-sigc::signal<void,boost::shared_ptr<ARDOUR::Region> > Region::RegionPropertyChanged;
+PBD::Signal1<void,boost::shared_ptr<ARDOUR::Region> > Region::RegionPropertyChanged;
 
 /* derived-from-derived constructor (no sources in constructor) */
 Region::Region (Session& s, nframes_t start, nframes_t length, const string& name, DataType type, layer_t layer, Region::Flag flags)
@@ -110,7 +109,7 @@ Region::Region (boost::shared_ptr<Source> src, nframes_t start, nframes_t length
        _sources.push_back (src);
        _master_sources.push_back (src);
 
-       src->GoingAway.connect (bind (mem_fun (*this, &Region::source_deleted), src));
+       src->DropReferences.connect_same_thread (*this, boost::bind (&Region::source_deleted, this, boost::weak_ptr<Source>(src)));
 
        assert(_sources.size() > 0);
        _positional_lock_style = AudioTime;
@@ -338,8 +337,7 @@ Region::Region (boost::shared_ptr<Source> src, const XMLNode& node)
 
 Region::~Region ()
 {
-       notify_callbacks ();
-       GoingAway (); /* EMIT SIGNAL */
+       DEBUG_TRACE (DEBUG::Destruction, string_compose ("Region %1 destructor @ %2\n", _name, this));
 }
 
 void
@@ -1150,6 +1148,7 @@ Region::set_live_state (const XMLNode& node, int /*version*/, Change& what_chang
                sscanf (prop->value().c_str(), "%" PRIu32, &val);
                if (val != _start) {
                        what_changed = Change (what_changed|StartChanged);
+                       cerr << _name << " start changed\n";
                        _start = val;
                }
        } else {
@@ -1160,6 +1159,7 @@ Region::set_live_state (const XMLNode& node, int /*version*/, Change& what_chang
                sscanf (prop->value().c_str(), "%" PRIu32, &val);
                if (val != _length) {
                        what_changed = Change (what_changed|LengthChanged);
+                       cerr << _name << " length changed\n";
                        _last_length = _length;
                        _length = val;
                }
@@ -1172,6 +1172,7 @@ Region::set_live_state (const XMLNode& node, int /*version*/, Change& what_chang
                sscanf (prop->value().c_str(), "%" PRIu32, &val);
                if (val != _position) {
                        what_changed = Change (what_changed|PositionChanged);
+                       cerr << _name << " position changed\n";
                        _last_position = _position;
                        _position = val;
                }
@@ -1185,6 +1186,7 @@ Region::set_live_state (const XMLNode& node, int /*version*/, Change& what_chang
                x = (layer_t) atoi (prop->value().c_str());
                if (x != _layer) {
                        what_changed = Change (what_changed|LayerChanged);
+                       cerr << _name << " layer changed\n";
                        _layer = x;
                }
        } else {
@@ -1195,6 +1197,7 @@ Region::set_live_state (const XMLNode& node, int /*version*/, Change& what_chang
                sscanf (prop->value().c_str(), "%" PRIu32, &val);
                if (val != _sync_position) {
                        what_changed = Change (what_changed|SyncOffsetChanged);
+                       cerr << _name << " sync changed\n";
                        _sync_position = val;
                }
        } else {
@@ -1283,6 +1286,7 @@ Region::set_live_state (const XMLNode& node, int /*version*/, Change& what_chang
        }
 
        if (send) {
+               cerr << _name << ": final change to be sent: " << hex << what_changed << dec << endl;
                send_change (what_changed);
        }
 
@@ -1348,12 +1352,13 @@ Region::thaw (const string& /*why*/)
                recompute_at_end ();
        }
 
-       StateChanged (what_changed);
+       send_change (what_changed);
 }
 
 void
 Region::send_change (Change what_changed)
 {
+
        {
                Glib::Mutex::Lock lm (_lock);
                if (_frozen) {
@@ -1362,7 +1367,9 @@ Region::send_change (Change what_changed)
                }
        }
 
+       cerr << _name << " actually sends " << hex << what_changed << dec << " @" << get_microseconds() << endl;
        StateChanged (what_changed);
+       cerr << _name << " done with " << hex << what_changed << dec << " @" << get_microseconds() << endl;
 
        if (!(_flags & DoNotSendPropertyChanges)) {
 
@@ -1372,7 +1379,9 @@ Region::send_change (Change what_changed)
 
                try {
                        boost::shared_ptr<Region> rptr = shared_from_this();
+                       cerr << _name << " actually sends prop change " << hex << what_changed << dec <<  " @ " << get_microseconds() << endl;
                        RegionPropertyChanged (rptr);
+                       cerr << _name << " done with prop change  @ " << get_microseconds() << endl;
                } catch (...) {
                        /* no shared_ptr available, relax; */
                }
@@ -1414,10 +1423,21 @@ Region::region_list_equivalent (boost::shared_ptr<const Region> other) const
 }
 
 void
-Region::source_deleted (boost::shared_ptr<Source>)
+Region::source_deleted (boost::weak_ptr<Source>)
 {
        _sources.clear ();
-       drop_references ();
+
+       if (!_session.deletion_in_progress()) {
+               /* this is a very special case: at least one of the region's
+                  sources has bee deleted, so invalidate all references to
+                  ourselves. Do NOT do this during session deletion, because
+                  then we run the risk that this will actually result
+                  in this object being deleted (as refcnt goes to zero)
+                  while emitting DropReferences.
+               */
+
+               drop_references ();
+       }
 }
 
 vector<string>
@@ -1586,14 +1606,14 @@ Region::use_sources (SourceList const & s)
 
        for (SourceList::const_iterator i = s.begin (); i != s.end(); ++i) {
                _sources.push_back (*i);
-               (*i)->GoingAway.connect (bind (mem_fun (*this, &Region::source_deleted), *i));
+               (*i)->DropReferences.connect_same_thread (*this, boost::bind (&Region::source_deleted, this, boost::weak_ptr<Source>(*i)));
                unique_srcs.insert (*i);
        }
 
        for (SourceList::const_iterator i = s.begin (); i != s.end(); ++i) {
                _master_sources.push_back (*i);
                if (unique_srcs.find (*i) == unique_srcs.end()) {
-                       (*i)->GoingAway.connect (bind (mem_fun (*this, &Region::source_deleted), *i));
+                       (*i)->DropReferences.connect_same_thread (*this, boost::bind (&Region::source_deleted, this, boost::weak_ptr<Source>(*i)));
                }
        }
 }