From aa6742897e86ffe26d2fb476d411ea682c96d5b8 Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Tue, 15 Oct 2024 16:09:00 +1100 Subject: [PATCH] Cleanup: spelling in comments --- source/blender/animrig/intern/nla.cc | 2 +- source/blender/blenkernel/intern/fcurve.cc | 6 ++---- source/blender/editors/space_node/node_draw.cc | 2 +- source/blender/gpu/glsl_preprocess/glsl_preprocess.hh | 2 +- source/blender/makesrna/intern/rna_constraint.cc | 4 ++-- 5 files changed, 7 insertions(+), 9 deletions(-) diff --git a/source/blender/animrig/intern/nla.cc b/source/blender/animrig/intern/nla.cc index 4776474f7b0..ba6a0fcbd46 100644 --- a/source/blender/animrig/intern/nla.cc +++ b/source/blender/animrig/intern/nla.cc @@ -35,7 +35,7 @@ bool assign_action(NlaStrip &strip, Action &action, ID &animated_id) * The generic code doesn't work for this. The first strip assignment would see the slot * `XXSlot`, and because it has never been used, just use it. This would change its name to, for * example, `OBSlot`. The second strip assignment would not see a 'virgin' slot, and thus not - * auto-select `OBSlot`. This behaviour makes sense when assigning Actions in the Action editor + * auto-select `OBSlot`. This behavior makes sense when assigning Actions in the Action editor * (it shouldn't automatically pick the first slot of matching ID type), but for the NLA I * (Sybren) feel that it could be a bit more 'enthousiastic' in auto-picking a slot. */ diff --git a/source/blender/blenkernel/intern/fcurve.cc b/source/blender/blenkernel/intern/fcurve.cc index 99f6f780f97..5881879178a 100644 --- a/source/blender/blenkernel/intern/fcurve.cc +++ b/source/blender/blenkernel/intern/fcurve.cc @@ -101,10 +101,8 @@ void BKE_fcurves_free(ListBase *list) return; } - /* Free data - no need to call remlink before freeing each curve, - * as we store reference to next, and freeing only touches the curve - * it's given. - */ + /* Free data, no need to call #BLI_remlink before freeing each curve, + * as we store reference to next, and freeing only touches the curve it's given. */ FCurve *fcn = nullptr; for (FCurve *fcu = static_cast(list->first); fcu; fcu = fcn) { fcn = fcu->next; diff --git a/source/blender/editors/space_node/node_draw.cc b/source/blender/editors/space_node/node_draw.cc index a454d5f413a..55b7fac257f 100644 --- a/source/blender/editors/space_node/node_draw.cc +++ b/source/blender/editors/space_node/node_draw.cc @@ -647,7 +647,7 @@ static void determine_visible_panels_impl_recursive(const bNode &node, r_result[panel_decl.index] = true; const bNodePanelState &panel_state = node.panel_states_array[panel_decl.index]; if (panel_state.is_collapsed()) { - /* The subpanels can't be visible if this panel is collapsed. */ + /* The sub-panels can't be visible if this panel is collapsed. */ return; } for (const nodes::ItemDeclaration *item_decl : panel_decl.items) { diff --git a/source/blender/gpu/glsl_preprocess/glsl_preprocess.hh b/source/blender/gpu/glsl_preprocess/glsl_preprocess.hh index 7662de0ce2e..e97128d68e0 100644 --- a/source/blender/gpu/glsl_preprocess/glsl_preprocess.hh +++ b/source/blender/gpu/glsl_preprocess/glsl_preprocess.hh @@ -106,7 +106,7 @@ class Preprocessor { return out_str; } } - /* Remove trailing whitespaces as they make the subsequent regex much slower. */ + /* Remove trailing white space as they make the subsequent regex much slower. */ std::regex regex("(\\ )*?\\n"); return std::regex_replace(out_str, regex, "\n"); } diff --git a/source/blender/makesrna/intern/rna_constraint.cc b/source/blender/makesrna/intern/rna_constraint.cc index c31aefd2709..2914d656637 100644 --- a/source/blender/makesrna/intern/rna_constraint.cc +++ b/source/blender/makesrna/intern/rna_constraint.cc @@ -755,11 +755,11 @@ static void rna_ActionConstraint_action_set(PointerRNA *ptr, PointerRNA value, R * The generic code doesn't work for this. The first assignment would see the slot * `XXSlot`, and because it has never been used, just use it. This would change its name to * `OBSlot`. The assignment to the Action Constraint would not see a 'virgin' slot, and thus not - * auto-select `OBSlot`. This behaviour makes sense when assigning Actions in the Action editor + * auto-select `OBSlot`. This behavior makes sense when assigning Actions in the Action editor * (it shouldn't automatically pick the first slot of matching ID type), but for the Action * Constraint I (Sybren) feel that it could be a bit more 'enthousiastic' in auto-picking a slot. * - * Note that this is the same behaviour as for NLA strips, albeit for a slightly different + * Note that this is the same behavior as for NLA strips, albeit for a slightly different * reason. Because of that it's not sharing code with the NLA. */ if (acon->action_slot_handle == Slot::unassigned && action->slots().size() == 1) {