Hi Jason,
I've fixed the warning by changing an int to std::string::size_type, I
presume you didn't spot the warning because you're on a 32bit OS while
I'm working under 64bit. Alas fixing this warning and associated code
didn't solve the crash.
On reviewing VBSPReader.cpp to fix this warning I noticed that you
declare all your variables at the top of each method. This is C style
coding that is best avoided as it can easily lead to uninitialized
variables. Best practice in C++ varable declaration is to move the
variable declaration to where it's first required and to always
initialize it if no default is otherwise provided.
I've moved the declarations and added initializers to a couple of the
methods in VBSOReader.cpp These changes are your fix to
CBSPGeometry.cpp are checked in as revision 9241. Could review this
and particular look at the variable declaration changes, and use this
type in the rest of the code as there is a chance that an
uninitialized variable is the cause of this problem.
http://www.openscenegraph.org/projects/osg/changeset/9241
Cheers,
Robert.
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
