Remove LengthChanged that no-one was listening to.

git-svn-id: svn://localhost/ardour2/branches/3.0@11023 d708f5d6-7413-0410-9779-e7cbd77b26cf

diff --git a/libs/ardour/ardour/playlist.h b/libs/ardour/ardour/playlist.h
index cce602437bc3385634ed7a6b37e9ab61bcb4a9de..6d7113b32b10ad17b9b44c63d7f27230b17f68e8 100644 (file)
--- a/libs/ardour/ardour/playlist.h
+++ b/libs/ardour/ardour/playlist.h
@@ -180,7 +180,6 @@ public:
        PBD::Signal1<void,boost::weak_ptr<Region> > RegionAdded;
        PBD::Signal1<void,boost::weak_ptr<Region> > RegionRemoved;
        PBD::Signal0<void>      NameChanged;
-       PBD::Signal0<void>      LengthChanged;
        PBD::Signal0<void>      LayeringChanged;
 
        /** Emitted when regions have moved (not when regions have only been trimmed) */
@@ -265,7 +264,6 @@ public:
        RegionList       pending_bounds;
        bool             pending_contents_change;
        bool             pending_layering;
-       bool             pending_length;
 
        /** Movements of time ranges caused by region moves; note that
         *  region trims are not included in this list; it is used to

diff --git a/libs/ardour/playlist.cc b/libs/ardour/playlist.cc
index e68c13b1733db9ec0de094e8bc935113e431c941..d234ab58c3174b56d395bc0b3dc50ed02326b080 100644 (file)
--- a/libs/ardour/playlist.cc
+++ b/libs/ardour/playlist.cc
@@ -304,7 +304,6 @@ Playlist::init (bool hide)
        g_atomic_int_set (&block_notifications, 0);
        g_atomic_int_set (&ignore_state_changes, 0);
        pending_contents_change = false;
-       pending_length = false;
        pending_layering = false;
        first_set_state = true;
        _refcnt = 0;
@@ -473,13 +472,10 @@ Playlist::notify_region_removed (boost::shared_ptr<Region> r)
        if (holding_state ()) {
                pending_removes.insert (r);
                pending_contents_change = true;
-               pending_length = true;
        } else {
                /* this might not be true, but we have to act
                   as though it could be.
                */
-               pending_length = false;
-               LengthChanged (); /* EMIT SIGNAL */
                pending_contents_change = false;
                RegionRemoved (boost::weak_ptr<Region> (r)); /* EMIT SIGNAL */
                ContentsChanged (); /* EMIT SIGNAL */
@@ -559,11 +555,8 @@ Playlist::notify_region_added (boost::shared_ptr<Region> r)
        if (holding_state()) {
                pending_adds.insert (r);
                pending_contents_change = true;
-               pending_length = true;
        } else {
                r->clear_changes ();
-               pending_length = false;
-               LengthChanged (); /* EMIT SIGNAL */
                pending_contents_change = false;
                RegionAdded (boost::weak_ptr<Region> (r)); /* EMIT SIGNAL */
                ContentsChanged (); /* EMIT SIGNAL */
@@ -573,11 +566,7 @@ Playlist::notify_region_added (boost::shared_ptr<Region> r)
 void
 Playlist::notify_length_changed ()
 {
-       if (holding_state ()) {
-               pending_length = true;
-       } else {
-               pending_length = false;
-               LengthChanged(); /* EMIT SIGNAL */
+       if (!holding_state ()) {
                pending_contents_change = false;
                ContentsChanged (); /* EMIT SIGNAL */
        }
@@ -590,8 +579,6 @@ Playlist::flush_notifications (bool from_undo)
        set<boost::shared_ptr<Region> > dependent_checks_needed;
        set<boost::shared_ptr<Region> >::iterator s;
        uint32_t regions_changed = false;
-       bool check_length = false;
-       framecnt_t old_length = 0;
 
        if (in_flush) {
                return;
@@ -601,10 +588,6 @@ Playlist::flush_notifications (bool from_undo)
 
        if (!pending_bounds.empty() || !pending_removes.empty() || !pending_adds.empty()) {
                regions_changed = true;
-               if (!pending_length) {
-                       old_length = _get_extent ().second;
-                       check_length = true;
-               }
        }
 
        /* we have no idea what order the regions ended up in pending
@@ -638,19 +621,6 @@ Playlist::flush_notifications (bool from_undo)
                 dependent_checks_needed.insert (*s);
         }
 
-        if (check_length) {
-                if (old_length != _get_extent().second) {
-                        pending_length = true;
-                        // cerr << _name << " length has changed\n";
-                }
-        }
-
-        if (pending_length || (freeze_length != _get_extent().second)) {
-                pending_length = false;
-                // cerr << _name << " sends LengthChanged\n";
-                LengthChanged(); /* EMIT SIGNAL */
-        }
-
         if (regions_changed || pending_contents_change) {
                 if (!in_set_state) {
                         relayer ();
@@ -692,7 +662,6 @@ Playlist::flush_notifications (bool from_undo)
         pending_range_moves.clear ();
         pending_region_extensions.clear ();
         pending_contents_change = false;
-        pending_length = false;
  }
 
  /*************************************************************
@@ -1701,8 +1670,6 @@ Playlist::flush_notifications (bool from_undo)
                 }
 
                 pending_removes.clear ();
-                pending_length = false;
-                LengthChanged ();
                 pending_contents_change = false;
                 ContentsChanged ();
         }