Remove collection-less constructor overloads from most loaders

[libs/gl.git] / source / materials / material.cpp

diff --git a/source/materials/material.cpp b/source/materials/material.cpp
index 810ecd174d81fb407730b33b0488b456dcee4f86..71610c7cd9174957e0bf650f5f50631c2812ddb7 100644 (file)
--- a/source/materials/material.cpp
+++ b/source/materials/material.cpp
@@ -1,34 +1,40 @@
 #include <msp/core/hash.h>
 #include <msp/strings/format.h>
 #include "basicmaterial.h"
-#include "gl.h"
 #include "pbrmaterial.h"
+#include "program.h"
 #include "resources.h"
-#include "texturing.h"
-#include "uniform.h"
+#include "unlitmaterial.h"
 
 using namespace std;
 
 namespace Msp {
 namespace GL {
 
-Program *Material::create_compatible_shader() const
+const Program *Material::create_compatible_shader(const map<string, int> &extra_spec) const
 {
-       return new Program(create_program_source());
-}
+       string module_name;
+       map<string, int> spec_values;
+       fill_program_info(module_name, spec_values);
 
-const Program *Material::create_compatible_shader(DataFile::Collection &coll) const
-{
-       string source = create_program_source();
-       string name = format("_material_%016x.glsl", hash64(source));
-       Program *shprog = coll.find<Program>(name);
+       for(const auto &kvp: extra_spec)
+               spec_values[kvp.first] = kvp.second;
+
+       string info = module_name;
+       for(const auto &kvp: spec_values)
+               info += format(",%s:%d", kvp.first, kvp.second);
+
+       Resources &res = Resources::get_global();
+       string name = format("_material_%016x.shader", hash64(info));
+       Program *shprog = res.find<Program>(name);
        if(shprog)
                return shprog;
 
-       shprog = new Program(create_program_source());
+       const Module &module = res.get<Module>(module_name);
+       shprog = new Program(module, spec_values);
        try
        {
-               coll.add(name, shprog);
+               res.add(name, shprog);
        }
        catch(...)
        {
@@ -39,24 +45,13 @@ const Program *Material::create_compatible_shader(DataFile::Collection &coll) co
        return shprog;
 }
 
-void Material::attach_texture_to(const Texture *tex, Texturing &texturing, ProgramData &tex_shdata, const string &name) const
+void Material::set_debug_name(const string &name)
 {
-       if(!tex)
-               return;
-
-       int unit = -1;
-
-       if(const Uniform *uni = tex_shdata.find_uniform(name))
-               if(const Uniform1i *uni_int = dynamic_cast<const Uniform1i *>(uni))
-                       unit = uni_int->get();
-
-       if(unit<0)
-               unit = texturing.find_free_unit(name);
-       if(unit<0)
-               throw runtime_error("no free texunit");
-
-       texturing.attach(unit, *tex, sampler);
-       tex_shdata.uniform(name, unit);
+#ifdef DEBUG
+       shdata.set_debug_name(name+" [UBO]");
+#else
+       (void)name;
+#endif
 }
 
 Material::MaterialRegistry &Material::get_material_registry()
@@ -68,15 +63,12 @@ Material::MaterialRegistry &Material::get_material_registry()
                initialized = true;
                registry.register_type<BasicMaterial>("basic");
                registry.register_type<PbrMaterial>("pbr");
+               registry.register_type<UnlitMaterial>("unlit");
        }
        return registry;
 }
 
 
-Material::Loader::Loader(Material &m):
-       CollectionObjectLoader(m, 0)
-{ }
-
 Material::Loader::Loader(Material &m, Collection &c):
        CollectionObjectLoader(m, &c)
 { }
@@ -86,7 +78,7 @@ void Material::Loader::init_actions()
        add("sampler", &Loader::sampler);
 }
 
-void Material::Loader::sampler(const std::string &name)
+void Material::Loader::sampler(const string &name)
 {
        obj.sampler = &get_collection().get<Sampler>(name);
 }
@@ -94,15 +86,28 @@ void Material::Loader::sampler(const std::string &name)
 
 DataFile::Loader::ActionMap Material::GenericLoader::shared_actions;
 
-Material::GenericLoader::GenericLoader(DataFile::Collection *c):
-       coll(c)
+Material::GenericLoader::GenericLoader(DataFile::Collection &c):
+       coll(c),
+       material(0),
+       mat_loader(0)
 {
        set_actions(shared_actions);
 }
 
+Material::GenericLoader::~GenericLoader()
+{
+       delete material;
+       delete mat_loader;
+}
+
 void Material::GenericLoader::init_actions()
 {
-       get_material_registry().add_all(*this);
+       add("type", &GenericLoader::type);
+}
+
+void Material::GenericLoader::type(const DataFile::Symbol &sym)
+{
+       get_material_registry().invoke(sym.name, *this);
 }
 
 } // namespace GL