lesliejaneth / opencollada

Automatically exported from code.google.com/p/opencollada
0 stars 0 forks source link

Material ID's not respected in instance (patch included) #111

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Parsing the mesh at this location
http://graphics.stanford.edu/~danielrh/sportive.dae
will result in the hair material being attached to the body since the materials 
are out of order
likewise this mesh
http://graphics.stanford.edu/~danielrh/reverse.dae
"fixes" the problem by reversing the material library definitions

The patch is not to use a UniqueId for making the mMaterialIdInfo since there's 
really no harm in using the same ID for the same text string, but there is a 
harm in using different ID's for the sam text string.

Original issue reported on code.google.com by danie...@gmail.com on 23 Aug 2010 at 5:22

Attachments:

GoogleCodeExporter commented 8 years ago
Missed this one. Gets applied tomorrow. Thanks!

Original comment by opencollada on 28 Sep 2010 at 6:18

GoogleCodeExporter commented 8 years ago
Has been fixed in rev 769.

However, I have not applied the patch as it was but removed the map of 
GeometryMaterialIdInfo by on global GeometryMaterialIdInfo, which basically 
leads to the same behavior as the provided patch.

Thanks for your effort.

Original comment by opencollada2@googlemail.com on 29 Sep 2010 at 9:06