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, 445 insertions, 426 deletions

diff --git a/source/blender/editors/armature/armature_naming.c b/source/blender/editors/armature/armature_naming.c
index 6d4babe9674..b1f0297bb29 100644
--- a/source/blender/editors/armature/armature_naming.c
+++ b/source/blender/editors/armature/armature_naming.c
@@ -71,18 +71,24 @@
 /* note: there's a unique_bone_name() too! */
 static bool editbone_unique_check(void *arg, const char *name)
 {
-	struct {ListBase *lb; void *bone; } *data = arg;
-	EditBone *dupli = ED_armature_ebone_find_name(data->lb, name);
-	return dupli && dupli != data->bone;
+  struct {
+    ListBase *lb;
+    void *bone;
+  } *data = arg;
+  EditBone *dupli = ED_armature_ebone_find_name(data->lb, name);
+  return dupli && dupli != data->bone;
 }
 
 void ED_armature_ebone_unique_name(ListBase *edbo, char *name, EditBone *bone)
 {
-	struct {ListBase *lb; void *bone; } data;
-	data.lb = edbo;
-	data.bone = bone;
-
-	BLI_uniquename_cb(editbone_unique_check, &data, DATA_("Bone"), '.', name, sizeof(bone->name));
+  struct {
+    ListBase *lb;
+    void *bone;
+  } data;
+  data.lb = edbo;
+  data.bone = bone;
+
+  BLI_uniquename_cb(editbone_unique_check, &data, DATA_("Bone"), '.', name, sizeof(bone->name));
 }
 
 /* ************************************************** */
@@ -90,261 +96,265 @@ void ED_armature_ebone_unique_name(ListBase *edbo, char *name, EditBone *bone)
 
 static bool bone_unique_check(void *arg, const char *name)
 {
-	return BKE_armature_find_bone_name((bArmature *)arg, name) != NULL;
+  return BKE_armature_find_bone_name((bArmature *)arg, name) != NULL;
 }
 
 static void unique_bone_name(bArmature *arm, char *name)
 {
-	BLI_uniquename_cb(bone_unique_check, (void *)arm, DATA_("Bone"), '.', name, sizeof(((Bone *)NULL)->name));
+  BLI_uniquename_cb(
+      bone_unique_check, (void *)arm, DATA_("Bone"), '.', name, sizeof(((Bone *)NULL)->name));
 }
 
 /* helper call for armature_bone_rename */
-static void constraint_bone_name_fix(Object *ob, ListBase *conlist, const char *oldname, const char *newname)
+static void constraint_bone_name_fix(Object *ob,
+                                     ListBase *conlist,
+                                     const char *oldname,
+                                     const char *newname)
 {
-	bConstraint *curcon;
-	bConstraintTarget *ct;
-
-	for (curcon = conlist->first; curcon; curcon = curcon->next) {
-		const bConstraintTypeInfo *cti = BKE_constraint_typeinfo_get(curcon);
-		ListBase targets = {NULL, NULL};
-
-		/* constraint targets */
-		if (cti && cti->get_constraint_targets) {
-			cti->get_constraint_targets(curcon, &targets);
-
-			for (ct = targets.first; ct; ct = ct->next) {
-				if (ct->tar == ob) {
-					if (STREQ(ct->subtarget, oldname)) {
-						BLI_strncpy(ct->subtarget, newname, MAXBONENAME);
-					}
-				}
-			}
-
-			if (cti->flush_constraint_targets)
-				cti->flush_constraint_targets(curcon, &targets, 0);
-		}
-
-		/* action constraints */
-		if (curcon->type == CONSTRAINT_TYPE_ACTION) {
-			bActionConstraint *actcon = (bActionConstraint *)curcon->data;
-			BKE_action_fix_paths_rename(&ob->id, actcon->act, "pose.bones", oldname, newname, 0, 0, 1);
-		}
-	}
+  bConstraint *curcon;
+  bConstraintTarget *ct;
+
+  for (curcon = conlist->first; curcon; curcon = curcon->next) {
+    const bConstraintTypeInfo *cti = BKE_constraint_typeinfo_get(curcon);
+    ListBase targets = {NULL, NULL};
+
+    /* constraint targets */
+    if (cti && cti->get_constraint_targets) {
+      cti->get_constraint_targets(curcon, &targets);
+
+      for (ct = targets.first; ct; ct = ct->next) {
+        if (ct->tar == ob) {
+          if (STREQ(ct->subtarget, oldname)) {
+            BLI_strncpy(ct->subtarget, newname, MAXBONENAME);
+          }
+        }
+      }
+
+      if (cti->flush_constraint_targets)
+        cti->flush_constraint_targets(curcon, &targets, 0);
+    }
+
+    /* action constraints */
+    if (curcon->type == CONSTRAINT_TYPE_ACTION) {
+      bActionConstraint *actcon = (bActionConstraint *)curcon->data;
+      BKE_action_fix_paths_rename(&ob->id, actcon->act, "pose.bones", oldname, newname, 0, 0, 1);
+    }
+  }
 }
 
 /* called by UI for renaming a bone */
 /* warning: make sure the original bone was not renamed yet! */
 /* seems messy, but that's what you get with not using pointers but channel names :) */
