Code cleanup: use r_ prefix for return args
[blender.git] / source / blender / editors / object / object_constraint.c
index 1bed7d7dd11c05b7fb969d20c22bc9dd0f6557d2..1e7e543ccb8c44cce72168a3955358098046a148 100644 (file)
@@ -105,10 +105,10 @@ ListBase *get_active_constraints(Object *ob)
 }
 
 /* Find the list that a given constraint belongs to, and/or also get the posechannel this is from (if applicable) */
-ListBase *get_constraint_lb(Object *ob, bConstraint *con, bPoseChannel **pchan_r)
+ListBase *get_constraint_lb(Object *ob, bConstraint *con, bPoseChannel **r_pchan)
 {
-       if (pchan_r)
-               *pchan_r = NULL;
+       if (r_pchan)
+               *r_pchan = NULL;
        
        if (ELEM(NULL, ob, con))
                return NULL;
@@ -128,8 +128,8 @@ ListBase *get_constraint_lb(Object *ob, bConstraint *con, bPoseChannel **pchan_r
                for (pchan = ob->pose->chanbase.first; pchan; pchan = pchan->next) {
                        if ((BLI_findindex(&pchan->constraints, con) != -1)) {
                                
-                               if (pchan_r)
-                                       *pchan_r = pchan;
+                               if (r_pchan)
+                                       *r_pchan = pchan;
                                
                                return &pchan->constraints;
                        }
@@ -435,7 +435,9 @@ static void test_constraints(Object *owner, bPoseChannel *pchan)
                                                                curcon->flag |= CONSTRAINT_DISABLE;
                                                }
                                        }
-                                       else curcon->flag |= CONSTRAINT_DISABLE;
+                                       else {
+                                               curcon->flag |= CONSTRAINT_DISABLE;
+                                       }
                                }
                        }
                        else if (curcon->type == CONSTRAINT_TYPE_CAMERASOLVER) {
@@ -540,7 +542,7 @@ static int edit_constraint_poll_generic(bContext *C, StructRNA *rna_type)
        PointerRNA ptr = CTX_data_pointer_get_type(C, "constraint", rna_type);
        Object *ob = (ptr.id.data) ? ptr.id.data : ED_object_active_context(C);
 
-       if (!ob || ob->id.lib) return 0;
+       if (!ptr.data || !ob || ob->id.lib) return 0;
        if (ptr.id.data && ((ID *)ptr.id.data)->lib) return 0;
 
        return 1;
@@ -553,7 +555,7 @@ static int edit_constraint_poll(bContext *C)
 
 static void edit_constraint_properties(wmOperatorType *ot)
 {
-       RNA_def_string(ot->srna, "constraint", "", MAX_NAME, "Constraint", "Name of the constraint to edit");
+       RNA_def_string(ot->srna, "constraint", NULL, MAX_NAME, "Constraint", "Name of the constraint to edit");
        RNA_def_enum(ot->srna, "owner", constraint_owner_items, 0, "Owner", "The owner of this constraint");
 }
 
@@ -602,7 +604,7 @@ static bConstraint *edit_constraint_property_get(wmOperator *op, Object *ob, int
                        list = &pchan->constraints;
                else {
                        //if (G.debug & G_DEBUG)
-                       //printf("edit_constraint_property_get: No active bone for object '%s'\n", (ob)? ob->id.name+2 : "<None>");
+                       //printf("edit_constraint_property_get: No active bone for object '%s'\n", (ob) ? ob->id.name + 2 : "<None>");
                        return NULL;
                }
        }
@@ -614,7 +616,7 @@ static bConstraint *edit_constraint_property_get(wmOperator *op, Object *ob, int
        
        con = BKE_constraints_findByName(list, constraint_name);
        //if (G.debug & G_DEBUG)
-       //printf("constraint found = %p, %s\n", (void *)con, (con)?con->name:"<Not found>");
+       //printf("constraint found = %p, %s\n", (void *)con, (con) ? con->name : "<Not found>");
 
        if (con && (type != 0) && (con->type != type))
                con = NULL;
@@ -645,7 +647,7 @@ static int stretchto_reset_exec(bContext *C, wmOperator *op)
        return OPERATOR_FINISHED;
 }
 
-static int stretchto_reset_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int stretchto_reset_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        if (edit_constraint_invoke_properties(C, op))
                return stretchto_reset_exec(C, op);
@@ -691,7 +693,7 @@ static int limitdistance_reset_exec(bContext *C, wmOperator *op)
        return OPERATOR_FINISHED;
 }
 
