[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [50870] trunk/blender/source/blender/ modifiers/intern/MOD_solidify.c: fix for regression in solidify modifier copying loop customdata to rim faces (UV's and vertex colors).

Campbell Barton ideasman42 at gmail.com
Tue Sep 25 03:21:27 CEST 2012


Revision: 50870
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=50870
Author:   campbellbarton
Date:     2012-09-25 01:21:21 +0000 (Tue, 25 Sep 2012)
Log Message:
-----------
fix for regression in solidify modifier copying loop customdata to rim faces (UV's and vertex colors).

Own fault when fixing rim face flipping r46924

Revision Links:
--------------
    http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=46924

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

Modified: trunk/blender/source/blender/modifiers/intern/MOD_solidify.c
===================================================================
--- trunk/blender/source/blender/modifiers/intern/MOD_solidify.c	2012-09-25 00:25:17 UTC (rev 50869)
+++ trunk/blender/source/blender/modifiers/intern/MOD_solidify.c	2012-09-25 01:21:21 UTC (rev 50870)
@@ -635,15 +635,15 @@
 			/* notice we use 'mp->totloop' which is later overwritten,
 			 * we could lookup the original face but theres no point since this is a copy
 			 * and will have the same value, just take care when changing order of assignment */
-			k1 = mpoly[fidx].loopstart + ((edge_order[eidx] + 1) % mp->totloop);
+			k1 = mpoly[fidx].loopstart + (((edge_order[eidx] - 1) + mp->totloop) % mp->totloop);  /* prev loop */
 			k2 = mpoly[fidx].loopstart +  (edge_order[eidx]);
 
 			mp->totloop = 4;
 
-			CustomData_copy_data(&dm->loopData, &result->loopData, k1, numLoops * 2 + j + 0, 1);
-			CustomData_copy_data(&dm->loopData, &result->loopData, k2, numLoops * 2 + j + 1, 1);
-			CustomData_copy_data(&dm->loopData, &result->loopData, k2, numLoops * 2 + j + 2, 1);
-			CustomData_copy_data(&dm->loopData, &result->loopData, k1, numLoops * 2 + j + 3, 1);
+			CustomData_copy_data(&dm->loopData, &result->loopData, k2, numLoops * 2 + j + 0, 1);
+			CustomData_copy_data(&dm->loopData, &result->loopData, k1, numLoops * 2 + j + 1, 1);
+			CustomData_copy_data(&dm->loopData, &result->loopData, k1, numLoops * 2 + j + 2, 1);
+			CustomData_copy_data(&dm->loopData, &result->loopData, k2, numLoops * 2 + j + 3, 1);
 
 			if (flip == FALSE) {
 				ml[j].v = ed->v1;




More information about the Bf-blender-cvs mailing list