From 0b837a49861c4d4a413ba282124ecd8a6a2efd79 Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Thu, 8 Nov 2018 08:02:02 +1100 Subject: Cleanup: style, use const args --- source/blender/blenkernel/intern/mball.c | 77 +++++++++++++------------------- 1 file changed, 31 insertions(+), 46 deletions(-) (limited to 'source/blender/blenkernel/intern/mball.c') diff --git a/source/blender/blenkernel/intern/mball.c b/source/blender/blenkernel/intern/mball.c index 4c634ed9140..63adcfc3cae 100644 --- a/source/blender/blenkernel/intern/mball.c +++ b/source/blender/blenkernel/intern/mball.c @@ -66,8 +66,6 @@ #include "BKE_object.h" #include "BKE_material.h" -//#include "DEG_depsgraph.h" - /* Functions */ /** Free (or release) any data used by this mball (does not free the mball itself). */ @@ -443,17 +441,17 @@ Object *BKE_mball_basis_find(Scene *scene, Object *basis) return basis; } -bool BKE_mball_minmax_ex(MetaBall *mb, float min[3], float max[3], - float obmat[4][4], const short flag) +bool BKE_mball_minmax_ex( + const MetaBall *mb, float min[3], float max[3], + const float obmat[4][4], const short flag) { const float scale = obmat ? mat4_to_scale(obmat) : 1.0f; - MetaElem *ml; bool changed = false; float centroid[3], vec[3]; INIT_MINMAX(min, max); - for (ml = mb->elems.first; ml; ml = ml->next) { + for (const MetaElem *ml = mb->elems.first; ml; ml = ml->next) { if ((ml->flag & flag) == flag) { const float scale_mb = (ml->rad * 0.5f) * scale; int i; @@ -480,27 +478,24 @@ bool BKE_mball_minmax_ex(MetaBall *mb, float min[3], float max[3], /* basic vertex data functions */ -bool BKE_mball_minmax(MetaBall *mb, float min[3], float max[3]) +bool BKE_mball_minmax(const MetaBall *mb, float min[3], float max[3]) { - MetaElem *ml; - INIT_MINMAX(min, max); - for (ml = mb->elems.first; ml; ml = ml->next) { + for (const MetaElem *ml = mb->elems.first; ml; ml = ml->next) { minmax_v3v3_v3(min, max, &ml->x); } return (BLI_listbase_is_empty(&mb->elems) == false); } -bool BKE_mball_center_median(MetaBall *mb, float r_cent[3]) +bool BKE_mball_center_median(const MetaBall *mb, float r_cent[3]) { - MetaElem *ml; int total = 0; zero_v3(r_cent); - for (ml = mb->elems.first; ml; ml = ml->next) { + for (const MetaElem *ml = mb->elems.first; ml; ml = ml->next) { add_v3_v3(r_cent, &ml->x); total++; } @@ -512,7 +507,7 @@ bool BKE_mball_center_median(MetaBall *mb, float r_cent[3]) return (total != 0); } -bool BKE_mball_center_bounds(MetaBall *mb, float r_cent[3]) +bool BKE_mball_center_bounds(const MetaBall *mb, float r_cent[3]) { float min[3], max[3]; @@ -526,26 +521,25 @@ bool BKE_mball_center_bounds(MetaBall *mb, float r_cent[3]) void BKE_mball_transform(MetaBall *mb, float mat[4][4], const bool do_props) { - MetaElem *me; float quat[4]; const float scale = mat4_to_scale(mat); const float scale_sqrt = sqrtf(scale); mat4_to_quat(quat, mat); - for (me = mb->elems.first; me; me = me->next) { - mul_m4_v3(mat, &me->x); - mul_qt_qtqt(me->quat, quat, me->quat); + for (MetaElem *ml = mb->elems.first; ml; ml = ml->next) { + mul_m4_v3(mat, &ml->x); + mul_qt_qtqt(ml->quat, quat, ml->quat); if (do_props) { - me->rad *= scale; + ml->rad *= scale; /* hrmf, probably elems shouldn't be * treating scale differently - campbell */ - if (!MB_TYPE_SIZE_SQUARED(me->type)) { - mul_v3_fl(&me->expx, scale); + if (!MB_TYPE_SIZE_SQUARED(ml->type)) { + mul_v3_fl(&ml->expx, scale); } else { - mul_v3_fl(&me->expx, scale_sqrt); + mul_v3_fl(&ml->expx, scale_sqrt); } } } @@ -553,19 +547,16 @@ void BKE_mball_transform(MetaBall *mb, float mat[4][4], const bool do_props) void BKE_mball_translate(MetaBall *mb, const float offset[3]) { - MetaElem *ml; - - for (ml = mb->elems.first; ml; ml = ml->next) { + for (MetaElem *ml = mb->elems.first; ml; ml = ml->next) { add_v3_v3(&ml->x, offset); } } /* *** select funcs *** */ -int BKE_mball_select_count(struct MetaBall *mb) { +int BKE_mball_select_count(const MetaBall *mb) +{ int sel = 0; - MetaElem *ml; - - for (ml = mb->editelems->first; ml; ml = ml->next) { + for (const MetaElem *ml = mb->editelems->first; ml; ml = ml->next) { if (ml->flag & SELECT) { sel++; } @@ -573,22 +564,20 @@ int BKE_mball_select_count(struct MetaBall *mb) { return sel; } -int BKE_mball_select_count_multi(Object **objects, int objects_len) { - +int BKE_mball_select_count_multi(Object **objects, int objects_len) +{ int sel = 0; for (uint ob_index = 0; ob_index < objects_len; ob_index++) { - Object *obedit = objects[ob_index]; - MetaBall *mb = (MetaBall *)obedit->data; + const Object *obedit = objects[ob_index]; + const MetaBall *mb = (MetaBall *)obedit->data; sel += BKE_mball_select_count(mb); } return sel; } -void BKE_mball_select_all(struct MetaBall *mb) +void BKE_mball_select_all(MetaBall *mb) { - MetaElem *ml; - - for (ml = mb->editelems->first; ml; ml = ml->next) { + for (MetaElem *ml = mb->editelems->first; ml; ml = ml->next) { ml->flag |= SELECT; } } @@ -597,16 +586,14 @@ void BKE_mball_select_all_multi(Object **objects, int objects_len) { for (uint ob_index = 0; ob_index < objects_len; ob_index++) { Object *obedit = objects[ob_index]; - MetaBall *mb = (MetaBall *)obedit->data; + MetaBall *mb = obedit->data; BKE_mball_select_all(mb); } } void BKE_mball_deselect_all(MetaBall *mb) { - MetaElem *ml; - - for (ml = mb->editelems->first; ml; ml = ml->next) { + for (MetaElem *ml = mb->editelems->first; ml; ml = ml->next) { ml->flag &= ~SELECT; } } @@ -615,17 +602,15 @@ void BKE_mball_deselect_all_multi(Object **objects, int objects_len) { for (uint ob_index = 0; ob_index < objects_len; ob_index++) { Object *obedit = objects[ob_index]; - MetaBall *mb = (MetaBall *)obedit->data; + MetaBall *mb = obedit->data; BKE_mball_deselect_all(mb); } } -void BKE_mball_select_swap(struct MetaBall *mb) +void BKE_mball_select_swap(MetaBall *mb) { - MetaElem *ml; - - for (ml = mb->editelems->first; ml; ml = ml->next) { + for (MetaElem *ml = mb->editelems->first; ml; ml = ml->next) { ml->flag ^= SELECT; } } -- cgit v1.2.3