background scene file loading introduces race condition with datamaps
[laserbrain_demo] / src / metascene.cc
index 65c9d66..21e9093 100644 (file)
@@ -1,3 +1,4 @@
+#include <string>
 #include <regex>
 #include "metascene.h"
 #include "scene.h"
 #include <alloca.h>
 #endif
 
+struct MaterialEdit {
+       std::regex name_re;
+       int attr;
+       Texture *tex;
+};
+
 static bool proc_node(MetaScene *mscn, struct ts_node *node);
 static bool proc_scenefile(MetaScene *mscn, struct ts_node *node);
-static bool proc_mtledit(Scene *scn, struct ts_node *node);
+static bool proc_mtledit(MaterialEdit *med, struct ts_node *node, TextureSet *texset);
+static void apply_mtledit(Scene *scn, const MaterialEdit &med);
+static void apply_mtledit(Material *mtl, const MaterialEdit &med);
 static struct ts_attr *attr_inscope(struct ts_node *node, const char *name);
 
 static void print_scene_graph(SceneNode *n, int level);
@@ -92,10 +101,19 @@ static bool proc_node(MetaScene *mscn, struct ts_node *node)
        return true;
 }
 
+
+
+struct SceneData {
+       std::string walkmesh_regexp;
+       std::vector<MaterialEdit> mtledit;
+};
+
 static bool proc_scenefile(MetaScene *mscn, struct ts_node *node)
 {
        const char *fname = ts_get_attr_str(node, "file");
        if(fname) {
+               SceneData *sdat = new SceneData;
+
                // datapath
                struct ts_attr *adpath = attr_inscope(node, "datapath");
                if(adpath && adpath->val.type == TS_STRING) {
@@ -104,10 +122,9 @@ static bool proc_scenefile(MetaScene *mscn, struct ts_node *node)
                }
 
                // walkmesh
-               char *walkmesh_regexp = 0;
                struct ts_attr *awmesh = attr_inscope(node, "walkmesh");
                if(awmesh && awmesh->val.type == TS_STRING) {
-                       walkmesh_regexp = awmesh->val.str;
+                       sdat->walkmesh_regexp = std::string(awmesh->val.str);
                }
 
                int namesz = datamap_lookup(fname, 0, 0);
@@ -116,48 +133,80 @@ static bool proc_scenefile(MetaScene *mscn, struct ts_node *node)
                        fname = namebuf;
                }
 
-               Scene *newscn = new Scene(mscn->texman);
-               if(!(newscn->load(fname, SCNLOAD_FLIPTEX))) {   // TODO unhardcode FLIPTEX
-                       return false;
+               // material edits
+               struct ts_node *child = node->child_list;
+               while(child) {
+                       MaterialEdit medit;
+                       if(proc_mtledit(&medit, child, mscn->texman)) {
+                               sdat->mtledit.push_back(medit);
+                       }
+                       child = child->next;
                }
 
-               // extract the walk mesh if necessary
-               Scene *wscn;
-               if(walkmesh_regexp && (wscn = newscn->extract_nodes(walkmesh_regexp))) {
-                       // apply all transformations to the meshes
-                       wscn->apply_xform();
+               Scene *newscn = mscn->sceneman->get(fname);
+               /* NOTE: setting this after get() is not a race condition, because
+                * scene_loaded() which uses this, will only run in our main loop during
+                * SceneSet::update() on the main thread.
+                */
+               newscn->metascn = mscn;
+               mscn->scndata[newscn] = sdat;
+       }
+       return true;
+}
+
+bool MetaScene::scene_loaded(Scene *newscn)
+{
+       SceneData *sdat = (SceneData*)scndata[newscn];
+       if(!sdat) {
+               error_log("MetaScene::scene_loaded called, but corresponding SceneData not found\n");
+               return false;
+       }
 
-                       int nmeshes = wscn->meshes.size();
-                       for(int i=0; i<nmeshes; i++) {
-                               Mesh *m = wscn->meshes[i];
+       // extract the walk mesh if necessary
+       Scene *wscn;
+       if(!sdat->walkmesh_regexp.empty() && (wscn = newscn->extract_nodes(sdat->walkmesh_regexp.c_str()))) {
+               // apply all transformations to the meshes
+               wscn->apply_xform();
 
-                               if(mscn->walk_mesh) {
-                                       mscn->walk_mesh->append(*m);
-                               } else {
-                                       mscn->walk_mesh = m;
-                                       wscn->remove_mesh(m);   // to save it from destruction
-                               }
-                       }
+               int nmeshes = wscn->meshes.size();
+               for(int i=0; i<nmeshes; i++) {
+                       Mesh *m = wscn->meshes[i];
 
-                       delete wscn;
+                       if(walk_mesh) {
+                               walk_mesh->append(*m);
+                       } else {
+                               walk_mesh = m;
+                               wscn->remove_mesh(m);   // to save it from destruction
+                       }
                }
 
-               // perform material edits
-               struct ts_node *child = node->child_list;
-               while(child) {
-                       proc_mtledit(newscn, child); // ignores any non-mtledit nodes, no need to check
-                       child = child->next;
-               }
+               delete wscn;
+       }
 
-               mscn->scenes.push_back(newscn);
+       int num_medits = sdat->mtledit.size();
+       for(int i=0; i<num_medits; i++) {
+               // perform material edits
+               apply_mtledit(newscn, sdat->mtledit[i]);
        }
-       //datamap_reset();      // TODO this should be push/pop instead of hard reset
 
+       scenes.push_back(newscn);
        return true;
 }
 
-static void mtledit(Material *mtl, TextureSet *texset, struct ts_node *node)
+static bool proc_mtledit(MaterialEdit *med, struct ts_node *node, TextureSet *texset)
 {
+       if(strcmp(node->name, "mtledit") != 0) {
+               return false;
+       }
+
+       const char *restr = ".*";
+       struct ts_attr *amtl = ts_get_attr(node, "material");
+       if(amtl && amtl->val.type == TS_STRING) {
+               restr = amtl->val.str;
+       }
+
+       med->name_re = std::regex(restr);
+
        node = node->child_list;
        while(node) {
                struct ts_node *cn = node;
@@ -184,42 +233,44 @@ static void mtledit(Material *mtl, TextureSet *texset, struct ts_node *node)
                                }
                        }
 
+                       med->attr = textype;
+
                        if(!afile || !afile->val.str || !*afile->val.str) {
                                // remove
-                               mtl->textures[textype] = 0;
+                               med->tex = 0;
                        } else {
-                               Texture *tex = texset->get_texture(afile->val.str, TEX_2D);
-                               mtl->add_texture(tex, textype);
+                               med->tex = texset->get_texture(afile->val.str, TEX_2D);
                        }
                }
                // TODO add more edit modes
        }
