X-Git-Url: http://git.tdb.fi/?p=libs%2Fgl.git;a=blobdiff_plain;f=source%2Fmaterials%2Fmaterial.h;h=98baaa54ff2faf41b7975427dc4412306ac2e906;hp=40d2b7ee96193627e148c2e4bf7189513cfbfc01;hb=38712d8ecc57d043a2419ffbaeeb57f7a6586f14;hpb=6065f6622cc275dc0b20baaf7c267e71169d18f3

diff --git a/source/materials/material.h b/source/materials/material.h
index 40d2b7ee..98baaa54 100644
--- a/source/materials/material.h
+++ b/source/materials/material.h
@@ -1,8 +1,8 @@
 #ifndef MSP_GL_MATERIAL_H_
 #define MSP_GL_MATERIAL_H_
 
-#include <msp/core/typeregistry.h>
 #include <msp/datafile/collection.h>
+#include <msp/datafile/dynamicobjectloader.h>
 #include <msp/datafile/objectloader.h>
 #include "color.h"
 #include "programdata.h"
@@ -19,7 +19,6 @@ private:
 	class Loader: public DataFile::CollectionObjectLoader<Material>
 	{
 	protected:
-		Loader(Material &);
 		Loader(Material &, Collection &);
 
 		virtual void init_actions();
@@ -42,7 +41,6 @@ protected:
 	class PropertyLoader: public DataFile::DerivedObjectLoader<Material, Loader>
 	{
 	protected:
-		PropertyLoader(C &m): DerivedObjectLoader<Material, Loader>(m) { }
 		PropertyLoader(C &m, Collection &c): DerivedObjectLoader<Material, Loader>(m, c) { }
 
 		void add_property(const std::string &, void (C::*)(float), void (C::*)(const Texture *));
@@ -58,44 +56,24 @@ protected:
 	};
 
 public:
-	class GenericLoader: public DataFile::Loader
+	class GenericLoader: public DataFile::DynamicObjectLoader<Material>
 	{
-	private:
-		template<typename T>
-		struct CreateMaterial
-		{
-			void operator()(const std::string &, GenericLoader &) const;
-		};
-
-		DataFile::Collection *coll;
-		Material *material;
-		Loader *mat_loader;
-
-		static ActionMap shared_actions;
+		friend class Material;
 
 	public:
-		GenericLoader(DataFile::Collection * = 0);
-		~GenericLoader();
+		GenericLoader(Collection &c): DynamicObjectLoader<Material>(&c) { }
 
-		Material *get_material() { Material *m = material; material = 0; return m; }
-	private:
-		virtual void init_actions();
-
-		void type(const DataFile::Symbol &);
-
-		friend class Material;
+	protected:
+		virtual const TypeRegistry &get_type_registry() const { return get_material_registry(); }
 	};
 
-private:
-	typedef TypeRegistry<GenericLoader::CreateMaterial, GenericLoader &> MaterialRegistry;
-
 protected:
-	const Sampler *sampler;
+	const Sampler *sampler = 0;
 	ProgramData shdata;
 
-	Material(): sampler(0) { }
+	Material() = default;
 public:
-	virtual ~Material() { }
+	virtual ~Material() = default;
 
 	virtual const Program *create_compatible_shader(const std::map<std::string, int> & = std::map<std::string, int>()) const;
 protected:
@@ -114,7 +92,7 @@ public:
 	template<typename T>
 	static void register_type(const std::string &);
 private:
-	static MaterialRegistry &get_material_registry();
+	static GenericLoader::TypeRegistry &get_material_registry();
 };
 
 template<typename T>
@@ -190,22 +168,6 @@ void Material::PropertyLoader<C>::property_texture(void (C::*set_texture)(const
 	(static_cast<C &>(obj).*set_texture)(&static_cast<Collection &>(get_collection()).get<Texture>(name));
 }
 
-
-template<typename T>
-void Material::GenericLoader::CreateMaterial<T>::operator()(const std::string &, GenericLoader &ldr) const
-{
-	if(ldr.material)
-		throw std::logic_error("Material type was already specified");
-
-	T *mat = new T;
-	ldr.material = mat;
-	if(ldr.coll)
-		ldr.mat_loader = new typename T::Loader(*mat, *ldr.coll);
-	else
-		ldr.mat_loader = new typename T::Loader(*mat);
-	ldr.add_auxiliary_loader(*ldr.mat_loader);
-}
-
 } // namespace GL
 } // namespace Msp