Merge remote-tracking branch 'chris/nif-fixes'

components/nif/data.hpp

@@ -142,14 +142,12 @@ public:
     {
         ShapeData::read(nif);
 
-        int tris = nif->getUShort();
-        if(tris)
-        {
+        /*int tris =*/ nif->getUShort();
+
         // We have three times as many vertices as triangles, so this
         // is always equal to tris*3.
         int cnt = nif->getInt();
         nif->getShorts(triangles, cnt);
-        }
 
         // Read the match list, which lists the vertices that are equal to
         // vertices. We don't actually need need this for anything, so

components/nif/nif_file.cpp

@@ -220,11 +220,10 @@ void NIFFile::parse()
 
   for(size_t i = 0;i < recNum;i++)
     {
-      std::string rec = getString();
-      //cout << i << ": " << rec.toString() << endl;
-
       Record *r = NULL;
 
+      std::string rec = getString();
+
       /* These are all the record types we know how to read.
 
          This can be heavily optimized later if needed. For example, a

components/nifogre/ogre_nif_loader.cpp

@@ -844,9 +844,12 @@ class NIFMeshLoader : Ogre::ManualResourceLoader
         // Set the bounding box first
         BoundsFinder bounds;
         bounds.add(&srcVerts[0][0], srcVerts.size());
-        // No idea why this offset is needed. It works fine without it if the
-        // vertices weren't transformed first, but otherwise it fails later on
-        // when the object is being inserted into the scene.
+        if(!bounds.isValid())
+        {
+            float v[3] = { 0.0f, 0.0f, 0.0f };
+            bounds.add(&v[0], 1);
+        }
+
         mesh->_setBounds(Ogre::AxisAlignedBox(bounds.minX()-0.5f, bounds.minY()-0.5f, bounds.minZ()-0.5f,
                                               bounds.maxX()+0.5f, bounds.maxY()+0.5f, bounds.maxZ()+0.5f));
         mesh->_setBoundingSphereRadius(bounds.getRadius());