dqn
/
FBX2glTF
mirror of https://github.com/facebookincubator/FBX2glTF.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Cope with geometry opting out of geometry. 

Some FBX files have index arrays that contain -1 (indeed, that are
nothing but negative ones). Presumably the intention is to specify "no
material". In any case, let's not segfault.
This commit is contained in:
Par Winzell 2017-10-24 20:54:59 -07:00
parent 4e0f5e4fcf
commit db3f232fc9
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/Fbx2Raw.cpp
View File

@ -265,6 +265,9 @@ public:
for (int ii = 0; ii < indices->GetCount(); ii++) {
int materialNum = indices->GetAt(ii);
if (materialNum < 0) {
continue;
}
if (materialNum >= summaries.size()) {
summaries.resize(materialNum + 1);
}
@ -280,9 +283,11 @@ public:
const std::shared_ptr<FbxMaterialAccess> GetMaterial(const int polygonIndex) const
{
if (mappingMode != FbxGeometryElement::eNone) {
const auto materialNum = static_cast<unsigned int>(indices->GetAt(
(mappingMode == FbxGeometryElement::eByPolygon) ? polygonIndex : 0));
return summaries.at(materialNum);
const int materialNum = indices->GetAt((mappingMode == FbxGeometryElement::eByPolygon) ? polygonIndex : 0);
if (materialNum < 0) {
return nullptr;
}
return summaries.at((unsigned long) materialNum);
}
return nullptr;
}