[Bf-blender-cvs] [dc316f72b37] blender2.8: Multires: Promote OSD CCG to be officially supported

Sergey Sharybin noreply at git.blender.org
Wed Sep 26 14:42:26 CEST 2018


Commit: dc316f72b376d49a921cce0fcc718ec0feaa3feb
Author: Sergey Sharybin
Date:   Wed Sep 26 14:14:12 2018 +0200
Branches: blender2.8
https://developer.blender.org/rBdc316f72b376d49a921cce0fcc718ec0feaa3feb

Multires: Promote OSD CCG to be officially supported

Only active when WITH_OPENSUBDIV_MODIFIER is set to ON.

There is still some work to be done to make that one enabled by
default.

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

M	source/blender/modifiers/intern/MOD_multires.c

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

diff --git a/source/blender/modifiers/intern/MOD_multires.c b/source/blender/modifiers/intern/MOD_multires.c
index c3cfa1d8cd3..8c055536834 100644
--- a/source/blender/modifiers/intern/MOD_multires.c
+++ b/source/blender/modifiers/intern/MOD_multires.c
@@ -228,16 +228,12 @@ static Mesh *applyModifier_subdiv(ModifierData *md,
 		/* Happens on bad topology, ut also on empty input mesh. */
 		return result;
 	}
-	/* TODO(sergey): Some of production machines are using OpenSubdiv already.
-	 * so better not enable semi-finished multires sculpting for now. Will give
-	 * a wrong impression that things do work, even though crucial areas are
-	 * still missing in implementation.
+	/* NOTE: Orco needs final coordinates on CPU side, which are expected to be
+	 * accessible via MVert. For this reason we do not evaluate multires to
+	 * grids when orco is requested.
 	 */
 	const bool for_orco = (ctx->flag & MOD_APPLY_ORCO) != 0;
-	if ((ctx->object->mode & OB_MODE_SCULPT) &&
-	    G.debug_value == 128 &&
-	    !for_orco)
-	{
+	if ((ctx->object->mode & OB_MODE_SCULPT) && !for_orco) {
 		/* NOTE: CCG takes ownership over Subdiv. */
 		result = multires_as_ccg(mmd, ctx, mesh, subdiv);
 		// BKE_subdiv_stats_print(&subdiv->stats);



More information about the Bf-blender-cvs mailing list