Replace MFace Usage by MLoopTri in dynamic_paint
AbandonedPublic
Actions

Authored by Campbell Barton (campbellbarton) on Jul 21 2015, 6:03 PM.

Details

Reviewers

Philipp Oeser (lichtwerk)
Antonis Ryakiotakis (psy-fi)

Summary

This patch replaces MFaces by MLoopTris in dynamic_paint
Subtask of T45474: Deprecate legacy MFace data storage

Diff Detail

Event Timeline

Philipp Oeser (lichtwerk) updated this revision to Diff 4725.Jul 21 2015, 6:03 PM

Philipp Oeser (lichtwerk) retitled this revision from to Replace MFace Usage by MLoopTri in dynamic_paint.

Philipp Oeser (lichtwerk) updated this object.

Philipp Oeser (lichtwerk) added reviewers: Campbell Barton (campbellbarton), Antonis Ryakiotakis (psy-fi).

Philipp Oeser (lichtwerk) added a project: BF Blender: Next.

I still havent touched the 2 occasions where MFace is in concert with BVHTree here (see comments in code).
I am not tooo familiar with dynamic paint but from my (limited) tests [tried to use most available options once], everything seems to be working as before [master]

now that we have "bvhtree_from_mesh_looptri" --> will replace the last occurences of MFace as well...

replaced the last occurences of MFace here (thanks to the new looptri funcs for the BVHTree)
had to replace "RE_sample_material_color" as well (and made a looptri version "RE_sample_material_color_looptri")

Campbell Barton (campbellbarton) mentioned this in rBe48c4d73d378: Replace MFace with looptri for dynamicpaint.Jul 27 2015, 8:07 AM