diff options
author | Hans Goudey <h.goudey@me.com> | 2020-11-17 21:00:43 +0300 |
---|---|---|
committer | Hans Goudey <h.goudey@me.com> | 2020-11-17 21:00:43 +0300 |
commit | df2bf164c788c367709351aa5c34d843a3639a9c (patch) | |
tree | 704bc168b257129fcf82c17433fc4479cef51867 /source/blender | |
parent | da6c968a955083dcf9da69f002244f50affee936 (diff) |
Cleanup: Use LISTBASE_FOREACH macro
Diffstat (limited to 'source/blender')
-rw-r--r-- | source/blender/blenkernel/intern/constraint.c | 26 |
1 files changed, 9 insertions, 17 deletions
diff --git a/source/blender/blenkernel/intern/constraint.c b/source/blender/blenkernel/intern/constraint.c index a1b53b721d3..974d536d83e 100644 --- a/source/blender/blenkernel/intern/constraint.c +++ b/source/blender/blenkernel/intern/constraint.c @@ -2263,10 +2263,9 @@ static int pycon_get_tars(bConstraint *con, ListBase *list) static void pycon_id_looper(bConstraint *con, ConstraintIDFunc func, void *userdata) { bPythonConstraint *data = con->data; - bConstraintTarget *ct; /* targets */ - for (ct = data->targets.first; ct; ct = ct->next) { + LISTBASE_FOREACH (bConstraintTarget *, ct, &data->targets) { func(con, (ID **)&ct->tar, false, userdata); } @@ -2381,10 +2380,9 @@ static int armdef_get_tars(bConstraint *con, ListBase *list) static void armdef_id_looper(bConstraint *con, ConstraintIDFunc func, void *userdata) { bArmatureConstraint *data = con->data; - bConstraintTarget *ct; /* Target list. */ - for (ct = data->targets.first; ct; ct = ct->next) { + LISTBASE_FOREACH (bConstraintTarget *, ct, &data->targets) { func(con, (ID **)&ct->tar, false, userdata); } } @@ -5370,10 +5368,8 @@ void BKE_constraint_free_data(bConstraint *con) /* Free all constraints from a constraint-stack */ void BKE_constraints_free_ex(ListBase *list, bool do_id_user) { - bConstraint *con; - /* Free constraint data and also any extra data */ - for (con = list->first; con; con = con->next) { + LISTBASE_FOREACH (bConstraint *, con, list) { BKE_constraint_free_data_ex(con, do_id_user); } @@ -5554,9 +5550,7 @@ bConstraint *BKE_constraint_add_for_object(Object *ob, const char *name, short t /* Run the given callback on all ID-blocks in list of constraints */ void BKE_constraints_id_loop(ListBase *conlist, ConstraintIDFunc func, void *userdata) { - bConstraint *con; - - for (con = conlist->first; con; con = con->next) { + LISTBASE_FOREACH (bConstraint *, con, conlist) { const bConstraintTypeInfo *cti = BKE_constraint_typeinfo_get(con); if (cti) { @@ -5686,11 +5680,10 @@ bConstraint *BKE_constraints_find_name(ListBase *list, const char *name) /* finds the 'active' constraint in a constraint stack */ bConstraint *BKE_constraints_active_get(ListBase *list) { - bConstraint *con; /* search for the first constraint with the 'active' flag set */ if (list) { - for (con = list->first; con; con = con->next) { + LISTBASE_FOREACH (bConstraint *, con, list) { if (con->flag & CONSTRAINT_ACTIVE) { return con; } @@ -5704,15 +5697,14 @@ bConstraint *BKE_constraints_active_get(ListBase *list) /* Set the given constraint as the active one (clearing all the others) */ void BKE_constraints_active_set(ListBase *list, bConstraint *con) { - bConstraint *c; if (list) { - for (c = list->first; c; c = c->next) { - if (c == con) { - c->flag |= CONSTRAINT_ACTIVE; + LISTBASE_FOREACH (bConstraint *, con_iter, list) { + if (con_iter == con) { + con_iter->flag |= CONSTRAINT_ACTIVE; } else { - c->flag &= ~CONSTRAINT_ACTIVE; + con_iter->flag &= ~CONSTRAINT_ACTIVE; } } } |