Cleanup: remove useless `DM_ensure_looptri()`.
That one was doing exactly same thing as `dm->getLoopTriArray()`, no point in having twice the same code here...
This commit is contained in:
parent
9cc7e32f39
commit
7c3b435c3a
|
@ -625,7 +625,6 @@ void DM_ensure_normals(DerivedMesh *dm);
|
|||
void DM_ensure_tessface(DerivedMesh *dm);
|
||||
|
||||
void DM_ensure_looptri_data(DerivedMesh *dm);
|
||||
void DM_ensure_looptri(DerivedMesh *dm);
|
||||
void DM_verttri_from_looptri(MVertTri *verttri, const MLoop *mloop, const MLoopTri *looptri, int looptri_num);
|
||||
|
||||
void DM_update_tessface_data(DerivedMesh *dm);
|
||||
|
|
|
@ -238,7 +238,7 @@ static int dm_getNumLoopTri(DerivedMesh *dm)
|
|||
static const MLoopTri *dm_getLoopTriArray(DerivedMesh *dm)
|
||||
{
|
||||
if (dm->looptris.array) {
|
||||
BLI_assert(poly_to_tri_count(dm->numPolyData, dm->numLoopData) == dm->looptris.num);
|
||||
BLI_assert(dm->getNumLoopTri(dm) == dm->looptris.num);
|
||||
}
|
||||
else {
|
||||
dm->recalcLoopTri(dm);
|
||||
|
@ -510,19 +510,6 @@ void DM_ensure_looptri_data(DerivedMesh *dm)
|
|||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* The purpose of this function is that we can call:
|
||||
* `dm->getLoopTriArray(dm)` and get the array returned.
|
||||
*/
|
||||
void DM_ensure_looptri(DerivedMesh *dm)
|
||||
{
|
||||
const int numPolys = dm->getNumPolys(dm);
|
||||
|
||||
if ((dm->looptris.num == 0) && (numPolys != 0)) {
|
||||
dm->recalcLoopTri(dm);
|
||||
}
|
||||
}
|
||||
|
||||
void DM_verttri_from_looptri(MVertTri *verttri, const MLoop *mloop, const MLoopTri *looptri, int looptri_num)
|
||||
{
|
||||
int i;
|
||||
|
@ -2214,7 +2201,6 @@ static void mesh_calc_modifiers(
|
|||
if (dataMask & CD_MASK_MFACE) {
|
||||
DM_ensure_tessface(finaldm);
|
||||
}
|
||||
DM_ensure_looptri(finaldm);
|
||||
|
||||
/* without this, drawing ngon tri's faces will show ugly tessellated face
|
||||
* normals and will also have to calculate normals on the fly, try avoid
|
||||
|
|
|
@ -807,7 +807,6 @@ static int cloth_from_object(Object *ob, ClothModifierData *clmd, DerivedMesh *d
|
|||
if ( !dm )
|
||||
return 0;
|
||||
|
||||
DM_ensure_looptri(dm);
|
||||
cloth_from_mesh ( clmd, dm );
|
||||
|
||||
// create springs
|
||||
|
|
|
@ -2060,9 +2060,6 @@ DerivedMesh *dynamicPaint_Modifier_do(DynamicPaintModifierData *pmd, Scene *scen
|
|||
if (pmd->canvas) {
|
||||
DerivedMesh *ret;
|
||||
|
||||
/* For now generate looptris in every case */
|
||||
DM_ensure_looptri(dm);
|
||||
|
||||
/* Update canvas data for a new frame */
|
||||
dynamicPaint_frameUpdate(pmd, scene, ob, dm);
|
||||
|
||||
|
@ -2072,9 +2069,6 @@ DerivedMesh *dynamicPaint_Modifier_do(DynamicPaintModifierData *pmd, Scene *scen
|
|||
return ret;
|
||||
}
|
||||
else {
|
||||
/* For now generate looptris in every case */
|
||||
DM_ensure_looptri(dm);
|
||||
|
||||
/* Update canvas data for a new frame */
|
||||
dynamicPaint_frameUpdate(pmd, scene, ob, dm);
|
||||
|
||||
|
|
|
@ -80,8 +80,6 @@ void initElbeemMesh(struct Scene *scene, struct Object *ob,
|
|||
|
||||
dm = mesh_create_derived_index_render(scene, ob, CD_MASK_BAREMESH, modifierIndex);
|
||||
|
||||
DM_ensure_looptri(dm);
|
||||
|
||||
mvert = dm->getVertArray(dm);
|
||||
mloop = dm->getLoopArray(dm);
|
||||
looptri = dm->getLoopTriArray(dm);
|
||||
|
|
|
@ -1373,7 +1373,6 @@ void BKE_mesh_remap_calc_loops_from_dm(
|
|||
if (dirty_tess_flag) {
|
||||
dm_src->dirty &= ~dirty_tess_flag;
|
||||
}
|
||||
DM_ensure_looptri(dm_src);
|
||||
if (dirty_tess_flag) {
|
||||
dm_src->dirty |= dirty_tess_flag;
|
||||
}
|
||||
|
|
|
@ -291,8 +291,6 @@ static rbCollisionShape *rigidbody_get_shape_trimesh_from_mesh(Object *ob)
|
|||
if (dm == NULL)
|
||||
return NULL;
|
||||
|
||||
DM_ensure_looptri(dm);
|
||||
|
||||
mvert = dm->getVertArray(dm);
|
||||
totvert = dm->getNumVerts(dm);
|
||||
looptri = dm->getLoopTriArray(dm);
|
||||
|
@ -524,8 +522,6 @@ void BKE_rigidbody_calc_volume(Object *ob, float *r_vol)
|
|||
if (dm == NULL)
|
||||
return;
|
||||
|
||||
DM_ensure_looptri(dm);
|
||||
|
||||
mvert = dm->getVertArray(dm);
|
||||
totvert = dm->getNumVerts(dm);
|
||||
lt = dm->getLoopTriArray(dm);
|
||||
|
@ -609,8 +605,6 @@ void BKE_rigidbody_calc_center_of_mass(Object *ob, float r_center[3])
|
|||
if (dm == NULL)
|
||||
return;
|
||||
|
||||
DM_ensure_looptri(dm);
|
||||
|
||||
mvert = dm->getVertArray(dm);
|
||||
totvert = dm->getNumVerts(dm);
|
||||
looptri = dm->getLoopTriArray(dm);
|
||||
|
|
|
@ -2692,7 +2692,6 @@ static void smokeModifier_process(SmokeModifierData *smd, Scene *scene, Object *
|
|||
|
||||
if (smd->flow->dm) smd->flow->dm->release(smd->flow->dm);
|
||||
smd->flow->dm = CDDM_copy(dm);
|
||||
DM_ensure_looptri(smd->flow->dm);
|
||||
|
||||
if (scene->r.cfra > smd->time)
|
||||
{
|
||||
|
@ -2715,7 +2714,6 @@ static void smokeModifier_process(SmokeModifierData *smd, Scene *scene, Object *
|
|||
smd->coll->dm->release(smd->coll->dm);
|
||||
|
||||
smd->coll->dm = CDDM_copy(dm);
|
||||
DM_ensure_looptri(smd->coll->dm);
|
||||
}
|
||||
|
||||
smd->time = scene->r.cfra;
|
||||
|
|
|
@ -424,7 +424,6 @@ static bool PE_create_shape_tree(PEData *data, Object *shapeob)
|
|||
return false;
|
||||
}
|
||||
|
||||
DM_ensure_looptri(dm);
|
||||
return (bvhtree_from_mesh_looptri(&data->shape_bvh, dm, 0.0f, 4, 8) != NULL);
|
||||
}
|
||||
|
||||
|
|
|
@ -153,8 +153,6 @@ static void deformVerts(ModifierData *md, Object *ob,
|
|||
collmd->current_v = MEM_dupallocN(collmd->x); // inter-frame
|
||||
|
||||
collmd->mvert_num = mvert_num;
|
||||
|
||||
DM_ensure_looptri(dm);
|
||||
|
||||
collmd->tri_num = dm->getNumLoopTri(dm);
|
||||
{
|
||||
|
|
|
@ -539,7 +539,7 @@ static void initSystem(LaplacianDeformModifierData *lmd, Object *ob, DerivedMesh
|
|||
STACK_PUSH(index_anchors, i);
|
||||
}
|
||||
}
|
||||
DM_ensure_looptri(dm);
|
||||
|
||||
total_anchors = STACK_SIZE(index_anchors);
|
||||
lmd->cache_system = initLaplacianSystem(numVerts, dm->getNumEdges(dm), dm->getNumLoopTri(dm),
|
||||
total_anchors, lmd->anchor_grp_name, lmd->repeat);
|
||||
|
|
|
@ -1156,7 +1156,6 @@ static PyObject *C_BVHTree_FromObject(PyObject *UNUSED(cls), PyObject *args, PyO
|
|||
|
||||
/* Get data for tessellation */
|
||||
{
|
||||
DM_ensure_looptri(dm);
|
||||
lt = dm->getLoopTriArray(dm);
|
||||
|
||||
tris_len = (unsigned int)dm->getNumLoopTri(dm);
|
||||
|
|
Loading…
Reference in New Issue