From: Carlos Lopez <genetita@gmail.com>
Date: Mon, 26 Jul 2010 17:52:57 +0000 (+0200)
Subject: Delete commented code and add some documentation.
X-Git-Url: https://git.pterodactylus.net/?a=commitdiff_plain;h=c81544a192b7f971ec65498048c004fb86bce8ac;p=synfig.git

Delete commented code and add some documentation.
---

diff --git a/synfig-core/src/synfig/savecanvas.cpp b/synfig-core/src/synfig/savecanvas.cpp
index fa984ab..7682195 100644
--- a/synfig-core/src/synfig/savecanvas.cpp
+++ b/synfig-core/src/synfig/savecanvas.cpp
@@ -38,13 +38,9 @@
 #include "general.h"
 #include "valuenode.h"
 #include "valuenode_animated.h"
-//#include "valuenode_composite.h"
 #include "valuenode_const.h"
-//#include "valuenode_linear.h"
 #include "valuenode_dynamiclist.h"
 #include "valuenode_reference.h"
-//#include "valuenode_segcalctangent.h"
-//#include "valuenode_segcalcvertex.h"
 #include "valuenode_bline.h"
 #include "time.h"
 #include "keyframe.h"
@@ -265,10 +261,8 @@ xmlpp::Element* encode_animated(xmlpp::Element* root,ValueNode_Animated::ConstHa
 	for(iter=waypoint_list.begin();iter!=waypoint_list.end();++iter)
 	{
 		xmlpp::Element *waypoint_node=root->add_child("waypoint");
-		//waypoint_node->set_attribute("time",iter->get_time().get_string(canvas->rend_desc().get_frame_rate()));
 		waypoint_node->set_attribute("time",iter->get_time().get_string());
 
-		//waypoint_node->add_child(encode_value(iter->get_value(),canvas));
 		if(iter->get_value_node()->is_exported())
 			waypoint_node->set_attribute("use",iter->get_value_node()->get_relative_id(canvas));
 		else {
@@ -437,9 +431,6 @@ xmlpp::Element* encode_linkable_value_node(xmlpp::Element* root,LinkableValueNod
 		ValueBase value((*value_node)(0));
 		encode_value(root,value,canvas);
 
-		// ValueNode_Const::ConstHandle const_value(ValueNode_Const::create((*value_node)(0)));
-		// encode_value_node(root,const_value,canvas);
-
 		return root;
 	}
 
@@ -651,7 +642,11 @@ xmlpp::Element* encode_canvas(xmlpp::Element* root,Canvas::ConstHandle canvas)
 	}
 
 	// Output the <defs> section
-	//! \todo check where the parentheses should really go - around the && or the ||?
+	//! Check where the parentheses should really go - around the && or the ||?
+	//! If children is not empty (there are exported canvases in the current canvas)
+	//! they must be listed in the defs section regardless the result of check the
+	//! Value Node list (exported value nodes in the canvas) and if the canvas is
+	//! in line or not. Inline canvases cannot have exported canvases inside.
 	if((!canvas->is_inline() && !canvas->value_node_list().empty()) || !canvas->children().empty())
 	{
 		xmlpp::Element *node=root->add_child("defs");