[Bf-blender-cvs] [be27159527c] master: Cleanup: rename base_index to ob_index in knife logic

Campbell Barton noreply at git.blender.org
Tue Oct 11 11:30:52 CEST 2022


Commit: be27159527c09d79a662bdef3f663f0d5b8d9a7e
Author: Campbell Barton
Date:   Tue Oct 11 20:28:04 2022 +1100
Branches: master
https://developer.blender.org/rBbe27159527c09d79a662bdef3f663f0d5b8d9a7e

Cleanup: rename base_index to ob_index in knife logic

This indexes the "objects" array, so rename 'b' and 'base_index'
to 'ob_index'.

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

M	source/blender/editors/mesh/editmesh_knife.c

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

diff --git a/source/blender/editors/mesh/editmesh_knife.c b/source/blender/editors/mesh/editmesh_knife.c
index d13103623f9..a147594b25b 100644
--- a/source/blender/editors/mesh/editmesh_knife.c
+++ b/source/blender/editors/mesh/editmesh_knife.c
@@ -100,7 +100,7 @@ typedef struct KnifeColors {
 /* Knife-tool Operator. */
 typedef struct KnifeVert {
   Object *ob;
-  uint base_index;
+  uint ob_index;
   BMVert *v; /* Non-NULL if this is an original vert. */
   ListBase edges;
   ListBase faces;
@@ -142,7 +142,7 @@ typedef struct KnifeLineHit {
   KnifeVert *v;
   BMFace *f;
   Object *ob;
-  uint base_index;
+  uint ob_index;
 } KnifeLineHit;
 
 typedef struct KnifePosData {
@@ -156,7 +156,7 @@ typedef struct KnifePosData {
   KnifeEdge *edge;
   BMFace *bmface;
   Object *ob; /* Object of the vert, edge or bmface. */
-  uint base_index;
+  uint ob_index;
 
   /* When true, the cursor isn't over a face. */
   bool is_space;
@@ -182,7 +182,7 @@ typedef struct KnifeBVH {
   BVHTree *tree;          /* Knife Custom BVH Tree. */
   BMLoop *(*looptris)[3]; /* Used by #knife_bvh_raycast_cb to store the intersecting looptri. */
   float uv[2];            /* Used by #knife_bvh_raycast_cb to store the intersecting uv. */
-  uint base_index;
+  uint ob_index;
 
   /* Use #bm_ray_cast_cb_elem_not_in_face_check. */
   bool (*filter_cb)(BMFace *f, void *userdata);
@@ -1159,11 +1159,11 @@ static void knife_update_header(bContext *C, wmOperator *op, KnifeTool_OpData *k
  * \{ */
 
 static const int *knife_bm_tri_index_get(const KnifeTool_OpData *kcd,
-                                         int base_index,
+                                         int ob_index,
                                          int tri_index,
                                          int tri_index_buf[3])
 {
-  const KnifeObjectInfo *obinfo = &kcd->objects_info[base_index];
+  const KnifeObjectInfo *obinfo = &kcd->objects_info[ob_index];
   if (obinfo->tri_indices) {
     return obinfo->tri_indices[tri_index];
   }
@@ -1174,25 +1174,25 @@ static const int *knife_bm_tri_index_get(const KnifeTool_OpData *kcd,
 }
 
 static void knife_bm_tri_cagecos_get(const KnifeTool_OpData *kcd,
-                                     int base_index,
+                                     int ob_index,
                                      int tri_index,
                                      float cos[3][3])
 {
-  const KnifeObjectInfo *obinfo = &kcd->objects_info[base_index];
+  const KnifeObjectInfo *obinfo = &kcd->objects_info[ob_index];
   int tri_ind_buf[3];
-  const int *tri_ind = knife_bm_tri_index_get(kcd, base_index, tri_index, tri_ind_buf);
+  const int *tri_ind = knife_bm_tri_index_get(kcd, ob_index, tri_index, tri_ind_buf);
   for (int i = 0; i < 3; i++) {
     copy_v3_v3(cos[i], obinfo->cagecos[tri_ind[i]]);
   }
 }
 
 static void knife_bm_tri_cagecos_get_worldspace(const KnifeTool_OpData *kcd,
-                                                int base_index,
+                                                int ob_index,
                                                 int tri_index,
                                                 float cos[3][3])
 {
-  knife_bm_tri_cagecos_get(kcd, base_index, tri_index, cos);
-  const Object *ob = kcd->objects[base_index];
+  knife_bm_tri_cagecos_get(kcd, ob_index, tri_index, cos);
+  const Object *ob = kcd->objects[ob_index];
   for (int i = 0; i < 3; i++) {
     mul_m4_v3(ob->obmat, cos[i]);
   }
@@ -1237,9 +1237,9 @@ static void knife_bvh_init(KnifeTool_OpData *kcd)
   bool test_fn_ret = false;
 
   /* Calculate tottri. */
-  for (uint b = 0; b < kcd->objects_len; b++) {
+  for (uint ob_index = 0; ob_index < kcd->objects_len; ob_index++) {
     ob_tottri = 0;
-    ob = kcd->objects[b];
+    ob = kcd->objects[ob_index];
     em = BKE_editmesh_from_object(ob);
 
     for (int i = 0; i < em->tottri; i++) {
@@ -1269,8 +1269,8 @@ static void knife_bvh_init(KnifeTool_OpData *kcd)
    * Don't forget to update #knife_bvh_intersect_plane!
    */
   tottri = 0;
-  for (uint b = 0; b < kcd->objects_len; b++) {
-    ob = kcd->objects[b];
+  for (uint ob_index = 0; ob_index < kcd->objects_len; ob_index++) {
+    ob = kcd->objects[ob_index];
     em = BKE_editmesh_from_object(ob);
     looptris = em->looptris;
 
@@ -1287,7 +1287,7 @@ static void knife_bvh_init(KnifeTool_OpData *kcd)
       }
 
       float tri_cos[3][3];
-      knife_bm_tri_cagecos_get_worldspace(kcd, b, i, tri_cos);
+      knife_bm_tri_cagecos_get_worldspace(kcd, ob_index, i, tri_cos);
       BLI_bvhtree_insert(kcd->bvh.tree, i + tottri, &tri_cos[0][0], 3);
     }
 
@@ -1325,10 +1325,10 @@ static void knife_bvh_raycast_cb(void *userdata,
   int tottri;
 
   tottri = 0;
-  uint b = 0;
-  for (; b < kcd->objects_len; b++) {
+  uint ob_index = 0;
+  for (; ob_index < kcd->objects_len; ob_index++) {
     index -= tottri;
-    ob = kcd->objects[b];
+    ob = kcd->objects[ob_index];
     em = BKE_editmesh_from_object(ob);
     tottri = em->tottri;
     if (index < tottri) {
@@ -1344,7 +1344,7 @@ static void knife_bvh_raycast_cb(void *userdata,
   }
 
   float tri_cos[3][3];
-  knife_bm_tri_cagecos_get_worldspace(kcd, b, index, tri_cos);
+  knife_bm_tri_cagecos_get_worldspace(kcd, ob_index, index, tri_cos);
   isect = (ray->radius > 0.0f ?
                isect_ray_tri_epsilon_v3(
                    ray->origin, ray->direction, UNPACK3(tri_cos), &dist, uv, ray->radius) :
@@ -1371,7 +1371,7 @@ static void knife_bvh_raycast_cb(void *userdata,
 
     kcd->bvh.looptris = em->looptris;
     copy_v2_v2(kcd->bvh.uv, uv);
-    kcd->bvh.base_index = b;
+    kcd->bvh.ob_index = ob_index;
   }
 }
 
@@ -1383,7 +1383,7 @@ static BMFace *knife_bvh_raycast(KnifeTool_OpData *kcd,
                                  float *r_dist,
                                  float r_hitout[3],
                                  float r_cagehit[3],
-                                 uint *r_base_index)
+                                 uint *r_ob_index)
 {
   BMFace *face;
   BVHTreeRayHit hit;
@@ -1400,7 +1400,7 @@ static BMFace *knife_bvh_raycast(KnifeTool_OpData *kcd,
     /* Hits returned in world space. */
     if (r_hitout) {
       float tri_cos[3][3];
-      knife_bm_tri_cagecos_get_worldspace(kcd, kcd->bvh.base_index, hit.index, tri_cos);
+      knife_bm_tri_cagecos_get_worldspace(kcd, kcd->bvh.ob_index, hit.index, tri_cos);
       interp_v3_v3v3v3_uv(r_hitout, UNPACK3(tri_cos), kcd->bvh.uv);
 
       if (r_cagehit) {
@@ -1412,8 +1412,8 @@ static BMFace *knife_bvh_raycast(KnifeTool_OpData *kcd,
       *r_dist = hit.dist;
     }
 
-    if (r_base_index) {
-      *r_base_index = kcd->bvh.base_index;
+    if (r_ob_index) {
+      *r_ob_index = kcd->bvh.ob_index;
     }
 
     return face;
@@ -1429,7 +1429,7 @@ static BMFace *knife_bvh_raycast_filter(KnifeTool_OpData *kcd,
                                         float *r_dist,
                                         float r_hitout[3],
                                         float r_cagehit[3],
-                                        uint *r_base_index,
+                                        uint *r_ob_index,
                                         bool (*filter_cb)(BMFace *f, void *userdata),
                                         void *filter_userdata)
 {
@@ -1454,7 +1454,7 @@ static BMFace *knife_bvh_raycast_filter(KnifeTool_OpData *kcd,
     /* Hits returned in world space. */
     if (r_hitout) {
       float tri_cos[3][3];
-      knife_bm_tri_cagecos_get_worldspace(kcd, kcd->bvh.base_index, hit.index, tri_cos);
+      knife_bm_tri_cagecos_get_worldspace(kcd, kcd->bvh.ob_index, hit.index, tri_cos);
       interp_v3_v3v3v3_uv(r_hitout, UNPACK3(tri_cos), kcd->bvh.uv);
 
       if (r_cagehit) {
@@ -1466,8 +1466,8 @@ static BMFace *knife_bvh_raycast_filter(KnifeTool_OpData *kcd,
       *r_dist = hit.dist;
     }
 
-    if (r_base_index) {
-      *r_base_index = kcd->bvh.base_index;
+    if (r_ob_index) {
+      *r_ob_index = kcd->bvh.ob_index;
     }
 
     return face;
@@ -1727,7 +1727,7 @@ static KnifeEdge *new_knife_edge(KnifeTool_OpData *kcd)
 }
 
 /* Get a KnifeVert wrapper for an existing BMVert. */
-static KnifeVert *get_bm_knife_vert(KnifeTool_OpData *kcd, BMVert *v, Object *ob, uint base_index)
+static KnifeVert *get_bm_knife_vert(KnifeTool_OpData *kcd, BMVert *v, Object *ob, uint ob_index)
 {
   KnifeVert *kfv = BLI_ghash_lookup(kcd->origvertmap, v);
   const float *cageco;
@@ -1737,7 +1737,7 @@ static KnifeVert *get_bm_knife_vert(KnifeTool_OpData *kcd, BMVert *v, Object *ob
     BMFace *f;
 
     if (BM_elem_index_get(v) >= 0) {
-      cageco = kcd->objects_info[base_index].cagecos[BM_elem_index_get(v)];
+      cageco = kcd->objects_info[ob_index].cagecos[BM_elem_index_get(v)];
     }
     else {
       cageco = v->co;
@@ -1749,7 +1749,7 @@ static KnifeVert *get_bm_knife_vert(KnifeTool_OpData *kcd, BMVert *v, Object *ob
     kfv = new_knife_vert(kcd, v->co, cageco_ws);
     kfv->v = v;
     kfv->ob = ob;
-    kfv->base_index = base_index;
+    kfv->ob_index = ob_index;
 
     BLI_ghash_insert(kcd->origvertmap, v, kfv);
     BM_ITER_ELEM (f, &bmiter, v, BM_FACES_OF_VERT) {
@@ -1761,7 +1761,7 @@ static KnifeVert *get_bm_knife_vert(KnifeTool_OpData *kcd, BMVert *v, Object *ob
 }
 
 /* Get a KnifeEdge wrapper for an existing BMEdge. */
-static KnifeEdge *get_bm_knife_edge(KnifeTool_OpData *kcd, BMEdge *e, Object *ob, uint base_index)
+static KnifeEdge *get_bm_knife_edge(KnifeTool_OpData *kcd, BMEdge *e, Object *ob, uint ob_index)
 {
   KnifeEdge *kfe = BLI_ghash_lookup(kcd->origedgemap, e);
   if (!kfe) {
@@ -1770,8 +1770,8 @@ static KnifeEdge *get_bm_knife_edge(KnifeTool_OpData *kcd, BMEdge *e, Object *ob
 
     kfe = new_knife_edge(kcd);
     kfe->e = e;
-    kfe->v1 = get_bm_knife_vert(kcd, e->v1, ob, base_index);
-    kfe->v2 = get_bm_knife_vert(kcd, e->v2, ob, base_index);
+    kfe->v1 = get_bm_knife_vert(kcd, e->v1, ob, ob_index);
+    kfe->v2 = get_bm_knife_vert(kcd, e->v2, ob, ob_index);
 
     knife_add_to_vert_edges(kcd, kfe);
 
@@ -1785,10 +1785,7 @@ static KnifeEdge *get_bm_knife_edge(KnifeTool_OpData *kcd, BMEdge *e, Object *ob
   return kfe;
 }
 
-static ListBase *knife_get_face_kedges(KnifeTool_OpData *kcd,
-                                       Object 

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list