X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=synfig-studio%2Ftrunk%2Fsrc%2Fgtkmm%2Fcanvastreestore.cpp;h=f49d7a3e4c8f04856d5634c02e40ef36df71d5be;hb=756c0d29ac1742f231e6615f9a577e574e35a4af;hp=d9bacd384ca5803311160d6a470aa8d9f63f44cd;hpb=3c7ce5771ea086b4caa2c33c60547586b1c997e2;p=synfig.git

diff --git a/synfig-studio/trunk/src/gtkmm/canvastreestore.cpp b/synfig-studio/trunk/src/gtkmm/canvastreestore.cpp
index d9bacd3..f49d7a3 100644
--- a/synfig-studio/trunk/src/gtkmm/canvastreestore.cpp
+++ b/synfig-studio/trunk/src/gtkmm/canvastreestore.cpp
@@ -6,6 +6,7 @@
 **
 **	\legal
 **	Copyright (c) 2002-2005 Robert B. Quattlebaum Jr., Adrian Bentley
+**	Copyright (c) 2008 Chris Moore
 **
 **	This package is free software; you can redistribute it and/or
 **	modify it under the terms of the GNU General Public License as
@@ -286,7 +287,8 @@ CanvasTreeStore::find_first_value_desc(const synfigapp::ValueDesc& value_desc, G
 		if(!iter->children().empty())
 		{
 			Gtk::TreeIter iter2(iter->children().begin());
-			if(iter2 && value_desc==(*iter2)[model.value_desc] || find_next_value_desc(value_desc, iter2))
+			//! \todo confirm that the && should be done before the ||
+			if((iter2 && value_desc==(*iter2)[model.value_desc]) || find_next_value_desc(value_desc, iter2))
 			{
 				iter=iter2;
 				return true;
@@ -310,7 +312,8 @@ CanvasTreeStore::find_next_value_desc(const synfigapp::ValueDesc& value_desc, Gt
 		if(!iter->children().empty())
 		{
 			Gtk::TreeIter iter2(iter->children().begin());
-			if(iter2 && value_desc==(*iter2)[model.value_desc] || find_next_value_desc(value_desc, iter2))
+			//! \todo confirm that the && should be done before the ||
+			if((iter2 && value_desc==(*iter2)[model.value_desc]) || find_next_value_desc(value_desc, iter2))
 			{
 				iter=iter2;
 				return true;
@@ -337,7 +340,8 @@ CanvasTreeStore::find_first_value_node(const synfig::ValueNode::Handle& value_no
 		if(!iter->children().empty())
 		{
 			Gtk::TreeIter iter2(iter->children().begin());
-			if(iter2 && value_node==(ValueNode::Handle)(*iter2)[model.value_node] || find_next_value_node(value_node, iter2))
+			//! \todo confirm that the && should be done before the ||
+			if((iter2 && value_node==(ValueNode::Handle)(*iter2)[model.value_node]) || find_next_value_node(value_node, iter2))
 			{
 				iter=iter2;
 				return true;
@@ -361,7 +365,8 @@ CanvasTreeStore::find_next_value_node(const synfig::ValueNode::Handle& value_nod
 		if(!iter->children().empty())
 		{
 			Gtk::TreeIter iter2(iter->children().begin());
-			if(iter2 && value_node==(ValueNode::Handle)(*iter2)[model.value_node] || find_next_value_node(value_node, iter2))
+			//! \todo confirm that the && should be done before the ||
+			if((iter2 && value_node==(ValueNode::Handle)(*iter2)[model.value_node]) || find_next_value_node(value_node, iter2))
 			{
 				iter=iter2;
 				return true;
@@ -383,7 +388,8 @@ void
 CanvasTreeStore::set_row(Gtk::TreeRow row,synfigapp::ValueDesc value_desc, bool do_children)
 {
 	Gtk::TreeModel::Children children = row.children();
-	while(!children.empty() && erase(children.begin()));
+	while(!children.empty() && erase(children.begin()))
+		;
 
 	row[model.value_desc]=value_desc;
 	try
@@ -564,7 +570,8 @@ CanvasTreeStore::rebuild_row(Gtk::TreeModel::Row &row, bool do_children)
 		row[model.is_value_node] = false;
 		row[model.is_editable] = true;
 		Gtk::TreeModel::Children children = row.children();
-		while(!children.empty() && erase(children.begin()));
+		while(!children.empty() && erase(children.begin()))
+			;
 	}
 	if(!do_children)
 		return;