X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=synfig-core%2Ftrunk%2Fsrc%2Fsynfig%2Fcanvas.cpp;h=b60734543b3c902ccee998ad3134ba53e19e0443;hb=0313f3153df081c11ff8a57d1ded6864142b21e2;hp=355a7381a08a98bf5d72125cda628166dd57f027;hpb=493e279ad4100cd467b906fd8b3eb106b2cef428;p=synfig.git

diff --git a/synfig-core/trunk/src/synfig/canvas.cpp b/synfig-core/trunk/src/synfig/canvas.cpp
index 355a738..b607345 100644
--- a/synfig-core/trunk/src/synfig/canvas.cpp
+++ b/synfig-core/trunk/src/synfig/canvas.cpp
@@ -526,7 +526,7 @@ Canvas::surefind_canvas(const String &id)
 			if(is_absolute_path(file_name))
 				external_canvas=open_canvas(file_name);
 			else
-				external_canvas=open_canvas(get_file_path()+ETL_DIRECTORY_SEPERATOR+file_name);
+				external_canvas=open_canvas(get_file_path()+ETL_DIRECTORY_SEPARATOR+file_name);
 
 			if(!external_canvas)
 				throw Exception::FileNotFound(file_name);
@@ -553,7 +553,7 @@ Canvas::surefind_canvas(const String &id)
 		return new_child_canvas(id);
 	}
 
-	// If the first character is the seperator, then
+	// If the first character is the separator, then
 	// this references the root canvas.
 	if(id[0]==':')
 		return get_root()->surefind_canvas(string(id,1));
@@ -611,7 +611,7 @@ Canvas::find_canvas(const String &id)const
 			if(is_absolute_path(file_name))
 				external_canvas=open_canvas(file_name);
 			else
-				external_canvas=open_canvas(get_file_path()+ETL_DIRECTORY_SEPERATOR+file_name);
+				external_canvas=open_canvas(get_file_path()+ETL_DIRECTORY_SEPARATOR+file_name);
 
 			if(!external_canvas)
 				throw Exception::FileNotFound(file_name);
@@ -636,7 +636,7 @@ Canvas::find_canvas(const String &id)const
 		throw Exception::IDNotFound("Child Canvas in Parent Canvas: (child)"+id);
 	}
 
-	// If the first character is the seperator, then
+	// If the first character is the separator, then
 	// this references the root canvas.
 	if(id.find_first_of(':')==0)
 		return get_root()->find_canvas(string(id,1));
@@ -700,24 +700,22 @@ Canvas::insert(iterator iter,etl::handle<Layer> x)
 	//while(correct_canvas->is_inline())correct_canvas=correct_canvas->parent();
 	Layer::LooseHandle loose_layer(x);
 
-	x->signal_added_to_group().connect(
-		sigc::bind(
-			sigc::mem_fun(
-				*correct_canvas,
-				&Canvas::add_group_pair
-			),
-			loose_layer
-		)
-	);
-	x->signal_removed_from_group().connect(
-		sigc::bind(
-			sigc::mem_fun(
-				*correct_canvas,
-				&Canvas::remove_group_pair
-			),
-			loose_layer
-		)
-	);
+	add_connection(loose_layer,
+				   sigc::connection::connection(
+					   x->signal_added_to_group().connect(
+						   sigc::bind(
+							   sigc::mem_fun(
+								   *correct_canvas,
+								   &Canvas::add_group_pair),
+							   loose_layer))));
+	add_connection(loose_layer,
+				   sigc::connection::connection(
+					   x->signal_removed_from_group().connect(
+						   sigc::bind(
+							   sigc::mem_fun(
+								   *correct_canvas,
+								   &Canvas::remove_group_pair),
+							   loose_layer))));
 
 
 	if(!x->get_group().empty())
@@ -749,8 +747,10 @@ Canvas::erase(Canvas::iterator iter)
 	// is using these signals, so I'll just
 	// leave these next two lines like they
 	// are for now - darco 07-30-2004
-	(*iter)->signal_added_to_group().clear();
-	(*iter)->signal_removed_from_group().clear();
+
+	// so don't wipe them out entirely
+	// - dooglus 09-21-2007
+	disconnect_connections(*iter);
 
 	if(!op_flag_)remove_child(iter->get());
 
@@ -1179,6 +1179,21 @@ Canvas::remove_group_pair(String group, etl::handle<Layer> layer)
 }
 
 void
+Canvas::add_connection(Layer::LooseHandle layer, sigc::connection connection)
+{
+	connections_[layer].push_back(connection);
+}
+
+void
+Canvas::disconnect_connections(Layer::LooseHandle layer)
+{
+	std::vector<sigc::connection>::iterator iter;
+	for(iter=connections_[layer].begin();iter!=connections_[layer].end();++iter)
+		iter->disconnect();
+	connections_[layer].clear();
+}
+
+void
 Canvas::rename_group(const String&old_name,const String&new_name)
 {
 	if(is_inline() && parent_)