Hi Aurelien, I have now implemented and checked in the new osg::PatchParameter class. I have implement the implemented the GL_PATCH_VERTICES, GL_PATCH_DEFAULT_INNER_LEVEL and GL_PATCH_DEFAULT_OUTER_LEVEL support, along with associated serializer. The previous .osg and .osgb/.osgt/.osgx serializers didn't support Program::setParameter(GL_PATCH_VERTICES,..) so there hasn't been any need to jump through hoops with keeping backwards compatibility on the IO front.
Having implemented the new class, and read up on the associated OpenGL docs for glPatchParameter I believe the separate osg::StateAttribute is the way to go, it feels very much a natural fit to how OpenGL manages this state and how the OSG has traditional implemented such state - what was out of sync was the jamming this functionality into osg::Program where it never really belonged. I have added an OSG_WARN notification if one attempts an osg::Program::setParameter(GL_PATCH_VERTICES,..) including a mention that one should now use osg::PatchParameters. Could you try out the new class and let me know how you get on, Robert. _______________________________________________ osg-submissions mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
