On 5/11/2011 11:20 AM, Ryan Pavlik wrote:


On Tue, May 10, 2011 at 8:24 PM, Paul Martz <[email protected]
<mailto:[email protected]>> wrote:

    On 5/10/2011 7:11 PM, Paul Martz wrote:

        On 5/3/2011 3:02 PM, Ryan Pavlik wrote:

            I'd like to propose my fixes that I sent in on -submissions
            pre-2.8.4: all
            except the VC2010 build fix still apply. They are these two:

            Improvements to osgconv:
            
https://github.com/rpavlik/osg/compare/OpenSceneGraph-2.8...improve-osgconv

            (I've also forward-ported these to trunk:
            
https://github.com/rpavlik/osg/compare/master...improve-osgconv-trunk )


        So, if I understand correctly, you did not post this change to
        osg-submissions
        for inclusion in OSG's svn trunk, is that correct? Or, you did post
        them, but
        they are awaiting action by Robert?

        Has anyone other than you tested the changes?

        What, exactly, are these improvements to osgconv? (No, I haven't looked
        at the
        code -- yet.)


    Having reviewed the code, I see two changes:

    1) A simple change to an error message in osgconv.cpp. Looks 
straightforward.

    2) A change to the image file name format as output by the dot OSG plugin.
    Not sure I understand this one, but it doesn't appear to be an "improvement
    to osgconv". Did you include this by mistake?

       -Paul


No, the .osg plugin modification essentially only affects use of osgconv.  What
it does is, if  the OutputTextureFiles option is turned on (such as by passing
-O OutputTextureFiles to osgconv),

...or anytime "OutputTextureFiles" is present in the Options string. This functionality is orthogonal to osgconv. Any application can access it anytime they write a dot OSG file. This change will affect more than osgconv.
   -Paul
_______________________________________________
osg-users mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org

Reply via email to