TraversableNodeSet migrated to use UndoSystem signals

libs/scene/TraversableNodeSet.cpp

@@ -263,20 +263,19 @@ void TraversableNodeSet::importState(const IUndoMementoPtr& state)
 	if (!_undoInsertBuffer.empty())
 	{
 		// Register to get notified when the undo operation is complete
-		GlobalUndoSystem().addObserver(this);
+		_undoHandler = GlobalUndoSystem().signal_postUndo().connect(
+			sigc::mem_fun(this, &TraversableNodeSet::onUndoRedoOperationFinished));
+		_redoHandler = GlobalUndoSystem().signal_postRedo().connect(
+			sigc::mem_fun(this, &TraversableNodeSet::onUndoRedoOperationFinished));
 	}
 }
 
-void TraversableNodeSet::postUndo()
+void TraversableNodeSet::onUndoRedoOperationFinished()
 {
-	processInsertBuffer();
-	GlobalUndoSystem().removeObserver(this);
-}
+	_undoHandler.disconnect();
+	_redoHandler.disconnect();
 
-void TraversableNodeSet::postRedo()
-{
 	processInsertBuffer();
-	GlobalUndoSystem().removeObserver(this);
 }
 
 void TraversableNodeSet::processInsertBuffer()

libs/scene/TraversableNodeSet.h

@@ -4,6 +4,8 @@
 #include "iundo.h"
 #include <list>
 #include <boost/noncopyable.hpp>
+#include <sigc++/connection.h>
+#include <sigc++/trackable.h>
 
 class IMapFileChangeTracker;
 
@@ -24,7 +26,7 @@ class Node;
 class TraversableNodeSet :
 	public IUndoable,
 	public boost::noncopyable,
-	public UndoSystem::Observer
+	public sigc::trackable
 {
 public:
 	typedef std::list<INodePtr> NodeList;
@@ -40,6 +42,9 @@ class TraversableNodeSet :
 	// A list collecting nodes for insertion in postUndo/postRedo
 	NodeList _undoInsertBuffer;
 
+	sigc::connection _undoHandler;
+	sigc::connection _redoHandler;
+
 public:
 	// Default constructor, creates an empty set
 	TraversableNodeSet(Node& owner);
@@ -91,13 +96,12 @@ class TraversableNodeSet :
 	IUndoMementoPtr exportState() const;
 	void importState(const IUndoMementoPtr& state);
 
-	// UndoSystem::Observer implementation
-	void postUndo();
-	void postRedo();
-
 	void setRenderSystem(const RenderSystemPtr& renderSystem);
 
 private:
+	// UndoSystem event handler
+	void onUndoRedoOperationFinished();
+
 	// Sends the current state to the undosystem
 	void undoSave();