Make insertBegin internal

apps/openmw/mwclass/apparatus.cpp

@@ -25,9 +25,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/armor.cpp

@@ -30,9 +30,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/book.cpp

@@ -23,9 +23,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/clothing.cpp

@@ -27,9 +27,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/container.cpp

@@ -61,9 +61,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/door.cpp

@@ -29,9 +29,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/ingredient.cpp

@@ -36,9 +36,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/light.cpp

@@ -26,18 +26,10 @@ namespace MWClass
 {
     void Light::insertObjectRendering (const MWWorld::Ptr& ptr, MWRender::RenderingInterface& renderingInterface) const
     {
-        MWWorld::LiveCellRef<ESM::Light> *ref =
+        const std::string model = getModel(ptr);
-            ptr.get<ESM::Light>();
+        if(!model.empty()) {
-        assert (ref->mBase != NULL);
+            renderingInterface.getObjects().insertModel(ptr, model);
-
+        }
-        const std::string &model = ref->mBase->mModel;
-
-        MWRender::Objects& objects = renderingInterface.getObjects();
-        objects.insertBegin(ptr);
-        if(!model.empty())
-            objects.insertMesh(ptr, "meshes\\" + model);
-        else
-            objects.insertMesh(ptr, "");
     }
 
     void Light::insertObject(const MWWorld::Ptr& ptr, MWWorld::PhysicsSystem& physics) const

apps/openmw/mwclass/lockpick.cpp

@@ -26,9 +26,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/misc.cpp

@@ -42,9 +42,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/potion.cpp

@@ -28,9 +28,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/probe.cpp

@@ -26,9 +26,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/repair.cpp

@@ -25,9 +25,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/static.cpp

@@ -15,9 +15,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwclass/weapon.cpp

@@ -33,9 +33,7 @@ namespace MWClass
     {
         const std::string model = getModel(ptr);
         if (!model.empty()) {
-            MWRender::Objects& objects = renderingInterface.getObjects();
+            renderingInterface.getObjects().insertModel(ptr, model);
-            objects.insertBegin(ptr);
-            objects.insertMesh(ptr, model);
         }
     }
 

apps/openmw/mwrender/actors.hpp

@@ -27,6 +27,8 @@ namespace MWRender
         CellSceneNodeMap mCellSceneNodes;
         PtrAnimationMap mAllActors;
 
+        void insertBegin(const MWWorld::Ptr &ptr);
+
     public:
         Actors(OEngine::Render::OgreRenderer& _rend, MWRender::RenderingManager* rendering)
             : mRend(_rend)
@@ -36,7 +38,7 @@ namespace MWRender
         ~Actors();
 
         void setRootNode(Ogre::SceneNode* root);
-        void insertBegin (const MWWorld::Ptr& ptr);
+
         void insertNPC(const MWWorld::Ptr& ptr, MWWorld::InventoryStore& inv);
         void insertCreature (const MWWorld::Ptr& ptr);
         void insertActivator (const MWWorld::Ptr& ptr);

apps/openmw/mwrender/objects.cpp

@@ -69,16 +69,15 @@ void Objects::insertBegin (const MWWorld::Ptr& ptr)
     ptr.getRefData().setBaseNode(insert);
 }
 
-void Objects::insertMesh (const MWWorld::Ptr& ptr, const std::string& mesh)
+void Objects::insertModel(const MWWorld::Ptr &ptr, const std::string &mesh)
 {
-    Ogre::SceneNode* insert = ptr.getRefData().getBaseNode();
+    insertBegin(ptr);
-    assert(insert);
 
     std::auto_ptr<ObjectAnimation> anim(new ObjectAnimation(ptr, mesh));
 
     Ogre::AxisAlignedBox bounds = anim->getWorldBounds();
     Ogre::Vector3 extents = bounds.getSize();
-    extents *= insert->getScale();
+    extents *= ptr.getRefData().getBaseNode()->getScale();
     float size = std::max(std::max(extents.x, extents.y), extents.z);
 
     bool small = (size < Settings::Manager::getInt("small object size", "Viewing distance")) &&

apps/openmw/mwrender/objects.hpp

@@ -31,14 +31,15 @@ class Objects{
 
     static int uniqueID;
 
+    void insertBegin(const MWWorld::Ptr& ptr);
+
 public:
     Objects(OEngine::Render::OgreRenderer &renderer)
         : mRenderer(renderer)
         , mRootNode(NULL)
     {}
     ~Objects(){}
-    void insertBegin (const MWWorld::Ptr& ptr);
+    void insertModel(const MWWorld::Ptr& ptr, const std::string &model);
-    void insertMesh (const MWWorld::Ptr& ptr, const std::string& mesh);
 
     void enableLights();
     void disableLights();