-void ED_armature_bone_rename(Main *bmain, bArmature *arm, const char *oldnamep, const char *newnamep)
+void ED_armature_bone_rename(Main *bmain,
+                             bArmature *arm,
+                             const char *oldnamep,
+                             const char *newnamep)
 {
-	Object *ob;
-	char newname[MAXBONENAME];
-	char oldname[MAXBONENAME];
-
-	/* names better differ! */
-	if (!STREQLEN(oldnamep, newnamep, MAXBONENAME)) {
-
-		/* we alter newname string... so make copy */
-		BLI_strncpy(newname, newnamep, MAXBONENAME);
-		/* we use oldname for search... so make copy */
-		BLI_strncpy(oldname, oldnamep, MAXBONENAME);
-
-		/* now check if we're in editmode, we need to find the unique name */
-		if (arm->edbo) {
-			EditBone *eBone = ED_armature_ebone_find_name(arm->edbo, oldname);
-
-			if (eBone) {
-				ED_armature_ebone_unique_name(arm->edbo, newname, NULL);
-				BLI_strncpy(eBone->name, newname, MAXBONENAME);
-			}
-			else {
-				return;
-			}
-		}
-		else {
-			Bone *bone = BKE_armature_find_bone_name(arm, oldname);
-
-			if (bone) {
-				unique_bone_name(arm, newname);
-				BLI_strncpy(bone->name, newname, MAXBONENAME);
-			}
-			else {
-				return;
-			}
-		}
-
-		/* force copy on write to update database */
-		DEG_id_tag_update(&arm->id, ID_RECALC_COPY_ON_WRITE);
-
-		/* do entire dbase - objects */
-		for (ob = bmain->objects.first; ob; ob = ob->id.next) {
-			ModifierData *md;
-
-			/* we have the object using the armature */
-			if (arm == ob->data) {
-				Object *cob;
-
-				/* Rename the pose channel, if it exists */
-				if (ob->pose) {
-					bPoseChannel *pchan = BKE_pose_channel_find_name(ob->pose, oldname);
-					if (pchan) {
-						GHash *gh = ob->pose->chanhash;
-
-						/* remove the old hash entry, and replace with the new name */
-						if (gh) {
-							BLI_assert(BLI_ghash_haskey(gh, pchan->name));
-							BLI_ghash_remove(gh, pchan->name, NULL, NULL);
-						}
-
-						BLI_strncpy(pchan->name, newname, MAXBONENAME);
-
-						if (gh) {
-							BLI_ghash_insert(gh, pchan->name, pchan);
-						}
-					}
-
-					BLI_assert(BKE_pose_channels_is_valid(ob->pose) == true);
-				}
-
-				/* Update any object constraints to use the new bone name */
-				for (cob = bmain->objects.first; cob; cob = cob->id.next) {
-					if (cob->constraints.first)
-						constraint_bone_name_fix(ob, &cob->constraints, oldname, newname);
-					if (cob->pose) {
-						bPoseChannel *pchan;
-						for (pchan = cob->pose->chanbase.first; pchan; pchan = pchan->next) {
-							constraint_bone_name_fix(ob, &pchan->constraints, oldname, newname);
-						}
-					}
-				}
-			}
-
-			/* See if an object is parented to this armature */
-			if (ob->parent && (ob->parent->data == arm)) {
-				if (ob->partype == PARBONE) {
-					/* bone name in object */
-					if (STREQ(ob->parsubstr, oldname))
-						BLI_strncpy(ob->parsubstr, newname, MAXBONENAME);
-				}
-			}
-
-			if (modifiers_usesArmature(ob, arm)) {
-				bDeformGroup *dg = defgroup_find_name(ob, oldname);
-				if (dg) {
-					BLI_strncpy(dg->name, newname, MAXBONENAME);
-				}
-			}
-
-			/* fix modifiers that might be using this name */
-			for (md = ob->modifiers.first; md; md = md->next) {
-				switch (md->type) {
-					case eModifierType_Hook:
-					{
-						HookModifierData *hmd = (HookModifierData *)md;
-
-						if (hmd->object && (hmd->object->data == arm)) {
-							if (STREQ(hmd->subtarget, oldname))
-								BLI_strncpy(hmd->subtarget, newname, MAXBONENAME);
-						}
-						break;
-					}
-					case eModifierType_UVWarp:
-					{
-						UVWarpModifierData *umd = (UVWarpModifierData *)md;
-
-						if (umd->object_src && (umd->object_src->data == arm)) {
-							if (STREQ(umd->bone_src, oldname))
-								BLI_strncpy(umd->bone_src, newname, MAXBONENAME);
-						}
-						if (umd->object_dst && (umd->object_dst->data == arm)) {
-							if (STREQ(umd->bone_dst, oldname))
-								BLI_strncpy(umd->bone_dst, newname, MAXBONENAME);
-						}
-						break;
-					}
-					default:
-						break;
-				}
-			}
-
-			/* fix grease pencil modifiers and vertex groups */
-			if (ob->type == OB_GPENCIL) {
-
-				bGPdata *gpd = (bGPdata *)ob->data;
-				for (bGPDlayer *gpl = gpd->layers.first; gpl; gpl = gpl->next) {
-					if ((gpl->parent != NULL) && (gpl->parent->data == arm)) {
-						if (STREQ(gpl->parsubstr, oldname))
-							BLI_strncpy(gpl->parsubstr, newname, MAXBONENAME);
-					}
-				}
-
-				for (GpencilModifierData *gp_md = ob->greasepencil_modifiers.first; gp_md; gp_md = gp_md->next) {
-					switch (gp_md->type) {
-						case eGpencilModifierType_Armature:
-						{
-							ArmatureGpencilModifierData *mmd = (ArmatureGpencilModifierData *)gp_md;
-							if (mmd->object && mmd->object->data == arm) {
-								bDeformGroup *dg = defgroup_find_name(ob, oldname);
-								if (dg) {
-									BLI_strncpy(dg->name, newname, MAXBONENAME);
-								}
-							}
-							break;
-						}
-						case eGpencilModifierType_Hook:
-						{
-							HookGpencilModifierData *hgp_md = (HookGpencilModifierData *)gp_md;
-							if (hgp_md->object && (hgp_md->object->data == arm)) {
-								if (STREQ(hgp_md->subtarget, oldname))
-									BLI_strncpy(hgp_md->subtarget, newname, MAXBONENAME);
-							}
-							break;
-						}
-						default:
-							break;
-					}
-				}
-			}
-			DEG_id_tag_update(&ob->id, ID_RECALC_COPY_ON_WRITE);
-		}
-
-		/* Fix all animdata that may refer to this bone - we can't just do the ones attached to objects, since
-		 * other ID-blocks may have drivers referring to this bone [#29822]
-		 */
-		// XXX: the ID here is for armatures, but most bone drivers are actually on the object instead...
-		{
-
-			BKE_animdata_fix_paths_rename_all(&arm->id, "pose.bones", oldname, newname);
-		}
-
-		/* correct view locking */
-		{
-			bScreen *screen;
-			for (screen = bmain->screens.first; screen; screen = screen->id.next) {
-				ScrArea *sa;
-				/* add regions */
-				for (sa = screen->areabase.first; sa; sa = sa->next) {
-					SpaceLink *sl;
-					for (sl = sa->spacedata.first; sl; sl = sl->next) {
-						if (sl->spacetype == SPACE_VIEW3D) {
-							View3D *v3d = (View3D *)sl;
-							if (v3d->ob_centre && v3d->ob_centre->data == arm) {
-								if (STREQ(v3d->ob_centre_bone, oldname)) {
-									BLI_strncpy(v3d->ob_centre_bone, newname, MAXBONENAME);
-								}
-							}
-						}
-					}
-				}
-			}
-		}
-	}
+  Object *ob;
+  char newname[MAXBONENAME];
+  char oldname[MAXBONENAME];
+
+  /* names better differ! */
+  if (!STREQLEN(oldnamep, newnamep, MAXBONENAME)) {
+
+    /* we alter newname string... so make copy */
+    BLI_strncpy(newname, newnamep, MAXBONENAME);
+    /* we use oldname for search... so make copy */
+    BLI_strncpy(oldname, oldnamep, MAXBONENAME);
+
+    /* now check if we're in editmode, we need to find the unique name */
+    if (arm->edbo) {
+      EditBone *eBone = ED_armature_ebone_find_name(arm->edbo, oldname);
+
+      if (eBone) {
+        ED_armature_ebone_unique_name(arm->edbo, newname, NULL);
+        BLI_strncpy(eBone->name, newname, MAXBONENAME);
+      }
+      else {
+        return;
+      }
+    }
+    else {
+      Bone *bone = BKE_armature_find_bone_name(arm, oldname);
+
+      if (bone) {
+        unique_bone_name(arm, newname);
+        BLI_strncpy(bone->name, newname, MAXBONENAME);
+      }
+      else {
+        return;
+      }
+    }
+
+    /* force copy on write to update database */
+    DEG_id_tag_update(&arm->id, ID_RECALC_COPY_ON_WRITE);
+
+    /* do entire dbase - objects */
+    for (ob = bmain->objects.first; ob; ob = ob->id.next) {
+      ModifierData *md;
+
+      /* we have the object using the armature */
+      if (arm == ob->data) {
+        Object *cob;
+
+        /* Rename the pose channel, if it exists */
+        if (ob->pose) {
+          bPoseChannel *pchan = BKE_pose_channel_find_name(ob->pose, oldname);
+          if (pchan) {
+            GHash *gh = ob->pose->chanhash;
+
+            /* remove the old hash entry, and replace with the new name */
+            if (gh) {
+              BLI_assert(BLI_ghash_haskey(gh, pchan->name));
+              BLI_ghash_remove(gh, pchan->name, NULL, NULL);
+            }
+
+            BLI_strncpy(pchan->name, newname, MAXBONENAME);
+
+            if (gh) {
+              BLI_ghash_insert(gh, pchan->name, pchan);
+            }
+          }
+
+          BLI_assert(BKE_pose_channels_is_valid(ob->pose) == true);
+        }
+
+        /* Update any object constraints to use the new bone name */
+        for (cob = bmain->objects.first; cob; cob = cob->id.next) {
+          if (cob->constraints.first)
+            constraint_bone_name_fix(ob, &cob->constraints, oldname, newname);
+          if (cob->pose) {
+            bPoseChannel *pchan;
+            for (pchan = cob->pose->chanbase.first; pchan; pchan = pchan->next) {
+              constraint_bone_name_fix(ob, &pchan->constraints, oldname, newname);
+            }
+          }
+        }
+      }
+
+      /* See if an object is parented to this armature */
+      if (ob->parent && (ob->parent->data == arm)) {
+        if (ob->partype == PARBONE) {
+          /* bone name in object */
+          if (STREQ(ob->parsubstr, oldname))
+            BLI_strncpy(ob->parsubstr, newname, MAXBONENAME);
+        }
+      }
+
+      if (modifiers_usesArmature(ob, arm)) {
+        bDeformGroup *dg = defgroup_find_name(ob, oldname);
+        if (dg) {
+          BLI_strncpy(dg->name, newname, MAXBONENAME);
+        }
+      }
+
+      /* fix modifiers that might be using this name */
+      for (md = ob->modifiers.first; md; md = md->next) {
+        switch (md->type) {
+          case eModifierType_Hook: {
+            HookModifierData *hmd = (HookModifierData *)md;
+
+            if (hmd->object && (hmd->object->data == arm)) {
+              if (STREQ(hmd->subtarget, oldname))
+                BLI_strncpy(hmd->subtarget, newname, MAXBONENAME);
+            }
+            break;
+          }
+          case eModifierType_UVWarp: {
+            UVWarpModifierData *umd = (UVWarpModifierData *)md;
+
+            if (umd->object_src && (umd->object_src->data == arm)) {
+              if (STREQ(umd->bone_src, oldname))
+                BLI_strncpy(umd->bone_src, newname, MAXBONENAME);
+            }
+            if (umd->object_dst && (umd->object_dst->data == arm)) {
+              if (STREQ(umd->bone_dst, oldname))
+                BLI_strncpy(umd->bone_dst, newname, MAXBONENAME);
+            }
+            break;
+          }
+          default:
+            break;
+        }
+      }
+
+      /* fix grease pencil modifiers and vertex groups */
+      if (ob->type == OB_GPENCIL) {
+
+        bGPdata *gpd = (bGPdata *)ob->data;
+        for (bGPDlayer *gpl = gpd->layers.first; gpl; gpl = gpl->next) {
+          if ((gpl->parent != NULL) && (gpl->parent->data == arm)) {
+            if (STREQ(gpl->parsubstr, oldname))
+              BLI_strncpy(gpl->parsubstr, newname, MAXBONENAME);
+          }
+        }
+
+        for (GpencilModifierData *gp_md = ob->greasepencil_modifiers.first; gp_md;
+             gp_md = gp_md->next) {
+          switch (gp_md->type) {
+            case eGpencilModifierType_Armature: {
+              ArmatureGpencilModifierData *mmd = (ArmatureGpencilModifierData *)gp_md;
+              if (mmd->object && mmd->object->data == arm) {
+                bDeformGroup *dg = defgroup_find_name(ob, oldname);
+                if (dg) {
+                  BLI_strncpy(dg->name, newname, MAXBONENAME);
+                }
+              }
+              break;
+            }
+            case eGpencilModifierType_Hook: {
+              HookGpencilModifierData *hgp_md = (HookGpencilModifierData *)gp_md;
+              if (hgp_md->object && (hgp_md->object->data == arm)) {
+                if (STREQ(hgp_md->subtarget, oldname))
+                  BLI_strncpy(hgp_md->subtarget, newname, MAXBONENAME);
+              }
+              break;
+            }
+            default:
+              break;
+          }
+        }
+      }
+      DEG_id_tag_update(&ob->id, ID_RECALC_COPY_ON_WRITE);
+    }
+
+    /* Fix all animdata that may refer to this bone - we can't just do the ones attached to objects, since
+     * other ID-blocks may have drivers referring to this bone [#29822]
+     */
+    // XXX: the ID here is for armatures, but most bone drivers are actually on the object instead...
+    {
+
+      BKE_animdata_fix_paths_rename_all(&arm->id, "pose.bones", oldname, newname);
+    }
+
+    /* correct view locking */
+    {
+      bScreen *screen;
+      for (screen = bmain->screens.first; screen; screen = screen->id.next) {
+        ScrArea *sa;
+        /* add regions */
+        for (sa = screen->areabase.first; sa; sa = sa->next) {
+          SpaceLink *sl;
+          for (sl = sa->spacedata.first; sl; sl = sl->next) {
+            if (sl->spacetype == SPACE_VIEW3D) {
+              View3D *v3d = (View3D *)sl;
+              if (v3d->ob_centre && v3d->ob_centre->data == arm) {
+                if (STREQ(v3d->ob_centre_bone, oldname)) {
+                  BLI_strncpy(v3d->ob_centre_bone, newname, MAXBONENAME);
+                }
+              }
+            }
+          }
+        }
+      }
+    }
+  }
 }
 
 typedef struct BoneFlipNameData {
-	struct BoneFlipNameData *next, *prev;
-	char *name;
-	char name_flip[MAXBONENAME];
+  struct BoneFlipNameData *next, *prev;
+  char *name;
+  char name_flip[MAXBONENAME];
 } BoneFlipNameData;
 
 /**
@@ -357,38 +367,41 @@ typedef struct BoneFlipNameData {
  * \param bones_names: List of BoneConflict elems.
  * \param do_strip_numbers: if set, try to get rid of dot-numbers at end of bone names.
  */
