griefith/test2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Cleanup: spelling in comments

Browse Source
This commit is contained in:
Campbell Barton
2024-10-15 16:09:00 +11:00
parent 11e41f7276
commit aa6742897e
5 changed files with 7 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
source/blender/animrig/intern/nla.cc
View File

@@ -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.
*/

6
source/blender/blenkernel/intern/fcurve.cc
View File

@@ -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<FCurve *>(list->first); fcu; fcu = fcn) {
fcn = fcu->next;

2
source/blender/editors/space_node/node_draw.cc
View File

@@ -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) {

2
source/blender/gpu/glsl_preprocess/glsl_preprocess.hh
View File

@@ -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");
}

4
source/blender/makesrna/intern/rna_constraint.cc
View File

@@ -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) {