From f6c011cb651f5765f67f4fed473c3ee098767d5c Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Fri, 7 May 2021 22:47:57 +0200
Subject: [PATCH] Remove old RegionPropertyChanged signal (3/3)

---
 libs/ardour/ardour/region.h | 2 --
 libs/ardour/playlist.cc     | 1 -
 libs/ardour/region.cc       | 3 ---
 libs/ardour/track.cc        | 9 ---------
 4 files changed, 15 deletions(-)

diff --git a/libs/ardour/ardour/region.h b/libs/ardour/ardour/region.h
index b66b3b8e7d..89ae1883a7 100644
--- a/libs/ardour/ardour/region.h
+++ b/libs/ardour/ardour/region.h
@@ -99,8 +99,6 @@ public:
 
 	static void make_property_quarks ();
 
-	static PBD::Signal2<void,boost::shared_ptr<ARDOUR::Region>, const PBD::PropertyChange&> RegionPropertyChanged;
-
 	static PBD::Signal2<void,boost::shared_ptr<RegionList>, const PBD::PropertyChange&> RegionsPropertyChanged;
 
 	typedef std::map <PBD::PropertyChange, RegionList> ChangeMap;
diff --git a/libs/ardour/playlist.cc b/libs/ardour/playlist.cc
index b4c64c88e3..98a70d0518 100644
--- a/libs/ardour/playlist.cc
+++ b/libs/ardour/playlist.cc
@@ -626,7 +626,6 @@ Playlist::flush_notifications (bool from_undo)
 		remove_dependents (*s);
 		RegionRemoved (boost::weak_ptr<Region> (*s)); /* EMIT SIGNAL */
 		rl->push_back (*s);
-		Region::RegionPropertyChanged (*s, Properties::hidden); // XXX remove me
 	}
 	if (rl->size () > 0) {
 		Region::RegionsPropertyChanged (rl, Properties::hidden);
diff --git a/libs/ardour/region.cc b/libs/ardour/region.cc
index 66640a55b0..c4c6d9534b 100644
--- a/libs/ardour/region.cc
+++ b/libs/ardour/region.cc
@@ -86,7 +86,6 @@ namespace ARDOUR {
 	}
 }
 
-PBD::Signal2<void,boost::shared_ptr<ARDOUR::Region>,const PropertyChange&> Region::RegionPropertyChanged;
 PBD::Signal2<void,boost::shared_ptr<ARDOUR::RegionList>,const PropertyChange&> Region::RegionsPropertyChanged;
 
 void
@@ -1507,8 +1506,6 @@ Region::send_change (const PropertyChange& what_changed)
 				boost::shared_ptr<RegionList> rl (new RegionList);
 				rl->push_back (rptr);
 				RegionsPropertyChanged (rl, what_changed);
-
-				RegionPropertyChanged (rptr, what_changed); // XXX remove me
 			}
 		} catch (...) {
 			/* no shared_ptr available, relax; */
diff --git a/libs/ardour/track.cc b/libs/ardour/track.cc
index f1b3119e2b..181f8b6eba 100644
--- a/libs/ardour/track.cc
+++ b/libs/ardour/track.cc
@@ -659,13 +659,6 @@ Track::find_and_use_playlist (DataType dt, PBD::ID const & id)
 	return use_playlist (dt, playlist);
 }
 
-void
-update_region_visibility(boost::shared_ptr<Region> r)
-{
-	Region::RegionPropertyChanged(r, Properties::hidden); // XXX remove me
-}
-
-
 int
 Track::use_playlist (DataType dt, boost::shared_ptr<Playlist> p)
 {
@@ -688,14 +681,12 @@ Track::use_playlist (DataType dt, boost::shared_ptr<Playlist> p)
 		if (rl->size () > 0) {
 			Region::RegionsPropertyChanged (rl, Properties::hidden);
 		}
-		old->foreach_region(update_region_visibility); // XXX remove me
 	}
 	if (p) {
 		boost::shared_ptr<RegionList> rl (new RegionList (p->region_list_property ().rlist ()));
 		if (rl->size () > 0) {
 			Region::RegionsPropertyChanged (rl, Properties::hidden);
 		}
-		p->foreach_region(update_region_visibility); // XXX remove me
 	}
 
 	_session.set_dirty ();