Merge pull request tinyobjloader#235 from SergeyXa/shapes-being-skipped-bugfix

syoyo · web-flow · commit 8920b25ac66e · 2019-11-06T18:45:43.000+09:00
Fix: Some data may be ignored when loading objects from .obj
diff --git a/tiny_obj_loader.h b/tiny_obj_loader.h
@@ -2438,7 +2438,10 @@ bool LoadObj(attrib_t *attrib, std::vector<shape_t> *shapes,
       // flush previous face group.
       bool ret = exportGroupsToShape(&shape, prim_group, tags, material, name,
                                      triangulate, v);
-      if (ret) {
+      (void)ret;  // return value not used.
+
+      if (shape.mesh.indices.size() > 0 || shape.lines.indices.size() > 0 ||
+          shape.points.indices.size() > 0) {
         shapes->push_back(shape);
       }
 
