Cleanup: code comments punctuation / spacing

15 files changed, 29 insertions, 24 deletions

diff --git a/source/blender/editors/armature/pose_lib_2.c b/source/blender/editors/armature/pose_lib_2.c
index e146799a23c..32440f941ba 100644
--- a/source/blender/editors/armature/pose_lib_2.c
+++ b/source/blender/editors/armature/pose_lib_2.c
@@ -130,7 +130,7 @@ static void poselib_keytag_pose(bContext *C, Scene *scene, PoseBlendData *pbd)
   /* start tagging/keying */
   const bArmature *armature = pbd->ob->data;
   LISTBASE_FOREACH (bActionGroup *, agrp, &act->groups) {
-    /* only for selected bones unless there aren't any selected, in which case all are included  */
+    /* Only for selected bones unless there aren't any selected, in which case all are included. */
     bPoseChannel *pchan = BKE_pose_channel_find_name(pose, agrp->name);
     if (pchan == NULL) {
       continue;
diff --git a/source/blender/editors/interface/interface_handlers.c b/source/blender/editors/interface/interface_handlers.c
index b41db7818da..bfc03a95949 100644
--- a/source/blender/editors/interface/interface_handlers.c
+++ b/source/blender/editors/interface/interface_handlers.c
@@ -3929,7 +3929,7 @@ static void ui_do_but_textedit(
 
       /* exception that's useful for number buttons, some keyboard
        * numpads have a comma instead of a period */
-      if (ELEM(but->type, UI_BTYPE_NUM, UI_BTYPE_NUM_SLIDER)) { /* could use data->min*/
+      if (ELEM(but->type, UI_BTYPE_NUM, UI_BTYPE_NUM_SLIDER)) { /* Could use `data->min`. */
         if (event->type == EVT_PADPERIOD && ascii == ',') {
           ascii = '.';
           utf8_buf = NULL; /* force ascii fallback */
@@ -9486,7 +9486,7 @@ static void ui_list_activate_row_from_index(
     /* A bit ugly, set the active index in RNA directly. That's because a button that's
      * scrolled away in the list box isn't created at all.
      * The custom activate operator (#uiList.custom_activate_opname) is not called in this case
-     * (which may need the row button context).*/
+     * (which may need the row button context). */
     RNA_property_int_set(&listbox->rnapoin, listbox->rnaprop, index);
     RNA_property_update(C, &listbox->rnapoin, listbox->rnaprop);
     ui_apply_but_undo(listbox);
@@ -9505,7 +9505,7 @@ static int ui_list_get_increment(const uiList *ui_list, const int type, const in
     increment = (type == EVT_UPARROWKEY) ? -columns : columns;
   }
   else {
-    /* Left or right in grid layouts or any direction in single column layouts increments by 1.  */
+    /* Left or right in grid layouts or any direction in single column layouts increments by 1. */
     increment = ELEM(type, EVT_UPARROWKEY, EVT_LEFTARROWKEY, WHEELUPMOUSE) ? -1 : 1;
   }
 
diff --git a/source/blender/editors/render/render_update.c b/source/blender/editors/render/render_update.c
index 80b5623b9c3..fb9d11feb63 100644
--- a/source/blender/editors/render/render_update.c
+++ b/source/blender/editors/render/render_update.c
@@ -65,7 +65,7 @@
 
 /***************************** Render Engines ********************************/
 
-/* Update 3D viewport render or draw engine on changes to the scene or view settings . */
+/* Update 3D viewport render or draw engine on changes to the scene or view settings. */
 void ED_render_view3d_update(Depsgraph *depsgraph,
                              wmWindow *window,
                              ScrArea *area,
diff --git a/source/blender/editors/sculpt_paint/sculpt_filter_color.c b/source/blender/editors/sculpt_paint/sculpt_filter_color.c
index de9511bab6f..4b49bf2cefb 100644
--- a/source/blender/editors/sculpt_paint/sculpt_filter_color.c
+++ b/source/blender/editors/sculpt_paint/sculpt_filter_color.c
@@ -10,7 +10,7 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software  Foundation,
+ * along with this program; if not, write to the Free Software Foundation,
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  *
  * The Original Code is Copyright (C) 2020 Blender Foundation.
diff --git a/source/blender/editors/sculpt_paint/sculpt_paint_color.c b/source/blender/editors/sculpt_paint/sculpt_paint_color.c
index c3977b28178..936ebb7e8f7 100644
--- a/source/blender/editors/sculpt_paint/sculpt_paint_color.c
+++ b/source/blender/editors/sculpt_paint/sculpt_paint_color.c
@@ -10,7 +10,7 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software  Foundation,
+ * along with this program; if not, write to the Free Software Foundation,
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  *
  * The Original Code is Copyright (C) 2020 Blender Foundation.
diff --git a/source/blender/editors/space_graph/graph_edit.c b/source/blender/editors/space_graph/graph_edit.c
index 6f1b0bb0d7d..9f9869a854c 100644
--- a/source/blender/editors/space_graph/graph_edit.c
+++ b/source/blender/editors/space_graph/graph_edit.c
@@ -261,7 +261,7 @@ static int graphkeys_insertkey_exec(bContext *C, wmOperator *op)
     return OPERATOR_CANCELLED;
   }
 
-  /* Which channels to affect?. */
+  /* Which channels to affect? */
   mode = RNA_enum_get(op->ptr, "type");
 
   /* Insert keyframes. */
@@ -2814,7 +2814,7 @@ static int graph_fmodifier_paste_exec(bContext *C, wmOperator *op)
   }
   ANIM_animdata_freelist(&anim_data);
 
-  /* Successful or not?. */
+  /* Successful or not? */
   if (ok) {
     /* Set notifier that keyframes have changed. */
     WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
@@ -2873,7 +2873,7 @@ static int graph_driver_vars_copy_exec(bContext *C, wmOperator *op)
     ok = ANIM_driver_vars_copy(op->reports, fcu);
   }
 
-  /* Successful or not?. */
+  /* Successful or not? */
   if (ok) {
     return OPERATOR_FINISHED;
   }
@@ -2915,7 +2915,7 @@ static int graph_driver_vars_paste_exec(bContext *C, wmOperator *op)
     ok = ANIM_driver_vars_paste(op->reports, fcu, replace);
   }
 
-  /* Successful or not?. */
+  /* Successful or not? */
   if (ok) {
     /* Rebuild depsgraph, now that there are extra deps here. */
     DEG_relations_tag_update(CTX_data_main(C));
diff --git a/source/blender/editors/space_graph/graph_view.c b/source/blender/editors/space_graph/graph_view.c
index 31c53cde62c..036fd354c18 100644
--- a/source/blender/editors/space_graph/graph_view.c
+++ b/source/blender/editors/space_graph/graph_view.c
@@ -398,7 +398,7 @@ static void create_ghost_curves(bAnimContext *ac, int start, int end)
             ANIMFILTER_NODUPLIS);
   ANIM_animdata_filter(ac, &anim_data, filter, ac->data, ac->datatype);
 
-  /* Loop through filtered data and add keys between selected keyframes on every frame . */
+  /* Loop through filtered data and add keys between selected keyframes on every frame. */
   for (ale = anim_data.first; ale; ale = ale->next) {
     FCurve *fcu = (FCurve *)ale->key_data;
     FCurve *gcu = BKE_fcurve_create();
diff --git a/source/blender/editors/space_image/image_buttons.c b/source/blender/editors/space_image/image_buttons.c
index 4779a82948d..86349a64681 100644
--- a/source/blender/editors/space_image/image_buttons.c
+++ b/source/blender/editors/space_image/image_buttons.c
@@ -1189,7 +1189,7 @@ void uiTemplateImageLayers(uiLayout *layout, bContext *C, Image *ima, ImageUser
     const int menus_width = 160 * dpi_fac;
     const bool is_render_result = (ima->type == IMA_TYPE_R_RESULT);
 
-    /* use BKE_image_acquire_renderresult  so we get the correct slot in the menu */
+    /* Use BKE_image_acquire_renderresult so we get the correct slot in the menu. */
     rr = BKE_image_acquire_renderresult(scene, ima);
     uiblock_layer_pass_buttons(
         layout, ima, rr, iuser, menus_width, is_render_result ? &ima->render_slot : NULL);
diff --git a/source/blender/editors/space_node/node_relationships.cc b/source/blender/editors/space_node/node_relationships.cc
index 725c872e98f..61a874b2b2d 100644
--- a/source/blender/editors/space_node/node_relationships.cc
+++ b/source/blender/editors/space_node/node_relationships.cc
@@ -299,7 +299,7 @@ static void pick_input_link_by_link_intersect(const bContext *C,
 
   /* If no linked was picked in this call, try using the one picked in the previous call.
    * Not essential for the basic behavior, but can make interaction feel a bit better if
-   * the  mouse moves to the right and loses the "selection." */
+   * the mouse moves to the right and loses the "selection." */
   if (!link_to_pick) {
     bNodeLink *last_picked_link = nldrag->last_picked_multi_input_socket_link;
     if (last_picked_link) {
diff --git a/source/blender/editors/space_node/node_select.cc b/source/blender/editors/space_node/node_select.cc
index a081cc83481..6c07c41f451 100644
--- a/source/blender/editors/space_node/node_select.cc
+++ b/source/blender/editors/space_node/node_select.cc
@@ -1196,7 +1196,7 @@ static void node_find_create_label(const bNode *node, char *str, int maxlen)
   }
 }
 
-/* generic  search invoke */
+/* Generic search invoke. */
 static void node_find_update_fn(const struct bContext *C,
                                 void *UNUSED(arg),
                                 const char *str,
diff --git a/source/blender/editors/space_outliner/outliner_select.c b/source/blender/editors/space_outliner/outliner_select.c
index 35015356f0b..aaa52f6b649 100644
--- a/source/blender/editors/space_outliner/outliner_select.c
+++ b/source/blender/editors/space_outliner/outliner_select.c
@@ -697,7 +697,9 @@ static void tree_element_sequence_dup_activate(Scene *scene, TreeElement *UNUSED
 {
   Editing *ed = SEQ_editing_get(scene, false);
 
-  /* XXX  select_single_seq(seq, 1); */
+#if 0
+  select_single_seq(seq, 1);
+#endif
   Sequence *p = ed->seqbasep->first;
   while (p) {
     if ((!p->strip) || (!p->strip->stripdata) || (p->strip->stripdata->name[0] == '\0')) {
@@ -705,8 +707,11 @@ static void tree_element_sequence_dup_activate(Scene *scene, TreeElement *UNUSED
       continue;
     }
 
-    /* XXX: if (STREQ(p->strip->stripdata->name, seq->strip->stripdata->name)) select_single_seq(p,
-     * 0); */
+#if 0
+    if (STREQ(p->strip->stripdata->name, seq->strip->stripdata->name)) {
+      select_single_seq(p, 0);
+    }
+#endif
     p = p->next;
   }
 }
diff --git a/source/blender/editors/transform/transform_convert_mesh.c b/source/blender/editors/transform/transform_convert_mesh.c
index c1ee6edfef6..7377e47da3d 100644
--- a/source/blender/editors/transform/transform_convert_mesh.c
+++ b/source/blender/editors/transform/transform_convert_mesh.c
@@ -1981,7 +1981,7 @@ static void tc_mesh_partial_update(TransInfo *t,
 
   /* Promote the partial update types based on the previous state
    * so the values that no longer modified are reset before being left as-is.
-   * Needed for translation which can toggle snap-to-normal during transform.  */
+   * Needed for translation which can toggle snap-to-normal during transform. */
   const enum ePartialType partial_for_looptri = MAX2(partial_state->for_looptri,
                                                      partial_state_prev->for_looptri);
   const enum ePartialType partial_for_normals = MAX2(partial_state->for_normals,
diff --git a/source/blender/editors/transform/transform_generics.c b/source/blender/editors/transform/transform_generics.c
index aaac8e21cb9..e89ab6729d2 100644
--- a/source/blender/editors/transform/transform_generics.c
+++ b/source/blender/editors/transform/transform_generics.c
@@ -343,9 +343,9 @@ void initTransInfo(bContext *C, TransInfo *t, wmOperator *op, const wmEvent *eve
   }
   else {
     if (region) {
-      /* XXX for now, get View2D  from the active region */
+      /* XXX: For now, get View2D from the active region. */
       t->view = &region->v2d;
-      /* XXX for now, the center point is the midpoint of the data */
+      /* XXX: For now, the center point is the midpoint of the data. */
     }
     else {
       t->view = NULL;
diff --git a/source/blender/editors/transform/transform_orientations.c b/source/blender/editors/transform/transform_orientations.c
index 155250261de..33f4b06eb0e 100644
--- a/source/blender/editors/transform/transform_orientations.c
+++ b/source/blender/editors/transform/transform_orientations.c
@@ -537,10 +537,10 @@ short ED_transform_calc_orientation_from_type_ex(const bContext *C,
     case V3D_ORIENT_LOCAL: {
       if (ob) {
         if (ob->mode & OB_MODE_POSE) {
-          /* each bone moves on its own local axis, but  to avoid confusion,
+          /* Each bone moves on its own local axis, but to avoid confusion,
            * use the active pones axis for display T33575, this works as expected on a single
            * bone and users who select many bones will understand what's going on and what local
-           * means when they start transforming */
+           * means when they start transforming. */
           ED_getTransformOrientationMatrix(C, ob, obedit, pivot_point, r_mat);
         }
         else {
diff --git a/source/blender/editors/uvedit/uvedit_select.c b/source/blender/editors/uvedit/uvedit_select.c
index 7709b76290f..20aadb84b7b 100644
--- a/source/blender/editors/uvedit/uvedit_select.c
+++ b/source/blender/editors/uvedit/uvedit_select.c
@@ -1126,7 +1126,7 @@ static void uv_select_edgeloop_single_side_tag(const Scene *scene,
     while (l_step != NULL) {
 
       if (!uvedit_face_visible_test(scene, l_step->f) ||
-          /* Check the boundary is still a  boundary. */
+          /* Check the boundary is still a boundary. */
           (uvedit_loop_find_other_radial_loop_with_visible_face(
                scene, l_step, cd_loop_uv_offset) != NULL)) {
         break;