Hi,

On Tue, 2009-02-24 at 18:03 -0600, Patrick Hartling wrote:
> In an effort to help clarify these additions, I have attached two files that 
> show how the code looks when merged to the trunk:
> 
>     double_transform.patch: Modifications to existing files
>     double_transform_files.tar.bz2: New files to be added
> 
> I am still working on testing OSG::DoubleTransform on the trunk, but so far, 
> things are looking good.
> 

looks ok to me, the only very minor comment would be to switch to

#ifndef OSG_ENABLE_DOUBLE_MATRIX_STACK
     typedef std::pair<UInt32, Matrix>              MatrixStore;
#else
    typedef std::pair<UInt32, Matrix4d>             DoubleMatrixStore;
#endif
 
in RenderPartition instead of having two MatrixStore / MatrixStack defs.



kind regards,
  gerrit



------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA
-OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise
-Strategies to boost innovation and cut costs with open source participation
-Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
_______________________________________________
Opensg-core mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensg-core

Reply via email to