Squash change signals with the same property and type.

There's a bit of an assumption here that handlers don't need to be told multiple
times about the same change (to different pieces of content) but that seems
reasonable.

1 files changed, 8 insertions, 1 deletions

diff --git a/src/lib/change_signaller.h b/src/lib/change_signaller.h
index 0b2c1539d..094716598 100644
--- a/src/lib/change_signaller.h
+++ b/src/lib/change_signaller.h
@@ -71,7 +71,14 @@ public:
 	{
 		if (_suspended) {
 			boost::mutex::scoped_lock lm(_mutex);
-			_pending.push_back(signal);
+			auto iter = std::find_if(
+				_pending.begin(),
+				_pending.end(),
+				[signal](ChangeSignal<T, P> const& s) { return s.property == signal.property && s.type == signal.type; }
+			);
+			if (iter == _pending.end()) {
+				_pending.push_back(signal);
+			}
 		} else {
 			signal.thing->signal_change(signal.type, signal.property);
 		}