From a170b06d6d29e91d7792f3da115198b143116c66 Mon Sep 17 00:00:00 2001
From: Don BURNS <don@andesengineering.com>
Date: Sun, 14 Oct 2001 05:33:36 +0000
Subject: [PATCH] Mr. O added fancy, shmanzy C++ code, but broke normal
 transform functionality Fixed.

---
 src/Demos/osgconv/orientationconverter.cpp | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/Demos/osgconv/orientationconverter.cpp b/src/Demos/osgconv/orientationconverter.cpp
index ca1ee7ad6..52bcee740 100644
--- a/src/Demos/osgconv/orientationconverter.cpp
+++ b/src/Demos/osgconv/orientationconverter.cpp
@@ -32,7 +32,15 @@ class TransformFunctor : public osg::Drawable::AttributeFunctor
                 for (osg::Vec3* itr=begin;itr<end;++itr)
                 {
                     // note post mult rather than pre mult of value.
-                    (*itr) = osg::Matrix::transform3x3(_m,(*itr));
+
+		    // Yep,  I noted that post mult doesn't work.
+		    // All the normals are upside down or facing the 
+		    // wrong way
+		    // -don
+                    //(*itr) = osg::Matrix::transform3x3(_m,(*itr));
+                    //(*itr).normalize();
+
+		    (*itr) = (*itr) * _m;
                     (*itr).normalize();
                 }
                 return true;
@@ -72,7 +80,6 @@ void OrientationConverter::ConvertVisitor::apply( Geode &geode )
    
    TransformFunctor tf(_mat);
 
-   // We assume all Drawables are GeoSets ?!!?
    for( int i = 0; i < numdrawables; i++ )
    {
         geode.getDrawable(i)->applyAttributeOperation(tf);