+
+       return true;
 }
 
-static bool proc_mtledit(Scene *scn, struct ts_node *node)
+static void apply_mtledit(Scene *scn, const MaterialEdit &med)
 {
-       if(strcmp(node->name, "mtledit") != 0) {
-               return false;
-       }
-
-       const char *mtl_regexp = ".*";
-       struct ts_attr *amtl = ts_get_attr(node, "material");
-       if(amtl && amtl->val.type == TS_STRING) {
-               mtl_regexp = amtl->val.str;
-       }
-
        // search all the objects to find matching material names
-       std::regex re{mtl_regexp};
-
        int nobj = scn->objects.size();
        for(int i=0; i<nobj; i++) {
                Object *obj = scn->objects[i];
-               if(std::regex_match(obj->get_name(), re)) {
-                       mtledit(&obj->mtl, scn->texset, node);
+               if(std::regex_match(obj->get_name(), med.name_re)) {
+                       apply_mtledit(&obj->mtl, med);
                }
        }
+}
 
-       return true;
+static void apply_mtledit(Material *mtl, const MaterialEdit &med)
+{
+       // TODO more edit modes...
+       if(med.tex) {
+               mtl->add_texture(med.tex, med.attr);
+       } else {
+               Texture *tex = mtl->stdtex[med.attr];
+               if(tex) {
+                       mtl->remove_texture(tex);
+               }
+       }
 }
 
 static struct ts_attr *attr_inscope(struct ts_node *node, const char *name)