organicmaps/fast_obj
13
Fork 0
mirror of https://github.com/thisistherk/fast_obj.git synced 2025-04-04 13:05:02 +00:00
Code Issues 1 Projects Releases Wiki Activity Actions

Create materials for 'usemtl' declaration

Browse source 
Problem: I read a bare OBJ so no MTL is provided. I still want
to keep the materials setup across the scene for further edit.

Fix: Instead of using a fallback material (idx 0), create a material
for each not-defined material while keeping the name
so that the structure of the model is conserved.
This commit is contained in:
Aurélien Chatelain 2020-10-19 07:30:47 +00:00
parent 4bc640697f
commit 76143f7ef2
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
fast_obj.h
View file

@ -820,12 +820,6 @@ const char* parse_usemtl(fastObjData* data, const char* ptr)
e = ptr;
/* If there are no materials yet, add a dummy invalid material at index 0 */
if (array_empty(data->mesh->materials))
array_push(data->mesh->materials, mtl_default());
/* Find an existing material with the same name */
idx = 0;
while (idx < array_size(data->mesh->materials))
@ -837,8 +831,14 @@ const char* parse_usemtl(fastObjData* data, const char* ptr)
idx++;
}
/* If doesn't exists, create a default one with this name
Note: this case happens when OBJ doesn't have its MTL */
if (idx == array_size(data->mesh->materials))
idx = 0;
{
fastObjMaterial new_mtl = mtl_default();
new_mtl.name = string_copy(s, e);
array_push(data->mesh->materials, new_mtl);
}
data->material = idx;