diff options
author | Campbell Barton <ideasman42@gmail.com> | 2017-10-01 13:12:14 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2017-10-01 13:21:24 +0300 |
commit | 22ecea9e38e708056262f3d98b0d0c6210da75cf (patch) | |
tree | 1551b8cc4a35863bd9098a2bb12cdb161c8196f7 /source/blender | |
parent | 9139532dc8655f23b59da3ee399cc01d1fa2b7b2 (diff) |
Weight Paint: restore non-spray functionality
When painting with spray disabled - we need to re-apply
on top of the original each time.
Applying the soc-2016-pbvh-painting branch removed this.
While I'd added back a simple previous weight array,
this won't work when multiple groups are painted at once.
Diffstat (limited to 'source/blender')
-rw-r--r-- | source/blender/blenkernel/BKE_paint.h | 5 | ||||
-rw-r--r-- | source/blender/blenkernel/intern/paint.c | 7 | ||||
-rw-r--r-- | source/blender/editors/sculpt_paint/paint_vertex.c | 85 |
3 files changed, 67 insertions, 30 deletions
diff --git a/source/blender/blenkernel/BKE_paint.h b/source/blender/blenkernel/BKE_paint.h index 6cae8d28947..fe7461c810d 100644 --- a/source/blender/blenkernel/BKE_paint.h +++ b/source/blender/blenkernel/BKE_paint.h @@ -228,9 +228,12 @@ typedef struct SculptSession { /* Vertex aligned arrays of weights. */ float *previous_accum; - float *previous_weight; /* Keep track of how much each vertex has been painted (non-airbrush only). */ float *alpha_weight; + + /* Needed to continuously re-apply over the same weights (VP_SPRAY disabled). + * Lazy initialize as needed (flag is set to 1 to tag it as uninitialized). */ + struct MDeformVert *dvert_prev; } wpaint; //struct { diff --git a/source/blender/blenkernel/intern/paint.c b/source/blender/blenkernel/intern/paint.c index 849fb5c6b5a..c2f1d05dd89 100644 --- a/source/blender/blenkernel/intern/paint.c +++ b/source/blender/blenkernel/intern/paint.c @@ -49,6 +49,7 @@ #include "BKE_brush.h" #include "BKE_colortools.h" +#include "BKE_deform.h" #include "BKE_main.h" #include "BKE_context.h" #include "BKE_crazyspace.h" @@ -686,8 +687,12 @@ void BKE_sculptsession_free_vwpaint_data(struct SculptSession *ss) gmap = &ss->mode.wpaint.gmap; MEM_SAFE_FREE(ss->mode.wpaint.alpha_weight); - MEM_SAFE_FREE(ss->mode.wpaint.previous_weight); MEM_SAFE_FREE(ss->mode.wpaint.previous_accum); + if (ss->mode.wpaint.dvert_prev) { + BKE_defvert_array_free_elems(ss->mode.wpaint.dvert_prev, ss->totvert); + MEM_freeN(ss->mode.wpaint.dvert_prev); + ss->mode.wpaint.dvert_prev = NULL; + } } else { return; diff --git a/source/blender/editors/sculpt_paint/paint_vertex.c b/source/blender/editors/sculpt_paint/paint_vertex.c index 13923b50430..47b84dda1af 100644 --- a/source/blender/editors/sculpt_paint/paint_vertex.c +++ b/source/blender/editors/sculpt_paint/paint_vertex.c @@ -88,10 +88,16 @@ struct WPaintAverageAccum { double value; }; -static void defweight_prev_init(const MDeformWeight *dw, float *weight_prev) +static void defweight_prev_restore_or_init(MDeformVert *dvert_prev, MDeformVert *dvert_curr, int index) { - if (UNLIKELY(*weight_prev == -1.0f)) { - *weight_prev = dw ? dw->weight : 0.0f; + MDeformVert *dv_curr = &dvert_curr[index]; + MDeformVert *dv_prev = &dvert_prev[index]; + if (dv_prev->flag == 1) { + dv_prev->flag = 0; + defvert_copy(dv_prev, dv_curr); + } + else { + defvert_copy(dv_curr, dv_prev); } } @@ -645,18 +651,6 @@ static void do_weight_paint_vertex_single( MDeformVert *dv_mirr; MDeformWeight *dw_mirr; - if (wp->flag & VP_ONLYVGROUP) { - dw = defvert_find_index(dv, wpi->active.index); - } - else { - dw = defvert_verify_index(dv, wpi->active.index); - } - - if (dw == NULL) { - return; - } - - /* from now on we can check if mirrors enabled if this var is -1 and not bother with the flag */ if (me->editflag & ME_EDIT_MIRROR_X) { index_mirr = mesh_get_x_mirror_vert(ob, NULL, index, topology); @@ -672,6 +666,24 @@ static void do_weight_paint_vertex_single( index_mirr = vgroup_mirr = -1; } + if ((wp->flag & VP_SPRAY) == 0) { + struct MDeformVert *dvert_prev = ob->sculpt->mode.wpaint.dvert_prev; + defweight_prev_restore_or_init(dvert_prev, me->dvert, index); + if (index_mirr != -1) { + defweight_prev_restore_or_init(dvert_prev, me->dvert, index_mirr); + } + } + + if (wp->flag & VP_ONLYVGROUP) { + dw = defvert_find_index(dv, wpi->active.index); + } + else { + dw = defvert_verify_index(dv, wpi->active.index); + } + + if (dw == NULL) { + return; + } /* get the mirror def vars */ if (index_mirr != -1) { @@ -792,6 +804,17 @@ static void do_weight_paint_vertex_multi( if (index_mirr != -1 && index_mirr != index) { dv_mirr = &me->dvert[index_mirr]; } + else { + index_mirr = -1; + } + } + + if ((wp->flag & VP_SPRAY) == 0) { + struct MDeformVert *dvert_prev = ob->sculpt->mode.wpaint.dvert_prev; + defweight_prev_restore_or_init(dvert_prev, me->dvert, index); + if (index_mirr != -1) { + defweight_prev_restore_or_init(dvert_prev, me->dvert, index_mirr); + } } /* compute weight change by applying the brush to average or sum of group weights */ @@ -945,13 +968,23 @@ static void vertex_paint_init_session_data(const ToolSettings *ts, Object *ob) ob->sculpt->mode.wpaint.alpha_weight = MEM_callocN(me->totvert * sizeof(float), __func__); } - if (ob->sculpt->mode.wpaint.previous_weight == NULL) { - ob->sculpt->mode.wpaint.previous_weight = - MEM_mallocN(me->totvert * sizeof(float), __func__); + if (ob->sculpt->mode.wpaint.dvert_prev == NULL) { + ob->sculpt->mode.wpaint.dvert_prev = + MEM_callocN(me->totvert * sizeof(MDeformVert), __func__); + MDeformVert *dv = ob->sculpt->mode.wpaint.dvert_prev; + for (int i = 0; i < me->totvert; i++, dv++) { + /* Use to show this isn't initialized, never apply to the mesh data. */ + dv->flag = 1; + } } } else { MEM_SAFE_FREE(ob->sculpt->mode.wpaint.alpha_weight); + if (ob->sculpt->mode.wpaint.dvert_prev != NULL) { + BKE_defvert_array_free_elems(ob->sculpt->mode.wpaint.dvert_prev, me->totvert); + MEM_freeN(ob->sculpt->mode.wpaint.dvert_prev); + ob->sculpt->mode.wpaint.dvert_prev = NULL; + } } if (brush && brush->flag & BRUSH_ACCUMULATE) { if (ob->sculpt->mode.wpaint.previous_accum == NULL) { @@ -1335,8 +1368,12 @@ static bool wpaint_stroke_test_start(bContext *C, wmOperator *op, const float mo vwpaint_update_cache_invariants(C, vd, ss, op, mouse); vertex_paint_init_session_data(ts, ob); - if (ss->mode.wpaint.previous_weight != NULL) { - copy_vn_fl(ss->mode.wpaint.previous_weight, me->totvert, -1.0f); + if (ob->sculpt->mode.wpaint.dvert_prev != NULL) { + MDeformVert *dv = ob->sculpt->mode.wpaint.dvert_prev; + for (int i = 0; i < me->totvert; i++, dv++) { + /* Use to show this isn't initialized, never apply to the mesh data. */ + dv->flag = 1; + } } return true; @@ -1605,14 +1642,6 @@ static void do_wpaint_brush_draw_task_cb_ex( else { continue; } - - MDeformVert *dv = &data->me->dvert[v_index]; - MDeformWeight *dw = defvert_find_index(dv, data->wpi->active.index); - float *weight_prev = &ss->mode.wpaint.previous_weight[v_index]; - defweight_prev_init(dw, weight_prev); - if (dw) { - dw->weight = *weight_prev; - } } do_weight_paint_vertex( |