Use DynamicObjectLoader for Material and Scene generic loaders

[libs/gl.git] / source / render / scene.cpp

diff --git a/source/render/scene.cpp b/source/render/scene.cpp
index cec2683abd2f0303b0fe90e1b6d3fc7db90e6127..75d1859bdd4995d8c42d8c0aa6f4b9ca747d7928 100644 (file)
--- a/source/render/scene.cpp
+++ b/source/render/scene.cpp
@@ -1,4 +1,6 @@
 #include <msp/datafile/collection.h>
+#include <msp/fs/utils.h>
+#include <msp/strings/format.h>
 #include "camera.h"
 #include "objectinstance.h"
 #include "occludedscene.h"
@@ -13,12 +15,6 @@ using namespace std;
 namespace Msp {
 namespace GL {
 
-Scene::~Scene()
-{
-       for(vector<Renderable *>::iterator i=owned_data.begin(); i!=owned_data.end(); ++i)
-               delete *i;
-}
-
 bool Scene::setup_frustum(const Renderer &renderer) const
 {
        const Camera *camera = renderer.get_camera();
@@ -77,9 +73,9 @@ bool Scene::frustum_cull(const Renderable &renderable) const
        return false;
 }
 
-Scene::SceneRegistry &Scene::get_scene_registry()
+Scene::GenericLoader::TypeRegistry &Scene::get_scene_registry()
 {
-       static SceneRegistry registry;
+       static Scene::GenericLoader::TypeRegistry registry;
        static bool initialized = false;
        if(!initialized)
        {
@@ -93,21 +89,10 @@ Scene::SceneRegistry &Scene::get_scene_registry()
 }
 
 
-Scene::Loader::Loader(Scene &s, Collection &c):
-       DataFile::CollectionObjectLoader<Scene>(s, &c),
-       content(0)
-{
-       init();
-}
-
-Scene::Loader::Loader(Scene &s, Collection &c, ContentMap &m):
+Scene::Loader::Loader(Scene &s, Collection &c, ContentMap *m):
        DataFile::CollectionObjectLoader<Scene>(s, &c),
-       content(&m)
-{
-       init();
-}
-
-void Scene::Loader::init()
+       content(m),
+       inst_counter(0)
 {
        add("object", &Loader::object);
        add("object", &Loader::object_tagged);
@@ -123,10 +108,10 @@ void Scene::Loader::object_tagged(const string &n, const string &t)
 {
        RefPtr<ObjectInstance> inst = new ObjectInstance(get_collection().get<GL::Object>(n));
        load_sub(*inst);
-       obj.add(*inst);
+       get_collection().add(format("%s/%d.inst", FS::basename(get_source()), inst_counter++), inst.get());
        if(content && !t.empty())
                (*content)[t] = inst.get();
-       obj.owned_data.push_back(inst.release());
+       obj.add(*inst.release());
 }
 
 void Scene::Loader::scene(const string &n)
@@ -134,32 +119,5 @@ void Scene::Loader::scene(const string &n)
        obj.add(get_collection().get<Scene>(n));
 }
 
-
-DataFile::Loader::ActionMap Scene::GenericLoader::shared_actions;
-
-Scene::GenericLoader::GenericLoader(DataFile::Collection &c):
-       coll(c),
-       scene(0),
-       scene_loader(0)
-{
-       set_actions(shared_actions);
-}
-
-Scene::GenericLoader::~GenericLoader()
-{
-       delete scene;
-       delete scene_loader;
-}
-
-void Scene::GenericLoader::init_actions()
-{
-       add("type", &GenericLoader::type);
-}
-
-void Scene::GenericLoader::type(const DataFile::Symbol &sym)
-{
-       get_scene_registry().invoke(sym.name, *this);
-}
-
 } // namespace GL
 } // namespace Msp