diff options
| author | Bastien Montagne <montagne29@wanadoo.fr> | 2018-12-03 18:19:08 +0300 |
|---|---|---|
| committer | Bastien Montagne <montagne29@wanadoo.fr> | 2018-12-03 18:22:27 +0300 |
| commit | 3235a3081c12e99d4bec350eff04b3073c07cdf8 (patch) | |
| tree | c75a7e0edd39f81ca879de8cc2d27c595c6c35fd /source/blender/blenkernel/intern/DerivedMesh.c | |
| parent | 957435785cd04dd68307c390a5063026917e836e (diff) | |
Fix T57858: Add validation callback to CustomData layers.
Our mesh validation was only checking cd layout so far, not their actual
data. While this might only be needed for a few types, this is a
required addition for things like imported UVs, else we have no way to
avoid nasty things like NANs & co.
Note that more layer types may need that callback, time will say. For
now added it to some obvious missing cases...
Diffstat (limited to 'source/blender/blenkernel/intern/DerivedMesh.c')
| -rw-r--r-- | source/blender/blenkernel/intern/DerivedMesh.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/source/blender/blenkernel/intern/DerivedMesh.c b/source/blender/blenkernel/intern/DerivedMesh.c index f0097c46a71..98d2e77c62f 100644 --- a/source/blender/blenkernel/intern/DerivedMesh.c +++ b/source/blender/blenkernel/intern/DerivedMesh.c @@ -2770,10 +2770,10 @@ bool DM_is_valid(DerivedMesh *dm) bool changed = true; is_valid &= BKE_mesh_validate_all_customdata( - dm->getVertDataLayout(dm), - dm->getEdgeDataLayout(dm), - dm->getLoopDataLayout(dm), - dm->getPolyDataLayout(dm), + dm->getVertDataLayout(dm), dm->getNumVerts(dm), + dm->getEdgeDataLayout(dm), dm->getNumEdges(dm), + dm->getLoopDataLayout(dm), dm->getNumLoops(dm), + dm->getPolyDataLayout(dm), dm->getNumPolys(dm), false, /* setting mask here isn't useful, gives false positives */ do_verbose, do_fixes, &changed); |