-static int limitdistance_reset_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int limitdistance_reset_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        if (edit_constraint_invoke_properties(C, op))
                return limitdistance_reset_exec(C, op);
@@ -771,7 +773,7 @@ static void child_get_inverse_matrix(Scene *scene, Object *ob, bConstraint *con,
                         * the effect of the constraint
                         */
                        invert_m4_m4(imat, pchan->pose_mat);
-                       mult_m4_m4m4(tmat, pmat, imat);
+                       mul_m4_m4m4(tmat, pmat, imat);
                        invert_m4_m4(invmat, tmat);
 
                        /* 5. restore constraints */
@@ -827,7 +829,7 @@ static int childof_set_inverse_exec(bContext *C, wmOperator *op)
        return OPERATOR_FINISHED;
 }
 
-static int childof_set_inverse_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int childof_set_inverse_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        if (edit_constraint_invoke_properties(C, op))
                return childof_set_inverse_exec(C, op);
@@ -874,7 +876,7 @@ static int childof_clear_inverse_exec(bContext *C, wmOperator *op)
        return OPERATOR_FINISHED;
 }
 
-static int childof_clear_inverse_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int childof_clear_inverse_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        if (edit_constraint_invoke_properties(C, op))
                return childof_clear_inverse_exec(C, op);
@@ -925,8 +927,8 @@ static int followpath_path_animate_exec(bContext *C, wmOperator *op)
        if (data->tar) {
                Curve *cu = (Curve *)data->tar->data;
                
-               if ( ELEM(NULL, cu->adt, cu->adt->action) ||
-                       (list_find_fcurve(&cu->adt->action->curves, "eval_time", 0) == NULL))
+               if (ELEM(NULL, cu->adt, cu->adt->action) ||
+                   (list_find_fcurve(&cu->adt->action->curves, "eval_time", 0) == NULL))
                {
                        /* create F-Curve for path animation */
                        act = verify_adt_action(&cu->id, 1);
@@ -991,7 +993,7 @@ static int followpath_path_animate_exec(bContext *C, wmOperator *op)
        return OPERATOR_FINISHED;
 }
 
-static int followpath_path_animate_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int followpath_path_animate_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        /* hook up invoke properties for figuring out which constraint we're dealing with */
        if (edit_constraint_invoke_properties(C, op)) {
@@ -1049,7 +1051,7 @@ static int objectsolver_set_inverse_exec(bContext *C, wmOperator *op)
        return OPERATOR_FINISHED;
 }
 
-static int objectsolver_set_inverse_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int objectsolver_set_inverse_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        if (edit_constraint_invoke_properties(C, op))
                return objectsolver_set_inverse_exec(C, op);
@@ -1095,7 +1097,7 @@ static int objectsolver_clear_inverse_exec(bContext *C, wmOperator *op)
        return OPERATOR_FINISHED;
 }
 
-static int objectsolver_clear_inverse_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int objectsolver_clear_inverse_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        if (edit_constraint_invoke_properties(C, op))
                return objectsolver_clear_inverse_exec(C, op);
@@ -1169,7 +1171,7 @@ static int constraint_delete_exec(bContext *C, wmOperator *UNUSED(op))
        Object *ob = ptr.id.data;
        bConstraint *con = ptr.data;
        ListBase *lb = get_constraint_lb(ob, con, NULL);
-       const short is_ik = ELEM(con->type, CONSTRAINT_TYPE_KINEMATIC, CONSTRAINT_TYPE_SPLINEIK);
+       const bool is_ik = ELEM(con->type, CONSTRAINT_TYPE_KINEMATIC, CONSTRAINT_TYPE_SPLINEIK);
 
        /* free the constraint */
        if (BKE_remove_constraint(lb, con)) {
@@ -1231,7 +1233,7 @@ static int constraint_move_down_exec(bContext *C, wmOperator *op)
        return OPERATOR_CANCELLED;
 }
 
-static int constraint_move_down_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int constraint_move_down_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        if (edit_constraint_invoke_properties(C, op))
                return constraint_move_down_exec(C, op);
