Fix T57366: Mesh.from_pydata invalid loose-edge state
authorCampbell Barton <ideasman42@gmail.com>
Wed, 24 Oct 2018 00:37:44 +0000 (11:37 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 24 Oct 2018 00:37:44 +0000 (11:37 +1100)
release/scripts/modules/bpy_types.py
source/blender/blenkernel/BKE_mesh.h
source/blender/blenkernel/intern/mesh_validate.c
source/blender/editors/include/ED_mesh.h
source/blender/editors/mesh/mesh_data.c
source/blender/makesrna/intern/rna_mesh_api.c

index 9fe45d223f54ff8437a060e60cb983923d4fb1ae..84262e618d860c8bdc7164f4c98527df0a8379be 100644 (file)
@@ -454,6 +454,8 @@ class Mesh(bpy_types.ID):
         # if no edges - calculate them
         if faces and (not edges):
             self.update(calc_edges=True)
+        elif edges:
+            self.update(calc_edges_loose=True)
 
     @property
     def edge_keys(self):
index 9cc2bd8c15276f9a715c96070b2b11fdcd83496a..56241a192216431fabad7b57fafe0ee722bcc825 100644 (file)
@@ -497,6 +497,7 @@ void BKE_mesh_strip_loose_polysloops(struct Mesh *me);
 void BKE_mesh_strip_loose_edges(struct Mesh *me);
 
 void BKE_mesh_calc_edges_legacy(struct Mesh *me, const bool use_old);
+void BKE_mesh_calc_edges_loose(struct Mesh *mesh);
 void BKE_mesh_calc_edges(struct Mesh *mesh, bool update, const bool select);
 void BKE_mesh_calc_edges_tessface(struct Mesh *mesh);
 
index dd9d6fa448a208840a98503d85ecde9bdace3100..af4d1265cfd62972bb555757af94183ed3323e05 100644 (file)
@@ -1445,6 +1445,18 @@ void BKE_mesh_calc_edges(Mesh *mesh, bool update, const bool select)
        BLI_edgehash_free(eh, NULL);
 }
 
+void BKE_mesh_calc_edges_loose(Mesh *mesh)
+{
+       MEdge *med = mesh->medge;
+       for (int i = 0; i < mesh->totedge; i++, med++) {
+               med->flag |= ME_LOOSEEDGE;
+       }
+       MLoop *ml = mesh->mloop;
+       for (int i = 0; i < mesh->totloop; i++, ml++) {
+               mesh->medge[ml->e].flag &= ~ME_LOOSEEDGE;
+       }
+}
+
 /**
  * Calculate/create edges from tessface data
  *
index da6fd1a6b53718031557d339fed0ef178e327a3b..a6db1870106b5c3f593c20783a9fb0a417925280 100644 (file)
@@ -319,7 +319,7 @@ void ED_mesh_edges_remove(struct Mesh *mesh, struct ReportList *reports, int cou
 void ED_mesh_vertices_remove(struct Mesh *mesh, struct ReportList *reports, int count);
 
 void ED_mesh_calc_tessface(struct Mesh *mesh, bool free_mpoly);
-void ED_mesh_update(struct Mesh *mesh, struct bContext *C, bool calc_edges, bool calc_tessface);
+void ED_mesh_update(struct Mesh *mesh, struct bContext *C, bool calc_edges, bool calc_edges_loose, bool calc_tessface);
 
 void ED_mesh_uv_texture_ensure(struct Mesh *me, const char *name);
 int  ED_mesh_uv_texture_add(struct Mesh *me, const char *name, const bool active_set);
index 438cbb8da91a172d8c74c3e99f9b92c64ab2965f..2a918560ba89eda7acbd1758fc43ea8a8da2c571 100644 (file)
@@ -877,7 +877,7 @@ void MESH_OT_customdata_custom_splitnormals_clear(wmOperatorType *ot)
 
 /************************** Add Geometry Layers *************************/
 
-void ED_mesh_update(Mesh *mesh, bContext *C, bool calc_edges, bool calc_tessface)
+void ED_mesh_update(Mesh *mesh, bContext *C, bool calc_edges, bool calc_edges_loose, bool calc_tessface)
 {
        bool tessface_input = false;
 
@@ -888,6 +888,10 @@ void ED_mesh_update(Mesh *mesh, bContext *C, bool calc_edges, bool calc_tessface
                tessface_input = true;
        }
 
+       if (calc_edges_loose && mesh->totedge) {
+               BKE_mesh_calc_edges_loose(mesh);
+       }
+
        if (calc_edges || ((mesh->totpoly || mesh->totface) && mesh->totedge == 0))
                BKE_mesh_calc_edges(mesh, calc_edges, true);
 
index b1556edd4b7cd589ab05d396bc44050bde319ce8..a08555794d8f7157d2c380bd751327e9807f5a50 100644 (file)
@@ -306,6 +306,7 @@ void RNA_api_mesh(StructRNA *srna)
 
        func = RNA_def_function(srna, "update", "ED_mesh_update");
        RNA_def_boolean(func, "calc_edges", 0, "Calculate Edges", "Force recalculation of edges");
+       RNA_def_boolean(func, "calc_edges_loose", 0, "Calculate Loose Edges", "Calculate the loose state of each edge");
        RNA_def_boolean(func, "calc_loop_triangles", 0, "Calculate Triangules", "Force recalculation of triangle tessellation");
        RNA_def_function_flag(func, FUNC_USE_CONTEXT);