-void ED_armature_bones_flip_names(Main *bmain, bArmature *arm, ListBase *bones_names, const bool do_strip_numbers)
+void ED_armature_bones_flip_names(Main *bmain,
+                                  bArmature *arm,
+                                  ListBase *bones_names,
+                                  const bool do_strip_numbers)
 {
-	ListBase bones_names_conflicts = {NULL};
-	BoneFlipNameData *bfn;
-
-	/* First pass: generate flip names, and blindly rename.
-	 * If rename did not yield expected result, store both bone's name and expected flipped one into temp list
-	 * for second pass. */
-	for (LinkData *link = bones_names->first; link; link = link->next) {
-		char name_flip[MAXBONENAME];
-		char *name = link->data;
-
-		/* WARNING: if do_strip_numbers is set, expect completely mismatched names in cases like
-		 * Bone.R, Bone.R.001, Bone.R.002, etc. */
-		BLI_string_flip_side_name(name_flip, name, do_strip_numbers, sizeof(name_flip));
-
-		ED_armature_bone_rename(bmain, arm, name, name_flip);
-
-		if (!STREQ(name, name_flip)) {
-			bfn = alloca(sizeof(BoneFlipNameData));
-			bfn->name = name;
-			BLI_strncpy(bfn->name_flip, name_flip, sizeof(bfn->name_flip));
-			BLI_addtail(&bones_names_conflicts, bfn);
-		}
-	}
-
-	/* Second pass to handle the bones that have naming conflicts with other bones.
-	 * Note that if the other bone was not selected, its name was not flipped, so conflict remains and that second
-	 * rename simply generates a new numbered alternative name. */
-	for (bfn = bones_names_conflicts.first; bfn; bfn = bfn->next) {
-		ED_armature_bone_rename(bmain, arm, bfn->name, bfn->name_flip);
-	}
+  ListBase bones_names_conflicts = {NULL};
+  BoneFlipNameData *bfn;
+
+  /* First pass: generate flip names, and blindly rename.
+   * If rename did not yield expected result, store both bone's name and expected flipped one into temp list
+   * for second pass. */
+  for (LinkData *link = bones_names->first; link; link = link->next) {
+    char name_flip[MAXBONENAME];
+    char *name = link->data;
+
+    /* WARNING: if do_strip_numbers is set, expect completely mismatched names in cases like
+     * Bone.R, Bone.R.001, Bone.R.002, etc. */
+    BLI_string_flip_side_name(name_flip, name, do_strip_numbers, sizeof(name_flip));
+
+    ED_armature_bone_rename(bmain, arm, name, name_flip);
+
+    if (!STREQ(name, name_flip)) {
+      bfn = alloca(sizeof(BoneFlipNameData));
+      bfn->name = name;
+      BLI_strncpy(bfn->name_flip, name_flip, sizeof(bfn->name_flip));
+      BLI_addtail(&bones_names_conflicts, bfn);
+    }
+  }
+
+  /* Second pass to handle the bones that have naming conflicts with other bones.
+   * Note that if the other bone was not selected, its name was not flipped, so conflict remains and that second
+   * rename simply generates a new numbered alternative name. */
+  for (bfn = bones_names_conflicts.first; bfn; bfn = bfn->next) {
+    ED_armature_bone_rename(bmain, arm, bfn->name, bfn->name_flip);
+  }
 }
 
 /* ************************************************** */