@@ -1280,7 +1282,7 @@ static int constraint_move_up_exec(bContext *C, wmOperator *op)
        return OPERATOR_CANCELLED;
 }
 
-static int constraint_move_up_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(event))
+static int constraint_move_up_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        if (edit_constraint_invoke_properties(C, op))
                return constraint_move_up_exec(C, op);
@@ -1329,7 +1331,7 @@ static int pose_constraints_clear_exec(bContext *C, wmOperator *UNUSED(op))
 
        /* do updates */
        DAG_id_tag_update(&ob->id, OB_RECALC_DATA);
-       WM_event_add_notifier(C, NC_OBJECT | ND_CONSTRAINT, ob);
+       WM_event_add_notifier(C, NC_OBJECT | ND_CONSTRAINT | NA_REMOVED, ob);
        
        return OPERATOR_FINISHED;
 }
@@ -1363,7 +1365,7 @@ static int object_constraints_clear_exec(bContext *C, wmOperator *UNUSED(op))
        DAG_relations_tag_update(bmain);
        
        /* do updates */
-       WM_event_add_notifier(C, NC_OBJECT | ND_CONSTRAINT, NULL);
+       WM_event_add_notifier(C, NC_OBJECT | ND_CONSTRAINT | NA_REMOVED, NULL);
        
        return OPERATOR_FINISHED;
 }
@@ -1570,12 +1572,13 @@ static short get_new_constraint_target(bContext *C, int con_type, Object **tar_o
        
        /* if still not found, add a new empty to act as a target (if allowed) */
        if ((found == 0) && (add)) {
+               Main *bmain = CTX_data_main(C);
                Scene *scene = CTX_data_scene(C);
                Base *base = BASACT, *newbase = NULL;
                Object *obt;
                
                /* add new target object */
-               obt = BKE_object_add(scene, OB_EMPTY);
+               obt = BKE_object_add(bmain, scene, OB_EMPTY);
                
                /* set layers OK */
                newbase = BASACT;
@@ -1610,7 +1613,7 @@ static short get_new_constraint_target(bContext *C, int con_type, Object **tar_o
 }
 
 /* used by add constraint operators to add the constraint required */
-static int constraint_add_exec(bContext *C, wmOperator *op, Object *ob, ListBase *list, int type, short setTarget)
+static int constraint_add_exec(bContext *C, wmOperator *op, Object *ob, ListBase *list, int type, const bool setTarget)
 {
        Main *bmain = CTX_data_main(C);
        bPoseChannel *pchan;
@@ -1713,7 +1716,7 @@ static int constraint_add_exec(bContext *C, wmOperator *op, Object *ob, ListBase
                DAG_id_tag_update(&ob->id, OB_RECALC_DATA | OB_RECALC_OB);
        }
        else
-               DAG_id_tag_update(&ob->id, OB_RECALC_DATA);
+               DAG_id_tag_update(&ob->id, OB_RECALC_OB);
        
        /* notifiers for updates */
        WM_event_add_notifier(C, NC_OBJECT | ND_CONSTRAINT | NA_ADDED, ob);
@@ -1848,7 +1851,7 @@ void POSE_OT_constraint_add_with_targets(wmOperatorType *ot)
 // TODO: should these be here, or back in editors/armature/poseobject.c again?
 
 /* present menu with options + validation for targets to use */
-static int pose_ik_add_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(evt))
+static int pose_ik_add_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
 {
        Object *ob = BKE_object_pose_armature_get(CTX_data_active_object(C));
        bPoseChannel *pchan = BKE_pose_channel_active(ob);
@@ -1904,7 +1907,7 @@ static int pose_ik_add_invoke(bContext *C, wmOperator *op, wmEvent *UNUSED(evt))
 static int pose_ik_add_exec(bContext *C, wmOperator *op)
 {
        Object *ob = CTX_data_active_object(C);
-       int with_targets = RNA_boolean_get(op->ptr, "with_targets");
+       const bool with_targets = RNA_boolean_get(op->ptr, "with_targets");
        
        /* add the constraint - all necessary checks should have been done by the invoke() callback already... */
        return constraint_add_exec(C, op, ob, get_active_constraints(ob), CONSTRAINT_TYPE_KINEMATIC, with_targets);