[Bf-blender-cvs] [55daa0d4448] master: Fix T72251: Add rotate brush as constrained by radius for automasking

Pablo Dobarro noreply at git.blender.org
Tue Jan 7 17:08:11 CET 2020


Commit: 55daa0d44483e165d5ce10d161489d136fa0c4e4
Author: Pablo Dobarro
Date:   Sat Dec 7 20:01:39 2019 +0100
Branches: master
https://developer.blender.org/rB55daa0d44483e165d5ce10d161489d136fa0c4e4

Fix T72251: Add rotate brush as constrained by radius for automasking

This brush should be added to the set of brushes where we know which
vertices are going to be affected by the brush when starting the stroke.
This way we can limit the automasking only to those vertices instead of
flood filling the whole mesh from the active vertex.

All brushes that are not in this set will automask by flood filling the
mesh when starting the stroke. To improve this and make it work as most
users expect, we need a fast way to calculate topological distances on
high poly meshes.

Reviewed By: jbakker

Maniphest Tasks: T72251

Differential Revision: https://developer.blender.org/D6376

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

M	source/blender/editors/sculpt_paint/sculpt.c

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

diff --git a/source/blender/editors/sculpt_paint/sculpt.c b/source/blender/editors/sculpt_paint/sculpt.c
index 0ac43f18344..c350cb3b48a 100644
--- a/source/blender/editors/sculpt_paint/sculpt.c
+++ b/source/blender/editors/sculpt_paint/sculpt.c
@@ -1256,7 +1256,7 @@ static bool sculpt_automasking_is_constrained_by_radius(Brush *br)
     return false;
   }
 
-  if (ELEM(br->sculpt_tool, SCULPT_TOOL_GRAB, SCULPT_TOOL_THUMB)) {
+  if (ELEM(br->sculpt_tool, SCULPT_TOOL_GRAB, SCULPT_TOOL_THUMB, SCULPT_TOOL_ROTATE)) {
     return true;
   }
   return false;



More information about the Bf-blender-cvs mailing list