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 2d7e448788e0d81612bb191c7b4733b793e2c6c1..71610c7cd9174957e0bf650f5f50631c2812ddb7 100644 (file)
--- a/source/materials/material.cpp
+++ b/source/materials/material.cpp
@@ -1,10 +1,9 @@
 #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 "uniform.h"
 #include "unlitmaterial.h"
 
 using namespace std;
@@ -18,12 +17,12 @@ const Program *Material::create_compatible_shader(const map<string, int> &extra_
        map<string, int> spec_values;
        fill_program_info(module_name, spec_values);
 
-       for(map<string, int>::const_iterator i=extra_spec.begin(); i!=extra_spec.end(); ++i)
-               spec_values[i->first] = i->second;
+       for(const auto &kvp: extra_spec)
+               spec_values[kvp.first] = kvp.second;
 
        string info = module_name;
-       for(map<string, int>::const_iterator i=spec_values.begin(); i!=spec_values.end(); ++i)
-               info += format(",%s:%d", i->first, i->second);
+       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));
@@ -70,10 +69,6 @@ Material::MaterialRegistry &Material::get_material_registry()
 }
 
 
-Material::Loader::Loader(Material &m):
-       CollectionObjectLoader(m, 0)
-{ }
-
 Material::Loader::Loader(Material &m, Collection &c):
        CollectionObjectLoader(m, &c)
 { }
@@ -91,7 +86,7 @@ void Material::Loader::sampler(const string &name)
 
 DataFile::Loader::ActionMap Material::GenericLoader::shared_actions;
 
-Material::GenericLoader::GenericLoader(DataFile::Collection *c):
+Material::GenericLoader::GenericLoader(DataFile::Collection &c):
        coll(c),
        material(0),
        mat_loader(0)