Don't add a face that contains only two vertices

CesiumGS / obj2gltf

Convert OBJ assets to glTF

Apache License 2.0

1.71k stars 307 forks source link

Don't add a face that contains only two vertices #120

Closed lilleyse closed 6 years ago

lilleyse commented 6 years ago

Fixes https://github.com/AnalyticalGraphicsInc/obj2gltf/issues/119

emackey commented 6 years ago

Are primitives like POINTS and LINES supported by this converter? Just asking in case this fix might impact them. Not even sure if OBJ does them at all.

lilleyse commented 6 years ago

The converter doesn't support points or lines.

The obj format supports them

p v1 v2 v3 l v1 v2 v3

But in this case it was actually an f line that only had two vertices, which seems like a problem with the obj.

emackey commented 6 years ago

Sounds good, thanks!