X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=synfig-core%2Ftrunk%2Fsrc%2Fsynfig%2Fvaluenode_timedswap.cpp;h=1f369a1c134e6b83b3a11b402f2c40a621586bef;hb=334e15ce6c4d9b1f30a168a55e7ef4d31320d568;hp=39cc91d97ca0800fe16cbe79b8ec23e5fbdf27ff;hpb=b37086823c5bfee9c76f6b265584a414100963ef;p=synfig.git

diff --git a/synfig-core/trunk/src/synfig/valuenode_timedswap.cpp b/synfig-core/trunk/src/synfig/valuenode_timedswap.cpp
index 39cc91d..1f369a1 100644
--- a/synfig-core/trunk/src/synfig/valuenode_timedswap.cpp
+++ b/synfig-core/trunk/src/synfig/valuenode_timedswap.cpp
@@ -83,7 +83,7 @@ ValueNode_TimedSwap::ValueNode_TimedSwap(const ValueBase &value):
 		set_link("after",ValueNode_Const::create(value.get(Vector())));
 		break;
 	default:
-		throw Exception::BadType(ValueBase::type_name(get_type()));
+		throw Exception::BadType(ValueBase::type_local_name(get_type()));
 	}
 
 	set_link("time",ValueNode_Const::create(Time(2)));
@@ -98,95 +98,17 @@ ValueNode_TimedSwap::create(const ValueBase& x)
 	return new ValueNode_TimedSwap(x);
 }
 
-
 LinkableValueNode*
 ValueNode_TimedSwap::create_new()const
 {
 	return new ValueNode_TimedSwap(get_type());
 }
 
-
 synfig::ValueNode_TimedSwap::~ValueNode_TimedSwap()
 {
 	unlink_all();
 }
 
-
-
-bool
-ValueNode_TimedSwap::set_before(const ValueNode::Handle &x)
-{
-	if(!x || x->get_type()!=get_type()
-		&& !PlaceholderValueNode::Handle::cast_dynamic(x))
-		return false;
-
-	before=x;
-
-	return true;
-}
-
-ValueNode::Handle
-ValueNode_TimedSwap::get_before()const
-{
-	return before;
-}
-
-
-bool
-ValueNode_TimedSwap::set_after(const ValueNode::Handle &x)
-{
-	if(!x || x->get_type()!=get_type()
-		&& !PlaceholderValueNode::Handle::cast_dynamic(x))
-		return false;
-
-	after=x;
-
-	return true;
-}
-
-ValueNode::Handle
-ValueNode_TimedSwap::get_after()const
-{
-	return after;
-}
-
-
-bool
-ValueNode_TimedSwap::set_swap_time(const ValueNode::Handle &x)
-{
-	if(!x || (!ValueBase(x->get_type()).same_type_as(ValueBase::TYPE_TIME) &&
-			  !PlaceholderValueNode::Handle::cast_dynamic(x)))
-		return false;
-
-	swap_time=x;
-	return true;
-}
-
-ValueNode::Handle
-ValueNode_TimedSwap::get_swap_time()const
-{
-	return swap_time;
-}
-
-bool
-ValueNode_TimedSwap::set_swap_length(const ValueNode::Handle &x)
-{
-	if(!x || (!ValueBase(x->get_type()).same_type_as(ValueBase::TYPE_TIME) &&
-			  !PlaceholderValueNode::Handle::cast_dynamic(x)))
-		return false;
-
-	swap_length=x;
-	return true;
-}
-
-ValueNode::Handle
-ValueNode_TimedSwap::get_swap_length()const
-{
-	return swap_length;
-}
-
-
-
 synfig::ValueBase
 synfig::ValueNode_TimedSwap::operator()(Time t)const
 {
@@ -246,7 +168,6 @@ synfig::ValueNode_TimedSwap::operator()(Time t)const
 		}
 	}
 
-
 	/*! \todo this should interpolate from
 	**	before to after over the period defined
 	**	by swap_length */
@@ -254,17 +175,17 @@ synfig::ValueNode_TimedSwap::operator()(Time t)const
 	return (*before)(t);
 }
 
-
 bool
-ValueNode_TimedSwap::set_link_vfunc(int i,ValueNode::Handle x)
+ValueNode_TimedSwap::set_link_vfunc(int i,ValueNode::Handle value)
 {
-	assert(i>=0 && i<4);
+	assert(i>=0 && i<link_count());
+
 	switch(i)
 	{
-	case 0: return set_before(x);
-	case 1: return set_after(x);
-	case 2: return set_swap_time(x);
-	case 3: return set_swap_length(x);
+	case 0: CHECK_TYPE_AND_SET_VALUE(before,      get_type());
+	case 1: CHECK_TYPE_AND_SET_VALUE(after,       get_type());
+	case 2: CHECK_TYPE_AND_SET_VALUE(swap_time,   ValueBase::TYPE_TIME);
+	case 3: CHECK_TYPE_AND_SET_VALUE(swap_length, ValueBase::TYPE_TIME);
 	}
 	return false;
 }
@@ -272,13 +193,14 @@ ValueNode_TimedSwap::set_link_vfunc(int i,ValueNode::Handle x)
 ValueNode::LooseHandle
 ValueNode_TimedSwap::get_link_vfunc(int i)const
 {
-	assert(i>=0 && i<4);
+	assert(i>=0 && i<link_count());
+
 	switch(i)
 	{
-	case 0: return get_before();
-	case 1: return get_after();
-	case 2: return get_swap_time();
-	case 3: return get_swap_length();
+	case 0: return before;
+	case 1: return after;
+	case 2: return swap_time;
+	case 3: return swap_length;
 	}
 	return 0;
 }
@@ -292,7 +214,8 @@ ValueNode_TimedSwap::link_count()const
 String
 ValueNode_TimedSwap::link_local_name(int i)const
 {
-	assert(i>=0 && i<4);
+	assert(i>=0 && i<link_count());
+
 	switch(i)
 	{
 	case 0: return _("Before");
@@ -306,7 +229,8 @@ ValueNode_TimedSwap::link_local_name(int i)const
 String
 ValueNode_TimedSwap::link_name(int i)const
 {
-	assert(i>=0 && i<4);
+	assert(i>=0 && i<link_count());
+
 	switch(i)
 	{
 	case 0: return "before";