Fix T56872: Edge slide gives invalid normals
authorCampbell Barton <ideasman42@gmail.com>
Tue, 27 Nov 2018 00:39:51 +0000 (11:39 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 27 Nov 2018 00:40:32 +0000 (11:40 +1100)
Add the ability to skip overwriting normals.

source/blender/bmesh/intern/bmesh_construct.c
source/blender/bmesh/intern/bmesh_construct.h
source/blender/editors/transform/transform.c

index 1b300b248085f2ddc70639a9190cdecfe9675a48..8f124afc55b8fae969e5c2d19191dea23f7a89e8 100644 (file)
@@ -99,7 +99,7 @@ void BM_edges_from_verts_ensure(BMesh *bm, BMEdge **edge_arr, BMVert **vert_arr,
 /* prototypes */
 static void bm_loop_attrs_copy(
         BMesh *source_mesh, BMesh *target_mesh,
-        const BMLoop *source_loop, BMLoop *target_loop);
+        const BMLoop *source_loop, BMLoop *target_loop, uint64_t cd_mask);
 
 /**
  * \brief Make Quad/Triangle
@@ -169,7 +169,7 @@ void BM_face_copy_shared(
                                BLI_assert(l_dst[j]->v == l_src[j]->v);
                                if (BM_ELEM_API_FLAG_TEST(l_dst[j], _FLAG_OVERLAP) == 0) {
                                        if ((filter_fn == NULL) || filter_fn(l_src[j], user_data)) {
-                                               bm_loop_attrs_copy(bm, bm, l_src[j], l_dst[j]);
+                                               bm_loop_attrs_copy(bm, bm, l_src[j], l_dst[j], 0x0);
                                                BM_ELEM_API_FLAG_ENABLE(l_dst[j], _FLAG_OVERLAP);
                                        }
                                }
@@ -486,13 +486,15 @@ void BM_verts_sort_radial_plane(BMVert **vert_arr, int len)
 
 static void bm_vert_attrs_copy(
         BMesh *source_mesh, BMesh *target_mesh,
-        const BMVert *source_vertex, BMVert *target_vertex)
+        const BMVert *source_vertex, BMVert *target_vertex, uint64_t cd_mask)
 {
        if ((source_mesh == target_mesh) && (source_vertex == target_vertex)) {
                BLI_assert(!"BMVert: source and targer match");
                return;
        }
-       copy_v3_v3(target_vertex->no, source_vertex->no);
+       if ((cd_mask & CD_MASK_NORMAL) == 0) {
+               copy_v3_v3(target_vertex->no, source_vertex->no);
+       }
        CustomData_bmesh_free_block_data(&target_mesh->vdata, target_vertex->head.data);
        CustomData_bmesh_copy_data(&source_mesh->vdata, &target_mesh->vdata,
                                   source_vertex->head.data, &target_vertex->head.data);
@@ -500,7 +502,7 @@ static void bm_vert_attrs_copy(
 
 static void bm_edge_attrs_copy(
         BMesh *source_mesh, BMesh *target_mesh,
-        const BMEdge *source_edge, BMEdge *target_edge)
+        const BMEdge *source_edge, BMEdge *target_edge, uint64_t UNUSED(cd_mask))
 {
        if ((source_mesh == target_mesh) && (source_edge == target_edge)) {
                BLI_assert(!"BMEdge: source and targer match");
@@ -513,7 +515,7 @@ static void bm_edge_attrs_copy(
 
 static void bm_loop_attrs_copy(
         BMesh *source_mesh, BMesh *target_mesh,
-        const BMLoop *source_loop, BMLoop *target_loop)
+        const BMLoop *source_loop, BMLoop *target_loop, uint64_t UNUSED(cd_mask))
 {
        if ((source_mesh == target_mesh) && (source_loop == target_loop)) {
                BLI_assert(!"BMLoop: source and targer match");
@@ -526,13 +528,15 @@ static void bm_loop_attrs_copy(
 
 static void bm_face_attrs_copy(
         BMesh *source_mesh, BMesh *target_mesh,
-        const BMFace *source_face, BMFace *target_face)
+        const BMFace *source_face, BMFace *target_face, uint64_t cd_mask)
 {
        if ((source_mesh == target_mesh) && (source_face == target_face)) {
                BLI_assert(!"BMFace: source and targer match");
                return;
        }
-       copy_v3_v3(target_face->no, source_face->no);
+       if ((cd_mask & CD_MASK_NORMAL) == 0) {
+               copy_v3_v3(target_face->no, source_face->no);
+       }
        CustomData_bmesh_free_block_data(&target_mesh->pdata, target_face->head.data);
        CustomData_bmesh_copy_data(&source_mesh->pdata, &target_mesh->pdata,
                                   source_face->head.data, &target_face->head.data);
@@ -548,7 +552,7 @@ static void bm_face_attrs_copy(
  */
 void BM_elem_attrs_copy_ex(
         BMesh *bm_src, BMesh *bm_dst, const void *ele_src_v, void *ele_dst_v,
-        const char hflag_mask)
+        const char hflag_mask, const uint64_t cd_mask)
 {
        const BMHeader *ele_src = ele_src_v;
        BMHeader *ele_dst = ele_dst_v;
@@ -556,6 +560,9 @@ void BM_elem_attrs_copy_ex(
        BLI_assert(ele_src->htype == ele_dst->htype);
        BLI_assert(ele_src != ele_dst);
 
+       /* Only support normal layer at the moment. */
+       BLI_assert((cd_mask & ~CD_MASK_NORMAL) == 0);
+
        if ((hflag_mask & BM_ELEM_SELECT) == 0) {
                /* First we copy select */
                if (BM_elem_flag_test((BMElem *)ele_src, BM_ELEM_SELECT)) {
@@ -577,16 +584,16 @@ void BM_elem_attrs_copy_ex(
        /* Copy specific attributes */
        switch (ele_dst->htype) {
                case BM_VERT:
-                       bm_vert_attrs_copy(bm_src, bm_dst, (const BMVert *)ele_src, (BMVert *)ele_dst);
+                       bm_vert_attrs_copy(bm_src, bm_dst, (const BMVert *)ele_src, (BMVert *)ele_dst, cd_mask);
                        break;
                case BM_EDGE:
-                       bm_edge_attrs_copy(bm_src, bm_dst, (const BMEdge *)ele_src, (BMEdge *)ele_dst);
+                       bm_edge_attrs_copy(bm_src, bm_dst, (const BMEdge *)ele_src, (BMEdge *)ele_dst, cd_mask);
                        break;
                case BM_LOOP:
-                       bm_loop_attrs_copy(bm_src, bm_dst, (const BMLoop *)ele_src, (BMLoop *)ele_dst);
+                       bm_loop_attrs_copy(bm_src, bm_dst, (const BMLoop *)ele_src, (BMLoop *)ele_dst, cd_mask);
                        break;
                case BM_FACE:
-                       bm_face_attrs_copy(bm_src, bm_dst, (const BMFace *)ele_src, (BMFace *)ele_dst);
+                       bm_face_attrs_copy(bm_src, bm_dst, (const BMFace *)ele_src, (BMFace *)ele_dst, cd_mask);
                        break;
                default:
                        BLI_assert(0);
@@ -597,7 +604,7 @@ void BM_elem_attrs_copy_ex(
 void BM_elem_attrs_copy(BMesh *bm_src, BMesh *bm_dst, const void *ele_src, void *ele_dst)
 {
        /* BMESH_TODO, default 'use_flags' to false */
-       BM_elem_attrs_copy_ex(bm_src, bm_dst, ele_src, ele_dst, BM_ELEM_SELECT);
+       BM_elem_attrs_copy_ex(bm_src, bm_dst, ele_src, ele_dst, BM_ELEM_SELECT, 0x0);
 }
 
 void BM_elem_select_copy(BMesh *bm_dst, void *ele_dst_v, const void *ele_src_v)
@@ -644,7 +651,7 @@ static BMFace *bm_mesh_copy_new_face(
        /* use totface in case adding some faces fails */
        BM_elem_index_set(f_new, (bm_new->totface - 1)); /* set_inline */
 
-       BM_elem_attrs_copy_ex(bm_old, bm_new, f, f_new, 0xff);
+       BM_elem_attrs_copy_ex(bm_old, bm_new, f, f_new, 0xff, 0x0);
        f_new->head.hflag = f->head.hflag;  /* low level! don't do this for normal api use */
 
        j = 0;
@@ -701,7 +708,7 @@ BMesh *BM_mesh_copy(BMesh *bm_old)
        BM_ITER_MESH_INDEX (v, &iter, bm_old, BM_VERTS_OF_MESH, i) {
                /* copy between meshes so cant use 'example' argument */
                v_new = BM_vert_create(bm_new, v->co, NULL, BM_CREATE_SKIP_CD);
-               BM_elem_attrs_copy_ex(bm_old, bm_new, v, v_new, 0xff);
+               BM_elem_attrs_copy_ex(bm_old, bm_new, v, v_new, 0xff, 0x0);
                v_new->head.hflag = v->head.hflag;  /* low level! don't do this for normal api use */
                vtable[i] = v_new;
                BM_elem_index_set(v, i); /* set_inline */
@@ -719,7 +726,7 @@ BMesh *BM_mesh_copy(BMesh *bm_old)
                                       vtable[BM_elem_index_get(e->v2)],
                                       e, BM_CREATE_SKIP_CD);
 
-               BM_elem_attrs_copy_ex(bm_old, bm_new, e, e_new, 0xff);
+               BM_elem_attrs_copy_ex(bm_old, bm_new, e, e_new, 0xff, 0x0);
                e_new->head.hflag = e->head.hflag;  /* low level! don't do this for normal api use */
                etable[i] = e_new;
                BM_elem_index_set(e, i); /* set_inline */
index a52a17cd2f357fa697ef5150ee9ca6ba73f14861..1a2b6e46c399d2e7fd03037dbea4e4260aafe390 100644 (file)
@@ -55,7 +55,7 @@ BMFace *BM_face_create_ngon_verts(
 
 void BM_elem_attrs_copy_ex(
         BMesh *bm_src, BMesh *bm_dst, const void *ele_src_v, void *ele_dst_v,
-        const char hflag_mask);
+        const char hflag_mask, const uint64_t cd_mask);
 void BM_elem_attrs_copy(BMesh *bm_src, BMesh *bm_dst, const void *ele_src_v, void *ele_dst_v);
 void BM_elem_select_copy(BMesh *bm_dst, void *ele_dst_v, const void *ele_src_v);
 
index f0e90a05a3b22b9ed637f6def5589e51633ee6bd..3276c4af8fc6834bce13b9337c9ee442c2df19a0 100644 (file)
@@ -5550,7 +5550,7 @@ static void slide_origdata_interp_data_vert(
                BM_loop_interp_from_face(bm, l, f_copy, false, false);
 
                /* make sure face-attributes are correct (e.g. MTexPoly) */
-               BM_elem_attrs_copy(sod->bm_origfaces, bm, f_copy, l->f);
+               BM_elem_attrs_copy_ex(sod->bm_origfaces, bm, f_copy, l->f, 0x0, CD_MASK_NORMAL);
 
                /* weight the loop */
                if (do_loop_weight) {