Hi Dan, Thanks for the changes. I originally didn't add a getAxisAlignment() as all setAxisAlignment does is set the Rotation member variable, and potentially one could apply user defined Rotation setting after the setAxisAlignment() which would bring it out of sync with the setAxisAlignment.
Rather than reject your submission on the ground of potentially getting out of sync and therefore misleading users I've added a USED_DEFINED_ROTATION to AxisAlignment enum, and set this in the serRotation and then override this setting of _axisAlignment in the setAxisAlingment method. I've also removed the lazy updating optimization you've added to the top of setAxisAlignment to avoid potential problems as well. These changes are now checked into SVN. Could you please checked them out. Cheers, Robert. On 9/28/07, Dan Minor <[EMAIL PROTECTED]> wrote: > osgText::Text does not currently have a getAxisAlignment method. The > attached code adds this, along with a member variable to keep track of > the setting. It is based on the latest subversion version, and was > tested by creating a new text object with the same axis alignment as an > existing one (e.g. > new_text->setAxisAlignment(old_text->getAxisAlignment()); ). > > Thanks, > > Daniel > > _______________________________________________ > 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
