[Bf-blender-cvs] [1e8cc72f85f] master: Cleanup: corrections, clarification to do comments

Campbell Barton noreply at git.blender.org
Fri Jan 20 05:20:58 CET 2023


Commit: 1e8cc72f85f0c759696176011ce13f0e7475d0a4
Author: Campbell Barton
Date:   Fri Jan 20 15:20:02 2023 +1100
Branches: master
https://developer.blender.org/rB1e8cc72f85f0c759696176011ce13f0e7475d0a4

Cleanup: corrections, clarification to do comments

===================================================================

M	source/blender/editors/include/UI_interface.h
M	source/blender/editors/mesh/editmesh_utils.c
M	source/blender/editors/transform/transform_snap_object.cc
M	source/blender/io/collada/MeshImporter.cpp
M	source/blender/modifiers/intern/MOD_solidify_extrude.cc

===================================================================

diff --git a/source/blender/editors/include/UI_interface.h b/source/blender/editors/include/UI_interface.h
index 108f8e46966..db3f0dc41fb 100644
--- a/source/blender/editors/include/UI_interface.h
+++ b/source/blender/editors/include/UI_interface.h
@@ -2036,7 +2036,7 @@ enum {
   UI_TEMPLATE_OP_PROPS_NO_SPLIT_LAYOUT = 1 << 4,
 };
 
-/* used for transp checkers */
+/* Used for transparent checkers shown under color buttons that have an alpha component. */
 #define UI_ALPHA_CHECKER_DARK 100
 #define UI_ALPHA_CHECKER_LIGHT 160
 
diff --git a/source/blender/editors/mesh/editmesh_utils.c b/source/blender/editors/mesh/editmesh_utils.c
index 27a43ab93a0..f54284ef81a 100644
--- a/source/blender/editors/mesh/editmesh_utils.c
+++ b/source/blender/editors/mesh/editmesh_utils.c
@@ -912,6 +912,7 @@ static bool loop_uv_match(BMLoop *loop,
  * \param edge: Search for `needle` in all loops connected to `edge` (recursively).
  * \param luv_anchor: The UV of the anchor (vertex that's being stepped around).
  * \param luv_fan: The UV of the outer edge, this changes as the fan is stepped over.
+ * \param needle: Search for this loop, also defines the vertex at the center of the face-fan.
  * \param visited: A set of edges to prevent recursing down the same edge multiple times.
  * \param cd_loop_uv_offset: The UV layer.
  * \return true if there are edges that fan between them that are seam-free.
diff --git a/source/blender/editors/transform/transform_snap_object.cc b/source/blender/editors/transform/transform_snap_object.cc
index 39e0fa1fc16..923c1a2ff99 100644
--- a/source/blender/editors/transform/transform_snap_object.cc
+++ b/source/blender/editors/transform/transform_snap_object.cc
@@ -63,13 +63,13 @@ enum eViewProj {
   VIEW_PROJ_PERSP = -1,
 };
 
-/* SnapObjectContext.cache.editmesh_map */
+/** #SnapObjectContext.editmesh_caches */
 struct SnapData_EditMesh {
   /* Verts, Edges. */
   BVHTree *bvhtree[2];
   bool cached[2];
 
-  /* Looptris. */
+  /* BVH tree from #BMEditMesh.looptris. */
   BVHTreeFromEditMesh treedata_editmesh;
 
   blender::bke::MeshRuntime *mesh_runtime;
diff --git a/source/blender/io/collada/MeshImporter.cpp b/source/blender/io/collada/MeshImporter.cpp
index 096376cfe19..52eb852e1ed 100644
--- a/source/blender/io/collada/MeshImporter.cpp
+++ b/source/blender/io/collada/MeshImporter.cpp
@@ -717,7 +717,6 @@ void MeshImporter::read_polys(COLLADAFW::Mesh *collada_mesh,
 
         for (uint uvset_index = 0; uvset_index < index_list_array_uvcoord.getCount();
              uvset_index++) {
-          /* get mtface by face index and uv set index */
           COLLADAFW::IndexList &index_list = *index_list_array_uvcoord[uvset_index];
           blender::float2 *mloopuv = static_cast<blender::float2 *>(
               CustomData_get_layer_named_for_write(
diff --git a/source/blender/modifiers/intern/MOD_solidify_extrude.cc b/source/blender/modifiers/intern/MOD_solidify_extrude.cc
index cbca9c3527d..2c51c430e1e 100644
--- a/source/blender/modifiers/intern/MOD_solidify_extrude.cc
+++ b/source/blender/modifiers/intern/MOD_solidify_extrude.cc
@@ -392,7 +392,7 @@ Mesh *MOD_solidify_extrude_modifyMesh(ModifierData *md, const ModifierEvalContex
         &result->edata, CD_BWEIGHT, CD_SET_DEFAULT, nullptr, result->totedge));
   }
 
-  /* initializes: (i_end, do_shell_align, mv). */
+  /* Initializes: (`i_end`, `do_shell_align`, `vert_index`). */
 #define INIT_VERT_ARRAY_OFFSETS(test) \
   if (((ofs_new >= ofs_orig) == do_flip) == test) { \
     i_end = verts_num; \



More information about the Bf-blender-cvs mailing list