Hey.

We discovered a few problems in the current osgt serialization code...

First, in src/osgWrappers/serializers/osgAnimation/RigGeometry.cpp:

The "VertexInfluence" property in readInfluenceMap() /
writeInfluenceMap(), does not use readWrappedString() /
writeWrappedString() - therefore things break when you happen to have
spaces in bone names. Also there is very questionable "" => "Empty"
mapping...

A similar one can be found in
src/osgWrappers/serializers/osgAnimation/Animation.cpp:

The "Name" and "TargetName" properties in readChannel() and
writeChannel() have the same problem.

A straight-forward patch (tested) is attached to this mail.

This patch obviously forward way breaks backwards compatibility... not
sure whether a check for a " char to determine the "old" and "new"
format would be appropriate here... If so... tell me. :-)

Of course with osgb everything works fine...

Another minor issue is in
src/osgWrappers/serializers/osgAnimation/RigGeometry.cpp:

The "Skeleton" property is not mapped - this forces to use a specific
hierarchy which forbids multi-referenced skeletons. I'm currently
adding that to our local build... If you like I can submit a patch for
that when it's done.

Cheers,

Felix
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to