djsundog
/
obj2gltf
mirror of https://github.com/CesiumGS/obj2gltf.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Don't add a face that contains only two vertices

This commit is contained in:
Sean Lilley 2017-12-12 09:34:35 -05:00
parent e85d64f021
commit 2869560aab
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/loadObj.js
View File

@ -329,7 +329,7 @@ function loadObj(objPath, options) {
function isConvex(positions2D) { function isConvex(positions2D) {
var turnDirection = getTurnDirection(positions2D[0], positions2D[1], positions2D[2]); var turnDirection = getTurnDirection(positions2D[0], positions2D[1], positions2D[2]);
for (var i=1; i < positions2D.length-2; ++i) { for (var i=1; i < positions2D.length-2; ++i) {
var currentTurnDirection = getTurnDirection(positions2D[i], positions2D[i+1], positions2D[i+2]); var currentTurnDirection = getTurnDirection(positions2D[i], positions2D[i+1], positions2D[i+2]);
if (turnDirection * currentTurnDirection < 0) { if (turnDirection * currentTurnDirection < 0) {
return false; return false;
} }
@ -465,7 +465,9 @@ function loadObj(objPath, options) {
faceUvs.push(result[2]); faceUvs.push(result[2]);
faceNormals.push(result[3]); faceNormals.push(result[3]);
} }
addFace(faceVertices, facePositions, faceUvs, faceNormals); if (faceVertices.length > 2) {
addFace(faceVertices, facePositions, faceUvs, faceNormals);
}
faceVertices.length = 0; faceVertices.length = 0;
facePositions.length = 0; facePositions.length = 0;