Removed a bunch more DEBUGPOINT()s.
[synfig.git] / synfig-core / trunk / src / synfig / loadcanvas.cpp
index bbb6d77..199a609 100644 (file)
@@ -6,6 +6,7 @@
 **
 **     \legal
 **     Copyright (c) 2002-2005 Robert B. Quattlebaum Jr., Adrian Bentley
+**     Copyright (c) 2007 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
@@ -234,7 +235,7 @@ CanvasParser::parse_keyframe(xmlpp::Element *element,Canvas::Handle canvas)
 
 
 Real
-CanvasParser::parse_real(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_real(xmlpp::Element *element)
 {
        assert(element->get_name()=="real");
 
@@ -272,7 +273,7 @@ CanvasParser::parse_time(xmlpp::Element *element,Canvas::Handle canvas)
 }
 
 int
-CanvasParser::parse_integer(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_integer(xmlpp::Element *element)
 {
        assert(element->get_name()=="integer");
 
@@ -290,8 +291,17 @@ CanvasParser::parse_integer(xmlpp::Element *element,Canvas::Handle canvas)
        return atoi(val.c_str());
 }
 
+// see 'minor hack' at the end of parse_vector() below
+// making this 'static' to give it file local scope
+// stops it working (where working means working around
+// bug #1509627)
+Vector &canvas_parser_vector_id(Vector &vector)
+{
+       return vector;
+}
+
 Vector
-CanvasParser::parse_vector(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_vector(xmlpp::Element *element)
 {
        assert(element->get_name()=="vector");
 
@@ -332,11 +342,16 @@ CanvasParser::parse_vector(xmlpp::Element *element,Canvas::Handle canvas)
                else
                        error_unexpected_element(child,child->get_name());
        }
-       return vect;
+       // Minor hack - gcc 4.1.2 and earlier think that we're not using
+       // 'vect' and optimize it out at -O2 and higher.  This convinces
+       // them that we are really using it.
+       return canvas_parser_vector_id(vect);
+       // When the bug is fixed, we can just do this instead:
+       // return vect;
 }
 
 Color
-CanvasParser::parse_color(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_color(xmlpp::Element *element)
 {
        assert(element->get_name()=="color");
 
@@ -346,7 +361,7 @@ CanvasParser::parse_color(xmlpp::Element *element,Canvas::Handle canvas)
                return Color();
        }
 
-       Color color;
+       Color color(0);
 
        xmlpp::Element::NodeList list = element->get_children();
        for(xmlpp::Element::NodeList::iterator iter = list.begin(); iter != list.end(); ++iter)
@@ -402,7 +417,7 @@ CanvasParser::parse_color(xmlpp::Element *element,Canvas::Handle canvas)
 }
 
 synfig::String
-CanvasParser::parse_string(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_string(xmlpp::Element *element)
 {
        assert(element->get_name()=="string");
 
@@ -422,7 +437,7 @@ CanvasParser::parse_string(xmlpp::Element *element,Canvas::Handle canvas)
 }
 
 bool
-CanvasParser::parse_bool(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_bool(xmlpp::Element *element)
 {
        assert(element->get_name()=="bool");
 
@@ -448,7 +463,7 @@ CanvasParser::parse_bool(xmlpp::Element *element,Canvas::Handle canvas)
 }
 
 Gradient
-CanvasParser::parse_gradient(xmlpp::Element *node,Canvas::Handle canvas)
+CanvasParser::parse_gradient(xmlpp::Element *node)
 {
        assert(node->get_name()=="gradient");
        Gradient ret;
@@ -462,7 +477,7 @@ CanvasParser::parse_gradient(xmlpp::Element *node,Canvas::Handle canvas)
                else
                {
                        Gradient::CPoint cpoint;
-                       cpoint.color=parse_color(child,canvas);
+                       cpoint.color=parse_color(child);
 
                        if(!child->get_attribute("pos"))
                        {
@@ -505,7 +520,7 @@ CanvasParser::parse_list(xmlpp::Element *element,Canvas::Handle canvas)
 }
 
 Segment
-CanvasParser::parse_segment(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_segment(xmlpp::Element *element)
 {
        assert(element->get_name()=="segment");
 
@@ -545,7 +560,7 @@ CanvasParser::parse_segment(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       seg.p1=parse_vector(dynamic_cast<xmlpp::Element*>(*iter),canvas);
+                       seg.p1=parse_vector(dynamic_cast<xmlpp::Element*>(*iter));
                }
                else
                if(child->get_name()=="t1")
@@ -569,7 +584,7 @@ CanvasParser::parse_segment(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       seg.t1=parse_vector(dynamic_cast<xmlpp::Element*>(*iter),canvas);
+                       seg.t1=parse_vector(dynamic_cast<xmlpp::Element*>(*iter));
                }
                else
                if(child->get_name()=="p2")
@@ -593,7 +608,7 @@ CanvasParser::parse_segment(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       seg.p2=parse_vector(dynamic_cast<xmlpp::Element*>(*iter),canvas);
+                       seg.p2=parse_vector(dynamic_cast<xmlpp::Element*>(*iter));
                }
                else
                if(child->get_name()=="t2")
@@ -617,7 +632,7 @@ CanvasParser::parse_segment(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       seg.t2=parse_vector(dynamic_cast<xmlpp::Element*>(*iter),canvas);
+                       seg.t2=parse_vector(dynamic_cast<xmlpp::Element*>(*iter));
                }
                else
                        error_unexpected_element(child,child->get_name());
@@ -626,7 +641,7 @@ CanvasParser::parse_segment(xmlpp::Element *element,Canvas::Handle canvas)
 }
 
 BLinePoint
-CanvasParser::parse_bline_point(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_bline_point(xmlpp::Element *element)
 {
        assert(element->get_name()=="bline_point");
        if(element->get_children().empty())
@@ -667,7 +682,7 @@ CanvasParser::parse_bline_point(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       ret.set_vertex(parse_vector(dynamic_cast<xmlpp::Element*>(*iter),canvas));
+                       ret.set_vertex(parse_vector(dynamic_cast<xmlpp::Element*>(*iter)));
                }
                else
                // Tangent 1
@@ -692,7 +707,7 @@ CanvasParser::parse_bline_point(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       ret.set_tangent1(parse_vector(dynamic_cast<xmlpp::Element*>(*iter),canvas));
+                       ret.set_tangent1(parse_vector(dynamic_cast<xmlpp::Element*>(*iter)));
                }
                else
                // Tangent 2
@@ -717,7 +732,7 @@ CanvasParser::parse_bline_point(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       ret.set_tangent2(parse_vector(dynamic_cast<xmlpp::Element*>(*iter),canvas));
+                       ret.set_tangent2(parse_vector(dynamic_cast<xmlpp::Element*>(*iter)));
                        ret.set_split_tangent_flag(true);
                }
                else
@@ -743,7 +758,7 @@ CanvasParser::parse_bline_point(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       ret.set_width(parse_real(dynamic_cast<xmlpp::Element*>(*iter),canvas));
+                       ret.set_width(parse_real(dynamic_cast<xmlpp::Element*>(*iter)));
                }
                else
                // origin
@@ -768,7 +783,7 @@ CanvasParser::parse_bline_point(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       ret.set_origin(parse_real(dynamic_cast<xmlpp::Element*>(*iter),canvas));
+                       ret.set_origin(parse_real(dynamic_cast<xmlpp::Element*>(*iter)));
                }
                else
                        error_unexpected_element(child,child->get_name());
@@ -777,7 +792,7 @@ CanvasParser::parse_bline_point(xmlpp::Element *element,Canvas::Handle canvas)
 }
 
 Angle
-CanvasParser::parse_angle(xmlpp::Element *element,Canvas::Handle canvas)
+CanvasParser::parse_angle(xmlpp::Element *element)
 {
        assert(element->get_name()=="angle");
 
@@ -799,51 +814,48 @@ ValueBase
 CanvasParser::parse_value(xmlpp::Element *element,Canvas::Handle canvas)
 {
        if(element->get_name()=="real")
-               return parse_real(element,canvas);
+               return parse_real(element);
        else
        if(element->get_name()=="time")
                return parse_time(element,canvas);
        else
        if(element->get_name()=="integer")
-               return parse_integer(element,canvas);
+               return parse_integer(element);
        else
        if(element->get_name()=="string")
-               return parse_string(element,canvas);
+               return parse_string(element);
        else
        if(element->get_name()=="vector")
-       {
-               return parse_vector(element,canvas);
-       }
+               return parse_vector(element);
        else
        if(element->get_name()=="color")
-               return parse_color(element,canvas);
+               return parse_color(element);
        else
        if(element->get_name()=="segment")
-               return parse_segment(element,canvas);
+               return parse_segment(element);
        else
        if(element->get_name()=="list")
                return parse_list(element,canvas);
        else
        if(element->get_name()=="gradient")
-               return parse_gradient(element,canvas);
+               return parse_gradient(element);
        else
        if(element->get_name()=="bool")
-               return parse_bool(element,canvas);
+               return parse_bool(element);
        else
        //if(element->get_name()=="canvas")
        //      return parse_canvas(element,canvas,true);       // inline canvas
        //else
        if(element->get_name()=="angle" || element->get_name()=="degrees" || element->get_name()=="radians" || element->get_name()=="rotations")
-               return parse_angle(element,canvas);
+               return parse_angle(element);
        else
        if(element->get_name()=="bline_point")
-               return parse_bline_point(element,canvas);
+               return parse_bline_point(element);
        else
        if(element->get_name()=="canvas")
                return ValueBase(parse_canvas(element,canvas,true));
        else
        {
-               DEBUGPOINT();
                error_unexpected_element(element,element->get_name());
        }
 
@@ -876,7 +888,7 @@ CanvasParser::parse_animated(xmlpp::Element *element,Canvas::Handle canvas)
 
        if(!value_node)
        {
-               error(element,strprintf(_("Unable to create <animated> with type \"%s\""),ValueBase::type_name(type).c_str()));
+               error(element,strprintf(_("Unable to create <animated> with type \"%s\""),ValueBase::type_local_name(type).c_str()));
                return ValueNode_Animated::Handle();
        }
 
@@ -901,10 +913,22 @@ CanvasParser::parse_animated(xmlpp::Element *element,Canvas::Handle canvas)
 
 
                        ValueNode::Handle waypoint_value_node;
+                       xmlpp::Element::NodeList list = child->get_children();
 
                        if(child->get_attribute("use"))
                        {
-                               waypoint_value_node=canvas->surefind_value_node(child->get_attribute("use")->get_value());
+                               if(!list.empty())
+                                       warning(child,_("Found \"use\" attribute for <waypoint>, but it wasn't empty. Ignoring contents..."));
+
+                               // the waypoint might look like this, in which case we won't find "mycanvas" in the list of valuenodes, 'cos it's a canvas
+                               //
+                               //      <animated type="canvas">
+                               //        <waypoint time="0s" use="mycanvas"/>
+                               //      </animated>
+                               if (type==ValueBase::TYPE_CANVAS)
+                                       waypoint_value_node=ValueNode_Const::create(canvas->surefind_canvas(child->get_attribute("use")->get_value()));
+                               else
+                                       waypoint_value_node=canvas->surefind_value_node(child->get_attribute("use")->get_value());
                        }
                        else
                        {
@@ -914,7 +938,6 @@ CanvasParser::parse_animated(xmlpp::Element *element,Canvas::Handle canvas)
                                        continue;
                                }
 
-                               xmlpp::Element::NodeList list = child->get_children();
                                xmlpp::Element::NodeList::iterator iter;
 
                                // Search for the first non-text XML element
@@ -1027,220 +1050,47 @@ CanvasParser::parse_animated(xmlpp::Element *element,Canvas::Handle canvas)
                else
                        error_unexpected_element(child,child->get_name());
        }
-       value_node->changed();
-       return value_node;
-}
-
-// This function is a phase-out hack for the timed swap value node
-etl::handle<ValueNode_Animated>
-CanvasParser::parse_timedswap(xmlpp::Element *node,Canvas::Handle canvas)
-{
-       ValueNode_TimedSwap::Handle timed_swap(parse_linkable_value_node(node,canvas));
-
-       assert(timed_swap);
-
-       ValueNode_Animated::Handle animated(ValueNode_Animated::create(timed_swap->get_type()));
-
-       animated->set_root_canvas(canvas->get_root());
-
-       assert(animated);
-
-       Time swap_time, swap_length;
-       (*timed_swap->get_swap_time())(0).put(&swap_time);
-       (*timed_swap->get_swap_length())(0).put(&swap_length);
-
-       animated->new_waypoint(swap_time-swap_length,timed_swap->get_before());
-       animated->new_waypoint(swap_time,timed_swap->get_after());
-
-       return animated;
-}
-
-
-handle<ValueNode_Subtract>
-CanvasParser::parse_subtract(xmlpp::Element *element,Canvas::Handle canvas)
-{
-       assert(element->get_name()=="subtract");
-
-       handle<ValueNode_Subtract> value_node;
-       handle<ValueNode> lhs,rhs,scalar;
-
-       if(element->get_attribute("type"))
-       {
-               ValueBase::Type type=ValueBase::ident_type(element->get_attribute("type")->get_value());
-
-               if(!type)
-               {
-                       error(element,"Bad type in <subtract>");
-                       return ValueNode_Subtract::Handle();
-               }
-               value_node=ValueNode_Subtract::create(type);
-       }
-       else
-               value_node=ValueNode_Subtract::create();
-
-       if(!value_node)
-       {
-               error(element,strprintf(_("Unable to create <subtract>")));
-               return handle<ValueNode_Subtract>();
-       }
-
-       //if(element->get_attribute("scalar"))
-       //{
-       //      value_node->set_scalar(atof(element->get_attribute("scalar")->get_value().c_str()));
-       //}
-
-       try
-       {
-               if(element->get_attribute("scalar"))
-               {
-                       // This is for compatibility with older versions of the file format
-                       String value(element->get_attribute("scalar")->get_value());
-                       if((value[0]<='9' && value[0]>='0')      || value[0]=='-')
-                       {
-                               warning(element, _("Use of a real value where the ID should be is deprecated"));
-                               value_node->set_scalar(atof(value.c_str()));
-                       }
-                       else
-                               value_node->set_scalar(canvas->surefind_value_node(value));
-                       scalar=value_node->get_scalar();
-               }
 
-               if(element->get_attribute("lhs"))
-               {
-                       lhs=canvas->surefind_value_node(element->get_attribute("lhs")->get_value());
-                       value_node->set_lhs(lhs);
-               }
-
-               if(element->get_attribute("rhs"))
-               {
-                       rhs=canvas->surefind_value_node(element->get_attribute("rhs")->get_value());
-                       value_node->set_rhs(rhs);
-               }
-       }
-       catch (Exception::IDNotFound)
-       {
-               error(element,"attribute in <subtract> references unknown ID");
-       }
-       catch (Exception::FileNotFound)
-       {
-               error(element,"Unable to open external file referenced in ID");
-       }
+       // in canvas version 0.1, angles used to wrap, so to get from -179
+       // degrees to 180 degrees meant a 1 degree change
+       // in canvas version 0.2 they don't, so that's a 359 degree change
 
-       xmlpp::Element::NodeList list = element->get_children();
-       for(xmlpp::Element::NodeList::iterator iter = list.begin(); iter != list.end(); ++iter)
+       // when loading a version 0.1 canvas, modify constant angle
+       // waypoints to that they are within 180 degrees of the previous
+       // waypoint's value
+       if (type == ValueBase::TYPE_ANGLE) 
        {
-               xmlpp::Element *child(dynamic_cast<xmlpp::Element*>(*iter));
-               if(!child)
-                       continue;
-               else
-               if(child->get_name()=="lhs")
-               {
-                       if(lhs)
-                       {
-                               error(child,"LHS component of <subtract> already defined!");
-                               continue;
-                       }
-
-                       xmlpp::Element::NodeList list = child->get_children();
-                       xmlpp::Element::NodeList::iterator iter;
-
-                       // Search for the first non-text XML element
-                       for(iter = list.begin(); iter != list.end(); ++iter)
-                               if(dynamic_cast<xmlpp::Element*>(*iter)) break;
-
-                       if(iter==list.end())
-                       {
-                               error(child,_("<lhs> is missing its contents"));
-                               continue;
-                       }
-
-                       lhs=parse_value_node(dynamic_cast<xmlpp::Element*>(*iter),canvas);
-
-                       if(lhs)
-                               value_node->set_lhs(lhs);
-                       else
-                       {
-                               error((*iter),"Parse of LHS ValueNode failed");
-                       }
-
-                       // \todo do a search for more elements and warn if they are found
-               }
-               else
-               if(child->get_name()=="rhs")
-               {
-                       if(rhs)
-                       {
-                               error(child,"RHS component of <subtract> already defined!");
-                               continue;
-                       }
-
-                       xmlpp::Element::NodeList list = child->get_children();
-                       xmlpp::Element::NodeList::iterator iter;
-
-                       // Search for the first non-text XML element
-                       for(iter = list.begin(); iter != list.end(); ++iter)
-                               if(dynamic_cast<xmlpp::Element*>(*iter)) break;
-
-                       if(iter==list.end())
-                       {
-                               error(child,_("<rhs> is missing its contents"));
-                               continue;
-                       }
+               Canvas::Handle parent = canvas;
+               while (!parent->is_root())
+                       parent = parent->parent();
 
-                       rhs=parse_value_node(dynamic_cast<xmlpp::Element*>(*iter),canvas);
-
-                       if(rhs)
-                               value_node->set_rhs(rhs);
-                       else
-                       {
-                               error((*iter),"Parse of RHS ValueNode failed");
-                       }
-
-                       // \todo do a search for more elements and warn if they are found
-               }
-               else
-               if(child->get_name()=="scalar")
+               if (parent->get_version() == "0.1")
                {
-                       if(scalar)
-                       {
-                               error(child,"scalar component of <subtract> already defined!");
-                               continue;
-                       }
-
-                       xmlpp::Element::NodeList list = child->get_children();
-                       xmlpp::Element::NodeList::iterator iter;
-
-                       // Search for the first non-text XML element
-                       for(iter = list.begin(); iter != list.end(); ++iter)
-                               if(dynamic_cast<xmlpp::Element*>(*iter)) break;
-
-                       if(iter==list.end())
-                       {
-                               error(child,_("<scalar> is missing its contents"));
-                               continue;
-                       }
-
-                       scalar=parse_value_node(dynamic_cast<xmlpp::Element*>(*iter),canvas);
-
-                       if(scalar)
-                               value_node->set_scalar(scalar);
-                       else
-                       {
-                               error((*iter),"Parse of scalar ValueNode failed");
+                       bool first = true;
+                       Real angle, prev = 0;
+                       WaypointList &wl = value_node->waypoint_list();
+                       for (WaypointList::iterator iter = wl.begin(); iter != wl.end(); iter++)
+                       {
+                               angle = Angle::deg(iter->get_value(iter->get_time()).get(Angle())).get();
+                               if (first)
+                                       first = false;
+                               else if (iter->get_value_node()->get_name() == "constant")
+                                       if (angle - prev > 180)
+                                       {
+                                               while (angle - prev > 180) angle -= 360;
+                                               iter->set_value(Angle::deg(angle));
+                                       }
+                                       else if (prev - angle > 180)
+                                       {
+                                               while (prev - angle > 180) angle += 360;
+                                               iter->set_value(Angle::deg(angle));
+                                       }
+                               prev = angle;
                        }
-
-                       // \todo do a search for more elements and warn if they are found
                }
-               else
-                       error_unexpected_element(child,child->get_name());
        }
 
-       if(!value_node->get_rhs() || !value_node->get_lhs() || !value_node->get_scalar())
-               error(element,"<subtract> is missing LHS, RHS, or SCALAR");
-
-       if(value_node->get_rhs() == value_node->get_lhs())
-               warning(element,"LHS is equal to RHS in <subtract>, so this value_node will always be zero!");
-
+       value_node->changed();
        return value_node;
 }
 
@@ -1347,8 +1197,8 @@ CanvasParser::parse_linkable_value_node(xmlpp::Element *element,Canvas::Handle c
                        else
                        if(!value_node->set_link(index,link))
                        {
-                               //error(dynamic_cast<xmlpp::Element*>(*iter),strprintf("Unable to connect value node ('%s' of type '%s') to link %d",link->get_name().c_str(),ValueBase::type_name(link->get_type()).c_str(),index));
-                               error(element,strprintf("Unable to connect value node ('%s' of type '%s') to link %d",link->get_name().c_str(),ValueBase::type_name(link->get_type()).c_str(),index));
+                               //error(dynamic_cast<xmlpp::Element*>(*iter),strprintf(_("Unable to connect value node ('%s' of type '%s') to link %d"),link->get_name().c_str(),ValueBase::type_local_name(link->get_type()).c_str(),index));
+                               error(element,strprintf(_("Unable to connect value node ('%s' of type '%s') to link %d"),link->get_name().c_str(),ValueBase::type_local_name(link->get_type()).c_str(),index));
                        }
 
                        // \todo do a search for more elements and warn if they are found
@@ -1601,6 +1451,10 @@ CanvasParser::parse_dynamic_list(xmlpp::Element *element,Canvas::Handle canvas)
                                        String timecode(begin_sequence.begin(), iter);
                                        int priority=0;
 
+                                       // skip whitespace before checking for a priority
+                                       while (isspace(timecode[0]))
+                                               timecode=timecode.substr(1);
+
                                        // If there is a priority, then grab it and remove
                                        // it from the timecode
                                        if(timecode[0]=='p')
@@ -1637,6 +1491,10 @@ CanvasParser::parse_dynamic_list(xmlpp::Element *element,Canvas::Handle canvas)
                                        String timecode(end_sequence.begin(),   iter);
                                        int priority=0;
 
+                                       // skip whitespace before checking for a priority
+                                       while (isspace(timecode[0]))
+                                               timecode=timecode.substr(1);
+
                                        // If there is a priority, then grab it and remove
                                        // it from the timecode
                                        if(timecode[0]=='p')
@@ -1670,6 +1528,7 @@ CanvasParser::parse_dynamic_list(xmlpp::Element *element,Canvas::Handle canvas)
 
                        if(child->get_attribute("use"))
                        {
+                               // \todo does this need to be able to read 'use="canvas"', like waypoints can now?  (see 'surefind_canvas' in this file)
                                string id=child->get_attribute("use")->get_value();
                                try
                                {
@@ -1730,7 +1589,7 @@ CanvasParser::parse_value_node(xmlpp::Element *element,Canvas::Handle canvas)
                        return value_node;
        }
 
-       // If ValueBase::ident_type() recognises the name, then we know it's a ValueBase
+       // If ValueBase::ident_type() recognizes the name, then we know it's a ValueBase
        if(element->get_name()!="canvas" && ValueBase::ident_type(element->get_name()))
        {
                ValueBase data=parse_value(element,canvas);
@@ -1752,9 +1611,6 @@ CanvasParser::parse_value_node(xmlpp::Element *element,Canvas::Handle canvas)
        if(element->get_name()=="hermite" || element->get_name()=="animated")
                value_node=parse_animated(element,canvas);
        else
-       if(element->get_name()=="subtract")
-               value_node=parse_subtract(element,canvas);
-       else
        if(element->get_name()=="composite")
                value_node=parse_composite(element,canvas);
        else
@@ -1764,9 +1620,6 @@ CanvasParser::parse_value_node(xmlpp::Element *element,Canvas::Handle canvas)
        if(element->get_name()=="bline") // This is not a typo. The dynamic list parser will parse a bline.
                value_node=parse_dynamic_list(element,canvas);
        else
-       if(element->get_name()=="timed_swap")
-               value_node=parse_timedswap(element,canvas);
-       else
        if(LinkableValueNode::book().count(element->get_name()))
                value_node=parse_linkable_value_node(element,canvas);
        else
@@ -1776,6 +1629,7 @@ CanvasParser::parse_value_node(xmlpp::Element *element,Canvas::Handle canvas)
        {
                error_unexpected_element(element,element->get_name());
                error(element, "Expected a ValueNode");
+               value_node=PlaceholderValueNode::create();
        }
 
 
@@ -1860,7 +1714,7 @@ CanvasParser::parse_layer(xmlpp::Element *element,Canvas::Handle canvas)
        {
                String version(element->get_attribute("version")->get_value());
                if(version>layer->get_version())
-                       warning(element,_("Installed layer version is larger than layer version in file"));
+                       warning(element,_("Installed layer version is smaller than layer version in file"));
                if(version!=layer->get_version())
                        layer->set_version(version);
        }
@@ -1942,7 +1796,7 @@ CanvasParser::parse_layer(xmlpp::Element *element,Canvas::Handle canvas)
                                continue;
                        }
 
-                       // If we recognise the element name as a
+                       // If we recognize the element name as a
                        // ValueBase, then treat is at one
                        if(/*(*iter)->get_name()!="canvas" && */ValueBase::ident_type((*iter)->get_name()) && !dynamic_cast<xmlpp::Element*>(*iter)->get_attribute("guid"))
                        {
@@ -2041,6 +1895,9 @@ CanvasParser::parse_canvas(xmlpp::Element *element,Canvas::Handle parent,bool in
                        canvas->set_guid(guid);
        }
 
+       if(element->get_attribute("version"))
+               canvas->set_version(element->get_attribute("version")->get_value());
+
        if(element->get_attribute("width"))
                canvas->rend_desc().set_w(atoi(element->get_attribute("width")->get_value().c_str()));
 
@@ -2175,7 +2032,7 @@ CanvasParser::parse_canvas(xmlpp::Element *element,Canvas::Handle parent,bool in
 
                                // If we don't have any name, warn
                                if(list.empty())
-                                       warning(child,_("blank \"name\" entitity"));
+                                       warning(child,_("blank \"name\" entity"));
 
                                string tmp;
                                for(xmlpp::Element::NodeList::iterator iter = list.begin(); iter != list.end(); ++iter)
@@ -2190,7 +2047,7 @@ CanvasParser::parse_canvas(xmlpp::Element *element,Canvas::Handle parent,bool in
 
                                // If we don't have any description, warn
                                if(list.empty())
-                                       warning(child,_("blank \"desc\" entitity"));
+                                       warning(child,_("blank \"desc\" entity"));
 
                                string tmp;
                                for(xmlpp::Element::NodeList::iterator iter = list.begin(); iter != list.end(); ++iter)
@@ -2205,7 +2062,7 @@ CanvasParser::parse_canvas(xmlpp::Element *element,Canvas::Handle parent,bool in
 
                                // If we don't have any description, warn
                                if(list.empty())
-                                       warning(child,_("blank \"author\" entitity"));
+                                       warning(child,_("blank \"author\" entity"));
 
                                string tmp;
                                for(xmlpp::Element::NodeList::iterator iter = list.begin(); iter != list.end(); ++iter)
@@ -2230,9 +2087,20 @@ CanvasParser::parse_canvas(xmlpp::Element *element,Canvas::Handle parent,bool in
 
        if(canvas->value_node_list().placeholder_count())
        {
-               error(element,strprintf(_("Canvas %s has undefined ValueNodes"),canvas->get_id().c_str()));
+               String nodes;
+               for (ValueNodeList::const_iterator iter = canvas->value_node_list().begin(); iter != canvas->value_node_list().end(); iter++)
+                       if(PlaceholderValueNode::Handle::cast_dynamic(*iter))
+                       {
+                               if (nodes != "") nodes += ", ";
+                               nodes += "'" + (*iter)->get_id() + "'";
+                       }
+               error(element,strprintf(_("Canvas '%s' has undefined %s: %s"),
+                                                               canvas->get_id().c_str(),
+                                                               canvas->value_node_list().placeholder_count() == 1 ? _("ValueNode") : _("ValueNodes"),
+                                                               nodes.c_str()));
        }
 
+       canvas->set_version(CURRENT_CANVAS_VERSION);
        return canvas;
 }
 
@@ -2245,7 +2113,6 @@ CanvasParser::parse_from_file(const String &file)
 Canvas::Handle
 CanvasParser::parse_from_file_as(const String &file_,const String &as_)
 {
-       CHECK_EXPIRE_TIME();
        try
        {
         ChangeLocale change_locale(LC_NUMERIC, "C");
@@ -2306,8 +2173,6 @@ CanvasParser::parse_from_file_as(const String &file_,const String &as_)
 Canvas::Handle
 CanvasParser::parse_from_string(const String &data)
 {
-       CHECK_EXPIRE_TIME();
-
        try
        {
         ChangeLocale change_locale(LC_NUMERIC, "C");