Cleanup: remove tessface loop from set-smooth function
authorCampbell Barton <ideasman42@gmail.com>
Sun, 25 Aug 2019 02:00:03 +0000 (12:00 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Sun, 25 Aug 2019 02:08:18 +0000 (12:08 +1000)
source/blender/blenkernel/BKE_mesh.h
source/blender/blenkernel/intern/mesh.c
source/blender/editors/object/object_edit.c
source/blender/editors/object/object_remesh.c

index 7986bf947e51ed8e05b876b2712b1b8260d35689..4233c140dad5ba49cc92ce0357c6bb03adac8d9e 100644 (file)
@@ -185,7 +185,7 @@ void BKE_mesh_material_index_remove(struct Mesh *me, short index);
 bool BKE_mesh_material_index_used(struct Mesh *me, short index);
 void BKE_mesh_material_index_clear(struct Mesh *me);
 void BKE_mesh_material_remap(struct Mesh *me, const unsigned int *remap, unsigned int remap_len);
-void BKE_mesh_smooth_flag_set(struct Object *meshOb, int enableSmooth);
+void BKE_mesh_smooth_flag_set(struct Mesh *me, const bool use_smooth);
 
 const char *BKE_mesh_cmp(struct Mesh *me1, struct Mesh *me2, float thresh);
 
index 38e4527fd17991a82f52e16778001e501667d2f4..6102fec963ee5385e40c8428da4d1f99b8eba7fd 100644 (file)
@@ -1273,30 +1273,16 @@ void BKE_mesh_material_remap(Mesh *me, const unsigned int *remap, unsigned int r
 #undef MAT_NR_REMAP
 }
 
-void BKE_mesh_smooth_flag_set(Object *meshOb, int enableSmooth)
+void BKE_mesh_smooth_flag_set(Mesh *me, const bool use_smooth)
 {
-  Mesh *me = meshOb->data;
-  int i;
-
-  for (i = 0; i < me->totpoly; i++) {
-    MPoly *mp = &me->mpoly[i];
-
-    if (enableSmooth) {
-      mp->flag |= ME_SMOOTH;
-    }
-    else {
-      mp->flag &= ~ME_SMOOTH;
+  if (use_smooth) {
+    for (int i = 0; i < me->totpoly; i++) {
+      me->mpoly[i].flag |= ME_SMOOTH;
     }
   }
-
-  for (i = 0; i < me->totface; i++) {
-    MFace *mf = &me->mface[i];
-
-    if (enableSmooth) {
-      mf->flag |= ME_SMOOTH;
-    }
-    else {
-      mf->flag &= ~ME_SMOOTH;
+  else {
+    for (int i = 0; i < me->totpoly; i++) {
+      me->mpoly[i].flag &= ~ME_SMOOTH;
     }
   }
 }
index 16d21a11e2b6784449b249c8a8842ff059545e74..12f7ecb89dd2704cd23d718ed8effd99ffc78dad 100644 (file)
@@ -1200,7 +1200,7 @@ static int shade_smooth_exec(bContext *C, wmOperator *op)
     }
 
     if (ob->type == OB_MESH) {
-      BKE_mesh_smooth_flag_set(ob, !clear);
+      BKE_mesh_smooth_flag_set(ob->data, !clear);
 
       BKE_mesh_batch_cache_dirty_tag(ob->data, BKE_MESH_BATCH_DIRTY_ALL);
       DEG_id_tag_update(&ob->id, ID_RECALC_GEOMETRY);
index bc94b3f7875a66044e4b75f8f4add2143917c68d..5f464084a9b7bef5fb825bad6cc16c2a70c5832d 100644 (file)
@@ -126,7 +126,7 @@ static int voxel_remesh_exec(bContext *C, wmOperator *op)
   }
 
   if (mesh->flag & ME_REMESH_SMOOTH_NORMALS) {
-    BKE_mesh_smooth_flag_set(ob, true);
+    BKE_mesh_smooth_flag_set(ob->data, true);
   }
 
   if (ob->mode == OB_MODE_SCULPT) {