[Bf-blender-cvs] [b4b224dc083] blender2.8: Shrinkwrap: use polygon normals for flat faces in Align To Normal.

[Alexander Gavrilov]

Commit: b4b224dc083c0e490eebc5c98646d791a41b8563
Author: Alexander Gavrilov
Date:   Wed Dec 5 20:24:05 2018 +0300
Branches: blender2.8
https://developer.blender.org/rBb4b224dc083c0e490eebc5c98646d791a41b8563

Shrinkwrap: use polygon normals for flat faces in Align To Normal.

Hit normal originates from tesselated triangles and isn't the
actual normal used for shading of flat faces. Thus, it is better
to use the actual polygon normals when available.

===================================================================

M	source/blender/blenkernel/BKE_shrinkwrap.h
M	source/blender/blenkernel/intern/shrinkwrap.c

===================================================================

diff --git a/source/blender/blenkernel/BKE_shrinkwrap.h b/source/blender/blenkernel/BKE_shrinkwrap.h
index d703e8819d7..0959e8a72f0 100644
--- a/source/blender/blenkernel/BKE_shrinkwrap.h
+++ b/source/blender/blenkernel/BKE_shrinkwrap.h
@@ -90,6 +90,7 @@ typedef struct ShrinkwrapTreeData {
 	BVHTree *bvh;
 	BVHTreeFromMesh treeData;
 
+	float (*pnors)[3];
 	float (*clnors)[3];
 	ShrinkwrapBoundaryData *boundary;
 } ShrinkwrapTreeData;
diff --git a/source/blender/blenkernel/intern/shrinkwrap.c b/source/blender/blenkernel/intern/shrinkwrap.c
index ee35876f2a3..06e5cc9e849 100644
--- a/source/blender/blenkernel/intern/shrinkwrap.c
+++ b/source/blender/blenkernel/intern/shrinkwrap.c
@@ -144,6 +144,7 @@ bool BKE_shrinkwrap_init_tree(ShrinkwrapTreeData *data, Mesh *mesh, int shrinkTy
 		}
 
 		if (force_normals || BKE_shrinkwrap_needs_normals(shrinkType, shrinkMode)) {
+			data->pnors = CustomData_get_layer(&mesh->pdata, CD_NORMAL);
 			if ((mesh->flag & ME_AUTOSMOOTH) != 0) {
 				data->clnors = CustomData_get_layer(&mesh->ldata, CD_NORMAL);
 			}
@@ -1164,7 +1165,11 @@ void BKE_shrinkwrap_compute_smooth_normal(
 			normalize_v3(r_no);
 		}
 	}
-	/* Use the looptri normal if flat. */
+	/* Use the polygon normal if flat. */
+	else if (tree->pnors != NULL) {
+		copy_v3_v3(r_no, tree->pnors[tri->poly]);
+	}
+	/* Finally fallback to the looptri normal. */
 	else {
 		copy_v3_v3(r_no, hit_no);
 	}