From ebc49de85117bca4137b45cf95fe4a9e9a98d2fd Mon Sep 17 00:00:00 2001
From: Roman Melnik <kromgart@gmail.com>
Date: Thu, 22 Mar 2012 00:39:19 +0200
Subject: [PATCH] Fix Bug #1: "Meshes rendered with wrong orientation"

Discard the tranformation of the root NiNode when loading nif files
(set the 'identity' transformation instead).

After applying the fix test the following types of interiors:

 - Dwemer ruins (the bug was common here before the fix)

 - Ancestral tombs (the bug was common here before the fix)

 - Daedric shrines

 - Caves/grottos

 - Dunmer strongholds

 - Telvanni, imperial, redoran houses/towers/castles

Also checked exteriors (although it is hard to do without terrain
rendering)
---
 components/nif/nif_file.cpp  |  9 +++++++++
 components/nif/nif_types.hpp | 15 +++++++++++++++
 2 files changed, 24 insertions(+)

diff --git a/components/nif/nif_file.cpp b/components/nif/nif_file.cpp
index 52a37ba5c..80ea7a0b7 100644
--- a/components/nif/nif_file.cpp
+++ b/components/nif/nif_file.cpp
@@ -162,6 +162,15 @@ void NIFFile::parse()
       r->recName = rec;
       records[i] = r;
       r->read(this);
+
+      // Discard tranformations for the root node, otherwise some meshes
+      // occasionally get wrong orientation. Only for NiNode-s for now, but
+      // can be expanded if needed.
+      // This should be rewritten when the method is cleaned up.
+      if (0 == i && rec == "NiNode")
+      {
+          static_cast<Nif::Node*>(r)->trafo = Nif::Transformation::getIdentity();
+      }
     }
 
   /* After the data, the nif contains an int N and then a list of N
diff --git a/components/nif/nif_types.hpp b/components/nif/nif_types.hpp
index 17cec19bc..ce7a0de62 100644
--- a/components/nif/nif_types.hpp
+++ b/components/nif/nif_types.hpp
@@ -59,6 +59,21 @@ struct Transformation
   Matrix rotation;
   float scale;
   Vector velocity;
+
+  static const Transformation* getIdentity()
+  {
+	  static Transformation* identity = NULL;
+	  if (NULL == identity)
+      {
+        identity = new Transformation();
+        identity->scale = 1.0f;
+        identity->rotation.v[0].array[0] = 1.0f;
+        identity->rotation.v[1].array[1] = 1.0f;
+        identity->rotation.v[2].array[2] = 1.0f;
+      }
+
+	  return identity;
+  }
 };
 #pragma pack(pop)