Cleanup: consistent use of tags: NOTE/TODO/FIXME/XXX

Also use doxy style function reference `#` prefix chars when
referencing identifiers.

12 files changed, 19 insertions, 19 deletions

diff --git a/source/blender/bmesh/operators/bmo_connect_pair.c b/source/blender/bmesh/operators/bmo_connect_pair.c
index b57c8686e1c..660633e8a0f 100644
--- a/source/blender/bmesh/operators/bmo_connect_pair.c
+++ b/source/blender/bmesh/operators/bmo_connect_pair.c
@@ -571,7 +571,7 @@ static void bm_vert_pair_to_matrix(BMVert *v_pair[2], float r_unit_mat[3][3])
     }
 
     /* create a new 'basis_nor' from the best direction.
-     * note: we could add the directions,
+     * NOTE: we could add the directions,
      * but this more often gives 45d rotated matrix, so just use the best one. */
     copy_v3_v3(basis_nor, axis_pair[axis_pair[0].angle_cos < axis_pair[1].angle_cos].nor);
     project_plane_normalized_v3_v3v3(basis_nor, basis_nor, basis_dir);
diff --git a/source/blender/bmesh/operators/bmo_create.c b/source/blender/bmesh/operators/bmo_create.c
index f38e2e08f33..2e2a7e0964e 100644
--- a/source/blender/bmesh/operators/bmo_create.c
+++ b/source/blender/bmesh/operators/bmo_create.c
@@ -184,7 +184,7 @@ void bmo_contextual_create_exec(BMesh *bm, BMOperator *op)
   /* -------------------------------------------------------------------- */
   /* Dissolve Face */
   if (totf != 0) { /* should be (totf > 1)... see below */
-    /* note: allow this to run on single faces so running on a single face
+    /* NOTE: allow this to run on single faces so running on a single face
      * won't go on to create a face, treating them as random */
     BMOperator op_sub;
     BMO_op_initf(bm, &op_sub, op->flag, "dissolve_faces faces=%ff", ELE_NEW);
@@ -204,7 +204,7 @@ void bmo_contextual_create_exec(BMesh *bm, BMOperator *op)
   /* Fill EdgeLoop's - fills isolated loops, different from edgenet */
   if (tote > 2) {
     BMOperator op_sub;
-    /* note: in most cases 'edgenet_fill' will handle this case since in common cases
+    /* NOTE: in most cases 'edgenet_fill' will handle this case since in common cases
      * users fill in empty spaces, however its possible to have an edge selection around
      * existing geometry that makes 'edgenet_fill' fail. */
     BMO_op_initf(bm, &op_sub, op->flag, "edgeloop_fill edges=%fe", ELE_NEW);
@@ -280,7 +280,7 @@ void bmo_contextual_create_exec(BMesh *bm, BMOperator *op)
    * last resort when all else fails.
    */
   if (totv > 2) {
-    /* TODO, some of these vertices may be connected by edges,
+    /* TODO: some of these vertices may be connected by edges,
      * this connectivity could be used rather than treating
      * them as a bunch of isolated verts. */
 
diff --git a/source/blender/bmesh/operators/bmo_edgenet.c b/source/blender/bmesh/operators/bmo_edgenet.c
index 53a2a3b2bba..5449762b83c 100644
--- a/source/blender/bmesh/operators/bmo_edgenet.c
+++ b/source/blender/bmesh/operators/bmo_edgenet.c
@@ -52,7 +52,7 @@ void bmo_edgenet_fill_exec(BMesh *bm, BMOperator *op)
   BMO_slot_buffer_hflag_enable(bm, op->slots_in, "edges", BM_EDGE, BM_ELEM_TAG, false);
 
   BM_mesh_elem_hflag_disable_all(bm, BM_FACE, BM_ELEM_TAG, false);
-  BM_mesh_edgenet(bm, true, true); /* TODO, sides */
+  BM_mesh_edgenet(bm, true, true); /* TODO: sides. */
 
   BMO_slot_buffer_from_enabled_hflag(bm, op, op->slots_out, "faces.out", BM_FACE, BM_ELEM_TAG);
 
diff --git a/source/blender/bmesh/operators/bmo_fill_edgeloop.c b/source/blender/bmesh/operators/bmo_fill_edgeloop.c
index 16f9922b21d..2b481542463 100644
--- a/source/blender/bmesh/operators/bmo_fill_edgeloop.c
+++ b/source/blender/bmesh/operators/bmo_fill_edgeloop.c
@@ -85,7 +85,7 @@ void bmo_edgeloop_fill_exec(BMesh *bm, BMOperator *op)
   }
 
   if (ok) {
-    /* note: in the case of multiple loops, this over-allocs (which is fine) */
+    /* NOTE: in the case of multiple loops, this over-allocs (which is fine). */
     BMVert **f_verts = MEM_mallocN(sizeof(*verts) * totv, __func__);
     BMIter eiter;
 
diff --git a/source/blender/bmesh/operators/bmo_fill_holes.c b/source/blender/bmesh/operators/bmo_fill_holes.c
index 5801db582e1..64107fefe73 100644
--- a/source/blender/bmesh/operators/bmo_fill_holes.c
+++ b/source/blender/bmesh/operators/bmo_fill_holes.c
@@ -35,7 +35,7 @@ void bmo_holes_fill_exec(BMesh *bm, BMOperator *op)
   BM_mesh_elem_hflag_disable_all(bm, BM_EDGE | BM_FACE, BM_ELEM_TAG, false);
   BMO_slot_buffer_hflag_enable(bm, op->slots_in, "edges", BM_EDGE, BM_ELEM_TAG, false);
 
-  BM_mesh_edgenet(bm, true, true);  // TODO, sides
+  BM_mesh_edgenet(bm, true, true);  // TODO: sides
 
   /* bad - remove faces after as a workaround */
   if (sides != 0) {
diff --git a/source/blender/bmesh/operators/bmo_hull.c b/source/blender/bmesh/operators/bmo_hull.c
index 956d55695d6..dd2fe52293f 100644
--- a/source/blender/bmesh/operators/bmo_hull.c
+++ b/source/blender/bmesh/operators/bmo_hull.c
@@ -278,7 +278,7 @@ static void hull_remove_overlapping(BMesh *bm,
         }
       }
 
-      /* Note: can't change ghash while iterating, so mark
+      /* NOTE: can't change ghash while iterating, so mark
        * with 'skip' flag rather than deleting triangles */
       if (BM_vert_in_face(t->v[1], f) && BM_vert_in_face(t->v[2], f) && f_on_hull) {
         t->skip = true;
@@ -505,7 +505,7 @@ static void hull_from_bullet(BMesh *bm, BMOperator *op, BLI_mempool *hull_triang
       BLI_array_grow_items(fvi, len);
       plConvexHullGetFaceVertices(hull, i, fvi);
 
-      /* Note: here we throw away any NGons from Bullet and turn
+      /* NOTE: here we throw away any NGons from Bullet and turn
        * them into triangle fans. Would be nice to use these
        * directly, but will have to wait until HullTriangle goes
        * away (TODO) */
diff --git a/source/blender/bmesh/operators/bmo_inset.c b/source/blender/bmesh/operators/bmo_inset.c
index 3f6619cc2ee..4833290aa0b 100644
--- a/source/blender/bmesh/operators/bmo_inset.c
+++ b/source/blender/bmesh/operators/bmo_inset.c
@@ -822,7 +822,7 @@ void bmo_inset_region_exec(BMesh *bm, BMOperator *op)
     /* this could go in its own loop,
      * only use the 'es->l->f' so we don't store loops for faces which have no mixed selection
      *
-     * note: faces on the other side of the inset will be interpolated too since this is hard to
+     * NOTE: faces on the other side of the inset will be interpolated too since this is hard to
      * detect, just allow it even though it will cause some redundant interpolation */
     if (use_interpolate) {
       BMIter viter;
diff --git a/source/blender/bmesh/operators/bmo_join_triangles.c b/source/blender/bmesh/operators/bmo_join_triangles.c
index c3e96a9b08e..8b456bb0925 100644
--- a/source/blender/bmesh/operators/bmo_join_triangles.c
+++ b/source/blender/bmesh/operators/bmo_join_triangles.c
@@ -44,7 +44,7 @@ static float quad_calc_error(const float v1[3],
 {
   /* Gives a 'weight' to a pair of triangles that join an edge
    * to decide how good a join they would make. */
-  /* Note: this is more complicated than it needs to be and should be cleaned up. */
+  /* NOTE: this is more complicated than it needs to be and should be cleaned up. */
   float error = 0.0f;
 
   /* Normal difference */
diff --git a/source/blender/bmesh/operators/bmo_offset_edgeloops.c b/source/blender/bmesh/operators/bmo_offset_edgeloops.c
index 2c7e478b549..98a8432cfb1 100644
--- a/source/blender/bmesh/operators/bmo_offset_edgeloops.c
+++ b/source/blender/bmesh/operators/bmo_offset_edgeloops.c
@@ -221,7 +221,7 @@ void bmo_offset_edgeloops_exec(BMesh *bm, BMOperator *op)
               }
             }
 
-            /* Note: instead of duplicate code in alternate direction,
+            /* NOTE: instead of duplicate code in alternate direction,
              * we can be sure to hit the other vertex, so the code above runs. */
 #if 0
             else if (BM_elem_index_get(l->prev->v) == -1) {
diff --git a/source/blender/bmesh/operators/bmo_rotate_edges.c b/source/blender/bmesh/operators/bmo_rotate_edges.c
index b684aa390cf..f05f6489c99 100644
--- a/source/blender/bmesh/operators/bmo_rotate_edges.c
+++ b/source/blender/bmesh/operators/bmo_rotate_edges.c
@@ -191,7 +191,7 @@ static void bm_rotate_edges_shared(
 
           edges_len_rotate += 1;
 
-          /* Note: we could validate all edges which have not been rotated
+          /* NOTE: we could validate all edges which have not been rotated
            * (not just previously degenerate edges).
            * However there is no real need -
            * they can be left until they're popped off the queue. */
diff --git a/source/blender/bmesh/operators/bmo_subdivide.c b/source/blender/bmesh/operators/bmo_subdivide.c
index ae4fa65b55c..7311c94a0d8 100644
--- a/source/blender/bmesh/operators/bmo_subdivide.c
+++ b/source/blender/bmesh/operators/bmo_subdivide.c
@@ -79,7 +79,7 @@ typedef void (*subd_pattern_fill_fp)(BMesh *bm,
                                      const SubDParams *params);
 
 /*
- * note: this is a pattern-based edge subdivider.
+ * NOTE: this is a pattern-based edge subdivider.
  * it tries to match a pattern to edge selections on faces,
  * then executes functions to cut them.
  */
@@ -138,7 +138,7 @@ static BMEdge *connect_smallest_face(BMesh *bm, BMVert *v_a, BMVert *v_b, BMFace
    * multiple faces yet.  that might require a convexity test to figure out which
    * face is "best" and who knows what for non-manifold conditions.
    *
-   * note: we allow adjacent here, since there's no chance this happens.
+   * NOTE: we allow adjacent here, since there's no chance this happens.
    */
   f = BM_vert_pair_share_face_by_len(v_a, v_b, &l_a, &l_b, true);
 
@@ -469,7 +469,7 @@ static void bm_subdivide_multicut(
   alter_co(v2, &e_tmp, params, 1.0, &v1_tmp, &v2_tmp);
 }
 
-/* note: the patterns are rotated as necessary to
+/* NOTE: the patterns are rotated as necessary to
  * match the input geometry.  they're based on the
  * pre-split state of the  face */
 
diff --git a/source/blender/bmesh/operators/bmo_subdivide_edgering.c b/source/blender/bmesh/operators/bmo_subdivide_edgering.c
index d9bfe96ddc4..caca24f4409 100644
--- a/source/blender/bmesh/operators/bmo_subdivide_edgering.c
+++ b/source/blender/bmesh/operators/bmo_subdivide_edgering.c
@@ -83,8 +83,8 @@ static float bezier_handle_calc_length_v3(const float co_a[3],
   float fac = 1.333333f;
   float len;
   if (dot < 0.0f) {
-    /* scale down to 0.666 if we point directly at each other rough but ok */
-    /* TODO, current blend from dot may not be optimal but its also a detail */
+    /* Scale down to 0.666 if we point directly at each other rough but ok. */
+    /* TODO: current blend from dot may not be optimal but its also a detail. */
     const float t = 1.0f + dot;
     fac = (fac * t) + (0.75f * (1.0f - t));
   }
@@ -821,7 +821,7 @@ static void bm_edgering_pair_interpolate(BMesh *bm,
  */
 static void bm_face_slice(BMesh *bm, BMLoop *l, const int cuts)
 {
-  /* TODO, interpolate edge data */
+  /* TODO: interpolate edge data. */
   BMLoop *l_new = l;
   int i;