[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [42683] trunk/blender/source/blender/ editors/sculpt_paint/sculpt.c: Calculation of the center and normal for sculpting would ignore whether the sculpting tool is using the current vertex positions /normals or the positions/normals from before the current edit.

[Jason Wilkins]

Revision: 42683
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=42683
Author:   jwilkins
Date:     2011-12-17 03:49:39 +0000 (Sat, 17 Dec 2011)
Log Message:
-----------
Calculation of the center and normal for sculpting would ignore whether the sculpting tool is using the current vertex positions/normals or the positions/normals from before the current edit.  Most of the time this will lead to subtle and hard to predict differences from what a user might expect.  There was sum testing and discussion about at the end of last summer and even some screen shots showing the difference.  I'm going to go ahead and apply this since I do not think it is controversial.

Modified Paths:
--------------
    trunk/blender/source/blender/editors/sculpt_paint/sculpt.c

Modified: trunk/blender/source/blender/editors/sculpt_paint/sculpt.c
===================================================================
--- trunk/blender/source/blender/editors/sculpt_paint/sculpt.c	2011-12-17 03:46:38 UTC (rev 42682)
+++ trunk/blender/source/blender/editors/sculpt_paint/sculpt.c	2011-12-17 03:49:39 UTC (rev 42683)
@@ -1688,7 +1688,7 @@
 		if(ss->cache->original) {
 			BLI_pbvh_vertex_iter_begin(ss->pbvh, nodes[n], vd, PBVH_ITER_UNIQUE) {
 				if(sculpt_brush_test_fast(&test, unode->co[vd.i])) {
-					add_v3_v3(private_fc, vd.co);
+					add_v3_v3(private_fc, unode->co[vd.i]);
 					private_count++;
 				}
 			}
@@ -1758,7 +1758,7 @@
 					add_norm_if(ss->cache->view_normal, private_an, private_out_flip, fno);
 
 					// fc
-					add_v3_v3(private_fc, vd.co);
+					add_v3_v3(private_fc, unode->co[vd.i]);
 					private_count++;
 				}
 			}