Hi Rui,

Thanks for the fixes, now merged and checked into svn/trunk and the
OSG-3.0 branch.

When reviewing the changes to
AsciiStreamOperator::readWrappedString(..) struck me as a potential
bug.  I understand how these changes are required, but I wonder if
that they aren't sufficient as they don't handle the possibility that
the _preReadString starts with ", and if it does whether it ends with
a ".   I strikes me that if it does begin with a " then one should
then search for any enclosed " and either return the substring and set
the  _preReadString to the remainder if it exists, or continue adding
characters to the string by reading them from the stream until the
full balanced string is read.

Thoughts?
Robert.

On Mon, Jul 4, 2011 at 4:23 AM, Wang Rui <[email protected]> wrote:
> Hi Robert,
>
> The files attached should be separately put into the directories
> src/osgPlugins/osg and src/osgWrappers/serializers/osgSim. They fix a
> serious infinite loop problem that may be caused by the stream buffer
> mechanism under Windows and some osgSim wrapper bugs pointed by
> Andreas. I've asked the community to help test them and hope now we
> can solve these recent .osgt file reading issues.
>
> Cheers,
>
> Wang Rui
>
> _______________________________________________
> osg-submissions mailing list
> [email protected]
> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
>
>
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to