From ca8a3909be79c6a04b40c5f0d090e6f4843c905b Mon Sep 17 00:00:00 2001 From: Madrich Date: Fri, 1 May 2015 21:40:51 +0200 Subject: [PATCH 1/5] Add StepExporter.cpp(stp) Export Fix XFile --- CREDITS | 4 +- code/CMakeLists.txt | 7 + code/Exporter.cpp | 7 +- code/ProcessHelper.cpp | 20 +++ code/ProcessHelper.h | 10 ++ code/StepExporter.cpp | 374 +++++++++++++++++++++++++++++++++++++++++ code/StepExporter.h | 108 ++++++++++++ code/XFileExporter.cpp | 20 ++- code/XFileExporter.h | 2 - 9 files changed, 540 insertions(+), 12 deletions(-) create mode 100644 code/StepExporter.cpp create mode 100644 code/StepExporter.h diff --git a/CREDITS b/CREDITS index 5028bc306..774bed5f8 100644 --- a/CREDITS +++ b/CREDITS @@ -153,6 +153,8 @@ Ogre Binary format support Android JNI asset extraction support - Richard Steffen -Contributed X File exporter Contributed ExportProperties interface +Contributed X File exporter +Contributed Step (stp) exporter + diff --git a/code/CMakeLists.txt b/code/CMakeLists.txt index e14215838..30f685a84 100644 --- a/code/CMakeLists.txt +++ b/code/CMakeLists.txt @@ -579,6 +579,12 @@ SET( XFile_SRCS ) SOURCE_GROUP( XFile FILES ${XFile_SRCS}) +SET( Step_SRCS + StepExporter.h + StepExporter.cpp +) +SOURCE_GROUP( Step FILES ${Step_SRCS}) + SET( Exporter_SRCS Exporter.cpp AssimpCExport.cpp @@ -716,6 +722,7 @@ SET( assimp_src ${Terragen_SRCS} ${Unreal_SRCS} ${XFile_SRCS} + ${Step_SRCS} ${Extra_SRCS} ${MS3D_SRCS} ${COB_SRCS} diff --git a/code/Exporter.cpp b/code/Exporter.cpp index c6fea8856..f33a08ddf 100644 --- a/code/Exporter.cpp +++ b/code/Exporter.cpp @@ -73,7 +73,8 @@ void GetPostProcessingStepInstanceList(std::vector< BaseProcess* >& out); // Exporter worker function prototypes. Should not be necessary to #ifndef them, it's just a prototype // do not use const, because some exporter need to convert the scene temporary void ExportSceneCollada(const char*,IOSystem*, const aiScene*, const ExportProperties*); -void ExportSceneXFile(const char*,IOSystem*, const aiScene*, const ExportProperties*); +void ExportSceneXFile(const char*,IOSystem*, const aiScene*, const ExportProperties*); +void ExportSceneStep(const char*,IOSystem*, const aiScene*, const ExportProperties*); void ExportSceneObj(const char*,IOSystem*, const aiScene*, const ExportProperties*); void ExportSceneSTL(const char*,IOSystem*, const aiScene*, const ExportProperties*); void ExportSceneSTLBinary(const char*,IOSystem*, const aiScene*, const ExportProperties*); @@ -96,6 +97,10 @@ Exporter::ExportFormatEntry gExporters[] = aiProcess_MakeLeftHanded | aiProcess_FlipWindingOrder | aiProcess_FlipUVs), #endif +#ifndef ASSIMP_BUILD_NO_STEP_EXPORTER + Exporter::ExportFormatEntry( "stp", "Step Files", "stp", &ExportSceneStep, 0), +#endif + #ifndef ASSIMP_BUILD_NO_OBJ_EXPORTER Exporter::ExportFormatEntry( "obj", "Wavefront OBJ format", "obj", &ExportSceneObj, aiProcess_GenSmoothNormals /*| aiProcess_PreTransformVertices */), diff --git a/code/ProcessHelper.cpp b/code/ProcessHelper.cpp index 9250b239e..a3c0cad09 100644 --- a/code/ProcessHelper.cpp +++ b/code/ProcessHelper.cpp @@ -94,6 +94,26 @@ void FindMeshCenter (aiMesh* mesh, aiVector3D& out, aiVector3D& min, aiVector3D& out = min + (max-min)*0.5f; } +// ------------------------------------------------------------------------------- +void FindSceneCenter (aiScene* scene, aiVector3D& out, aiVector3D& min, aiVector3D& max) +{ + if (scene->mNumMeshes == 0) return; + FindMeshCenter(scene->mMeshes[0], out, min, max); + for (int i = 1; i < scene->mNumMeshes; ++i) + { + aiVector3D tout, tmin, tmax; + FindMeshCenter(scene->mMeshes[i], tout, tmin, tmax); + if (min[0] > tmin[0]) min[0] = tmin[0]; + if (min[1] > tmin[1]) min[1] = tmin[1]; + if (min[2] > tmin[2]) min[2] = tmin[2]; + if (max[0] < tmax[0]) max[0] = tmax[0]; + if (max[1] < tmax[1]) max[1] = tmax[1]; + if (max[2] < tmax[2]) max[2] = tmax[2]; + } + out = min + (max-min)*0.5f; +} + + // ------------------------------------------------------------------------------- void FindMeshCenterTransformed (aiMesh* mesh, aiVector3D& out, aiVector3D& min, aiVector3D& max, const aiMatrix4x4& m) diff --git a/code/ProcessHelper.h b/code/ProcessHelper.h index 1c95c61f3..cf6252aa8 100644 --- a/code/ProcessHelper.h +++ b/code/ProcessHelper.h @@ -258,6 +258,16 @@ void FindAABBTransformed (const aiMesh* mesh, aiVector3D& min, aiVector3D& max, * @param[out] out Center point */ void FindMeshCenter (aiMesh* mesh, aiVector3D& out, aiVector3D& min, aiVector3D& max); +// ------------------------------------------------------------------------------- +/** @brief Helper function to determine the 'real' center of a scene + * + * That is the center of its axis-aligned bounding box. + * @param scene Input scene + * @param[out] min Minimum vertex of the scene + * @param[out] max maximum vertex of the scene + * @param[out] out Center point */ +void FindSceneCenter (aiScene* scene, aiVector3D& out, aiVector3D& min, aiVector3D& max); + // ------------------------------------------------------------------------------- // Helper function to determine the 'real' center of a mesh after applying a given transform diff --git a/code/StepExporter.cpp b/code/StepExporter.cpp new file mode 100644 index 000000000..fa970d4f7 --- /dev/null +++ b/code/StepExporter.cpp @@ -0,0 +1,374 @@ +/* +Open Asset Import Library (assimp) +---------------------------------------------------------------------- + +Copyright (c) 2006-2015, assimp team +All rights reserved. + +Redistribution and use of this software in source and binary forms, +with or without modification, are permitted provided that the +following conditions are met: + +* Redistributions of source code must retain the above + copyright notice, this list of conditions and the + following disclaimer. + +* Redistributions in binary form must reproduce the above + copyright notice, this list of conditions and the + following disclaimer in the documentation and/or other + materials provided with the distribution. + +* Neither the name of the assimp team, nor the names of its + contributors may be used to endorse or promote products + derived from this software without specific prior + written permission of the assimp team. + +THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS +"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT +LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR +A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT +OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, +SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT +LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, +DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY +THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT +(INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE +OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + +@author: Richard Steffen, 2015 +---------------------------------------------------------------------- +*/ + +#include "AssimpPCH.h" + +#ifndef ASSIMP_BUILD_NO_EXPORT +#ifndef ASSIMP_BUILD_NO_STEP_EXPORTER +#include "StepExporter.h" +#include "ConvertToLHProcess.h" +#include "Bitmap.h" +#include "fast_atof.h" +#include "SceneCombiner.h" + +#include +#include +#include +#include + +/* Tested with Step viewer v4 from www.ida-step.net */ + +using namespace Assimp; + +namespace Assimp +{ + +// ------------------------------------------------------------------------------------------------ +// Worker function for exporting a scene to Collada. Prototyped and registered in Exporter.cpp +void ExportSceneStep(const char* pFile,IOSystem* pIOSystem, const aiScene* pScene, const ExportProperties* pProperties) +{ + std::string path = ""; + std::string file = pFile; + + // We need to test both types of folder separators because pIOSystem->getOsSeparator() is not reliable. + // Moreover, the path given by some applications is not even consistent with the OS specific type of separator. + const char* end_path = std::max(strrchr(pFile, '\\'), strrchr(pFile, '/')); + + if(end_path != NULL) { + path = std::string(pFile, end_path + 1 - pFile); + file = file.substr(end_path + 1 - pFile, file.npos); + + std::size_t pos = file.find_last_of('.'); + if(pos != file.npos) { + file = file.substr(0, pos); + } + } + + // create/copy Properties + ExportProperties props(*pProperties); + + // set standard properties if not set + //if (!props.HasPropertyBool(AI_CONFIG_EXPORT_XFILE_64BIT)) props.SetPropertyBool(AI_CONFIG_EXPORT_XFILE_64BIT, false); + + // invoke the exporter + StepExporter iDoTheExportThing( pScene, pIOSystem, path, file, &props); + + // we're still here - export successfully completed. Write result to the given IOSYstem + boost::scoped_ptr outfile (pIOSystem->Open(pFile,"wt")); + if(outfile == NULL) { + throw DeadlyExportError("could not open output .stp file: " + std::string(pFile)); + } + + // XXX maybe use a small wrapper around IOStream that behaves like std::stringstream in order to avoid the extra copy. + outfile->Write( iDoTheExportThing.mOutput.str().c_str(), static_cast(iDoTheExportThing.mOutput.tellp()),1); +} + +} // end of namespace Assimp + + +namespace { +inline uint64_t toIndexHash(int32_t id1, int32_t id2) +{ + // dont wonder that -1/-1 -> hash=-1 + uint64_t hash = (uint32_t) id1; + hash = (hash << 32); + hash += (uint32_t) id2; + return hash; +} + +inline void fromIndexHash(uint64_t hash, int32_t &id1, int32_t &id2) +{ + id1 = (hash & 0xFFFFFFFF00000000) >> 32; + id2 = (hash & 0xFFFFFFFF); +} + +// Collect world transformations for each node + void CollectTrafos(const aiNode* node, std::map& trafos) { + const aiMatrix4x4& parent = node->mParent ? trafos[node->mParent] : aiMatrix4x4(); + trafos[node] = parent * node->mTransformation; + for (unsigned int i = 0; i < node->mNumChildren; ++i) { + CollectTrafos(node->mChildren[i], trafos); + } + } + + // Generate a flat list of the meshes (by index) assigned to each node + void CollectMeshes(const aiNode* node, std::multimap& meshes) { + for (unsigned int i = 0; i < node->mNumMeshes; ++i) { + meshes.insert(std::make_pair(node, node->mMeshes[i])); + } + for (unsigned int i = 0; i < node->mNumChildren; ++i) { + CollectMeshes(node->mChildren[i], meshes); + } + } +} + +// ------------------------------------------------------------------------------------------------ +// Constructor for a specific scene to export +StepExporter::StepExporter(const aiScene* pScene, IOSystem* pIOSystem, const std::string& path, const std::string& file, const ExportProperties* pProperties) : mIOSystem(pIOSystem), mPath(path), mFile(file), mProperties(pProperties) +{ + CollectTrafos(pScene->mRootNode, trafos); + CollectMeshes(pScene->mRootNode, meshes); + + // make sure that all formatting happens using the standard, C locale and not the user's current locale + mOutput.imbue( std::locale("C") ); + + mFile = std::string(file); + mPath = std::string(path); + mScene = pScene; + mSceneOwned = false; + + // set up strings + endstr = ";\n"; + + // start writing + WriteFile(); + + + +} + +// ------------------------------------------------------------------------------------------------ +// Destructor +StepExporter::~StepExporter() +{ + if(mSceneOwned) { + delete mScene; + } +} + +// ------------------------------------------------------------------------------------------------ +// Starts writing the contents +void StepExporter::WriteFile() +{ + // see http://shodhganga.inflibnet.ac.in:8080/jspui/bitstream/10603/14116/11/11_chapter%203.pdf + // note, that all realnumber values must be comma separated in x files + mOutput.setf(std::ios::fixed); + mOutput.precision(16); // precission for double + + aiMatrix4x4 baseTransform = mScene->mRootNode->mTransformation; + + int ind = 100; // the start index to be used + int faceEntryLen = 30; // number of entries for a triangle/face + // prepare unique (count triangles and vertices) + std::map uniqueVerts; // use a map to reduce find complexity to log(n) + std::map::iterator it; + int countFace = 0; + + for (unsigned int i=0; imNumMeshes; ++i) + { + aiMesh* mesh = mScene->mMeshes[i]; + for (unsigned int j=0; jmNumFaces; ++j) + { + aiFace* face = &(mesh->mFaces[j]); + + if (face->mNumIndices == 3) countFace++; + } + for (unsigned int j=0; jmNumVertices; ++j) + { + aiVector3D* v = &(mesh->mVertices[j]); + it =uniqueVerts.find(v); + if (it == uniqueVerts.end()) + { + uniqueVerts[v] = -1; // first mark the vector as not transformed + } + } + } + + // write the header + mOutput << "ISO-10303-21" << endstr; + mOutput << "HEADER" << endstr; + mOutput << "FILE_DESCRIPTION(('STEP AP214'),'1')" << endstr; + mOutput << "FILE_NAME('" << mFile << ".stp','2015-04-23T09:26:41',(' '),(' '),'Spatial InterOp 3D',' ',' ')" << endstr; + mOutput << "FILE_SCHEMA(('automotive_design'))" << endstr; + mOutput << "ENDSEC" << endstr; + + // write the top of data + mOutput << "DATA" << endstr; + mOutput << "#1=MECHANICAL_DESIGN_GEOMETRIC_PRESENTATION_REPRESENTATION(' ',("; + for (int i=0; imMeshes[mesh_idx]; + aiMatrix4x4& trafo = trafos[&node]; + for (unsigned int i = 0; i < mesh->mNumVertices; ++i) + { + aiVector3D* v = &(mesh->mVertices[i]); + it = uniqueVerts.find(v); + if (it->second >=0 ) continue; + it->second = ind; // this one is new, so set the index (ind) + aiVector3D vt = trafo * (*v); // transform the coordinate + mOutput << "#" << it->second << "=CARTESIAN_POINT('',(" << vt.x << "," << vt.y << "," << vt.z << "))" << endstr; + mOutput << "#" << it->second+1 << "=VERTEX_POINT('',#" << it->second << ")" << endstr; + ind += 2; + } + } + + // write the triangles + for (unsigned int i=0; imNumMeshes; ++i) + { + aiMesh* mesh = mScene->mMeshes[i]; + for (unsigned int j=0; jmNumFaces; ++j) + { + aiFace* face = &(mesh->mFaces[j]); + + if (face->mNumIndices != 3) continue; + + aiVector3D* v1 = &(mesh->mVertices[face->mIndices[0]]); + aiVector3D* v2 = &(mesh->mVertices[face->mIndices[1]]); + aiVector3D* v3 = &(mesh->mVertices[face->mIndices[2]]); + aiVector3D dv12 = *v2 - *v1; + aiVector3D dv23 = *v3 - *v2; + aiVector3D dv31 = *v1 - *v3; + aiVector3D dv13 = *v3 - *v1; + dv12.Normalize(); + dv23.Normalize(); + dv31.Normalize(); + dv13.Normalize(); + + int pid1 = uniqueVerts.find(v1)->second; + int pid2 = uniqueVerts.find(v2)->second; + int pid3 = uniqueVerts.find(v3)->second; + + int sid = ind; // the sub index + mOutput << "#" << sid << "=STYLED_ITEM('',(#" << sid+1 << "),#" << sid+8 << ")" << endstr; /* the item that must be referenced in #1 */ + /* This is the color information of the Triangle */ + mOutput << "#" << sid+1 << "=PRESENTATION_STYLE_ASSIGNMENT((#" << sid+2 << "))" << endstr; + mOutput << "#" << sid+2 << "=SURFACE_STYLE_USAGE(.BOTH.,#" << sid+3 << ")" << endstr; + mOutput << "#" << sid+3 << "=SURFACE_SIDE_STYLE('',(#" << sid+4 << "))" << endstr; + mOutput << "#" << sid+4 << "=SURFACE_STYLE_FILL_AREA(#" << sid+5 << ")" << endstr; + mOutput << "#" << sid+5 << "=FILL_AREA_STYLE('',(#" << sid+6 << "))" << endstr; + mOutput << "#" << sid+6 << "=FILL_AREA_STYLE_COLOUR('',#" << sid+7 << ")" << endstr; + mOutput << "#" << sid+7 << "=COLOUR_RGB('',0.0,0.0,1.0)" << endstr; + + /* this is the geometry */ + mOutput << "#" << sid+8 << "=FACE_SURFACE('',(#" << sid+13 << "),#" << sid+9<< ",.T.)" << endstr; /* the face that must be referenced in 29 */ + + /* 2 directions of the plane */ + mOutput << "#" << sid+9 << "=PLANE('',#" << sid+10 << ")" << endstr; + mOutput << "#" << sid+10 << "=AXIS2_PLACEMENT_3D('',#" << pid1 << ", #" << sid+11 << ",#" << sid+12 << ")" << endstr; + + mOutput << "#" << sid+11 << "=DIRECTION('',(" << dv12.x << "," << dv12.y << "," << dv12.z << "))" << endstr; + mOutput << "#" << sid+12 << "=DIRECTION('',(" << dv13.x << "," << dv13.y << "," << dv13.z << "))" << endstr; + + mOutput << "#" << sid+13 << "=FACE_BOUND('',#" << sid+14 << ",.T.)" << endstr; + mOutput << "#" << sid+14 << "=EDGE_LOOP('',(#" << sid+15 << ",#" << sid+16 << ",#" << sid+17 << "))" << endstr; + + /* edge loop */ + mOutput << "#" << sid+15 << "=ORIENTED_EDGE('',*,*,#" << sid+18 << ",.T.)" << endstr; + mOutput << "#" << sid+16 << "=ORIENTED_EDGE('',*,*,#" << sid+19 << ",.T.)" << endstr; + mOutput << "#" << sid+17 << "=ORIENTED_EDGE('',*,*,#" << sid+20 << ",.T.)" << endstr; + + /* oriented edges */ + mOutput << "#" << sid+18 << "=EDGE_CURVE('',#" << pid1+1 << ",#" << pid2+1 << ",#" << sid+21 << ",.F.)" << endstr; + mOutput << "#" << sid+19 << "=EDGE_CURVE('',#" << pid2+1 << ",#" << pid3+1 << ",#" << sid+22 << ",.T.)" << endstr; + mOutput << "#" << sid+20 << "=EDGE_CURVE('',#" << pid3+1 << ",#" << pid1+1 << ",#" << sid+23 << ",.T.)" << endstr; + + /* 3 lines and 3 vectors for the lines for the 3 edge curves */ + mOutput << "#" << sid+21 << "=LINE('',#" << pid1 << ",#" << sid+24 << ")" << endstr; + mOutput << "#" << sid+22 << "=LINE('',#" << pid2 << ",#" << sid+25 << ")" << endstr; + mOutput << "#" << sid+23 << "=LINE('',#" << pid3 << ",#" << sid+26 << ")" << endstr; + mOutput << "#" << sid+24 << "=VECTOR('',#" << sid+27 << ",1.0)" << endstr; + mOutput << "#" << sid+25 << "=VECTOR('',#" << sid+28 << ",1.0)" << endstr; + mOutput << "#" << sid+26 << "=VECTOR('',#" << sid+29 << ",1.0)" << endstr; + mOutput << "#" << sid+27 << "=DIRECTION('',(" << dv12.x << "," << dv12.y << "," << dv12.z << "))" << endstr; + mOutput << "#" << sid+28 << "=DIRECTION('',(" << dv23.x << "," << dv23.y << "," << dv23.z << "))" << endstr; + mOutput << "#" << sid+29 << "=DIRECTION('',(" << dv31.x << "," << dv31.y << "," << dv31.z << "))" << endstr; + ind += faceEntryLen; // increase counter + } + } + + mOutput << "ENDSEC" << endstr; // end of data section + mOutput << "END-ISO-10303-21" << endstr; // end of file +} + + + +#endif +#endif + diff --git a/code/StepExporter.h b/code/StepExporter.h new file mode 100644 index 000000000..1ff0ab7de --- /dev/null +++ b/code/StepExporter.h @@ -0,0 +1,108 @@ +/* +Open Asset Import Library (assimp) +---------------------------------------------------------------------- + +Copyright (c) 2006-2012, assimp team +All rights reserved. + +Redistribution and use of this software in source and binary forms, +with or without modification, are permitted provided that the +following conditions are met: + +* Redistributions of source code must retain the above + copyright notice, this list of conditions and the + following disclaimer. + +* Redistributions in binary form must reproduce the above + copyright notice, this list of conditions and the + following disclaimer in the documentation and/or other + materials provided with the distribution. + +* Neither the name of the assimp team, nor the names of its + contributors may be used to endorse or promote products + derived from this software without specific prior + written permission of the assimp team. + +THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS +"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT +LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR +A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT +OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, +SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT +LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, +DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY +THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT +(INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE +OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + +@author: Richard Steffen, 2014 + +---------------------------------------------------------------------- +*/ + +/** @file StepExporter.h + * Declares the exporter class to write a scene to a Collada file + */ +#ifndef AI_STEPEXPORTER_H_INC +#define AI_STEPEXPORTER_H_INC + +#include "../include/assimp/ai_assert.h" +#include + +struct aiScene; +struct aiNode; + +namespace Assimp +{ + +/// Helper class to export a given scene to a StepFile. +/// Note: an StepFile uses a left hand system. Assimp used a right hand system (OpenGL), therefore we have to transform everything +class StepExporter +{ +public: + /// Constructor for a specific scene to export + StepExporter(const aiScene* pScene, IOSystem* pIOSystem, const std::string& path, const std::string& file, const ExportProperties* pProperties); + + /// Destructor + virtual ~StepExporter(); + +protected: + /// Starts writing the contents + void WriteFile(); + +public: + + /// Stringstream to write all output into + std::stringstream mOutput; + +protected: + + /// hold the properties pointer + const ExportProperties* mProperties; + + /// The IOSystem for output + IOSystem* mIOSystem; + + /// Name of the file (without extension) where the scene will be exported + std::string mFile; + + /// Path of the directory where the scene will be exported + std::string mPath; + + /// The scene to be written + const aiScene* mScene; + bool mSceneOwned; + + /// current line end string for simple stream insertion + std::string endstr; + + std::map trafos; + + typedef std::multimap MeshesByNodeMap; + MeshesByNodeMap meshes; + +}; + +} + +#endif // !! AI_STEPEXPORTER_H_INC diff --git a/code/XFileExporter.cpp b/code/XFileExporter.cpp index ce87f3168..a4ab68746 100644 --- a/code/XFileExporter.cpp +++ b/code/XFileExporter.cpp @@ -319,8 +319,8 @@ void XFileExporter::WriteNode( aiNode* pNode) WriteMesh(mScene->mMeshes[pNode->mMeshes[i]]); // recursive call the Nodes - for (size_t a = 0; a < pNode->mNumChildren; a++) - WriteNode( mScene->mRootNode->mChildren[a]); + for (size_t i = 0; i < pNode->mNumChildren; ++i) + WriteNode( mScene->mRootNode->mChildren[i]); PopTag(); @@ -507,12 +507,16 @@ void XFileExporter::WriteMesh(aiMesh* mesh) std::string XFileExporter::toXFileString(aiString &name) { - std::string str = std::string(name.C_Str()); - std::replace(str.begin(), str.end(), '<', '_'); - std::replace(str.begin(), str.end(), '>', '_'); - std::replace(str.begin(), str.end(), '{', '_'); - std::replace(str.begin(), str.end(), '}', '_'); - std::replace(str.begin(), str.end(), '$', '_'); + std::string pref = "NN_"; // node name prefix to prevent unexpected start of string + std::string str = pref + std::string(name.C_Str()); + for (int i=0; i= 48 && str[i] <= 57) || // 0-9 + (str[i] >= 65 && str[i] <= 90) || // A-Z + (str[i] >= 97 && str[i] <= 122)) // a-z + continue; + str[i] = '_'; + } return str; } diff --git a/code/XFileExporter.h b/code/XFileExporter.h index 5c9f56701..b2f4e03ab 100644 --- a/code/XFileExporter.h +++ b/code/XFileExporter.h @@ -122,8 +122,6 @@ protected: /// current line end string for simple stream insertion std::string endstr; -protected: - }; } From 58a72d93026b924ba9f23b5d65b01da8e9627bbd Mon Sep 17 00:00:00 2001 From: Madrich Date: Fri, 1 May 2015 22:19:40 +0200 Subject: [PATCH 2/5] Fix Warnings Add Time in Step File --- code/ProcessHelper.cpp | 2 +- code/StepExporter.cpp | 13 +++++++------ code/XFileExporter.cpp | 4 ++-- 3 files changed, 10 insertions(+), 9 deletions(-) diff --git a/code/ProcessHelper.cpp b/code/ProcessHelper.cpp index a3c0cad09..0bde4be93 100644 --- a/code/ProcessHelper.cpp +++ b/code/ProcessHelper.cpp @@ -99,7 +99,7 @@ void FindSceneCenter (aiScene* scene, aiVector3D& out, aiVector3D& min, aiVector { if (scene->mNumMeshes == 0) return; FindMeshCenter(scene->mMeshes[0], out, min, max); - for (int i = 1; i < scene->mNumMeshes; ++i) + for (unsigned int i = 1; i < scene->mNumMeshes; ++i) { aiVector3D tout, tmin, tmax; FindMeshCenter(scene->mMeshes[i], tout, tmin, tmax); diff --git a/code/StepExporter.cpp b/code/StepExporter.cpp index fa970d4f7..e609ae821 100644 --- a/code/StepExporter.cpp +++ b/code/StepExporter.cpp @@ -150,8 +150,6 @@ StepExporter::StepExporter(const aiScene* pScene, IOSystem* pIOSystem, const std // make sure that all formatting happens using the standard, C locale and not the user's current locale mOutput.imbue( std::locale("C") ); - mFile = std::string(file); - mPath = std::string(path); mScene = pScene; mSceneOwned = false; @@ -182,9 +180,7 @@ void StepExporter::WriteFile() // note, that all realnumber values must be comma separated in x files mOutput.setf(std::ios::fixed); mOutput.precision(16); // precission for double - - aiMatrix4x4 baseTransform = mScene->mRootNode->mTransformation; - + int ind = 100; // the start index to be used int faceEntryLen = 30; // number of entries for a triangle/face // prepare unique (count triangles and vertices) @@ -212,11 +208,16 @@ void StepExporter::WriteFile() } } + static const unsigned int date_nb_chars = 20; + char date_str[date_nb_chars]; + std::time_t date = std::time(NULL); + std::strftime(date_str, date_nb_chars, "%Y-%m-%dT%H:%M:%S", std::localtime(&date)); + // write the header mOutput << "ISO-10303-21" << endstr; mOutput << "HEADER" << endstr; mOutput << "FILE_DESCRIPTION(('STEP AP214'),'1')" << endstr; - mOutput << "FILE_NAME('" << mFile << ".stp','2015-04-23T09:26:41',(' '),(' '),'Spatial InterOp 3D',' ',' ')" << endstr; + mOutput << "FILE_NAME('" << mFile << ".stp','" << date_str << "',(' '),(' '),'Spatial InterOp 3D',' ',' ')" << endstr; mOutput << "FILE_SCHEMA(('automotive_design'))" << endstr; mOutput << "ENDSEC" << endstr; diff --git a/code/XFileExporter.cpp b/code/XFileExporter.cpp index a4ab68746..002f5a8df 100644 --- a/code/XFileExporter.cpp +++ b/code/XFileExporter.cpp @@ -111,7 +111,7 @@ XFileExporter::XFileExporter(const aiScene* pScene, IOSystem* pIOSystem, const s mSceneOwned = false; // set up strings - endstr = "\n"; + endstr = "\n"; // start writing WriteFile(); @@ -509,7 +509,7 @@ std::string XFileExporter::toXFileString(aiString &name) { std::string pref = "NN_"; // node name prefix to prevent unexpected start of string std::string str = pref + std::string(name.C_Str()); - for (int i=0; i= 48 && str[i] <= 57) || // 0-9 (str[i] >= 65 && str[i] <= 90) || // A-Z From 148207a07309feab55da7fd5d644c3078bbd83c6 Mon Sep 17 00:00:00 2001 From: Madrich Date: Fri, 1 May 2015 23:16:44 +0200 Subject: [PATCH 3/5] Fix removed AssimpPCH --- code/StepExporter.cpp | 7 ++++++- code/StepExporter.h | 2 ++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/code/StepExporter.cpp b/code/StepExporter.cpp index e609ae821..a8fca4613 100644 --- a/code/StepExporter.cpp +++ b/code/StepExporter.cpp @@ -39,13 +39,13 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. ---------------------------------------------------------------------- */ -#include "AssimpPCH.h" #ifndef ASSIMP_BUILD_NO_EXPORT #ifndef ASSIMP_BUILD_NO_STEP_EXPORTER #include "StepExporter.h" #include "ConvertToLHProcess.h" #include "Bitmap.h" +#include "BaseImporter.h" #include "fast_atof.h" #include "SceneCombiner.h" @@ -53,6 +53,11 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #include #include #include +#include +#include "Exceptional.h" +#include "../include/assimp/IOSystem.hpp" +#include "../include/assimp/scene.h" +#include "../include/assimp/light.h" /* Tested with Step viewer v4 from www.ida-step.net */ diff --git a/code/StepExporter.h b/code/StepExporter.h index 1ff0ab7de..0439bef6f 100644 --- a/code/StepExporter.h +++ b/code/StepExporter.h @@ -47,6 +47,8 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #define AI_STEPEXPORTER_H_INC #include "../include/assimp/ai_assert.h" +#include "../include/assimp/matrix4x4.h" +#include "../include/assimp/Exporter.hpp" #include struct aiScene; From 2dde962f0d4851eb13092e061df70139b33fa357 Mon Sep 17 00:00:00 2001 From: Madrich Date: Sat, 2 May 2015 01:56:20 +0200 Subject: [PATCH 4/5] FilePath Functions in DefaultIOSystems Replacements in Exporters --- code/ColladaExporter.cpp | 19 ++------ code/DefaultIOSystem.cpp | 27 +++++++++++ code/DefaultIOSystem.h | 15 ++++++ code/StepExporter.cpp | 98 ++++++++++++++++++---------------------- code/StepExporter.h | 7 ++- code/XFileExporter.cpp | 20 ++------ 6 files changed, 94 insertions(+), 92 deletions(-) diff --git a/code/ColladaExporter.cpp b/code/ColladaExporter.cpp index 380b971f2..078ac0a56 100644 --- a/code/ColladaExporter.cpp +++ b/code/ColladaExporter.cpp @@ -47,6 +47,7 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #include "Bitmap.h" #include "fast_atof.h" #include "SceneCombiner.h" +#include "DefaultIOSystem.h" #include "XMLTools.h" #include "../include/assimp/IOSystem.hpp" #include "../include/assimp/Exporter.hpp" @@ -67,22 +68,8 @@ namespace Assimp // Worker function for exporting a scene to Collada. Prototyped and registered in Exporter.cpp void ExportSceneCollada(const char* pFile, IOSystem* pIOSystem, const aiScene* pScene, const ExportProperties* pProperties) { - std::string path = ""; - std::string file = pFile; - - // We need to test both types of folder separators because pIOSystem->getOsSeparator() is not reliable. - // Moreover, the path given by some applications is not even consistent with the OS specific type of separator. - const char* end_path = std::max(strrchr(pFile, '\\'), strrchr(pFile, '/')); - - if(end_path != NULL) { - path = std::string(pFile, end_path + 1 - pFile); - file = file.substr(end_path + 1 - pFile, file.npos); - - std::size_t pos = file.find_last_of('.'); - if(pos != file.npos) { - file = file.substr(0, pos); - } - } + std::string path = DefaultIOSystem::absolutePath(std::string(pFile)); + std::string file = DefaultIOSystem::completeBaseName(std::string(pFile)); // invoke the exporter ColladaExporter iDoTheExportThing( pScene, pIOSystem, path, file); diff --git a/code/DefaultIOSystem.cpp b/code/DefaultIOSystem.cpp index 58a84aafa..421615088 100644 --- a/code/DefaultIOSystem.cpp +++ b/code/DefaultIOSystem.cpp @@ -167,4 +167,31 @@ bool DefaultIOSystem::ComparePaths (const char* one, const char* second) const return !ASSIMP_stricmp(temp1,temp2); } + +std::string DefaultIOSystem::fileName(std::string path) +{ + std::string ret = path; + std::less comp; + const char* end_path = comp(strrchr(path.c_str(), '\\'), strrchr(path.c_str(), '/')) ? strrchr(path.c_str(), '/') : strrchr(path.c_str(), '\\'); + if(end_path != NULL) ret = ret.substr(end_path + 1 - path.c_str(), ret.npos); + return ret; +} + +std::string DefaultIOSystem::completeBaseName(std::string path) +{ + std::string ret = fileName(path); + std::size_t pos = ret.find_last_of('.'); + if(pos != ret.npos) ret = ret.substr(0, pos); + return ret; +} + +std::string DefaultIOSystem::absolutePath(std::string path) +{ + std::string ret; + std::less comp; + const char* end_path = comp(strrchr(path.c_str(), '\\'), strrchr(path.c_str(), '/')) ? strrchr(path.c_str(), '/') : strrchr(path.c_str(), '\\'); + if(end_path != NULL) ret = std::string(path.c_str(), end_path + 1 - path.c_str()); + return ret; +} + #undef PATHLIMIT diff --git a/code/DefaultIOSystem.h b/code/DefaultIOSystem.h index fa3e4c625..de79df329 100644 --- a/code/DefaultIOSystem.h +++ b/code/DefaultIOSystem.h @@ -76,6 +76,21 @@ public: // ------------------------------------------------------------------- /** Compare two paths */ bool ComparePaths (const char* one, const char* second) const; + + /** @brief get the file name of a full filepath + * example: /tmp/archive.tar.gz -> archive.tar.gz + */ + static std::string fileName(std::string path); + + /** @brief get the complete base name of a full filepath + * example: /tmp/archive.tar.gz -> archive.tar + */ + static std::string completeBaseName(std::string path); + + /** @brief get the path of a full filepath + * example: /tmp/archive.tar.gz -> /tmp/ + */ + static std::string absolutePath(std::string path); }; } //!ns Assimp diff --git a/code/StepExporter.cpp b/code/StepExporter.cpp index a8fca4613..877a64217 100644 --- a/code/StepExporter.cpp +++ b/code/StepExporter.cpp @@ -48,7 +48,8 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #include "BaseImporter.h" #include "fast_atof.h" #include "SceneCombiner.h" - +#include "DefaultIOSystem.h" +#include #include #include #include @@ -59,6 +60,27 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #include "../include/assimp/scene.h" #include "../include/assimp/light.h" +// +#if _MSC_VER > 1500 || (defined __GNUC___) +# define ASSIMP_STEP_USE_UNORDERED_MULTIMAP +# else +# define step_unordered_map map +# define step_unordered_multimap multimap +#endif + +#ifdef ASSIMP_STEP_USE_UNORDERED_MULTIMAP +# include +# if _MSC_VER > 1600 +# define step_unordered_map unordered_map +# define step_unordered_multimap unordered_multimap +# else +# define step_unordered_map tr1::unordered_map +# define step_unordered_multimap tr1::unordered_multimap +# endif +#endif + +typedef std::step_unordered_map VectorIndexUMap; + /* Tested with Step viewer v4 from www.ida-step.net */ using namespace Assimp; @@ -70,29 +92,12 @@ namespace Assimp // Worker function for exporting a scene to Collada. Prototyped and registered in Exporter.cpp void ExportSceneStep(const char* pFile,IOSystem* pIOSystem, const aiScene* pScene, const ExportProperties* pProperties) { - std::string path = ""; - std::string file = pFile; - - // We need to test both types of folder separators because pIOSystem->getOsSeparator() is not reliable. - // Moreover, the path given by some applications is not even consistent with the OS specific type of separator. - const char* end_path = std::max(strrchr(pFile, '\\'), strrchr(pFile, '/')); - - if(end_path != NULL) { - path = std::string(pFile, end_path + 1 - pFile); - file = file.substr(end_path + 1 - pFile, file.npos); - - std::size_t pos = file.find_last_of('.'); - if(pos != file.npos) { - file = file.substr(0, pos); - } - } + std::string path = DefaultIOSystem::absolutePath(std::string(pFile)); + std::string file = DefaultIOSystem::completeBaseName(std::string(pFile)); // create/copy Properties ExportProperties props(*pProperties); - // set standard properties if not set - //if (!props.HasPropertyBool(AI_CONFIG_EXPORT_XFILE_64BIT)) props.SetPropertyBool(AI_CONFIG_EXPORT_XFILE_64BIT, false); - // invoke the exporter StepExporter iDoTheExportThing( pScene, pIOSystem, path, file, &props); @@ -110,22 +115,22 @@ void ExportSceneStep(const char* pFile,IOSystem* pIOSystem, const aiScene* pScen namespace { -inline uint64_t toIndexHash(int32_t id1, int32_t id2) -{ - // dont wonder that -1/-1 -> hash=-1 - uint64_t hash = (uint32_t) id1; - hash = (hash << 32); - hash += (uint32_t) id2; - return hash; -} + inline uint64_t toIndexHash(int32_t id1, int32_t id2) + { + // dont wonder that -1/-1 -> hash=-1 + uint64_t hash = (uint32_t) id1; + hash = (hash << 32); + hash += (uint32_t) id2; + return hash; + } -inline void fromIndexHash(uint64_t hash, int32_t &id1, int32_t &id2) -{ - id1 = (hash & 0xFFFFFFFF00000000) >> 32; - id2 = (hash & 0xFFFFFFFF); -} + inline void fromIndexHash(uint64_t hash, int32_t &id1, int32_t &id2) + { + id1 = (hash & 0xFFFFFFFF00000000) >> 32; + id2 = (hash & 0xFFFFFFFF); + } -// Collect world transformations for each node + // Collect world transformations for each node void CollectTrafos(const aiNode* node, std::map& trafos) { const aiMatrix4x4& parent = node->mParent ? trafos[node->mParent] : aiMatrix4x4(); trafos[node] = parent * node->mTransformation; @@ -156,25 +161,12 @@ StepExporter::StepExporter(const aiScene* pScene, IOSystem* pIOSystem, const std mOutput.imbue( std::locale("C") ); mScene = pScene; - mSceneOwned = false; // set up strings - endstr = ";\n"; + endstr = ";\n"; // start writing WriteFile(); - - - -} - -// ------------------------------------------------------------------------------------------------ -// Destructor -StepExporter::~StepExporter() -{ - if(mSceneOwned) { - delete mScene; - } } // ------------------------------------------------------------------------------------------------ @@ -189,8 +181,9 @@ void StepExporter::WriteFile() int ind = 100; // the start index to be used int faceEntryLen = 30; // number of entries for a triangle/face // prepare unique (count triangles and vertices) - std::map uniqueVerts; // use a map to reduce find complexity to log(n) - std::map::iterator it; + + VectorIndexUMap uniqueVerts; // use a map to reduce find complexity to log(n) + VectorIndexUMap::iterator it; int countFace = 0; for (unsigned int i=0; imNumMeshes; ++i) @@ -262,10 +255,7 @@ void StepExporter::WriteFile() mOutput << "#25=CARTESIAN_POINT('',(0.0,0.0,0.0))" << endstr; mOutput << "#26=DIRECTION('',(0.0,0.0,1.0))" << endstr; mOutput << "#27=DIRECTION('',(1.0,0.0,0.0))" << endstr; - mOutput << "#28= (NAMED_UNIT(#21)LENGTH_UNIT()SI_UNIT(.MILLI.,.METRE.))" << endstr; - - mOutput << "#29=CLOSED_SHELL('',("; for (int i=0; i + struct aiScene; struct aiNode; @@ -65,9 +66,6 @@ public: /// Constructor for a specific scene to export StepExporter(const aiScene* pScene, IOSystem* pIOSystem, const std::string& path, const std::string& file, const ExportProperties* pProperties); - /// Destructor - virtual ~StepExporter(); - protected: /// Starts writing the contents void WriteFile(); @@ -93,13 +91,14 @@ protected: /// The scene to be written const aiScene* mScene; - bool mSceneOwned; /// current line end string for simple stream insertion std::string endstr; + /// accumultated transformations for nodes std::map trafos; + /// map to all meshed of nodes typedef std::multimap MeshesByNodeMap; MeshesByNodeMap meshes; diff --git a/code/XFileExporter.cpp b/code/XFileExporter.cpp index 0bc436e14..74e1f0389 100644 --- a/code/XFileExporter.cpp +++ b/code/XFileExporter.cpp @@ -48,7 +48,7 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #include "BaseImporter.h" #include "fast_atof.h" #include "SceneCombiner.h" - +#include "DefaultIOSystem.h" #include #include #include @@ -68,22 +68,8 @@ namespace Assimp // Worker function for exporting a scene to Collada. Prototyped and registered in Exporter.cpp void ExportSceneXFile(const char* pFile,IOSystem* pIOSystem, const aiScene* pScene, const ExportProperties* pProperties) { - std::string path = ""; - std::string file = pFile; - - // We need to test both types of folder separators because pIOSystem->getOsSeparator() is not reliable. - // Moreover, the path given by some applications is not even consistent with the OS specific type of separator. - const char* end_path = std::max(strrchr(pFile, '\\'), strrchr(pFile, '/')); - - if(end_path != NULL) { - path = std::string(pFile, end_path + 1 - pFile); - file = file.substr(end_path + 1 - pFile, file.npos); - - std::size_t pos = file.find_last_of('.'); - if(pos != file.npos) { - file = file.substr(0, pos); - } - } + std::string path = DefaultIOSystem::absolutePath(std::string(pFile)); + std::string file = DefaultIOSystem::completeBaseName(std::string(pFile)); // create/copy Properties ExportProperties props(*pProperties); From b6d91ec11aba91d0661be64965e5a20cae2b1ab6 Mon Sep 17 00:00:00 2001 From: Madrich Date: Sat, 2 May 2015 12:19:20 +0200 Subject: [PATCH 5/5] Fix Path functions Fix XFilePrefix for mesh name Add Color for faces in Step Export --- code/DefaultIOSystem.cpp | 14 +++++++------- code/StepExporter.cpp | 20 +++++++++++++++++++- code/XFileExporter.cpp | 8 ++++---- 3 files changed, 30 insertions(+), 12 deletions(-) diff --git a/code/DefaultIOSystem.cpp b/code/DefaultIOSystem.cpp index 421615088..5ce86eadc 100644 --- a/code/DefaultIOSystem.cpp +++ b/code/DefaultIOSystem.cpp @@ -171,12 +171,12 @@ bool DefaultIOSystem::ComparePaths (const char* one, const char* second) const std::string DefaultIOSystem::fileName(std::string path) { std::string ret = path; - std::less comp; - const char* end_path = comp(strrchr(path.c_str(), '\\'), strrchr(path.c_str(), '/')) ? strrchr(path.c_str(), '/') : strrchr(path.c_str(), '\\'); - if(end_path != NULL) ret = ret.substr(end_path + 1 - path.c_str(), ret.npos); + std::size_t last = ret.find_last_of("\\/"); + if (last != std::string::npos) ret = ret.substr(last + 1); return ret; } + std::string DefaultIOSystem::completeBaseName(std::string path) { std::string ret = fileName(path); @@ -185,12 +185,12 @@ std::string DefaultIOSystem::completeBaseName(std::string path) return ret; } + std::string DefaultIOSystem::absolutePath(std::string path) { - std::string ret; - std::less comp; - const char* end_path = comp(strrchr(path.c_str(), '\\'), strrchr(path.c_str(), '/')) ? strrchr(path.c_str(), '/') : strrchr(path.c_str(), '\\'); - if(end_path != NULL) ret = std::string(path.c_str(), end_path + 1 - path.c_str()); + std::string ret = path; + std::size_t last = ret.find_last_of("\\/"); + if (last != std::string::npos) ret = ret.substr(0, last); return ret; } diff --git a/code/StepExporter.cpp b/code/StepExporter.cpp index 877a64217..0ce273313 100644 --- a/code/StepExporter.cpp +++ b/code/StepExporter.cpp @@ -178,6 +178,12 @@ void StepExporter::WriteFile() mOutput.setf(std::ios::fixed); mOutput.precision(16); // precission for double + // standard color + aiColor4D fColor; + fColor.r = 0.8f; + fColor.g = 0.8f; + fColor.b = 0.8f; + int ind = 100; // the start index to be used int faceEntryLen = 30; // number of entries for a triangle/face // prepare unique (count triangles and vertices) @@ -310,6 +316,18 @@ void StepExporter::WriteFile() int pid1 = uniqueVerts.find(v1)->second; int pid2 = uniqueVerts.find(v2)->second; int pid3 = uniqueVerts.find(v3)->second; + + // mean vertex color for the face if available + if (mesh->HasVertexColors(0)) + { + fColor.r = 0.0; + fColor.g = 0.0; + fColor.b = 0.0; + fColor += mesh->mColors[0][face->mIndices[0]]; + fColor += mesh->mColors[0][face->mIndices[1]]; + fColor += mesh->mColors[0][face->mIndices[2]]; + fColor /= 3.0f; + } int sid = ind; // the sub index mOutput << "#" << sid << "=STYLED_ITEM('',(#" << sid+1 << "),#" << sid+8 << ")" << endstr; /* the item that must be referenced in #1 */ @@ -320,7 +338,7 @@ void StepExporter::WriteFile() mOutput << "#" << sid+4 << "=SURFACE_STYLE_FILL_AREA(#" << sid+5 << ")" << endstr; mOutput << "#" << sid+5 << "=FILL_AREA_STYLE('',(#" << sid+6 << "))" << endstr; mOutput << "#" << sid+6 << "=FILL_AREA_STYLE_COLOUR('',#" << sid+7 << ")" << endstr; - mOutput << "#" << sid+7 << "=COLOUR_RGB('',0.0,0.0,1.0)" << endstr; + mOutput << "#" << sid+7 << "=COLOUR_RGB(''," << fColor.r << "," << fColor.g << "," << fColor.b << ")" << endstr; /* this is the geometry */ mOutput << "#" << sid+8 << "=FACE_SURFACE('',(#" << sid+13 << "),#" << sid+9<< ",.T.)" << endstr; /* the face that must be referenced in 29 */ diff --git a/code/XFileExporter.cpp b/code/XFileExporter.cpp index 74e1f0389..276b309dd 100644 --- a/code/XFileExporter.cpp +++ b/code/XFileExporter.cpp @@ -500,13 +500,13 @@ void XFileExporter::WriteMesh(aiMesh* mesh) std::string XFileExporter::toXFileString(aiString &name) { - std::string pref = "NN_"; // node name prefix to prevent unexpected start of string + std::string pref = ""; // node name prefix to prevent unexpected start of string std::string str = pref + std::string(name.C_Str()); for (int i=0; i < (int) str.length(); ++i) { - if ((str[i] >= 48 && str[i] <= 57) || // 0-9 - (str[i] >= 65 && str[i] <= 90) || // A-Z - (str[i] >= 97 && str[i] <= 122)) // a-z + if ((str[i] >= '0' && str[i] <= '9') || // 0-9 + (str[i] >= 'A' && str[i] <= 'Z') || // A-Z + (str[i] >= 'a' && str[i] <= 'z')) // a-z continue; str[i] = '_'; }