Cycles: Fix compilation error with latest OIIO
authorSergey Sharybin <sergey.vfx@gmail.com>
Fri, 10 Nov 2017 09:04:33 +0000 (10:04 +0100)
committerSergey Sharybin <sergey.vfx@gmail.com>
Fri, 10 Nov 2017 09:04:33 +0000 (10:04 +0100)
There was some changes about namespaces, which causes ambiguities.

Replaces using namespace with an explicit symbols we need. Is good idea to NOT
pull in the whole namespace anyway!

intern/cycles/app/cycles_xml.cpp
intern/cycles/graph/node_xml.cpp
intern/cycles/graph/node_xml.h
intern/cycles/util/util_xml.h

index 04f00ef0e10e07f349656aba5fa052d661cd488d..3a1c7205c34965682a23706068a802c7668dff5b 100644 (file)
@@ -70,9 +70,9 @@ struct XMLReadState : public XMLReader {
 
 /* Attribute Reading */
 
-static bool xml_read_int(int *value, pugi::xml_node node, const char *name)
+static bool xml_read_int(int *value, xml_node node, const char *name)
 {
-       pugi::xml_attribute attr = node.attribute(name);
+       xml_attribute attr = node.attribute(name);
 
        if(attr) {
                *value = atoi(attr.value());
@@ -82,9 +82,9 @@ static bool xml_read_int(int *value, pugi::xml_node node, const char *name)
        return false;
 }
 
-static bool xml_read_int_array(vector<int>& value, pugi::xml_node node, const char *name)
+static bool xml_read_int_array(vector<int>& value, xml_node node, const char *name)
 {
-       pugi::xml_attribute attr = node.attribute(name);
+       xml_attribute attr = node.attribute(name);
 
        if(attr) {
                vector<string> tokens;
@@ -99,9 +99,9 @@ static bool xml_read_int_array(vector<int>& value, pugi::xml_node node, const ch
        return false;
 }
 
-static bool xml_read_float(float *value, pugi::xml_node node, const char *name)
+static bool xml_read_float(float *value, xml_node node, const char *name)
 {
-       pugi::xml_attribute attr = node.attribute(name);
+       xml_attribute attr = node.attribute(name);
 
        if(attr) {
                *value = (float)atof(attr.value());
@@ -111,9 +111,9 @@ static bool xml_read_float(float *value, pugi::xml_node node, const char *name)
        return false;
 }
 
-static bool xml_read_float_array(vector<float>& value, pugi::xml_node node, const char *name)
+static bool xml_read_float_array(vector<float>& value, xml_node node, const char *name)
 {
-       pugi::xml_attribute attr = node.attribute(name);
+       xml_attribute attr = node.attribute(name);
 
        if(attr) {
                vector<string> tokens;
@@ -128,7 +128,7 @@ static bool xml_read_float_array(vector<float>& value, pugi::xml_node node, cons
        return false;
 }
 
-static bool xml_read_float3(float3 *value, pugi::xml_node node, const char *name)
+static bool xml_read_float3(float3 *value, xml_node node, const char *name)
 {
        vector<float> array;
 
@@ -140,7 +140,7 @@ static bool xml_read_float3(float3 *value, pugi::xml_node node, const char *name
        return false;
 }
 
-static bool xml_read_float3_array(vector<float3>& value, pugi::xml_node node, const char *name)
+static bool xml_read_float3_array(vector<float3>& value, xml_node node, const char *name)
 {
        vector<float> array;
 
@@ -154,7 +154,7 @@ static bool xml_read_float3_array(vector<float3>& value, pugi::xml_node node, co
        return false;
 }
 
-static bool xml_read_float4(float4 *value, pugi::xml_node node, const char *name)
+static bool xml_read_float4(float4 *value, xml_node node, const char *name)
 {
        vector<float> array;
 
@@ -166,9 +166,9 @@ static bool xml_read_float4(float4 *value, pugi::xml_node node, const char *name
        return false;
 }
 
-static bool xml_read_string(string *str, pugi::xml_node node, const char *name)
+static bool xml_read_string(string *str, xml_node node, const char *name)
 {
-       pugi::xml_attribute attr = node.attribute(name);
+       xml_attribute attr = node.attribute(name);
 
        if(attr) {
                *str = attr.value();
@@ -178,9 +178,9 @@ static bool xml_read_string(string *str, pugi::xml_node node, const char *name)
        return false;
 }
 
-static bool xml_equal_string(pugi::xml_node node, const char *name, const char *value)
+static bool xml_equal_string(xml_node node, const char *name, const char *value)
 {
-       pugi::xml_attribute attr = node.attribute(name);
+       xml_attribute attr = node.attribute(name);
 
        if(attr)
                return string_iequals(attr.value(), value);
@@ -190,7 +190,7 @@ static bool xml_equal_string(pugi::xml_node node, const char *name, const char *
 
 /* Camera */
 
-static void xml_read_camera(XMLReadState& state, pugi::xml_node node)
+static void xml_read_camera(XMLReadState& state, xml_node node)
 {
        Camera *cam = state.scene->camera;
 
@@ -210,7 +210,7 @@ static void xml_read_camera(XMLReadState& state, pugi::xml_node node)
 
 /* Shader */
 
-static void xml_read_shader_graph(XMLReadState& state, Shader *shader, pugi::xml_node graph_node)
+static void xml_read_shader_graph(XMLReadState& state, Shader *shader, xml_node graph_node)
 {
        xml_read_node(state, shader, graph_node);
 
@@ -220,7 +220,7 @@ static void xml_read_shader_graph(XMLReadState& state, Shader *shader, pugi::xml
        XMLReader graph_reader;
        graph_reader.node_map[ustring("output")] = graph->output();
 
-       for(pugi::xml_node node = graph_node.first_child(); node; node = node.next_sibling()) {
+       for(xml_node node = graph_node.first_child(); node; node = node.next_sibling()) {
                ustring node_name(node.name());
 
                if(node_name == "connect") {
@@ -349,7 +349,7 @@ static void xml_read_shader_graph(XMLReadState& state, Shader *shader, pugi::xml
        shader->tag_update(state.scene);
 }
 
-static void xml_read_shader(XMLReadState& state, pugi::xml_node node)
+static void xml_read_shader(XMLReadState& state, xml_node node)
 {
        Shader *shader = new Shader();
        xml_read_shader_graph(state, shader, node);
@@ -385,7 +385,7 @@ static Mesh *xml_add_mesh(Scene *scene, const Transform& tfm)
        return mesh;
 }
 
-static void xml_read_mesh(const XMLReadState& state, pugi::xml_node node)
+static void xml_read_mesh(const XMLReadState& state, xml_node node)
 {
        /* add mesh */
        Mesh *mesh = xml_add_mesh(state.scene, state.tfm);
@@ -531,7 +531,7 @@ static void xml_read_mesh(const XMLReadState& state, pugi::xml_node node)
 
 /* Light */
 
-static void xml_read_light(XMLReadState& state, pugi::xml_node node)
+static void xml_read_light(XMLReadState& state, xml_node node)
 {
        Light *light = new Light();
 
@@ -543,7 +543,7 @@ static void xml_read_light(XMLReadState& state, pugi::xml_node node)
 
 /* Transform */
 
-static void xml_read_transform(pugi::xml_node node, Transform& tfm)
+static void xml_read_transform(xml_node node, Transform& tfm)
 {
        if(node.attribute("matrix")) {
                vector<float> matrix;
@@ -572,7 +572,7 @@ static void xml_read_transform(pugi::xml_node node, Transform& tfm)
 
 /* State */
 
-static void xml_read_state(XMLReadState& state, pugi::xml_node node)
+static void xml_read_state(XMLReadState& state, xml_node node)
 {
        /* read shader */
        string shadername;
@@ -605,9 +605,9 @@ static void xml_read_state(XMLReadState& state, pugi::xml_node node)
 
 static void xml_read_include(XMLReadState& state, const string& src);
 
-static void xml_read_scene(XMLReadState& state, pugi::xml_node scene_node)
+static void xml_read_scene(XMLReadState& state, xml_node scene_node)
 {
-       for(pugi::xml_node node = scene_node.first_child(); node; node = node.next_sibling()) {
+       for(xml_node node = scene_node.first_child(); node; node = node.next_sibling()) {
                if(string_iequals(node.name(), "film")) {
                        xml_read_node(state, state.scene->film, node);
                }
@@ -657,8 +657,8 @@ static void xml_read_scene(XMLReadState& state, pugi::xml_node scene_node)
 static void xml_read_include(XMLReadState& state, const string& src)
 {
        /* open XML document */
-       pugi::xml_document doc;
-       pugi::xml_parse_result parse_result;
+       xml_document doc;
+       xml_parse_result parse_result;
 
        string path = path_join(state.base, src);
        parse_result = doc.load_file(path.c_str());
@@ -667,7 +667,7 @@ static void xml_read_include(XMLReadState& state, const string& src)
                XMLReadState substate = state;
                substate.base = path_dirname(path);
 
-               pugi::xml_node cycles = doc.child("cycles");
+               xml_node cycles = doc.child("cycles");
                xml_read_scene(substate, cycles);
        }
        else {
index aad2740ffc01e0e796f92c75341f22ef8c2ff64f..d26b3b2c2c885dc66984bb94e5dd5ad0e7b332be 100644 (file)
@@ -33,7 +33,7 @@ static const char *xml_write_boolean(bool value)
 }
 
 template<int VECTOR_SIZE, typename T>
-static void xml_read_float_array(T& value, pugi::xml_attribute attr)
+static void xml_read_float_array(T& value, xml_attribute attr)
 {
        vector<string> tokens;
        string_split(tokens, attr.value());
@@ -51,9 +51,9 @@ static void xml_read_float_array(T& value, pugi::xml_attribute attr)
        }
 }
 
-void xml_read_node(XMLReader& reader, Node *node, pugi::xml_node xml_node)
+void xml_read_node(XMLReader& reader, Node *node, xml_node xml_node)
 {
-       pugi::xml_attribute name_attr = xml_node.attribute("name");
+       xml_attribute name_attr = xml_node.attribute("name");
        if(name_attr) {
                node->name = ustring(name_attr.value());
        }
@@ -66,7 +66,7 @@ void xml_read_node(XMLReader& reader, Node *node, pugi::xml_node xml_node)
                        continue;
                }
 
-               pugi::xml_attribute attr = xml_node.attribute(socket.name.c_str());
+               xml_attribute attr = xml_node.attribute(socket.name.c_str());
 
                if(!attr) {
                        continue;
@@ -254,9 +254,9 @@ void xml_read_node(XMLReader& reader, Node *node, pugi::xml_node xml_node)
                reader.node_map[node->name] = node;
 }
 
-pugi::xml_node xml_write_node(Node *node, pugi::xml_node xml_root)
+xml_node xml_write_node(Node *node, xml_node xml_root)
 {
-       pugi::xml_node xml_node = xml_root.append_child(node->type->name.c_str());
+       xml_node xml_node = xml_root.append_child(node->type->name.c_str());
 
        xml_node.append_attribute("name") = node->name.c_str();
 
@@ -271,7 +271,7 @@ pugi::xml_node xml_write_node(Node *node, pugi::xml_node xml_root)
                        continue;
                }
 
-               pugi::xml_attribute attr = xml_node.append_attribute(socket.name.c_str());
+               xml_attribute attr = xml_node.append_attribute(socket.name.c_str());
 
                switch(socket.type)
                {
index 63e80bf79f279fc48fc824a92c052e0be2749fa2..b648c9666c1ac938655c001b70998d38b9de3462 100644 (file)
@@ -28,8 +28,8 @@ struct XMLReader {
        map<ustring, Node*> node_map;
 };
 
-void xml_read_node(XMLReader& reader, Node *node, pugi::xml_node xml_node);
-pugi::xml_node xml_write_node(Node *node, pugi::xml_node xml_root);
+void xml_read_node(XMLReader& reader, Node *node, xml_node xml_node);
+xml_node xml_write_node(Node *node, xml_node xml_root);
 
 CCL_NAMESPACE_END
 
index cfd0afc95f7428a578c449ea07c90116b34a492f..e1a28df94336ccdc8c65caee45ac5b37f2cf9f9f 100644 (file)
@@ -23,7 +23,8 @@
 
 CCL_NAMESPACE_BEGIN
 
-OIIO_NAMESPACE_USING
+using OIIO_NAMESPACE::pugi::xml_node;
+using OIIO_NAMESPACE::pugi::xml_attribute;
 
 CCL_NAMESPACE_END