Forgot to mention: There is a chance that lib3DS files in OSG have tiny 
modifications...

Sukender
PVLE - Lightweight cross-platform game engine - http://pvle.sourceforge.net/

----- "Sukender" <[email protected]> a écrit :

> Hi Chris,
> 
> This was discussed multiple times before. For now we just include the
> lib3DS. I guess it would be nice if you could update to the latest
> version. I personnally made the last update... I guess it was 1.2 but
> I'm not sure.
> 
> Sukender
> PVLE - Lightweight cross-platform game engine -
> http://pvle.sourceforge.net/
> 
> ----- "Chris Denham" <[email protected]> a écrit :
> 
> > Hi Sukender,
> > I just downloaded lib3ds version 1.3.0 (2007-06-28 ) and was amazed
> to
> > find that the change I submitted to the OSG version of the library
> was
> > already there in v1.3.0! So looks like the leak was plugged in
> lib3ds
> > quite a while back. I suppose that adds confidence that my
> submission
> > is good, but then begs the question, "What version of lib3ds is osg
> > plugin based on?" Is there a plan to bring in lib3ds as a OSG third
> > party dependency, or to bring the OSG code inline with it?
> > Thanks.
> > Chris.
> >
> >
> >
> > Sukender wrote:
> > > Hi Chris,
> > >
> > > As you moidified a lib3DS file, could you send it back to the
> lib3DS
> > project, please?
> > > (I didn't check your change yet)
> > > Thanks.
> > >
> > > Sukender
> > > PVLE - Lightweight cross-platform game engine -
> > http://pvle.sourceforge.net/
> > >
> > > ----- "Chris Denham" <> a écrit :
> > >
> > >
> > > > I've discovered a memory leak in the 3DS plugin.
> > > > Seems occur only for some 3DS files.
> > > > The leak reported is for an allocation in:
> > > > file: viewport.cpp
> > > > function: lib3ds_viewport_read
> > > > line:
> > > >
> > viewport->layout.viewL=(Lib3dsView*)realloc(viewport->layout.viewL,
> > > > sizeof(Lib3dsView)*views);
> > > >
> > > > It seems that the Lib3dsFile object may contain two viewport
> > objects,
> > > > but the 'lib3ds_file_free' function only releases one of them.
> > > > My submission adds the line:
> > > > lib3ds_viewport_set_views(&file->viewport_keyf,0);
> > > > after the line:
> > > > lib3ds_viewport_set_views(&file->viewport,0);
> > > > which seems to plug the leak I see when loading a file that
> causes
> > > > creation of the 'viewport_keyf' object.
> > > >
> > > > I have attached the modification to file.cpp relative to
> revision
> > > > [10850]
> > > > I have also attached a modification to viewport.cpp which adds
> the
> > > > ommited malloc debug section that I needed track down the source
> > of
> > > > the leak.
> > > >
> > > > Chris Denham.
> > > >
> > > > ------------------
> > > > Read this topic online here:
> > > > http://forum.openscenegraph.org/viewtopic.php?p=30193#30193
> > > >
> > > >
> > > >
> > > >
> > > > Attachments:
> > > > http://forum.openscenegraph.org//files/gsk0013ds_693.zip
> > > > http://forum.openscenegraph.org//files/viewport_585.cpp
> > > > http://forum.openscenegraph.org//files/file_150.cpp
> > > >
> > > >
> > > > _______________________________________________
> > > > osg-submissions mailing list
> > > >
> > > >
> >
> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
> > > >
> > > _______________________________________________
> > > osg-submissions mailing list
> > >
> > >
> >
> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
> > >
> > >  ------------------
> > > Post generated by Mail2Forum
> >
> >
> > ------------------
> > Read this topic online here:
> > http://forum.openscenegraph.org/viewtopic.php?p=30272#30272
> >
> >
> >
> >
> >
> > _______________________________________________
> > 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
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to