griefith/test2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Cleanup: Remove unnecessary conditional when restoring the mesh

Browse Source 
The `SCULPT_BRUSH_TYPE_GRAB` and `SCULPT_BRUSH_TYPE_ELASTIC_DEFORM`
types are already handled above the removed lines, generically, without
checking for size pressure. This check of the if statement was
effectively always false as it would never be encountered.

Pull Request: https://projects.blender.org/blender/blender/pulls/146826
This commit is contained in:
Sean Kim
2025-09-26 12:24:30 +02:00
committed by Sean Kim
parent 2b0a1cae06
commit 6375b10039
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
source/blender/editors/sculpt_paint/sculpt.cc
View File

@@ -5030,11 +5030,7 @@ static void restore_from_undo_step_if_necessary(const Depsgraph &depsgraph,
}
/* Restore the mesh before continuing with anchored stroke. */
if ((brush->flag & BRUSH_ANCHORED) ||
(ELEM(brush->sculpt_brush_type, SCULPT_BRUSH_TYPE_GRAB, SCULPT_BRUSH_TYPE_ELASTIC_DEFORM) &&
BKE_brush_use_size_pressure(brush)) ||
(brush->flag & BRUSH_DRAG_DOT))
{
if (brush->flag & BRUSH_ANCHORED || brush->flag & BRUSH_DRAG_DOT) {
undo::restore_from_undo_step(depsgraph, sd, ob);