ClangFormat: apply to source, most of intern

Apply clang format as proposed in T53211.

For details on usage and instructions for migrating branches
without conflicts, see:

https://wiki.blender.org/wiki/Tools/ClangFormat

1 files changed, 279 insertions, 259 deletions

diff --git a/source/blender/editors/interface/interface_anim.c b/source/blender/editors/interface/interface_anim.c
index a4476b4d2f1..b9de504f3b2 100644
--- a/source/blender/editors/interface/interface_anim.c
+++ b/source/blender/editors/interface/interface_anim.c
@@ -52,83 +52,85 @@
 
 #include "interface_intern.h"
 
-static FCurve *ui_but_get_fcurve(uiBut *but, AnimData **adt, bAction **action, bool *r_driven, bool *r_special)
+static FCurve *ui_but_get_fcurve(
+    uiBut *but, AnimData **adt, bAction **action, bool *r_driven, bool *r_special)
 {
-	/* for entire array buttons we check the first component, it's not perfect
-	 * but works well enough in typical cases */
-	int rnaindex = (but->rnaindex == -1) ? 0 : but->rnaindex;
+  /* for entire array buttons we check the first component, it's not perfect
+   * but works well enough in typical cases */
+  int rnaindex = (but->rnaindex == -1) ? 0 : but->rnaindex;
 
-	return rna_get_fcurve_context_ui(but->block->evil_C, &but->rnapoin, but->rnaprop, rnaindex, adt, action, r_driven, r_special);
+  return rna_get_fcurve_context_ui(
+      but->block->evil_C, &but->rnapoin, but->rnaprop, rnaindex, adt, action, r_driven, r_special);
 }
 
 void ui_but_anim_flag(uiBut *but, float cfra)
 {
-	AnimData *adt;
-	bAction *act;
-	FCurve *fcu;
-	bool driven;
-	bool special;
-
-	but->flag &= ~(UI_BUT_ANIMATED | UI_BUT_ANIMATED_KEY | UI_BUT_DRIVEN);
-	but->drawflag &= ~UI_BUT_ANIMATED_CHANGED;
-
-	/* NOTE: "special" is reserved for special F-Curves stored on the animation data
-	 *        itself (which are used to animate properties of the animation data).
-	 *        We count those as "animated" too for now
-	 */
-	fcu = ui_but_get_fcurve(but, &adt, &act, &driven, &special);
-
-	if (fcu) {
-		if (!driven) {
-			but->flag |= UI_BUT_ANIMATED;
-
-			/* T41525 - When the active action is a NLA strip being edited,
-			 * we need to correct the frame number to "look inside" the
-			 * remapped action
-			 */
-			if (adt) {
-				cfra = BKE_nla_tweakedit_remap(adt, cfra, NLATIME_CONVERT_UNMAP);
-			}
-
-			if (fcurve_frame_has_keyframe(fcu, cfra, 0)) {
-				but->flag |= UI_BUT_ANIMATED_KEY;
-			}
-
-			/* XXX: this feature is totally broken and useless with NLA */
-			if (adt == NULL || adt->nla_tracks.first == NULL) {
-				if (fcurve_is_changed(but->rnapoin, but->rnaprop, fcu, cfra)) {
-					but->drawflag |= UI_BUT_ANIMATED_CHANGED;
-				}
-			}
-		}
-		else {
-			but->flag |= UI_BUT_DRIVEN;
-		}
-	}
+  AnimData *adt;
+  bAction *act;
+  FCurve *fcu;
+  bool driven;
+  bool special;
+
+  but->flag &= ~(UI_BUT_ANIMATED | UI_BUT_ANIMATED_KEY | UI_BUT_DRIVEN);
+  but->drawflag &= ~UI_BUT_ANIMATED_CHANGED;
+
+  /* NOTE: "special" is reserved for special F-Curves stored on the animation data
+   *        itself (which are used to animate properties of the animation data).
+   *        We count those as "animated" too for now
+   */
+  fcu = ui_but_get_fcurve(but, &adt, &act, &driven, &special);
+
+  if (fcu) {
+    if (!driven) {
+      but->flag |= UI_BUT_ANIMATED;
+
+      /* T41525 - When the active action is a NLA strip being edited,
+       * we need to correct the frame number to "look inside" the
+       * remapped action
+       */
+      if (adt) {
+        cfra = BKE_nla_tweakedit_remap(adt, cfra, NLATIME_CONVERT_UNMAP);
+      }
+
+      if (fcurve_frame_has_keyframe(fcu, cfra, 0)) {
+        but->flag |= UI_BUT_ANIMATED_KEY;
+      }
+
+      /* XXX: this feature is totally broken and useless with NLA */
+      if (adt == NULL || adt->nla_tracks.first == NULL) {
+        if (fcurve_is_changed(but->rnapoin, but->rnaprop, fcu, cfra)) {
+          but->drawflag |= UI_BUT_ANIMATED_CHANGED;
+        }
+      }
+    }
+    else {
+      but->flag |= UI_BUT_DRIVEN;
+    }
+  }
 }
 
 void ui_but_anim_decorate_update_from_flag(uiBut *but)
 {
-	BLI_assert(UI_but_is_decorator(but) && but->prev);
-	int flag = but->prev->flag;
-	if (flag & UI_BUT_DRIVEN) {
-		but->icon = ICON_DECORATE_DRIVER;
-	}
-	else if (flag & UI_BUT_ANIMATED_KEY) {
-		but->icon = ICON_DECORATE_KEYFRAME;
-	}
-	else if (flag & UI_BUT_ANIMATED) {
-		but->icon = ICON_DECORATE_ANIMATE;
-	}
-	else if (flag & UI_BUT_OVERRIDEN) {
-		but->icon = ICON_DECORATE_OVERRIDE;
-	}
-	else {
-		but->icon = ICON_DECORATE;
-	}
-
-	const int flag_copy = (UI_BUT_DISABLED | UI_BUT_INACTIVE);
-	but->flag = (but->flag & ~flag_copy) | (flag & flag_copy);
+  BLI_assert(UI_but_is_decorator(but) && but->prev);
+  int flag = but->prev->flag;
+  if (flag & UI_BUT_DRIVEN) {
+    but->icon = ICON_DECORATE_DRIVER;
+  }
+  else if (flag & UI_BUT_ANIMATED_KEY) {
+    but->icon = ICON_DECORATE_KEYFRAME;
+  }
+  else if (flag & UI_BUT_ANIMATED) {
+    but->icon = ICON_DECORATE_ANIMATE;
+  }
+  else if (flag & UI_BUT_OVERRIDEN) {
+    but->icon = ICON_DECORATE_OVERRIDE;
+  }
+  else {
+    but->icon = ICON_DECORATE;
+  }
+
+  const int flag_copy = (UI_BUT_DISABLED | UI_BUT_INACTIVE);
+  but->flag = (but->flag & ~flag_copy) | (flag & flag_copy);
 }
 
 /**
@@ -137,233 +139,251 @@ void ui_but_anim_decorate_update_from_flag(uiBut *but)
  */
 bool ui_but_anim_expression_get(uiBut *but, char *str, size_t maxlen)
 {
-	FCurve *fcu;
-	ChannelDriver *driver;
-	bool driven, special;
+  FCurve *fcu;
+  ChannelDriver *driver;
+  bool driven, special;
 
-	fcu = ui_but_get_fcurve(but, NULL, NULL, &driven, &special);
+  fcu = ui_but_get_fcurve(but, NULL, NULL, &driven, &special);
 
-	if (fcu && driven) {
-		driver = fcu->driver;
+  if (fcu && driven) {
+    driver = fcu->driver;
 
-		if (driver && driver->type == DRIVER_TYPE_PYTHON) {
-			if (str) {
-				BLI_strncpy(str, driver->expression, maxlen);
-			}
-			return true;
-		}
-	}
+    if (driver && driver->type == DRIVER_TYPE_PYTHON) {
+      if (str) {
+        BLI_strncpy(str, driver->expression, maxlen);
+      }
+      return true;
+    }
+  }
 
-	return false;
+  return false;
 }
 
 bool ui_but_anim_expression_set(uiBut *but, const char *str)
 {
-	FCurve *fcu;
-	ChannelDriver *driver;
-	bool driven, special;
+  FCurve *fcu;
+  ChannelDriver *driver;
+  bool driven, special;
 
-	fcu = ui_but_get_fcurve(but, NULL, NULL, &driven, &special);
+  fcu = ui_but_get_fcurve(but, NULL, NULL, &driven, &special);
 
-	if (fcu && driven) {
-		driver = fcu->driver;
+  if (fcu && driven) {
+    driver = fcu->driver;
 
-		if (driver && (driver->type == DRIVER_TYPE_PYTHON)) {
-			BLI_strncpy_utf8(driver->expression, str, sizeof(driver->expression));
+    if (driver && (driver->type == DRIVER_TYPE_PYTHON)) {
+      BLI_strncpy_utf8(driver->expression, str, sizeof(driver->expression));
 
-			/* tag driver as needing to be recompiled */
-			BKE_driver_invalidate_expression(driver, true, false);
+      /* tag driver as needing to be recompiled */
+      BKE_driver_invalidate_expression(driver, true, false);
 
-			/* clear invalid flags which may prevent this from working */
-			driver->flag &= ~DRIVER_FLAG_INVALID;
-			fcu->flag &= ~FCURVE_DISABLED;
+      /* clear invalid flags which may prevent this from working */
+      driver->flag &= ~DRIVER_FLAG_INVALID;
+      fcu->flag &= ~FCURVE_DISABLED;
 
-			/* this notifier should update the Graph Editor and trigger depsgraph refresh? */
-			WM_event_add_notifier(but->block->evil_C, NC_ANIMATION | ND_KEYFRAME, NULL);
+      /* this notifier should update the Graph Editor and trigger depsgraph refresh? */
+      WM_event_add_notifier(but->block->evil_C, NC_ANIMATION | ND_KEYFRAME, NULL);
 
-			return true;
-		}
-	}
+      return true;
+    }
+  }
 
-	return false;
+  return false;
 }
 
 /* create new expression for button (i.e. a "scripted driver"), if it can be created... */
 bool ui_but_anim_expression_create(uiBut *but, const char *str)
 {
-	bContext *C = but->block->evil_C;
-	ID *id;
-	FCurve *fcu;
-	char *path;
-	bool ok = false;
-
-	/* button must have RNA-pointer to a numeric-capable property */
-	if (ELEM(NULL, but->rnapoin.data, but->rnaprop)) {
-		if (G.debug & G_DEBUG) {
-			printf("ERROR: create expression failed - button has no RNA info attached\n");
-		}
-		return false;
-	}
-
-	if (RNA_property_array_check(but->rnaprop) != 0) {
-		if (but->rnaindex == -1) {
-			if (G.debug & G_DEBUG) {
-				printf("ERROR: create expression failed - can't create expression for entire array\n");
-			}
-			return false;
-		}
-	}
-
-	/* make sure we have animdata for this */
-	/* FIXME: until materials can be handled by depsgraph,
-	 * don't allow drivers to be created for them */
-	id = (ID *)but->rnapoin.id.data;
-	if ((id == NULL) || (GS(id->name) == ID_MA) || (GS(id->name) == ID_TE)) {
-		if (G.debug & G_DEBUG) {
-			printf("ERROR: create expression failed - invalid data-block for adding drivers (%p)\n", id);
-		}
-		return false;
-	}
-
-	/* get path */
-	path = RNA_path_from_ID_to_property(&but->rnapoin, but->rnaprop);
-	if (path == NULL) {
-		return false;
-	}
-
-	/* create driver */
-	fcu = verify_driver_fcurve(id, path, but->rnaindex, 1);
-	if (fcu) {
-		ChannelDriver *driver = fcu->driver;
-
-		if (driver) {
-			/* set type of driver */
-			driver->type = DRIVER_TYPE_PYTHON;
-
-			/* set the expression */
-			/* TODO: need some way of identifying variables used */
-			BLI_strncpy_utf8(driver->expression, str, sizeof(driver->expression));
-
-			/* updates */
-			BKE_driver_invalidate_expression(driver, true, false);
-			DEG_relations_tag_update(CTX_data_main(C));
-			WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME, NULL);
-			ok = true;
-		}
-	}
-
-	MEM_freeN(path);
-
-	return ok;
+  bContext *C = but->block->evil_C;
+  ID *id;
+  FCurve *fcu;
+  char *path;
+  bool ok = false;
+
+  /* button must have RNA-pointer to a numeric-capable property */
+  if (ELEM(NULL, but->rnapoin.data, but->rnaprop)) {
+    if (G.debug & G_DEBUG) {
+      printf("ERROR: create expression failed - button has no RNA info attached\n");
+    }
+    return false;
+  }
+
+  if (RNA_property_array_check(but->rnaprop) != 0) {
+    if (but->rnaindex == -1) {
+      if (G.debug & G_DEBUG) {
+        printf("ERROR: create expression failed - can't create expression for entire array\n");
+      }
+      return false;
+    }
+  }
+
+  /* make sure we have animdata for this */
+  /* FIXME: until materials can be handled by depsgraph,
+   * don't allow drivers to be created for them */
+  id = (ID *)but->rnapoin.id.data;
+  if ((id == NULL) || (GS(id->name) == ID_MA) || (GS(id->name) == ID_TE)) {
+    if (G.debug & G_DEBUG) {
+      printf("ERROR: create expression failed - invalid data-block for adding drivers (%p)\n", id);
+    }
+    return false;
+  }
+
+  /* get path */
+  path = RNA_path_from_ID_to_property(&but->rnapoin, but->rnaprop);
+  if (path == NULL) {
+    return false;
+  }
+
+  /* create driver */
+  fcu = verify_driver_fcurve(id, path, but->rnaindex, 1);
+  if (fcu) {
+    ChannelDriver *driver = fcu->driver;
+
+    if (driver) {
+      /* set type of driver */
+      driver->type = DRIVER_TYPE_PYTHON;
+
+      /* set the expression */
+      /* TODO: need some way of identifying variables used */
+      BLI_strncpy_utf8(driver->expression, str, sizeof(driver->expression));
+
+      /* updates */
+      BKE_driver_invalidate_expression(driver, true, false);
+      DEG_relations_tag_update(CTX_data_main(C));
+      WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME, NULL);
+      ok = true;
+    }
+  }
+
+  MEM_freeN(path);
+
+  return ok;
 }
 
 void ui_but_anim_autokey(bContext *C, uiBut *but, Scene *scene, float cfra)
 {
-	Main *bmain = CTX_data_main(C);
-	ID *id;
-	bAction *action;
-	FCurve *fcu;
-	bool driven;
-	bool special;
-
-	fcu = ui_but_get_fcurve(but, NULL, &action, &driven, &special);
-
-	if (fcu == NULL) {
-		return;
-	}
-
-	if (special) {
-		/* NLA Strip property */
-		if (IS_AUTOKEY_ON(scene)) {
-			Depsgraph *depsgraph = CTX_data_depsgraph(C);
-			ReportList *reports = CTX_wm_reports(C);
-			ToolSettings *ts = scene->toolsettings;
-
-			insert_keyframe_direct(depsgraph, reports, but->rnapoin, but->rnaprop, fcu, cfra, ts->keyframe_type, NULL, 0);
-			WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
-		}
-	}
-	else if (driven) {
-		/* Driver - Try to insert keyframe using the driver's input as the frame,
-		 * making it easier to set up corrective drivers
-		 */
-		if (IS_AUTOKEY_ON(scene)) {
-			Depsgraph *depsgraph = CTX_data_depsgraph(C);
-			ReportList *reports = CTX_wm_reports(C);
-			ToolSettings *ts = scene->toolsettings;
-
-			insert_keyframe_direct(depsgraph, reports, but->rnapoin, but->rnaprop, fcu, cfra, ts->keyframe_type, NULL, INSERTKEY_DRIVER);
-			WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
-		}
-	}
-	else {
-		id = but->rnapoin.id.data;
-
-		/* TODO: this should probably respect the keyingset only option for anim */
-		if (autokeyframe_cfra_can_key(scene, id)) {
-			Depsgraph *depsgraph = CTX_data_depsgraph(C);
-			ReportList *reports = CTX_wm_reports(C);
-			ToolSettings *ts = scene->toolsettings;
-			short flag = ANIM_get_keyframing_flags(scene, 1);
-
-			fcu->flag &= ~FCURVE_SELECTED;
-
-			/* Note: We use but->rnaindex instead of fcu->array_index,
-			 *       because a button may control all items of an array at once.
-			 *       E.g., color wheels (see T42567). */
-			BLI_assert((fcu->array_index == but->rnaindex) || (but->rnaindex == -1));
-			insert_keyframe(bmain, depsgraph, reports, id, action,
-			                ((fcu->grp) ? (fcu->grp->name) : (NULL)),
-			                fcu->rna_path, but->rnaindex, cfra, ts->keyframe_type, NULL, flag);
-
-			WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
-		}
-	}
+  Main *bmain = CTX_data_main(C);
+  ID *id;
+  bAction *action;
+  FCurve *fcu;
+  bool driven;
+  bool special;
+
+  fcu = ui_but_get_fcurve(but, NULL, &action, &driven, &special);
+
+  if (fcu == NULL) {
+    return;
+  }
+
+  if (special) {
+    /* NLA Strip property */
+    if (IS_AUTOKEY_ON(scene)) {
+      Depsgraph *depsgraph = CTX_data_depsgraph(C);
+      ReportList *reports = CTX_wm_reports(C);
+      ToolSettings *ts = scene->toolsettings;
+
+      insert_keyframe_direct(
+          depsgraph, reports, but->rnapoin, but->rnaprop, fcu, cfra, ts->keyframe_type, NULL, 0);
+      WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+    }
+  }
+  else if (driven) {
+    /* Driver - Try to insert keyframe using the driver's input as the frame,
+     * making it easier to set up corrective drivers
+     */
+    if (IS_AUTOKEY_ON(scene)) {
+      Depsgraph *depsgraph = CTX_data_depsgraph(C);
+      ReportList *reports = CTX_wm_reports(C);
+      ToolSettings *ts = scene->toolsettings;
+
+      insert_keyframe_direct(depsgraph,
+                             reports,
+                             but->rnapoin,
+                             but->rnaprop,
+                             fcu,
+                             cfra,
+                             ts->keyframe_type,
+                             NULL,
+                             INSERTKEY_DRIVER);
+      WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+    }
+  }
+  else {
+    id = but->rnapoin.id.data;
+
+    /* TODO: this should probably respect the keyingset only option for anim */
+    if (autokeyframe_cfra_can_key(scene, id)) {
+      Depsgraph *depsgraph = CTX_data_depsgraph(C);
+      ReportList *reports = CTX_wm_reports(C);
+      ToolSettings *ts = scene->toolsettings;
+      short flag = ANIM_get_keyframing_flags(scene, 1);
+
+      fcu->flag &= ~FCURVE_SELECTED;
+
+      /* Note: We use but->rnaindex instead of fcu->array_index,
+       *       because a button may control all items of an array at once.
+       *       E.g., color wheels (see T42567). */
+      BLI_assert((fcu->array_index == but->rnaindex) || (but->rnaindex == -1));
+      insert_keyframe(bmain,
+                      depsgraph,
+                      reports,
+                      id,
+                      action,
+                      ((fcu->grp) ? (fcu->grp->name) : (NULL)),
+                      fcu->rna_path,
+                      but->rnaindex,
+                      cfra,
+                      ts->keyframe_type,
+                      NULL,
+                      flag);
+
+      WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
+    }
+  }
 }
 
 void ui_but_anim_copy_driver(bContext *C)
 {
-	/* this operator calls UI_context_active_but_prop_get */
-	WM_operator_name_call(C, "ANIM_OT_copy_driver_button", WM_OP_INVOKE_DEFAULT, NULL);
+  /* this operator calls UI_context_active_but_prop_get */
+  WM_operator_name_call(C, "ANIM_OT_copy_driver_button", WM_OP_INVOKE_DEFAULT, NULL);
 }
 
 void ui_but_anim_paste_driver(bContext *C)
 {
-	/* this operator calls UI_context_active_but_prop_get */
-	WM_operator_name_call(C, "ANIM_OT_paste_driver_button", WM_OP_INVOKE_DEFAULT, NULL);
+  /* this operator calls UI_context_active_but_prop_get */
+  WM_operator_name_call(C, "ANIM_OT_paste_driver_button", WM_OP_INVOKE_DEFAULT, NULL);
 }
 
 void ui_but_anim_decorate_cb(bContext *C, void *arg_but, void *UNUSED(arg_dummy))
 {
-	wmWindowManager *wm = CTX_wm_manager(C);
-	uiBut *but = arg_but;
-	but = but->prev;
-
-	/* FIXME(campbell), swapping active pointer is weak. */
-	SWAP(struct uiHandleButtonData *, but->active, but->next->active);
-	wm->op_undo_depth++;
-
-	if (but->flag & UI_BUT_DRIVEN) {
-		/* pass */
-		/* TODO: report? */
-	}
-	else if (but->flag & UI_BUT_ANIMATED_KEY) {
-		PointerRNA props_ptr;
-		wmOperatorType *ot = WM_operatortype_find("ANIM_OT_keyframe_delete_button", false);
-		WM_operator_properties_create_ptr(&props_ptr, ot);
-		RNA_boolean_set(&props_ptr, "all", but->rnaindex == -1);
-		WM_operator_name_call_ptr(C, ot, WM_OP_INVOKE_DEFAULT, &props_ptr);
-		WM_operator_properties_free(&props_ptr);
-	}
-	else {
-		PointerRNA props_ptr;
-		wmOperatorType *ot = WM_operatortype_find("ANIM_OT_keyframe_insert_button", false);
-		WM_operator_properties_create_ptr(&props_ptr, ot);
-		RNA_boolean_set(&props_ptr, "all", but->rnaindex == -1);
-		WM_operator_name_call_ptr(C, ot, WM_OP_INVOKE_DEFAULT, &props_ptr);
-		WM_operator_properties_free(&props_ptr);
-	}
-
-	SWAP(struct uiHandleButtonData *, but->active, but->next->active);
-	wm->op_undo_depth--;
+  wmWindowManager *wm = CTX_wm_manager(C);
+  uiBut *but = arg_but;
+  but = but->prev;
+
+  /* FIXME(campbell), swapping active pointer is weak. */
+  SWAP(struct uiHandleButtonData *, but->active, but->next->active);
+  wm->op_undo_depth++;
+
+  if (but->flag & UI_BUT_DRIVEN) {
+    /* pass */
+    /* TODO: report? */
+  }
+  else if (but->flag & UI_BUT_ANIMATED_KEY) {
+    PointerRNA props_ptr;
+    wmOperatorType *ot = WM_operatortype_find("ANIM_OT_keyframe_delete_button", false);
+    WM_operator_properties_create_ptr(&props_ptr, ot);
+    RNA_boolean_set(&props_ptr, "all", but->rnaindex == -1);
+    WM_operator_name_call_ptr(C, ot, WM_OP_INVOKE_DEFAULT, &props_ptr);
+    WM_operator_properties_free(&props_ptr);
+  }
+  else {
+    PointerRNA props_ptr;
+    wmOperatorType *ot = WM_operatortype_find("ANIM_OT_keyframe_insert_button", false);
+    WM_operator_properties_create_ptr(&props_ptr, ot);
+    RNA_boolean_set(&props_ptr, "all", but->rnaindex == -1);
+    WM_operator_name_call_ptr(C, ot, WM_OP_INVOKE_DEFAULT, &props_ptr);
+    WM_operator_properties_free(&props_ptr);
+  }
+
+  SWAP(struct uiHandleButtonData *, but->active, but->next->active);
+  wm->op_undo_depth--;
 }