Cleanup: style, use braces for editors
[blender.git] / source / blender / editors / armature / pose_edit.c
index 9c31b66..cf64210 100644 (file)
@@ -78,7 +78,8 @@ Object *ED_pose_object_from_context(bContext *C)
   ScrArea *sa = CTX_wm_area(C);
   Object *ob;
 
-  /* since this call may also be used from the buttons window, we need to check for where to get the object */
+  /* Since this call may also be used from the buttons window,
+   * we need to check for where to get the object. */
   if (sa && sa->spacetype == SPACE_PROPERTIES) {
     ob = ED_object_context(C);
   }
@@ -262,8 +263,9 @@ static int pose_calculate_paths_invoke(bContext *C, wmOperator *op, const wmEven
 {
   Object *ob = BKE_object_pose_armature_get(CTX_data_active_object(C));
 
-  if (ELEM(NULL, ob, ob->pose))
+  if (ELEM(NULL, ob, ob->pose)) {
     return OPERATOR_CANCELLED;
+  }
 
   /* set default settings from existing/stored settings */
   {
@@ -290,8 +292,9 @@ static int pose_calculate_paths_exec(bContext *C, wmOperator *op)
   Object *ob = BKE_object_pose_armature_get(CTX_data_active_object(C));
   Scene *scene = CTX_data_scene(C);
 
-  if (ELEM(NULL, ob, ob->pose))
+  if (ELEM(NULL, ob, ob->pose)) {
     return OPERATOR_CANCELLED;
+  }
 
   /* grab baking settings from operator settings */
   {
@@ -390,8 +393,9 @@ static int pose_update_paths_exec(bContext *C, wmOperator *UNUSED(op))
   Object *ob = BKE_object_pose_armature_get(CTX_data_active_object(C));
   Scene *scene = CTX_data_scene(C);
 
-  if (ELEM(NULL, ob, scene))
+  if (ELEM(NULL, ob, scene)) {
     return OPERATOR_CANCELLED;
+  }
 
   /* calculate the bones that now have motionpaths... */
   /* TODO: only make for the selected bones? */
@@ -426,8 +430,9 @@ static void ED_pose_clear_paths(Object *ob, bool only_selected)
   bPoseChannel *pchan;
   bool skipped = false;
 
-  if (ELEM(NULL, ob, ob->pose))
+  if (ELEM(NULL, ob, ob->pose)) {
     return;
+  }
 
   /* free the motionpath blocks for all bones - This is easier for users to quickly clear all */
   for (pchan = ob->pose->chanbase.first; pchan; pchan = pchan->next) {
@@ -443,8 +448,9 @@ static void ED_pose_clear_paths(Object *ob, bool only_selected)
   }
 
   /* if nothing was skipped, there should be no paths left! */
-  if (skipped == false)
+  if (skipped == false) {
     ob->pose->avs.path_bakeflag &= ~MOTIONPATH_BAKE_HAS_PATHS;
+  }
 
   /* tag armature object for copy on write - so removed paths don't still show */
   DEG_id_tag_update(&ob->id, ID_RECALC_COPY_ON_WRITE);
@@ -457,8 +463,9 @@ static int pose_clear_paths_exec(bContext *C, wmOperator *op)
   bool only_selected = RNA_boolean_get(op->ptr, "only_selected");
 
   /* only continue if there's an object */
-  if (ELEM(NULL, ob, ob->pose))
+  if (ELEM(NULL, ob, ob->pose)) {
     return OPERATOR_CANCELLED;
+  }
 
   /* use the backend function for this */
   ED_pose_clear_paths(ob, only_selected);
@@ -734,8 +741,9 @@ static int pose_armature_layers_showall_exec(bContext *C, wmOperator *op)
   int i;
 
   /* sanity checking */
-  if (arm == NULL)
+  if (arm == NULL) {
     return OPERATOR_CANCELLED;
+  }
 
   /* use RNA to set the layers
    * although it would be faster to just set directly using bitflags, we still
@@ -743,8 +751,9 @@ static int pose_armature_layers_showall_exec(bContext *C, wmOperator *op)
    */
   RNA_id_pointer_create(&arm->id, &ptr);
 
-  for (i = 0; i < maxLayers; i++)
+  for (i = 0; i < maxLayers; i++) {
     layers[i] = 1;
+  }
 
   RNA_boolean_set_array(&ptr, "layers", layers);
 
@@ -787,10 +796,12 @@ static int armature_layers_invoke(bContext *C, wmOperator *op, const wmEvent *ev
   bool layers[32];
 
   /* sanity checking */
-  if (arm == NULL)
+  if (arm == NULL) {
     return OPERATOR_CANCELLED;
+  }
 
-  /* get RNA pointer to armature data to use that to retrieve the layers as ints to init the operator */
+  /* Get RNA pointer to armature data to use that to retrieve the layers as ints
+   * to init the operator. */
   RNA_id_pointer_create((ID *)arm, &ptr);
   RNA_boolean_get_array(&ptr, "layers", layers);
   RNA_boolean_set_array(op->ptr, "layers", layers);