diff --git a/code/X/XFileImporter.cpp b/code/X/XFileImporter.cpp index 10547bb8a..be7256d5c 100644 --- a/code/X/XFileImporter.cpp +++ b/code/X/XFileImporter.cpp @@ -332,7 +332,7 @@ void XFileImporter::CreateMeshes( aiScene* pScene, aiNode* pNode, const std::vec // collect vertex data for indices of this face for( unsigned int d = 0; d < df.mNumIndices; ++d ) { - df.mIndices[d] = newIndex; + df.mIndices[ d ] = newIndex; const unsigned int newIdx( pf.mIndices[ d ] ); if ( newIdx > sourceMesh->mPositions.size() ) { continue; @@ -344,7 +344,10 @@ void XFileImporter::CreateMeshes( aiScene* pScene, aiNode* pNode, const std::vec mesh->mVertices[newIndex] = sourceMesh->mPositions[pf.mIndices[d]]; // Normal, if present if ( mesh->HasNormals() ) { - mesh->mNormals[ newIndex ] = sourceMesh->mNormals[ sourceMesh->mNormFaces[ f ].mIndices[ d ] ]; + if ( sourceMesh->mNormFaces[ f ].mIndices.size() > d ) { + const size_t idx( sourceMesh->mNormFaces[ f ].mIndices[ d ] ); + mesh->mNormals[ newIndex ] = sourceMesh->mNormals[ idx ]; + } } // texture coord sets diff --git a/code/X/XFileImporter.h b/code/X/XFileImporter.h index d1b8d3f47..7d12b6fdf 100644 --- a/code/X/XFileImporter.h +++ b/code/X/XFileImporter.h @@ -58,13 +58,13 @@ struct aiNode; namespace Assimp { namespace XFile { -struct Scene; -struct Node; + struct Scene; + struct Node; } // --------------------------------------------------------------------------- /** The XFileImporter is a worker class capable of importing a scene from a - * DirectX file .x + * DirectX file .x */ class XFileImporter : public BaseImporter { public: diff --git a/code/X/XFileParser.cpp b/code/X/XFileParser.cpp index ed715a19d..08d3c88da 100644 --- a/code/X/XFileParser.cpp +++ b/code/X/XFileParser.cpp @@ -583,24 +583,30 @@ void XFileParser::ParseDataObjectMeshNormals( Mesh* pMesh) pMesh->mNormals.resize( numNormals); // read normal vectors - for( unsigned int a = 0; a < numNormals; a++) + for( unsigned int a = 0; a < numNormals; ++a) { pMesh->mNormals[a] = ReadVector3(); + } // read normal indices unsigned int numFaces = ReadInt(); - if( numFaces != pMesh->mPosFaces.size()) + if( numFaces != pMesh->mPosFaces.size()) { ThrowException( "Normal face count does not match vertex face count."); + } - for( unsigned int a = 0; a < numFaces; a++) - { - unsigned int numIndices = ReadInt(); - pMesh->mNormFaces.push_back( Face()); - Face& face = pMesh->mNormFaces.back(); + // do not crah when no face definitions are there + if (numFaces > 0) { + // normal face creation + pMesh->mNormFaces.resize( pMesh->mNormFaces.size() + numFaces ); + for( unsigned int a = 0; a < numFaces; ++a ) { + unsigned int numIndices = ReadInt(); + pMesh->mNormFaces.push_back( Face() ); + Face& face = pMesh->mNormFaces.back(); + for( unsigned int b = 0; b < numIndices; ++b ) { + face.mIndices.push_back( ReadInt()); + } - for( unsigned int b = 0; b < numIndices; b++) - face.mIndices.push_back( ReadInt()); - - TestForSeparator(); + TestForSeparator(); + } } CheckForClosingBrace(); diff --git a/test/models/PLY/cube_test.ply b/test/models/PLY/cube_test.ply index bb0839286..194798cf0 100644 --- a/test/models/PLY/cube_test.ply +++ b/test/models/PLY/cube_test.ply @@ -1,6 +1,6 @@ ply format ascii 1.0 -comment Created by Open Asset Import Library - http://assimp.sf.net (v4.1.2760932948) +comment Created by Open Asset Import Library - http://assimp.sf.net (v4.1.3945266037) element vertex 8 property float x property float y diff --git a/test/unit/utBlendImportMaterials.cpp b/test/unit/utBlendImportMaterials.cpp index cefacbe27..151c614c0 100644 --- a/test/unit/utBlendImportMaterials.cpp +++ b/test/unit/utBlendImportMaterials.cpp @@ -133,10 +133,10 @@ TEST_F(BlendImportMaterials, testImportMaterialwith2texturesAnd2TexCoordMappings // material has 2 diffuse textures ASSERT_TRUE(pTest->HasMaterials()); - EXPECT_EQ(1, pTest->mNumMaterials); + EXPECT_EQ(1u, pTest->mNumMaterials); const aiMaterial *pMat = pTest->mMaterials[0]; ASSERT_TRUE(nullptr != pMat); - ASSERT_EQ(2, pMat->GetTextureCount(aiTextureType_DIFFUSE)); + ASSERT_EQ(2u, pMat->GetTextureCount(aiTextureType_DIFFUSE)); aiString aPath; aiTextureMapping tm = aiTextureMapping::aiTextureMapping_OTHER; aiReturn result = pMat->GetTexture(aiTextureType_DIFFUSE, 0, &aPath, &tm); @@ -146,7 +146,7 @@ TEST_F(BlendImportMaterials, testImportMaterialwith2texturesAnd2TexCoordMappings // mesh has 2 texturecoord sets ASSERT_TRUE(pTest->HasMeshes()); - EXPECT_EQ(1, pTest->mNumMeshes); + EXPECT_EQ(1u, pTest->mNumMeshes); const aiMesh *pMesh = pTest->mMeshes[0]; ASSERT_TRUE(nullptr != pMesh); ASSERT_TRUE(pMesh->HasTextureCoords(0));