@@ -396,165 +409,171 @@ void ED_armature_bones_flip_names(Main *bmain, bArmature *arm, ListBase *bones_n
 
 static int armature_flip_names_exec(bContext *C, wmOperator *op)
 {
-	Main *bmain = CTX_data_main(C);
-	ViewLayer *view_layer = CTX_data_view_layer(C);
-	Object *ob_active = CTX_data_edit_object(C);
-
-	const bool do_strip_numbers = RNA_boolean_get(op->ptr, "do_strip_numbers");
-
-	uint objects_len = 0;
-	Object **objects = BKE_view_layer_array_from_objects_in_edit_mode_unique_data(view_layer, CTX_wm_view3d(C), &objects_len);
-	for (uint ob_index = 0; ob_index < objects_len; ob_index++) {
-		Object *ob = objects[ob_index];
-		bArmature *arm = ob->data;
-
-		/* Paranoia check. */
-		if (ob_active->pose == NULL) {
-			continue;
-		}
-
-		ListBase bones_names = {NULL};
-
-		for (EditBone *ebone = arm->edbo->first; ebone; ebone = ebone->next) {
-			if (EBONE_VISIBLE(arm, ebone)) {
-				if (ebone->flag & BONE_SELECTED) {
-					BLI_addtail(&bones_names, BLI_genericNodeN(ebone->name));
-
-					if (arm->flag & ARM_MIRROR_EDIT) {
-						EditBone *flipbone = ED_armature_ebone_get_mirrored(arm->edbo, ebone);
-						if ((flipbone) && !(flipbone->flag & BONE_SELECTED)) {
-							BLI_addtail(&bones_names, BLI_genericNodeN(flipbone->name));
-						}
-					}
-				}
-			}
-		}
-
-		if (BLI_listbase_is_empty(&bones_names)) {
-			continue;
-		}
-
-		ED_armature_bones_flip_names(bmain, arm, &bones_names, do_strip_numbers);
-
-		BLI_freelistN(&bones_names);
-
-		/* since we renamed stuff... */
-		DEG_id_tag_update(&ob->id, ID_RECALC_GEOMETRY);
-
-		/* copied from #rna_Bone_update_renamed */
-		/* redraw view */
-		WM_event_add_notifier(C, NC_GEOM | ND_DATA, ob->data);
-
-		/* update animation channels */
-		WM_event_add_notifier(C, NC_ANIMATION | ND_ANIMCHAN, ob->data);
-
-	}
-	MEM_freeN(objects);
-
-	return OPERATOR_FINISHED;
+  Main *bmain = CTX_data_main(C);
+  ViewLayer *view_layer = CTX_data_view_layer(C);
+  Object *ob_active = CTX_data_edit_object(C);
+
+  const bool do_strip_numbers = RNA_boolean_get(op->ptr, "do_strip_numbers");
+
+  uint objects_len = 0;
+  Object **objects = BKE_view_layer_array_from_objects_in_edit_mode_unique_data(
+      view_layer, CTX_wm_view3d(C), &objects_len);
+  for (uint ob_index = 0; ob_index < objects_len; ob_index++) {
+    Object *ob = objects[ob_index];
+    bArmature *arm = ob->data;
+
+    /* Paranoia check. */
+    if (ob_active->pose == NULL) {
+      continue;
+    }
+
+    ListBase bones_names = {NULL};
+
+    for (EditBone *ebone = arm->edbo->first; ebone; ebone = ebone->next) {
+      if (EBONE_VISIBLE(arm, ebone)) {
+        if (ebone->flag & BONE_SELECTED) {
+          BLI_addtail(&bones_names, BLI_genericNodeN(ebone->name));
+
+          if (arm->flag & ARM_MIRROR_EDIT) {
+            EditBone *flipbone = ED_armature_ebone_get_mirrored(arm->edbo, ebone);
+            if ((flipbone) && !(flipbone->flag & BONE_SELECTED)) {
+              BLI_addtail(&bones_names, BLI_genericNodeN(flipbone->name));
+            }
+          }
+        }
+      }
+    }
+
+    if (BLI_listbase_is_empty(&bones_names)) {
+      continue;
+    }
+
+    ED_armature_bones_flip_names(bmain, arm, &bones_names, do_strip_numbers);
+
+    BLI_freelistN(&bones_names);
+
+    /* since we renamed stuff... */
+    DEG_id_tag_update(&ob->id, ID_RECALC_GEOMETRY);
+
+    /* copied from #rna_Bone_update_renamed */
+    /* redraw view */
+    WM_event_add_notifier(C, NC_GEOM | ND_DATA, ob->data);
+
+    /* update animation channels */
+    WM_event_add_notifier(C, NC_ANIMATION | ND_ANIMCHAN, ob->data);
+  }
+  MEM_freeN(objects);
+
+  return OPERATOR_FINISHED;
 }
 
 void ARMATURE_OT_flip_names(wmOperatorType *ot)
 {
-	/* identifiers */
-	ot->name = "Flip Names";
-	ot->idname = "ARMATURE_OT_flip_names";
-	ot->description = "Flips (and corrects) the axis suffixes of the names of selected bones";
-
-	/* api callbacks */
-	ot->exec = armature_flip_names_exec;
-	ot->poll = ED_operator_editarmature;
-
-	/* flags */
-	ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
-
-	RNA_def_boolean(ot->srna, "do_strip_numbers", false, "Strip Numbers",
-	                "Try to remove right-most dot-number from flipped names "
-	                "(WARNING: may result in incoherent naming in some cases)");
+  /* identifiers */
+  ot->name = "Flip Names";
+  ot->idname = "ARMATURE_OT_flip_names";
+  ot->description = "Flips (and corrects) the axis suffixes of the names of selected bones";
+
+  /* api callbacks */
+  ot->exec = armature_flip_names_exec;
+  ot->poll = ED_operator_editarmature;
+
+  /* flags */
+  ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
+
+  RNA_def_boolean(ot->srna,
+                  "do_strip_numbers",
+                  false,
+                  "Strip Numbers",
+                  "Try to remove right-most dot-number from flipped names "
+                  "(WARNING: may result in incoherent naming in some cases)");
 }
 
 static int armature_autoside_names_exec(bContext *C, wmOperator *op)
 {
-	ViewLayer *view_layer = CTX_data_view_layer(C);
-	Main *bmain = CTX_data_main(C);
-	char newname[MAXBONENAME];
-	const short axis = RNA_enum_get(op->ptr, "type");
-	bool changed_multi = false;
-
-	uint objects_len = 0;
-	Object **objects = BKE_view_layer_array_from_objects_in_edit_mode_unique_data(view_layer, CTX_wm_view3d(C), &objects_len);
-	for (uint ob_index = 0; ob_index < objects_len; ob_index++) {
-		Object *ob = objects[ob_index];
-		bArmature *arm = ob->data;
-		bool changed = false;
-
-		/* Paranoia checks. */
-		if (ELEM(NULL, ob, ob->pose)) {
-			continue;
-		}
-
-		for (EditBone *ebone = arm->edbo->first; ebone; ebone = ebone->next) {
-			if (EBONE_EDITABLE(ebone)) {
-
-				/* We first need to do the flipped bone, then the original one.
-				 * Otherwise we can't find the flipped one because of the bone name change. */
-				if (arm->flag & ARM_MIRROR_EDIT) {
-					EditBone *flipbone = ED_armature_ebone_get_mirrored(arm->edbo, ebone);
-					if ((flipbone) && !(flipbone->flag & BONE_SELECTED)) {
-						BLI_strncpy(newname, flipbone->name, sizeof(newname));
-						if (bone_autoside_name(newname, 1, axis, flipbone->head[axis], flipbone->tail[axis])) {
-							ED_armature_bone_rename(bmain, arm, flipbone->name, newname);
-							changed = true;
-						}
-					}
-				}
-
-				BLI_strncpy(newname, ebone->name, sizeof(newname));
-				if (bone_autoside_name(newname, 1, axis, ebone->head[axis], ebone->tail[axis])) {
-					ED_armature_bone_rename(bmain, arm, ebone->name, newname);
-					changed = true;
-				}
-			}
-		}
-
-		if (!changed) {
-			continue;
-		}
-
-		changed_multi = true;
-
-		/* Since we renamed stuff... */
-		DEG_id_tag_update(&ob->id, ID_RECALC_GEOMETRY);
-
-		/* Note, notifier might evolve. */
-		WM_event_add_notifier(C, NC_OBJECT | ND_POSE, ob);
-	}
-	MEM_freeN(objects);
-	return changed_multi ? OPERATOR_FINISHED : OPERATOR_CANCELLED;
+  ViewLayer *view_layer = CTX_data_view_layer(C);
+  Main *bmain = CTX_data_main(C);
+  char newname[MAXBONENAME];
+  const short axis = RNA_enum_get(op->ptr, "type");
+  bool changed_multi = false;
+
+  uint objects_len = 0;
+  Object **objects = BKE_view_layer_array_from_objects_in_edit_mode_unique_data(
+      view_layer, CTX_wm_view3d(C), &objects_len);
+  for (uint ob_index = 0; ob_index < objects_len; ob_index++) {
+    Object *ob = objects[ob_index];
+    bArmature *arm = ob->data;
+    bool changed = false;
+
+    /* Paranoia checks. */
+    if (ELEM(NULL, ob, ob->pose)) {
+      continue;
+    }
+
+    for (EditBone *ebone = arm->edbo->first; ebone; ebone = ebone->next) {
+      if (EBONE_EDITABLE(ebone)) {
+
+        /* We first need to do the flipped bone, then the original one.
+         * Otherwise we can't find the flipped one because of the bone name change. */
+        if (arm->flag & ARM_MIRROR_EDIT) {
+          EditBone *flipbone = ED_armature_ebone_get_mirrored(arm->edbo, ebone);
+          if ((flipbone) && !(flipbone->flag & BONE_SELECTED)) {
+            BLI_strncpy(newname, flipbone->name, sizeof(newname));
+            if (bone_autoside_name(newname, 1, axis, flipbone->head[axis], flipbone->tail[axis])) {
+              ED_armature_bone_rename(bmain, arm, flipbone->name, newname);
+              changed = true;
+            }
+          }
+        }
+
+        BLI_strncpy(newname, ebone->name, sizeof(newname));
+        if (bone_autoside_name(newname, 1, axis, ebone->head[axis], ebone->tail[axis])) {
+          ED_armature_bone_rename(bmain, arm, ebone->name, newname);
+          changed = true;
+        }
+      }
+    }
+
+    if (!changed) {
+      continue;
+    }
+
+    changed_multi = true;
+
+    /* Since we renamed stuff... */
+    DEG_id_tag_update(&ob->id, ID_RECALC_GEOMETRY);
+
+    /* Note, notifier might evolve. */
+    WM_event_add_notifier(C, NC_OBJECT | ND_POSE, ob);
+  }
+  MEM_freeN(objects);
+  return changed_multi ? OPERATOR_FINISHED : OPERATOR_CANCELLED;
 }
 
 void ARMATURE_OT_autoside_names(wmOperatorType *ot)
 {
-	static const EnumPropertyItem axis_items[] = {
-		{0, "XAXIS", 0, "X-Axis", "Left/Right"},
-		{1, "YAXIS", 0, "Y-Axis", "Front/Back"},
-		{2, "ZAXIS", 0, "Z-Axis", "Top/Bottom"},
-		{0, NULL, 0, NULL, NULL},
-	};
-
-	/* identifiers */
-	ot->name = "AutoName by Axis";
-	ot->idname = "ARMATURE_OT_autoside_names";
-	ot->description = "Automatically renames the selected bones according to which side of the target axis they fall on";
-
-	/* api callbacks */
-	ot->invoke = WM_menu_invoke;
-	ot->exec = armature_autoside_names_exec;
-	ot->poll = ED_operator_editarmature;
-
-	/* flags */
-	ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
-
-	/* settings */
-	ot->prop = RNA_def_enum(ot->srna, "type", axis_items, 0, "Axis", "Axis tag names with");
+  static const EnumPropertyItem axis_items[] = {
+      {0, "XAXIS", 0, "X-Axis", "Left/Right"},
+      {1, "YAXIS", 0, "Y-Axis", "Front/Back"},
+      {2, "ZAXIS", 0, "Z-Axis", "Top/Bottom"},
+      {0, NULL, 0, NULL, NULL},
+  };
+
+  /* identifiers */
+  ot->name = "AutoName by Axis";
+  ot->idname = "ARMATURE_OT_autoside_names";
+  ot->description =
+      "Automatically renames the selected bones according to which side of the target axis they "
+      "fall on";
+
+  /* api callbacks */
+  ot->invoke = WM_menu_invoke;
+  ot->exec = armature_autoside_names_exec;
+  ot->poll = ED_operator_editarmature;
+
+  /* flags */
+  ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
+
+  /* settings */
+  ot->prop = RNA_def_enum(ot->srna, "type", axis_items, 0, "Axis", "Axis tag names with");
 }