From 23acedd4329a9d9825a463e7eb1de1204d1ed78b Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Fri, 7 Jul 2023 15:11:19 +1000 Subject: [PATCH] Cleanup: code-comments --- intern/cycles/blender/util.h | 2 +- source/blender/blenkernel/BKE_lib_query.h | 2 +- source/blender/blenkernel/intern/armature.c | 14 ++++++++------ source/blender/blenkernel/intern/constraint.c | 6 +++--- source/blender/blenkernel/intern/ipo.c | 2 +- source/blender/blenkernel/intern/lib_remap.c | 2 +- source/blender/blenkernel/intern/ocean.c | 8 ++++---- source/blender/blenkernel/intern/pbvh.cc | 2 +- source/blender/blenlib/intern/math_interp.c | 2 +- source/blender/bmesh/tools/bmesh_bevel.c | 10 +++++----- .../depsgraph/intern/node/deg_node_operation.h | 2 +- source/blender/draw/intern/draw_manager.c | 2 +- source/blender/editors/animation/keyframing.cc | 2 +- source/blender/editors/armature/armature_add.cc | 8 ++------ .../blender/editors/armature/armature_relations.cc | 2 +- source/blender/editors/armature/armature_utils.cc | 8 ++++---- source/blender/editors/armature/pose_transform.cc | 3 ++- .../blender/editors/gpencil_legacy/gpencil_merge.c | 2 +- source/blender/editors/mesh/editmesh_bevel.cc | 4 ++-- source/blender/editors/mesh/editmesh_extrude.cc | 2 +- source/blender/editors/mesh/editmesh_knife.cc | 2 +- source/blender/editors/object/object_constraint.cc | 2 +- source/blender/editors/object/object_ops.cc | 2 +- source/blender/editors/object/object_relations.cc | 4 ++-- source/blender/editors/object/object_shapekey.cc | 10 ++++++---- source/blender/io/collada/ArmatureExporter.cpp | 2 +- source/blender/makesrna/intern/rna_define.c | 2 +- source/blender/makesrna/intern/rna_image.cc | 2 +- source/blender/makesrna/intern/rna_pose.cc | 2 +- source/blender/modifiers/intern/MOD_normal_edit.cc | 2 +- source/blender/python/intern/bpy_rna.c | 2 +- tools/check_source/check_spelling_c_config.py | 4 ++++ 32 files changed, 63 insertions(+), 58 deletions(-) diff --git a/intern/cycles/blender/util.h b/intern/cycles/blender/util.h index 93b1e500e1a..494a0c8bbcf 100644 --- a/intern/cycles/blender/util.h +++ b/intern/cycles/blender/util.h @@ -89,7 +89,7 @@ static inline BL::Mesh object_to_mesh(BL::BlendData & /*data*/, if (b_ob_info.is_real_object_data()) { if (mesh) { - /* Make a copy to split faces if we use autosmooth, otherwise not needed. + /* Make a copy to split faces if we use auto-smooth, otherwise not needed. * Also in edit mode do we need to make a copy, to ensure data layers like * UV are not empty. */ if (mesh.is_editmode() || diff --git a/source/blender/blenkernel/BKE_lib_query.h b/source/blender/blenkernel/BKE_lib_query.h index dd12b498935..b8000f4912d 100644 --- a/source/blender/blenkernel/BKE_lib_query.h +++ b/source/blender/blenkernel/BKE_lib_query.h @@ -50,7 +50,7 @@ enum { IDWALK_CB_DIRECT_WEAK_LINK = (1 << 3), /** - * That ID is used as mere sub-data by its owner (only case currently: those root nodetrees in + * That ID is used as mere sub-data by its owner (only case currently: those root node-trees in * materials etc., and the Scene's master collections). * This means callback shall not *do* anything, only use this as informative data if it needs it. */ diff --git a/source/blender/blenkernel/intern/armature.c b/source/blender/blenkernel/intern/armature.c index c0b551e684d..c82a8cdd418 100644 --- a/source/blender/blenkernel/intern/armature.c +++ b/source/blender/blenkernel/intern/armature.c @@ -989,9 +989,10 @@ void BKE_pchan_bbone_spline_params_get(bPoseChannel *pchan, /* Transform previous point inside this bone space. */ if (bone->bbone_prev_type == BBONE_HANDLE_RELATIVE) { - /* Use delta movement (from restpose), and apply this relative to the current bone's head. */ + /* Use delta movement (from rest-pose), + * and apply this relative to the current bone's head. */ if (rest) { - /* In restpose, arm_head == pose_head */ + /* In rest-pose, arm_head == pose_head */ zero_v3(param->prev_h); done = true; } @@ -1043,9 +1044,10 @@ void BKE_pchan_bbone_spline_params_get(bPoseChannel *pchan, /* Transform next point inside this bone space. */ if (bone->bbone_next_type == BBONE_HANDLE_RELATIVE) { - /* Use delta movement (from restpose), and apply this relative to the current bone's tail. */ + /* Use delta movement (from rest-pose), + * and apply this relative to the current bone's tail. */ if (rest) { - /* In restpose, arm_head == pose_head */ + /* In rest-pose, arm_head == pose_head */ copy_v3_fl3(param->next_h, 0.0f, param->length, 0.0); done = true; } @@ -1091,12 +1093,12 @@ void BKE_pchan_bbone_spline_params_get(bPoseChannel *pchan, * - These properties allow users to hand-animate the * bone curve/shape, without having to resort to using * extra bones - * - The "bone" level offsets are for defining the restpose + * - The "bone" level offsets are for defining the rest-pose * shape of the bone (e.g. for curved eyebrows for example). * -> In the viewport, it's needed to define what the rest pose * looks like * -> For "rest == 0", we also still need to have it present - * so that we can "cancel out" this restpose when it comes + * so that we can "cancel out" this rest-pose when it comes * time to deform some geometry, it won't cause double transforms. * - The "pchan" level offsets are the ones that animators actually * end up animating diff --git a/source/blender/blenkernel/intern/constraint.c b/source/blender/blenkernel/intern/constraint.c index 33b602bb578..c1cea986886 100644 --- a/source/blender/blenkernel/intern/constraint.c +++ b/source/blender/blenkernel/intern/constraint.c @@ -1439,7 +1439,7 @@ static int followpath_get_tars(bConstraint *con, ListBase *list) bFollowPathConstraint *data = con->data; bConstraintTarget *ct; - /* standard target-getting macro for single-target constraints without subtargets */ + /* Standard target-getting macro for single-target constraints without sub-targets. */ SINGLETARGETNS_GET_TARS(con, data->tar, ct, list); return 1; @@ -3787,7 +3787,7 @@ static int clampto_get_tars(bConstraint *con, ListBase *list) bClampToConstraint *data = con->data; bConstraintTarget *ct; - /* standard target-getting macro for single-target constraints without subtargets */ + /* Standard target-getting macro for single-target constraints without sub-targets. */ SINGLETARGETNS_GET_TARS(con, data->tar, ct, list); return 1; @@ -4609,7 +4609,7 @@ static int splineik_get_tars(bConstraint *con, ListBase *list) bSplineIKConstraint *data = con->data; bConstraintTarget *ct; - /* standard target-getting macro for single-target constraints without subtargets */ + /* Standard target-getting macro for single-target constraints without sub-targets. */ SINGLETARGETNS_GET_TARS(con, data->tar, ct, list); return 1; diff --git a/source/blender/blenkernel/intern/ipo.c b/source/blender/blenkernel/intern/ipo.c index 1e656d8b195..c897f919409 100644 --- a/source/blender/blenkernel/intern/ipo.c +++ b/source/blender/blenkernel/intern/ipo.c @@ -1098,7 +1098,7 @@ static char *get_rna_access(ID *id, propname = "eval_time"; break; - /* XXX problematic blocktypes */ + /* XXX problematic block-types. */ case ID_SEQ: /* sequencer strip */ /* SEQ_FAC1: */ switch (adrcode) { diff --git a/source/blender/blenkernel/intern/lib_remap.c b/source/blender/blenkernel/intern/lib_remap.c index a2776bdc8d4..c8e9ecfb794 100644 --- a/source/blender/blenkernel/intern/lib_remap.c +++ b/source/blender/blenkernel/intern/lib_remap.c @@ -185,7 +185,7 @@ static int foreach_libblock_remap_callback(LibraryIDLinkCallbackData *cb_data) IDRemap *id_remap_data = cb_data->user_data; /* Those asserts ensure the general sanity of ID tags regarding 'embedded' ID data (root - * nodetrees and co). */ + * node-trees and co). */ BLI_assert(id_owner == id_remap_data->id_owner); BLI_assert(id_self == id_owner || (id_self->flag & LIB_EMBEDDED_DATA) != 0); diff --git a/source/blender/blenkernel/intern/ocean.c b/source/blender/blenkernel/intern/ocean.c index a1543906445..ba8598f224e 100644 --- a/source/blender/blenkernel/intern/ocean.c +++ b/source/blender/blenkernel/intern/ocean.c @@ -881,22 +881,22 @@ bool BKE_ocean_init(Ocean *o, o->_Lz = 0.001f; } - /* the +ve components and DC */ + /* The +VE components and DC. */ for (i = 0; i <= o->_M / 2; i++) { o->_kx[i] = 2.0f * (float)M_PI * i / o->_Lx; } - /* the -ve components */ + /* The -VE components. */ for (i = o->_M - 1, ii = 0; i > o->_M / 2; i--, ii++) { o->_kx[i] = -2.0f * (float)M_PI * ii / o->_Lx; } - /* the +ve components and DC */ + /* The +VE components and DC. */ for (i = 0; i <= o->_N / 2; i++) { o->_kz[i] = 2.0f * (float)M_PI * i / o->_Lz; } - /* the -ve components */ + /* The -VE components. */ for (i = o->_N - 1, ii = 0; i > o->_N / 2; i--, ii++) { o->_kz[i] = -2.0f * (float)M_PI * ii / o->_Lz; } diff --git a/source/blender/blenkernel/intern/pbvh.cc b/source/blender/blenkernel/intern/pbvh.cc index f10e4f0e8b7..bb02ecc698f 100644 --- a/source/blender/blenkernel/intern/pbvh.cc +++ b/source/blender/blenkernel/intern/pbvh.cc @@ -2717,7 +2717,7 @@ void BKE_pbvh_raycast_project_ray_root( float dist = max[2] - min[2]; - /* Build ray interval from z dimen of bounds. */ + /* Build ray interval from z dimension of bounds. */ mid_v3_v3v3(cent, bb_min_root, bb_max_root); madd_v3_v3v3fl(ray_start_new, cent, ray_normal, -dist); madd_v3_v3v3fl(ray_end_new, cent, ray_normal, dist); diff --git a/source/blender/blenlib/intern/math_interp.c b/source/blender/blenlib/intern/math_interp.c index 017b839ca96..28e14e8e2ef 100644 --- a/source/blender/blenlib/intern/math_interp.c +++ b/source/blender/blenlib/intern/math_interp.c @@ -487,7 +487,7 @@ void BLI_bilinear_interpolation_wrap_char(const uchar *buffer, * by Ned Greene and Paul S. Heckbert (1986) ***************************************************************************/ -/* Table of (exp(ar) - exp(a)) / (1 - exp(a)) for r in range [0, 1] and a = -2 +/* Table of `(exp(ar) - exp(a)) / (1 - exp(a))` for `r` in range [0, 1] and `a = -2`. * used instead of actual gaussian, * otherwise at high texture magnifications circular artifacts are visible. */ #define EWA_MAXIDX 255 diff --git a/source/blender/bmesh/tools/bmesh_bevel.c b/source/blender/bmesh/tools/bmesh_bevel.c index 4103155d84a..558bca92fc9 100644 --- a/source/blender/bmesh/tools/bmesh_bevel.c +++ b/source/blender/bmesh/tools/bmesh_bevel.c @@ -2484,16 +2484,16 @@ static void bevel_harden_normals(BevelParams *bp, BMesh *bm) int cd_clnors_offset = CustomData_get_offset(&bm->ldata, CD_CUSTOMLOOPNORMAL); - /* If there is not already a custom split normal layer then making one (with BM_lnorspace_update) - * will not respect the autosmooth angle between smooth faces. To get that to happen, we have - * to mark the sharpen the edges that are only sharp because of the angle test -- otherwise would - * be smooth. */ + /* If there is not already a custom split normal layer then making one + * (with #BM_lnorspace_update) will not respect the auto-smooth angle between smooth faces. + * To get that to happen, we have to mark the sharpen the edges that are only sharp because + * of the angle test -- otherwise would be smooth. */ if (cd_clnors_offset == -1) { BM_edges_sharp_from_angle_set(bm, bp->smoothresh); bevel_edges_sharp_boundary(bm, bp); } - /* Ensure that bm->lnor_spacearr has properly stored loop normals. + /* Ensure that `bm->lnor_spacearr` has properly stored loop normals. * Side effect: ensures loop indices. */ BM_lnorspace_update(bm); diff --git a/source/blender/depsgraph/intern/node/deg_node_operation.h b/source/blender/depsgraph/intern/node/deg_node_operation.h index d027da8b33f..60681037b95 100644 --- a/source/blender/depsgraph/intern/node/deg_node_operation.h +++ b/source/blender/depsgraph/intern/node/deg_node_operation.h @@ -120,7 +120,7 @@ enum class OperationCode { /* Bone. ---------------------------------------------------------------- */ /* Bone local transforms - entry point */ BONE_LOCAL, - /* Pose-space conversion (includes parent + restpose, */ + /* Pose-space conversion (includes parent + rest-pose. */ BONE_POSE_PARENT, /* Constraints */ BONE_CONSTRAINTS, diff --git a/source/blender/draw/intern/draw_manager.c b/source/blender/draw/intern/draw_manager.c index c7cdb1176f3..e65bd206f91 100644 --- a/source/blender/draw/intern/draw_manager.c +++ b/source/blender/draw/intern/draw_manager.c @@ -828,7 +828,7 @@ typedef struct IdDdtTemplate { static bool id_type_can_have_drawdata(const short id_type) { /* Only some ID-blocks have this info for now */ - /* TODO: finish adding this for the other blocktypes */ + /* TODO: finish adding this for the other block-types. */ switch (id_type) { /* has DrawData */ case ID_OB: diff --git a/source/blender/editors/animation/keyframing.cc b/source/blender/editors/animation/keyframing.cc index cd52ad34cdc..b8530a89d70 100644 --- a/source/blender/editors/animation/keyframing.cc +++ b/source/blender/editors/animation/keyframing.cc @@ -856,7 +856,7 @@ enum { * This helper function determines if visual-keyframing should be used when * inserting keyframes for the given channel. As visual-keyframing only works * on Object and Pose-Channel blocks, this should only get called for those - * blocktypes, when using "standard" keying but 'Visual Keying' option in Auto-Keying + * block-types, when using "standard" keying but 'Visual Keying' option in Auto-Keying * settings is on. */ static bool visualkey_can_use(PointerRNA *ptr, PropertyRNA *prop) diff --git a/source/blender/editors/armature/armature_add.cc b/source/blender/editors/armature/armature_add.cc index 990315a51f3..770b947605b 100644 --- a/source/blender/editors/armature/armature_add.cc +++ b/source/blender/editors/armature/armature_add.cc @@ -1054,9 +1054,7 @@ static int armature_duplicate_selected_exec(bContext *C, wmOperator *op) ebone->bbone_next = ebone_iter->bbone_next->temp.ebone; } - /* Lets try to fix any constraint subtargets that might - * have been duplicated - */ + /* Lets try to fix any constraint subtargets that might have been duplicated. */ updateDuplicateSubtarget(ebone, arm->edbo, ob, false); } } @@ -1315,9 +1313,7 @@ static int armature_symmetrize_exec(bContext *C, wmOperator *op) ebone->bbone_prev_flag = ebone_iter->bbone_prev_flag; ebone->bbone_next_flag = ebone_iter->bbone_next_flag; - /* Lets try to fix any constraint subtargets that might - * have been duplicated - */ + /* Lets try to fix any constraint sub-targets that might have been duplicated. */ updateDuplicateSubtarget(ebone, arm->edbo, obedit, true); /* Try to update constraint options so that they are mirrored as well * (need to supply bone_iter as well in case we are working with existing bones) */ diff --git a/source/blender/editors/armature/armature_relations.cc b/source/blender/editors/armature/armature_relations.cc index bfa1d0d89d7..4429f166fc0 100644 --- a/source/blender/editors/armature/armature_relations.cc +++ b/source/blender/editors/armature/armature_relations.cc @@ -109,7 +109,7 @@ static void joined_armature_fix_links_constraints(Main *bmain, } } -/* userdata for joined_armature_fix_animdata_cb() */ +/** User-data for #joined_armature_fix_animdata_cb(). */ struct tJoinArmature_AdtFixData { Main *bmain; diff --git a/source/blender/editors/armature/armature_utils.cc b/source/blender/editors/armature/armature_utils.cc index b1a208b2c08..8faed298f66 100644 --- a/source/blender/editors/armature/armature_utils.cc +++ b/source/blender/editors/armature/armature_utils.cc @@ -598,8 +598,8 @@ static void armature_finalize_restpose(ListBase *bonelist, ListBase *editbonelis for (curBone = static_cast(bonelist->first); curBone; curBone = curBone->next) { /* Set bone's local head/tail. - * Note that it's important to use final parent's restpose (arm_mat) here, - * instead of setting those values from editbone's matrix (see #46010). */ + * Note that it's important to use final parent's rest-pose (arm_mat) here, + * instead of setting those values from edit-bone's matrix (see #46010). */ if (curBone->parent) { float parmat_inv[4][4]; @@ -617,7 +617,7 @@ static void armature_finalize_restpose(ListBase *bonelist, ListBase *editbonelis copy_v3_v3(curBone->tail, curBone->arm_tail); } - /* Set local matrix and arm_mat (restpose). + /* Set local matrix and arm_mat (rest-pose). * Do not recurse into children here, armature_finalize_restpose() is already recursive. */ BKE_armature_where_is_bone(curBone, curBone->parent, false); @@ -772,7 +772,7 @@ void ED_armature_from_edit(Main *bmain, bArmature *arm) } } - /* Finalize definition of restpose data (roll, bone_mat, arm_mat, head/tail...). */ + /* Finalize definition of rest-pose data (roll, bone_mat, arm_mat, head/tail...). */ armature_finalize_restpose(&arm->bonebase, arm->edbo); BKE_armature_bone_hash_make(arm); diff --git a/source/blender/editors/armature/pose_transform.cc b/source/blender/editors/armature/pose_transform.cc index c7f430bcee5..0013e063577 100644 --- a/source/blender/editors/armature/pose_transform.cc +++ b/source/blender/editors/armature/pose_transform.cc @@ -503,9 +503,10 @@ void POSE_OT_armature_apply(wmOperatorType *ot) /* -------------------------------------------------------------------- */ /** \name Apply Visual Transform Operator + * + * Set the current pose as the rest-pose. * \{ */ -/* set the current pose as the restpose */ static int pose_visual_transform_apply_exec(bContext *C, wmOperator * /*op*/) { const Scene *scene = CTX_data_scene(C); diff --git a/source/blender/editors/gpencil_legacy/gpencil_merge.c b/source/blender/editors/gpencil_legacy/gpencil_merge.c index 2fc7b9eaada..beac5513149 100644 --- a/source/blender/editors/gpencil_legacy/gpencil_merge.c +++ b/source/blender/editors/gpencil_legacy/gpencil_merge.c @@ -377,7 +377,7 @@ static int gpencil_analyze_strokes(tGPencilPointCache *src_array, for (i = 0; i < totpoints; i++) { sort_pt = &src_array[i]; - /* avoid dups */ + /* Avoid duplicates. */ if (BLI_ghash_haskey(all_strokes, sort_pt->gps)) { continue; } diff --git a/source/blender/editors/mesh/editmesh_bevel.cc b/source/blender/editors/mesh/editmesh_bevel.cc index 54aa4a49f1e..e2d5e29f60c 100644 --- a/source/blender/editors/mesh/editmesh_bevel.cc +++ b/source/blender/editors/mesh/editmesh_bevel.cc @@ -338,7 +338,7 @@ static bool edbm_bevel_calc(wmOperator *op) Mesh *me = static_cast(obedit->data); if (harden_normals && !(me->flag & ME_AUTOSMOOTH)) { - /* harden_normals only has a visible effect if autosmooth is on, so turn it on */ + /* `harden_normals` only has a visible effect if auto-smooth is on, so turn it on. */ me->flag |= ME_AUTOSMOOTH; } @@ -446,7 +446,7 @@ static void edbm_bevel_cancel(bContext *C, wmOperator *op) edbm_bevel_exit(C, op); - /* need to force redisplay or we may still view the modified result */ + /* Need to force re-display or we may still view the modified result. */ ED_region_tag_redraw(CTX_wm_region(C)); } diff --git a/source/blender/editors/mesh/editmesh_extrude.cc b/source/blender/editors/mesh/editmesh_extrude.cc index 38ee757b0ac..72090408922 100644 --- a/source/blender/editors/mesh/editmesh_extrude.cc +++ b/source/blender/editors/mesh/editmesh_extrude.cc @@ -862,7 +862,7 @@ static int edbm_dupli_extrude_cursor_invoke(bContext *C, wmOperator *op, const w EDBM_op_callf( vc.em, op, "rotate verts=%hv cent=%v matrix=%m3", BM_ELEM_SELECT, local_center, mat); - /* also project the source, for retopo workflow */ + /* Also project the source, for retopology workflow. */ if (use_proj) { EDBM_project_snap_verts(C, depsgraph, vc.region, vc.obedit, vc.em); } diff --git a/source/blender/editors/mesh/editmesh_knife.cc b/source/blender/editors/mesh/editmesh_knife.cc index 52431a32e20..f059a1c6c1c 100644 --- a/source/blender/editors/mesh/editmesh_knife.cc +++ b/source/blender/editors/mesh/editmesh_knife.cc @@ -1679,7 +1679,7 @@ static void knife_add_to_vert_edges(KnifeTool_OpData *kcd, KnifeEdge *kfe) knife_append_list(kcd, &kfe->v2->edges, kfe); } -/* Add faces of an edge to a KnifeVert's faces list. No checks for dups. */ +/* Add faces of an edge to a KnifeVert's faces list. No checks for duplicates. */ static void knife_add_edge_faces_to_vert(KnifeTool_OpData *kcd, KnifeVert *kfv, BMEdge *e) { BMIter bmiter; diff --git a/source/blender/editors/object/object_constraint.cc b/source/blender/editors/object/object_constraint.cc index c02af0ae211..c2d36a15ba6 100644 --- a/source/blender/editors/object/object_constraint.cc +++ b/source/blender/editors/object/object_constraint.cc @@ -2423,7 +2423,7 @@ static int constraint_add_exec( int scriptint = 0; /* popup a list of usable scripts */ menustr = buildmenu_pyconstraints(bmain, nullptr, &scriptint); - /* XXX scriptint = pupmenu(menustr); */ + // scriptint = pupmenu(menustr); /* XXX */ MEM_freeN(menustr); /* only add constraint if a script was chosen */ diff --git a/source/blender/editors/object/object_ops.cc b/source/blender/editors/object/object_ops.cc index 57ff2ac823d..edab2e6429f 100644 --- a/source/blender/editors/object/object_ops.cc +++ b/source/blender/editors/object/object_ops.cc @@ -332,7 +332,7 @@ void ED_keymap_object(wmKeyConfig *keyconf) keymap = WM_keymap_ensure(keyconf, "Object Non-modal", 0, 0); /* Object Mode ---------------------------------------------------------------- */ - /* NOTE: this keymap gets disabled in non-objectmode. */ + /* NOTE: this keymap gets disabled in non-object-mode. */ keymap = WM_keymap_ensure(keyconf, "Object Mode", 0, 0); keymap->poll = object_mode_poll; } diff --git a/source/blender/editors/object/object_relations.cc b/source/blender/editors/object/object_relations.cc index 1ce387f6345..758b893e391 100644 --- a/source/blender/editors/object/object_relations.cc +++ b/source/blender/editors/object/object_relations.cc @@ -2199,8 +2199,8 @@ static void make_local_material_tag(Material *ma) ma->id.tag &= ~LIB_TAG_PRE_EXISTING; make_local_animdata_tag(BKE_animdata_from_id(&ma->id)); - /* About nodetrees: root one is made local together with material, * others we keep linked - * for now... */ + /* About node-trees: root one is made local together with material, + * others we keep linked (for now). */ } } diff --git a/source/blender/editors/object/object_shapekey.cc b/source/blender/editors/object/object_shapekey.cc index 348e162e438..7f716e0b5ea 100644 --- a/source/blender/editors/object/object_shapekey.cc +++ b/source/blender/editors/object/object_shapekey.cc @@ -161,10 +161,12 @@ static bool object_shape_key_mirror( /* half but found up odd value */ const int pntsu_half = (lt->pntsu / 2) + (lt->pntsu % 2); - /* currently editmode isn't supported by mesh so - * ignore here for now too */ - - /* if (lt->editlatt) lt = lt->editlatt->latt; */ + /* Currently edit-mode isn't supported by mesh so ignore here for now too. */ +#if 0 + if (lt->editlatt) { + lt = lt->editlatt->latt; + } +#endif for (w = 0; w < lt->pntsw; w++) { for (v = 0; v < lt->pntsv; v++) { diff --git a/source/blender/io/collada/ArmatureExporter.cpp b/source/blender/io/collada/ArmatureExporter.cpp index cd69bd7c9d7..c8405e103d4 100644 --- a/source/blender/io/collada/ArmatureExporter.cpp +++ b/source/blender/io/collada/ArmatureExporter.cpp @@ -259,7 +259,7 @@ void ArmatureExporter::add_bone_transform(Object *ob_arm, Bone *bone, COLLADASW: if (!has_restmat) { - /* Have no restpose matrix stored, try old style <= Blender 2.78 */ + /* Have no rest-pose matrix stored, try old style <= Blender 2.78. */ bc_create_restpose_mat(this->export_settings, bone, bone_rest_mat, bone->arm_mat, true); diff --git a/source/blender/makesrna/intern/rna_define.c b/source/blender/makesrna/intern/rna_define.c index 48586d83b85..2b401ec3234 100644 --- a/source/blender/makesrna/intern/rna_define.c +++ b/source/blender/makesrna/intern/rna_define.c @@ -765,7 +765,7 @@ void RNA_define_fallback_property_update(int noteflag, const char *updatefunc) void RNA_struct_free_extension(StructRNA *srna, ExtensionRNA *rna_ext) { #ifdef RNA_RUNTIME - rna_ext->free(rna_ext->data); /* decref's the PyObject that the srna owns */ + rna_ext->free(rna_ext->data); /* Decrefs the PyObject that the `srna` owns. */ RNA_struct_blender_type_set(srna, NULL); /* FIXME: this gets accessed again. */ /* NULL the srna's value so RNA_struct_free won't complain of a leak */ diff --git a/source/blender/makesrna/intern/rna_image.cc b/source/blender/makesrna/intern/rna_image.cc index 2a4e9f0ecf5..82007630081 100644 --- a/source/blender/makesrna/intern/rna_image.cc +++ b/source/blender/makesrna/intern/rna_image.cc @@ -241,7 +241,7 @@ static void rna_ImageUser_update(Main *bmain, Scene *scene, PointerRNA *ptr) if (id) { if (GS(id->name) == ID_NT) { - /* Special update for nodetrees. */ + /* Special update for node-trees. */ BKE_ntree_update_tag_image_user_changed((bNodeTree *)id, iuser); ED_node_tree_propagate_change(nullptr, bmain, nullptr); } diff --git a/source/blender/makesrna/intern/rna_pose.cc b/source/blender/makesrna/intern/rna_pose.cc index 972b0d02f48..bcac0682cdc 100644 --- a/source/blender/makesrna/intern/rna_pose.cc +++ b/source/blender/makesrna/intern/rna_pose.cc @@ -1117,7 +1117,7 @@ static void rna_def_pose_channel(BlenderRNA *brna) RNA_def_property_ui_text(prop, "Rotation Mode", ""); RNA_def_property_update(prop, NC_OBJECT | ND_POSE, "rna_Pose_update"); - /* Curved bones settings - Applied on top of restpose values */ + /* Curved bones settings - Applied on top of rest-pose values. */ rna_def_bone_curved_common(srna, true, false); /* Custom BBone next/prev sources */ diff --git a/source/blender/modifiers/intern/MOD_normal_edit.cc b/source/blender/modifiers/intern/MOD_normal_edit.cc index 5c68ce3b40b..8cdba660565 100644 --- a/source/blender/modifiers/intern/MOD_normal_edit.cc +++ b/source/blender/modifiers/intern/MOD_normal_edit.cc @@ -486,7 +486,7 @@ static Mesh *normalEditModifier_do(NormalEditModifierData *enmd, (enmd->mix_factor == 1.0f) && (enmd->defgrp_name[0] == '\0') && (enmd->mix_limit == float(M_PI))); - /* Do not run that modifier at all if autosmooth is disabled! */ + /* Do not run that modifier at all if auto-smooth is disabled! */ if (!is_valid_target_with_error(ctx->object, enmd) || mesh->totloop == 0) { return mesh; } diff --git a/source/blender/python/intern/bpy_rna.c b/source/blender/python/intern/bpy_rna.c index 2086a79df9e..4c2d6e6ddf1 100644 --- a/source/blender/python/intern/bpy_rna.c +++ b/source/blender/python/intern/bpy_rna.c @@ -8026,7 +8026,7 @@ static int deferred_register_prop(StructRNA *srna, PyObject *key, PyObject *item Py_DECREF(args_fake); /* Free's py_srna_cobject too. */ } else { - /* _must_ print before decreffing args_fake. */ + /* _must_ print before decrefing args_fake. */ PyErr_Print(); PyErr_Clear(); diff --git a/tools/check_source/check_spelling_c_config.py b/tools/check_source/check_spelling_c_config.py index b8d15b35d8c..63374341c6c 100644 --- a/tools/check_source/check_spelling_c_config.py +++ b/tools/check_source/check_spelling_c_config.py @@ -129,6 +129,7 @@ dict_custom = { "extrema", "fallbacks", "finalizer", + "flippable", "flushable", "formatter", "formatters", @@ -265,6 +266,7 @@ dict_custom = { "recursing", "recursivity", "redefinitions", + "redisplay", "redistributions", "registerable", "reimplement", @@ -300,6 +302,7 @@ dict_custom = { "sidedness", "simplices", "situationally", + "skinnable", "skippable", "sortable", "stepsize", @@ -314,6 +317,7 @@ dict_custom = { "subrange", "subtractive", "superset", + "symmetrizable", "symmetrize", "symmetrized", "targetless",