Use a struct to store texture slots in Technique

[libs/gl.git] / source / technique.cpp

diff --git a/source/technique.cpp b/source/technique.cpp
index 74cd6c311be0699e10760922452c702507458e58..53c7a9ae76cacdc3ba68b05bd54db5000de452e2 100644 (file)
--- a/source/technique.cpp
+++ b/source/technique.cpp
@@ -45,8 +45,8 @@ const ObjectPass &Technique::get_pass(const GL::Tag &tag) const
 
 unsigned Technique::get_texture_index(const std::string &n) const
 {
-       for(unsigned i=0; i<tex_names.size(); ++i)
-               if(tex_names[i]==n)
+       for(unsigned i=0; i<textures.size(); ++i)
+               if(textures[i].name==n)
                        return i;
 
        throw KeyError("Unknown texture slot", n);
@@ -57,7 +57,7 @@ const Texture *Technique::get_texture(unsigned i) const
        if(i>=textures.size())
                throw KeyError("Texture index out of range");
 
-       return textures[i];
+       return textures[i].texture;
 }
 
 
@@ -80,7 +80,10 @@ void Technique::Loader::finish()
                if(i->second.shdata)
                {
                        for(unsigned j=0; j<tech.textures.size(); ++j)
-                               i->second.shdata->uniform(i->second.shprog->get_uniform_location(tech.tex_names[j]), static_cast<int>(j));
+                       {
+                               unsigned loc=i->second.shprog->get_uniform_location(tech.textures[j].name);
+                               i->second.shdata->uniform(loc, static_cast<int>(j));
+                       }
                }
 }
 
@@ -120,16 +123,22 @@ void Technique::Loader::shader(const string &n)
 void Technique::Loader::shader_texture(const string &n)
 {
        string::size_type eqsign=n.find('=');
+       TextureSlot tex;
        if(eqsign!=string::npos)
        {
-               tech.textures.push_back(coll.get<Texture>(n.substr(eqsign+1)));
-               tech.tex_names.push_back(n.substr(0, eqsign));
+               tex.name=n.substr(0, eqsign);
+               tex.texture=coll.get<Texture>(n.substr(eqsign+1));
        }
        else
        {
-               tech.textures.push_back(coll.get<Texture>(n));
-               tech.tex_names.push_back(n);
+               string::size_type dot=n.rfind('.');
+               tex.name=n.substr(0, dot);
+               tex.texture = coll.get<Texture>(n);
        }
+       for(string::iterator i=tex.name.begin(); i!=tex.name.end(); ++i)
+               if(!isalnum(*i))
+                       *i='_';
+       tech.textures.push_back(tex);
 }
 
 void Technique::Loader::texture(const string &n)
@@ -138,14 +147,17 @@ void Technique::Loader::texture(const string &n)
                throw Exception("Only one main texture may be specified");
 
        tech.main_texture=coll.get<Texture>(n);
-       tech.textures.push_back(tech.main_texture);
-       tech.tex_names.push_back("texture");
+       TextureSlot tex;
+       tex.name="texture";
+       tex.texture=tech.main_texture;
+       tech.textures.push_back(tex);
 }
 
 void Technique::Loader::texture_slot(const string &n)
 {
-       tech.tex_names.push_back(n);
-       tech.textures.push_back(0);
+       TextureSlot tex;
+       tex.name=n;
+       tech.textures.push_back(tex);
 }
 
 } // namespace GL