[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [52399] trunk/blender/source/blender/ modifiers/intern/MOD_mirror.c: Fix #33217, mirror modifier edges missing in wireframe mode.

Antony Riakiotakis kalast at gmail.com
Tue Nov 20 11:00:01 CET 2012


Revision: 52399
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=52399
Author:   psy-fi
Date:     2012-11-20 09:59:57 +0000 (Tue, 20 Nov 2012)
Log Message:
-----------
Fix #33217, mirror modifier edges missing in wireframe mode.

I see we explicitly set the edge visibility in the code for a few
modifiers because of this flag missing. The only place that this flag is
not set is during subsurf so maybe it would make a lot more sense to set
it by default on newly created edges (currently off by default) through
bmesh and turn it off for any modifiers that may need it to be off.

Modified Paths:
--------------
    trunk/blender/source/blender/modifiers/intern/MOD_mirror.c

Modified: trunk/blender/source/blender/modifiers/intern/MOD_mirror.c
===================================================================
--- trunk/blender/source/blender/modifiers/intern/MOD_mirror.c	2012-11-20 09:04:55 UTC (rev 52398)
+++ trunk/blender/source/blender/modifiers/intern/MOD_mirror.c	2012-11-20 09:59:57 UTC (rev 52399)
@@ -107,7 +107,7 @@
 	const int maxLoops = dm->getNumLoops(dm);
 	const int maxPolys = dm->getNumPolys(dm);
 	MVert *mv, *mv_prev;
-	MEdge *me;
+	MEdge *me, *orig_me;
 	MLoop *ml;
 	MPoly *mp;
 	float mtx[4][4];
@@ -209,11 +209,14 @@
 		}
 	}
 	
-	/* adjust mirrored edge vertex indices */
-	me = CDDM_get_edges(result) + maxEdges;
+	/* adjust mirrored edge vertex indices, also set visibility to true */
+	orig_me = CDDM_get_edges(result);
+	me = orig_me + maxEdges;
 	for (i = 0; i < maxEdges; i++, me++) {
 		me->v1 += maxVerts;
 		me->v2 += maxVerts;
+		me->flag |= ME_EDGEDRAW | ME_EDGERENDER;
+		orig_me->flag |= ME_EDGEDRAW | ME_EDGERENDER;
 	}
 	
 	/* adjust mirrored poly loopstart indices, and reverse loop order (normals) */




More information about the Bf-blender-cvs mailing list